Skip to content

Conversation

@sapphi-red
Copy link
Member

@sapphi-red sapphi-red commented Oct 23, 2025

Description

Let's add a revert for now. I'll backport this to 7.1 after this PR is merged.

fixes #20983

@sapphi-red sapphi-red requested a review from hi-ogawa October 23, 2025 03:41
@sapphi-red sapphi-red added the p3-downstream-blocker Blocking the downstream ecosystem to work properly (priority) label Oct 23, 2025
@sapphi-red

This comment was marked as outdated.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Oct 23, 2025

Open in StackBlitz

npm i https://pkg.pr.new/vite@20988

commit: e870983

@sapphi-red
Copy link
Member Author

Oops, the patch is wrong

@vite-ecosystem-ci

This comment was marked as outdated.

@sapphi-red
Copy link
Member Author

/ecosystem-ci run

@bluwy
Copy link
Member

bluwy commented Oct 23, 2025

Should we pin the to previous version instead of patching?

@vite-ecosystem-ci
Copy link

📝 Ran ecosystem CI on 1a8ee07: Open

suite result latest scheduled
analogjs failure success
astro failure success
histoire failure success
laravel failure failure
vite-environment-examples failure success
vite-plugin-vue failure success
qwik success failure
waku success failure
react-router success ⏹️ cancelled
vite-plugin-rsc failure failure
storybook failure success
vite-plugin-svelte failure success
vite-plugin-cloudflare failure success
vitest failure success

ladle, one, rakkas, marko, vike, nuxt, unocss, vite-plugin-react, sveltekit, vitepress, vite-plugin-pwa, quasar, vuepress, vite-setup-catalogue

@sapphi-red
Copy link
Member Author

Should we pin the to previous version instead of patching?

rollup/plugins#1909 was introduced in 28.0.7 which is two versions older. If we pin the version to 28.0.6, it'll also revert rollup/plugins#1914

@hi-ogawa
Copy link
Contributor

If it's easier, it looks fine to pin the old version for now as a quick fix. Is ecosystem CI failure still related to commonjs plugin or something else?

Copy link
Contributor

@hi-ogawa hi-ogawa left a comment

Choose a reason for hiding this comment

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

Oh, I just realized the same patch is for dist/cjs and dist/esm, so it's a simple revert of rollup PR. Isn't only dist/esm bundled to vite?

@sapphi-red
Copy link
Member Author

Is ecosystem CI failure still related to commonjs plugin or something else?

I think it's not related. This is the run for the main branch: https://github.com/vitejs/vite-ecosystem-ci/actions/runs/18737739165

@sapphi-red
Copy link
Member Author

I'll change to a simple revert as that feels safer.

@sapphi-red sapphi-red changed the title fix(deps): patch commonjs plugin to revert rollup/plugins#1909 fix(deps): downgrade commonjs plugin to 28.0.6 to avoid rollup/plugins#1909 Oct 23, 2025
Copy link
Member

@bluwy bluwy left a comment

Choose a reason for hiding this comment

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

We should make sure this isn't accidentally bumped again in the next renovate PR until it's fixed upstream

@sapphi-red sapphi-red merged commit 856e683 into vitejs:main Oct 23, 2025
19 checks passed
sapphi-red added a commit to vitejs/rolldown-vite that referenced this pull request Oct 27, 2025
@sapphi-red sapphi-red deleted the fix/revert-commonjs-1909 branch October 28, 2025 02:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

p3-downstream-blocker Blocking the downstream ecosystem to work properly (priority) trigger: preview

Projects

None yet

Development

Successfully merging this pull request may close these issues.

vite 7.1.11 causing issues CJS node builtin imports

3 participants