feat: use loc.file from rollup errors if available

The loc.file references the file where the error occurred (containing the displayed line and column), rather than e.id which is the module.
vitejs · Jan 19, 2025 · 709c291 · 709c291
1 parent 4f5845a
commit 709c291
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/packages/vite/src/node/build.ts b/packages/vite/src/node/build.ts
@@ -635,7 +635,12 @@ async function buildEnvironment(
     const stackOnly = extractStack(e)
 
     let msg = colors.red((e.plugin ? `[${e.plugin}] ` : '') + e.message)
-    if (e.id) {
+    if (e.loc && e.loc.file && e.loc.file !== e.id) {
+      msg += `\nfile: ${colors.cyan(
+        `${e.loc.file}:${e.loc.line}:${e.loc.column}` +
+          (e.id ? ` (${e.id})` : ''),
+      )}`
+    } else if (e.id) {
       msg += `\nfile: ${colors.cyan(
         e.id + (e.loc ? `:${e.loc.line}:${e.loc.column}` : ''),
       )}`