feat!: remove transformMode option

Author: sheremet-va

docs/config/index.md

@@ -1117,42 +1117,6 @@ Will call [`vi.unstubAllEnvs`](/api/vi#vi-unstuballenvs) before each test.
 
 Will call [`vi.unstubAllGlobals`](/api/vi#vi-unstuballglobals) before each test.
 
-### transformMode
-
-- **Type:** `{ web?, ssr? }`
-
-Determine the transform method of modules
-
-#### transformMode.ssr
-
-- **Type:** `RegExp[]`
-- **Default:** `[/\.([cm]?[jt]sx?|json)$/]`
-
-Use SSR transform pipeline for the specified files.<br>
-Vite plugins will receive `ssr: true` flag when processing those files.
-
-#### transformMode&#46;web
-
-- **Type:** `RegExp[]`
-- **Default:** *modules other than those specified in `transformMode.ssr`*
-
-First do a normal transform pipeline (targeting browser), then do a SSR rewrite to run the code in Node.<br>
-Vite plugins will receive `ssr: false` flag when processing those files.
-
-When you use JSX as component models other than React (e.g. Vue JSX or SolidJS), you might want to config as following to make `.tsx` / `.jsx` transformed as client-side components:
-
-```ts
-import { defineConfig } from 'vitest/config'
-
-export default defineConfig({
-  test: {
-    transformMode: {
-      web: [/\.[jt]sx$/],
-    },
-  },
-})
-```
-
 ### snapshotFormat<NonProjectOption />
 
 - **Type:** `PrettyFormatOptions`

examples/solid/vite.config.mjs

@@ -7,9 +7,6 @@ import solid from 'vite-plugin-solid'
 export default defineConfig({
   test: {
     environment: 'jsdom',
-    transformMode: {
-      web: [/.[jt]sx?/],
-    },
     threads: false,
     isolate: false,
   },

examples/vue-jsx/vite.config.ts

@@ -7,8 +7,5 @@ export default defineConfig({
   test: {
     globals: true,
     environment: 'jsdom',
-    transformMode: {
-      web: [/.[tj]sx$/],
-    },
   },
 })

packages/vitest/src/integrations/env/edge-runtime.ts

@@ -4,6 +4,7 @@ import { populateGlobal } from './utils'
 
 export default <Environment>({
   name: 'edge-runtime',
+  transformMode: 'ssr',
   async setup(global) {
     const { EdgeVM } = await importModule('@edge-runtime/vm') as typeof import('@edge-runtime/vm')
     const vm = new EdgeVM({

packages/vitest/src/integrations/env/happy-dom.ts

@@ -4,6 +4,7 @@ import { populateGlobal } from './utils'
 
 export default <Environment>({
   name: 'happy-dom',
+  transformMode: 'web',
   async setup(global) {
     // happy-dom v3 introduced a breaking change to Window, but
     // provides GlobalWindow as a way to use previous behaviour

packages/vitest/src/integrations/env/index.ts

@@ -1,4 +1,6 @@
-import type { VitestEnvironment } from '../../types/config'
+import type { BuiltinEnvironment, VitestEnvironment } from '../../types/config'
+import type { VitestExecutor } from '../../node'
+import type { Environment } from '../../types'
 import node from './node'
 import jsdom from './jsdom'
 import happy from './happy-dom'
@@ -19,10 +21,28 @@ export const envPackageNames: Record<Exclude<keyof typeof environments, 'node'>,
   'edge-runtime': '@edge-runtime/vm',
 }
 
+function isBuiltinEnvironment(env: VitestEnvironment): env is BuiltinEnvironment {
+  return env in environments
+}
+
 export function getEnvPackageName(env: VitestEnvironment) {
   if (env === 'node')
     return null
   if (env in envPackageNames)
     return (envPackageNames as any)[env]
   return `vitest-environment-${env}`
 }
+
+export async function loadEnvironment(name: VitestEnvironment, executor: VitestExecutor): Promise<Environment> {
+  if (isBuiltinEnvironment(name))
+    return environments[name]
+  const packageId = (name[0] === '.' || name[0] === '/') ? name : `vitest-environment-${name}`
+  const pkg = await executor.executeId(packageId)
+  if (!pkg || !pkg.default || typeof pkg.default !== 'object' || typeof pkg.default.setup !== 'function') {
+    throw new Error(
+      `Environment "${name}" is not a valid environment. `
+    + `Package "vitest-environment-${name}" should have default export with "setup" method.`,
+    )
+  }
+  return pkg.default
+}

packages/vitest/src/integrations/env/jsdom.ts

@@ -28,6 +28,7 @@ function catchWindowErrors(window: Window) {
 
 export default <Environment>({
   name: 'jsdom',
+  transformMode: 'web',
   async setup(global, { jsdom = {} }) {
     const {
       CookieJar,

packages/vitest/src/integrations/env/node.ts

@@ -3,6 +3,7 @@ import type { Environment } from '../../types'
 
 export default <Environment>({
   name: 'node',
+  transformMode: 'ssr',
   async setup(global) {
     global.console.Console = Console
     return {

packages/vitest/src/node/pools/rpc.ts

@@ -1,6 +1,5 @@
 import type { RawSourceMap } from 'vite-node'
 import type { RuntimeRPC } from '../../types'
-import { getEnvironmentTransformMode } from '../../utils/base'
 import type { WorkspaceProject } from '../workspace'
 
 export function createMethodsRPC(project: WorkspaceProject): RuntimeRPC {
@@ -25,12 +24,10 @@ export function createMethodsRPC(project: WorkspaceProject): RuntimeRPC {
       const r = await project.vitenode.transformRequest(id)
       return r?.map as RawSourceMap | undefined
     },
-    fetch(id, environment) {
-      const transformMode = getEnvironmentTransformMode(project.config, environment)
+    fetch(id, transformMode) {
       return project.vitenode.fetchModule(id, transformMode)
     },
-    resolveId(id, importer, environment) {
-      const transformMode = getEnvironmentTransformMode(project.config, environment)
+    resolveId(id, importer, transformMode) {
       return project.vitenode.resolveId(id, importer, transformMode)
     },
     onPathsCollected(paths) {

packages/vitest/src/runtime/child.ts

@@ -10,7 +10,7 @@ import { rpcDone } from './rpc'
 import { setupInspect } from './inspector'
 
 function init(ctx: ChildContext) {
-  const { config } = ctx
+  const { config, environment } = ctx
 
   process.env.VITEST_WORKER_ID = '1'
   process.env.VITEST_POOL_ID = '1'
@@ -21,7 +21,7 @@ function init(ctx: ChildContext) {
   })
 
   // @ts-expect-error untyped global
-  globalThis.__vitest_environment__ = config.environment
+  globalThis.__vitest_environment__ = environment.name
   // @ts-expect-error I know what I am doing :P
   globalThis.__vitest_worker__ = {
     ctx,
@@ -76,8 +76,8 @@ export async function run(ctx: ChildContext) {
 
   try {
     init(ctx)
-    const { run, executor } = await startViteNode(ctx)
-    await run(ctx.files, ctx.config, ctx.environment, executor)
+    const { run, executor, environment } = await startViteNode(ctx)
+    await run(ctx.files, ctx.config, { ...ctx.environment, environment }, executor)
     await rpcDone()
   }
   finally {

packages/vitest/src/runtime/entry.ts

@@ -2,7 +2,7 @@ import { performance } from 'node:perf_hooks'
 import type { VitestRunner, VitestRunnerConstructor } from '@vitest/runner'
 import { startTests } from '@vitest/runner'
 import { resolve } from 'pathe'
-import type { ContextTestEnvironment, ResolvedConfig } from '../types'
+import type { ResolvedConfig, ResolvedTestEnvironment } from '../types'
 import { getWorkerState, resetModules } from '../utils'
 import { vi } from '../integrations/vi'
 import { distDir } from '../paths'
@@ -89,7 +89,7 @@ async function getTestRunner(config: ResolvedConfig, executor: VitestExecutor):
 }
 
 // browser shouldn't call this!
-export async function run(files: string[], config: ResolvedConfig, environment: ContextTestEnvironment, executor: VitestExecutor): Promise<void> {
+export async function run(files: string[], config: ResolvedConfig, environment: ResolvedTestEnvironment, executor: VitestExecutor): Promise<void> {
   const workerState = getWorkerState()
 
   await setupGlobalEnv(config)
@@ -104,11 +104,11 @@ export async function run(files: string[], config: ResolvedConfig, environment:
   workerState.durations.prepare = performance.now() - workerState.durations.prepare
 
   // @ts-expect-error untyped global
-  globalThis.__vitest_environment__ = environment
+  globalThis.__vitest_environment__ = environment.name
 
   workerState.durations.environment = performance.now()
 
-  await withEnv(environment.name, environment.options || config.environmentOptions || {}, executor, async () => {
+  await withEnv(environment, environment.options || config.environmentOptions || {}, async () => {
     workerState.durations.environment = performance.now() - workerState.durations.environment
 
     for (const file of files) {

packages/vitest/src/runtime/execute.ts

@@ -6,8 +6,9 @@ import { normalize, relative, resolve } from 'pathe'
 import { processError } from '@vitest/runner/utils'
 import type { MockMap } from '../types/mocker'
 import { getCurrentEnvironment, getWorkerState } from '../utils/global'
-import type { ContextRPC, ContextTestEnvironment, ResolvedConfig } from '../types'
+import type { ContextRPC, Environment, ResolvedConfig, ResolvedTestEnvironment } from '../types'
 import { distDir } from '../paths'
+import { loadEnvironment } from '../integrations/env'
 import { VitestMocker } from './mocker'
 import { rpc } from './rpc'
 
@@ -25,8 +26,9 @@ export async function createVitestExecutor(options: ExecuteOptions) {
 }
 
 let _viteNode: {
-  run: (files: string[], config: ResolvedConfig, environment: ContextTestEnvironment, executor: VitestExecutor) => Promise<void>
+  run: (files: string[], config: ResolvedConfig, environment: ResolvedTestEnvironment, executor: VitestExecutor) => Promise<void>
   executor: VitestExecutor
+  environment: Environment
 }
 
 export const moduleCache = new ModuleCacheMap()
@@ -61,12 +63,14 @@ export async function startViteNode(ctx: ContextRPC) {
   process.on('uncaughtException', e => catchError(e, 'Uncaught Exception'))
   process.on('unhandledRejection', e => catchError(e, 'Unhandled Rejection'))
 
+  let transformMode: 'ssr' | 'web' = 'ssr'
+
   const executor = await createVitestExecutor({
     fetchModule(id) {
-      return rpc().fetch(id, ctx.environment.name)
+      return rpc().fetch(id, transformMode)
     },
     resolveId(id, importer) {
-      return rpc().resolveId(id, importer, ctx.environment.name)
+      return rpc().resolveId(id, importer, transformMode)
     },
     moduleCache,
     mockMap,
@@ -76,9 +80,12 @@ export async function startViteNode(ctx: ContextRPC) {
     base: config.base,
   })
 
+  const environment = await loadEnvironment(ctx.environment.name, executor)
+  transformMode = environment.transformMode ?? 'ssr'
+
   const { run } = await import(pathToFileURL(resolve(distDir, 'entry.js')).href)
 
-  _viteNode = { run, executor }
+  _viteNode = { run, executor, environment }
 
   return _viteNode
 }

packages/vitest/src/runtime/setup.node.ts

@@ -2,16 +2,14 @@ import { createRequire } from 'node:module'
 import { isatty } from 'node:tty'
 import { installSourcemapsSupport } from 'vite-node/source-map'
 import { createColors, setupColors } from '@vitest/utils'
-import { environments } from '../integrations/env'
-import type { Environment, ResolvedConfig } from '../types'
+import type { EnvironmentOptions, ResolvedConfig, ResolvedTestEnvironment } from '../types'
 import { VitestSnapshotEnvironment } from '../integrations/snapshot/environments/node'
 import { getSafeTimers, getWorkerState } from '../utils'
 import * as VitestIndex from '../index'
 import { RealDate } from '../integrations/mock/date'
 import { expect } from '../integrations/chai'
 import { rpc } from './rpc'
 import { setupCommonEnv } from './setup.common'
-import type { VitestExecutor } from './execute'
 
 // this should only be used in Node
 let globalSetup = false
@@ -157,30 +155,17 @@ export async function setupConsoleLogSpy() {
   })
 }
 
-async function loadEnvironment(name: string, executor: VitestExecutor) {
-  const pkg = await executor.executeId(`vitest-environment-${name}`)
-  if (!pkg || !pkg.default || typeof pkg.default !== 'object' || typeof pkg.default.setup !== 'function') {
-    throw new Error(
-      `Environment "${name}" is not a valid environment. `
-    + `Package "vitest-environment-${name}" should have default export with "setup" method.`,
-    )
-  }
-  return pkg.default
-}
-
 export async function withEnv(
-  name: ResolvedConfig['environment'],
-  options: ResolvedConfig['environmentOptions'],
-  executor: VitestExecutor,
+  { environment, name }: ResolvedTestEnvironment,
+  options: EnvironmentOptions,
   fn: () => Promise<void>,
 ) {
-  const config: Environment = (environments as any)[name] || await loadEnvironment(name, executor)
   // @ts-expect-error untyped global
-  globalThis.__vitest_environment__ = config.name || name
+  globalThis.__vitest_environment__ = name
   expect.setState({
-    environment: config.name || name || 'node',
+    environment: name,
   })
-  const env = await config.setup(globalThis, options)
+  const env = await environment.setup(globalThis, options)
   try {
     await fn()
   }

packages/vitest/src/runtime/worker.ts

@@ -24,7 +24,7 @@ function init(ctx: WorkerContext) {
   })
 
   // @ts-expect-error untyped global
-  globalThis.__vitest_environment__ = config.environment
+  globalThis.__vitest_environment__ = config.environment.name
   // @ts-expect-error I know what I am doing :P
   globalThis.__vitest_worker__ = {
     ctx,
@@ -62,8 +62,8 @@ export async function run(ctx: WorkerContext) {
 
   try {
     init(ctx)
-    const { run, executor } = await startViteNode(ctx)
-    await run(ctx.files, ctx.config, ctx.environment, executor)
+    const { run, executor, environment } = await startViteNode(ctx)
+    await run(ctx.files, ctx.config, { ...ctx.environment, environment }, executor)
     await rpcDone()
   }
   finally {

packages/vitest/src/types/config.ts

@@ -443,27 +443,6 @@ export interface InlineConfig {
    */
   uiBase?: string
 
-  /**
-   * Determine the transform method of modules
-   */
-  transformMode?: {
-    /**
-     * Use SSR transform pipeline for the specified files.
-     * Vite plugins will receive `ssr: true` flag when processing those files.
-     *
-     * @default [/\.([cm]?[jt]sx?|json)$/]
-     */
-    ssr?: RegExp[]
-    /**
-     * First do a normal transform pipeline (targeting browser),
-     * then then do a SSR rewrite to run the code in Node.
-     * Vite plugins will receive `ssr: false` flag when processing those files.
-     *
-     * @default other than `ssr`
-     */
-    web?: RegExp[]
-  }
-
   /**
    * Format options for snapshot testing.
    */

packages/vitest/src/types/general.ts

@@ -23,6 +23,7 @@ export interface EnvironmentReturn {
 
 export interface Environment {
   name: string
+  transformMode?: 'web' | 'ssr'
   setup(global: any, options: Record<string, any>): Awaitable<EnvironmentReturn>
 }