fix(llms-txt): move all handling to middleware for Vercel compatibility

Vercel ISR glob rules (/package/**/llms.txt) create catch-all serverless
functions that intercept requests before Nitro's file-based routes can
resolve them, breaking scoped packages and versioned paths. Move all
llms.txt/llms_full.txt handling into the middleware, remove ISR route
rules, and delete file-based route files.

Author: lukeocodes

nuxt.config.ts

@@ -112,9 +112,6 @@ export default defineNuxtConfig({
     '/api/registry/file/**': { isr: true, cache: { maxAge: 365 * 24 * 60 * 60 } },
     '/api/registry/provenance/**': { isr: true, cache: { maxAge: 365 * 24 * 60 * 60 } },
     '/api/registry/files/**': { isr: true, cache: { maxAge: 365 * 24 * 60 * 60 } },
-    '/package/**/llms.txt': { isr: 3600 },
-    '/package/**/llms_full.txt': { isr: 3600 },
-    '/llms.txt': { isr: 3600 },
     '/api/registry/package-meta/**': { isr: 300 },
     '/:pkg/.well-known/skills/**': { isr: 3600 },
     '/:scope/:pkg/.well-known/skills/**': { isr: 3600 },

server/middleware/llms-txt.ts

@@ -3,20 +3,27 @@ import { PackageRouteParamsSchema } from '#shared/schemas/package'
 import { handleApiError } from '#server/utils/error-handler'
 import { handleLlmsTxt, handleOrgLlmsTxt, generateRootLlmsTxt } from '#server/utils/llms-txt'
 
+const CACHE_HEADER = 's-maxage=3600, stale-while-revalidate=86400'
+
 /**
- * Middleware to handle llms.txt / llms_full.txt routes that can't be served
- * by Nitro's file-based routing (versioned paths hit a radix3 limitation
- * where parameterized intermediate segments don't resolve literal children).
+ * Middleware to handle ALL llms.txt / llms_full.txt routes.
  *
- * Handles:
- * - /llms.txt (root — file-based route is blocked by canonical-redirects)
- * - /package/:name/v/:version/llms.txt
- * - /package/:name/v/:version/llms_full.txt
- * - /package/@:org/:name/v/:version/llms.txt
- * - /package/@:org/:name/v/:version/llms_full.txt
- * - /package/@:org/llms.txt (org listing)
+ * All llms.txt handling lives here rather than in file-based routes because
+ * Vercel's ISR route rules with glob patterns (e.g. `/package/ ** /llms.txt`)
+ * create catch-all serverless functions that interfere with Nitro's file-based
+ * route resolution — scoped packages and versioned paths fail to match.
  *
- * Non-versioned package routes are left to file-based handlers.
+ * Handles:
+ * - /llms.txt (root discovery page)
+ * - /package/@:org/llms.txt (org package listing)
+ * - /package/:name/llms.txt (unscoped, latest)
+ * - /package/:name/llms_full.txt (unscoped, latest, full)
+ * - /package/@:org/:name/llms.txt (scoped, latest)
+ * - /package/@:org/:name/llms_full.txt (scoped, latest, full)
+ * - /package/:name/v/:version/llms.txt (unscoped, versioned)
+ * - /package/:name/v/:version/llms_full.txt (unscoped, versioned, full)
+ * - /package/@:org/:name/v/:version/llms.txt (scoped, versioned)
+ * - /package/@:org/:name/v/:version/llms_full.txt (scoped, versioned, full)
  */
 export default defineEventHandler(async event => {
   const path = event.path.split('?')[0]
@@ -31,7 +38,7 @@ export default defineEventHandler(async event => {
     const url = getRequestURL(event)
     const baseUrl = `${url.protocol}//${url.host}`
     setHeader(event, 'Content-Type', 'text/markdown; charset=utf-8')
-    setHeader(event, 'Cache-Control', 's-maxage=3600, stale-while-revalidate=86400')
+    setHeader(event, 'Cache-Control', CACHE_HEADER)
     return generateRootLlmsTxt(baseUrl)
   }
 
@@ -48,33 +55,37 @@ export default defineEventHandler(async event => {
       const baseUrl = `${url.protocol}//${url.host}`
       const content = await handleOrgLlmsTxt(orgName, baseUrl)
       setHeader(event, 'Content-Type', 'text/markdown; charset=utf-8')
-      setHeader(event, 'Cache-Control', 's-maxage=3600, stale-while-revalidate=86400')
+      setHeader(event, 'Cache-Control', CACHE_HEADER)
       return content
     } catch (error: unknown) {
       handleApiError(error, { statusCode: 502, message: 'Failed to generate org llms.txt.' })
     }
   }
 
-  // Versioned paths — only handle if /v/ is present (non-versioned are handled by file routes)
-  if (!inner.includes('/v/')) return
-
+  // Parse package name and optional version from inner path
   let rawPackageName: string
-  let rawVersion: string
+  let rawVersion: string | undefined
 
-  if (inner.startsWith('@')) {
-    // Scoped: @org/name/v/version
-    const match = inner.match(/^(@[^/]+\/[^/]+)\/v\/(.+)$/)
-    if (!match) return
-    rawPackageName = match[1]
-    rawVersion = match[2]
+  if (inner.includes('/v/')) {
+    // Versioned path
+    if (inner.startsWith('@')) {
+      const match = inner.match(/^(@[^/]+\/[^/]+)\/v\/(.+)$/)
+      if (!match) return
+      rawPackageName = match[1]
+      rawVersion = match[2]
+    } else {
+      const match = inner.match(/^([^/]+)\/v\/(.+)$/)
+      if (!match) return
+      rawPackageName = match[1]
+      rawVersion = match[2]
+    }
   } else {
-    // Unscoped: name/v/version
-    const match = inner.match(/^([^/]+)\/v\/(.+)$/)
-    if (!match) return
-    rawPackageName = match[1]
-    rawVersion = match[2]
+    // Latest version — inner is just the package name
+    rawPackageName = inner
   }
 
+  if (!rawPackageName) return
+
   try {
     const { packageName, version } = v.parse(PackageRouteParamsSchema, {
       packageName: rawPackageName,
@@ -83,7 +94,7 @@ export default defineEventHandler(async event => {
 
     const content = await handleLlmsTxt(packageName, version, { includeAgentFiles: full })
     setHeader(event, 'Content-Type', 'text/markdown; charset=utf-8')
-    setHeader(event, 'Cache-Control', 's-maxage=3600, stale-while-revalidate=86400')
+    setHeader(event, 'Cache-Control', CACHE_HEADER)
     return content
   } catch (error: unknown) {
     handleApiError(error, {

server/routes/package/[name]/llms.txt.get.ts

@@ -1,13 +1,6 @@
-import * as v from 'valibot'
 import type { Packument } from '@npm/types'
 import type { JsDelivrFileNode, AgentFile, LlmsTxtResult } from '#shared/types'
-import { PackageRouteParamsSchema } from '#shared/schemas/package'
-import {
-  NPM_MISSING_README_SENTINEL,
-  NPM_REGISTRY,
-  CACHE_MAX_AGE_ONE_HOUR,
-} from '#shared/utils/constants'
-import { handleApiError } from '#server/utils/error-handler'
+import { NPM_MISSING_README_SENTINEL, NPM_REGISTRY } from '#shared/utils/constants'
 
 /** Well-known agent instruction files at the package root */
 const ROOT_AGENT_FILES: Record<string, string> = {
@@ -368,55 +361,3 @@ export function generateRootLlmsTxt(baseUrl: string): string {
 
   return lines.join('\n').trimEnd() + '\n'
 }
-
-/**
- * Create a cached event handler for package-level llms.txt or llms_full.txt.
- *
- * Each route file should call this factory and `export default` the result.
- * This avoids the re-export pattern that Nitro doesn't register as routes.
- */
-export function createPackageLlmsTxtHandler(options?: { full?: boolean }) {
-  const full = options?.full ?? false
-
-  return defineCachedEventHandler(
-    async event => {
-      const org = getRouterParam(event, 'org')
-      const name = getRouterParam(event, 'name')
-      const rawVersion = getRouterParam(event, 'version')
-
-      if (!name) {
-        throw createError({ statusCode: 404, message: 'Package name is required.' })
-      }
-
-      const rawPackageName = org ? `${org}/${name}` : name
-
-      try {
-        const { packageName, version } = v.parse(PackageRouteParamsSchema, {
-          packageName: rawPackageName,
-          version: rawVersion,
-        })
-
-        const content = await handleLlmsTxt(packageName, version, { includeAgentFiles: full })
-        setHeader(event, 'Content-Type', 'text/markdown; charset=utf-8')
-        return content
-      } catch (error: unknown) {
-        handleApiError(error, {
-          statusCode: 502,
-          message: `Failed to generate ${full ? 'llms_full.txt' : 'llms.txt'}.`,
-        })
-      }
-    },
-    {
-      maxAge: CACHE_MAX_AGE_ONE_HOUR,
-      swr: true,
-      getKey: event => {
-        const org = getRouterParam(event, 'org')
-        const name = getRouterParam(event, 'name')
-        const version = getRouterParam(event, 'version')
-        const pkg = org ? `${org}/${name}` : name
-        const prefix = full ? 'llms-full-txt' : 'llms-txt'
-        return version ? `${prefix}:${pkg}@${version}` : `${prefix}:${pkg}`
-      },
-    },
-  )
-}