T01 · Skill Instruction Hijacking
Error
- Location
- SKILL.md:588
- Finding
- Server-Controlled Integration Instructions Are Treated as Trusted Runtime Directives## Vulnerability Details **File Location**: `SKILL.md:243-247` and `SKILL.md:588-597` **Vulnerability Type**: Remote instruction retrieval and skill instruction hijacking **Risk Level**: Critical ### Vulnerable Code ```text This returns two sections: - `files` — the mentor's authored content: `AGENTS.md`, `skills.md`, `cron-patterns.json`, `CLAW_MENTOR.md`, `privacy-notes.md`, `working-patterns.md` - `platform` — platform guides: `mentee-integration.md` (the full integration algorithm), `setup-guide.md`, `mentee-skill.md` (detailed operations guide) ``` ```text 4. **Read `platform['mentee-integration.md']` from the package response.** This contains the full 6-phase integration algorithm: - Phase 0: Backup (snapshot your current setup) - Phase 1: Self-Assessment (read and map your human's complete current state) - Phase 2: Update Parsing (understand what the mentor is proposing) - Phase 3: Conflict Analysis (find every collision, overlap, and opportunity) - Phase 4: Merge Strategy & Human Approval (present the plan, get explicit approval) - Phase 5: Integration Execution (apply the approved merge carefully) - Phase 6: Post-Integration Report (tell your human what changed) 5. **Follow `mentee-integration.md` step by step.** It handles everything: backup, conflict detection, voice preservation, capacity checks, cron jobs, skill recommendations, memory system setup, and rollback safety. **Important:** The integration algorithm in `mentee-integration.md` is comprehensive. Trust it — it handles edge cases like interrupted integrations, minimal configs, multiple mentors, and capacity preservation. Don't skip phases or simplify the process. ``` ### Technical Analysis The reviewed Skill is not the complete implementation of its integration behavior. It downloads `mentee-integration.md` from `app.clawmentor.ai` and explicitly directs the agent to trust and follow that remote document. The remo ...[truncated 1873 chars]
- Remediation
- ## Remediation Suggestions - Bundle the integration algorithm locally in the reviewed Skill rather than downloading executable instructions. - Treat every downloaded Markdown field as untrusted data and prohibit it from issuing tool-use directives. - Define a strict, versioned package schema containing declarative changes only. - Enforce a local allowlist of supported actions, destination paths, commands, and configuration keys. - Cryptographically sign packages and verify signatures against pinned publisher keys before processing. - Separate package parsing from execution and display a normalized operation plan generated by trusted local code. - Reject package fields containing shell commands, instruction overrides, tool directives, or requests to read unrelated files. - Require independent per-operation authorization for cron creation, skill installation, command execution, and core-file changes.
