Skip to content

chore: clear npm audit advisories in the dev dependency tree - #17

Merged
cuzzlor merged 1 commit into
mainfrom
chore/npm-audit
Aug 16, 2026
Merged

cuzzlor merged 1 commit into
mainfrom
chore/npm-audit

Conversation

@cuzzlor

@cuzzlor cuzzlor commented Aug 15, 2026 •

Copy link
Copy Markdown
Collaborator

Stack 1/3 — this PR is the base of the stack #17 → #15 → #16. Merge this first; GitHub will retarget #15 to main automatically when this branch is deleted on merge.

Problem

CI's audit step (better-npm-audit) fails on main with 24 advisories accumulated in the devDependency tree since the last release (vite, shell-quote, qs, fast-uri, launch-editor, …). This blocks every PR — #15 and #16 are stacked on top of this fix.

Fix

  • npm audit fix (lockfile-only, within existing semver ranges) resolves 23 of 24. No package.json ranges change and nothing shipped to consumers changes.
  • The remaining low advisory (GHSA-g7r4-m6w7-qqqr: esbuild 0.27.3–0.28.0 dev-server arbitrary file read on Windows) has no in-range fix reachable via tsx/vite yet, so it gets an .nsprc exception with notes, following the repo's existing pattern — esbuild is used here only as a build/test transform, never as a dev server.
  • Removed the stale uuid exception (better-npm-audit reports it no longer matches anything after the update).

Testing

npm run audit ("All good!"), npm run build (lint, check-types, rollup, attw all green), and npm test pass locally.

🤖 Generated with Claude Code

`npm audit fix` resolves 23 of the 24 advisories accumulated since the
last release (vite, shell-quote, qs, fast-uri, launch-editor, etc. —
all transitive devDependencies; nothing shipped changes).

The remaining low-severity advisory (GHSA-g7r4-m6w7-qqqr, esbuild
0.27.3–0.28.0 dev-server file read on Windows) has no in-range fix via
tsx/vite yet, so it gets an .nsprc exception with removal notes —
esbuild is only used here as a build/test transform, never as a dev
server. The stale uuid exception (fixed by the update) is removed.

Unblocks CI for #15 and #16, which fail on the audit step for
advisories predating those branches.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@cuzzlor
cuzzlor merged commit b30a47e into main Aug 16, 2026
1 check passed
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.

2 participants