feat: support route handlers (#106)

* feat: function entry matcher utility

* test: function entry matcher

* chore: changeset

* chore: prefer forEach over test.each

* chore: clearer parameter names

.changeset/wise-camels-sleep.md

@@ -0,0 +1,5 @@
+---
+"@cloudflare/next-on-pages": patch
+---
+
+Handle route handler function entries.

src/index.ts

@@ -8,7 +8,7 @@ import { watch } from "chokidar";
 import pLimit from "p-limit";
 import acorn, { parse, Node } from "acorn";
 import { generate } from "astring";
-import { normalizePath } from "./utils";
+import { matchFunctionEntry, normalizePath } from "./utils";
 
 import { version as nextOnPagesVersion } from "../package.json";
 
@@ -400,10 +400,7 @@ const transform = async ({
     }
 
     for (const entry of functionsEntries) {
-      if (
-        `pages/${name}` === entry?.name ||
-        `app${name !== "index" ? `/${name}` : ""}/page` === entry?.name
-      ) {
+      if (matchFunctionEntry(entry.name, name)) {
         hydratedFunctions.set(name, { matchers: entry.matchers, filepath });
       }
     }

src/utils/index.ts

@@ -1 +1,2 @@
-export { normalizePath } from "./paths";
+export { normalizePath } from './paths';
+export { matchFunctionEntry } from './transform';

src/utils/transform/index.ts

@@ -0,0 +1 @@
+export { matchFunctionEntry } from './manifest';

src/utils/transform/manifest.ts

@@ -0,0 +1,19 @@
+/**
+ * Check if a function file name matches an entry in the manifest.
+ *
+ * @param entryName Manifest entry name.
+ * @param fileName Function file name.
+ * @returns Whether the function file name matches the manifest entry name.
+ */
+export const matchFunctionEntry = (entryName: string, fileName: string) => {
+  // app directory
+  if (entryName.startsWith('app/')) {
+    const type = entryName.endsWith('/route') ? '/route' : '/page';
+    return (
+      `app${fileName !== 'index' ? `/${fileName}` : ''}${type}` === entryName
+    );
+  }
+
+  // pages directory
+  return entryName.startsWith('pages/') && `pages/${fileName}` === entryName;
+};

tests/manifest.test.ts

@@ -0,0 +1,21 @@
+import { describe, test, expect, suite } from 'vitest';
+import { matchFunctionEntry } from '../src/utils';
+
+const manifestEntries = [
+  { name: 'app dir - normal page', entry: 'app/test/page', file: 'test' },
+  { name: 'app dir - dynamic page', entry: 'app/[id]/page', file: '[id]' },
+  { name: 'app dir - nested page', entry: 'app/1/2/3/page', file: '1/2/3' },
+  { name: 'app dir - index page', entry: 'app/page', file: 'index' },
+  { name: 'app dir - route handler', entry: 'app/test/route', file: 'test' },
+  { name: 'pages dir - page', entry: 'pages/api/hello', file: 'api/hello' },
+];
+
+suite('manifest utils', () => {
+  describe('match function entries', () => {
+    manifestEntries.forEach(({ name, entry, file }) => {
+      test(name, () => {
+        expect(matchFunctionEntry(entry, file)).toEqual(true);
+      });
+    });
+  });
+});