Fix outputting un-necessary trace files for edge functions (vercel#43304)

<!--
Thanks for opening a PR! Your contribution is much appreciated.
To make sure your PR is handled as smoothly as possible we request that
you follow the checklist sections below.
Choose the right checklist for the change that you're making:
-->

## Bug

- [x] Related issues linked using `fixes #number`
- [ ] Integration tests added
- [ ] Errors have a helpful link attached, see
[`contributing.md`](https://github.com/vercel/next.js/blob/canary/contributing.md)

## Feature

- [ ] Implements an existing feature request or RFC. Make sure the
feature request has been accepted for implementation before opening a
PR.
- [ ] Related issues linked using `fixes #number`
- [ ]
[e2e](https://github.com/vercel/next.js/blob/canary/contributing/core/testing.md#writing-tests-for-nextjs)
tests added
- [ ] Documentation added
- [ ] Telemetry added. In case of a feature if it's used or not.
- [ ] Errors have a helpful link attached, see
[`contributing.md`](https://github.com/vercel/next.js/blob/canary/contributing.md)

## Documentation / Examples

- [ ] Make sure the linting passes by running `pnpm build && pnpm lint`
- [ ] The "examples guidelines" are followed from [our contributing
doc](https://github.com/vercel/next.js/blob/canary/contributing/examples/adding-examples.md)

fixes vercel#39275 and related issues like vercel#39858 and hopefully vercel#41395 (can't
test because `pnpm next-with-deps build ./dev-app` keeps crashing on
`Collecting page data...` - `useContext` seems to be null, and even
without the changes it still crashes).

This works by checking if the file is an edge function in Next.js's
Webpack plugin `TraceEntryPointsPlugin`, if true then output the nft
file to the output path, if not move up a folder like previously. I
tried changing Webpack config's output.path from `path: !dev &&
isNodeServer ? path.join(outputPath, 'chunks') : outputPath` to `path:
!dev && (isNodeServer || isEdgeServer) ? path.join(outputPath, 'chunks')
: outputPath`, but it just causes the build script to crash because
files like middleware-manifest.json are output to outputPath/chunks
instead of the expected outputPath. So I find this to be the better
solution.

Co-authored-by: JJ Kasper <[email protected]>

Author: DuCanhGH

packages/next/build/webpack-config.ts

@@ -2002,7 +2002,7 @@ export default async function getBaseWebpackConfig(
         }),
       (isClient || isEdgeServer) && new DropClientPage(),
       config.outputFileTracing &&
-        (isNodeServer || isEdgeServer) &&
+        isNodeServer &&
         !dev &&
         new (require('./webpack/plugins/next-trace-entrypoints-plugin')
           .TraceEntryPointsPlugin as typeof import('./webpack/plugins/next-trace-entrypoints-plugin').TraceEntryPointsPlugin)(

test/e2e/edge-render-getserversideprops/index.test.ts

@@ -4,6 +4,7 @@ import { fetchViaHTTP, normalizeRegEx, renderViaHTTP } from 'next-test-utils'
 import cheerio from 'cheerio'
 import { join } from 'path'
 import escapeStringRegexp from 'escape-string-regexp'
+import fs from 'fs-extra'
 
 describe('edge-render-getserversideprops', () => {
   let next: NextInstance
@@ -16,6 +17,28 @@ describe('edge-render-getserversideprops', () => {
   })
   afterAll(() => next.destroy())
 
+  if ((global as any).isNextStart) {
+    it('should not output trace files for edge routes', async () => {
+      expect(await fs.pathExists(join(next.testDir, '.next/pages'))).toBe(false)
+      expect(
+        await fs.pathExists(join(next.testDir, '.next/server/pages/[id].js'))
+      ).toBe(true)
+      expect(
+        await fs.pathExists(
+          join(next.testDir, '.next/server/pages/[id].js.nft.json')
+        )
+      ).toBe(false)
+      expect(
+        await fs.pathExists(join(next.testDir, '.next/server/pages/index.js'))
+      ).toBe(true)
+      expect(
+        await fs.pathExists(
+          join(next.testDir, '.next/server/pages/index.js.nft.json')
+        )
+      ).toBe(false)
+    })
+  }
+
   it('should have correct query/params on index', async () => {
     const html = await renderViaHTTP(next.url, '/')
     const $ = cheerio.load(html)