@rulvar/executor 1.80.0 → 1.81.1

package/dist/index.js

@@ -288,7 +288,12 @@ function subprocessExecutor(options = {}) {
 				const tail = child.stderr.trim().slice(-500);
 				throw new ExecutorError("exit", `tool '${request.tool}' exited ${child.code ?? "null"}${child.signal === null ? "" : ` (signal ${child.signal})`}${tail === "" ? "" : `: ${tail}`}`);
 			}
-			return parseToolResult(child.stdout, request.tool);
+			try {
+				return parseToolResult(child.stdout, request.tool);
+			} catch (err) {
+				outcome = "error";
+				throw err;
+			}
 		} finally {
 			const durationMs = now() - startedAt;
 			if (options.ledger !== void 0) await options.ledger.record({
@@ -483,7 +488,12 @@ function containerExecutor(options) {
 				const tail = child.stderr.trim().slice(-500);
 				throw new ExecutorError("exit", `container tool '${request.tool}' exited ${child.code ?? "null"}${tail === "" ? "" : `: ${tail}`}`);
 			}
-			return parseToolResult(child.stdout, request.tool);
+			try {
+				return parseToolResult(child.stdout, request.tool);
+			} catch (err) {
+				outcome = "error";
+				throw err;
+			}
 		} finally {
 			const durationMs = now() - startedAt;
 			if (options.ledger !== void 0) await options.ledger.record({
@@ -796,6 +806,24 @@ function executorConformance(factory, options = {}) {
 				ensure(bad?.outcome === "error" && bad.exitCode === 1, "e11", "the error record is wrong");
 				ensure(rows.every((r) => typeof r.argsHash === "string" && r.argsHash.length === 64), "e11", "every record must carry a content argsHash");
 			}
+		},
+		{
+			id: "e12",
+			title: "ledgers a protocol failure as an error outcome, never ok",
+			async run() {
+				const ledger = memoryEffectLedger();
+				const provider = factory({
+					command: runtime,
+					args: baseArgs,
+					ledger
+				});
+				const err = await asExecutorError("e12", provider.run(request("sloppy", { behavior: "garbage" }, { idempotencyKey: "k-proto" })));
+				ensure(err.code === "protocol", "e12", `expected code 'protocol', got '${err.code}'`);
+				const rows = ledger.entries();
+				ensure(rows.length === 1, "e12", `expected 1 ledger record, got ${rows.length}`);
+				ensure(rows[0]?.outcome === "error", "e12", `a protocol failure must ledger outcome 'error', got '${String(rows[0]?.outcome)}'`);
+				ensure(rows[0]?.exitCode === 0, "e12", "the clean exit code must still be recorded");
+			}
 		}
 	];
 	return {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rulvar/executor",
-  "version": "1.80.0",
+  "version": "1.81.1",
   "description": "Rulvar isolated tool executors: reference ToolExecutorProvider adapters that run tool work out of process (subprocess and container) so hostile or model-generated scripts cannot reach host capabilities.",
   "type": "module",
   "license": "Apache-2.0",
@@ -22,13 +22,13 @@
     "access": "public"
   },
   "dependencies": {
-    "@rulvar/core": "1.80.0"
+    "@rulvar/core": "1.81.1"
   },
   "devDependencies": {
     "@types/node": "^22.20.0",
     "tsdown": "^0.22.3",
     "typescript": "~6.0.3",
-    "@rulvar/testing": "1.80.0"
+    "@rulvar/testing": "1.81.1"
   },
   "repository": {
     "type": "git",