T08 · Insecure Dependencies
- Location
SKILL.md:22- Finding
Mandatory Unbundled Skill Dependency Can Alter Review Behavior
- Content
View full analysis
Vulnerability Details
File Location:
SKILL.md, lines 22-23
Vulnerability Type:T08: Insecure Dependencies
Risk Level: MediumVulnerable Code Snippet:
markdown 4. **Pre-report verification** — Load and follow [review-verification-protocol](../review-verification-protocol/SKILL.md). **Pass:** The protocol’s **Pre-Report Verification Checklist** is satisfied for each finding you will report (actual code read, surrounding context checked, “wrong” vs “different style” distinguished, etc.).The requirement is reiterated at line 154:
markdown Satisfy **step 4** in [Review Workflow](#review-workflow): load [review-verification-protocol](../review-verification-protocol/SKILL.md) and complete its pre-report checks for each issue.Technical Analysis
The Skill mandates loading and following instructions from
../review-verification-protocol/SKILL.md, a sibling path outside the audited project. That dependency is not included in the supplied package and therefore cannot be reviewed, integrity-checked, or version-pinned as part of this audit.Because the referenced file contains agent instructions rather than passive data, changing it can directly modify the effective behavior of this Skill. A malicious or compromised sibling Skill could instruct the agent to suppress legitimate findings, produce misleading results, access unrelated resources, or perform other actions beyond the intended Go code-review workflow.
Attack Path
- An attacker gains the ability to create or modify the sibling file at
../review-verification-protocol/SKILL.md. - A user invokes this Go code-review Skill.
- The workflow reaches its mandatory pre-report verification step.
- The agent loads and follows the attacker-controlled sibling Skill as required by
SKILL.md. - The injected instructions influence the review process or final report, potentially suppressing findings or directing unintended ac ...[truncated 720 chars]
- An attacker gains the ability to create or modify the sibling file at
- Remediation
View remediation
Remediation Suggestions
- Bundle the verification protocol inside this project so it is reviewed and distributed with the primary Skill.
- Replace the parent-directory reference with a package-local path, such as
references/review-verification-protocol.md. - If an external dependency is unavoidable, pin it to a trusted immutable version and verify its content using a cryptographic hash before loading it.
- Document the dependency’s trusted source, version, update process, and ownership.
- Explicitly state that dependency instructions cannot override system, developer, user, security, or least-privilege constraints.
- Treat missing or failed integrity verification as a hard failure rather than loading an unverified replacement.
- Include the dependency in future security audits and review all updates before deployment.
