fix: sourcemap tool get modules contents (#1400)

Author: yifancong

packages/core/src/build-utils/build/module-graph/webpack/transform.ts

@@ -91,7 +91,7 @@ function appendDependency(
   const request = rawRequest ?? resolveRequest;
 
   if (!module.getDependencyByRequest(request)) {
-    const depModule = graph.getModuleByFile(resolveRequest);
+    const depModule = graph.getModuleByFile(resolveRequest)[0];
 
     if (depModule) {
       const dep = module.addDependency(

packages/core/src/inner-plugins/plugins/sourcemapTool.ts

@@ -117,7 +117,37 @@ export async function collectSourceMaps(
         if (!m.source) continue;
 
         // The source in map.sources returned by sourceAndMap may be modified by Rsdoctor's loader
-        let realSource = m.source.split('!').pop();
+        // Handle loader chain paths (e.g., "loader1!loader2!/path/to/file.js")
+        // Extract the actual source file path by splitting on '!' and taking the last part
+        let realSource = m.source;
+
+        if (realSource.includes('!')) {
+          const parts = realSource.split('!');
+          for (let j = parts.length - 1; j >= 0; j--) {
+            const part = parts[j];
+            if (!part) continue;
+
+            const cleanPart = part.split('??')[0];
+
+            if (cleanPart.startsWith('builtin:')) {
+              continue;
+            }
+
+            // Check if it looks like a file path (absolute path starting with '/' or relative path)
+            if (cleanPart.startsWith('/') || cleanPart.includes('\\')) {
+              realSource = cleanPart;
+              break;
+            }
+          }
+
+          if (realSource === m.source) {
+            const lastPart = parts[parts.length - 1];
+            realSource = lastPart ? lastPart.split('??')[0] : lastPart;
+          }
+        } else if (realSource.includes('??')) {
+          realSource = realSource.split('??')[0];
+        }
+
         if (
           (realSource?.startsWith('webpack://') ||
             realSource?.startsWith('file://')) &&

packages/graph/src/graph/module-graph/graph.ts

@@ -325,7 +325,10 @@ export class ModuleGraph implements SDK.ModuleGraphInstance {
   }
 
   getModuleByFile(file: string) {
-    return this.getModules().find((item) => item.path === file);
+    const similarModules = this.getModules().filter(
+      (item) => item.path === file,
+    );
+    return similarModules || [];
   }
 
   addModule(...modules: SDK.ModuleInstance[]) {

packages/graph/src/transform/chunks/assetsModules.ts

@@ -79,20 +79,21 @@ export async function getAssetsModulesData(
   } else {
     time(`Start Parse bundle by sourcemap.`);
     for (const [modulePath, codes] of sourceMapSets.entries()) {
-      const module = moduleGraph.getModuleByFile(modulePath);
-      if (!module) continue;
+      const modules = moduleGraph.getModuleByFile(modulePath);
       let gzipSize = undefined;
       try {
         if (codes && typeof codes === 'string' && codes.length > 0) {
           const { gzipSync } = await import('node:zlib');
           gzipSize = gzipSync(codes, { level: 9 }).length;
         }
       } catch {}
-      module?.setSize({
-        parsedSize: codes.length,
-        gzipSize,
-      });
-      module?.setSource({ parsedSource: codes });
+      for (const module of modules) {
+        module?.setSize({
+          parsedSize: codes.length,
+          gzipSize,
+        });
+        module?.setSource({ parsedSource: codes });
+      }
     }
     timeEnd(`Start Parse bundle by sourcemap.`);
   }

packages/types/src/sdk/module.ts

@@ -311,7 +311,7 @@ export interface ModuleGraphInstance {
   getModuleByWebpackId(webpackId: string): ModuleInstance | undefined;
 
   /** Get module by path */
-  getModuleByFile(file: string): ModuleInstance | undefined;
+  getModuleByFile(file: string): ModuleInstance[] | [];
 
   /** Add Module */
   addModule(module: ModuleInstance): void;