@b9g/router 0.1.8 → 0.1.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@b9g/router",
-  "version": "0.1.8",
+  "version": "0.1.9",
   "description": "Universal request router for ServiceWorker applications. Built on web standards with cache-aware routing and generator-based middleware.",
   "keywords": [
     "router",
@@ -17,7 +17,7 @@
     "@b9g/match-pattern": "^0.1.7"
   },
   "devDependencies": {
-    "@b9g/libuild": "^0.1.11",
+    "@b9g/libuild": "^0.1.18",
     "bun-types": "latest"
   },
   "type": "module",

package/src/index.js

@@ -269,27 +269,21 @@ var Router = class {
         }
         const matchResult = this.#executor.match(request);
         if (matchResult) {
-          const mutableRequest = this.#createMutableRequest(request);
           return await this.#executeMiddlewareStack(
             this.#middlewares,
-            mutableRequest,
+            request,
             matchResult.context,
-            matchResult.handler,
-            request.url,
-            this.#executor
+            matchResult.handler
           );
         } else {
           const notFoundHandler = async () => {
             throw new NotFound();
           };
-          const mutableRequest = this.#createMutableRequest(request);
           return await this.#executeMiddlewareStack(
             this.#middlewares,
-            mutableRequest,
+            request,
             { params: {} },
-            notFoundHandler,
-            request.url,
-            this.#executor
+            notFoundHandler
           );
         }
       } catch (error) {
@@ -356,8 +350,6 @@ var Router = class {
       this.#executor = new RadixTreeExecutor(this.#routes);
       this.#dirty = false;
     }
-    const mutableRequest = this.#createMutableRequest(request);
-    const originalURL = mutableRequest.url;
     let matchResult = this.#executor.match(request);
     let handler;
     let context;
@@ -374,12 +366,9 @@ var Router = class {
     try {
       response = await this.#executeMiddlewareStack(
         this.#middlewares,
-        mutableRequest,
+        request,
         context,
-        handler,
-        originalURL,
-        this.#executor
-        // Pass executor for re-routing
+        handler
       );
     } catch (error) {
       if (!matchResult && isHTTPError(error) && error.status === 404) {
@@ -512,7 +501,7 @@ var Router = class {
   /**
    * Execute middleware stack with guaranteed execution using Rack-style LIFO order
    */
-  async #executeMiddlewareStack(middlewares, request, context, handler, originalURL, executor) {
+  async #executeMiddlewareStack(middlewares, request, context, handler) {
     const runningGenerators = [];
     let currentResponse = null;
     const requestPathname = new URL(request.url).pathname;
@@ -545,49 +534,19 @@ var Router = class {
       }
     }
     if (!currentResponse) {
-      let finalHandler = handler;
-      let finalContext = context;
-      if (request.url !== originalURL && executor) {
-        const newMatchResult = executor.match(
-          new Request(request.url, {
-            method: request.method,
-            headers: request.headers,
-            body: request.body
-          })
-        );
-        if (newMatchResult) {
-          finalHandler = newMatchResult.handler;
-          finalContext = newMatchResult.context;
-        }
-      }
       let handlerError = null;
       try {
-        currentResponse = await finalHandler(request, finalContext);
+        currentResponse = await handler(request, context);
       } catch (error) {
         handlerError = error;
       }
       if (handlerError) {
-        if (request.url !== originalURL) {
-          currentResponse = this.#handleAutomaticRedirect(
-            originalURL,
-            request.url,
-            request.method
-          );
-        } else {
-          currentResponse = await this.#handleErrorThroughGenerators(
-            handlerError,
-            runningGenerators
-          );
-        }
+        currentResponse = await this.#handleErrorThroughGenerators(
+          handlerError,
+          runningGenerators
+        );
       }
     }
-    if (request.url !== originalURL && currentResponse) {
-      currentResponse = this.#handleAutomaticRedirect(
-        originalURL,
-        request.url,
-        request.method
-      );
-    }
     for (let i = runningGenerators.length - 1; i >= 0; i--) {
       const { generator } = runningGenerators[i];
       const result = await generator.next(currentResponse);
@@ -616,59 +575,6 @@ var Router = class {
     }
     throw error;
   }
-  /**
-   * Create a mutable request wrapper that allows URL modification
-   */
-  #createMutableRequest(request) {
-    return {
-      url: request.url,
-      method: request.method,
-      headers: new Headers(request.headers),
-      body: request.body,
-      bodyUsed: request.bodyUsed,
-      cache: request.cache,
-      credentials: request.credentials,
-      destination: request.destination,
-      integrity: request.integrity,
-      keepalive: request.keepalive,
-      mode: request.mode,
-      redirect: request.redirect,
-      referrer: request.referrer,
-      referrerPolicy: request.referrerPolicy,
-      signal: request.signal,
-      // Add all other Request methods
-      arrayBuffer: () => request.arrayBuffer(),
-      blob: () => request.blob(),
-      clone: () => request.clone(),
-      formData: () => request.formData(),
-      json: () => request.json(),
-      text: () => request.text()
-    };
-  }
-  /**
-   * Handle automatic redirects when URL is modified
-   */
-  #handleAutomaticRedirect(originalURL, newURL, method) {
-    const originalURLObj = new URL(originalURL);
-    const newURLObj = new URL(newURL);
-    if (originalURLObj.hostname !== newURLObj.hostname || originalURLObj.port !== newURLObj.port && originalURLObj.port !== "" && newURLObj.port !== "") {
-      throw new Error(
-        `Cross-origin redirect not allowed: ${originalURL} -> ${newURL}`
-      );
-    }
-    let status = 302;
-    if (originalURLObj.protocol !== newURLObj.protocol) {
-      status = 301;
-    } else if (method.toUpperCase() !== "GET") {
-      status = 307;
-    }
-    return new Response(null, {
-      status,
-      headers: {
-        Location: newURL
-      }
-    });
-  }
   /**
    * Get route statistics
    */