Skip to content

Fix new high/critical audit findings via resolution bumps - #14

Merged
torreyatcitty merged 1 commit into
mainfrom
security/fix-new-audit-findings
Jul 22, 2026
Merged

Fix new high/critical audit findings via resolution bumps#14
torreyatcitty merged 1 commit into
mainfrom
security/fix-new-audit-findings

Conversation

@torreyatcitty

Copy link
Copy Markdown
Contributor

Summary

The audit gate added in #13 is failing on main because six new advisories published after that PR went green. All are fixable by resolution bumps — no allowlist additions needed:

Resolution Before After Advisory
tar ^7.5.16 ^7.5.19 GHSA-23hp-3jrh-7fpw (critical DoS) + GHSA-8x88-c5mf-7j5w (high)
js-yaml ^4.1.1 ^4.3.0 GHSA-52cp-r559-cp3m (high, quadratic CPU)
brace-expansion ^2.0.2 ^2.1.2 GHSA-3jxr-9vmj-r5cp (high, exponential expansion)
fast-uri (new) ^3.1.4 GHSA-v2hh-gcrm-f6hx + GHSA-4c8g-83qw-93j6 (high, host confusion)

Low risk: all four packages were already force-deduped to a single version by the existing resolutions block (e.g. every tar@^4/^6/^7 range already resolves to 7.5.x), so these bumps move versions the build already runs, within the same major.

Verification

  • node scripts/audit-check.js exits 0 (the two known allowlisted advisories still print with justifications)
  • Full production yarn build passes
  • Dependency Audit workflow green on this PR

🤖 Generated with Claude Code

New advisories published after the audit gate merged in #13. All four
packages were already force-deduped to a single version by the
existing resolutions block, so these are same-major bumps of what the
build already runs:

- tar ^7.5.16 -> ^7.5.19 (GHSA-23hp-3jrh-7fpw critical DoS,
  GHSA-8x88-c5mf-7j5w high infinite loop)
- js-yaml ^4.1.1 -> ^4.3.0 (GHSA-52cp-r559-cp3m high quadratic CPU)
- brace-expansion ^2.0.2 -> ^2.1.2 (GHSA-3jxr-9vmj-r5cp high
  exponential expansion)
- fast-uri ^3.1.4 added (GHSA-v2hh-gcrm-f6hx, GHSA-4c8g-83qw-93j6
  high host confusion)

Audit gate exits 0 again; production build verified passing.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@torreyatcitty
torreyatcitty merged commit bebe675 into main Jul 22, 2026
2 checks passed
@torreyatcitty
torreyatcitty deleted the security/fix-new-audit-findings branch July 22, 2026 20:15
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