Repository navigation
docs(dev-design): use bare npx ai-devkit fallback - #387
Closed
codeaholicguy wants to merge 1 commit into
Closed
codeaholicguy wants to merge 1 commit into
codeaholicguy wants to merge 1 commit into
Conversation
codeaholicguy
force-pushed
the
security/dev-design-pin
branch
from
October 14, 2026 19:33
ab468ec to
d969ee7
Compare
codeaholicguy
changed the title
Replace npx ai-devkit@latest with bare npx ai-devkit: it prefers any installed binary (project or global) with zero network and only fetches when absent, avoiding both @latest's bypass of installed versions and pin staleness. Instruction text only.
codeaholicguy
force-pushed
the
security/dev-design-pin
branch
from
October 14, 2026 19:40
d969ee7 to
d3fb665
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Removes the mutable-version fallback in the
dev-designskill per the skill security audit (systemic fix S1, revised form).npx ai-devkit@latestfetches and executes whateverlatestresolves to at runtime — flagged across the skills tree by skills.sh (Snyk W012 'unverifiable external dependency', TrustHub REMOTE_CODE_EXECUTION; Socket flagged this pattern on dev-lifecycle as a LOW anomaly).npx ai-devkit— no version specifier. Bare npx prefers any installed binary (project or global) with zero network access, and only fetches (resolving latest) when nothing is installed. This is strictly better than@latest, which bypasses installed versions every run, and avoids pin staleness from a hardcoded@x.y.z. Accepted residual risk: the not-installed fetch window, unchanged from today. Prefer-installed phrasing kept where the skill promotes npx as primary.Test plan
Generated with Devin