Merge pull request #104 from stainless-api/yolken-fix-hono-head2

yolken-stainless · web-flow · commit db725ff977a3 · 2026-01-09T16:17:33.000-05:00
fix: Add comment on routeMatcher fix
diff --git a/packages/hono/src/routeMatcher.ts b/packages/hono/src/routeMatcher.ts
@@ -35,8 +35,8 @@ export function makeRouteMatcher(endpoints: AnyEndpoint[]) {
     const [method, path] = endpointToHono(endpoint.endpoint);
     routeMatcher.add(method, path, endpoint);
     if (method === "GET") {
-      // Hono route matching is method-specific, so also add a
-      // HEAD route for GET endpoints
+      // Add HEAD route for every GET route so that HEAD requests
+      // are also matched.
       routeMatcher.add("HEAD", path, endpoint);
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -35,8 +35,8 @@ export function makeRouteMatcher(endpoints: AnyEndpoint[]) {`
`35`	`35`	`const [method, path] = endpointToHono(endpoint.endpoint);`
`36`	`36`	`routeMatcher.add(method, path, endpoint);`
`37`	`37`	`if (method === "GET") {`
`38`		`- // Hono route matching is method-specific, so also add a`
`39`		`- // HEAD route for GET endpoints`
	`38`	`+ // Add HEAD route for every GET route so that HEAD requests`
	`39`	`+ // are also matched.`
`40`	`40`	`routeMatcher.add("HEAD", path, endpoint);`
`41`	`41`	`}`
`42`	`42`	`}`