agent-dag 1.33.22 → 1.33.23

package/dist/web/index.html

@@ -5,7 +5,7 @@
     <meta name="viewport" content="width=device-width, initial-scale=1" />
     <title>agents-deck</title>
     <link rel="icon" href="data:image/svg+xml,%3Csvg xmlns='http://www.w3.org/2000/svg' viewBox='0 0 100 100'%3E%3Ctext y='84' font-size='84'%3E%E2%97%89%3C/text%3E%3C/svg%3E" />
-    <script type="module" crossorigin src="/assets/index-BC3e-3-4.js"></script>
+    <script type="module" crossorigin src="/assets/index-DTNEy5xO.js"></script>
     <link rel="stylesheet" crossorigin href="/assets/index-Cpi89XJ8.css">
   </head>
   <body>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-dag",
-  "version": "1.33.22",
+  "version": "1.33.23",
   "description": "Live deck of Claude Code and Codex agents — watch parallel subagents fork, call tools, and return on one calm canvas. Also available as npx ccdeck and npx agent-dag.",
   "type": "module",
   "bin": {

package/src/server/exec.mjs

@@ -224,7 +224,16 @@ export function runInteractive(cmd, args, { timeout = 300_000, maxOutput = 256 <
       if (tryNext(err) && !stdout && !stderr) { child = null; return attempt(i + 1); }
       finish(-1, err);
     });
-    proc.on("spawn", () => resolved.set(cmd, raw));
+    // Spawning proves a spelling exists — except a .cmd/.bat one, which is
+    // launched THROUGH cmd.exe. cmd.exe is always there, so it spawns just as
+    // happily for a batch file that is not, and only says so later by exiting
+    // non-zero with "is not recognized". Caching at spawn time therefore
+    // remembered a spelling that never existed, and since `candidates` then
+    // offers only the remembered one, the tool stayed unrunnable — with the
+    // false message "not on PATH" — even after it was installed. A batch
+    // spelling is confirmed by the clean exit below instead, which is the rule
+    // `run` already applies with `if (!err)`.
+    if (!isBatch(raw)) proc.on("spawn", () => resolved.set(cmd, raw));
     // Capped so a runaway child cannot grow the heap without bound; the tail is
     // what carries the error, so the head is what gets dropped.
     const keep = (buf, text) => (buf + text).slice(-maxOutput);
@@ -242,6 +251,9 @@ export function runInteractive(cmd, args, { timeout = 300_000, maxOutput = 256 <
         }
         return finish(-1, { code: "ENOENT" });
       }
+      // Ran to a clean exit, so this spelling is real — the only confirmation a
+      // batch one ever gets.
+      if (code === 0 && !killed && !timedOut) resolved.set(cmd, raw);
       finish(code ?? -1, null);
     });
   };
@@ -281,7 +293,11 @@ export function runDetached(cmd, args) {
     try {
       const child = spawn(file, argv, { stdio: "ignore", shell: false, windowsHide: true, ...opts });
       child.on("error", (err) => { if (tryNext(err)) attempt(i + 1); });
-      child.on("spawn", () => resolved.set(cmd, raw));
+      // Same trap as above: a batch spelling is spawned through cmd.exe, which
+      // succeeds whether or not the batch file is there, so only a clean exit
+      // proves this one is worth remembering.
+      if (isBatch(raw)) child.on("exit", (code) => { if (code === 0) resolved.set(cmd, raw); });
+      else child.on("spawn", () => resolved.set(cmd, raw));
       child.unref?.();
     } catch {
       attempt(i + 1);