Back to skill

Security audit

Hitchhikers Guide

Security checks for vulnerabilities and agentic risk

Overview

This is a coherent local text-adventure skill that saves game progress and lore files in its own directory, with no evidence of credential access, network behavior, or hidden execution.

Install only if you are comfortable with the skill keeping local game progress, action history, and generated guide entries in its own assets directory. Use the reset command carefully because it clears the current save.

Vulnerability Patterns
  • Skill Instruction HijackingAlters the agent's session goals or safety constraints when the skill loads
  • Agent Memory PoisoningWrites attacker-controlled rules into memory that affect later sessions
  • Remote Payload Retrieval and ExecutionFetches external code whose behavior can change after review
  • Embedded Malicious CodeShips malicious scripts inside the skill and executes them locally
  • Unauthorized Access and Privilege EscalationObtains permissions beyond the task's legitimate needs
Vulnerability Patterns
  • Memory PoisoningPersistent Context Injection, Context Window Stuffing, Memory Manipulation
  • MCP Least PrivilegeUnderdeclared Capability, Wildcard Permission, Missing Permission Declaration
  • Prompt InjectionInstruction Override, Hidden Instructions, Exfiltration Commands
  • Data ExfiltrationExternal Transmission, Env Variable Harvesting, File System Enumeration
  • Privilege EscalationExcessive Permissions, Sudo/Root Execution, Credential Access
Findings (7)

Memory Manipulation

High
Category
Memory Poisoning
Confidence
80% confidence
Finding

Skill manipulates agent memory, state, or stored context. Memory corruption can alter personality, override safety rules, or cause unpredictable behavior.

Content

Scanner excerpt · README.md (reported line 38)May include surrounding context.

md
### Key Commands

- **Load State**: `python scripts/game_manager.py load`
- **Reset State**: `python scripts/game_manager.py reset`
- **Update Location**: `python scripts/game_manager.py set_location "Vogon Hold"`
- **Add/Remove Items**:
  - `python scripts/game_manager.py add_item "Towel"`

Missing User Warnings

Medium
Category
Not specified by scanner
Confidence
95% confidence
Finding

The documented reset command is explicitly destructive and the README provides no warning, confirmation step, or backup guidance. In an agent-driven workflow, a model could invoke reset as part of normal operation and irreversibly erase the current game state, causing unintended data loss even if the impact is limited to local save data.

Content

No source excerpt is available for this finding.

Undeclared Tool Scope

Medium
Category
MCP Least Privilege
Confidence
94% confidence
Finding

The skill instructs the agent to read and write local files and invoke scripts, but it does not declare any explicit tool scope or permissions boundaries. This creates an authorization and transparency gap: an agent may perform filesystem operations the user did not clearly consent to, including modifying persistent state files.

Content

No source excerpt is available for this finding.

Missing User Warnings

Medium
Category
Not specified by scanner
Confidence
95% confidence
Finding

The workflow directs the agent to always continue and save game state to a local file, but it does not disclose the persistence implications or obtain consent. This can retain user interaction history, inventory, and other gameplay-derived data across sessions in ways the user may not expect.

Content

No source excerpt is available for this finding.

Missing User Warnings

Medium
Category
Not specified by scanner
Confidence
96% confidence
Finding

The skill says new guide entries should be saved to assets/GUIDE.md automatically, which causes local file writes without notifying the user at the time of action. Automatic persistence can unexpectedly store user-provided or model-generated content, creating privacy, integrity, and auditability concerns.

Content

No source excerpt is available for this finding.

Missing User Warnings

Low
Category
Not specified by scanner
Confidence
93% confidence
Finding

This markdown file documents that game state is saved persistently and later lists a reset command, but it does not warn users that save data can be modified or overwritten. For markdown files, omitting warnings about behaviors that affect user data qualifies as a missing user warning.

Content

No source excerpt is available for this finding.

Missing User Warnings

Low
Category
Not specified by scanner
Confidence
90% confidence
Finding

The reset_game function overwrites the existing save file with the initial state, which is a destructive operation affecting user data. The code performs the write directly and does not include any explicit warning or confirmation step near the operation itself.

Content

No source excerpt is available for this finding.

Static analysis

No suspicious patterns detected.