Skip to content

chore(deps): migrate app to vite 8 + @vitejs/plugin-react 6 (closes #336 path)#353

Merged
rz1989s merged 1 commit into
mainfrom
chore/vite-8-plugin-react-6
Jun 25, 2026
Merged

chore(deps): migrate app to vite 8 + @vitejs/plugin-react 6 (closes #336 path)#353
rz1989s merged 1 commit into
mainfrom
chore/vite-8-plugin-react-6

Conversation

@rz1989s

@rz1989s rz1989s commented Jun 25, 2026

Copy link
Copy Markdown
Member

Unblocks Dependabot #336 (@vitejs/plugin-react 4→6) by doing the coordinated vite 6→8 migration it requires. plugin-react 6 hard-requires vite ^8 (the Rolldown era).

Why this stayed small

  • plugin-react 6's new peers @rolldown/plugin-babel + babel-plugin-react-compiler are both optional (peerDependenciesMeta.optional) — only needed for the React Compiler / Rolldown-babel. Not installed.
  • The app's vite.config.ts is minimal (react() + tailwindcss() + a dev proxy) and uses none of vite 8's renamed options (rollupOptionsrolldownOptions, esbuildoxc), so no config changes.
  • @tailwindcss/vite 4.3.1 (already on main) and vitest 4 both already declare vite ^8 support.

Note: workspace-wide vite dedup

Pinning the app to vite ^8 dedups every workspace's vitest onto vite 8.1.0 (no more vite 7) — so the whole test suite now transforms under vite 8, not just the app. Verified accordingly below.

Verification (fresh worktree off latest main, all under vite 8.1.0)

  • pnpm typecheck — green (root / app / sdk / agent)
  • vite buildvite v8.1.0, 459 modules transformed, clean (Rolldown bundler)
  • Tests: root 563 / sdk 118 / app 577 / agent 1718 (+2 skipped) = 2976

One agent test flaked once in 7 full-suite runs (non-reproducible, passes on re-run) — a pre-existing state-pollution flake, not a vite-8 transform regression (which would fail deterministically). If CI trips it, re-run.

Supersedes #336 (closed as superseded on merge).

plugin-react 6 hard-requires vite ^8 (the Rolldown era), so #336 needs
the coordinated vite 6->8 bump. plugin-react 6's new peers
(@rolldown/plugin-babel, babel-plugin-react-compiler) are both OPTIONAL
(peerDependenciesMeta), so no extra deps are needed — just vite + the plugin.

The app's vite.config is minimal (react + tailwind + dev proxy) and uses
none of vite 8's renamed options (rollupOptions->rolldownOptions,
esbuild->oxc), so no config changes are required. @tailwindcss/vite 4.3.1
and vitest 4 already declare vite ^8 support.

Note: pinning the app to vite ^8 dedups every workspace's vitest onto
vite 8.1.0 (no more vite 7), so the whole suite now transforms under vite 8.

Verified off latest main, full suite under vite 8.1.0:
  typecheck green (root/app/sdk/agent)
  vite 8 production build — 459 modules, clean
  root 563 / sdk 118 / app 577 / agent 1718 (+2 skipped)

Supersedes #336.
@vercel

vercel Bot commented Jun 25, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sipher Ready Ready Preview, Comment Jun 25, 2026 5:45am

@rz1989s rz1989s merged commit e337eeb into main Jun 25, 2026
8 checks passed
@rz1989s rz1989s deleted the chore/vite-8-plugin-react-6 branch June 25, 2026 05:49
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