Skill v1.0.1
currentAutomated scan100/100+2 new
version: "1.0.1" name: ca-add-dep description: Vet a new or changed third-party dependency for license, provenance, and supply-chain risk before any install runs. argument-hint: "<package[@version]>"
/ca-add-dep — dependency review
For every security policy reference below, follow <plugin-root>/includes/knowledge.md ("Read security policy before a security decision"): select the repository's context format, read the complete policy from exact CTX-SECURITY-CONTROLS records in HTML mode or the full Markdown controls in Markdown mode, and read operative scope blocks when applicable. Required policy that is missing or unreadable blocks a security decision; scope blocks alone are not full policy.
Gate a new or changed third-party dependency through review before it lands. When you route through this command, the orchestrator runs no install until the dependency-reviewer agent clears the package. Specify the exact version if you have one; without one, the reviewer evaluates the latest available version.
Routes to
The dependency-reviewer agent (<plugin-root>/agents/dependency-reviewer.md). The agent reads <project-root>/.codearbiter/security-controls.md (allowed/denied licenses, provenance and supply-chain policy) and the format-selected tech-stack document (<plugin-root>/includes/knowledge.md; stack fit, dependency manager) to judge the package.
After the agent clears it, the orchestrator surfaces the install command for confirmation. The lock file change is committed alongside the manifest change — never one without the other.
When NOT to use
- Removing a dependency →
/ca-fixor/ca-featurewith the change described. - Updating an existing dependency as part of a code change →
/ca-feature//ca-fix; manifest
changes route to review at /ca-pr.
- Asking about a package without installing it → ask it directly; no command needed.
Ephemeral tool run — one invocation, nothing adopted
A one-time developer tool is not a dependency, and reviewing it as one is what issue #346 records going wrong: a duplicate-code investigation was interrupted and pushed toward project-dependency review for jscpd, which the operator had explicitly said must never be listed as a dependency. The redirect loop then reached for /ca-override, a bypass invented to cover missing coverage. This section is that coverage.
The distinction is the dependency GRAPH, not the download. Adopting or changing anything that enters package.json, a lockfile, or a base image is the review above, unchanged. Running a pinned tool once, against the repository, adopting nothing, is this.
Route here when ALL of these hold. If any is unclear, it is a dependency and takes the full review:
- The operator asked for this specific tool, by name, in this session.
- It runs ONCE, for inspection or analysis. It is not wired into a script, a
hook, or CI.
- Nothing it does may change a manifest, a lockfile, or a committed artifact.
Then:
- Pin the exact version.
npx jscpd@4.0.5 ., never barenpx jscpd— an
unpinned invocation resolves to whatever was published this morning, which is the supply-chain exposure the review exists for and the one part of it that still applies at full strength.
- Use the approved registry.
https://registry.npmjs.orgis the only one
(security-controls.md); no git+, no file:, no non-TLS source.
- Confirm before running, naming the exact command. One approval, not a
review — the operator has already made the adoption decision, which is "no".
- Isolate what you can. Prefer a cache directory under the system temp dir
over the project tree, and write any report there too, so the run leaves no residue a later commit could pick up.
- Report what ran — the pinned command and where its output went.
Hard gate: MUST NOT modify a manifest or a lockfile. If the tool writes one, that is adoption, and it stops here and routes to the review above. Verify rather than trust: git status --porcelain over the manifest and lockfile paths must be unchanged after the run. A tool that requires adoption to run at all is a dependency, and saying so is the correct answer.
This is a bounded carve-out inside this command, not a command of its own (ADR-0023). A new command would add a public surface — the catalog, the Pi command catalog, the README counts, the site sidebar — to govern an action whose whole definition is that it changes nothing.
Hard gate
MUST NOT install before dependency-reviewer clears the package. BLOCK on a denied license or any unresolved supply-chain or provenance concern.
This gate is orchestrator-enforced, not hook-enforced: unlike the crypto/secret gate (pre-bash.py H-09b/H-10b), there is no pre-bash rule that blocks a bare npm/pip/yarn/pnpm install typed outside this command. The discipline depends on routing installs through /ca-add-dep. A hook-level install block (parallel to the security-gate marker) would be a stronger posture, but is a deliberate behavior change — a possible future enhancement, not the current contract.