fix: source map sources in node modules should convert to absolute path (#11501)

Author: SyMind

Cargo.lock

@@ -35,6 +35,7 @@ rspack_workspace               = { workspace = true }
 rustc-hash                     = { workspace = true }
 serde                          = { workspace = true, features = ["derive"] }
 serde_json                     = { workspace = true }
+sugar_path                     = { workspace = true }
 swc                            = { workspace = true, features = ["manual-tokio-runtime"] }
 swc_config                     = { workspace = true }
 swc_core                       = { workspace = true, features = ["base", "ecma_ast", "common", "ecma_preset_env", "ecma_helpers_inline"] }

crates/rspack_loader_swc/Cargo.toml

@@ -3,7 +3,7 @@ mod options;
 mod plugin;
 mod transformer;
 
-use std::default::Default;
+use std::{default::Default, path::Path};
 
 use options::SwcCompilerOptionsWithAdditional;
 pub use options::SwcLoaderJsOptions;
@@ -14,6 +14,7 @@ use rspack_error::{Diagnostic, Result, miette};
 use rspack_javascript_compiler::{JavaScriptCompiler, TransformOutput};
 use rspack_loader_runner::{Identifier, Loader, LoaderContext};
 pub use rspack_workspace::rspack_swc_core_version;
+use sugar_path::SugarPath;
 use swc_config::{merge::Merge, types::MergingOption};
 use swc_core::{
   base::config::{InputSourceMap, TransformConfig},
@@ -95,7 +96,7 @@ impl SwcLoader {
     };
 
     let javascript_compiler = JavaScriptCompiler::new();
-    let filename = FileName::Real(resource_path.into_std_path_buf());
+    let filename = FileName::Real(resource_path.clone().into_std_path_buf());
 
     let source = content.into_string_lossy();
     let is_typescript =
@@ -104,7 +105,7 @@ impl SwcLoader {
 
     let TransformOutput {
       code,
-      map,
+      mut map,
       diagnostics,
     } = javascript_compiler.transform(
       source,
@@ -140,6 +141,29 @@ impl SwcLoader {
       );
     }
 
+    // When compiling target modules, SWC retrieves the source map via sourceMapUrl.
+    // The sources paths in the source map are relative to the target module. We need to resolve these paths
+    // to absolute paths using the resource path to avoid incorrect project path references.
+    if let (Some(map), Some(resource_dir)) = (map.as_mut(), resource_path.parent()) {
+      map.set_sources(
+        map
+          .sources()
+          .iter()
+          .map(|source| {
+            let source_path = Path::new(source);
+            if source_path.is_relative() {
+              source_path
+                .absolutize_with(resource_dir.as_std_path())
+                .to_string_lossy()
+                .into_owned()
+            } else {
+              source.to_string()
+            }
+          })
+          .collect::<Vec<_>>(),
+      );
+    }
+
     loader_context.finish_with((code, map));
 
     Ok(())

crates/rspack_loader_swc/src/lib.rs

@@ -0,0 +1,2 @@
+import sayHello from "lib-with-source-map"
+sayHello("SyMind");

packages/rspack-test-tools/tests/configCases/source-map/source-map-resolve-vendor-path/index.js

@@ -0,0 +1,29 @@
+/** @type {import("@rspack/core").Configuration} */
+module.exports = {
+    devtool: "source-map",
+    module: {
+        rules: [
+            {
+                test: /\.(jsx?|tsx?)$/,
+                use: [
+                    {
+                        loader: "builtin:swc-loader"
+                    }
+                ]
+            }
+        ]
+    },
+    plugins: [
+        {
+            apply(compiler) {
+                compiler.hooks.afterEmit.tap("PLUGIN", compilation => {
+                    const sourceMap = JSON.parse(compilation.assets["bundle0.js.map"].source());
+                    expect(sourceMap.sources).toEqual(expect.arrayContaining([
+                        'webpack:///./node_modules/lib-with-source-map/main.js',
+                        'webpack:///./index.js'
+                    ]));
+                });
+            }
+        }
+    ]
+};