@rulvar/cli 1.84.0 → 1.85.0

package/dist/index.js

@@ -142,6 +142,26 @@ function createServer(options) {
 		})().catch(() => {
 			pumpFailed = true;
 		});
+		/**
+		* The terminal release cascade shared by the settle and the
+		* rejection paths: durable retention first (deletes the record AND
+		* untracks), then memory retention (untracks only), then the
+		* settled cap. It runs after the drain so the released buffer is
+		* the full one.
+		*/
+		const releaseAfterTerminal = async () => {
+			const meta = options.retention === void 0 && options.memoryRetention === void 0 ? void 0 : await metaOf(run.runId);
+			if (meta !== void 0 && options.retention?.(meta) === true) {
+				await engine.deleteRun(run.runId);
+				runs.delete(run.runId);
+				return;
+			}
+			if (meta !== void 0 && options.memoryRetention?.(meta) === true) {
+				runs.delete(run.runId);
+				return;
+			}
+			enforceTrackedCap();
+		};
 		handle.result.then(async (outcome) => {
 			run.outcome = outcome;
 			if (outcome.status === "suspended") return;
@@ -149,20 +169,29 @@ function createServer(options) {
 			run.done = true;
 			for (const feed of [...run.feeds]) feed(null, pumpFailed ? "event pump failed; the stream may be incomplete, reconnect with Last-Event-ID to replay" : void 0);
 			run.feeds.clear();
-			(async () => {
-				const meta = options.retention === void 0 && options.memoryRetention === void 0 ? void 0 : await metaOf(run.runId);
-				if (meta !== void 0 && options.retention?.(meta) === true) {
-					await engine.deleteRun(run.runId);
-					runs.delete(run.runId);
-					return;
-				}
-				if (meta !== void 0 && options.memoryRetention?.(meta) === true) {
-					runs.delete(run.runId);
-					return;
-				}
-				enforceTrackedCap();
-			})().catch(() => void 0);
-		}).catch(() => void 0);
+			releaseAfterTerminal().catch(() => void 0);
+		}, handleRejection).catch(() => void 0);
+		/**
+		* The segment rejected: no outcome will ever arrive. The engine
+		* closes the segment's surfaces before every rejection (the boot
+		* refusal explicitly, the withheld settlement after run:end), so
+		* awaiting the pump here delivers whatever tail exists and then
+		* terminates. Without this branch the tracked run stayed 'running'
+		* for the life of the process, its SSE connections never closed,
+		* and neither retention nor the settled cap could ever release it.
+		*/
+		async function handleRejection(thrown) {
+			await pump;
+			run.done = true;
+			run.rejection = thrown instanceof RulvarError ? thrown.toWire() : {
+				code: "error",
+				message: thrown instanceof Error ? thrown.message : String(thrown),
+				retryable: false
+			};
+			for (const feed of [...run.feeds]) feed(null, `run failed before settling: ${run.rejection.message}`);
+			run.feeds.clear();
+			await releaseAfterTerminal();
+		}
 	}
 	function track(runId, workflowName, args, handle) {
 		const run = {
@@ -228,6 +257,13 @@ function createServer(options) {
 		const run = runs.get(runId);
 		if (run !== void 0) {
 			const outcome = run.outcome;
+			if (run.rejection !== void 0) return json(200, {
+				runId,
+				status: "error",
+				workflow: run.workflowName,
+				live: true,
+				error: run.rejection
+			});
 			if (outcome === void 0) return json(200, {
 				runId,
 				status: "running",
@@ -329,6 +365,7 @@ function createServer(options) {
 					controller.enqueue(encoder.encode(sseFrame(run.buffer[i])));
 				}
 				if (run.done) {
+					if (run.rejection !== void 0) controller.enqueue(encoder.encode(`: run failed before settling: ${run.rejection.message}\n\n`));
 					controller.close();
 					return;
 				}
@@ -369,7 +406,7 @@ function createServer(options) {
 		const section = run.queue.then(async () => {
 			if (run.done || run.outcome !== void 0 && run.outcome.status !== "suspended") return json(409, { error: {
 				code: "config",
-				message: `run '${run.runId}' already settled '${run.outcome?.status ?? "unknown"}'`
+				message: run.rejection === void 0 ? `run '${run.runId}' already settled '${run.outcome?.status ?? "unknown"}'` : `run '${run.runId}' failed before settling: ${run.rejection.message}`
 			} });
 			const settledSuspended = run.outcome?.status === "suspended";
 			if (settledSuspended) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rulvar/cli",
-  "version": "1.84.0",
+  "version": "1.85.0",
   "description": "Rulvar shell: run/resume/runs/inspect/plan/kb commands, TUI progress, createServer, createWorker, OTel exporter.",
   "type": "module",
   "license": "Apache-2.0",
@@ -22,17 +22,17 @@
     "access": "public"
   },
   "dependencies": {
-    "@rulvar/core": "1.84.0"
+    "@rulvar/core": "1.85.0"
   },
   "devDependencies": {
-    "@types/node": "^22.20.0",
-    "tsdown": "^0.22.3",
+    "@types/node": "^22.20.1",
+    "tsdown": "^0.22.14",
     "typescript": "~6.0.3",
-    "@rulvar/testing": "1.84.0",
-    "@rulvar/store-sqlite": "1.84.0",
-    "@rulvar/planner": "1.84.0",
-    "@rulvar/evals": "1.84.0",
-    "@rulvar/plan": "1.84.0"
+    "@rulvar/store-sqlite": "1.85.0",
+    "@rulvar/planner": "1.85.0",
+    "@rulvar/plan": "1.85.0",
+    "@rulvar/testing": "1.85.0",
+    "@rulvar/evals": "1.85.0"
   },
   "bin": {
     "rulvar": "./dist/cli.js"