Skip to content

chore: bump vite to 8.0.16, bump @vitejs/plugin-react to 6.0.2#1603

Open
jescalada wants to merge 2 commits into
mainfrom
vite-8-bump
Open

chore: bump vite to 8.0.16, bump @vitejs/plugin-react to 6.0.2#1603
jescalada wants to merge 2 commits into
mainfrom
vite-8-bump

Conversation

@jescalada

@jescalada jescalada commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Description

Aggregates vite and @vite/plugin-react bumps as they're dependent on one another. Supersedes related Dependabot PRs (#1597, #1574)

Note that there's a bug in a Vite 8 dependency (rolldown) which causes the UI/Cypress tests to error out. This is apparently caused by a circular dependency in MUI 4 which that dependency is choking on. I figured we could override it for now with a working version just to keep our dependencies up to date.

Checklist

General

Tests

  • Tests have been added/updated for new functionality
  • Unit tests pass (npm test)
  • Linting and formatting pass (npm run lint and npm run format:check)
  • Type checks pass (npm run check-types)

@netlify

netlify Bot commented Jun 19, 2026

Copy link
Copy Markdown

Deploy Preview for endearing-brigadeiros-63f9d0 canceled.

Name Link
🔨 Latest commit 27601db
🔍 Latest deploy log https://app.netlify.com/projects/endearing-brigadeiros-63f9d0/deploys/6a351f3327839000072b9580

@github-actions

github-actions Bot commented Jun 19, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

@codecov

codecov Bot commented Jun 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.51%. Comparing base (fc23d58) to head (27601db).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1603      +/-   ##
==========================================
+ Coverage   85.38%   85.51%   +0.12%     
==========================================
  Files          83       83              
  Lines        7878     7878              
  Branches     1312     1312              
==========================================
+ Hits         6727     6737      +10     
+ Misses       1123     1114       -9     
+ Partials       28       27       -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Workaround for a bug caused by a Vite dependency regression + Material UI's circular dependencies. We should remove the override once #1507 goes in
@jescalada jescalada marked this pull request as ready for review June 19, 2026 11:04
@jescalada jescalada requested a review from a team as a code owner June 19, 2026 11:04
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