feat(config): deprecate cache.dir option

docs/config/index.md

@@ -1732,18 +1732,10 @@ Test above this limit will be queued to run when available slot appears.
 
 ### cache<NonProjectOption />
 
-- **Type**: `false | { dir? }`
+- **Type**: `false`
 - **CLI**: `--no-cache`, `--cache=false`
 
-Options to configure Vitest cache policy. At the moment Vitest stores cache for test results to run the longer and failed tests first.
-
-#### cache.dir
-
-- **Type**: `string`
-- **Default**: `node_modules/.vitest`
-- **CLI**: `--cache.dir=./cache`
-
-Path to cache directory.
+Use this option if you want to disable the cache feature. At the moment Vitest stores cache file in Vite's [cacheDir](https://vitejs.dev/config/shared-options.html#cachedir) for test results to run the longer and failed tests first.
 
 ### sequence
 

packages/vitest/src/node/cache/index.ts

@@ -5,7 +5,6 @@ import { resolve } from 'pathe'
 import { loadConfigFromFile } from 'vite'
 import { configFiles } from '../../constants'
 import type { CliOptions } from '../cli/cli-api'
-import { slash } from '../../utils'
 import { FilesStatsCache } from './files'
 import { ResultsCache } from './results'
 
@@ -21,11 +20,10 @@ export class VitestCache {
     return this.stats.getStats(key)
   }
 
-  static resolveCacheDir(root: string, dir: string | undefined, projectName: string | undefined) {
-    const baseDir = slash(dir || 'node_modules/.vitest')
+  static resolveCacheDir(dir: string, projectName: string | undefined) {
     return projectName
-      ? resolve(root, baseDir, crypto.createHash('md5').update(projectName, 'utf-8').digest('hex'))
-      : resolve(root, baseDir)
+      ? resolve(dir, crypto.createHash('md5').update(projectName, 'utf-8').digest('hex'))
+      : dir
   }
 
   static async clearCache(options: CliOptions) {
@@ -47,7 +45,7 @@ export class VitestCache {
     if (cache === false)
       throw new Error('Cache is disabled')
 
-    const cachePath = VitestCache.resolveCacheDir(root, cache?.dir, projectName)
+    const cachePath = VitestCache.resolveCacheDir(config!.cacheDir!, projectName)
 
     let cleared = false
 

packages/vitest/src/node/cache/results.ts

@@ -1,6 +1,6 @@
 import fs from 'node:fs'
 import { dirname, relative, resolve } from 'pathe'
-import type { File, ResolvedConfig } from '../../types'
+import type { File } from '../../types'
 import { version } from '../../../package.json'
 
 export interface SuiteResultCache {
@@ -19,10 +19,9 @@ export class ResultsCache {
     return this.cachePath
   }
 
-  setConfig(root: string, config: ResolvedConfig['cache']) {
+  setConfig(root: string, cacheDir: string) {
     this.root = root
-    if (config)
-      this.cachePath = resolve(config.dir, 'results.json')
+    this.cachePath = resolve(cacheDir, 'results.json')
   }
 
   getResults(key: string) {

packages/vitest/src/node/cli/cli-config.ts

@@ -538,20 +538,13 @@ export const cliOptionsConfig: VitestCLIOptions = {
   cache: {
     description: 'Enable cache',
     argument: '', // allow only boolean
-    subcommands: {
-      dir: {
-        description: 'Path to the cache directory',
-        argument: '<path>',
-        normalize: true,
-      },
-    },
     // cache can only be "false" or an object
     transform(cache) {
       if (cache)
         return {}
       return cache
     },
-  },
+  } as VitestCLIOptions['cache'],
   maxConcurrency: {
     description: 'Maximum number of concurrent tests in a suite (default: 5)',
     argument: '<number>',

packages/vitest/src/node/config.ts

@@ -7,7 +7,6 @@ import { defaultBrowserPort, defaultPort, extraInlineDeps } from '../constants'
 import { benchmarkConfigDefaults, configDefaults } from '../defaults'
 import { isCI, stdProvider, toArray } from '../utils'
 import type { BuiltinPool } from '../types/pool-options'
-import { VitestCache } from './cache'
 import { BaseSequencer } from './sequencers/BaseSequencer'
 import { RandomSequencer } from './sequencers/RandomSequencer'
 import type { BenchmarkBuiltinReporters } from './reporters'
@@ -436,9 +435,13 @@ export function resolveConfig(
     resolved.css.modules.classNameStrategy ??= 'stable'
   }
 
-  resolved.cache ??= { dir: '' }
-  if (resolved.cache)
-    resolved.cache.dir = VitestCache.resolveCacheDir(resolved.root, resolved.cache.dir, resolved.name)
+  if (resolved.cache && resolved.cache.dir) {
+    console.warn(
+      c.yellow(
+        `${c.inverse(c.yellow(' Vitest '))} "cache.dir" is deprecated. Use vite's "cacheDir" instead.`,
+      ),
+    )
+  }
 
   resolved.sequence ??= {} as any
   if (!resolved.sequence?.sequencer) {

packages/vitest/src/node/core.ts

@@ -151,7 +151,7 @@ export class Vitest {
       ? await createBenchmarkReporters(toArray(resolved.benchmark?.reporters), this.runner)
       : await createReporters(resolved.reporters, this)
 
-    this.cache.results.setConfig(resolved.root, resolved.cache)
+    this.cache.results.setConfig(resolved.root, server.config.cacheDir)
     try {
       await this.cache.results.readFromCache()
     }

packages/vitest/src/node/plugins/optimizer.ts

@@ -8,8 +8,8 @@ export function VitestOptimizer(): Plugin {
       order: 'post',
       handler(viteConfig) {
         const testConfig = viteConfig.test || {}
-        const webOptimizer = resolveOptimizerConfig(testConfig.deps?.optimizer?.web, viteConfig.optimizeDeps, testConfig)
-        const ssrOptimizer = resolveOptimizerConfig(testConfig.deps?.optimizer?.ssr, viteConfig.ssr?.optimizeDeps, testConfig)
+        const webOptimizer = resolveOptimizerConfig(testConfig.deps?.optimizer?.web, viteConfig.optimizeDeps, viteConfig)
+        const ssrOptimizer = resolveOptimizerConfig(testConfig.deps?.optimizer?.ssr, viteConfig.ssr?.optimizeDeps, viteConfig)
 
         viteConfig.cacheDir = webOptimizer.cacheDir || ssrOptimizer.cacheDir || viteConfig.cacheDir
         viteConfig.optimizeDeps = webOptimizer.optimizeDeps

packages/vitest/src/node/plugins/utils.ts

@@ -1,11 +1,11 @@
 import { searchForWorkspaceRoot, version as viteVersion } from 'vite'
 import type { DepOptimizationOptions, ResolvedConfig, UserConfig as ViteConfig } from 'vite'
 import { dirname } from 'pathe'
-import type { DepsOptimizationOptions, InlineConfig } from '../../types'
+import type { DepsOptimizationOptions } from '../../types'
 import { VitestCache } from '../cache'
 import { rootDir } from '../../paths'
 
-export function resolveOptimizerConfig(_testOptions: DepsOptimizationOptions | undefined, viteOptions: DepOptimizationOptions | undefined, testConfig: InlineConfig) {
+export function resolveOptimizerConfig(_testOptions: DepsOptimizationOptions | undefined, viteOptions: DepOptimizationOptions | undefined, viteConfig: ViteConfig) {
   const testOptions = _testOptions || {}
   const newConfig: { cacheDir?: string; optimizeDeps: DepOptimizationOptions } = {} as any
   const [major, minor, fix] = viteVersion.split('.').map(Number)
@@ -24,8 +24,6 @@ export function resolveOptimizerConfig(_testOptions: DepsOptimizationOptions | u
     }
   }
   else {
-    const root = testConfig.root ?? process.cwd()
-    const cacheDir = testConfig.cache !== false ? testConfig.cache?.dir : undefined
     const currentInclude = (testOptions.include || viteOptions?.include || [])
     const exclude = [
       'vitest',
@@ -39,7 +37,7 @@ export function resolveOptimizerConfig(_testOptions: DepsOptimizationOptions | u
 
     const include = (testOptions.include || viteOptions?.include || []).filter((n: string) => !exclude.includes(n))
 
-    newConfig.cacheDir = cacheDir ?? VitestCache.resolveCacheDir(root, cacheDir, testConfig.name)
+    newConfig.cacheDir = viteConfig.test?.cache ? VitestCache.resolveCacheDir(viteConfig.cacheDir!, viteConfig.test?.name) : undefined
     newConfig.optimizeDeps = {
       ...viteOptions,
       ...testOptions,

packages/vitest/src/types/config.ts

@@ -606,10 +606,13 @@ export interface InlineConfig {
 
   /**
    * Options for configuring cache policy.
-   * @default { dir: 'node_modules/.vitest' }
+   * @default { dir: 'node_modules/.vite' }
    */
   cache?: false | {
-    dir?: string
+    /**
+     * @deprecated Use Vite's `cacheDir` instead.
+     */
+    dir: string
   }
 
   /**
@@ -816,6 +819,9 @@ export interface ResolvedConfig extends Omit<Required<UserConfig>, 'config' | 'f
   }
 
   cache: {
+    /**
+     * @deprecated
+     */
     dir: string
   } | false
 

test/cache/vitest-custom.config.ts

@@ -1,9 +1,6 @@
 import { defineConfig } from 'vite'
 
 export default defineConfig({
-  test: {
-    cache: {
-      dir: 'cache/.vitest-custom',
-    },
-  },
+  cacheDir: 'cache/.vitest-custom',
+  test: {},
 })

test/cache/vitest.config.ts

@@ -1,10 +1,8 @@
 import { defineConfig } from 'vite'
 
 export default defineConfig({
+  cacheDir: 'cache/.vitest-base',
   test: {
     pool: 'forks',
-    cache: {
-      dir: 'cache/.vitest-base',
-    },
   },
 })

test/core/test/cli-test.test.ts

@@ -212,13 +212,7 @@ test('cache is parsed correctly', () => {
   expect(getCLIOptions('--cache')).toEqual({ cache: {} })
   expect(getCLIOptions('--no-cache')).toEqual({ cache: false })
 
-  expect(getCLIOptions('--cache.dir=./test/cache.json')).toEqual({
-    cache: { dir: 'test/cache.json' },
-  })
-  expect(getCLIOptions('--cache.dir ./test/cache.json')).toEqual({
-    cache: { dir: 'test/cache.json' },
-  })
-  expect(getCLIOptions('--cache.dir .\\test\\cache.json')).toEqual({
+  expect(getCLIOptions('--cache.dir=./test/cache.json')).not.toEqual({
     cache: { dir: 'test/cache.json' },
   })
 })