fix: Resolve npm audit vulnerabilities in the TypeScript MCP server - #1949
Conversation
… failures npm audit reported 4 vulnerabilities (3 moderate, 1 high) in Packages/src/TypeScriptServer~ unrelated to any pending dependency-update PR: @hono/node-server <=2.0.4, hono 4.0.0-4.12.26, and fast-uri 3.0.0-3.1.3, all pulled in transitively via @modelcontextprotocol/sdk and pinned by the package.json overrides block. Raise the overrides to 2.0.11 / 4.12.31 and apply the resulting fast-uri lockfile update via `npm audit fix`, clearing all reported vulnerabilities without any breaking API changes (build, tests, and lint all pass unchanged).
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (3)
📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe TypeScript server package updates pinned Hono dependency overrides to newer versions. ChangesHono dependency updates
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 Checkov (3.3.8)Packages/src/TypeScriptServer~/package.jsonTraceback (most recent call last): Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
User Impact
npm auditwas reporting 4 vulnerabilities (3 moderate, 1 high) in the TypeScript MCP server's dependency tree, pulled in transitively through@modelcontextprotocol/sdkand pinned by thepackage.jsonoverrides block (@hono/node-server,hono,fast-uri).Changes
overridespins for@hono/node-server(1.19.13 → 2.0.11) andhono(4.12.25 → 4.12.31).fast-urilockfile update vianpm audit fix.Verification
npm audit --audit-level=moderate→ 0 vulnerabilitiesnpm run build→ succeedsnpm test→ 7 suites / 42 tests passednpm run lint:check→ 0 errors (pre-existing warnings only, unrelated to this change)