Skip to content

chore(deps): upgrade G5 — React core & router - #51

Closed
bravew wants to merge 8 commits into
mainfrom
chore/upgrade-G5-react-core-2026-09-27
Closed

bravew wants to merge 8 commits into
mainfrom
chore/upgrade-G5-react-core-2026-09-27

Conversation

@bravew

@bravew bravew commented Sep 27, 2026

Copy link
Copy Markdown
Owner

Summary

Group 5 of the 2026-09-27 dependency sweep (dev-doc/plan/_module-lib-upgrade.md rev 4). React core and its types move together as one peer group.

Package Workspaces Before After
react root / src-api (exact) ^19.2.8 / 19.2.8 ^19.3.0 / 19.3.0
react-dom root / src-api (exact) ^19.2.8 / 19.2.8 ^19.3.0 / 19.3.0
@types/react root ^19.2.18 ^19.3.0
@types/react-dom root ^19.2.4 ^19.3.0
react-router-dom root ^7.18.2 ^7.18.4
react-error-boundary root ^6.1.3 ^6.1.6

The lockfile resolves a single React 19.3.0 copy; no 19.2.8 references remain.

Test plan

Checks that already fail on main (typecheck 1, format 3 fixtures, API lint 4) produce identical results here.

  • pnpm install: clean, 0 peer warnings
  • pnpm typecheck:all, lint, API lint, format:check: same as baseline
  • pnpm test:fast: 1453 frontend + 3533 API tests passed (7 skipped)
  • pnpm build: PWA precache 26,836.92 → 26,867.38 KiB (+0.11%)
  • Headless navigation smoke of 8 routes (/, library, automation, dashboard, projects, design, video, approvals) against a worktree API with a throwaway HOME: no error boundaries or page errors
  • pnpm audit --audit-level moderate: 7, unchanged; signatures 2148/2148 verified
  • pnpm dev:app (Tauri shell): not run in this session

@playwright/test and playwright 1.62.1 → 1.63.0, @testing-library/react
16.3.2 → 16.3.3, @testing-library/user-event 14.6.5 → 14.6.7, jsdom
30.0.1 → 30.1.1. Refreshes the transitive
@csstools/css-parser-algorithms to 4.0.1 so jsdom's css-calc peer is met.

Held: vitest 5. @testing-library/jest-dom 7.0.1 still augments
Assertion<T>, which no longer merges with Vitest 5's Assertion<R, T>, so
~680 jest-dom matcher type errors appear (jest-dom #738, fix PR #742
open). @copilotkit/channels-core's latest release also peers vitest ^4.

Gate: install clean, no peer warnings; typecheck/lint/format identical
to the main baseline; test:fast 1453 + 3533 passed; browser and API e2e
failures identical to the baseline (23 and 1, pre-existing); audit
unchanged at 7; signatures 2148/2148 verified.
vite 8.2.2 → 8.3.1, @vitejs/plugin-react 6.1.0 → 6.1.1.
vite-plugin-pwa has no newer release, so the workbox-* override stays.

Gate: install clean, no peer warnings; typecheck/lint/format identical
to the main baseline; test:fast 1453 + 3533 passed; pnpm build green,
precache 26836.92 → 26836.04 KiB (540 entries, unchanged); preview
serves index and sw.js; audit unchanged at 7; signatures 2150/2150.
lucide-react 1.33.0 → 1.48.0, tailwind-merge 3.6.0 → 3.7.0.

Gate: install clean, no peer warnings; typecheck/lint/format identical
to the main baseline (no removed-icon type errors); test:fast 1453 +
3533 passed; check:design-system-flags passed; build precache
26836.92 → 26852.13 KiB (+0.06%); headless smoke of 8 routes against
an isolated API renders without crashes; audit unchanged at 7;
signatures 2148/2148.
react and react-dom 19.2.8 → 19.3.0 (root caret, src-api exact pin),
@types/react 19.2.18 → 19.3.0, @types/react-dom 19.2.4 → 19.3.0,
react-router-dom 7.18.2 → 7.18.4, react-error-boundary 6.1.3 → 6.1.6.
The lockfile resolves a single React 19.3.0 copy.

Gate: install clean, no peer warnings; typecheck/lint/format identical
to the main baseline; test:fast 1453 + 3533 passed; build precache
26836.92 → 26867.38 KiB (+0.11%); isolated headless smoke of 8 routes
without crashes or page errors; audit unchanged at 7; signatures
2148/2148.
@coderabbitai

coderabbitai Bot commented Sep 27, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 59 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 1f7a0018-d078-4238-9d12-d2da0e5ac000

📥 Commits

Reviewing files that changed from the base of the PR and between 7b26dc2 and 8d98081.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (3)
  • package.json
  • src-api/package.json
  • src-video/package.json

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

…3-bundler-2026-09-27

# Conflicts:
#	pnpm-lock.yaml
…-G4-design-system-2026-09-27

# Conflicts:
#	pnpm-lock.yaml
…pgrade-G5-react-core-2026-09-27

# Conflicts:
#	pnpm-lock.yaml
#	src-api/package.json
@bravew bravew mentioned this pull request Sep 27, 2026
2 of 3 tasks
@bravew

bravew commented Sep 27, 2026

Copy link
Copy Markdown
Owner Author

Superseded by #61, which lands G2–G14 as one commit.

@bravew bravew closed this Sep 27, 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.

1 participant