Vinxi's build command tries to run node.EXE even when using Deno

[Jari Pennanen]

I have example repo here:

https://github.com/Ciantic/deno-solid-test/

Some reason when running it through deno it still tries to run node.exe instances.

Notice the telltale warning:

(Use node.EXE --trace-warnings ... to show where the warning was created)

In following call:

$ deno run -A build 

⚙  Building your app...
vinxi v0.4.3
(node:33840) ExperimentalWarning: Importing JSON modules is an experimental feature and might change at any time
(Use `node.EXE --trace-warnings ...` to show where the warning was created)
ReferenceError: Deno is not defined
    at file:///C:/Source/JavaScript/solid-start-test/deno-solid-test/node_modules/.deno/@jsr+std__fs@0.221.0/node_modules/@jsr/std__fs/ensure_symlink.js:7:19
    at ModuleJob.run (node:internal/modules/esm/module_job:222:25)
    at async ModuleLoader.import (node:internal/modules/esm/loader:316:24)
    at async loadFile (file:///C:/Source/JavaScript/solid-start-test/deno-solid-test/node_modules/.deno/vinxi@0.4.3/node_modules/vinxi/lib/load-app.js:83:25)
    at async loadApp (file:///C:/Source/JavaScript/solid-start-test/deno-solid-test/node_modules/.deno/vinxi@0.4.3/node_modules/vinxi/lib/load-app.js:137:19)
    at async Object.run (file:///C:/Source/JavaScript/solid-start-test/deno-solid-test/node_modules/.deno/vinxi@0.4.3/node_modules/vinxi/bin/cli.mjs:229:17)
    at async runCommand (file:///C:/Source/JavaScript/solid-start-test/deno-solid-test/node_modules/.deno/citty@0.1.6/node_modules/citty/dist/index.mjs:316:16)
    at async runCommand (file:///C:/Source/JavaScript/solid-start-test/deno-solid-test/node_modules/.deno/citty@0.1.6/node_modules/citty/dist/index.mjs:307:11)
    at async runMain (file:///C:/Source/JavaScript/solid-start-test/deno-solid-test/node_modules/.deno/citty@0.1.6/node_modules/citty/dist/index.mjs:445:7)

(With Bun vinxi works only with --bun that does ugly hack of symlinking node.exe as bun.exe)

[Jari Pennanen]

Nitro does have deno support with preset deno_server:

https://github.com/unjs/nitro/blob/v2/docs/2.deploy/10.runtimes/deno.md

But Vinxi is not aware of that:

vinxi/packages/vinxi/bin/cli.mjs

Lines 325 to 351 in 432fdf5

 process.env.SERVER_PRESET ??= 
 	args.preset ?? 
 	process.env.TARGET ?? 
 	process.env.PRESET ?? 
 	process.env.SERVER_PRESET ?? 
 	process.env.SERVER_TARGET ?? 
 	process.env.NITRO_PRESET ?? 
 	process.env.NITRO_TARGET ?? 
 	"node-server"; 
  
 switch (process.env.SERVER_PRESET) { 
 	case "node-server": 
 		await import( 
 			pathToFileURL(process.cwd() + "/.output/server/index.mjs").href 
 		); 
 		break; 
  
 	case "bun": 
 		const { $ } = await import("../runtime/sh.js"); 
  
 		await $`bun run .output/server/index.mjs`; 
 		break; 
 	default: 
 		log( 
 			`Couldn't run an app built with the ${process.env.SERVER_PRESET} preset locally. Deploy the app to a provider that supports it.`, 
 		); 
 }