Allow server.open to be passed `readonly string[]` as per `open` spec

Description

Currently, as per the vite types, server.open can only accept boolean | string → https://github.com/vitejs/vite/blob/26bcdc3186807bb6f3817119cd7e64ae8308a057/packages/vite/src/node/http.ts#L37

Further more, the Vite docs say:

server.open See the open package for more details.

When checking the Open docs, for best cross-platform support, they suggest to set the browser like so:

const open = require('open');

await open('https://google.com', {
	app: {
		name: open.apps.chrome
	}
});

Ref: https://github.com/sindresorhus/open#openapps

However, trying to use open.apps.chrome in Vite like so, gives a type error:

server: { open: open.apps.chrome }

Suggested solution

We need to be able to set Chrome but make sure it works for both Windows and MacOS.

My suggestion is to allow readonly string[] which is returned by open.apps.chrome so that we can use this option in the Vite config.

Alternative

No response

Additional context

No response

Validations

• Follow our Code of Conduct
• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn’t already an issue that request the same feature to avoid creating a duplicate.

[bluwy]

Thanks @sapphi-red for the find! I think it’s nice if we can upstream the changes, we would probably have to have it as a normal dependency then, and re-export the app variable. But also, looks like Sindre prefers no to use an applescript: https://github.com/sindresorhus/open/issues/256#issuecomment-868984143

[sapphi-red]

@bluwy

I’m not sure what’s the reason of this mismatch though.

This is to reuse the existing tab. By using AppleScript, we can reuse it on macOS+Chromium base browsers. (It does not work with other OS/browers though…) open will open a new tab everytime.