node prefixes with require are only supported with node 14.18.0+, 16.0.0+.
But Vite and plugins supports 14.6.0+.

So it might need to strip it in build like tsup does or bump the support version to 14.18.0. Maybe unbuild handles this already?

node prefixes with require are only supported with node 14.18.0+, 16.0.0+. But Vite and plugins supports 14.6.0+.

vite/packages/vite/package.json

Line 36 in 45dba50

"node": ">=14.6.0"

So it might need to strip it in build like tsup does or bump the support version to 14.18.0. Maybe unbuild handles this already?

We could discuss this in the next team meeting 🤔

So it might need to strip it in build like tsup does or bump the support version to 14.18.0. Maybe unbuild handles this already?

unbuild doesn't handle unfortunately. It externalises node: and builtinModules by default, which makes it not possible for us to change the id. Setting the rollup.esbuild.target: 'node14.6' config doesn't work too.

For Vite core we have control of the build so it's likely possible with some grease. Otherwise, the easiest way seems to be bumping the node version, though I'm not sure if internal code cleanup justifies bumping 🤔

I vote to wait here, and add this to the reasons to bump (we're surely going to find more in the next months)

I found that Vite is using fs.rmSync which requires Node v14.14.0+.

Another trick would be to create a Rollup plugin to resolve node: imports and strips the protocol part, so that the final output remains not having the protocol. Even if we bump up to node 14.18, it'd help dedupe node:url and url, since non-protocol imports can happen in deps.

We should probably bump to 14.14 though per sapphi's comment.

EDIT: I just realized sapphi mentioned that tsup did this with an esbuild plugin too.

Rebased this.

1. Version upgrade is done at feat!: bump minimum node version to 14.18.0 #8662
2. Didn't add a rollup plugin to dedupe builtin imports, instead opened feat(node-resolve): dedupe preferBuiltins protocol rollup/plugins#1212
3. Thought to add an eslint plugin to ensure node:, but it seems to only exists in eslint-plugin-unicorn. Not sure if we should add the whole plugin just for one rule, was hoping eslint-plugin-import to support it.