Add support for Vite v6

[barrenechea]

Since the workaround in the issue #110 comment seems to work okay, this should do the trick.

Didn't bump the package version since you may want to update dependencies further before releasing it? :)

Fixes #110
Fixes #113

Note: If you're taking a look because this issue happened to you, I've published a fork of this repo as vite-plugin-node-polyfill with Vite v6 support 😄

[daniel-lauri-walsh]

+1 -- I would value this change as I am upgrading a legacy CRA app , and using Vite 6.0.1 and getting this error

[aleksraiden]

Great, waiting the approve and new version

[barrenechea]

@davidmyersdev, is this project still maintained? I see you’re pretty active on other GitHub projects, we could do a fork if not.

[daomtthuan]

Yeah. Just change package json for supporting Vite v6.
No impacts

[Sceat]

Anyone has a published fork ?

[barrenechea]

Anyone has a published fork ?

I'll try to give it a shot today, the CI workflow seems to be kinda broken, and I'd like for it to have a deployment workflow as well for transparency purposes :) I'm working on the CI workflow right now

[barrenechea]

Ok, I've just published vite-plugin-node-polyfill at version 0.22.1 based off my fork. Just worked on the workflows and added the Vite v6 support. The build and publish logs are here.

Remember to update the dependency and the import, from

import { nodePolyfills } from "vite-plugin-node-polyfills";

To

import { nodePolyfills } from "vite-plugin-node-polyfill";

I'm happy to mark the package as deprecated if this repo comes back to life at some point 😄

[O-Mutt]

Ok, I've just published vite-plugin-node-polyfill at version 0.22.1 based off my fork. Just worked on the workflows and added the Vite v6 support. The build and publish logs are here.

Remember to update the dependency and the import, from

import { nodePolyfills } from "vite-plugin-node-polyfills";

To

import { nodePolyfills } from "vite-plugin-node-polyfill";

I'm happy to mark the package as deprecated if this repo comes back to life at some point 😄

I could not see the difference. For anyone with weary eyes like myself the fork is sans "s". polyfill vs polyfills

[davidmyersdev]

Hey all. Apologies for the delay on this. I've been a bit busy with other projects (as you've noticed) and just hadn't come back to this one yet. I'll kick off CI and merge if everything passes.

[davidmyersdev]

Hm... It looks like GitHub does not allow running CI on PRs from forks? I'll have to cherry-pick your commits onto a new branch to run tests, I think.

[davidmyersdev]

Results of the CI run looks good. Merging it.

[davidmyersdev]

Released in v0.23.0.

[barrenechea]

Hey! Sorry for the delayed reply on my side. I just deprecated the forked package 😄

Thank you!!