fix gsp tracing issue (#57766)

This removes the ignores for dev react bundles which was added as an
optimization but causes issues when react is imported from an ESM module
since all requires are being analyzed for named exports.

Fixes #57582

packages/next/src/build/collect-build-traces.ts

@@ -266,7 +266,6 @@ export async function collectBuildTraces({
 
       const sharedIgnores = [
         '**/next/dist/compiled/next-server/**/*.dev.js',
-        '**/node_modules/react{,-dom,-dom-server-turbopack}/**/*.development.js',
         isStandalone ? null : '**/next/dist/compiled/jest-worker/**/*',
         '**/next/dist/compiled/webpack/(bundle4|bundle5).js',
         '**/node_modules/webpack5/**/*',
@@ -294,6 +293,7 @@ export async function collectBuildTraces({
 
       const serverIgnores = [
         ...sharedIgnores,
+        '**/node_modules/react{,-dom,-dom-server-turbopack}/**/*.development.js',
         '**/*.d.ts',
         '**/*.map',
         '**/next/dist/pages/**/*',

test/e2e/app-dir/app/standalone-gsp.test.ts

@@ -0,0 +1,88 @@
+import { createNextDescribe } from 'e2e-utils'
+import fs from 'fs-extra'
+import os from 'os'
+import path from 'path'
+import {
+  findPort,
+  initNextServerScript,
+  killApp,
+  fetchViaHTTP,
+} from 'next-test-utils'
+
+if (!(globalThis as any).isNextStart) {
+  it('should skip for non-next start', () => {})
+} else {
+  createNextDescribe(
+    'output: standalone with getStaticProps',
+    {
+      files: __dirname,
+      skipStart: true,
+      dependencies: {
+        swr: 'latest',
+      },
+    },
+    ({ next }) => {
+      beforeAll(async () => {
+        await next.patchFile(
+          'next.config.js',
+          (await next.readFile('next.config.js')).replace('// output', 'output')
+        )
+
+        await next.patchFile(
+          'pages/gsp.js',
+          `
+          import useSWR from 'swr'
+
+          console.log(useSWR)
+          
+          export default function Home() {
+            return <h1>Hello</h1>
+          }
+          
+          export async function getStaticProps() {
+            return {
+              props: {
+                foo: "bar",
+              },
+            };
+          }
+        `
+        )
+
+        await next.start()
+      })
+
+      it('should work correctly with output standalone', async () => {
+        const tmpFolder = path.join(
+          os.tmpdir(),
+          'next-standalone-' + Date.now()
+        )
+        await fs.move(path.join(next.testDir, '.next/standalone'), tmpFolder)
+        let server: any
+
+        try {
+          const testServer = path.join(tmpFolder, 'server.js')
+          const appPort = await findPort()
+          server = await initNextServerScript(
+            testServer,
+            /- Local:/,
+            {
+              ...process.env,
+              PORT: appPort.toString(),
+            },
+            undefined,
+            {
+              cwd: tmpFolder,
+            }
+          )
+
+          const res = await fetchViaHTTP(appPort, '/gsp')
+          expect(res.status).toBe(200)
+        } finally {
+          if (server) await killApp(server)
+          await fs.remove(tmpFolder)
+        }
+      })
+    }
+  )
+}