fix(coverage): remove unnecessary istanbul-lib-source-maps usage (#9344)

AriPerkkio · web-flow · commit b09405375d58 · 2025-12-26T21:29:53.000+01:00
diff --git a/packages/coverage-v8/package.json b/packages/coverage-v8/package.json
@@ -59,7 +59,6 @@
     "ast-v8-to-istanbul": "^0.3.8",
     "istanbul-lib-coverage": "catalog:",
     "istanbul-lib-report": "catalog:",
-    "istanbul-lib-source-maps": "catalog:",
     "istanbul-reports": "catalog:",
     "magicast": "catalog:",
     "obug": "catalog:",
@@ -69,7 +68,6 @@
   "devDependencies": {
     "@types/istanbul-lib-coverage": "catalog:",
     "@types/istanbul-lib-report": "catalog:",
-    "@types/istanbul-lib-source-maps": "catalog:",
     "@types/istanbul-reports": "catalog:",
     "@vitest/browser": "workspace:*",
     "pathe": "catalog:",
diff --git a/packages/coverage-v8/src/provider.ts b/packages/coverage-v8/src/provider.ts
@@ -9,7 +9,6 @@ import { mergeProcessCovs } from '@bcoe/v8-coverage'
 import astV8ToIstanbul from 'ast-v8-to-istanbul'
 import libCoverage from 'istanbul-lib-coverage'
 import libReport from 'istanbul-lib-report'
-import libSourceMaps from 'istanbul-lib-source-maps'
 import reports from 'istanbul-reports'
 import { parseModule } from 'magicast'
 import { createDebug } from 'obug'
@@ -61,16 +60,15 @@ export class V8CoverageProvider extends BaseCoverageProvider<ResolvedCoverageOpt
         })
       },
       onFinished: async (project, environment) => {
+        // Source maps can change based on projectName and transform mode.
+        // Coverage transform re-uses source maps so we need to separate transforms from each other.
         const converted = await this.convertCoverage(
           merged,
           project,
           environment,
         )
 
-        // Source maps can change based on projectName and transform mode.
-        // Coverage transform re-uses source maps so we need to separate transforms from each other.
-        const transformedCoverage = await transformCoverage(converted)
-        coverageMap.merge(transformedCoverage)
+        coverageMap.merge(converted)
 
         merged = { result: [] }
       },
@@ -83,7 +81,7 @@ export class V8CoverageProvider extends BaseCoverageProvider<ResolvedCoverageOpt
       const coveredFiles = coverageMap.files()
       const untestedCoverage = await this.getCoverageMapForUncoveredFiles(coveredFiles)
 
-      coverageMap.merge(await transformCoverage(untestedCoverage))
+      coverageMap.merge(untestedCoverage)
     }
 
     coverageMap.filter((filename) => {
@@ -458,11 +456,6 @@ export class V8CoverageProvider extends BaseCoverageProvider<ResolvedCoverageOpt
   }
 }
 
-async function transformCoverage(coverageMap: CoverageMap) {
-  const sourceMapStore = libSourceMaps.createSourceMapStore()
-  return await sourceMapStore.transformCoverage(coverageMap)
-}
-
 /**
  * Find the function with highest `endOffset` to determine the length of the file
  */
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
