[Feature?]: Vite 5 / Rollup 4 support

[birkskyum]

Duplicates

• I have searched the existing issues

Latest version

• I have tested the latest version

Summary 💡

Lots of good stuff here:

Vite 5

• https://vitejs.dev/blog/announcing-vite5
• https://vitejs.dev/guide/migration#deprecate-cjs-node-api

Rollup 4

• https://github.com/rollup/rollup/blob/master/CHANGELOG.md#400
• https://rollupjs.org/migration/

Examples 🌈

No response

Motivation 🔦

No response

[birkskyum]

Blocked by:

• vite 5 support nksaraf/vinxi#28

[LexSwed]

I assumed that vite 5 support was added here:
620b638

[birkskyum]

I assumed that vite 5 support was added here: 620b638

that was a first step, fixing simple usage of solidj.js with vite outside of Start, i.e. to make a SPA without a server part. With the 0.4 version of solid start where the dependency graph is start -> vinxi -> vite, it's necessary to update this in vinxi, and maybe adjust some things in start then as well.

[nksaraf]

Yup I had got it working earlier but found some issues in my projects where rollup 5 would choke on some syntax in some old libs.

I will add this back with optional support for falling back to vite 4 via overriding npm/yarn/pnpm resolutions.

[nksaraf]

Added this back in vinxi@0.0.54. I haven't made vite 5 the default, but you can use it by getting your own version of vite using npm/yarn/pnpm resolutions or overrides.

[birkskyum]

Thanks!

[nksaraf]

Please confirm if this actually works I haven't tried it out outside my repo

[birkskyum]

@nksaraf , it works :) Thanks!

• [Bug?]: New solid start doesn't work with yarn berry #1177 (comment)