-
Notifications
You must be signed in to change notification settings - Fork 225
Open
Labels
SEV-3Severity: 3Normal SeverityNormal SeverityType: BugSomething isn't workingSomething isn't working
Description
Description
@shopify/cli@3.90.0 (latest) bundles an outdated version of the @shopify/cli-hydrogen React Router version check that expects 7.9.2, even though:
- The Hydrogen team already updated
EXPECTED_VERSIONto"7.12.0"onmain(packages/cli/src/lib/react-router-version-check.ts) - This fix was included in
@shopify/cli-hydrogen@11.1.6(via PR #3346) - The skeleton template updated to React Router 7.12.0 in
skeleton@2025.7.1
Current behavior
Running shopify hydrogen dev with @shopify/cli@3.90.0 displays a warning telling developers to downgrade React Router from 7.12.0 to 7.9.2:
╭─ warning ────────────────────────────────────────────────────────────╮
│ │
│ React Router version mismatch detected │
│ │
│ Hydrogen requires React Router 7.9.x for proper functionality. │
│ │
│ Version mismatches found: │
│ • react-router: installed 7.12.0, expected 7.9.2 │
│ • @react-router/dev: installed 7.12.0, expected 7.9.2 │
│ • @react-router/fs-routes: installed 7.12.0, expected 7.9.2 │
│ │
│ To fix this issue, run: │
│ npm install react-router@7.9.2 │
│ npm install -D @react-router/dev@7.9.2 @react-router/fs-routes@7.9.2 │
│ │
╰──────────────────────────────────────────────────────────────────────╯
Evidence
Confirmed by inspecting the bundled CLI code:
# @shopify/cli@3.90.0 (npm, latest)
$ grep "EXPECTED_VERSION" node_modules/@shopify/cli/dist/index.js
], EXPECTED_VERSION = "7.9.2"; # ← stale, should be "7.12.0"
# @shopify/cli@3.89.0 (Homebrew, latest)
$ grep "EXPECTED_VERSION" /opt/homebrew/Cellar/shopify-cli/3.89.0/.../dist/index.js
], EXPECTED_VERSION = "7.9.2"; # ← same stale value
Meanwhile, the Hydrogen repo main branch already has the fix:
const EXPECTED_VERSION = '7.12.0'; // ← correctSecurity concern
React Router 7.9.2 has multiple known CVEs fixed in 7.12.0:
- GHSA-h5cw-625j-3rxh: CSRF in Action/Server Action Request Processing
- GHSA-2w69-qvjg-hvjx: XSS via Open Redirects
- GHSA-8v8x-cx79-35w7: SSR XSS in ScrollRestoration
- GHSA-9jcx-v3wj-wh4m: Unexpected external redirect via untrusted paths
- GHSA-9583-h5hc-x8cw: Path Traversal in @react-router/node
The CLI is actively advising developers to install a vulnerable version.
Expected behavior
The next @shopify/cli release should bundle @shopify/cli-hydrogen@11.1.6+ which contains the corrected version check.
Environment
@shopify/cli: 3.90.0 (npm) / 3.89.0 (Homebrew)- Node: 22.x
- OS: macOS
Related
- [Bug]: Shopify CLI React Router version check expects 7.9.2, but Hydrogen officially supports 7.12.0 hydrogen#3449 — companion issue filed on the Hydrogen repo
- chore(deps): bump RR7 to v7.12.0 hydrogen#3346 — PR that updated the version check to 7.12.0
@shopify/cli-hydrogen@11.1.6changelog confirms the fix was included
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
SEV-3Severity: 3Normal SeverityNormal SeverityType: BugSomething isn't workingSomething isn't working