T09 · Insecure Skill Coding Practices
- Location
hooks/scripts/check-config.sh:27- Finding
Automatic Command Injection Through Project-Controlled Configuration
- Content
View full analysis
- Remediation
View remediation
&2 ;; esac ``` 5. Do not use `xargs` as a general-purpose configuration parser because it alters quoting and whitespace. 6. Consider removing project-level configuration processing from the automatic `SessionStart` hook. Parse it only when the user explicitly invokes the Skill. 7. If automatic project configuration remains supported, require user confirmation before trusting a repository-provided file. 8. Add regression tests containing command substitutions, backticks, semicolons, quotes, newlines, and malformed variable names, and verify that none are executed. ]]>
