T09 · Insecure Skill Coding Practices
- Location
SKILL.md:61- Finding
Untrusted Job Content Is Copied into Privileged Sub-Agent Instructions
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
This skill is an agent-spawning workflow helper, but it delegates git pushes, SSH-related outreach work, and sensitive context without clear approval or least-privilege controls.
Review before installing. This skill is not deceptive or destructive by itself, but it is meant to coordinate autonomous sub-agents that may modify repositories, push commits, use SSH-related instructions, and handle sensitive business or personal files. Use it only in a controlled workspace with trusted JOB_BOARD.md authors, limited credentials, protected branches, and explicit approval before any push, remote command, or sensitive-data handling.
SKILL.md:61Untrusted Job Content Is Copied into Privileged Sub-Agent Instructions
SKILL.md:42Sensitive Context and High-Impact Git or SSH Operations Are Delegated without Least-Privilege Controls
Code accesses credential files (SSH keys, AWS credentials, etc.). This could indicate credential theft attempts.
The skill explicitly instructs spawned sub-agents to perform git commits and, for BUILD jobs, to push changes, but it provides no user-facing warning or consent checkpoint before initiating those actions. In an agent-spawning skill, this increases the risk of unintended repository modification or publication because downstream agents may act autonomously based on orchestrator input rather than explicit user approval.
Skill enables autonomous high-impact decisions without human-in-the-loop verification. Critical operations (destructive commands, financial transactions, data deletion) should require explicit user confirmation.
- ❌ Spawning 10 agents at once (max 3)
- ❌ Spawning an agent for a 2-minute task (do it inline)
- ❌ Not passing context (agent wastes time re-reading)
- ❌ Marking ✅ without checking output exists
- ❌ Polling subagents in a loop (wait for push events)
- ❌ Spawning duplicate agents for the same job
- ❌ Not killing stale agents (> timeout + 5 min buffer)