Skip to content

chore(security): bump mcp sdk, brace-expansion, body-parser (HIGH Dependabot) - #28

Closed
cdbartholomew wants to merge 1 commit into
mainfrom
chore/sec-high-npm-mcp-server
Closed

cdbartholomew wants to merge 1 commit into
mainfrom
chore/sec-high-npm-mcp-server

Conversation

@cdbartholomew

Copy link
Copy Markdown
Contributor

Human-authored bundle PR to clear open HIGH Dependabot alerts in package-lock.json (npm). Dependabot's own branches don't receive repo secrets, so this re-creates the bumps on a normal branch for full CI signal.

Base branch is main (this repo has no develop).

Alerts closed

Alert Package Change
#78, #86, #90 @modelcontextprotocol/sdk 1.29.0 → 1.32.2 (direct dep floor ^1.29.0 → ^1.32.0)
#82, #83 brace-expansion (1.x) 1.1.18 → 1.1.20 (override brace-expansion@^1.1.0)
#93 body-parser 2.3.0 → 2.4.0 (override; pulls iconv-lite 0.7.2 → 0.7.3)

Verification

  • Lockfile regenerated with npm@11 install --package-lock-only (no --force, no --legacy-peer-deps).
  • git diff scoped: 7 version bumps (3 targets + 4× brace-expansion 1.x instances) + iconv-lite 0.7.3 (required body-parser 2.4.0 sub-dep). No packages added/removed.
  • npm ci --dry-run resolves cleanly.
  • brace-expansion 2.x (override 2.1.4) left untouched — not in the HIGH set for this repo.

Do not merge until CI is green and reviewed.

https://claude.ai/code/session_014J7s5yzoVEqoZ8LmnD8bQd

…endabot)

Addresses open HIGH Dependabot alerts in package-lock.json:
- @modelcontextprotocol/sdk 1.29.0 -> 1.32.2 (alerts #78, #86, #90)
- brace-expansion (1.x line) 1.1.18 -> 1.1.20 (alerts #82, #83)
- body-parser 2.3.0 -> 2.4.0 (alert #93; pulls iconv-lite 0.7.2 -> 0.7.3)

Scoped change: @modelcontextprotocol/sdk floor raised in dependencies;
brace-expansion@^1.1.0 and body-parser overrides bumped. Lockfile
regenerated with npm 11 --package-lock-only; npm ci resolves clean.

Claude-Session: https://claude.ai/code/session_014J7s5yzoVEqoZ8LmnD8bQd
@cdbartholomew cdbartholomew added p1 High severity - remediate within 30 days security labels Oct 31, 2026
@cdbartholomew cdbartholomew self-assigned this Oct 31, 2026
@cdbartholomew

Copy link
Copy Markdown
Contributor Author

Superseded by #26, which covers these HIGH alerts plus the mediums/lows in the same root package-lock.json. Closing in favor of #26 to avoid a lockfile conflict.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

p1 High severity - remediate within 30 days security

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant