impossible to build with vitejs

[wadouk]

True to build unified with vite and it fail at runtime because of process.cwd() used without protection.
It should be great to have a pre-built version for browser because it's unusable without a bundler that override process.

[wadouk]

had to patch at beginning of a file

window.process = {
    cwd: () => '/'
}

use version 4.2

will try with an update

[ChristianMurphy]

@wadouk what version of vfile are you using?
In version 5 and higher, vfile provides

vfile/package.json

Lines 38 to 42 in b4c9fb8

	"browser": {
	"./lib/minpath.js": "./lib/minpath.browser.js",
	"./lib/minproc.js": "./lib/minproc.browser.js",
	"./lib/minurl.js": "./lib/minurl.browser.js"
	},

which removes the dependency on process, path, and some node specific url features for browser builds.
If you are using version 5 and vite is ignoring the browser field, it may be an issue to raise with vite, it appears that vite should support it (https://github.com/vitejs/vite/blob/fa6475fd4230fb2ab200485bc35b136b2474abfd/packages/vite/src/node/plugins/resolve.ts#L635-L640)

In the meantime you could try https://github.com/snowpackjs/rollup-plugin-polyfill-node

[wadouk]

Thanks, seen that.
Try to upgrade unified to have a more up to date version of vfile but some mandatory used plugin are not upgraded.
Partial upgrade with yarn to vfile@5 doesn't work because [email protected] specify vfile@4

Think will go back to my workaround and handle those later.

[ChristianMurphy]

right, vfile 5 should be used along side unified 10, and vfile 4 with unified 9.