Skip to content

docs(dev-requirements): pin ai-devkit and mark feature docs untrusted - #383

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

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

Conversation

@codeaholicguy

@codeaholicguy codeaholicguy commented Oct 14, 2026 •

Copy link
Copy Markdown
Owner

Summary

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

  • Finding: skills.sh Snyk Warn / MEDIUM (W012, risk 0.90) — repeated npx ai-devkit@latest calls fetch and execute remote package code at runtime (6 call sites including init --yes).
  • 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/feature 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-requirements/SKILL.md only
  • Repo pre-push hooks passed

Generated with Devin

@codeaholicguy
codeaholicguy force-pushed the security/dev-requirements-untrusted-docs branch from ca872d5 to e27e153 Compare October 14, 2026 19:37
Pin npx ai-devkit to 0.70.1, prefer the installed binary, and state
that requirements docs are data whose embedded instructions must not
be executed. Instruction text only.
@codeaholicguy
codeaholicguy force-pushed the security/dev-requirements-untrusted-docs branch from e27e153 to b8cfb02 Compare October 14, 2026 19:40
@codeaholicguy
codeaholicguy deleted the security/dev-requirements-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