@b9g/router 0.1.7 → 0.1.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@b9g/router",
-  "version": "0.1.7",
+  "version": "0.1.8",
   "description": "Universal request router for ServiceWorker applications. Built on web standards with cache-aware routing and generator-based middleware.",
   "keywords": [
     "router",
@@ -13,6 +13,7 @@
     "shovel"
   ],
   "dependencies": {
+    "@b9g/http-errors": "^0.1.5",
     "@b9g/match-pattern": "^0.1.7"
   },
   "devDependencies": {

package/src/index.d.ts

@@ -189,4 +189,28 @@ export declare class Router {
         compiled: boolean;
     };
 }
+/**
+ * Mode for trailing slash normalization
+ * - "strip": Redirect /path/ → /path (removes trailing slash)
+ * - "add": Redirect /path → /path/ (adds trailing slash)
+ */
+export type TrailingSlashMode = "strip" | "add";
+/**
+ * Middleware that normalizes trailing slashes via 301 redirect
+ *
+ * @param mode - "strip" removes trailing slash, "add" adds trailing slash
+ * @returns Function middleware that redirects non-canonical URLs
+ *
+ * @example
+ * ```typescript
+ * import { Router, trailingSlash } from "@b9g/router";
+ *
+ * const router = new Router();
+ * router.use(trailingSlash("strip")); // Redirect /path/ → /path
+ *
+ * // Can also be scoped to specific paths
+ * router.use("/api", trailingSlash("strip"));
+ * ```
+ */
+export declare function trailingSlash(mode: TrailingSlashMode): FunctionMiddleware;
 export {};

package/src/index.js

@@ -5,7 +5,11 @@ import {
   isSimplePattern,
   compilePathname
 } from "@b9g/match-pattern";
-import { InternalServerError, isHTTPError } from "@b9g/http-errors";
+import {
+  InternalServerError,
+  NotFound,
+  isHTTPError
+} from "@b9g/http-errors";
 var RadixNode = class {
   children;
   // char -> RadixNode
@@ -276,7 +280,7 @@ var Router = class {
           );
         } else {
           const notFoundHandler = async () => {
-            return new Response("Not Found", { status: 404 });
+            throw new NotFound();
           };
           const mutableRequest = this.#createMutableRequest(request);
           return await this.#executeMiddlewareStack(
@@ -361,18 +365,28 @@ var Router = class {
       handler = matchResult.handler;
       context = matchResult.context;
     } else {
-      handler = async () => new Response("Not Found", { status: 404 });
+      handler = async () => {
+        throw new NotFound();
+      };
       context = { params: {} };
     }
-    let response = await this.#executeMiddlewareStack(
-      this.#middlewares,
-      mutableRequest,
-      context,
-      handler,
-      originalURL,
-      this.#executor
-      // Pass executor for re-routing
-    );
+    let response;
+    try {
+      response = await this.#executeMiddlewareStack(
+        this.#middlewares,
+        mutableRequest,
+        context,
+        handler,
+        originalURL,
+        this.#executor
+        // Pass executor for re-routing
+      );
+    } catch (error) {
+      if (!matchResult && isHTTPError(error) && error.status === 404) {
+        return null;
+      }
+      throw error;
+    }
     if (!matchResult && response?.status === 404) {
       return null;
     }
@@ -553,10 +567,18 @@ var Router = class {
         handlerError = error;
       }
       if (handlerError) {
-        currentResponse = await this.#handleErrorThroughGenerators(
-          handlerError,
-          runningGenerators
-        );
+        if (request.url !== originalURL) {
+          currentResponse = this.#handleAutomaticRedirect(
+            originalURL,
+            request.url,
+            request.method
+          );
+        } else {
+          currentResponse = await this.#handleErrorThroughGenerators(
+            handlerError,
+            runningGenerators
+          );
+        }
       }
     }
     if (request.url !== originalURL && currentResponse) {
@@ -667,6 +689,28 @@ var Router = class {
     return httpError.toResponse(isDev);
   }
 };
+function trailingSlash(mode) {
+  return (request, _context) => {
+    const url = new URL(request.url);
+    const pathname = url.pathname;
+    if (pathname === "/")
+      return;
+    let newPathname = null;
+    if (mode === "strip" && pathname.endsWith("/")) {
+      newPathname = pathname.slice(0, -1);
+    } else if (mode === "add" && !pathname.endsWith("/")) {
+      newPathname = pathname + "/";
+    }
+    if (newPathname) {
+      url.pathname = newPathname;
+      return new Response(null, {
+        status: 301,
+        headers: { Location: url.toString() }
+      });
+    }
+  };
+}
 export {
-  Router
+  Router,
+  trailingSlash
 };