docs/site: fix Dependabot npm alerts via overrides#21168
Merged
Conversation
Add npm `overrides` for vulnerable transitive deps in the Docusaurus site and regenerate `package-lock.json`: - `@babel/plugin-transform-modules-systemjs` 7.29.0 → 7.29.4 (GHSA-fv7c-fp4j-7gwp, high) - `fast-uri` 3.1.0 → 3.1.2 (GHSA-v39h-62p7-jpjc, GHSA-q3j6-qgpj-74h6, both high) - `serialize-javascript` 6.0.2 → 7.0.5 (GHSA-5c6j-r48x-rmvq high, GHSA-qj8w-gfj5-8c6v medium) `npm audit` now reports 0 vulnerabilities. The site still builds via `npm run build` (Node 24). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR addresses Dependabot security alerts in the Docusaurus docs site by forcing patched versions of vulnerable transitive npm dependencies via overrides, and updating the lockfile accordingly.
Changes:
- Add npm
overridesfor@babel/plugin-transform-modules-systemjs,fast-uri, andserialize-javascriptto require non-vulnerable versions. - Update
package-lock.jsonto reflect the resolved patched versions (and removerandombytesas it’s no longer needed viaserialize-javascript@7).
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| docs/site/package.json | Adds security-motivated overrides entries for vulnerable transitive dependencies. |
| docs/site/package-lock.json | Updates resolved transitive dependency versions to patched releases and removes no-longer-required entries. |
Files not reviewed (1)
- docs/site/package-lock.json: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
AskAlexSharov
approved these changes
May 13, 2026
5 tasks
pull Bot
pushed a commit
to Dustin4444/erigon
that referenced
this pull request
Jun 9, 2026
…h#21691) ## Summary Fixes all 3 open [Dependabot alerts](https://github.com/erigontech/erigon/security/dependabot) (3 medium) — two transitive npm deps across two lockfiles. Both are fixed with npm `overrides`, the same approach as erigontech#21168. | Alert | Package | Lockfile | Bump | |---|---|---|---| | [84](https://github.com/erigontech/erigon/security/dependabot/84) | `uuid` (via `sockjs`) | `docs/site` | 8.3.2 → 11.1.1 | | [82](https://github.com/erigontech/erigon/security/dependabot/82) | `ws` (via `webpack-dev-server`) | `docs/site` | 8.20.0 → 8.21.0 | | [83](https://github.com/erigontech/erigon/security/dependabot/83) | `ws` (via `ethers`) | `execution/tracing/tracers/internal/tracetest/testgenerator` | 8.17.1 → 8.21.0 | - `uuid` < 11.1.1 — [CVE-2026-41907](GHSA-w5hq-g745-h8pq) / GHSA-w5hq-g745-h8pq (medium): missing buffer bounds check in `v3`/`v5`/`v6` when `buf` is provided. - `ws` >= 8.0.0, < 8.20.1 — [CVE-2026-45736](GHSA-58qx-3vcg-4xpx) / GHSA-58qx-3vcg-4xpx (medium): uninitialized memory disclosure. The overrides are **scoped to the vulnerable consumers**, so the non-vulnerable sibling copies in `docs/site` are left untouched: `ws@7.5.10` (`webpack-bundle-analyzer`, which needs `^7`) and `uuid@14.0.0` (`mermaid`). ## Test plan `docs/site` (mirrors the `Docs Site Build` CI workflow; run on Node 24): - [x] `npm ci` — clean, lockfile in sync with `package.json`, **0 vulnerabilities** - [x] `npm run typecheck` — passes - [x] `npm run build` — succeeds `testgenerator`: - [x] `npm ci` — **0 vulnerabilities** - [x] `npm run build` (`tsc`) — passes
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.
Summary
Fixes all 5 open Dependabot alerts (4 high, 1 medium) in
docs/site/package-lock.json. Each is a transitive npm dep, so the fix uses npmoverridesindocs/site/package.json:@babel/plugin-transform-modules-systemjs7.29.0 → 7.29.4 — GHSA-fv7c-fp4j-7gwp (high, arbitrary code generation)fast-uri3.1.0 → 3.1.2 — GHSA-v39h-62p7-jpjc + GHSA-q3j6-qgpj-74h6 (both high)serialize-javascript6.0.2 → 7.0.5 — GHSA-5c6j-r48x-rmvq (high, RCE) + GHSA-qj8w-gfj5-8c6v (medium, ReDoS)serialize-javascriptjumps a major version (6 → 7), but v7 only drops Node 14 support — theenginesfield indocs/site/package.jsonalready requires Node ≥ 20.Test plan
npm auditreports 0 vulnerabilitiesnpm run buildsucceeds (Node 24)🤖 Generated with Claude Code