agent-dag 1.33.22 → 1.33.24

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-Bt8oQ4sT.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.24",
   "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/cswap-install.mjs

@@ -166,20 +166,35 @@ async function safePythons() {
  * any `pip install --user pipx` where ~/.local/bin was never added to PATH. The
  * two-entry version of this list reported "needs uv or pipx" to people who had
  * pipx installed, which is the kind of wrong answer that stops someone looking.
+ *
+ * Every entry carries its UPGRADE command line as well as its install one. The
+ * upgrade used to be re-derived from the `via` label instead, and any label that
+ * derivation did not recognise fell through to `-m pipx upgrade` — so the
+ * bundled uv, which is the only installer present on a machine that had neither
+ * uv nor pipx nor a usable python, was asked to run a pipx command it rejects.
  */
 async function installers() {
   const out = [
-    { cmd: "uv",   probe: ["--version"], args: ["tool", "install", "claude-swap"], via: "uv" },
-    { cmd: "pipx", probe: ["--version"], args: ["install", "claude-swap"],         via: "pipx" },
+    { cmd: "uv",   probe: ["--version"], args: ["tool", "install", "claude-swap"], upgrade: ["tool", "upgrade", "claude-swap"], via: "uv" },
+    { cmd: "pipx", probe: ["--version"], args: ["install", "claude-swap"],         upgrade: ["upgrade", "claude-swap"],         via: "pipx" },
   ];
   // A uv fetched on an earlier run counts as installed tooling from here on.
   const own = existingBootstrappedUv();
-  if (own) out.push({ cmd: own, probe: ["--version"], args: ["tool", "install", "claude-swap"], via: "uv (bundled)" });
+  if (own) {
+    out.push({
+      cmd: own,
+      probe: ["--version"],
+      args: ["tool", "install", "claude-swap"],
+      upgrade: ["tool", "upgrade", "claude-swap"],
+      via: "uv (bundled)",
+    });
+  }
   for (const py of await safePythons()) {
     out.push({
       cmd: py,
       probe: ["-m", "pipx", "--version"],
       args: ["-m", "pipx", "install", "claude-swap"],
+      upgrade: ["-m", "pipx", "upgrade", "claude-swap"],
       via: `${py} -m pipx`,
     });
   }
@@ -270,21 +285,19 @@ function isOlder(a, b) {
  * Detached and unawaited: an upgrade resolves a Python environment and can
  * take tens of seconds, which is not a thing to put in front of the server
  * starting. The running copy keeps working; the new one is there next launch.
+ *
+ * The command line comes from the installer entry rather than from its label:
+ * runDetached captures nothing, so an upgrade aimed at the wrong tool fails
+ * where nobody can see it while the caller still reports "upgrading".
  */
-function upgradeInBackground(found) {
-  const { cmd, via } = found;
-  const args = via === "uv"
-    ? ["tool", "upgrade", "claude-swap"]
-    : via === "pipx"
-      ? ["upgrade", "claude-swap"]
-      : ["-m", "pipx", "upgrade", "claude-swap"];   // python -m pipx
-  runDetached(cmd, args);
+function upgradeInBackground({ cmd, upgrade }) {
+  runDetached(cmd, upgrade);
 }
 
 /** Whichever Python tool installer is available, or null. */
 async function findInstaller() {
-  for (const { cmd, probe, via } of await installers()) {
-    if ((await run(cmd, probe, { timeout: 8_000 })).ok) return { cmd, via };
+  for (const { cmd, probe, upgrade, via } of await installers()) {
+    if ((await run(cmd, probe, { timeout: 8_000 })).ok) return { cmd, upgrade, via };
   }
   return null;
 }

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);