@rdlabo/workers-hono-kit 0.3.2 → 0.3.3

package/dist/http/trailing-slash.d.ts

@@ -0,0 +1,21 @@
+/**
+ * Trailing-slash normalization for NestJS(Express) → Hono parity.
+ *
+ * Express runs with strict routing off by default, so `/x` and `/x/` resolve to the same handler.
+ * Hono distinguishes them, so a client that sends e.g. `GET /functions/timeline/` 404s against a
+ * Hono worker that only registered `/functions/timeline`. Apply this at the Worker `fetch` entry to
+ * strip a trailing slash before routing, preserving method / headers / body.
+ *
+ * We intentionally do NOT use `hono/trailing-slash`'s `trimTrailingSlash`, which responds with a 301
+ * redirect and thus breaks non-GET methods (POST/PUT/DELETE) and clients that don't follow redirects.
+ *
+ * @example
+ * export default {
+ *   fetch: (request, env, ctx) => app.fetch(normalizeTrailingSlash(request), env, ctx),
+ * };
+ *
+ * @param request - The incoming request.
+ * @returns The same request when the path has no trailing slash (or is exactly `/`), otherwise a new
+ *   Request with the trailing slash(es) removed.
+ */
+export declare const normalizeTrailingSlash: (request: Request) => Request;

package/dist/http/trailing-slash.js

@@ -0,0 +1,28 @@
+/**
+ * Trailing-slash normalization for NestJS(Express) → Hono parity.
+ *
+ * Express runs with strict routing off by default, so `/x` and `/x/` resolve to the same handler.
+ * Hono distinguishes them, so a client that sends e.g. `GET /functions/timeline/` 404s against a
+ * Hono worker that only registered `/functions/timeline`. Apply this at the Worker `fetch` entry to
+ * strip a trailing slash before routing, preserving method / headers / body.
+ *
+ * We intentionally do NOT use `hono/trailing-slash`'s `trimTrailingSlash`, which responds with a 301
+ * redirect and thus breaks non-GET methods (POST/PUT/DELETE) and clients that don't follow redirects.
+ *
+ * @example
+ * export default {
+ *   fetch: (request, env, ctx) => app.fetch(normalizeTrailingSlash(request), env, ctx),
+ * };
+ *
+ * @param request - The incoming request.
+ * @returns The same request when the path has no trailing slash (or is exactly `/`), otherwise a new
+ *   Request with the trailing slash(es) removed.
+ */
+export const normalizeTrailingSlash = (request) => {
+    const url = new URL(request.url);
+    if (url.pathname.length > 1 && url.pathname.endsWith('/')) {
+        url.pathname = url.pathname.replace(/\/+$/, '');
+        return new Request(url, request);
+    }
+    return request;
+};

package/dist/index.d.ts

@@ -24,6 +24,7 @@ export type { AppEnv } from './http/app-env.js';
 export { HttpStatus } from './http/http-status.js';
 export { createNestErrorHandler, nestNotFoundHandler, NEST_REASON_PHRASES } from './http/nest-error.js';
 export type { NestErrorHandlerOptions, ErrorReportContext, ErrorReporter } from './http/nest-error.js';
+export { normalizeTrailingSlash } from './http/trailing-slash.js';
 export { KVCache } from './cache/kv-cache.js';
 export type { KVNamespace, KVCacheOptions } from './cache/kv-cache.js';
 export { createStripeClient, verifyStripeWebhook } from './stripe/client.js';

package/dist/index.js

@@ -20,6 +20,7 @@ export { getAppInfo } from './http/app-info.js';
 export { resolveAppEnv, isProductionEnv } from './http/app-env.js';
 export { HttpStatus } from './http/http-status.js';
 export { createNestErrorHandler, nestNotFoundHandler, NEST_REASON_PHRASES } from './http/nest-error.js';
+export { normalizeTrailingSlash } from './http/trailing-slash.js';
 // cache
 export { KVCache } from './cache/kv-cache.js';
 // stripe

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rdlabo/workers-hono-kit",
-  "version": "0.3.2",
+  "version": "0.3.3",
   "private": false,
   "publishConfig": {
     "access": "public"

package/src/http/trailing-slash.ts

@@ -0,0 +1,28 @@
+/**
+ * Trailing-slash normalization for NestJS(Express) → Hono parity.
+ *
+ * Express runs with strict routing off by default, so `/x` and `/x/` resolve to the same handler.
+ * Hono distinguishes them, so a client that sends e.g. `GET /functions/timeline/` 404s against a
+ * Hono worker that only registered `/functions/timeline`. Apply this at the Worker `fetch` entry to
+ * strip a trailing slash before routing, preserving method / headers / body.
+ *
+ * We intentionally do NOT use `hono/trailing-slash`'s `trimTrailingSlash`, which responds with a 301
+ * redirect and thus breaks non-GET methods (POST/PUT/DELETE) and clients that don't follow redirects.
+ *
+ * @example
+ * export default {
+ *   fetch: (request, env, ctx) => app.fetch(normalizeTrailingSlash(request), env, ctx),
+ * };
+ *
+ * @param request - The incoming request.
+ * @returns The same request when the path has no trailing slash (or is exactly `/`), otherwise a new
+ *   Request with the trailing slash(es) removed.
+ */
+export const normalizeTrailingSlash = (request: Request): Request => {
+  const url = new URL(request.url);
+  if (url.pathname.length > 1 && url.pathname.endsWith('/')) {
+    url.pathname = url.pathname.replace(/\/+$/, '');
+    return new Request(url, request);
+  }
+  return request;
+};

package/src/index.ts

@@ -34,6 +34,7 @@ export type { AppEnv } from './http/app-env.js';
 export { HttpStatus } from './http/http-status.js';
 export { createNestErrorHandler, nestNotFoundHandler, NEST_REASON_PHRASES } from './http/nest-error.js';
 export type { NestErrorHandlerOptions, ErrorReportContext, ErrorReporter } from './http/nest-error.js';
+export { normalizeTrailingSlash } from './http/trailing-slash.js';
 
 // cache
 export { KVCache } from './cache/kv-cache.js';