refactor: share code with vite runtime (#15907)

Author: sapphi-red

.eslintrc.cjs

@@ -145,6 +145,7 @@ module.exports = defineConfig({
     },
     {
       files: ['packages/vite/src/node/**'],
+      excludedFiles: '**/__tests__/**',
       rules: {
         'no-console': ['error'],
       },

.prettierignore

@@ -1,4 +1,5 @@
 packages/*/CHANGELOG.md
+packages/vite/src/node/ssr/runtime/__tests__/fixtures
 playground-temp/
 dist/
 temp/

packages/vite/package.json

@@ -140,6 +140,7 @@
     "postcss-modules": "^6.0.0",
     "resolve.exports": "^2.0.2",
     "rollup-plugin-dts": "^6.1.0",
+    "rollup-plugin-esbuild": "^6.1.1",
     "rollup-plugin-license": "^3.2.0",
     "sirv": "^2.0.4",
     "source-map-support": "^0.5.21",

packages/vite/rollup.config.ts

@@ -8,6 +8,7 @@ import json from '@rollup/plugin-json'
 import MagicString from 'magic-string'
 import type { Plugin, RollupOptions } from 'rollup'
 import { defineConfig } from 'rollup'
+import { minify as esbuildMinifyPlugin } from 'rollup-plugin-esbuild'
 import licensePlugin from './rollupLicensePlugin'
 
 const pkg = JSON.parse(
@@ -153,13 +154,13 @@ function createNodeConfig(isProduction: boolean) {
       index: path.resolve(__dirname, 'src/node/index.ts'),
       cli: path.resolve(__dirname, 'src/node/cli.ts'),
       constants: path.resolve(__dirname, 'src/node/constants.ts'),
-      runtime: path.resolve(__dirname, 'src/node/ssr/runtime/index.ts'),
     },
     output: {
       ...sharedNodeOptions.output,
       sourcemap: !isProduction,
     },
     external: [
+      /^vite\//,
       'fsevents',
       'lightningcss',
       'rollup/parseAst',
@@ -176,6 +177,51 @@ function createNodeConfig(isProduction: boolean) {
   })
 }
 
+function createRuntimeConfig(isProduction: boolean) {
+  return defineConfig({
+    ...sharedNodeOptions,
+    input: {
+      runtime: path.resolve(__dirname, 'src/runtime/index.ts'),
+    },
+    output: {
+      ...sharedNodeOptions.output,
+      sourcemap: !isProduction,
+    },
+    external: [
+      'fsevents',
+      'lightningcss',
+      'rollup/parseAst',
+      ...Object.keys(pkg.dependencies),
+    ],
+    plugins: [
+      ...createNodePlugins(
+        false,
+        !isProduction,
+        // in production we use rollup.dts.config.ts for dts generation
+        // in development we need to rely on the rollup ts plugin
+        isProduction ? false : './dist/node',
+      ),
+      esbuildMinifyPlugin({ minify: false, minifySyntax: true }),
+      {
+        name: 'replace bias',
+        transform(code, id) {
+          if (id.includes('@jridgewell+trace-mapping')) {
+            return {
+              code: code.replaceAll(
+                'bias === LEAST_UPPER_BOUND',
+                'true' +
+                  `/*${'bias === LEAST_UPPER_BOUND'.length - '/**/'.length - 'true'.length}*/`,
+              ),
+              map: null,
+            }
+          }
+        },
+      },
+      bundleSizeLimit(45),
+    ],
+  })
+}
+
 function createCjsConfig(isProduction: boolean) {
   return defineConfig({
     ...sharedNodeOptions,
@@ -209,6 +255,7 @@ export default (commandLineArgs: any): RollupOptions[] => {
     envConfig,
     clientConfig,
     createNodeConfig(isProduction),
+    createRuntimeConfig(isProduction),
     createCjsConfig(isProduction),
   ])
 }

packages/vite/rollup.dts.config.ts

@@ -15,6 +15,7 @@ const pkg = JSON.parse(
 
 const external = [
   /^node:*/,
+  /^vite\//,
   'rollup/parseAst',
   ...Object.keys(pkg.dependencies),
   // lightningcss types are bundled
@@ -24,7 +25,7 @@ const external = [
 export default defineConfig({
   input: {
     index: './temp/node/index.d.ts',
-    runtime: './temp/node/ssr/runtime/index.d.ts',
+    runtime: './temp/runtime/index.d.ts',
   },
   output: {
     dir: './dist/node',
@@ -132,6 +133,7 @@ function validateChunkImports(this: PluginContext, chunk: RenderedChunk) {
       !id.startsWith('../') &&
       !id.startsWith('node:') &&
       !id.startsWith('types.d') &&
+      !id.startsWith('vite/') &&
       !deps.includes(id) &&
       !deps.some((name) => id.startsWith(name + '/'))
     ) {

packages/vite/src/node/__tests__/plugins/dynamicImportVar/parse.spec.ts

@@ -2,7 +2,8 @@ import { resolve } from 'node:path'
 import { fileURLToPath } from 'node:url'
 import { describe, expect, it } from 'vitest'
 import { transformDynamicImport } from '../../../plugins/dynamicImportVars'
-import { isWindows, normalizePath } from '../../../utils'
+import { normalizePath } from '../../../utils'
+import { isWindows } from '../../../../shared/utils'
 
 const __dirname = resolve(fileURLToPath(import.meta.url), '..')
 

packages/vite/src/node/__tests__/tsconfig.json

@@ -0,0 +1,8 @@
+{
+  "extends": "../../../tsconfig.base.json",
+  "compilerOptions": {
+    "esModuleInterop": true,
+    "declaration": false,
+    "resolveJsonModule": true
+  }
+}

packages/vite/src/node/__tests__/utils.spec.ts

@@ -10,11 +10,11 @@ import {
   getLocalhostAddressIfDiffersFromDNS,
   injectQuery,
   isFileReadable,
-  isWindows,
   posToNumber,
   processSrcSetSync,
   resolveHostname,
 } from '../utils'
+import { isWindows } from '../../shared/utils'
 
 describe('bareImportRE', () => {
   test('should work with normal package name', () => {

packages/vite/src/node/build.ts

@@ -21,6 +21,12 @@ import commonjsPlugin from '@rollup/plugin-commonjs'
 import type { RollupCommonJSOptions } from 'dep-types/commonjs'
 import type { RollupDynamicImportVarsOptions } from 'dep-types/dynamicImportVars'
 import type { TransformOptions } from 'esbuild'
+import { withTrailingSlash } from '../shared/utils'
+import {
+  DEFAULT_ASSETS_INLINE_LIMIT,
+  ESBUILD_MODULES_TARGET,
+  VERSION,
+} from './constants'
 import type { InlineConfig, ResolvedConfig } from './config'
 import { resolveConfig } from './config'
 import { buildReporterPlugin } from './plugins/reporter'
@@ -35,7 +41,6 @@ import {
   joinUrlSegments,
   normalizePath,
   requireResolveFromRootWithFallback,
-  withTrailingSlash,
 } from './utils'
 import { manifestPlugin } from './plugins/manifest'
 import type { Logger } from './logger'
@@ -45,11 +50,6 @@ import { ssrManifestPlugin } from './ssr/ssrManifestPlugin'
 import { loadFallbackPlugin } from './plugins/loadFallback'
 import { findNearestPackageData } from './packages'
 import type { PackageCache } from './packages'
-import {
-  DEFAULT_ASSETS_INLINE_LIMIT,
-  ESBUILD_MODULES_TARGET,
-  VERSION,
-} from './constants'
 import { resolveChokidarOptions } from './watch'
 import { completeSystemWrapPlugin } from './plugins/completeSystemWrap'
 import { mergeConfig } from './publicUtils'

packages/vite/src/node/cli.ts

@@ -3,12 +3,12 @@ import fs from 'node:fs'
 import { performance } from 'node:perf_hooks'
 import { cac } from 'cac'
 import colors from 'picocolors'
+import { VERSION } from './constants'
 import type { BuildOptions } from './build'
 import type { ServerOptions } from './server'
 import type { CLIShortcut } from './shortcuts'
 import type { LogLevel } from './logger'
 import { createLogger } from './logger'
-import { VERSION } from './constants'
 import { resolveConfig } from './config'
 
 const cli = cac('vite')