SRE-808: upgrade vulnerable npm dependencies (2026-07-13 weekly scan)#332
Draft
claude[bot] wants to merge 1 commit into
Draft
SRE-808: upgrade vulnerable npm dependencies (2026-07-13 weekly scan)#332claude[bot] wants to merge 1 commit into
claude[bot] wants to merge 1 commit into
Conversation
Re-resolve lockfile for vite (7.3.6 + nested 6.4.3), form-data 4.0.6, dompurify 3.4.12, mermaid 11.16.0, js-yaml 4.3.0, brace-expansion 5.0.7, qs 6.15.3, @babel/core 7.29.7, and release-it 20.2.1 (lifts its exact undici pin to 7.28.0). Bump chrome-devtools-mcp ^0.21.0 -> ^1.1.0 (dev-only tool, no in-repo consumers, identical bin/engines surface).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Requested by Tim Diekmann · Slack thread
Fixes SRE-808. Clears all 29 findings from the 2026-07-13 weekly OSV scan of
package-lock.jsonby bumping 10 flagged packages (11 locked copies) to at-or-above their fixed versions. Nine packages were resolved with a plain lockfile re-resolve — no version ranges changed; one (chrome-devtools-mcp) needed a manifest bump across its 0.x→1.x major. Nooverrideswere added, and the existingoverridesblock is untouched.Method follows the team's
transitive-dep-upgradeskill: trace every chain to its root, prefer lockfile re-resolve, bump the root parent only when ranges don't admit the fix, overrides as last resort (not needed here).Per-package notes
release-it, which exact-pins undicirelease-it20.0.1 → 20.2.1 (within existing^20.0.1range) lifts its pin to 7.28.0^7.0.4; all peer ranges (@tailwindcss/vite,@vitejs/plugin-react, vitest, …) admit@ladle/reactdep^6.0.5superagent^4.0.5,@types/superagent^4.0.0mermaiddep^3.3.1streamdown/@streamdown/mermaiddep^11.12.2cosmiconfigdep^4.1.0minimatchdep^5.0.5express^6.14.0,body-parser^6.14.1,superagent^6.14.1^0.21.0(nothing else depends on it)^0.21.0→^1.1.0@vitejs/plugin-react^7.29.0,@tanstack/router-plugin^7.28.5,@ladle/react^7.26.0,shadcn^7.28.0, + babel-plugin peers^7.0.0chrome-devtools-mcp 0.x → 1.x (reviewer attention)
This is the one in-major jump. Checked before bumping:
.mcp.json, not referenced by any script, config, or CI workflow. The only mention in the repo is.agents/skills/cli-chrome-axi/SKILL.md, which runs the separatechrome-devtools-axipackage vianpx -y(fetched at run time, independent of this devDependency).chrome-devtools,chrome-devtools-mcp), same node engines (^20.19.0 || ^22.12.0 || >=23), and a fully bundled dependency tree (its vulnerable transitives ship inside the tarball, which is why the fix requires the major bump rather than a re-resolve).Risk looks minimal, but please sanity-check if anyone relies on 0.21-era MCP tool names when launching it manually.
Verification
package-lock.json(all locked copies of all 10 packages enumerated and checked).npm ci(frozen install): clean.npm run check(oxfmt + oxlint type-aware): pass — 0 errors, 11 warnings (pre-existing, no source files touched).npm test(vitest): 2439 passed, 7 skipped, 3 failed — the same 3 tests (sandbox-guard.test.ts,petrinaut-events.test.ts) fail identically on unmodifiedmainin this container (sandbox-backend/environment-dependent), confirmed by re-running against the pre-change lockfile; not caused by this change.npm run build(client + server-runtime): pass.Linear: SRE-808
Generated by Claude Code