@rcrsr/rill-agent-http 0.20.0 → 0.21.0

package/dist/index.js

@@ -24,18 +24,33 @@ function createHarnessLifecycle(options) {
     if (server !== void 0) {
       throw new Error("Server is already listening");
     }
-    return new Promise((resolve) => {
-      server = serve({ fetch: app.fetch, port }, () => {
-        options?.serverTweaks?.(server);
+    return new Promise((resolve, reject) => {
+      const started = serve({ fetch: app.fetch, port }, () => {
+        started.off("error", onError);
+        options?.serverTweaks?.(started);
         resolve();
       });
+      function onError(err) {
+        server = void 0;
+        reject(err);
+      }
+      started.once("error", onError);
+      server = started;
     });
   }
   async function close() {
-    if (server !== void 0) {
-      server.close();
-      server = void 0;
+    if (server === void 0) return;
+    const current = server;
+    server = void 0;
+    if ("closeAllConnections" in current) {
+      current.closeAllConnections();
     }
+    await new Promise((resolve, reject) => {
+      current.close((err) => {
+        if (err) reject(err);
+        else resolve();
+      });
+    });
   }
   return { app, listen, close };
 }
@@ -82,8 +97,7 @@ function httpHarness(router) {
     }));
     return c.json({ agents });
   });
-  app.post("/agents/:name/run", async (c) => {
-    const name = c.req.param("name");
+  async function handleRun(c, name) {
     let body;
     try {
       const parsed = await c.req.json();
@@ -91,14 +105,32 @@ function httpHarness(router) {
     } catch {
       return c.json({ error: "Request body must be a JSON object" }, 400);
     }
-    const params = body["params"] ?? {};
+    let params;
+    try {
+      params = body["params"] === void 0 ? {} : assertJsonObject(body["params"]);
+    } catch {
+      return c.json({ error: 'Parameter "params" must be a JSON object' }, 400);
+    }
     const validationError = validateParams(params, name, router);
     if (validationError !== null) {
       return c.json({ error: validationError }, 400);
     }
+    const rawTimeout = body["timeout"];
+    let timeout;
+    if (rawTimeout !== void 0) {
+      if (typeof rawTimeout !== "number" || !Number.isFinite(rawTimeout) || rawTimeout <= 0) {
+        return c.json(
+          {
+            error: 'Parameter "timeout" must be a finite number greater than 0'
+          },
+          400
+        );
+      }
+      timeout = rawTimeout;
+    }
     const request = {
       params,
-      ...typeof body["timeout"] === "number" ? { timeout: body["timeout"] } : {}
+      ...timeout !== void 0 ? { timeout } : {}
     };
     try {
       const response = await router.run(name, request);
@@ -108,33 +140,9 @@ function httpHarness(router) {
       const message = err instanceof Error ? err.message : String(err);
       return c.json({ error: message }, status);
     }
-  });
-  app.post("/run", async (c) => {
-    let body;
-    try {
-      const parsed = await c.req.json();
-      body = assertJsonObject(parsed);
-    } catch {
-      return c.json({ error: "Request body must be a JSON object" }, 400);
-    }
-    const params = body["params"] ?? {};
-    const defaultName = router.defaultAgent();
-    const validationError = validateParams(params, defaultName, router);
-    if (validationError !== null) {
-      return c.json({ error: validationError }, 400);
-    }
-    const request = {
-      params,
-      ...typeof body["timeout"] === "number" ? { timeout: body["timeout"] } : {}
-    };
-    try {
-      const response = await router.run("", request);
-      return c.json(response);
-    } catch (err) {
-      const message = err instanceof Error ? err.message : String(err);
-      return c.json({ error: message }, 500);
-    }
-  });
+  }
+  app.post("/agents/:name/run", (c) => handleRun(c, c.req.param("name")));
+  app.post("/run", (c) => handleRun(c, ""));
   async function listen(port = 3e3) {
     return lifecycle.listen(port);
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rcrsr/rill-agent-http",
-  "version": "0.20.0",
+  "version": "0.21.0",
   "description": "rill agent HTTP harness — Hono-based HTTP server wrapping AgentRouter",
   "license": "MIT",
   "author": "Andre Bremer",
@@ -15,14 +15,14 @@
     }
   },
   "dependencies": {
-    "@hono/node-server": "^2.0.1",
-    "hono": "^4.12.16",
-    "@rcrsr/rill-agent": "~0.20.0"
+    "@hono/node-server": "^2.1.1",
+    "@rcrsr/rill-agent": "~0.21.0",
+    "hono": "^4.13.7"
   },
   "devDependencies": {
+    "@rcrsr/rill-agent-hono-kit": "^0.21.0",
     "dts-bundle-generator": "^9.5.1",
-    "tsup": "^8.5.0",
-    "@rcrsr/rill-agent-hono-kit": "^0.20.0"
+    "tsup": "^8.5.1"
   },
   "files": [
     "dist"