Skip to content

fix: override react-router-dom to 7.18.2 in dev-server UI - #825

Open
devin-ai-integration[bot] wants to merge 6 commits into
mainfrom
devin/dep-vulns-major-pkg/react-router-dom-160d5cb/2026-09-28
Open

devin-ai-integration[bot] wants to merge 6 commits into
mainfrom
devin/dep-vulns-major-pkg/react-router-dom-160d5cb/2026-09-28

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

Dependency vulnerability fixes (automated)

Generated by the Devin Dependency Security Vuln Fix automation run on 2026-10-26.
Change type: Major version bump(s), manifest/lockfile only.

Warning

We're not just looking for a review like a normal PR. Security can offer this fix as a recommendation, but doesn't have the tooling or domain knowledge to safely verify changes like this end-to-end for each repo. Please review, test, and own deployment before merging. For more information: https://launchdarkly.atlassian.net/wiki/spaces/SEC/pages/5360943572/Dependency+Vulnerability+Remediation+with+Devin.

Findings addressed

Package Ecosystem Bump type Current → Target Severity Age Source(s) Advisory
react-router (nested under react-router-dom 6) npm Major (override react-router-dom 6.30.1 → 7.18.2) 6.30.1 → 7.18.2 Medium 95d Dependabot, Wiz GHSA-337j-9hxr-rhxg, GHSA-wrjc-x8rr-h8h6

Notes:

Deferred / excluded findings

Verification

Re-run on 2026-10-26 after a merge of current main. Local tools: Node v24.19.0, npm 11.21.1, Go 1.26.10 (the toolchain in go.mod).

  • Install: ✅ (npm ci in internal/dev_server/ui, go mod tidy makes no changes)
  • Build: ✅ (npm run build, go build ./...)
  • Tests: ✅ (npm test, go test ./... with the sandbox LD_* env vars unset)
  • Lint: ✅ (npm run lint, go vet ./..., pre-commit hooks passed on the merge commit)

Written by Devin

Open in Devin Desktop: https://app.devin.ai/desktop/session/db4ffaacfbb14aa99a113cf588967538?variant=devin

…rjc-x8rr-h8h6)

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@ld-ahartmann ld-ahartmann left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

sure

@devin-ai-integration
devin-ai-integration Bot requested a review from a team October 12, 2026 13:12
@ld-ahartmann
ld-ahartmann enabled auto-merge (squash) October 20, 2026 19:52
devin-ai-integration Bot and others added 2 commits October 26, 2026 13:18
…d5cb/2026-09-28

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant