Update Vite to v3/v4

[zoton2]

Not experienced at developing Vite plugins but giving this a shot. Keeping this as draft for now as we work on it.

• Updates vite to the latest 3.x.x version.
• Updates rollup to the latest 2.x.x version; mainly had to bump this to fix some typing issues during build because of mismatched dependencies between rollup and Vite. There is also a newer major version (3.x.x) but it isn't supported by Vite 3.
• Updated pnpm-lock.yaml version to 5.4.
• Added "WebWorker" to "compilerOptions.lib" in tsconfig.json; not sure if this is needed but will not build without it because of type issues in dependencies.
  • Adding "skipLibCheck": true into "compilerOptions" also works, but wasn't sure if there was an important reason not to have it.

This successfully builds and when used in my nodecg-vue-ts-template, allows that to successfully build as well. You may need to check the Vite migration guide yourself to see if I missed anything as I'm not great at fully checking it when I didn't write the plugin.

We should also probably bump the other dependencies too, at least to minor versions, but I wanted to bump the least amount possible to see what was needed to get this to build.

[Dan-Shields]

Will review ASAP, thanks very much

[zoton2]

Vite v4 came out recently as well so will probably want to tweak this for that version. It also supports rollup v3 so we can bump that now as well.

[zoton2]

Just bumped all versions of dependencies up to the latest, including vite to 4.x.x and rollup to 3.x.x. No issues building, but have yet to test it with my template. The migration guide for vite v3 to v4 can be found here if it's needed for reference.

[Dan-Shields]

I can't see any issues with this from looking at the changes. I'll test it out tomorrow and merge if all is well.

I'd like to get #2 ironed out too before releasing a major version bump. Could also be good to do that before a RC.

[zoton2]

Just to update, I tested this myself on my template and it seems to build without issue.

[Dan-Shields]

Added "WebWorker" to "compilerOptions.lib" in tsconfig.json

Appears to be a reproduction of vitejs/vite#9813. If that gets resolved we can remove this, it shouldn't be necessary from what I can tell.

[Dan-Shields]

Can't see any issues with updating rollup. We only hook into writeBundle and buildStart and these haven't been changed.

[Dan-Shields]

Looks good to go. I'm going to merge into master and after I've tackled #2 and #3 this week I'll make a pre-release RC.