refactor: make a distinction between node and runtime types (#6214)

Author: sheremet-va

package.json

@@ -67,6 +67,7 @@
   },
   "pnpm": {
     "overrides": {
+      "acorn": "8.11.3",
       "mlly": "^1.7.1",
       "rollup": "$rollup",
       "vite": "$vite",

packages/vitest/LICENSE.md

@@ -438,6 +438,27 @@ License: MIT
 By: Mathias Bynens
 Repository: https://github.com/mathiasbynens/emoji-regex.git
 
+> Copyright Mathias Bynens <https://mathiasbynens.be/>
+>
+> Permission is hereby granted, free of charge, to any person obtaining
+> a copy of this software and associated documentation files (the
+> "Software"), to deal in the Software without restriction, including
+> without limitation the rights to use, copy, modify, merge, publish,
+> distribute, sublicense, and/or sell copies of the Software, and to
+> permit persons to whom the Software is furnished to do so, subject to
+> the following conditions:
+>
+> The above copyright notice and this permission notice shall be
+> included in all copies or substantial portions of the Software.
+>
+> THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+> EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+> MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+> NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+> LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+> OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+> WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+
 ---------------------------------------
 
 ## expect-type

packages/vitest/rollup.config.js

@@ -1,7 +1,7 @@
 import fs from 'node:fs'
 import { builtinModules, createRequire } from 'node:module'
 import { fileURLToPath } from 'node:url'
-import { dirname, join, normalize, relative, resolve } from 'pathe'
+import { dirname, join, normalize, resolve } from 'pathe'
 import esbuild from 'rollup-plugin-esbuild'
 import dts from 'rollup-plugin-dts'
 import nodeResolve from '@rollup/plugin-node-resolve'
@@ -17,13 +17,13 @@ const pkg = require('./package.json')
 
 const entries = {
   'path': 'src/paths.ts',
-  'index': 'src/index.ts',
+  'index': 'src/public/index.ts',
   'cli': 'src/node/cli.ts',
-  'node': 'src/node.ts',
+  'node': 'src/public/node.ts',
   'suite': 'src/suite.ts',
   'browser': 'src/browser.ts',
   'runners': 'src/runners.ts',
-  'environments': 'src/environments.ts',
+  'environments': 'src/public/environments.ts',
   'spy': 'src/integrations/spy.ts',
   'coverage': 'src/coverage.ts',
   'utils': 'src/public/utils.ts',
@@ -45,9 +45,9 @@ const entries = {
 }
 
 const dtsEntries = {
-  index: 'src/index.ts',
-  node: 'src/node.ts',
-  environments: 'src/environments.ts',
+  index: 'src/public/index.ts',
+  node: 'src/public/node.ts',
+  environments: 'src/public/environments.ts',
   browser: 'src/browser.ts',
   runners: 'src/runners.ts',
   suite: 'src/suite.ts',
@@ -107,35 +107,7 @@ export default ({ watch }) =>
       output: {
         dir: 'dist',
         format: 'esm',
-        chunkFileNames: (chunkInfo) => {
-          let id
-            = chunkInfo.facadeModuleId
-            || Object.keys(chunkInfo.moduleIds).find(
-              i =>
-                !i.includes('node_modules')
-                && (i.includes('src/') || i.includes('src\\')),
-            )
-          if (id) {
-            id = normalize(id)
-            const parts = Array.from(
-              new Set(
-                relative(process.cwd(), id)
-                  .split(/\//g)
-                  .map(i => i.replace(/\..*$/, ''))
-                  .filter(
-                    i =>
-                      !['src', 'index', 'dist', 'node_modules'].some(j =>
-                        i.includes(j),
-                      ) && i.match(/^[\w-]+$/),
-                  ),
-              ),
-            )
-            if (parts.length) {
-              return `chunks/${parts.slice(-2).join('-')}.[hash].js`
-            }
-          }
-          return 'vendor/[name].[hash].js'
-        },
+        chunkFileNames: 'chunks/[name].[hash].js',
       },
       external,
       plugins: [...plugins, !watch && licensePlugin()],
@@ -163,6 +135,7 @@ export default ({ watch }) =>
         entryFileNames: chunk =>
           `${normalize(chunk.name).replace('src/', '')}.d.ts`,
         format: 'esm',
+        chunkFileNames: 'chunks/[name].[hash].d.ts',
       },
       external,
       plugins: [dts({ respectExternal: true })],

packages/vitest/src/api/setup.ts

@@ -5,19 +5,14 @@ import { parse, stringify } from 'flatted'
 import type { WebSocket } from 'ws'
 import { WebSocketServer } from 'ws'
 import type { ViteDevServer } from 'vite'
+import type { File, TaskResultPack } from '@vitest/runner'
 import { API_PATH } from '../constants'
-import type { Vitest } from '../node'
-import type {
-  Awaitable,
-  File,
-  ModuleGraphData,
-  Reporter,
-  SerializableSpec,
-  TaskResultPack,
-  UserConsoleLog,
-} from '../types'
+import type { Vitest } from '../node/core'
+import type { Awaitable, ModuleGraphData, UserConsoleLog } from '../types/general'
+import type { Reporter } from '../node/types/reporter'
 import { getModuleGraph, isPrimitive, noop, stringifyReplace } from '../utils'
 import { parseErrorStacktrace } from '../utils/source-map'
+import type { SerializedSpec } from '../runtime/types/utils'
 import type {
   TransformResultWithSource,
   WebSocketEvents,
@@ -165,7 +160,7 @@ export class WebSocketReporter implements Reporter {
     })
   }
 
-  onSpecsCollected(specs?: SerializableSpec[] | undefined): Awaitable<void> {
+  onSpecsCollected(specs?: SerializedSpec[] | undefined): Awaitable<void> {
     if (this.clients.size === 0) {
       return
     }

packages/vitest/src/api/types.ts

@@ -1,23 +1,36 @@
-import type { TransformResult } from 'vite'
 import type { BirpcReturn } from 'birpc'
-import type {
-  File,
-  ModuleGraphData,
-  Reporter,
-  SerializableSpec,
-  SerializedConfig,
-  TaskResultPack,
-} from '../types'
+import type { File, TaskResultPack } from '@vitest/runner'
+import type { Awaitable, ModuleGraphData, UserConsoleLog } from '../types/general'
+import type { SerializedConfig } from '../runtime/config'
+import type { SerializedSpec } from '../runtime/types/utils'
 
-export interface TransformResultWithSource extends TransformResult {
+interface SourceMap {
+  file: string
+  mappings: string
+  names: string[]
+  sources: string[]
+  sourcesContent?: string[]
+  version: number
+  toString: () => string
+  toUrl: () => string
+}
+
+export interface TransformResultWithSource {
+  code: string
+  map: SourceMap | {
+    mappings: ''
+  } | null
+  etag?: string
+  deps?: string[]
+  dynamicDeps?: string[]
   source?: string
 }
 
 export interface WebSocketHandlers {
   onCollected: (files?: File[]) => Promise<void>
   onTaskUpdate: (packs: TaskResultPack[]) => void
   getFiles: () => File[]
-  getTestFiles: () => Promise<SerializableSpec[]>
+  getTestFiles: () => Promise<SerializedSpec[]>
   getPaths: () => string[]
   getConfig: () => SerializedConfig
   getModuleGraph: (
@@ -37,16 +50,17 @@ export interface WebSocketHandlers {
   getUnhandledErrors: () => unknown[]
 }
 
-export interface WebSocketEvents
-  extends Pick<
-    Reporter,
-    | 'onCollected'
-    | 'onFinished'
-    | 'onTaskUpdate'
-    | 'onUserConsoleLog'
-    | 'onPathsCollected'
-    | 'onSpecsCollected'
-  > {
+export interface WebSocketEvents {
+  onCollected?: (files?: File[]) => Awaitable<void>
+  onFinished?: (
+    files: File[],
+    errors: unknown[],
+    coverage?: unknown
+  ) => Awaitable<void>
+  onTaskUpdate?: (packs: TaskResultPack[]) => Awaitable<void>
+  onUserConsoleLog?: (log: UserConsoleLog) => Awaitable<void>
+  onPathsCollected?: (paths?: string[]) => Awaitable<void>
+  onSpecsCollected?: (specs?: SerializedSpec[]) => Awaitable<void>
   onFinishedReportCoverage: () => void
 }
 

packages/vitest/src/config.ts

@@ -1,5 +1,7 @@
+import './node/types/vite'
+
 import type { ConfigEnv, UserConfig as ViteUserConfig } from 'vite'
-import type { ProjectConfig } from './types'
+import type { ProjectConfig } from './node/types/config'
 
 export interface UserWorkspaceConfig extends ViteUserConfig {
   test?: ProjectConfig

packages/vitest/src/create/browser/creator.ts

@@ -7,7 +7,7 @@ import type { Agent } from '@antfu/install-pkg'
 import { detectPackageManager, installPackage } from '@antfu/install-pkg'
 import { findUp } from 'find-up'
 import { execa } from 'execa'
-import type { BrowserBuiltinProvider } from '../../types/browser'
+import type { BrowserBuiltinProvider } from '../../node/types/browser'
 import { configFiles } from '../../constants'
 import { generateExampleFiles } from './examples'
 

packages/vitest/src/defaults.ts

@@ -4,7 +4,7 @@ import type {
   CoverageV8Options,
   ResolvedCoverageOptions,
   UserConfig,
-} from './types'
+} from './node/types/config'
 import { isCI } from './utils/env'
 
 export { defaultBrowserPort } from './constants'