Skip to content

fix: resolve security vulnerabilities in dependencies#47

Merged
freema merged 1 commit intomainfrom
fix/security-audit-fixes
Mar 28, 2026
Merged

fix: resolve security vulnerabilities in dependencies#47
freema merged 1 commit intomainfrom
fix/security-audit-fixes

Conversation

@freema
Copy link
Copy Markdown
Collaborator

@freema freema commented Mar 28, 2026

Summary

  • Ran npm audit fix to resolve 11 of 17 reported vulnerabilities
  • All runtime vulnerabilities are fixed (path-to-regexp, undici, hono, @modelcontextprotocol/sdk, qs)
  • Dev-only vulnerabilities fixed: rollup, picomatch, flatted, ajv, brace-expansion

Remaining (dev-only, non-breaking fix not available)

  • minimatch (6 high) — requires breaking upgrade of @typescript-eslint/parser v6 → v8. Tracked separately, does not affect published package.

Vulnerability details

Package Severity Scope Issue
@modelcontextprotocol/sdk high runtime cross-client data leak
path-to-regexp high runtime ReDoS via sequential optional groups
undici high runtime WebSocket overflow, request smuggling
hono high runtime XSS, cache deception, prototype pollution
rollup high dev arbitrary file write via path traversal
picomatch high dev method injection, ReDoS
flatted high dev DoS, prototype pollution
qs, ajv, brace-expansion moderate mixed ReDoS, DoS

Test plan

  • npm run typecheck — pass
  • npm run lint — pass
  • npm run build — pass
  • npm run test:unit — 380/380 tests pass

cc @juliandescottes @padenot for review

Updates transitive dependencies to fix 11 vulnerabilities:
- @modelcontextprotocol/sdk 1.25.3 → 1.28.0 (fixes cross-client data leak)
- path-to-regexp 8.3.0 → 8.4.0 (fixes ReDoS)
- undici → 7.24.6 (fixes WebSocket and smuggling issues)
- hono → 4.12.9 (fixes XSS, cache deception, prototype pollution)
- rollup → 4.60.0 (fixes path traversal)
- picomatch → 2.3.2/4.0.4 (fixes method injection)
- flatted → 3.4.2 (fixes DoS and prototype pollution)
- qs, ajv, brace-expansion (moderate fixes)

Remaining: minimatch (dev-only, requires breaking @typescript-eslint upgrade)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@freema freema merged commit 66a8240 into main Mar 28, 2026
1 check 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