fix: now only custom rewrites are created for background and scheduled functions

Author: nickytonline

packages/runtime/src/helpers/functions.ts

@@ -171,12 +171,15 @@ export const getApiRouteConfigs = async (publish: string, baseDir: string): Prom
   const pages = await readJSON(join(publish, 'server', 'pages-manifest.json'))
   const apiRoutes = Object.keys(pages).filter((page) => page.startsWith('/api/'))
   const pagesDir = join(baseDir, 'pages')
-  return Promise.all(
+
+  const settledApiRoutes = await Promise.all(
     apiRoutes.map(async (apiRoute) => {
       const filePath = getSourceFileForPage(apiRoute, pagesDir)
       return { route: apiRoute, config: await extractConfigFromFile(filePath), compiled: pages[apiRoute] }
     }),
   )
+
+  return settledApiRoutes.filter((apiRoute) => apiRoute.config.type !== undefined)
 }
 
 interface FunctionsManifest {
@@ -210,7 +213,7 @@ export const warnOnApiRoutes = async ({
   if (functions.some((func) => func.schedule)) {
     console.warn(
       outdent`
-        ${chalk.yellowBright`Using scheduled API routes`} 
+        ${chalk.yellowBright`Using scheduled API routes`}
         These are run on a schedule when deployed to production.
         You can test them locally by loading them in your browser but this will not be available when deployed, and any returned value is ignored.
       `,

test/__snapshots__/index.js.snap

@@ -1095,21 +1095,6 @@ Array [
     "status": 200,
     "to": "/.netlify/functions/___netlify-handler",
   },
-  Object {
-    "from": "/api/enterPreview",
-    "status": 200,
-    "to": "/.netlify/functions/_api_enterPreview-handler",
-  },
-  Object {
-    "from": "/api/exitPreview",
-    "status": 200,
-    "to": "/.netlify/functions/_api_exitPreview-handler",
-  },
-  Object {
-    "from": "/api/hello",
-    "status": 200,
-    "to": "/.netlify/functions/_api_hello-handler",
-  },
   Object {
     "from": "/api/hello-background",
     "status": 200,
@@ -1120,21 +1105,6 @@ Array [
     "status": 404,
     "to": "/404.html",
   },
-  Object {
-    "from": "/api/og",
-    "status": 200,
-    "to": "/.netlify/functions/_api_og-handler",
-  },
-  Object {
-    "from": "/api/shows/:id",
-    "status": 200,
-    "to": "/.netlify/functions/_api_shows_id-PARAM-handler",
-  },
-  Object {
-    "from": "/api/shows/:params/*",
-    "status": 200,
-    "to": "/.netlify/functions/_api_shows_params-SPLAT-handler",
-  },
   Object {
     "force": false,
     "from": "/broken-image",

test/index.js

@@ -1627,28 +1627,16 @@ describe('api route file analysis', () => {
     // Using a Set means the order doesn't matter
     expect(new Set(configs)).toEqual(
       new Set([
-        { compiled: 'pages/api/enterPreview.js', config: {}, route: '/api/enterPreview' },
         {
           compiled: 'pages/api/hello-background.js',
           config: { type: 'experimental-background' },
           route: '/api/hello-background',
         },
-        { compiled: 'pages/api/exitPreview.js', config: {}, route: '/api/exitPreview' },
-        { 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.js', config: {}, route: '/api/hello' },
         {
           compiled: 'pages/api/hello-scheduled.js',
           config: { schedule: '@hourly', type: 'experimental-scheduled' },
           route: '/api/hello-scheduled',
         },
-        {
-          compiled: 'pages/api/og.js',
-          config: {
-            runtime: 'experimental-edge',
-          },
-          route: '/api/og',
-        },
       ]),
     )
   })