fix: externalize before caching (#9077)

Author: sheremet-va

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

@@ -34,7 +34,7 @@ export class FileSystemModuleCache {
   private rootCache: string
   private metadataFilePath: string
 
-  private version = '1.0.0-beta.1'
+  private version = '1.0.0-beta.2'
   private fsCacheRoots = new WeakMap<ResolvedConfig, string>()
   private fsEnvironmentHashMap = new WeakMap<DevEnvironment, string>()
   private fsCacheKeyGenerators = new Set<CacheKeyIdGenerator>()

packages/vitest/src/node/environments/fetchModule.ts

@@ -66,6 +66,11 @@ class ModuleFetcher {
       return { cache: true }
     }
 
+    const externalize = await this.resolver.shouldExternalize(moduleGraphModule.id!)
+    if (externalize) {
+      return { externalize, type: 'module' }
+    }
+
     const cachePath = await this.getCachePath(
       environment,
       moduleGraphModule,
@@ -271,11 +276,6 @@ class ModuleFetcher {
     moduleGraphModule: EnvironmentModuleNode,
     options?: FetchFunctionOptions,
   ): Promise<FetchResult> {
-    const externalize = await this.resolver.shouldExternalize(moduleGraphModule.id!)
-    if (externalize) {
-      return { externalize, type: 'module' }
-    }
-
     const moduleRunnerModule = await fetchModule(
       environment,
       url,