| name | dependency-audit |
| description | Audit Node.js dependency tree (security, health, compliance, hygiene). Static-first with optional --with-network enrichment for vulnerabilities, outdated, and abandonment data. Optionally generates an implementation plan. |
| trigger | /dependency-audit |
/dependency-audit
Audit a Node.js project's dependency tree against an opinionated baseline organised in four layers — security, health, compliance, hygiene — preceded by a diagnostic snapshot. Then offer to generate an implementation plan for the gaps.
The default mental model is TypeScript and React. The skill works on any Node.js project that uses one of the recognised package managers: npm, pnpm, yarn, or bun.
Static-first design with optional network enrichment
Vulnerability data, "is this package outdated", and "is this package abandoned" all need information that does not live in the repository. This skill is read-only and never installs, updates, or modifies anything. It runs in two modes:
- Static (default). Read
package.json, the lockfile, and (when present) the package.json of each installed package under node_modules/. Security and outdated checks degrade with a clear "needs --with-network" gap.
- Static plus opt-in
--with-network. When the flag is passed, the skill additionally runs the package manager's own read-only audit and outdated commands (npm audit --json, npm outdated --json, or pnpm/yarn/bun equivalents) and parses their output. No install, no update, no lockfile rewrite — only the read-only registry queries the package manager already exposes.
Running npm install, npm update, or any mutating operation is the responsibility of the user or a separate fix-and-validate skill. Keeping this audit fully read-only is what makes it safe to run in any working tree at any time.
The three input tiers
The audit's accuracy scales with what is available. Each check declares which tier it needs; tier-dependent checks degrade with a clear gap rather than silently passing or failing.
| Tier | What is read | What it unlocks |
|---|
| 1 — Lockfile only | package.json, lockfile (package-lock.json, pnpm-lock.yaml, yarn.lock, or bun.lockb) | Versions, transitive tree, duplicates, misplaced dependencies, lockfile hygiene. License and abandonment are unknown. |
2 — Lockfile + node_modules (default static run) | Everything in tier 1 plus the package.json of each installed package | Adds license fields, install scripts, disk size per package. |
3 — Lockfile + node_modules + network (--with-network) | Everything above plus registry queries via the package manager's own audit and outdated commands | Adds vulnerabilities, outdated versions, abandonment signals, deprecation messages. |
The skill detects which tier is available and records it in the diagnostic snapshot. There is no automatic install: if node_modules is absent, the audit runs at tier 1 and tier-2 checks degrade.
Usage
/dependency-audit # default: concise Top 5 + full report saved + ask about plan
/dependency-audit --worktree # create an isolated Git worktree, then run the audit there
/dependency-audit --learn # mid-level engineer teaching mode (detailed explanations + file/line examples)
/dependency-audit --teach # alias for --learn
/dependency-audit --with-network # tier 3: enrich with audit and outdated registry data
/dependency-audit --threshold-major-versions-behind=3 # override default 2
/dependency-audit --threshold-abandonment-months=18 # override default 24
/dependency-audit --security-critical-packages=react,next,@remix-run/react # override the default critical-package list
💡 Pro tip: Add --worktree to run this audit in an isolated Git worktree.
The skill never accepts --apply. The implementation plan is descriptive Markdown.
The defaults baked into the skill are the recommended baseline. Threshold flags exist as an escape hatch; the canonical path to evolving the defaults themselves is /system-self-improve.
💡 Pro tip: Run /preflight --audit=dependency first to confirm the package manager and lockfile are detected — --with-network calls the package manager's audit subcommand (npm audit, pnpm audit, yarn audit, or bun pm audit), so no install is needed, but a missing or unrecognised lockfile will silently degrade the enrichment.
The opinionated baseline
A check resolves to one of four statuses:
- present — the invariant holds.
- partial — most signals resolve, with a small number of exceptions, or the check needs a higher tier than what the run provided.
- missing — a structural prerequisite is absent.
- violation — concrete evidence in the project breaks the invariant.
Layer 0 is informational only and has no status.
Layer 0 — Diagnostic snapshot (always written, no pass/fail)
- Detected package manager and version (npm 10.x, pnpm 9.x, yarn 4.x, bun 1.x).
- Lockfile path and last-modified timestamp.
node_modules present: yes/no.- Counts: direct dependencies, dev dependencies, transitive total.
- Tree depth statistics: deepest path, average depth.
- Top 10 largest direct dependencies on disk (when
node_modules present).
- Duplicate package count: same package name, multiple resolved versions in the lockfile.
- Workspace count when the project is a monorepo.
- Tier-of-input the audit ran with (1, 2, or 3).
Layer 1 — Security
| Check | Tier | Expectation | Violation signal |
|---|
| No high or critical vulnerabilities | 3 | The package manager's audit reports no high or critical advisories. | Any high or critical advisory. Reported with package, severity, advisory ID, and fix availability. |
| No moderate vulnerabilities | 3 | Soft check. Moderate advisories report as partial rather than violation. | Moderate advisories present. |
| Lockfile present and committed | 1 | A lockfile exists and is tracked by git (not in .gitignore). | No lockfile, or lockfile in .gitignore. |
| Lockfile integrity verifiable | 1 | The lockfile carries integrity hashes and continuous integration uses the frozen-install variant (npm ci, pnpm install --frozen-lockfile, yarn install --immutable, bun install --frozen-lockfile). | Lockfile in a format without integrity hashes; or continuous-integration scripts not using the frozen-install variant. |
| No install scripts from untrusted packages | 2 | Direct dependencies that declare postinstall (or other lifecycle) scripts are limited to a known-trusted allowlist (typing definitions, build tools, framework-installed essentials such as husky, playwright, puppeteer, esbuild, sharp). | Direct-dependency lifecycle scripts present from packages outside the allowlist. Reported with the package, the script, and the script body. |
| Continuous-integration vulnerability scanning enabled | 1 | A continuous-integration workflow runs the package manager's audit, or Dependabot/Snyk/Renovate configuration is present. | No such automation detected (no dependabot.yml, no renovate.json/renovate.json5, no npm audit step in any workflow). |
Layer 2 — Health
| Check | Tier | Expectation | Violation signal |
|---|
| No dependencies more than two majors behind | 3 | Each direct dependency is at most the threshold number of major versions behind its latest release (default 2; tunable via --threshold-major-versions-behind). | Direct dependency exceeds the threshold. Reported with current and latest. |
| No deprecated-major usage | 3 | When a package's current major is upstream-deprecated, even a one-minor lag is reported as partial. Soft check — registry deprecation semantics are advisory and ecosystem-dependent. | Deprecated-major usage. |
| No abandoned packages | 3 | No direct dependency has gone more than the threshold months without a publish (default 24; tunable via --threshold-abandonment-months). Soft check — reported as partial. | Direct dependencies whose latest publish is older than the threshold. |
| No officially deprecated packages | 3 | No direct dependency carries a deprecated flag in the registry. | Deprecated direct dependency. Reported with the deprecation message and the recommended replacement when the registry provides one. |
| Peer dependencies satisfied | 2 | Every declared peer dependency in the tree resolves to a satisfying version. | Unsatisfied peer dependency warnings from npm ls --all/pnpm list --depth Infinity/equivalent. |
Layer 3 — Compliance
| Check | Tier | Expectation | Violation signal |
|---|
| Every package has a license declared | 2 | No package in the dependency tree is missing a license field. | Packages with license: "UNLICENSED", "SEE LICENSE IN ..." without a resolvable file, or a missing field. |
| Strong-copyleft licenses are flagged for human review | 2 | The audit surfaces, but does not enforce, the presence of GPL, AGPL, or LGPL packages in the tree. When .architect-playbook/licenses.json declares an allowlist that includes them, they pass; otherwise the situation is reported as partial with a recommendation to consult legal and decide. | GPL, AGPL, or LGPL in the tree with no allowlist file. |
| Source-available restrictive licenses are flagged for human review | 2 | Same posture as copyleft: surface the presence of SSPL, BUSL, Commons Clause, or similar. Allowlist via .architect-playbook/licenses.json. | Restrictive licenses present without allowlist. |
| License-checker tool present | 1 | A license-enforcement tool (license-checker, license-compliance, or equivalent) is configured and a continuous-integration step runs it. | No license tool detected, or tool present without a continuous-integration step invoking it. |
The skill never encodes a legal policy. Compliance findings are signals for a human (and where appropriate, a lawyer) to act on; they are not pass/fail gates with built-in opinions about which licenses are acceptable.
Layer 4 — Hygiene
| Check | Tier | Expectation | Violation signal |
|---|
| No unused dependencies | 1 (low confidence) or graph-enhanced (high confidence) | Every entry in dependencies and devDependencies is imported somewhere in the project. When the Graphify graph is present, this check uses the graph as the import truth (high confidence). Without the graph, falls back to a regex/AST sweep across source and configuration files (low confidence). | Packages declared but never imported. The findings record the confidence level. |
| No misplaced dependencies | 1 (low confidence) or graph-enhanced | Packages used in production code (under src/, framework conventions) live in dependencies; packages used only in tests, build, or development tooling live in devDependencies. | A devDependency imported by production code, or a dependency imported only by test/build files. |
| No duplicate packages | 1 | Each package appears at exactly one resolved version in the lockfile. | Same package name with multiple versions in the lockfile. Reported with the package and the conflicting versions. |
| No package-manager mixing | 1 | Exactly one lockfile is present. The continuous-integration workflow uses the same package manager that produced the lockfile. | Multiple lockfiles in the repository, or a continuous-integration workflow using a different package manager than the lockfile implies. |
| Security-critical packages exact-pinned | 1 | A configurable allowlist of security-critical packages (default: react, react-dom, next, @remix-run/react, @remix-run/node) is exact-pinned in package.json (no ^, no ~). Soft check — reported as partial. | Security-critical packages declared with ^ or ~ ranges. |
engines field declared | 1 | package.json declares the supported node version range. | Field absent. |
What this skill does
-
Reads the knowledge graph when present. Soft dependency: when graphify-out/graph.json exists, the unused-dependency and misplaced-dependency checks use the graph as the import truth (high confidence). When absent, both checks fall back to a regex/AST sweep and record confidence: "low" on their findings. The audit still runs in full either way.
-
Confirms a Node.js project. Detects package.json. If absent at the intended repository root, the skill writes a canonical not-applicable audit; if no project root can be established, it asks the user to change directory.
-
Detects the package manager and tier. Infers the package manager from the lockfile present (and falls back to the packageManager field in package.json). Determines the input tier based on the presence of node_modules and the --with-network flag.
-
When --with-network is set, runs the package manager's read-only audit and outdated commands and captures their JSON output. Never runs install, update, or any mutating operation.
-
Writes Layer 0 — the diagnostic snapshot to .architect-audits/dependency-audit/snapshot.md and prepends the same content to findings.md.
-
Walks each check in the active layer list, applying any --include, --exclude, threshold overrides, and the security-critical-package list. Tier-dependent checks running below their required tier emit partial with a "needs tier N — pass --with-network (or run npm install first)" gap.
-
Writes phase 1 outputs to .architect-audits/dependency-audit/:
findings.md — diagnostic snapshot followed by check results, grouped by layer.findings.json — machine-readable.snapshot.md — diagnostic snapshot on its own.metadata.json — skill version, run timestamp, Graphify revision (when present), package manager, tier, applied thresholds, applied filters.
-
Phase 2 — offers to plan the gaps. Summarises the findings in chat and asks the user a single yes-or-no question:
Implementation steps
Step 1 — Confirm the prerequisites
test -f package.json || { echo "dependency-audit: no package.json detected. This skill currently supports Node.js projects only."; exit 1; }
Detect the package manager:
package-lock.json → npm.pnpm-lock.yaml → pnpm.yarn.lock → yarn (Berry detected by absence of node-modules linker default and presence of .yarnrc.yml).bun.lockb → bun.
When multiple lockfiles are present, fall back to the packageManager field in package.json. When none of these resolves, continue: record the package-manager foundation check as missing, mark package-manager-command and lockfile-dependent checks applicable but not-evaluated, and evaluate manifest-only checks where possible.
Step 2 — Determine the input tier
test -d node_modules && tier=2 || tier=1
[ "$WITH_NETWORK" = "1" ] && tier=3
Record the resolved tier in metadata.json.
Step 3 — Run network commands when tier 3
When --with-network is set, run the package manager's read-only commands and capture their JSON:
- npm:
npm audit --json and npm outdated --json.
- pnpm:
pnpm audit --json and pnpm outdated --format=json.
- yarn (Berry):
yarn npm audit --json --recursive and yarn outdated --json (or via the relevant Berry plugin).
- bun:
bun pm audit --json (when supported) and bun outdated --json.
If any of these commands fails (network blocked, registry unreachable), record the failure in metadata.json, degrade tier-3 checks to partial, and continue.
Step 4 — Build the diagnostic snapshot
Compute the items listed in Layer 0. Write snapshot.md and prepend the same content to findings.md.
Step 5 — Resolve each check
For each check in the active layer list, walk its detection logic. Tier-dependent checks running below their required tier record partial with a gap explaining the missing tier and the flag or command needed to unlock it.
Step 6 — Write phase 1 outputs
Create .architect-audits/dependency-audit/ if needed. Write findings.md, findings.json, snapshot.md, metadata.json. Overwrite previous runs of these four; preserve implementation-plan.md unless the user agrees to regenerate it.
Step 7 — Print the concise chat summary and offer phase 2
Print a human-first, scannable summary in the chat. Do not print the full layered findings — those are written to disk in Step 6. The chat output has exactly this shape:
- Short header — audit name, timestamp, and a one-line summary of the codebase state.
- Top 5 Highest-Leverage Recommendations — ordered by architectural principles: test philosophy, maintainability, risk reduction, velocity, long-term health. For fewer than five findings, print what exists. For each recommendation (numbered 1–5):
- Title (one clear line).
- Why it matters (explain the principle in 1–2 sentences).
- Real consequences if ignored (honest downside for the team or project).
- Smallest high-leverage fix (exact next step, effort level, and which files to touch).
- At the end, add a lettered sub-list of concrete actions if useful (e.g. 2a, 2b) so the user can reply with "2b" or "1 and 3" to trigger implementation.
- Bottom line:
Full detailed audit report (layered findings, snapshot, metadata, implementation plan) → .architect-audits/dependency-audit/findings.md
When --learn or --teach is set, expand each recommendation into mid-level engineer teaching mode:
- For every item, explain as if teaching a mid-level engineer, pointing to specific files and line numbers from the current codebase.
- Use educational language: "Here's why this pattern bites teams in the long run…", "This is the exact mistake I see in most codebases at your stage…", "The fix is small but pays off huge because…".
- Include a short "What you'll learn from fixing this" section for each recommendation.
- Keep the numbered/lettered structure so the user can still reply with "2b" or "1 and 3".
- End with the same bottom-line link to the full report.
After printing, ask the single yes-or-no question: "Generate an implementation plan for the gaps identified above? (yes/no)" Do not proceed to phase 2 without an explicit affirmative.
Step 8 — Phase 2: generate the implementation plan
When the user agrees, build implementation-plan.md:
- Header — repository name, baseline version, package manager, tier, timestamp, total counts per layer.
- Layer 1 — security plan: per advisory, the upgrade or replacement command. Continuous-integration scanning step snippet when missing. Frozen-install switch for the relevant package manager when needed.
- Layer 2 — health plan: per outdated package, the upgrade target and the migration notes available from the registry. Per abandoned or deprecated package, the recommended replacement.
- Layer 3 — compliance plan: per flagged license, the package, the license type, and the recommended next step (allowlist via
.architect-playbook/licenses.json after legal review, or remove). License-checker tool installation and configuration snippet when missing.
- Layer 4 — hygiene plan: per unused package, the removal command. Per misplaced package, the move command. Per duplicate, the resolution strategy (overrides, dedupe, version alignment). Per package-manager-mixing finding, the cleanup command.
- Closing checklist — flat checkbox list mirroring the gaps, suitable for pasting into a pull-request description.
The plan is descriptive, not executable. It does not run any install, upgrade, or removal commands.
Repository Quality Score findings contract
findings.json is the scoring source and MUST use schema 2.0.0. Emit the
top-level fields schemaVersion, runIdentifier, skillName, skillVersion,
checkCatalogSchemaVersion, checkCatalogVersion, runStartedAt,
runFinishedAt, target, execution, and checks. target records the
repository, exact Git commit, and whether the audited source tree was clean.
execution records filtersApplied, filterArguments, threshold and policy
overrides, enrichmentArguments, and graph availability.
Emit every entry from checks.json exactly once and use its full checkId and
layer. Each result records applicability, evaluationState, evidenceQuality,
classification, canonical status, and evidence. A check that does not apply
is not-applicable/not-evaluated with a null status. A filtered or otherwise
unresolved applicable check is applicable/not-evaluated with a null status;
never guess a pass or failure. metadata.json repeats the common run, target,
catalog, and execution identity. The complete contract is
.agents/AUDIT_FINDINGS_CONTRACT.md in the playbook repository.
Findings file shape
findings.json (the example shows one representative check; emitted output
contains every catalog check exactly once):
{
"schemaVersion": "2.0.0",
"runIdentifier": "<uuid>",
"skillName": "dependency-audit",
"skillVersion": "1.0.0",
"checkCatalogSchemaVersion": "1.1.0",
"checkCatalogVersion": "1.0.0",
"target": { "repository": "example", "gitCommit": "<full-commit-sha>", "sourceWorkingTreeClean": true },
"execution": { "filtersApplied": false, "filterArguments": [], "thresholdOverrides": {
findings.md mirrors the same content in human-readable form, with the diagnostic snapshot at the top and one section per check, grouped by layer. snapshot.md contains only the snapshot. metadata.json carries skill identity, timestamps, Graphify revision (when present), and the configuration of the run (including withNetwork: true|false and networkCommandsExecuted: ["npm audit --json", "npm outdated --json"]).
Idempotency rules
- Re-running with no flags overwrites
findings.md, findings.json, snapshot.md, and metadata.json in place.
implementation-plan.md is preserved across runs unless the user agrees to regenerate it.- Filter flags, threshold overrides, the
--with-network flag, and the security-critical-package list are recorded in metadata.json so a partial run can be reproduced.
- Network-derived data (vulnerabilities, outdated, abandonment) is timestamp-tagged in the findings; staleness is the user's responsibility to manage by re-running.
Failure modes and remediation
| Symptom | Cause | Fix |
|---|
no package.json detected | The skill is run outside a Node.js project root. | Change directory into the project root and re-run. |
| No supported package manager | None of the recognised lockfiles is present and packageManager is unset. | Continue. Record the package-manager foundation as missing, dependent checks as applicable but not-evaluated, and evaluate manifest-only checks. List npm, pnpm, yarn, and bun in the remediation. |
node_modules absent | The user has not run install, or node_modules is in a non-standard location. | Continue at tier 1. Record tier: 1 in metadata. Tier-2 checks degrade to partial with the gap "needs tier 2 — run install first". |
--with-network set but network command fails | Registry unreachable, authentication required, audit endpoint rate-limited. | Record the failure and the command output in metadata. Tier-3 checks degrade to partial. Do not crash. |
| Multiple lockfiles present | Mid-migration between package managers, or accidental commit. | Pick the lockfile referenced by the packageManager field if set; otherwise the most recently modified. Record the conflict in the snapshot. The package-manager-mixing hygiene check will report violation. |
| Knowledge graph missing | /pre-audit-setup has not been run. | Continue. Record noGraphify: true in metadata. The unused-dependency and misplaced-dependency checks fall back to regex/AST sweep with confidence: "low". |
.architect-playbook/licenses.json is malformed | User has hand-edited the file. | Treat as if absent; record the parse error in metadata; report compliance flagged-licenses checks as partial rather than present. |
What this skill explicitly does NOT do
- Install, update, or remove any package.
- Modify
package.json, the lockfile, or node_modules/.
- Encode or enforce a license policy. Compliance findings surface signals; the human (and, where appropriate, legal counsel) makes the decision.
- Make registry calls beyond the package manager's own read-only audit and outdated commands.
- Open pull requests or commit anything to git.
- Audit non-Node.js ecosystems. Python, Go, Rust, and Ruby dependency audits are out of scope.
- Audit individual workspaces in a monorepo separately. The skill audits the repository root and reports cross-workspace duplication; per-workspace deep-dives are recommended as follow-up.
- Run any kind of supply-chain analysis beyond what the package manager's audit and the local installation surface — full provenance, signature verification, and attestations are out of scope for this version.
