fix(windows): detect node path correctly on windows with cross-spawn (#6069)

[Losses]

This Pull request would fix #6069

[changeset-bot]

🦋 Changeset detected

Latest commit: aac9218

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 10 packages

Name	Type
builder-util	Patch
app-builder-lib	Patch
dmg-builder	Patch
electron-builder-squirrel-windows	Patch
electron-builder	Patch
electron-publish	Patch
electron-forge-maker-appimage	Patch
electron-forge-maker-nsis-web	Patch
electron-forge-maker-nsis	Patch
electron-forge-maker-snap	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[mmaietta]

Nice detective work!

Do we need to worry about a similar issue in NsisUpdater using child_process?

electron-builder/packages/electron-updater/src/NsisUpdater.ts

Lines 231 to 234 in 695f89a

	const process = spawn(exe, args, {
	detached: true,
	stdio: "ignore",
	})

[mmaietta]

Please take a look at the failing tests:

Error: Cannot spawn /__w/electron-builder/electron-builder/node_modules/.pnpm/app-builder-bin@3.7.1/node_modules/app-builder-bin/linux/x64/app-builder: TypeError: cross_spawn_1.default is not a function

[Losses]

Nice detective work!

Do we need to worry about a similar issue in NsisUpdater using child_process?

electron-builder/packages/electron-updater/src/NsisUpdater.ts

Lines 231 to 234 in 695f89a

	const process = spawn(exe, args, {
	detached: true,
	stdio: "ignore",
	})

I think these code doesn't need to be modified, I didn't find any error in this place when I packaged our application.