T09 · Insecure Skill Coding Practices
- Location
scripts/metrics_card.py:27- Finding
Unrestricted Output Path Allows Arbitrary File Overwrite
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
This stock-card skill is purpose-aligned overall, but needs review because its script can overwrite arbitrary local files if given a crafted output path or stock symbol.
Install only if you are comfortable with a skill that writes local files. Use a normal six-digit stock symbol and keep output paths under notes/stocks/cards; review or patch the script to validate symbols, reject path traversal, and avoid overwriting existing files without confirmation.
scripts/metrics_card.py:27Unrestricted Output Path Allows Arbitrary File Overwrite
The skill explicitly plans to use network access to fetch stock data and to write a Markdown file, but it does not declare any tool restrictions or permissions. This creates an overbroad execution profile where an agent runtime may grant more capabilities than are necessary or fail to surface the risk to users, increasing the chance of unintended file writes or unreviewed external data access.
The skill says it will write a generated Markdown card to outputPath by default, but it does not clearly warn the user that invoking the skill modifies local files. Even though the default path appears intended for notes, silent file creation or overwrite can surprise users and becomes riskier if outputPath is user-controlled or if existing files are replaced.
No suspicious patterns detected.