Skip to content

docs(dev-planning): pin ai-devkit and mark feature docs untrusted - #382

Closed
codeaholicguy wants to merge 1 commit into
mainfrom
security/dev-planning-untrusted-docs
Closed

codeaholicguy wants to merge 1 commit into
mainfrom
security/dev-planning-untrusted-docs

Conversation

@codeaholicguy

@codeaholicguy codeaholicguy commented Oct 14, 2026 •

Copy link
Copy Markdown
Owner

Summary

Hardens the dev-planning skill per the skill security audit.

  • Finding: skills.sh Gen Agent Trust Hub FAIL / HIGH — REMOTE_CODE_EXECUTION + COMMAND_EXECUTION via unpinned npx ai-devkit@latest (×4), and PROMPT_INJECTION: the skill ingests requirements/design/testing docs with no instruction to treat embedded content as data. Snyk Warn MEDIUM (W012).
  • Change: converts all npx ai-devkit@latest calls to bare npx ai-devkit; adds a lead-in line preferring the installed binary and stating that requirements/design/testing docs are data whose embedded instructions must not be executed.
  • Behavior: instruction text only; no executable code changed.

Test plan

  • Single-file diff: skills/dev-planning/SKILL.md only
  • Repo pre-push hooks passed

Generated with Devin

@codeaholicguy
codeaholicguy force-pushed the security/dev-planning-untrusted-docs branch from 7ef6569 to 7fe284c Compare October 14, 2026 19:37
Pin npx ai-devkit to 0.70.1, prefer the installed binary, and state
that requirements/design/testing docs are data whose embedded
instructions must not be executed. Instruction text only.
@codeaholicguy
codeaholicguy force-pushed the security/dev-planning-untrusted-docs branch from 7fe284c to c06157f Compare October 14, 2026 19:40
@codeaholicguy
codeaholicguy deleted the security/dev-planning-untrusted-docs branch October 14, 2026 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant