fix: make it easier to configure environment runner

[sheremet-va]

Description

Hide the complexity of the custom runner configuration. If you need to configure the runner, you will most likely use the createServerModuleRunner. It just has a single option argument, let's pass it down directly instead of requiring the user to import the function - they shouldn't even need to know it exists.

[stackblitz]

Run & review this pull request in StackBlitz Codeflow.

[sapphi-red]

Won't this make it impossible to swap the ModuleRunner while keeping isRunnableDevEnvironment(devEnv) to be true?

[sheremet-va]

Won't this make it impossible to swap the ModuleRunner while keeping isRunnableDevEnvironment(devEnv) to be true?

What do you mean by "swap"? It always requires the ModuleRunner instance. The createServerModuleRunner/new ModuleRunner creates that runner. The runnerOptions can configure any options that you can already configure by calling new ModuleRunner:

vite/packages/vite/src/node/ssr/runtime/serverModuleRunner.ts

Line 16 in 51bb168

export interface ServerModuleRunnerOptions

This PR just hides the complexity of calling the createServerModuleRunner/new ModuleRunner directly. I've noticed in a lot of feedback that the new APIs feel too "boilerplate"-y, this PR tries to remove that friction and make it easier to work with

Basically, what I am saying is that runner: () => ModuleRunner and runnerOptions are equivalent in power, but one is easier to use than the other.

[sapphi-red]

What do you mean by "swap"?

I mean using a custom ModuleRunner that extends the ModuleRunner class. I guess it's allowed to be used like that, reading this comment.

vite/packages/vite/src/module-runner/runner.ts

Line 291 in a99a0aa

// override is allowed, consider this a public API

I think this PR makes it more easier and that is good, but I guess we need to give a way to declare a RunnableDevEnvironment with a custom ModuleRunner.

[sheremet-va]

I see what you mean now. I will revert the runner change and also add the options as the argument in the factory so they are not lost.