T01 · Skill Instruction Hijacking
- Location
SKILL.md:588- Finding
Server-Controlled Integration Instructions Are Treated as Trusted Runtime Directives
- Content
View full analysis
Vulnerability Details
File Location:
SKILL.md:243-247andSKILL.md:588-597
Vulnerability Type: Remote instruction retrieval and skill instruction hijacking
Risk Level: CriticalVulnerable 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.mdfromapp.clawmentor.aiand explicitly directs the agent to trust and follow that remote document. The remo ...[truncated 1873 chars]- Remediation
View 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.
