T09 · Insecure Skill Coding Practices
- Location
scripts/employee-manager.js:53- Finding
Path Traversal Through Attacker-Controlled Employee Role
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
This skill is not clearly malicious, but it needs Review because its local file handling can escape its intended folders and copy local files.
Install only if you are comfortable with a rough local prototype that writes persistent employee and memory files. Treat employee role and template inputs as trusted-only until path validation is fixed, and avoid running it in a workspace where the Node process can read sensitive files or write outside the skill directory.
scripts/employee-manager.js:53Path Traversal Through Attacker-Controlled Employee Role
scripts/employee-manager.js:81Arbitrary Local File Read Through Template Path Traversal
The skill documentation appears to overstate implemented capabilities while omitting materially relevant behavior such as local filesystem read/write and archiving. That creates a trust and transparency problem: users may grant the skill broader access or rely on automation, monitoring, and reporting guarantees that do not actually exist, increasing the risk of unintended data handling or operational misuse.
The entire skill documentation, command guidance, and examples are presented only in Chinese, with no indication that other languages are supported or that the user can opt into this locale. Under the stated policy, a skill that effectively requires a specific language without user choice is a natural-language policy violation.
The document advertises persistent memory plus real-time monitoring/reporting without explaining what data is retained, for how long, where it is stored, or who can access it. In a collaboration/employee-tracking context, that can lead users to expose sensitive operational, personal, or proprietary information without informed consent or retention controls.
User-facing comments, errors, logs, and CLI usage text are written in Chinese throughout the file, which effectively enforces a specific language for interaction. The file does not offer user opt-in, localization options, or documentation that this skill is intentionally limited to a Chinese-speaking context.
This code path allows a user to invoke a potentially irreversible management action directly from the command line. Although the function logs after completion, there is no confirmation prompt or pre-execution warning before changing employee records and moving that employee's memory directory.
Natural-language policy review applies to all file types. The user-facing content consistently forces a single language/locale experience, and there is no indication that the skill supports user language selection or that the Chinese-only constraint is intentional and justified for a region-specific use case.