Repository navigation
chore(deps): update dependency @vitejs/plugin-react to v6 - #13
Conversation
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
🔦 Lighthouse CI
This repo’s lighthouserc assertions can fail this check. |
rishitank
left a comment
There was a problem hiding this comment.
Self-review by Claude
Checked package.json and bun.lock.
- The manifest and lockfile agree: plugin-react 6.1.2 is locked, the Babel transitive dependencies it no longer needs are dropped, and
@rolldown/pluginutils1.0.1 is hoisted. - v6 requires
vite ^8, and the repo pinsvite 8.1.5, so the peer dependency is satisfied. - Lint/typecheck/test/build and Browser tests are green.
- The Lighthouse failure is the same one on every open PR in this repo. It does not come from this bump and is being handled in preview-proof #11.
Nothing to fix. This can merge once Lighthouse is green.
5a2e2e7 to
528ff14
Compare
🧪 Runtime gate✅ Runtime crawl: no runtime errors1 page(s) opened in a real browser (desktop + phone), safe controls exercised, forms tried with dummy data, links followed client-side, missing-route probe run. 17s. Coverage: 1/1 routes · 2/3 controls operated · 1/1 forms submitted · 39.1% of the app's own JavaScript executed Against the default branch: routes ±0, controls −1, JS +0.1 pts. No visual changes against the last default-branch run. Pages visited
⏭️ AI explorer skippedNo Screenshots, diffs and the app log: runtime-gate artifact. |
528ff14 to
35e7973
Compare
35e7973 to
9edc392
Compare
Independent Claude review (cloud session)Head reviewed: Recommendation: MERGE (low risk). Merging is the owner's call; I have not approved or merged anything. What changed
Release notes (6.0.0 to 6.1.2)v6 moves the plugin from Babel to Oxc/Rolldown. The 6.0.0 changelog entry is empty, so I checked the peer-dependency and dependency changes in the lockfile instead. The 6.0.x to 6.1.2 notes are fixes and the new opt-in React Compiler ( How the plugin is used
Verification (fresh checkout of the head,
|
| Step | Result |
|---|---|
| lint | pass (0 errors, 8 warnings, the same 8 as on main) |
| typecheck | pass |
| test | pass (8 files, 264 tests) |
| build | pass (client and SSR/nitro) |
dev server (vite dev --host 127.0.0.1) |
starts on Vite 8.3.3, / returns 200, and the served button.tsx contains the Fast Refresh transform ($RefreshReg$/$RefreshSig$) |
| merged together with #12 | no conflicts; with #12 merged onto main, lint, typecheck, test and build also pass |
Local runs used Node v22.22.0 and Bun 1.4.2; CI pins Bun 1.3.13. I did not exercise HMR in a live browser (the Runtime gate and Browser tests in CI cover the rendered app). The dev server needed --host 127.0.0.1 here only because the sandbox has no IPv6.
Findings (ranked)
- Info: the dev server logs a Vite notice that
vite-tsconfig-pathscan be replaced byresolve.tsconfigPaths. This is unrelated to this PR (the plugin comes from the Lovable config) and not actionable here. - Info: Runtime gate shows "controls −1" against the default branch (2/3 operated, same as chore(deps): update dependency @types/node to v26 #12's baseline). There are no runtime problems and no visual changes. It looks like crawl variance, but it is worth knowing about.
- No blocking or medium findings.
Fixes
None needed, so no commits were pushed to this branch.
Gate status
- CI on this head: green. Lint/typecheck/test/build, Browser tests, Lighthouse CI, Runtime gate, Vulnerable dependencies and Secrets all pass.
security / Ecosystem auditandflowsare skipped, as in the workflow config. - CodeRabbit: skipped. It posted "Review skipped: Bot user detected" on this PR. I did not trigger it, per instructions. Whether that satisfies "checked on the head" is the owner's call.
- Unresolved threads: none (0 review threads).
- Caveat: Renovate may recreate this branch, which would change the head SHA and make this review stale.
Generated by Claude Code
This PR contains the following updates:
^5.2.0→^6.1.2Release Notes
vitejs/vite-plugin-react (@vitejs/plugin-react)
v6.1.2Compare Source
Fix HMR for compound components (#1484)
Adding or removing a component from a compound component was not handled, creating stale updates. Doing this now triggers hmr invalidate on the file and propagates the invalidation to importers.
Fix
compiler.logDiagnosticsoption not working (#1483)With
oxc-transform-react>= 0.148, thelogDiagnosticswas not working. This version fixes it and requiresoxc-transform-react>= 0.152 as a peer dependency if you are using thecompileroption.compiler.logDiagnosticsoption is now deprecated and should be swapped for the builtincompiler.reportDiagnostics.Disable refresh for non-jsx with compiler enabled (fix #1478) (#1485)
When the compiler was enabled, TS files could trigger fast refresh which would cause runtime errors. The logic now follows what the builtin oxc plugin in Vite does.
v6.1.1Compare Source
Add
compiler.logDiagnosticsoptionRecoverable React Compiler diagnostics are no longer logged by default. Set
compiler.logDiagnosticstotrueto log them through Vite. Fatal diagnostics are always logged and fail the transform.Respect environment sourcemap option for React Compiler transform when
builder.sharedPluginsis enabled (#1439)The React Compiler transform was using the top-level sourcemap option instead of the environment sourcemap option. This caused a problem when the experimental
builder.sharedPluginswas enabled.v6.1.0Compare Source
Add experimental native React Compiler support (#1419)
Add experimental native React Compiler support.
You can use it by installing
oxc-transform-reactand enabling it via thecompileroption:v6.0.5Compare Source
Fixed the react compiler preset filter to be linear (#1353)
The improved filter in v6.0.3 was non-linear and caused a performance regression (#1349). The filter was changed to be linear to avoid that.
v6.0.4Compare Source
Fixed
$RefreshSig$ is not definederror when runningvite devwithNODE_ENV=productionWhen running
vite devwithNODE_ENV=production, the app errored with$RefreshSig$ is not defined.This error is now fixed.
v6.0.3Compare Source
Improve the react compiler preset filter to reduce false-positives (#1138)
Improved the filter in the react compiler babel preset to reduce the false-positives so that less modules are processed by the react compiler.
v6.0.2Compare Source
Allow all options in reactCompilerPreset (#1189)
This is a type only change. Only
compilationModeandtargetoptions were available forreactCompilerPreset.v6.0.1Compare Source
Expand
@rolldown/plugin-babelpeer dep range (#1146)Expanded
@rolldown/plugin-babelpeer dep range to include^0.2.0.v6.0.0Compare Source
Configuration
📅 Schedule: (in timezone Europe/London)
* 0-6 * * 1)🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.