Revert "Revert "Revert "Initial support for metadata (#44729)"" (#45113…

…)" (#45130)

This reverts commit 259cbc1.

It does not have necessary changes to make Turbopack work.



## Bug

- [ ] 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)

packages/next/src/build/webpack/loaders/next-app-loader.ts

@@ -7,7 +7,6 @@ import { sep } from 'path'
 import { verifyRootLayout } from '../../../lib/verifyRootLayout'
 import * as Log from '../../../build/output/log'
 import { APP_DIR_ALIAS } from '../../../lib/constants'
-import { resolveFileBasedMetadataForLoader } from '../../../lib/metadata/resolve-metadata'
 
 const FILE_TYPES = {
   layout: 'layout',
@@ -37,26 +36,21 @@ async function createTreeCodeFromPath({
   resolveParallelSegments,
 }: {
   pagePath: string
-  resolve: (
-    pathname: string,
-    resolveDir?: boolean
-  ) => Promise<string | undefined>
+  resolve: (pathname: string) => Promise<string | undefined>
   resolveParallelSegments: (
     pathname: string
   ) => [key: string, segment: string][]
 }) {
   const splittedPath = pagePath.split(/[\\/]/)
   const appDirPrefix = splittedPath[0]
   const pages: string[] = []
-
   let rootLayout: string | undefined
   let globalError: string | undefined
 
   async function createSubtreePropsFromSegmentPath(
     segments: string[]
   ): Promise<{
     treeCode: string
-    treeMetadataCode: string
   }> {
     const segmentPath = segments.join('/')
 
@@ -71,26 +65,12 @@ async function createTreeCodeFromPath({
       parallelSegments.push(...resolveParallelSegments(segmentPath))
     }
 
-    let metadataCode = ''
-
     for (const [parallelKey, parallelSegment] of parallelSegments) {
       if (parallelSegment === PAGE_SEGMENT) {
         const matchedPagePath = `${appDirPrefix}${segmentPath}/page`
         const resolvedPagePath = await resolve(matchedPagePath)
         if (resolvedPagePath) pages.push(resolvedPagePath)
 
-        metadataCode += `{
-          type: 'page',
-          layer: ${
-            // There's an extra virtual segment.
-            segments.length - 1
-          },
-          mod: () => import(/* webpackMode: "eager" */ ${JSON.stringify(
-            resolvedPagePath
-          )}),
-          path: ${JSON.stringify(resolvedPagePath)},
-        },`
-
         // Use '' for segment as it's the page. There can't be a segment called '' so this is the safest way to add it.
         props[parallelKey] = `['', {}, {
           page: [() => import(/* webpackMode: "eager" */ ${JSON.stringify(
@@ -100,8 +80,9 @@ async function createTreeCodeFromPath({
       }
 
       const parallelSegmentPath = segmentPath + '/' + parallelSegment
-      const { treeCode: subtreeCode, treeMetadataCode: subTreeMetadataCode } =
-        await createSubtreePropsFromSegmentPath([...segments, parallelSegment])
+      const { treeCode: subtreeCode } = await createSubtreePropsFromSegmentPath(
+        [...segments, parallelSegment]
+      )
 
       // `page` is not included here as it's added above.
       const filePaths = await Promise.all(
@@ -120,27 +101,6 @@ async function createTreeCodeFromPath({
         rootLayout = layoutPath
       }
 
-      // Collect metadata for the layout
-      if (layoutPath) {
-        metadataCode += `{
-          type: 'layout',
-          layer: ${segments.length},
-          mod: () => import(/* webpackMode: "eager" */ ${JSON.stringify(
-            layoutPath
-          )}),
-          path: ${JSON.stringify(layoutPath)},
-        },`
-      }
-      metadataCode += await resolveFileBasedMetadataForLoader(
-        segments.length,
-        (await resolve(`${appDirPrefix}${parallelSegmentPath}/`, true))!
-      )
-      metadataCode += subTreeMetadataCode
-
-      if (!rootLayout) {
-        rootLayout = layoutPath
-      }
-
       if (!globalError) {
         globalError = await resolve(
           `${appDirPrefix}${parallelSegmentPath}/${GLOBAL_ERROR_FILE_TYPE}`
@@ -173,16 +133,13 @@ async function createTreeCodeFromPath({
           .map(([key, value]) => `${key}: ${value}`)
           .join(',\n')}
       }`,
-      treeMetadataCode: metadataCode,
     }
   }
 
-  const { treeCode, treeMetadataCode } =
-    await createSubtreePropsFromSegmentPath([])
+  const { treeCode } = await createSubtreePropsFromSegmentPath([])
   return {
     treeCode: `const tree = ${treeCode}.children;`,
-    treeMetadataCode: `const metadata = [${treeMetadataCode}];`,
-    pages: `const pages = ${JSON.stringify(pages)};`,
+    pages,
     rootLayout,
     globalError,
   }
@@ -240,7 +197,7 @@ const nextAppLoader: webpack.LoaderDefinitionFunction<{
         const rest = path.slice(pathname.length + 1).split('/')
 
         let matchedSegment = rest[0]
-        // It is the actual page, mark it specially.
+        // It is the actual page, mark it sepcially.
         if (rest.length === 1 && matchedSegment === 'page') {
           matchedSegment = PAGE_SEGMENT
         }
@@ -255,11 +212,7 @@ const nextAppLoader: webpack.LoaderDefinitionFunction<{
     return Object.entries(matched)
   }
 
-  const resolver = async (pathname: string, resolveDir?: boolean) => {
-    if (resolveDir) {
-      return createAbsolutePath(appDir, pathname)
-    }
-
+  const resolver = async (pathname: string) => {
     try {
       const resolved = await resolve(this.rootContext, pathname)
       this.addDependency(resolved)
@@ -277,17 +230,12 @@ const nextAppLoader: webpack.LoaderDefinitionFunction<{
     }
   }
 
-  const {
-    treeCode,
-    treeMetadataCode,
-    pages: pageListCode,
-    rootLayout,
-    globalError,
-  } = await createTreeCodeFromPath({
-    pagePath,
-    resolve: resolver,
-    resolveParallelSegments,
-  })
+  const { treeCode, pages, rootLayout, globalError } =
+    await createTreeCodeFromPath({
+      pagePath,
+      resolve: resolver,
+      resolveParallelSegments,
+    })
 
   if (!rootLayout) {
     const errorMessage = `${chalk.bold(
@@ -315,8 +263,7 @@ const nextAppLoader: webpack.LoaderDefinitionFunction<{
 
   const result = `
     export ${treeCode}
-    export ${treeMetadataCode}
-    export ${pageListCode}
+    export const pages = ${JSON.stringify(pages)}
 
     export { default as AppRouter } from 'next/dist/client/components/app-router'
     export { default as LayoutRouter } from 'next/dist/client/components/layout-router'

packages/next/src/build/webpack/plugins/flight-types-plugin.ts

@@ -64,7 +64,6 @@ interface IEntry {
       ? "runtime?: 'nodejs' | 'experimental-edge' | 'edge'"
       : ''
   }
-  metadata?: any
 }
 
 // =============

packages/next/src/client/components/head.tsx

@@ -0,0 +1,10 @@
+import React from 'react'
+
+export function DefaultHead() {
+  return (
+    <>
+      <meta charSet="utf-8" />
+      <meta name="viewport" content="width=device-width, initial-scale=1" />
+    </>
+  )
+}