fix: harden pnpm audit overrides for new advisories - #851
Merged
Conversation
herbert-julio-azion
approved these changes
Aug 4, 2026
guilherme-santana-azion
approved these changes
Aug 4, 2026
The Security Scans / "Audit dependencies" step (pnpm audit) fails on three transitive advisories that the current overrides no longer cover: - brace-expansion — GHSA-rgw5-rvv9-x895 (DoS, CVE-2026-14257 mitigation bypass): bump the existing override from >=5.0.8 to >=5.0.9. - fast-uri — GHSA-7p8r-x3mc-p8w7 (host confusion via backslash): add >=3.1.5 <4 (bounded to 3.x, ajv pins ^3). - hono — GHSA-8j4g-w8fx-2239 (ReDoS in CORS middleware, via @modelcontextprotocol/sdk): add >=4.12.34 <5 (bounded to 4.x). pnpm audit now exits clean (no known vulnerabilities). Bounds keep every force within the dependents' expected major.
isaque-bock-azion
force-pushed
the
fix/harden-audit-overrides
branch
from
August 4, 2026 14:04
0eb5b4c to
2265c52
Compare
guilherme-santana-azion
approved these changes
Aug 4, 2026
herbert-julio-azion
approved these changes
Aug 4, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The Security Scans → "Audit dependencies" step (
pnpm run security:audit=pnpm audit) ingovernance.ymlstarted failing on three transitive advisories that the existingpnpm-workspace.yamloverrides no longer cover. Example failing run: runs/30902738667 · Security Scans (exit 1,1 moderate | 2 high).This is a repo-wide gate (it fails on every open PR, e.g. #850), so the fix lands in its own PR off
mainper the dependency-change convention; merging it unblocks the audit step everywhere.The three advisories
brace-expansion@typescript-eslint/*,eslint, … →minimatch→brace-expansion>=5.0.8→>=5.0.9fast-uri@commitlint/*,stylelint→ajv→fast-uri>=3.1.5 <4hono@modelcontextprotocol/sdk→ (@hono/node-server→)hono>=4.12.34 <5Why these bounds
Each force stays inside the dependents' expected major so nothing else in the tree breaks:
fast-uribounded to<4—ajvpins^3.honobounded to<5— the MCP SDK pins^4.brace-expansionstays in major 5, kept there by the existingminimatch: ">=9.0.0"override.The
brace-expansioncomment now references the superseding advisory (GHSA-rgw5-rvv9-x895) instead of the older GHSA-mh99-v99m-4gvg it already patched.Verification
pnpm install --lockfile-onlyresolvesbrace-expansion@5.0.9,fast-uri@3.1.5,hono@4.12.34.pnpm audit→No known vulnerabilities found(exit 0), down from1 moderate | 2 high.pnpm-workspace.yaml(4 lines) andpnpm-lock.yaml.Release effect:
fix→ patch.