Handle for Webpack

Author: devjiwonchoi

packages/next/src/build/analysis/get-page-static-info.ts

@@ -40,6 +40,7 @@ import {
 } from '../segment-config/middleware/middleware-config'
 import { normalizeAppPath } from '../../shared/lib/router/utils/app-paths'
 import { normalizePagePath } from '../../shared/lib/page-path/normalize-page-path'
+import { isProxyFile } from '../utils'
 
 const PARSE_PATTERN =
   /(?<!(_jsx|jsx-))runtime|preferredRegion|getStaticProps|getServerSideProps|generateStaticParams|export const|generateImageMetadata|generateSitemaps|middleware|proxy/
@@ -756,14 +757,15 @@ export async function getPagesPageStaticInfo({
     warnAboutExperimentalEdge(isAnAPIRoute ? page! : null)
   }
 
-  if (
-    (page === `/${PROXY_FILENAME}` || page === `/src/${PROXY_FILENAME}`) &&
-    isEdgeRuntime(resolvedRuntime)
-  ) {
+  if (isProxyFile(page) && isEdgeRuntime(resolvedRuntime)) {
     resolvedRuntime = SERVER_RUNTIME.nodejs
-    const message = `Proxy does not support Edge runtime.`
+    const relativePath = relative(process.cwd(), pageFilePath)
+    const resolvedPath = relativePath.startsWith('.')
+      ? relativePath
+      : `./${relativePath}`
+    const message = `Route segment config is not allowed in Proxy file at "${resolvedPath}". Proxy always runs on Node.js runtime.  Learn more: https://nextjs.org/docs/messages/middleware-to-proxy`
     if (isDev) {
-      Log.error(message)
+      Log.errorOnce(message)
     } else {
       throw new Error(message)
     }

packages/next/src/build/entries.ts

@@ -43,9 +43,9 @@ import type { __ApiPreviewProps } from '../server/api-utils'
 import {
   isMiddlewareFile,
   isMiddlewareFilename,
+  isProxyFile,
   isInstrumentationHookFile,
   isInstrumentationHookFilename,
-  isProxyFilename,
 } from './utils'
 import { getPageStaticInfo } from './analysis/get-page-static-info'
 import { normalizePathSep } from '../shared/lib/page-path/normalize-path-sep'
@@ -767,12 +767,12 @@ export function runDependingOnPageType<T>(params: {
     return
   }
 
-  if (isProxyFilename(params.page)) {
+  if (isProxyFile(params.page)) {
     params.onServer()
     return
   }
 
-  if (isMiddlewareFile(params.page)) {
+  if (isMiddlewareFile(params.page) && !isProxyFile(params.page)) {
     if (params.pageRuntime === 'nodejs') {
       params.onServer()
       return
@@ -1036,11 +1036,12 @@ export async function createEntrypoints(
                   : undefined,
               }).import
             }
-            const edgeServerBundlePath = isMiddlewareFile(page)
-              ? serverBundlePath
-                  .replace(PROXY_FILENAME, MIDDLEWARE_FILENAME)
-                  .replace('src/', '')
-              : serverBundlePath
+            const edgeServerBundlePath =
+              isMiddlewareFile(page) && !isProxyFile(page)
+                ? serverBundlePath
+                    .replace(PROXY_FILENAME, MIDDLEWARE_FILENAME)
+                    .replace('src/', '')
+                : serverBundlePath
             edgeServer[edgeServerBundlePath] = getEdgeServerEntry({
               ...params,
               rootDir,

packages/next/src/build/index.ts

@@ -1214,13 +1214,7 @@ export default async function build(
 
       for (const rootPath of rootPaths) {
         const { name: fileBaseName, dir: fileDir } = path.parse(rootPath)
-        const isAtConventionLevel =
-          fileDir === '/' ||
-          fileDir === '/src' ||
-          // rootPaths are currently relative paths from the root directory.
-          // Add safety check here for unexpected future changes.
-          fileDir === dir ||
-          fileDir === path.join(dir, 'src')
+        const isAtConventionLevel = fileDir === '/' || fileDir === '/src'
 
         if (isAtConventionLevel && fileBaseName === MIDDLEWARE_FILENAME) {
           middlewareFilePath = rootPath
@@ -2610,9 +2604,7 @@ export default async function build(
           return serverFilesManifest
         })
 
-      const middlewareFile = rootPaths.find(
-        (p) => p.includes(MIDDLEWARE_FILENAME) || p.includes(PROXY_FILENAME)
-      )
+      const middlewareFile = proxyFilePath || middlewareFilePath
       let hasNodeMiddleware = false
 
       if (middlewareFile) {
@@ -2623,7 +2615,10 @@ export default async function build(
           appDir,
           pageExtensions: config.pageExtensions,
           isDev: false,
-          page: 'middleware',
+          // Is format of `(/src)/(proxy|middleware).<ext>`, so split by
+          // "." and get the first part, regard rest of the extensions
+          // to match the `page` value format.
+          page: middlewareFile.split('.')[0],
         })
 
         if (staticInfo.hadUnsupportedValue) {

packages/next/src/build/output/log.ts

@@ -85,3 +85,12 @@ export function warnOnce(...message: any[]) {
     warn(...message)
   }
 }
+
+const errorOnceCache = new LRUCache<string>(10_000, (value) => value.length)
+export function errorOnce(...message: any[]) {
+  const key = message.join(' ')
+  if (!errorOnceCache.has(key)) {
+    errorOnceCache.set(key, key)
+    error(...message)
+  }
+}

packages/next/src/build/utils.ts

@@ -105,14 +105,6 @@ export function isMiddlewareFilename(file?: string | null) {
   )
 }
 
-export function isMiddlewareOnlyFilename(file?: string | null) {
-  return file === MIDDLEWARE_FILENAME || file === `src/${MIDDLEWARE_FILENAME}`
-}
-
-export function isProxyFilename(file?: string | null) {
-  return file === PROXY_FILENAME || file === `src/${PROXY_FILENAME}`
-}
-
 export function isInstrumentationHookFilename(file?: string | null) {
   return (
     file === INSTRUMENTATION_HOOK_FILENAME ||
@@ -1429,6 +1421,10 @@ export function isMiddlewareFile(file: string) {
   )
 }
 
+export function isProxyFile(file: string) {
+  return file === `/${PROXY_FILENAME}` || file === `/src/${PROXY_FILENAME}`
+}
+
 export function isInstrumentationHookFile(file: string) {
   return (
     file === `/${INSTRUMENTATION_HOOK_FILENAME}` ||

test/e2e/app-dir/proxy-runtime/proxy-runtime.test.ts

@@ -35,7 +35,9 @@ Next.js can't recognize the exported \`config\` field in route. Proxy does not s
 
 The exported configuration object in a source file needs to have a very specific format from which some properties can be statically parsed at compiled-time.`)
     } else {
-      expect(cliOutput).toContain(`Proxy does not support Edge runtime.`)
+      expect(cliOutput).toContain(
+        `Route segment config is not allowed in Proxy file at "./proxy.ts". Proxy always runs on Node.js runtime.  Learn more: https://nextjs.org/docs/messages/middleware-to-proxy`
+      )
     }
 
     await next.stop()