feat: split up API Routes

Brings back the functionality that was reverted in
#1731, but under a flag.
This will be utterly slow in building,
so let's try to speed that up in the next step!

Author: Skn0tt

packages/runtime/src/index.ts

@@ -24,6 +24,7 @@ import {
   generatePagesResolver,
   getExtendedApiRouteConfigs,
   warnOnApiRoutes,
+  getApiRouteConfigs,
 } from './helpers/functions'
 import { generateRedirects, generateStaticRedirects } from './helpers/redirects'
 import { shouldSkip, isNextAuthInstalled, getCustomImageResponseHeaders, getRemotePatterns } from './helpers/utils'
@@ -150,7 +151,11 @@ const plugin: NetlifyPlugin = {
     const buildId = readFileSync(join(publish, 'BUILD_ID'), 'utf8').trim()
 
     await configureHandlerFunctions({ netlifyConfig, ignore, publish: relative(process.cwd(), publish) })
-    const apiRoutes = await getExtendedApiRouteConfigs(publish, appDir)
+
+    const useNoneBundler = Boolean(process.env.NEXT_SPLIT_API_ROUTES)
+    const apiRoutes = useNoneBundler
+      ? await getApiRouteConfigs(publish, appDir)
+      : await getExtendedApiRouteConfigs(publish, appDir)
 
     await generateFunctions(constants, appDir, apiRoutes)
     await generatePagesResolver(constants)

test/helpers/functions.spec.ts

@@ -1,8 +1,65 @@
-import { getExtendedApiRouteConfigs } from "../../packages/runtime/src/helpers/functions"
-import { describeCwdTmpDir, moveNextDist } from "../test-utils"
+import { getApiRouteConfigs, getExtendedApiRouteConfigs } from '../../packages/runtime/src/helpers/functions'
+import { describeCwdTmpDir, moveNextDist } from '../test-utils'
 
 describeCwdTmpDir('api route file analysis', () => {
   it('extracts correct route configs from source files', async () => {
+    await moveNextDist()
+    const configs = await getApiRouteConfigs('.next', process.cwd())
+    // Using a Set means the order doesn't matter
+    expect(new Set(configs)).toEqual(
+      new Set([
+        {
+          compiled: 'pages/api/og.js',
+          config: {
+            runtime: 'edge',
+          },
+          route: '/api/og',
+        },
+        {
+          compiled: 'pages/api/enterPreview.js',
+          config: {},
+          route: '/api/enterPreview',
+        },
+        {
+          compiled: 'pages/api/exitPreview.js',
+          config: {},
+          route: '/api/exitPreview',
+        },
+        {
+          compiled: 'pages/api/hello.js',
+          config: {},
+          route: '/api/hello',
+        },
+        {
+          compiled: 'pages/api/shows/[...params].js',
+          config: {},
+          route: '/api/shows/[...params]',
+        },
+        {
+          compiled: 'pages/api/shows/[id].js',
+          config: {},
+          route: '/api/shows/[id]',
+        },
+        {
+          compiled: 'pages/api/hello-background.js',
+          config: { type: 'experimental-background' },
+          route: '/api/hello-background',
+        },
+        {
+          compiled: 'pages/api/hello-scheduled.js',
+          config: { schedule: '@hourly', type: 'experimental-scheduled' },
+          route: '/api/hello-scheduled',
+        },
+        {
+          compiled: 'pages/api/revalidate.js',
+          config: {},
+          route: '/api/revalidate',
+        },
+      ]),
+    )
+  })
+
+  it('only shows scheduled/background functions as extended funcs', async () => {
     await moveNextDist()
     const configs = await getExtendedApiRouteConfigs('.next', process.cwd())
     // Using a Set means the order doesn't matter
@@ -21,4 +78,4 @@ describeCwdTmpDir('api route file analysis', () => {
       ]),
     )
   })
-})
+})