@pylonsync/sdk 0.3.383 → 0.3.384

package/package.json

@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "0.3.383",
+  "version": "0.3.384",
   "type": "module",
   "main": "./src/index.ts",
   "types": "./dist/index.d.ts",

package/src/index.ts

@@ -1291,13 +1291,35 @@ export async function discoverAppRoutes(opts?: {
     kind: "og-image" as const,
   }));
 
-  return [
+  // Every navigable route — pages, method handlers, sitemap/robots, dynamic
+  // OG images — shares one first-match-wins table in the Rust matcher, so
+  // they must ALL sort literal-before-parameterized together. Appending the
+  // special kinds after pages let a top-level dynamic page shadow its
+  // literal siblings: with `app/[orgSlug]/page.tsx` in the table first,
+  // GET /opengraph-image (and /sitemap.xml, /robots.txt) matched `/:orgSlug`
+  // and rendered the page's 404 instead of the special route. Boundaries are
+  // excluded — they're picked by longest-prefix, not table order.
+  const tokenRank = (s: string): number =>
+    s.startsWith("*?") ? 3 : s.startsWith("*") ? 2 : s.startsWith(":") ? 1 : 0;
+  const navigable = [
     ...pageRoutes,
-    ...boundaryRoutes,
     ...routeRoutes,
     ...dataRoutes,
     ...ogImageRoutes,
-  ];
+  ].sort((a, b) => {
+    const as = a.path.split("/").filter(Boolean);
+    const bs = b.path.split("/").filter(Boolean);
+    const minLen = Math.min(as.length, bs.length);
+    for (let i = 0; i < minLen; i++) {
+      const ar = tokenRank(as[i]);
+      const br = tokenRank(bs[i]);
+      if (ar !== br) return ar - br; // more specific (lower rank) first
+      if (as[i] !== bs[i]) return as[i] < bs[i] ? -1 : 1;
+    }
+    return as.length - bs.length;
+  });
+
+  return [...navigable, ...boundaryRoutes];
 }
 
 export function queriesToManifest(queries: QueryDefinition[]): ManifestQuery[] {

package/src/route-discovery.test.ts

@@ -82,4 +82,26 @@ describe("discoverAppRoutes — opengraph-image convention", () => {
     const routes = await discoverAppRoutes();
     expect(routes.some((r) => r.kind === "og-image")).toBe(false);
   });
+
+  test("special routes sort before a top-level dynamic segment", async () => {
+    // The Rust matcher is first-match-wins over the whole table, so the
+    // literal special routes must come before `/:orgSlug` — otherwise
+    // GET /opengraph-image (and /sitemap.xml, /robots.txt) binds
+    // orgSlug="opengraph-image" and renders the page's 404.
+    app({
+      "page.tsx": PAGE,
+      "[orgSlug]/page.tsx": PAGE,
+      "opengraph-image.tsx": OG,
+      "sitemap.ts": "export default function S(){ return []; }",
+      "robots.ts": "export default function R(){ return {}; }",
+    });
+    const routes = await discoverAppRoutes();
+    const idx = (p: string) => routes.findIndex((r) => r.path === p);
+    const dynamic = idx("/:orgSlug");
+    expect(dynamic).toBeGreaterThan(-1);
+    for (const literal of ["/opengraph-image", "/sitemap.xml", "/robots.txt"]) {
+      expect(idx(literal)).toBeGreaterThan(-1);
+      expect(idx(literal)).toBeLessThan(dynamic);
+    }
+  });
 });