Skip to content

build(deps): raise brace-expansion and nanoid past their advisories - #981

Merged
samwiseg0 merged 1 commit into
devfrom
samwiseg0/npm-audit-fix
Aug 31, 2026
Merged

build(deps): raise brace-expansion and nanoid past their advisories#981
samwiseg0 merged 1 commit into
devfrom
samwiseg0/npm-audit-fix

Conversation

@samwiseg0

Copy link
Copy Markdown
Contributor

npm audit reported a high-severity denial of service in two transitive dev dependencies of
the frontend. npm audit fix --package-lock-only resolves both, and the change is confined to
the lockfile.

Neither package reaches the shipped bundle. npm audit --omit=dev was already clean, so what
was exposed is a lint or build step reading our own source.

Dependabot had not proposed these on its own because its version updates sit behind a 14-day
cooldown. A security update is exempt from that cooldown, but neither of these arrived as one.

website/package-lock.json carries 18 more high-severity findings, all of them the same
image-size parser reached through Docusaurus. Every published version is affected and no
patched release exists, so that one waits on upstream. Left as a note on #980 rather than
folded in here.

Closes #980.

Tested: npm audit now reports zero, plus the frontend's lint, format check, test
(1661 passed) and build, and pytest tests/test_repo_hygiene.py for the lockfile guards.

Co-Authored-By: Claude Opus 5 (1M context) noreply@anthropic.com

Both are transitive dev dependencies, and npm audit reported a high-severity
denial of service in each. Neither reaches the shipped bundle, so the exposure
was a lint or build step. brace-expansion moves 5.0.7 to 5.0.9 under eslint's
minimatch, and nanoid moves 3.3.16 to 3.3.18 under vite's postcss.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@samwiseg0 samwiseg0 added Kind/Security This is security issue Priority/Low The priority is low labels Aug 31, 2026
@codecov

codecov Bot commented Aug 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@samwiseg0
samwiseg0 merged commit 3ec4cfe into dev Aug 31, 2026
15 checks passed
@samwiseg0
samwiseg0 deleted the samwiseg0/npm-audit-fix branch August 31, 2026 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Kind/Security This is security issue Priority/Low The priority is low

Projects

None yet

Development

Successfully merging this pull request may close these issues.

npm audit reports two high-severity advisories in dev-only frontend dependencies

1 participant