[devtools] Stop sending codeframes for ignored frames

packages/next/src/build/webpack/plugins/wellknown-errors-plugin/parseNotFoundError.ts

@@ -4,6 +4,7 @@ import {
   createOriginalStackFrame,
   getIgnoredSources,
 } from '../../../../server/dev/middleware-webpack'
+import isInternal from '../../../../shared/lib/is-internal'
 import type { webpack } from 'next/dist/compiled/webpack/webpack'
 import type { RawSourceMap } from 'next/dist/compiled/source-map08'
 
@@ -89,10 +90,22 @@ async function getSourceFrame(
         },
       })
 
+      if (result === null || result.originalStackFrame === null) {
+        return {
+          column1: '',
+          frame: '',
+          line1: '',
+        }
+      }
+
+      const originalStackFrame = result.originalStackFrame
       return {
-        frame: result?.originalCodeFrame ?? '',
-        line1: result?.originalStackFrame?.line1?.toString() ?? '',
-        column1: result?.originalStackFrame?.column1?.toString() ?? '',
+        frame:
+          (!isInternal(originalStackFrame.file)
+            ? result.originalCodeFrame
+            : null) ?? '',
+        line1: originalStackFrame.line1?.toString() ?? '',
+        column1: originalStackFrame.column1?.toString() ?? '',
       }
     }
   } catch {}

packages/next/src/next-devtools/server/shared.ts

@@ -1,5 +1,4 @@
 import { codeFrameColumns } from 'next/dist/compiled/babel/code-frame'
-import isInternal from '../../shared/lib/is-internal'
 import type { StackFrame } from '../../server/lib/parse-stack'
 import { ignoreListAnonymousStackFramesIfSandwiched as ignoreListAnonymousStackFramesIfSandwichedGeneric } from '../../server/lib/source-maps'
 
@@ -68,7 +67,7 @@ export function getOriginalCodeFrame(
   source: string | null,
   colors: boolean = process.stdout.isTTY
 ): string | null {
-  if (!source || isInternal(frame.file)) {
+  if (!source) {
     return null
   }
 

packages/next/src/server/dev/middleware-turbopack.ts

@@ -307,17 +307,32 @@ async function createOriginalStackFrame(
     )
   }
 
-  return {
-    originalStackFrame: {
-      arguments: traced.frame.arguments,
-      file: normalizedStackFrameLocation,
-      line1: traced.frame.line1,
-      column1: traced.frame.column1,
-      ignored: traced.frame.ignored,
-      methodName: traced.frame.methodName,
+  /** undefined = not yet computed */
+  let originalCodeFrame: string | null | undefined
+
+  const tracedFrame = traced.frame
+  return Object.defineProperty(
+    {
+      originalStackFrame: {
+        arguments: tracedFrame.arguments,
+        file: normalizedStackFrameLocation,
+        line1: tracedFrame.line1,
+        column1: tracedFrame.column1,
+        ignored: tracedFrame.ignored,
+        methodName: tracedFrame.methodName,
+      },
+      originalCodeFrame: null,
     },
-    originalCodeFrame: getOriginalCodeFrame(traced.frame, traced.source),
-  }
+    'originalCodeFrame',
+    {
+      get: () => {
+        if (originalCodeFrame === undefined) {
+          originalCodeFrame = getOriginalCodeFrame(tracedFrame, traced.source)
+        }
+        return originalCodeFrame
+      },
+    }
+  )
 }
 
 export function getOverlayMiddleware({
@@ -507,7 +522,18 @@ export async function getOriginalStackFrames({
             reason: 'Failed to create original stack frame',
           }
         }
-        return { status: 'fulfilled', value: stackFrame }
+        const originalStackFrame = stackFrame.originalStackFrame
+        return {
+          status: 'fulfilled',
+          value: {
+            originalStackFrame,
+            originalCodeFrame:
+              originalStackFrame?.ignored === false
+                ? // TODO: Don't get all codeframes of non-ignored frames eagerly.
+                  stackFrame.originalCodeFrame
+                : null,
+          },
+        }
       } catch (error) {
         return {
           status: 'rejected',

packages/next/src/server/dev/middleware-webpack.ts

@@ -254,10 +254,24 @@ export async function createOriginalStackFrame({
     ignored,
   }
 
-  return {
-    originalStackFrame: traced,
-    originalCodeFrame: getOriginalCodeFrame(traced, sourceContent),
-  }
+  /** undefined = not yet computed */
+  let originalCodeFrame: string | null | undefined
+
+  return Object.defineProperty(
+    {
+      originalStackFrame: traced,
+      originalCodeFrame: null,
+    },
+    'originalCodeFrame',
+    {
+      get: () => {
+        if (originalCodeFrame === undefined) {
+          originalCodeFrame = getOriginalCodeFrame(traced, sourceContent)
+        }
+        return originalCodeFrame
+      },
+    }
+  )
 }
 
 async function getSourceMapFromCompilation(
@@ -530,7 +544,15 @@ async function getOriginalStackFrame({
     }
   }
 
-  return originalStackFrameResponse
+  const originalStackFrame = originalStackFrameResponse.originalStackFrame
+  return {
+    originalStackFrame,
+    originalCodeFrame:
+      originalStackFrame?.ignored === false
+        ? // TODO: Don't get all codeframes of non-ignored frames eagerly.
+          originalStackFrameResponse.originalCodeFrame
+        : null,
+  }
 }
 
 export function getOverlayMiddleware(options: {

packages/next/src/server/patch-error-inspect.ts

@@ -303,7 +303,7 @@ function getSourcemappedFrameIfPossible(
     ignored,
   }
 
-  /** undefined = not yet computed*/
+  /** undefined = not yet computed */
   let codeFrame: string | null | undefined
 
   return Object.defineProperty(