@proagentstore/cli 0.4.51 → 0.4.53

package/dist/browser-runner/coding/engine-acts.js

@@ -84,6 +84,44 @@ export function splitSegments(command) {
         .map((s) => s.trim())
         .filter(Boolean);
 }
+/**
+ * The engine's verdict on ONE tool call — the single expression of it (#597).
+ *
+ * `is_error` is the only outcome Claude Code's stream-json protocol states for a `tool_result`: it
+ * sets the flag when the call failed and omits it otherwise, so `!== true` is the protocol's own
+ * reading rather than an inference about the result text. This is the fact `settleAct` has always
+ * published as a consequential act's `ok`.
+ *
+ * It is a function because two call sites now need it — the act record and the transcript line the
+ * cloud parses — and a verdict stated twice is a verdict that can disagree with itself.
+ */
+export function toolCallOk(block) {
+    return block.is_error !== true;
+}
+/**
+ * The outcome marker on the transcript's `↳` result line — `↳✓` succeeded, `↳✗` failed (#597).
+ *
+ * The runner computed this and threw it away: `settleAct` read `is_error` while the line pushed
+ * into the transcript carried only the result text. The transcript is the ONLY channel the cloud
+ * reads for an ordinary tool call (a read, a grep, a file open leaves no `agent_events` row), so
+ * the per-tool-call record of #581 AC7 could state the argument and the result and never whether
+ * the call worked.
+ *
+ * ── Why the marker is welded to the arrow, with no space
+ *
+ * The reader (`workers/api/src/lib/engine-tool-calls.ts`) takes the outcome from the character at a
+ * FIXED offset after `↳`, and every runner predating this wrote a space there. So a row written by
+ * an old runner reads *unknown* and can never read as a pass, whatever the tool's own output
+ * happens to start with — which is the property AC1 asks for and the reason a marker after the
+ * space would not do: `toolResult()` collapses a result to one line, and vitest's begins
+ * `✓ src/foo.test.ts`. An old row carrying that text must not be read as a success claim.
+ *
+ * `terminal-render.ts` and `engine-auth-prompt.ts`'s `QUOTED_LINE_RE` both match `↳` unanchored on
+ * its right, so the pane still renders and quoted-line filtering still fires.
+ */
+export function toolResultMark(block) {
+    return toolCallOk(block) ? "✓" : "✗";
+}
 /** A token that is exactly the trunk, not a branch merely containing the word (`feature/main-fix`). */
 const TRUNK_TOKEN = /(?:^|\s)(?:HEAD:)?(?:refs\/heads\/)?(?:main|master)(?:\s|$)/;
 function prRef(segment) {

package/dist/browser-runner/coding/headless.js

@@ -1,5 +1,5 @@
 import { spawn } from "node:child_process";
-import { classifyCommand, commandFromToolInput, fillTargetFromResult } from "./engine-acts.js";
+import { classifyCommand, commandFromToolInput, fillTargetFromResult, toolCallOk, toolResultMark } from "./engine-acts.js";
 import { parseEngineUsage } from "./engine-usage.js";
 import { turnReportFromExit, turnReportFromResult } from "./engine-turn.js";
 /**
@@ -605,7 +605,7 @@ export class HeadlessSession {
             case "user": // tool results come back as a synthetic user message
                 for (const block of ev.message?.content ?? []) {
                     if (block.type === "tool_result") {
-                        this.push(`  ↳ ${toolResult(block.content)}`); // ↳
+                        this.push(`  ↳${toolResultMark(block)} ${toolResult(block.content)}`); // ↳✓ / ↳✗ (#597)
                         this.settleAct(block);
                     }
                 }
@@ -679,7 +679,7 @@ export class HeadlessSession {
         if (!acts)
             return;
         this.awaitingResult.delete(id);
-        const ok = block.is_error !== true;
+        const ok = toolCallOk(block);
         for (const a of fillTargetFromResult(acts, block.content))
             this.publishAct({ ...a, ok });
     }

package/dist/index.js

@@ -1217,6 +1217,9 @@ function diffMembership(attached, eligible, thisNode, blocked = /* @__PURE__ */
 function shouldRegisterOnOpen(reconnect, alreadyRegistered) {
   return reconnect || !alreadyRegistered;
 }
+function pendingRegistrations(attached, registered) {
+  return [...attached].filter((id) => !registered.has(id));
+}
 function instanceLabel(inst) {
   const short = `${inst.id.slice(0, 8)}\u2026`;
   return inst.name ? `${inst.name} (${short})` : short;
@@ -1336,7 +1339,7 @@ async function connectViaRelay(instanceIds, localUrl, runnerToken, opts, force =
   };
   for (const id of instanceIds) attach(id, "");
   reportRegistration();
-  writeLine(registered.size === instanceIds.length ? `Runtime registered with PAGS \u2713 (${registered.size}/${instanceIds.length} agents)` : `Runtime registration incomplete: ${registered.size}/${instanceIds.length} agents \u2014 retried on each relay (re)connect.`);
+  writeLine(registered.size === instanceIds.length ? `Runtime registered with PAGS \u2713 (${registered.size}/${instanceIds.length} agents)` : `Runtime registration incomplete: ${registered.size}/${instanceIds.length} agents \u2014 retried on each relay (re)connect${watchInstances ? " and every 20s while this runs" : ""}.`);
   writeLine("");
   writeLine("\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550");
   writeLine(`  \u2705 CONNECTED \u2014 WebSocket relay \xB7 ${hostname3()}`);
@@ -1406,6 +1409,11 @@ async function connectViaRelay(instanceIds, localUrl, runnerToken, opts, force =
             attach(inst.id, instanceLabel(inst));
           }
           for (const id of toDetach) detach(id);
+          const pending = pendingRegistrations(attached.keys(), registered);
+          if (pending.length) {
+            for (const id of pending) await registerRuntime(id);
+            reportRegistration();
+          }
         } catch {
         }
         tick();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@proagentstore/cli",
-  "version": "0.4.51",
+  "version": "0.4.53",
   "description": "CLI for creating, publishing, and running ProAgentStore agents",
   "license": "MIT",
   "type": "module",