-
Notifications
You must be signed in to change notification settings - Fork 4.1k
[Snyk] Security upgrade react-router-dom from 6.30.1 to 7.0.0 #9469
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-JS-REMIXRUNROUTER-14908530 - https://snyk.io/vuln/SNYK-JS-REACTROUTER-14908286 - https://snyk.io/vuln/SNYK-JS-REMIXRUNROUTER-14908287
Learn moreAll Green is an AI agent that automatically: ✅ Addresses code review comments ✅ Fixes failing CI checks ✅ Resolves merge conflicts |
1 similar comment
Learn moreAll Green is an AI agent that automatically: ✅ Addresses code review comments ✅ Fixes failing CI checks ✅ Resolves merge conflicts |
|
|
✅ Review Complete Code Review SummaryTroubleshooting
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1 issue found across 1 file
Prompt for AI agents (all issues)
Check if these issues are valid — if so, understand the root cause of each and fix them.
<file name="gui/package.json">
<violation number="1" location="gui/package.json:64">
P0: Incomplete React Router v7 migration. This upgrade requires changing all imports from 'react-router-dom' to 'react-router' across 22+ files, but only the package version was updated. The application will likely fail at runtime.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
| "react-redux": "^8.0.5", | ||
| "react-remark": "^2.1.0", | ||
| "react-router-dom": "^6.14.2", | ||
| "react-router-dom": "^7.0.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
P0: Incomplete React Router v7 migration. This upgrade requires changing all imports from 'react-router-dom' to 'react-router' across 22+ files, but only the package version was updated. The application will likely fail at runtime.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At gui/package.json, line 64:
<comment>Incomplete React Router v7 migration. This upgrade requires changing all imports from 'react-router-dom' to 'react-router' across 22+ files, but only the package version was updated. The application will likely fail at runtime.</comment>
<file context>
@@ -61,7 +61,7 @@
"react-redux": "^8.0.5",
"react-remark": "^2.1.0",
- "react-router-dom": "^6.14.2",
+ "react-router-dom": "^7.0.0",
"react-switch": "^7.0.0",
"react-syntax-highlighter": "^15.5.0",
</file context>
|
Failed to update package lock. Consolidating onto new PR |

Snyk has created this PR to fix 3 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):
gui/package.jsonVulnerabilities that will be fixed with an upgrade:
SNYK-JS-REMIXRUNROUTER-14908530
SNYK-JS-REACTROUTER-14908286
SNYK-JS-REMIXRUNROUTER-14908287
Important
Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic
Learn how to fix vulnerabilities with free interactive lessons:
🦉 Open Redirect
🦉 Cross-site Scripting (XSS)
Continue Tasks
Powered by Continue
Summary by cubic
Upgrade react-router-dom to 7.0.0 to fix high-severity XSS and open redirect vulnerabilities. Improves routing security and URL handling.
Dependencies
Migration
Written for commit beda9ce. Summary will update on new commits.