fix(browserLogs): correct source path resolution

e2e/cases/browser-logs/custom-source-map/index.test.ts

@@ -0,0 +1,48 @@
+import { join, relative } from 'node:path';
+import { rspackTest, toPosixPath } from '@e2e/helper';
+
+const EXPECTED_LOG =
+  'error   [browser] Uncaught Error: test (src/index.js:1:0)';
+
+rspackTest(
+  'should parse source map correctly if source path is absolute',
+  async ({ dev }) => {
+    const rsbuild = await dev({
+      config: {
+        tools: {
+          rspack: {
+            output: {
+              devtoolModuleFilenameTemplate(info) {
+                return toPosixPath(info.absoluteResourcePath);
+              },
+            },
+          },
+        },
+      },
+    });
+    await rsbuild.expectLog(EXPECTED_LOG, { posix: true });
+  },
+);
+
+rspackTest(
+  'should parse source map correctly if source path is relative',
+  async ({ dev }) => {
+    const distPath = join(__dirname, 'dist');
+    const rsbuild = await dev({
+      config: {
+        tools: {
+          rspack: {
+            output: {
+              devtoolModuleFilenameTemplate(info) {
+                return toPosixPath(
+                  relative(distPath, info.absoluteResourcePath),
+                );
+              },
+            },
+          },
+        },
+      },
+    });
+    await rsbuild.expectLog(EXPECTED_LOG, { posix: true });
+  },
+);

e2e/cases/browser-logs/custom-source-map/src/index.js

@@ -0,0 +1 @@
+throw new Error('test');

packages/core/src/server/browserLogs.ts

@@ -120,6 +120,17 @@ const resolveOriginalLocation = async (
   };
 };
 
+/**
+ * Get relative source path that is relative to the project root.
+ * By default, the source path is relative to the dist path or is absolute.
+ */
+const getRelativeSourcePath = (source: string, context: InternalContext) => {
+  const absoluteSourcePath = path.isAbsolute(source)
+    ? source
+    : path.join(context.distPath, source);
+  return path.relative(context.rootPath, absoluteSourcePath);
+};
+
 const formatOriginalLocation = (
   originalMapping: OriginalMapping | InvalidOriginalMapping,
   context: InternalContext,
@@ -129,7 +140,7 @@ const formatOriginalLocation = (
     return;
   }
 
-  let result = path.relative(context.rootPath, source);
+  let result = getRelativeSourcePath(source, context);
   if (line !== null) {
     result += column === null ? `:${line}` : `:${line}:${column}`;
   }