Skip to content

Clear the two advisories that turned the audit gate red - #181

Merged
rouges78 merged 1 commit into
mainfrom
audit-gate-sharp-jsyaml
Sep 9, 2026
Merged

rouges78 merged 1 commit into
mainfrom
audit-gate-sharp-jsyaml

Conversation

@rouges78

@rouges78 rouges78 commented Sep 9, 2026

Copy link
Copy Markdown
Owner

Merging #177 took the critical Next.js RCE pair and the four fast-uri advisories off the board, and two advisories published since took their place. The gate is red on main and therefore on every open PR — it runs per-PR, which is why the documentation-only #179 also shows npm audit + SBOM failing.

Both fixes are patch-level:

Advisory Package Scope Fix
GHSA-rgj7-g3m4-5g8c (libheif) sharp <0.35.4 PROD devDependency and existing override to ^0.35.4
GHSA-2883-xcg3-v3hh (maxTotalMergeKeys does not limit CPU on empty merge sources) js-yaml >=4.0.0 <4.3.2 dev only, via eslint@8.57.0 override to ^4.3.2

next 15.5.25 wants the newer sharp anyway: it re-enables AVIF image optimization only when a sharp new enough to carry the libheif fix is installed. The js-yaml override stays inside the same major, so eslint keeps the API it expects.

Lockfile refreshed with npm install --package-lock-only. Locally node scripts/audit-gate.js --level=high now passes, with one active exception: the dated deepmerge-ts one that expires 30/09.

🤖 Generated with Claude Code

Merging #177 took the critical Next.js pair and the four fast-uri advisories
off the board, and two published since took their place, so the gate stayed
red on main and on every open PR — it runs per-PR, which is why a
documentation-only branch also shows it failing.

Both fixes are patch-level:

- sharp <0.35.4 (GHSA-rgj7-g3m4-5g8c, libheif) — the devDependency and the
  existing override both moved to ^0.35.4. next 15.5.25 wants that version
  anyway: it re-enables AVIF image optimization only when a sharp new enough
  to carry the libheif fix is installed.
- js-yaml >=4.0.0 <4.3.2 (GHSA-2883-xcg3-v3hh, maxTotalMergeKeys does not
  limit CPU on empty merge sources) — dev-only, reached through eslint 8.57.0,
  which pins the vulnerable range. An override to ^4.3.2 is a patch bump
  inside the same major, so eslint keeps the API it expects.

Lockfile refreshed with --package-lock-only. `node scripts/audit-gate.js
--level=high` now passes with one active exception, the dated deepmerge-ts one
that expires 30/09.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@rouges78
rouges78 merged commit 828005a into main Sep 9, 2026
7 checks 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.

1 participant