@rhythmjs/router 0.0.21 → 0.0.22

package/dist/path.js

@@ -1,4 +1,8 @@
 // @bun
+import {
+  pathnameOf
+} from "./rhythm-router-t7qys9ar.js";
+
 // src/path.ts
 import { addRoute, createRouter, findRoute } from "rou3";
 function pathIs(pattern) {
@@ -6,7 +10,7 @@ function pathIs(pattern) {
     throw new TypeError(`path must start with "/", got "${pattern}"`);
   const router = createRouter();
   addRoute(router, "", pattern, true);
-  return (ctx) => findRoute(router, ctx.request.method, new URL(ctx.request.url).pathname) !== undefined;
+  return (ctx) => findRoute(router, ctx.request.method, pathnameOf(ctx.request.url)) !== undefined;
 }
 export {
   pathIs

package/dist/pathname.d.ts

@@ -0,0 +1 @@
+export declare function pathnameOf(url: string): string;

package/dist/rhythm-router-t7qys9ar.js

@@ -0,0 +1,20 @@
+// @bun
+// src/pathname.ts
+var NEEDS_PARSING = /[\\\u0000- \u007f-\uFFFF"<>^`{|}]|\/\.|%2e/i;
+function pathnameOf(url) {
+  const authority = url.indexOf("://") + 3;
+  const start = url.indexOf("/", authority);
+  if (authority < 3 || start === -1)
+    return new URL(url).pathname;
+  let end = url.length;
+  const query = url.indexOf("?", start);
+  if (query !== -1)
+    end = query;
+  const hash = url.indexOf("#", start);
+  if (hash !== -1 && hash < end)
+    end = hash;
+  const pathname = url.slice(start, end);
+  return NEEDS_PARSING.test(pathname) ? new URL(url).pathname : pathname;
+}
+
+export { pathnameOf };

package/dist/rhythm-router.js

@@ -1,4 +1,8 @@
 // @bun
+import {
+  pathnameOf
+} from "./rhythm-router-t7qys9ar.js";
+
 // src/rhythm-router.ts
 import { addRoute, createRouter, findRoute } from "rou3";
 import {
@@ -7,9 +11,12 @@ import {
 } from "@rhythmjs/rhythm";
 function decodeParams(params) {
   const decoded = {};
+  if (!params)
+    return decoded;
   try {
-    for (const [key, value] of Object.entries(params ?? {})) {
-      decoded[key] = decodeURIComponent(value);
+    for (const key in params) {
+      const value = params[key];
+      decoded[key] = value.includes("%") ? decodeURIComponent(value) : value;
     }
   } catch {
     return;
@@ -42,18 +49,21 @@ class RhythmRouter extends Pipeline {
   }
   callback() {
     const run = this.chain();
-    return async (input) => {
+    return (input) => {
       const ctx = input;
-      const match = this.#match(ctx);
-      if (!match)
-        return ctx;
-      Object.assign(ctx, { params: match.params });
-      await run(ctx, () => match.route(ctx));
-      return ctx;
+      try {
+        const match = this.#match(ctx);
+        if (!match)
+          return Promise.resolve(ctx);
+        Object.assign(ctx, { params: match.params });
+        return run(ctx, () => match.route(ctx)).then(() => ctx);
+      } catch (error) {
+        return Promise.reject(error);
+      }
     };
   }
   #match(ctx) {
-    const found = findRoute(this.#routes, ctx.request.method, new URL(ctx.request.url).pathname);
+    const found = findRoute(this.#routes, ctx.request.method, pathnameOf(ctx.request.url));
     if (!found)
       return;
     const params = decodeParams(found.params);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rhythmjs/router",
-  "version": "0.0.21",
+  "version": "0.0.22",
   "description": "Bun-native HTTP router for Rhythm: RhythmRouter plus a fetch handler for Bun.serve, matching with rou3.",
   "homepage": "https://rhythm.js.org/router",
   "license": "ISC",
@@ -36,8 +36,16 @@
   "publishConfig": {
     "access": "public"
   },
+  "scripts": {
+    "clean": "rm -rf dist",
+    "prebuild": "bun run clean",
+    "build": "bun build src/rhythm-router.ts src/context.ts src/fetch.ts src/path.ts --outdir dist --root src --format esm --target bun --packages external --splitting && tsc -p tsconfig.build.json",
+    "prepublishOnly": "bun run typecheck && bun test && bun run build",
+    "typecheck": "tsc --noEmit",
+    "test": "bun test"
+  },
   "dependencies": {
-    "@rhythmjs/rhythm": "0.0.21",
+    "@rhythmjs/rhythm": "workspace:*",
     "rou3": "^1.0.0"
   },
   "devDependencies": {
@@ -47,10 +55,5 @@
   },
   "engines": {
     "bun": ">=1.2.0"
-  },
-  "scripts": {
-    "build": "rm -rf dist && bun build src/rhythm-router.ts src/context.ts src/fetch.ts src/path.ts --outdir dist --root src --format esm --target bun --packages external --splitting && tsc -p tsconfig.build.json",
-    "typecheck": "tsc --noEmit",
-    "test": "bun test"
   }
-}
+}