Skip to content

chore(deps): npm audit fix for transitive security advisories#1380

Open
olaservo wants to merge 1 commit into
modelcontextprotocol:mainfrom
olaservo:fix/dependabot-audit-fixes
Open

chore(deps): npm audit fix for transitive security advisories#1380
olaservo wants to merge 1 commit into
modelcontextprotocol:mainfrom
olaservo:fix/dependabot-audit-fixes

Conversation

@olaservo
Copy link
Copy Markdown
Member

Summary

Resolves the bulk of the open Dependabot alerts via a lockfile-only npm audit fixno breaking major version bumps.

Fixes 15 of 16 local npm audit findings (1 critical, 5 high, 8 moderate, 1 low):

Severity Packages
🔴 critical handlebars → 4.7.9 (JS injection / prototype pollution)
🟠 high fast-uri, flatted, hono, path-to-regexp, rollup, vite, express-rate-limit, @hono/node-server
🟡 moderate ajv, brace-expansion, ip-address, picomatch, postcss, qs, ws, yaml
🟢 low @tootallnate/once

Remaining (not fixed here)

One advisory remains: minimatch@3.1.2 (high, ReDoS), pulled in transitively by @eslint/config-array — a dev/lint-only dependency. It is never shipped (the published package contains only build outputs) and never runs on untrusted input, so real-world exploitability is negligible. Forcing it via overrides required a full lockfile regen that bloated the diff to ~16k lines, so it was left out. It will clear automatically once ESLint bumps @eslint/config-array to minimatch 9+.

Verification

  • npm run build — server, client, and cli all build clean
  • ✅ Tests — 515/515 pass (one full-suite flake in App.samplingNavigation is pre-existing seed-dependent timing, unrelated to these bumps; passes in isolation)

Changes

package-lock.json only (+369 / −796).

🤖 Generated with Claude Code

Resolves 15 of 16 Dependabot alerts via lockfile-only updates (no
breaking major bumps): handlebars (critical), fast-uri, flatted, hono,
path-to-regexp, rollup, vite, express-rate-limit, @hono/node-server,
ajv, brace-expansion, ip-address, picomatch, postcss, qs, ws, yaml,
and @tootallnate/once.

The remaining minimatch@3.1.2 advisory comes from @eslint/config-array
(dev/lint-only, not shipped) and will clear when ESLint bumps it.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@olaservo olaservo marked this pull request as ready for review May 30, 2026 17:19
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