fix: externalize before caching (#9077)

sheremet-va · web-flow · commit e1b2e086a40c · 2025-11-24T16:29:28.000+01:00
diff --git a/packages/vitest/src/node/cache/fsModuleCache.ts b/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>()
diff --git a/packages/vitest/src/node/environments/fetchModule.ts b/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,

-Original file line number
+Diff line change
 privaterootCache: string
 privatemetadataFilePath: string
 -privateversion='1.0.0-beta.1'
 +privateversion='1.0.0-beta.2'
 privatefsCacheRoots=newWeakMap<ResolvedConfig,string>()
 privatefsEnvironmentHashMap=newWeakMap<DevEnvironment,string>()
 privatefsCacheKeyGenerators=newSet<CacheKeyIdGenerator>()
-Original file line number
+Diff line change
 return{cache: true}
+}
 +constexternalize=awaitthis.resolver.shouldExternalize(moduleGraphModule.id!)
 +if(externalize){
 +return{ externalize,type: 'module'}
 +}
++
 constcachePath=awaitthis.getCachePath(
 environment,
 moduleGraphModule,
 moduleGraphModule: EnvironmentModuleNode,
 options?: FetchFunctionOptions,
 ): Promise<FetchResult>{
 -constexternalize=awaitthis.resolver.shouldExternalize(moduleGraphModule.id!)
 -if(externalize){
 -return{ externalize,type: 'module'}
 -}
+-
 constmoduleRunnerModule=awaitfetchModule(
 environment,
 url,