Skip to content

chore: upgrade vite to v8#359

Merged
amareshsm merged 5 commits intomainfrom
chore/upgrade-vite-8
Mar 25, 2026
Merged

chore: upgrade vite to v8#359
amareshsm merged 5 commits intomainfrom
chore/upgrade-vite-8

Conversation

@Pixel998
Copy link
Copy Markdown
Contributor

@Pixel998 Pixel998 commented Mar 21, 2026

Prerequisites checklist

AI acknowledgment

  • I did not use AI to generate this PR.
  • (If the above is not checked) I have reviewed the AI-generated content before submitting.

What is the purpose of this pull request?

This PR migrates the project to Vite 8 and updates the React plugin setup to the Vite 8-compatible @vitejs/plugin-react v6 configuration. It also cleans up the related build config changes required by Rolldown/Oxc and fixes the Code Path regression that surfaced during the migration.

In local testing, production builds are about 6x faster and the generated build output is about 5% smaller than with the previous Vite 7 setup.

What changes did you make? (Give an overview)

  • Upgraded vite to 8.0.1 and @vitejs/plugin-react to 6.0.1.
  • Added @rolldown/plugin-babel and @babel/core, and migrated the React Compiler setup from react({ babel: ... }) to react() plus babel({ presets: [reactCompilerPreset()] }).
  • Updated the Vite build config to use Vite 8 / Rolldown-compatible output settings, replacing the old manualChunks setup with codeSplitting.groups.
  • Removed old compatibility/configuration bits that were tied to the previous Vite/plugin-react setup, including stale dependency optimizer and chunking config.
  • Replaced the use-debounced-effect dependency in the Code Path view with a local debounced effect to avoid the runtime interop error introduced by the tooling migration.

Related Issues

Is there anything you'd like reviewers to focus on?

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 21, 2026

Deploy Preview for eslint-code-explorer ready!

Name Link
🔨 Latest commit 3bee6e3
🔍 Latest deploy log https://app.netlify.com/projects/eslint-code-explorer/deploys/69c2c048ac44260008544a87
😎 Deploy Preview https://deploy-preview-359--eslint-code-explorer.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@eslintbot eslintbot added this to Triage Mar 21, 2026
@github-project-automation github-project-automation Bot moved this to Needs Triage in Triage Mar 21, 2026
@Pixel998 Pixel998 marked this pull request as ready for review March 21, 2026 10:49
Comment thread vite.config.ts
@DMartens
Copy link
Copy Markdown
Contributor

Other than the pre-existing error, changes LGTM.
Would like someone else with more vite experience to double-check.

@DMartens DMartens moved this from Needs Triage to Second Review Needed in Triage Mar 24, 2026
Copy link
Copy Markdown
Member

@amareshsm amareshsm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

overall it LGTM.

Vite 8 introduces an experimental resolve.tsconfigPaths option that automatically reads path mappings from your tsconfig.json, eliminating the need to manually configure resolve.alias.

resolve: {
  tsconfigPaths: true
}

it’s still marked as experimental so will take it up later.

@amareshsm amareshsm added build chore accepted There is consensus among the team that this change meets the criteria for inclusion and removed chore labels Mar 25, 2026
@amareshsm amareshsm merged commit 582f8a9 into main Mar 25, 2026
9 checks passed
@amareshsm amareshsm deleted the chore/upgrade-vite-8 branch March 25, 2026 10:19
@github-project-automation github-project-automation Bot moved this from Second Review Needed to Complete in Triage Mar 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

accepted There is consensus among the team that this change meets the criteria for inclusion build chore

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

4 participants