T01 · Skill Instruction Hijacking
- Location
SKILL.md:67- Finding
Mandatory Delegation to an Unverified External Skill
- Content
View full analysis
Vulnerability Details
File Location:
SKILL.md:67-72
Vulnerability Type: T01: Skill Instruction Hijacking
Risk Level: HighVulnerable code snippet:
markdown 3. **Verification protocol** — Load and apply [review-verification-protocol](../review-verification-protocol/SKILL.md) for the issue type. **Pass:** that skill’s pre-report checks for that finding class are satisfied before you write the finding. 4. **Finding shape** — Emit each issue as `[FILE:LINE] ISSUE_TITLE` with a one-line rationale tied to the cited code. **Pass:** every line matches that pattern. ## Before Submitting Findings Do not report until **Gates** above pass. For full anti-false-positive steps, follow [review-verification-protocol](../review-verification-protocol/SKILL.md).Technical Analysis
The Skill requires the agent to load and apply instructions from the sibling path
../review-verification-protocol/SKILL.md, which is outside the audited package and was not included in the supplied project. The referenced instructions therefore cannot be reviewed or integrity-verified as part of this audit.This is an instruction-hijacking risk because the external Skill is made a mandatory prerequisite for producing findings. Its content would execute within the agent’s current review context and could introduce unrelated directives, alter review criteria, suppress findings, redirect tool usage, or modify the final output. The instruction to refrain from reporting until the external gate passes increases the external file’s control over the agent’s behavior.
Attack Path
- An attacker creates or modifies
../review-verification-protocol/SKILL.mdin the environment containing the audited Skill. - A user invokes
phoenix-code-reviewfor a Phoenix code review. - The agent reaches the mandatory verification gate in
SKILL.md. - The agent loads the attacker-controlled sibling Skill as instructed.
- The sibling Skill su ...[truncated 879 chars]
- An attacker creates or modifies
- Remediation
View remediation
Remediation Suggestions
- Remove the mandatory dependency on the sibling Skill.
- Incorporate all required verification rules directly into this package so the complete instruction set can be audited together.
- If modularization is necessary, vendor the dependency inside the project rather than referencing a mutable parent-directory path.
- Pin the dependency to a reviewed version and verify its cryptographic digest before use.
- Treat external Skill content as untrusted reference material rather than authoritative instructions.
- Do not make report generation conditional on compliance with instructions obtained from outside the reviewed package.
- Explicitly state that any referenced guidance remains subordinate to system instructions, user requirements, and security policies.
