Skip to content

Fix remaining npm audit vulnerabilities via overrides - #1773

Open
TwitchBronBron wants to merge 1 commit into
masterfrom
fix/npm-audit-vulnerabilities
Open

Fix remaining npm audit vulnerabilities via overrides#1773
TwitchBronBron wants to merge 1 commit into
masterfrom
fix/npm-audit-vulnerabilities

Conversation

@TwitchBronBron

Copy link
Copy Markdown
Member

Summary

  • After the recent `qs` bump (Bump qs from 6.14.2 to 6.15.3 #1766), `npm audit` still reported 7 vulnerabilities: brace-expansion, js-yaml, fast-uri (all high), @babel/core (low), and the already-allowlisted uuid/istanbul-lib-processinfo/nyc chain (moderate)
  • All are pulled in exclusively through devDependencies (eslint, mocha, nyc, ajv) — none touch the published package
  • No non-major version resolves brace-expansion/js-yaml/@babel/core/fast-uri through normal resolution, so pins them via the existing `overrides` block (same pattern already used for serialize-javascript)
  • The remaining moderate uuid finding stays allowlisted in audit-ci.jsonc (unreachable v3/v5/v6 buffer-bounds path — both consumers only call v4())

Test plan

  • `npm run audit` passes
  • `npm run build` passes
  • `npm run lint` passes
  • `npm run test` passes (2979 passing)

🤖 Generated with Claude Code

brace-expansion, js-yaml, @babel/core, and fast-uri are pulled in
exclusively through eslint/mocha/nyc/ajv (dev tooling), never
shipped. No non-major version resolves them through normal
resolution, so pin patched versions via overrides, same pattern
already used for serialize-javascript.

The remaining moderate uuid/istanbul-lib-processinfo/nyc chain stays
allowlisted in audit-ci.jsonc (unreachable v3/v5/v6 buffer-bounds
path; both consumers only call v4()).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@TwitchBronBron TwitchBronBron added the Security enhancements Security-related dependency/vulnerability fixes label Aug 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Security enhancements Security-related dependency/vulnerability fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant