T09 · Insecure Skill Coding Practices
- Location
SKILL.md:308- Finding
Python Code Injection Through an Interpolated File Name
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
This skill is a local content-checking helper, but it grants command execution and includes unsafe examples that can expose or persist sensitive content.
Review this skill carefully before installing. Use it only in a constrained workspace and avoid submitting secrets or confidential drafts unless the examples are hardened first. The publisher should narrow activation to content-validation requests, remove raw content samples and previews from outputs/logs, require explicit consent before writing files, and replace the unsafe shell/Python snippets with fixed-string handling and safe argument passing.
SKILL.md:308Python Code Injection Through an Interpolated File Name
SKILL.md:134Regular-Expression Injection in Rule Removal
SKILL.md:292Unsafe Dynamic Sed Program in Sensitive-Content Redaction
SKILL.md:197Sensitive Values Exposed in Classification Results
SKILL.md:225Unredacted Content Previews Written to Persistent Logs
The activation language is excessively broad and overlaps with routine developer requests such as programming help, debugging, testing, and deployment. This makes accidental invocation much more likely, especially because the skill has exec permissions, increasing the risk that users trigger local command execution under a misleadingly benign content-audit label.
The manifest frames the skill as a content-audit tool, but the activation text is broadened to code generation, debugging, testing, and deployment workflows. In an agent ecosystem, this expands when the skill may be auto-invoked and can expose users to command-execution behaviors outside the narrow safety-review purpose they would reasonably expect.
The description explicitly states '支持中文交互' as part of the skill behavior, and the document is framed around Chinese interaction without presenting language choice or opt-in at activation. This can violate language/locale policy expectations when users have not chosen Chinese as their preferred interaction language.
The free-vs-pro comparison explicitly states that the free version does not support real-time monitoring. Yet the top-level '核心功能' text says the skill provides '化工作流场景等能力', and elsewhere the document overstates broad capabilities; the clearest contradiction is the free-tier limitation versus later generalized capability claims that blur edition boundaries.
The verification logging function stores raw content previews in local log files, which can include sensitive user-provided data such as credentials, personal information, or proprietary text. Because this skill is specifically designed to inspect potentially sensitive content, logging previews creates a realistic secondary leakage channel and expands data retention without clear user warning or minimization.
Storing natural-language content previews in verification logs creates a direct data leakage path for exactly the kinds of sensitive inputs this tool is meant to inspect. This is especially dangerous in the skill context because users may send secrets, personal data, or confidential drafts for screening, and the logging behavior increases persistence and discoverability of that data.
The filtering workflow rewrites user content into a new local output file, persisting transformed versions of potentially sensitive input without an explicit warning. In a content-screening skill, silent file creation/modification can surprise users, leave recoverable copies on disk, and broaden exposure of sensitive material to other local processes or users.
Lines L396-L397 state that the free version is purely local and does not send content externally. Later, the security section discusses network communication safety, HTTPS, and SSL certificate validation, which directly implies networked behavior inconsistent with the earlier documented intent for this edition.
The skill advertises broad file handling, API integration, and system command execution beyond what is necessary for basic local content validation. In the context of an agent with exec access, unnecessary capability breadth increases the chance of unsafe actions, misuse of local files, and over-collection or modification of user data.
The manifest and main description consistently position the skill as a local AI-generated content validation tool with blacklist/allowlist management, classification, and basic policy enforcement. However, the later '核心功能' section explicitly adds 'API集成' via external service calls, which is a materially broader behavior not justified by the stated product scope, especially since earlier text says the free version is pure local processing and does not send content externally.