T09 · Insecure Skill Coding Practices
- Location
SKILL.md:122- Finding
Command Injection Through Unsanitized File Arguments
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
The Beamer skill is coherent and not malicious, but it needs review because it combines broad activation, shell compilation, file mutation, and a few under-scoped instructions.
Install only if you are comfortable with an agent editing Beamer projects and running LaTeX/PDF commands. Prefer explicit Beamer requests, confirm presenter/institute metadata, use safe filenames, avoid raw shell metacharacters in paths, and review any extracted files before they overwrite existing figures.
SKILL.md:122Command Injection Through Unsanitized File Arguments
SKILL.md:28Unpinned Python Dependency Installation
The skill instructs the agent to default to a specific presenter format and institutional affiliation unless the user overrides it. This can cause the agent to insert false or unauthorized attribution into academic slides, creating reputational, misrepresentation, or policy/compliance issues when users do not explicitly request that affiliation.
The trigger phrases are broad enough to match many generic presentation requests, which can cause the skill to activate outside its intended Beamer-only scope. Because this skill enables powerful tools including Bash, Write, Edit, Agent, and task orchestration, accidental invocation can lead to unnecessary file modification, command execution, or workflow hijacking in unrelated contexts. The risk is elevated by the large trigger surface and the skill's strong automation behavior.
The manifest declares allowed tools only as Read, Write, Edit, Bash, Grep, Glob, Agent, AskUserQuestion, and task tools, but the extract-figures workflow later instructs use of mcp__pdf-mcp__pdf_get_toc, mcp__pdf-mcp__pdf_read_pages, and mcp__pdf-mcp__pdf_extract_images. That is a concrete capability beyond the declared permission/tool surface.
The invocation pattern 'review [file] or proofread [file]' is described without any limiting context, negative examples, or scope constraints. In a markdown rules file, this can be considered an ambiguous trigger because terms like 'review' and 'audit' are common requests and the document does not clarify when these action names should or should not activate the skill.
The file introduces action names like 'audit [file]' as standalone commands, but does not document boundaries for activation beyond the placeholder syntax. Because this is a markdown file and the trigger guidance is sparse, users or systems may interpret everyday uses of 'audit', 'validate', or 'review' as skill invocations.
The extract-figures workflow explicitly instructs saving decoded images into the project's figures/ directory, which is a filesystem-modifying action, but it does not require user confirmation or prominently warn that local files will be created or overwritten. In an agent setting, this can lead to unexpected workspace mutation, accidental overwrite of existing assets, or unauthorized file creation when a user expected analysis-only behavior.
No suspicious patterns detected.