refactor(vite-node-miniflare): use ssr.optimizeDeps for vite-node

[hi-ogawa]

• closes figure out ssr.optimizeDeps for vite-node-miniflare #151

(To be merged after Vite 5.1 release with vitejs/vite#15561)

summary

• call ssrLoadModule to trigger ssr deps optimization
  • fix: init dev ssr optimizer on server init vitejs/vite#15561 will make this unnecessary
• set ssr.target: "webworker" so that esbuild pre-bundling target is "browser"
• set ViteNodeServerOptions.cacheDir to random directory to disable vite-node's externalization

old comment

Vite/esbuild doesn't look happy with bundling react-dom/server:

 [ERROR] Cannot bundle Node.js built-in "stream" imported from "../../../../node_modules/.pnpm/[email protected][email protected]/node_modules/react-dom/cjs/react-dom-server-legacy.node.development.js". Consider disabling ssr.noExternal or remove the built-in dependency. [plugin vite:dep-pre-bundle]

Probably tweaking ssr.optimizeDeps.esbuildOptions would make it work?
(Either force resolve.conditions to browser or just importing react-dom/server.browser is fine)

[hi-ogawa]

Looks like ViteNodeServer.fetchModule returns externalize for pre-bundled deps, which tells ViteNodeRunner to directly import which, of course, is not possible on Workerd.

Error: No such module "home/hiroshi/code/personal/vite-plugins/packages/vite-node-miniflare/examples/react/node_modules/.vite/deps_ssr/react_jsx-dev-runtime.js".
    at ViteNodeRunner.importExternalModule (__vite_node_miniflare_entry.js:1677:5)
    at ViteNodeRunner.interopedImport (__vite_node_miniflare_entry.js:1683:39)
    at ViteNodeRunner.directRequest (__vite_node_miniflare_entry.js:1563:35)
    at ViteNodeRunner.cachedRequest (__vite_node_miniflare_entry.js:1499:14)
    at ViteNodeRunner.dependencyRequest (__vite_node_miniflare_entry.js:1543:12)
    at /home/hiroshi/code/personal/vite-plugins/packages/vite-node-miniflare/examples/react/src/worker-entry.tsx:1:154
    at ViteNodeRunner.runModule (__vite_node_miniflare_entry.js:1662:5)
    at ViteNodeRunner.directRequest (__vite_node_miniflare_entry.js:1646:5)
    at ViteNodeRunner.cachedRequest (__vite_node_miniflare_entry.js:1499:14)
    at ViteNodeRunner.executeId (__vite_node_miniflare_entry.js:1475:12)

https://github.com/vitest-dev/vitest/blob/043b78f3257b266302cdd68849a76b8ed343bba1/packages/vite-node/src/client.ts#L439-L441

This decision is coming from here:

  // always externalize Vite deps, they are too big to inline
  if (options?.cacheDir && id.includes(options.cacheDir))
    return id

https://github.com/vitest-dev/vitest/blob/043b78f3257b266302cdd68849a76b8ed343bba1/packages/vite-node/src/externalize.ts#L104-L106