fix(typecheck): avoid creating a temporary tsconfig file when typechecking (#7967)

Author: sheremet-va

docs/advanced/api/test-project.md

@@ -90,6 +90,12 @@ vitest.config === vitest.projects[0].globalConfig
 
 This is the project's resolved test config.
 
+## hash <Version>3.2.0</Version> {#hash}
+
+The unique hash of this project. This value is consistent between the reruns.
+
+It is based on the root of the project and its name. Note that the root path is not consistent between different OS, so the hash will also be different.
+
 ## vite
 
 This is project's [`ViteDevServer`](https://vite.dev/guide/api-javascript#vitedevserver). All projects have their own Vite servers.

packages/vitest/LICENSE.md

@@ -268,35 +268,6 @@ Repository: git+https://github.com/WebReflection/flatted.git
 
 ---------------------------------------
 
-## get-tsconfig
-License: MIT
-By: Hiroki Osame
-Repository: privatenumber/get-tsconfig
-
-> MIT License
->
-> Copyright (c) Hiroki Osame <[email protected]>
->
-> 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.
-
----------------------------------------
-
 ## js-tokens
 License: MIT
 By: Simon Lydell
@@ -550,35 +521,6 @@ Repository: git+https://github.com/quansync-dev/quansync.git
 
 ---------------------------------------
 
-## resolve-pkg-maps
-License: MIT
-By: Hiroki Osame
-Repository: privatenumber/resolve-pkg-maps
-
-> MIT License
->
-> Copyright (c) Hiroki Osame <[email protected]>
->
-> 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.
-
----------------------------------------
-
 ## sisteransi
 License: MIT
 By: Terkel Gjervig

packages/vitest/package.json

@@ -198,7 +198,6 @@
     "chai-subset": "^1.6.0",
     "find-up": "^6.3.0",
     "flatted": "catalog:",
-    "get-tsconfig": "^4.10.0",
     "happy-dom": "^17.4.4",
     "jsdom": "^26.1.0",
     "local-pkg": "^1.1.1",

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

@@ -9,7 +9,7 @@ import { createDefer } from '@vitest/utils'
 import { Typechecker } from '../../typecheck/typechecker'
 import { groupBy } from '../../utils/base'
 
-export function createTypecheckPool(ctx: Vitest): ProcessPool {
+export function createTypecheckPool(vitest: Vitest): ProcessPool {
   const promisesMap = new WeakMap<TestProject, DeferPromise<void>>()
   const rerunTriggered = new WeakSet<TestProject>()
 
@@ -20,11 +20,11 @@ export function createTypecheckPool(ctx: Vitest): ProcessPool {
     const checker = project.typechecker!
 
     const { packs, events } = checker.getTestPacksAndEvents()
-    await ctx._testRun.updated(packs, events)
+    await vitest._testRun.updated(packs, events)
 
     if (!project.config.typecheck.ignoreSourceErrors) {
       sourceErrors.forEach(error =>
-        ctx.state.catchError(error, 'Unhandled Source Error'),
+        vitest.state.catchError(error, 'Unhandled Source Error'),
       )
     }
 
@@ -33,19 +33,19 @@ export function createTypecheckPool(ctx: Vitest): ProcessPool {
     if (processError) {
       const error = new Error(checker.getOutput())
       error.stack = ''
-      ctx.state.catchError(error, 'Typecheck Error')
+      vitest.state.catchError(error, 'Typecheck Error')
     }
 
     promisesMap.get(project)?.resolve()
 
     rerunTriggered.delete(project)
 
     // triggered by TSC watcher, not Vitest watcher, so we need to emulate what Vitest does in this case
-    if (ctx.config.watch && !ctx.runningPromise) {
-      await ctx.report('onFinished', files, [])
-      await ctx.report('onWatcherStart', files, [
+    if (vitest.config.watch && !vitest.runningPromise) {
+      await vitest.report('onFinished', files, [])
+      await vitest.report('onWatcherStart', files, [
         ...(project.config.typecheck.ignoreSourceErrors ? [] : sourceErrors),
-        ...ctx.state.getUnhandledErrors(),
+        ...vitest.state.getUnhandledErrors(),
       ])
     }
   }
@@ -65,19 +65,19 @@ export function createTypecheckPool(ctx: Vitest): ProcessPool {
     checker.onParseStart(async () => {
       const files = checker.getTestFiles()
       for (const file of files) {
-        await ctx._testRun.enqueued(project, file)
+        await vitest._testRun.enqueued(project, file)
       }
-      await ctx._testRun.collected(project, files)
+      await vitest._testRun.collected(project, files)
     })
 
     checker.onParseEnd(result => onParseEnd(project, result))
 
     checker.onWatcherRerun(async () => {
       rerunTriggered.add(project)
 
-      if (!ctx.runningPromise) {
-        ctx.state.clearErrors()
-        await ctx.report(
+      if (!vitest.runningPromise) {
+        vitest.state.clearErrors()
+        await vitest.report(
           'onWatcherRerun',
           files,
           'File change detected. Triggering rerun.',
@@ -88,15 +88,14 @@ export function createTypecheckPool(ctx: Vitest): ProcessPool {
 
       const testFiles = checker.getTestFiles()
       for (const file of testFiles) {
-        await ctx._testRun.enqueued(project, file)
+        await vitest._testRun.enqueued(project, file)
       }
-      await ctx._testRun.collected(project, testFiles)
+      await vitest._testRun.collected(project, testFiles)
 
       const { packs, events } = checker.getTestPacksAndEvents()
-      await ctx._testRun.updated(packs, events)
+      await vitest._testRun.updated(packs, events)
     })
 
-    await checker.prepare()
     return checker
   }
 
@@ -118,7 +117,7 @@ export function createTypecheckPool(ctx: Vitest): ProcessPool {
       checker.setFiles(files)
       await checker.collectTests()
       const testFiles = checker.getTestFiles()
-      ctx.state.collectFiles(project, testFiles)
+      vitest.state.collectFiles(project, testFiles)
     }
   }
 
@@ -147,9 +146,9 @@ export function createTypecheckPool(ctx: Vitest): ProcessPool {
       if (project.typechecker && !triggered) {
         const testFiles = project.typechecker.getTestFiles()
         for (const file of testFiles) {
-          await ctx._testRun.enqueued(project, file)
+          await vitest._testRun.enqueued(project, file)
         }
-        await ctx._testRun.collected(project, testFiles)
+        await vitest._testRun.collected(project, testFiles)
         await onParseEnd(project, project.typechecker.getResult())
         continue
       }
@@ -166,7 +165,7 @@ export function createTypecheckPool(ctx: Vitest): ProcessPool {
     runTests,
     collectTests,
     async close() {
-      const promises = ctx.projects.map(project =>
+      const promises = vitest.projects.map(project =>
         project.typechecker?.stop(),
       )
       await Promise.all(promises)

packages/vitest/src/node/project.ts

@@ -70,6 +70,7 @@ export class TestProject {
   /** @internal */ typechecker?: Typechecker
   /** @internal */ _config?: ResolvedConfig
   /** @internal */ _vite?: ViteDevServer
+  /** @internal */ _hash?: string
 
   private runner!: ViteNodeRunner
 
@@ -92,6 +93,18 @@ export class TestProject {
     this.globalConfig = vitest.config
   }
 
+  /**
+   * The unique hash of this project. This value is consistent between the reruns.
+   *
+   * It is based on the root of the project (not consistent between OS) and its name.
+   */
+  public get hash(): string {
+    if (!this._hash) {
+      throw new Error('The server was not set. It means that `project.hash` was called before the Vite server was established.')
+    }
+    return this._hash
+  }
+
   // "provide" is a property, not a method to keep the context when destructed in the global setup,
   // making it a method would be a breaking change, and can be done in Vitest 3 at minimum
   /**
@@ -601,6 +614,12 @@ export class TestProject {
     return this._configureServer(options, server)
   }
 
+  private _setHash() {
+    this._hash = generateHash(
+      this._config!.root + this._config!.name,
+    )
+  }
+
   /** @internal */
   async _configureServer(options: UserConfig, server: ViteDevServer): Promise<void> {
     this._config = resolveConfig(
@@ -611,6 +630,7 @@ export class TestProject {
       },
       server.config,
     )
+    this._setHash()
     for (const _providedKey in this.config.provide) {
       const providedKey = _providedKey as keyof ProvidedContext
       // type is very strict here, so we cast it to any
@@ -699,6 +719,7 @@ export class TestProject {
     project.runner = vitest.runner
     project._vite = vitest.server
     project._config = vitest.config
+    project._setHash()
     project._provideObject(vitest.config.provide)
     return project
   }
@@ -713,6 +734,7 @@ export class TestProject {
     clone.runner = parent.runner
     clone._vite = parent._vite
     clone._config = config
+    clone._setHash()
     clone._parent = parent
     clone._provideObject(config.provide)
     return clone
@@ -771,3 +793,16 @@ export async function initializeProject(
 
   return project
 }
+
+function generateHash(str: string): string {
+  let hash = 0
+  if (str.length === 0) {
+    return `${hash}`
+  }
+  for (let i = 0; i < str.length; i++) {
+    const char = str.charCodeAt(i)
+    hash = (hash << 5) - hash + char
+    hash = hash & hash // Convert to 32bit integer
+  }
+  return `${hash}`
+}

packages/vitest/src/typecheck/parse.ts

@@ -1,12 +1,5 @@
-import type { TypecheckConfig } from '../node/types/config'
 import type { RawErrsMap, TscErrorInfo } from './types'
-import { writeFile } from 'node:fs/promises'
-import os from 'node:os'
-import url from 'node:url'
-import { getTsconfig as getTsconfigContent } from 'get-tsconfig'
-import { basename, dirname, join, resolve } from 'pathe'
 
-const __dirname = url.fileURLToPath(new URL('.', import.meta.url))
 const newLineRegExp = /\r?\n/
 const errCodeRegExp = /error TS(?<errCode>\d+)/
 
@@ -63,50 +56,6 @@ export async function makeTscErrorInfo(
   ]
 }
 
-export async function getTsconfig(root: string, config: TypecheckConfig): Promise<{
-  path: string
-  config: Record<string, any>
-}> {
-  const configName = config.tsconfig ? basename(config.tsconfig) : undefined
-  const configSearchPath = config.tsconfig
-    ? dirname(resolve(root, config.tsconfig))
-    : root
-
-  const tsconfig = getTsconfigContent(configSearchPath, configName)
-
-  if (!tsconfig) {
-    throw new Error('no tsconfig.json found')
-  }
-
-  const tsconfigName = basename(tsconfig.path, '.json')
-  const tempTsConfigName = `${tsconfigName}.vitest-temp.json`
-  const tempTsbuildinfoName = `${tsconfigName}.tmp.tsbuildinfo`
-
-  const tempConfigPath = join(
-    dirname(tsconfig.path),
-    tempTsConfigName,
-  )
-
-  try {
-    const tmpTsConfig: Record<string, any> = { ...tsconfig.config }
-
-    tmpTsConfig.compilerOptions = tmpTsConfig.compilerOptions || {}
-    tmpTsConfig.compilerOptions.emitDeclarationOnly = false
-    tmpTsConfig.compilerOptions.incremental = true
-    tmpTsConfig.compilerOptions.tsBuildInfoFile = join(
-      process.versions.pnp ? join(os.tmpdir(), 'vitest') : __dirname,
-      tempTsbuildinfoName,
-    )
-
-    const tsconfigFinalContent = JSON.stringify(tmpTsConfig, null, 2)
-    await writeFile(tempConfigPath, tsconfigFinalContent)
-    return { path: tempConfigPath, config: tmpTsConfig }
-  }
-  catch (err) {
-    throw new Error(`failed to write ${tempTsConfigName}`, { cause: err })
-  }
-}
-
 export async function getRawErrsMapFromTsCompile(tscErrorStdout: string): Promise<RawErrsMap> {
   const rawErrsMap: RawErrsMap = new Map()