Skip to content

fix(deps): align react-dom with react - #705

Merged
birme merged 1 commit into
Eyevinn:mainfrom
bjowe:fix/react-dom-version-match
Sep 21, 2026
Merged

birme merged 1 commit into
Eyevinn:mainfrom
bjowe:fix/react-dom-version-match

Conversation

@bjowe

@bjowe bjowe commented Sep 21, 2026

Copy link
Copy Markdown
Collaborator

#650 bumped react to ^19.2.7 but left react-dom at ^19.2.3, which the lockfile resolves to 19.2.4. React 19 refuses to run when the two differ:

Error: Incompatible React versions: The "react" and "react-dom" packages
must have the exact same version. Instead got:
- react: 19.2.7
- react-dom: 19.2.4

Every test that renders fails on main as a result.

before: Test Files 15 failed | 9 passed (24)
after: Test Files 24 passed (24) Tests 171 passed

Match react-dom's range to react's so the two cannot drift apart again.

Found while rebasing the video-support branch (#684) onto main. That branch is internally
consistent at 19.2.4 and so is green today, but it inherits the breakage as soon as its merge
conflict with main is resolved.

Eyevinn#650 bumped react to ^19.2.7 but left react-dom at ^19.2.3, which the
lockfile resolves to 19.2.4. React 19 refuses to run when the two differ:

  Error: Incompatible React versions: The "react" and "react-dom" packages
  must have the exact same version. Instead got:
    - react:      19.2.7
    - react-dom:  19.2.4

Every test that renders fails on main as a result.

  before: Test Files  15 failed | 9 passed (24)
  after:  Test Files  24 passed (24)   Tests  171 passed

Match react-dom's range to react's so the two cannot drift apart again.

Co-Authored-By: Claude Opus 5 (1M context) <[email protected]>

@birme birme 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.

LGTM. Fixes a real breakage: react 19.2.7 vs react-dom 19.2.4 mismatch makes React 19 refuse to run, failing every render test on main. Diff is minimal (package.json + lockfile alignment to ^19.2.7), CI green (builds, e2e, lint, prettier). Merging.

@birme
birme merged commit 5f5713d into Eyevinn:main Sep 21, 2026
6 checks passed
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