Skip to content

[codex] fix dependabot xmldom pin#2459

Merged
jcstein merged 1 commit intomainfrom
codex/fix-dependabot-2457
Apr 1, 2026
Merged

[codex] fix dependabot xmldom pin#2459
jcstein merged 1 commit intomainfrom
codex/fix-dependabot-2457

Conversation

@jcstein
Copy link
Copy Markdown
Member

@jcstein jcstein commented Apr 1, 2026

Summary

Pins the vulnerable transitive @xmldom/xmldom dependency to 0.9.9 for both npm and Yarn installs.

Why

nextra pulls in better-react-mathjax -> mathjax-full -> speech-rule-engine, which currently requests @xmldom/xmldom@0.9.8. That leaves the repo with an open Dependabot alert on yarn.lock.

What changed

  • added an npm overrides pin for @xmldom/xmldom in package.json
  • added a matching Yarn resolutions pin in package.json
  • updated package-lock.json and yarn.lock to resolve @xmldom/xmldom to 0.9.9

Validation

  • confirmed the updated lockfiles now point @xmldom/xmldom at 0.9.9
  • kept the diff scoped to the manifest and lockfiles only

Open with Devin

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

🚀 Preview Deployment

Your preview is ready: https://celestiaorg.github.io/docs-preview/pr-2459/

@jcstein jcstein marked this pull request as ready for review April 1, 2026 16:44
@jcstein jcstein requested a review from gbarros April 1, 2026 16:44
@jcstein jcstein self-assigned this Apr 1, 2026
Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@jcstein jcstein merged commit d1b76bd into main Apr 1, 2026
10 checks passed
@jcstein jcstein deleted the codex/fix-dependabot-2457 branch April 1, 2026 21:04
@jcstein jcstein mentioned this pull request Apr 8, 2026
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