Skip to content

fix(deps): resolve npm audit vulnerabilities#26

Closed
ben-miru wants to merge 1 commit intomainfrom
fix/audit-vulnerabilities
Closed

fix(deps): resolve npm audit vulnerabilities#26
ben-miru wants to merge 1 commit intomainfrom
fix/audit-vulnerabilities

Conversation

@ben-miru
Copy link
Copy Markdown
Contributor

Summary

  • Add npm overrides to force patched transitive dependency versions
  • brace-expansion >=5.0.5 — fixes ReDoS via extglob quantifiers (GHSA-f886-m6hf-6m8v)
  • handlebars >=4.7.9 — fixes prototype pollution leading to XSS (GHSA-2qvq-rjwj-gvw9)
  • minimatch >=10.0.0 scoped to prettier-eslint — fixes multiple ReDoS vulnerabilities (GHSA-3ppc-4f35-3m26, GHSA-7r86-cg39-jmmj, GHSA-23c5-xmqv-rm74)
  • Resolves all 38 npm audit findings (34 moderate, 4 high) → 0 vulnerabilities
  • All overrides target dev dependencies only — no impact on the bundled GitHub Action

Test plan

  • npm audit returns 0 vulnerabilities
  • npm run lint passes
  • npm run ci-test — all 30 tests pass
  • npm run package — bundle builds successfully

🤖 Generated with Claude Code

Add npm overrides to force patched versions of transitive
dependencies with known vulnerabilities:
- brace-expansion >=5.0.5 (ReDoS via extglob quantifiers)
- handlebars >=4.7.9 (prototype pollution leading to XSS)
- minimatch >=10.0.0 scoped to prettier-eslint (multiple ReDoS)

Resolves all 38 npm audit findings (34 moderate, 4 high).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@ben-miru ben-miru force-pushed the fix/audit-vulnerabilities branch from d5f5041 to 9bd4f2a Compare March 27, 2026 02:15
@ben-miru
Copy link
Copy Markdown
Contributor Author

There actually were no audit issues with the repository, I ill-prompted it with some failed CI checks from docs repository so we can just ignore this.

@ben-miru ben-miru closed this Mar 27, 2026
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