v0.14.0 - see CHANGELOG for details

Author: Thesephi

CHANGELOG.md

@@ -1,3 +1,26 @@
+## [0.14.0] - 2025-02-03
+
+## Changed
+
+- multiple paths can now be registered on the same HTTP verb, and if the
+  "static" and the "parameter" portions of the registered paths somehow overlap,
+  then the handler is invoked **once for each matched path**; for example if we
+  register `@Get("/foo/:bar")` and `@Get("/foo/bar")` (in that order) on the
+  same handler function, then for every request to `/foo/bar`, the same handler
+  is invoked **twice**, the first time having no "param" at all, and the 2nd
+  time having the param `bar` with the value `"bar"`. This behavior follows TC39
+  decorator specs where all decorators to a function are applied "from inside
+  out" aka: the last declared decorator gets applied first, and so on
+
+## Added
+
+- `ctx.state._oakRoutingCtrl_regPath` is available as a pointer to the
+  registered path that matches the URL request currently being handled; this is
+  helpful in rare situations where multiple similar paths are registered on the
+  same handler function, causing it to be invoked multiple times, and so we need
+  a machanism to control when to write to the response body (as it can only be
+  done once)
+
 ## [0.13.0] - 2025-02-01
 
 ### Added

deno.jsonc

@@ -1,6 +1,6 @@
 {
   "name": "@dklab/oak-routing-ctrl",
-  "version": "0.13.0",
+  "version": "0.14.0",
   "exports": {
     ".": "./mod.ts",
     "./mod": "./mod.ts"
@@ -15,7 +15,7 @@
   },
   "tasks": {
     "pretty": "deno lint --ignore=docs && deno check . && deno fmt",
-    "test": "deno test -RE",
+    "test": "deno test -REI -N=0.0.0.0,127.0.0.1",
     "check-doc": "deno check --doc .",
     "doc": "deno doc --html mod.ts"
   },
@@ -26,7 +26,8 @@
     "@std/io": "jsr:@std/io@^0.225.2",
     "@std/path": "jsr:@std/path@^1.0.8",
     "@std/testing": "jsr:@std/testing@^1.0.9",
-    "zod": "npm:zod@^3.24.1"
+    "zod": "npm:zod@^3.24.1",
+    "superoak": "https://deno.land/x/[email protected]/mod.ts"
   },
   "fmt": {
     "useTabs": false,

deno.lock

@@ -18,3 +18,11 @@ export {
   z,
   type zInfer,
 } from "./src/utils/schema_utils.ts";
+
+export type {
+  /**
+   * re-exporting from oak for convenient uses
+   * @ignore
+   */
+  Context,
+} from "@oak/oak";

mod.ts

@@ -35,10 +35,12 @@ export const Controller =
     for (const fnName of fnNames) {
       const pair = store.get(fnName);
       if (!pair) continue;
-      pair.forEach((path, verb, p) => {
+      const patchedPair = new Map();
+      pair.forEach((verb, path) => {
         const fullPath = join(pathPrefix, path);
-        p.set(verb, fullPath);
+        patchedPair.set(fullPath, verb);
         patchOasPath(fnName, verb, fullPath);
       });
+      store.set(fnName, patchedPair);
     }
   };

src/Controller.ts

@@ -23,5 +23,5 @@ Deno.test("@Delete decorator", () => {
   assertSpyCall(Delete, 0, { args: ["/bar"] });
   assertInstanceOf(Delete.calls[0].returned, Function);
   assertSpyCalls(Delete, 1);
-  assertEquals(store.get("doSomething")?.get("delete"), "/bar");
+  assertEquals(store.get("doSomething")?.get("/bar"), "delete");
 });

src/Delete_test.ts

@@ -23,5 +23,5 @@ Deno.test("@Get decorator", () => {
   assertSpyCall(Get, 0, { args: ["/bar"] });
   assertInstanceOf(Get.calls[0].returned, Function);
   assertSpyCalls(Get, 1);
-  assertEquals(store.get("doSomething")?.get("get"), "/bar");
+  assertEquals(store.get("doSomething")?.get("/bar"), "get");
 });

src/Get_test.ts

@@ -23,5 +23,5 @@ Deno.test("@Head decorator", () => {
   assertSpyCall(Head, 0, { args: ["/bar"] });
   assertInstanceOf(Head.calls[0].returned, Function);
   assertSpyCalls(Head, 1);
-  assertEquals(store.get("doSomething")?.get("head"), "/bar");
+  assertEquals(store.get("doSomething")?.get("/bar"), "head");
 });

src/Head_test.ts

@@ -23,5 +23,5 @@ Deno.test("@Options decorator", () => {
   assertSpyCall(Options, 0, { args: ["/bar"] });
   assertInstanceOf(Options.calls[0].returned, Function);
   assertSpyCalls(Options, 1);
-  assertEquals(store.get("doSomething")?.get("options"), "/bar");
+  assertEquals(store.get("doSomething")?.get("/bar"), "options");
 });

src/Options_test.ts

@@ -23,5 +23,5 @@ Deno.test("@Patch decorator", () => {
   assertSpyCall(Patch, 0, { args: ["/bar"] });
   assertInstanceOf(Patch.calls[0].returned, Function);
   assertSpyCalls(Patch, 1);
-  assertEquals(store.get("doSomething")?.get("patch"), "/bar");
+  assertEquals(store.get("doSomething")?.get("/bar"), "patch");
 });