Skip to content

fix(security): bump High-severity advisory dependencies - #535

Merged
gaurav-singh-9227 merged 2 commits into
mainfrom
security/bump-high-advisory-deps
Oct 30, 2026
Merged

gaurav-singh-9227 merged 2 commits into
mainfrom
security/bump-high-advisory-deps

Conversation

@SavioBS629

@SavioBS629 SavioBS629 commented Oct 30, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Bumps dependencies to resolve High-severity advisories reported by npm audit. All fixes were applied via in-range npm update or minimal non-breaking direct-dependency range bumps. No npm overrides were added and no breaking major bumps were performed.

Included (fixed)

Package Old → New Advisory
sharp 0.35.4 → 0.35.5 GHSA-wq5f-xc86-pv6w / CVE-2026-96889 (librsvg)
zod 4.4.3 → 4.6.7 GHSA-mrhx-c7xc-m3w5 (memory exhaustion)
vite (dev) 7.3.6 → 7.3.7 GHSA-587h-7wmj-h5pm (launch-editor fs.existsSync bypass)
body-parser (transitive) 2.2.2 → 2.4.0 GHSA-27r3-jxmh-p828 (DoS via trailing bytes)
source-map-js (transitive) 1.2.1 → 1.2.2 GHSA-68fv-2mgg-jv7q (event-loop DoS)
follow-redirects (transitive) 1.16.0 → 1.16.2 GHSA-8r9p-f939-6h3c (header leak)

Already satisfied (no change needed)

Deferred

  • @modelcontextprotocol/sdk (GHSA-6qxp-vccf-f47h): MCP SDK bump pulled out — needs a dedicated OAuth/transport test pass. Will be handled in a separate PR.

Not included (excluded)

Package Advisory Reason
extract-zip GHSA-jmr9-qjv8-65gv (symlink traversal) Only fixable via a breaking major bump of webdriverio (9 → 10, isSemVerMajor). Not in-range; excluded to avoid a breaking change.
basic-ftp GHSA-c475-qrg2-pj4r (quadratic ReDoS) Same: only reachable via the webdriverio 9 → 10 major bump (or a forbidden override). Excluded.

Validation

  • npx tsc passes clean; eslint passes clean.
  • Test suite: the only failing tests are pre-existing fake-timer/polling timeouts that fail identically on a pristine main checkout (same 27 tests), so they are unrelated to these bumps.
  • npm audit after the change: all included target advisories clear; only the two excluded webdriverio-chain advisories remain.

🤖 Generated with Claude Code

Bump dependencies to resolve High-severity advisories reported by npm audit.
Fixes applied via in-range npm update / minimal direct-dependency range bumps
only (no npm overrides, no breaking major bumps).

Direct dependencies:
- @modelcontextprotocol/sdk 1.29.0 -> 1.32.2 (GHSA-6qxp-vccf-f47h)
- sharp 0.35.4 -> 0.35.5 (GHSA-wq5f-xc86-pv6w / CVE-2026-96889, librsvg)
- zod 4.4.3 -> 4.6.7 (GHSA-mrhx-c7xc-m3w5)
- vite 7.3.6 -> 7.3.7 (GHSA-587h-7wmj-h5pm, launch-editor; dev dependency)

Transitive dependencies (lockfile):
- body-parser 2.2.2 -> 2.4.0 (GHSA-27r3-jxmh-p828)
- source-map-js 1.2.1 -> 1.2.2 (GHSA-68fv-2mgg-jv7q)
- follow-redirects 1.16.0 -> 1.16.2 (GHSA-8r9p-f939-6h3c)

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Central YAML (base), Organization UI (inherited), Workspace UI (inherited); Enforced overrides: Organization
  • Review profile: ASSERTIVE
  • Plan: Enterprise
  • Run ID: 2b5606f2-4d20-48c4-ae5b-a9054d31296f

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

Revert @modelcontextprotocol/sdk back to the default-branch version (1.29.0).
The SDK bump (GHSA-6qxp-vccf-f47h) needs a dedicated OAuth/transport test pass
and will be handled separately. All other advisory bumps are kept unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@gaurav-singh-9227
gaurav-singh-9227 merged commit f969678 into main Oct 30, 2026
6 checks 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.

2 participants