Skip to content

fix(deps): resolve outstanding npm audit advisories - #604

Merged
lourens-octopus merged 1 commit into
mainfrom
lourens/fix-npm-audit
Jul 27, 2026
Merged

fix(deps): resolve outstanding npm audit advisories#604
lourens-octopus merged 1 commit into
mainfrom
lourens/fix-npm-audit

Conversation

@lourens-octopus

Copy link
Copy Markdown
Contributor

What

This action was missed by the earlier dependency sweeps. Brings it to 0 vulnerabilities.

  • brace-expansion — a new advisory (GHSA-mh99-v99m-4gvg) flags every version ≤5.0.7, including the 1.1.16/2.1.2 that the previous fix landed on. Only 5.0.8 is patched, and there are no fixed 1.x/2.x releases — so I added an overrides entry forcing brace-expansion@^5.0.8 across the whole tree. Its expand() API is unchanged across majors, so minimatch/glob keep working (build + unit tests pass). The glob copy is bundled, so dist is rebuilt.
  • jest-junit → 17.0.0 (also clears its transitive uuid advisory)
  • js-yaml → 5.2.2

Note

The brace-expansion override approach (and the js-yaml 5.2.2 bump) likely apply to the rest of the fleet too — these are new advisories that supersede the earlier fixes. Flagging separately.

🤖 Generated with Claude Code

This action was missed by the earlier dependency sweeps. Clears all
remaining advisories:
- brace-expansion: new advisory GHSA-mh99-v99m-4gvg flags every version
  <=5.0.7 (including 1.1.16/2.1.2), and only 5.0.8 is patched. Added an
  overrides entry forcing brace-expansion ^5.0.8 across the tree; its
  expand() API is unchanged so minimatch/glob still work (build + tests
  pass). The bundled glob copy means dist is rebuilt.
- jest-junit -> 17.0.0 (clears its transitive uuid advisory)
- js-yaml -> 5.2.2

npm audit now reports 0 vulnerabilities.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@lourens-octopus
lourens-octopus requested a review from a team as a code owner July 26, 2026 22:47
Comment thread package.json
"version": "4.0.3"
"version": "4.0.3",
"overrides": {
"brace-expansion": "^5.0.8"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overrides always make me think twice. I appreciate you noting in the PR description that the current API surface for what we use is unchanged, but there could be future changes that do. I guess we rely on tests to break...

@lourens-octopus
lourens-octopus merged commit 295a4ee into main Jul 27, 2026
7 checks passed
@lourens-octopus
lourens-octopus deleted the lourens/fix-npm-audit branch July 27, 2026 19:40
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