Skip to content

Bump brace-expansion and nanoid via npm audit fix - #83

Merged
bmichelsen merged 1 commit into
mainfrom
npm-audit-fix
Aug 15, 2026
Merged

Bump brace-expansion and nanoid via npm audit fix#83
bmichelsen merged 1 commit into
mainfrom
npm-audit-fix

Conversation

@bmichelsen

@bmichelsen bmichelsen commented Aug 15, 2026

Copy link
Copy Markdown
Member

What does this PR do?

Lockfile-only: npm audit fix bumps two dev-only transitive dependencies flagged as high severity - brace-expansion (DoS via crafted globs, under eslint) and nanoid (zero-size generator loop, under vitest). Neither ships; the built extension contains only src/ and tom-select.

The remaining audit findings are all one root cause: image-size under addons-linter/web-ext. Deliberately deferred - npm audit fix --force would downgrade web-ext from 10 to 5.5.0 to dodge a dev-only DoS. Dependabot will pick up the real fix when addons-linter bumps image-size.

Checklist

  • npm test passes (56)
  • npm run lint passes
  • npm run build succeeds
  • No runtime dependency changes

🤖 Generated with Claude Code

Clears the fixable audit findings, all dev-only transitives (eslint's
glob expansion, vitest's id generator). The remaining image-size chain
under addons-linter/web-ext is deliberately left: the only offered fix
downgrades web-ext from 10 to 5.5, so it waits for addons-linter to
bump image-size upstream.
@bmichelsen
bmichelsen merged commit 9143713 into main Aug 15, 2026
3 checks passed
@bmichelsen
bmichelsen deleted the npm-audit-fix branch August 15, 2026 16:51
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.

1 participant