@agentcash/router 1.0.0 → 1.0.1

package/dist/index.cjs

@@ -90,33 +90,46 @@ module.exports = __toCommonJS(index_exports);
 // src/registry.ts
 var RouteRegistry = class {
   routes = /* @__PURE__ */ new Map();
-  // Silently overwrites on duplicate key. Next.js module loading order is
-  // non-deterministic during build — discovery stubs and real handlers may
-  // register the same route key in either order. Last writer wins.
+  // Internal map key includes the HTTP method so that POST and DELETE on the
+  // same path coexist. Within the same path+method, last-write-wins is still
+  // intentional — Next.js module loading order is non-deterministic during
+  // build and discovery stubs may register the same route in either order.
   // Prior art: ElysiaJS uses the same pattern (silent overwrite in router.history).
+  mapKey(entry) {
+    return `${entry.key}:${entry.method}`;
+  }
   register(entry) {
-    if (this.routes.has(entry.key) && typeof process !== "undefined" && process.env?.["NODE_ENV"] !== "production") {
+    const k = this.mapKey(entry);
+    if (this.routes.has(k) && typeof process !== "undefined" && process.env?.["NODE_ENV"] !== "production") {
       console.warn(
-        `[agentcash/router] route '${entry.key}' registered twice \u2014 overwriting (this is expected for discovery stubs during next build)`
+        `[agentcash/router] route '${entry.key}' (${entry.method}) registered twice \u2014 overwriting (this is expected for discovery stubs during next build)`
       );
     }
-    this.routes.set(entry.key, entry);
+    this.routes.set(k, entry);
   }
+  // Accepts either a compound key ("site/domain:DELETE") or a path-only key
+  // ("site/domain") — path-only returns the first registered method for that path.
   get(key) {
-    return this.routes.get(key);
+    const direct = this.routes.get(key);
+    if (direct) return direct;
+    for (const entry of this.routes.values()) {
+      if (entry.key === key) return entry;
+    }
+    return void 0;
   }
   entries() {
     return this.routes.entries();
   }
   has(key) {
-    return this.routes.has(key);
+    return this.get(key) !== void 0;
   }
   get size() {
     return this.routes.size;
   }
   validate(expectedKeys) {
     if (!expectedKeys) return;
-    const missing = expectedKeys.filter((k) => !this.routes.has(k));
+    const registeredPathKeys = new Set([...this.routes.values()].map((e) => e.key));
+    const missing = expectedKeys.filter((k) => !registeredPathKeys.has(k));
     if (missing.length > 0) {
       throw new Error(
         `route${missing.length > 1 ? "s" : ""} ${missing.map((k) => `'${k}'`).join(", ")} in prices map but not registered \u2014 add to barrel imports`
@@ -1475,8 +1488,8 @@ function createWellKnownHandler(registry, baseUrl, pricesKeys, discovery) {
     const x402Set = /* @__PURE__ */ new Set();
     const mppSet = /* @__PURE__ */ new Set();
     const methodHints = discovery.methodHints ?? "non-default";
-    for (const [key, entry] of registry.entries()) {
-      const url = `${normalizedBase}/api/${entry.path ?? key}`;
+    for (const [, entry] of registry.entries()) {
+      const url = `${normalizedBase}/api/${entry.path ?? entry.key}`;
       const resource = toDiscoveryResource(entry.method, url, methodHints);
       if (entry.authMode !== "unprotected") x402Set.add(resource);
       if (entry.protocols.includes("mpp")) mppSet.add(resource);
@@ -1532,12 +1545,12 @@ function createOpenAPIHandler(registry, baseUrl, pricesKeys, discovery) {
     const tagSet = /* @__PURE__ */ new Set();
     let requiresSiwxScheme = false;
     let requiresApiKeyScheme = false;
-    for (const [key, entry] of registry.entries()) {
-      const apiPath = `/api/${entry.path ?? key}`;
+    for (const [, entry] of registry.entries()) {
+      const apiPath = `/api/${entry.path ?? entry.key}`;
       const method = entry.method.toLowerCase();
-      const tag = deriveTag(key);
+      const tag = deriveTag(entry.key);
       tagSet.add(tag);
-      const built = buildOperation(key, entry, tag);
+      const built = buildOperation(entry.key, entry, tag);
       if (built.requiresSiwxScheme) requiresSiwxScheme = true;
       if (built.requiresApiKeyScheme) requiresApiKeyScheme = true;
       paths[apiPath] = { ...paths[apiPath], [method]: built.operation };

package/dist/index.d.cts

@@ -349,6 +349,7 @@ interface RouterConfig {
 
 declare class RouteRegistry {
     private routes;
+    private mapKey;
     register(entry: RouteEntry): void;
     get(key: string): RouteEntry | undefined;
     entries(): IterableIterator<[string, RouteEntry]>;

package/dist/index.d.ts

@@ -349,6 +349,7 @@ interface RouterConfig {
 
 declare class RouteRegistry {
     private routes;
+    private mapKey;
     register(entry: RouteEntry): void;
     get(key: string): RouteEntry | undefined;
     entries(): IterableIterator<[string, RouteEntry]>;

package/dist/index.js

@@ -51,33 +51,46 @@ var init_server = __esm({
 // src/registry.ts
 var RouteRegistry = class {
   routes = /* @__PURE__ */ new Map();
-  // Silently overwrites on duplicate key. Next.js module loading order is
-  // non-deterministic during build — discovery stubs and real handlers may
-  // register the same route key in either order. Last writer wins.
+  // Internal map key includes the HTTP method so that POST and DELETE on the
+  // same path coexist. Within the same path+method, last-write-wins is still
+  // intentional — Next.js module loading order is non-deterministic during
+  // build and discovery stubs may register the same route in either order.
   // Prior art: ElysiaJS uses the same pattern (silent overwrite in router.history).
+  mapKey(entry) {
+    return `${entry.key}:${entry.method}`;
+  }
   register(entry) {
-    if (this.routes.has(entry.key) && typeof process !== "undefined" && process.env?.["NODE_ENV"] !== "production") {
+    const k = this.mapKey(entry);
+    if (this.routes.has(k) && typeof process !== "undefined" && process.env?.["NODE_ENV"] !== "production") {
       console.warn(
-        `[agentcash/router] route '${entry.key}' registered twice \u2014 overwriting (this is expected for discovery stubs during next build)`
+        `[agentcash/router] route '${entry.key}' (${entry.method}) registered twice \u2014 overwriting (this is expected for discovery stubs during next build)`
       );
     }
-    this.routes.set(entry.key, entry);
+    this.routes.set(k, entry);
   }
+  // Accepts either a compound key ("site/domain:DELETE") or a path-only key
+  // ("site/domain") — path-only returns the first registered method for that path.
   get(key) {
-    return this.routes.get(key);
+    const direct = this.routes.get(key);
+    if (direct) return direct;
+    for (const entry of this.routes.values()) {
+      if (entry.key === key) return entry;
+    }
+    return void 0;
   }
   entries() {
     return this.routes.entries();
   }
   has(key) {
-    return this.routes.has(key);
+    return this.get(key) !== void 0;
   }
   get size() {
     return this.routes.size;
   }
   validate(expectedKeys) {
     if (!expectedKeys) return;
-    const missing = expectedKeys.filter((k) => !this.routes.has(k));
+    const registeredPathKeys = new Set([...this.routes.values()].map((e) => e.key));
+    const missing = expectedKeys.filter((k) => !registeredPathKeys.has(k));
     if (missing.length > 0) {
       throw new Error(
         `route${missing.length > 1 ? "s" : ""} ${missing.map((k) => `'${k}'`).join(", ")} in prices map but not registered \u2014 add to barrel imports`
@@ -1436,8 +1449,8 @@ function createWellKnownHandler(registry, baseUrl, pricesKeys, discovery) {
     const x402Set = /* @__PURE__ */ new Set();
     const mppSet = /* @__PURE__ */ new Set();
     const methodHints = discovery.methodHints ?? "non-default";
-    for (const [key, entry] of registry.entries()) {
-      const url = `${normalizedBase}/api/${entry.path ?? key}`;
+    for (const [, entry] of registry.entries()) {
+      const url = `${normalizedBase}/api/${entry.path ?? entry.key}`;
       const resource = toDiscoveryResource(entry.method, url, methodHints);
       if (entry.authMode !== "unprotected") x402Set.add(resource);
       if (entry.protocols.includes("mpp")) mppSet.add(resource);
@@ -1493,12 +1506,12 @@ function createOpenAPIHandler(registry, baseUrl, pricesKeys, discovery) {
     const tagSet = /* @__PURE__ */ new Set();
     let requiresSiwxScheme = false;
     let requiresApiKeyScheme = false;
-    for (const [key, entry] of registry.entries()) {
-      const apiPath = `/api/${entry.path ?? key}`;
+    for (const [, entry] of registry.entries()) {
+      const apiPath = `/api/${entry.path ?? entry.key}`;
       const method = entry.method.toLowerCase();
-      const tag = deriveTag(key);
+      const tag = deriveTag(entry.key);
       tagSet.add(tag);
-      const built = buildOperation(key, entry, tag);
+      const built = buildOperation(entry.key, entry, tag);
       if (built.requiresSiwxScheme) requiresSiwxScheme = true;
       if (built.requiresApiKeyScheme) requiresApiKeyScheme = true;
       paths[apiPath] = { ...paths[apiPath], [method]: built.operation };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agentcash/router",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "Unified route builder for Next.js App Router APIs with x402, MPP, SIWX, and API key auth",
   "type": "module",
   "exports": {