Skip to content

chore(deps): bump @astrojs/react to ^7.0.0 - #269

Closed
spencer-osbrjp wants to merge 1 commit into
mainfrom
bump/@astrojs-react
Closed

spencer-osbrjp wants to merge 1 commit into
mainfrom
bump/@astrojs-react

Conversation

@spencer-osbrjp

Copy link
Copy Markdown
Collaborator

⚠️ MAJOR / needs template review

From ^6.0.6 to ^7.0.0 — bump type: MAJOR.

  • @astrojs/react 7.0.0 moves the Vite integration to @vitejs/plugin-react v6, using Oxc for JSX transforms and Fast Refresh, and adds an opt-in Oxc-based React Compiler (compiler option, needs oxc-transform-react). Source: CHANGELOG.md and the GitHub releases for @astrojs/react@7.0.0.

Breaking changes

  • babel integration option removed. Projects passing react({ babel: ... }) must migrate: install @rolldown/plugin-babel and @babel/core, then move Babel plugins/presets into a babel() call under vite.plugins instead of the integration option.
    • The Babel plugin enables parsing for .jsx, .ts, .tsx but does not load babel.config.js/.babelrc — options must be passed directly to babel().
    • The old babel callback form is unsupported; use the plugin's override/preset hooks for conditional transforms instead.
    • Babel runs before Oxc's TS/JSX transform, and this still works alongside the new compiler: true option — but "do not enable babel-plugin-react-compiler on the same components as the Oxc compiler."
  • New optional peer dependency oxc-transform-react: ^0.145.0 (only required if you opt into compiler: true).
  • Internal dependency bumps as part of this release: @vitejs/plugin-react ^5.2.0 → ^6.1.1, vite ^8.0.13 → ^8.3.0 (transitive, not directly pinned by this repo).

Not changed / no action needed: react/react-dom peer range stays ^17.0.2 || ^18.0.0 || ^19.0.0; Node engine requirement stays >=22.12.0 in 7.0.0 itself (note: 7.1.0, which ^7.0.0 also permits installing, raises the Node minimum to 22.22.0 — worth knowing even though it's outside the pinned version this PR sets).

Repo impact

  • config/templates/base/astro/astro.config.mjs.tmpl calls the integration as plain react() with no options — it does not use the removed babel option, so no template edit is required for this bump.
  • config/templates/base/vite/vite.config.ts.tmpl uses @vitejs/plugin-react directly (the vite-react base) — unrelated to @astrojs/react, not affected.
  • pkg/add_test.go references @astrojs/react only as a detection-fixture version string ("1"); unaffected by the real version bump.
  • No .tmpl files were edited in this PR.

Build/test

  • go build -o /dev/null .: pass.
  • go test ./...: pass except a pre-existing, unrelated failure in internal/tui (TestSinglePaneUnder100Columns — terminal-width pane-title rendering). Independently confirmed present on main HEAD da18a3f with no registry.json changes, so it is not caused by this bump.

Soak window: 14 days


🤖 Generated with Claude Code

https://claude.ai/code/session_017rn9NQPRgXaaTPHcjbRv9y


Generated by Claude Code

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017rn9NQPRgXaaTPHcjbRv9y
@spencer-osbrjp

Copy link
Copy Markdown
Collaborator Author

Closing without merging: per task instructions, go test ./... must pass cleanly before this PR is opened. It currently fails in internal/tui (TestSinglePaneUnder100Columns). Although this failure reproduces identically on a clean main checkout (HEAD da18a3f) and appears unrelated to this dependency bump, the explicit verification rule for this task does not permit proceeding past any test failure, so this PR is being closed rather than left open. The branch bump/@astrojs-react and its one-line config/registry.json change (@astrojs/react ^6.0.6 → ^7.0.0) remain pushed for reference; no code was merged.


Generated by Claude Code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants