Skip to content

(janitor/dependencies): patch body-parser and qs vulnerabilities - #8990

Open
kilo-code-bot[bot] wants to merge 1 commit into
mainfrom
janitor/dependencies/dependabot-body-parser-qs
Open

kilo-code-bot[bot] wants to merge 1 commit into
mainfrom
janitor/dependencies/dependabot-body-parser-qs

Conversation

@kilo-code-bot

@kilo-code-bot kilo-code-bot Bot commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

Changes by use

One PR: both overrides share pnpm-workspace.yaml and the generated lockfile.

  • Express request parsing (MCP SDK, OpenClaw, web-ext tooling): body-parser override 2.3.0 → 2.4.0 (published 2026-10-22, now past the unchanged 6842-minute release-age window). Same major as Express 5; node >=18. Pulls content-type 2.1.0 and iconv-lite 0.7.3 (declared by body-parser 2.4.0).
  • Query-string parsing (express, body-parser, googleapis-common, url): qs override 6.16.0 → 6.16.1 (published 2026-10-14, eligible). 6.16.2 (2026-10-27) is still inside the release-age window and was not used. Patch bump, node >=0.6; satisfies body-parser 2.4.0's ^6.16.1.

Lockfile regenerated with pinned pnpm 11.1.2 (pnpm install --lockfile-only); no hand edits. Release-age policy, exclusions, patches and other pins unchanged.

Alerts addressed

Open alerts inspected, not fixed here

Validation (Node 24, pnpm 11.1.2; PostgreSQL/Redis not started)

  • pnpm install --lockfile-only — PASS; diff limited to body-parser, qs, content-type, iconv-lite entries.
  • pnpm install --frozen-lockfile — PASS.
  • pnpm why -r qs body-parser — only qs 6.16.1 and body-parser 2.4.0 remain.
  • pnpm format:changed, git diff --check — clean.
  • Not run: typecheck/tests (no source changes; transitive patch/minor bumps only).

@kilo-code-bot kilo-code-bot Bot added the janitor Automated dead-code/duplication cleanup label Oct 28, 2026
@kilo-code-bot
kilo-code-bot Bot enabled auto-merge October 28, 2026 05:08
@kilo-code-bot

kilo-code-bot Bot commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor Author

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

The PR updates two reviewed overrides (qs 6.16.0 → 6.16.1, body-parser 2.3.0 → 2.4.0) in pnpm-workspace.yaml and regenerates pnpm-lock.yaml; the lockfile is internally consistent with no stale qs@6.16.0 or body-parser@2.3.0 entries, and every dependent snapshot (express@5.2.1, googleapis-common, url@0.11.4) points at the new versions.

Assumptions: [INFERENCE] Publish dates for the two newly introduced transitive packages, content-type@2.1.0 and iconv-lite@0.7.3, were not stated in the PR body and could not be verified offline; the reported successful pnpm install --lockfile-only under minimumReleaseAge: 6842 is the only evidence they are past the release-age window. [INFERENCE] Actual vulnerability resolution was not independently verified; the PR relies on the Dependabot fixed versions (body-parser 2.4.0, qs 6.16.1), and no typecheck/test run was claimed since only dependency pins change.

Files Reviewed (2 files)
  • pnpm-workspace.yaml
  • pnpm-lock.yaml

Reviewed by deepseek-flash-latest · Input: 58.1K · Output: 4.9K · Cached: 314.9K

Review guidance: REVIEW.md from base branch main

This branch has not been deployed

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

Labels

janitor Automated dead-code/duplication cleanup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants