T09 · Insecure Skill Coding Practices
- Location
SKILL.md:319- Finding
Scanner Fails at Startup Due to Function Use Before Definition
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
The skill looks like a legitimate path traversal audit tool, but the shipped artifact does not provide a working scanner and could give users false security confidence.
Treat this as a Review item before installing. It does not show hidden data theft, persistence, or network behavior, but do not rely on it as a CI security gate until the scanner is packaged as an actual Python file, the import-time bug is fixed, and tests prove it catches and reports expected vulnerabilities.
SKILL.md:319Scanner Fails at Startup Due to Function Use Before Definition
SKILL.md:438Unrelated Nearby Guard Can Suppress Genuine Path-Traversal Findings
The implementation references _build_js_path_patterns() inside the PATTERNS dictionary before that helper is defined, which would raise a runtime error during module import and prevent the scanner from running as documented. A broken security scanner creates a dangerous false sense of protection because users may believe path traversal checks are being enforced when no scan actually occurs.
Code accesses credential files (SSH keys, AWS credentials, etc.). This could indicate credential theft attempts.
An attacker passes ../../etc/passwd or ..%2F..%2Fetc%2Fshadow as a filename parameter. Without validation, your code reads arbitrary files outside the intended base directory. With PHP include, it can lead to Remote Code Execution.
Classic exploit:
Code accesses credential files (SSH keys, AWS credentials, etc.). This could indicate credential theft attempts.
An attacker passes ../../etc/passwd or ..%2F..%2Fetc%2Fshadow as a filename parameter. Without validation, your code reads arbitrary files outside the intended base directory. With PHP include, it can lead to Remote Code Execution.
Classic exploit:
Code accesses credential files (SSH keys, AWS credentials, etc.). This could indicate credential theft attempts.
An attacker passes ../../etc/passwd or ..%2F..%2Fetc%2Fshadow as a filename parameter. Without validation, your code reads arbitrary files outside the intended base directory. With PHP include, it can lead to Remote Code Execution.
Classic exploit:
The manifest and surrounding documentation describe a path traversal and LFI scanner focused on user-controlled paths reaching file system sinks, with examples centered on reads/includes and CWE-22/CWE-23/CWE-98 style traversal issues. The implementation additionally includes a Node.js pattern for fs.writeFile/writeFileSync/appendFile with request input, extending behavior into arbitrary file overwrite/write detection that is not disclosed in the manifest's stated scope.
No suspicious patterns detected.