Skip to content

chore: reduce high severity yarn audit findings#347

Open
forge0x wants to merge 1 commit into
mainfrom
fix/security-audit
Open

chore: reduce high severity yarn audit findings#347
forge0x wants to merge 1 commit into
mainfrom
fix/security-audit

Conversation

@forge0x
Copy link
Copy Markdown
Contributor

@forge0x forge0x commented May 11, 2026

Summary

  • Updates direct security-sensitive dependencies to patched versions (axios, express, body-parser, cookie, jsonwebtoken, mongoose, @google-cloud/storage, @grpc/proto-loader, etc.)
  • Adds Yarn resolutions for vulnerable transitive packages that are still pulled in by older tooling/runtime dependencies
  • Keeps the resolution set compatible with install/build tooling after the chore: fix main TypeScript baseline #346 typecheck cleanup

Audit impact

Before:

  • 500 total advisories
  • 18 critical, 238 high, 195 moderate, 49 low

After:

  • 47 total advisories
  • 0 critical, 0 high, 28 moderate, 19 low
  • yarn audit --level high emits no high/critical advisories

Verification

Rebased on current main after #346.

  • yarn install --frozen-lockfile
  • yarn build
  • yarn tsc-check-noimplicitany
  • yarn tsc-check
  • npx prettier --check package.json
  • yarn audit --level high ✅ no high/critical advisories

Notes

  • yarn audit --level high exits non-zero because Yarn v1 reports remaining low/moderate advisories in the summary, but the JSON audit stream contains no high/critical auditAdvisory entries.
  • This PR intentionally uses Yarn resolutions for some transitive fixes because several vulnerable packages are buried under older dependency chains like ln-service, spectaql, api/oas, and older toolchain packages.

@forge0x forge0x force-pushed the fix/security-audit branch 2 times, most recently from 79dbda2 to 1fae4b8 Compare May 11, 2026 14:36
@forge0x forge0x force-pushed the fix/security-audit branch from 1fae4b8 to e1c7e1d Compare May 11, 2026 15:46
@forge0x
Copy link
Copy Markdown
Contributor Author

forge0x commented May 11, 2026

Rebased onto current main after #346 and force-pushed e1c7e1d34.

Verification after rebase:

  • yarn install --frozen-lockfile
  • yarn build
  • yarn tsc-check-noimplicitany
  • yarn tsc-check
  • npx prettier --check package.json
  • yarn audit --level high ✅ no high/critical advisories in JSON audit stream

GitHub reports the branch as mergeable now; current blocked state appears to be review/check gating rather than conflicts.

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