Revert "Sourcemap errors in terminal by default (#71444)"

This reverts commit af6f6aa.

packages/next/src/client/components/react-dev-overlay/server/shared.ts

@@ -50,7 +50,7 @@ export function findSourcePackage({
 export function getOriginalCodeFrame(
   frame: StackFrame,
   source: string | null
-): string | null {
+): string | null | undefined {
   if (!source || isInternal(frame.file)) {
     return null
   }
@@ -65,7 +65,6 @@ export function getOriginalCodeFrame(
         column: frame.column ?? 0,
       },
     },
-    // TODO: Only in TTY
     { forceColor: true }
   )
 }

packages/next/src/server/node-environment.ts

@@ -1,9 +1,6 @@
 // This file should be imported before any others. It sets up the environment
 // for later imports to work properly.
 
-// Import before anything else so that unexpected errors in other extensions are properly formatted.
-import './node-environment-extensions/error-inspect'
-
 import './node-environment-baseline'
 import './node-environment-extensions/random'
 import './node-environment-extensions/date'

test/e2e/app-dir/actions/app-action-size-limit-invalid.test.ts

@@ -1,6 +1,5 @@
 import { nextTestSetup } from 'e2e-utils'
 import { retry } from 'next-test-utils'
-import stripAnsi from 'strip-ansi'
 import { accountForOverhead } from './account-for-overhead'
 
 const CONFIG_ERROR =
@@ -27,7 +26,7 @@ describe('app-dir action size limit invalid config', () => {
 
   beforeAll(() => {
     const onLog = (log: string) => {
-      logs.push(stripAnsi(log.trim()))
+      logs.push(log.trim())
     }
 
     next.on('stdout', onLog)
@@ -116,7 +115,7 @@ describe('app-dir action size limit invalid config', () => {
 
       await retry(() => {
         expect(logs).toContainEqual(
-          expect.stringContaining('Error: Body exceeded 1.5mb limit')
+          expect.stringContaining('[Error]: Body exceeded 1.5mb limit')
         )
         expect(logs).toContainEqual(
           expect.stringContaining(

test/e2e/app-dir/dynamic-io-errors/dynamic-io-errors.prospective-fallback.test.ts

@@ -27,7 +27,6 @@ describe(`Dynamic IO Prospective Fallback`, () => {
         // we expect the build to fail
       }
 
-      // TODO: Assert on component stack
       expect(next.cliOutput).toContain(
         'Route "/blog/[slug]": A component accessed data, headers, params, searchParams, or a short-lived cache without a Suspense boundary nor a "use cache" above it.'
       )

test/e2e/app-dir/dynamic-io-errors/fixtures/prospective-fallback/next.config.js

@@ -1,6 +1,5 @@
 module.exports = {
   experimental: {
     dynamicIO: true,
-    serverSourceMaps: true,
   },
 }