castle-web-cli 0.4.35 → 0.4.36

package/dist/agent.js

@@ -543,13 +543,14 @@ function createTaskStore(opts) {
         });
     }
     function spawnFromDirective(directive, originMessageId) {
-        // A fix/redo task sweeps the rows it obsoletes off the user's board. A
-        // superseded task that never started must not start later either.
+        // A fix/redo task sweeps the rows it obsoletes off the user's board. Halt
+        // first -- a superseded task that is still running must have its process
+        // killed (not just hidden, which left an invisible zombie the rest of the
+        // board could depend on), and a waiting one must not start later.
         for (const id of resolveDeps(tasks, directive.supersedes)) {
             const old = tasks.get(id);
             if (old && !old.acknowledged) {
-                if (old.status === 'waiting')
-                    old.status = 'interrupted';
+                haltTask(old);
                 old.acknowledged = true;
                 touch(old);
                 opts.onSuperseded(old);
@@ -620,6 +621,28 @@ function createTaskStore(opts) {
     // stop everything still active. Waiting tasks are cancelled outright;
     // running ones get their agent process killed and finalize as interrupted
     // via the stopRequested path.
+    // Halt an active task: a waiting one is cancelled (-> interrupted); a
+    // running one gets its agent process killed and finalizes as interrupted
+    // via the stopRequested path. No-op on terminal tasks.
+    function haltTask(task) {
+        if (task.status === 'waiting') {
+            task.status = 'interrupted';
+            touch(task);
+        }
+        else if (task.status === 'running') {
+            stopRequested.add(task.id);
+            for (const child of children) {
+                if (child.pid === task.pid) {
+                    try {
+                        child.kill('SIGKILL');
+                    }
+                    catch {
+                        /* already gone */
+                    }
+                }
+            }
+        }
+    }
     function stop(tokens) {
         const ids = meansAll(tokens)
             ? [...tasks.values()]
@@ -628,25 +651,8 @@ function createTaskStore(opts) {
             : resolveDeps(tasks, tokens);
         for (const id of ids) {
             const task = tasks.get(id);
-            if (!task)
-                continue;
-            if (task.status === 'waiting') {
-                task.status = 'interrupted';
-                touch(task);
-            }
-            else if (task.status === 'running') {
-                stopRequested.add(task.id);
-                for (const child of children) {
-                    if (child.pid === task.pid) {
-                        try {
-                            child.kill('SIGKILL');
-                        }
-                        catch {
-                            /* already gone */
-                        }
-                    }
-                }
-            }
+            if (task)
+                haltTask(task);
         }
     }
     return { sorted, get: (id) => tasks.get(id), spawnFromDirective, acknowledge, checkOff, stop, shutdown };
@@ -779,10 +785,13 @@ function runRouterTurnIn(ctx, instruction) {
         messages: ctx.log.messages
             .filter((m) => m.role !== 'log' && m.id !== message.id && m.status !== 'streaming')
             .map((m) => ({ role: m.role, text: m.text })),
-        // Only the live board (unacknowledged tasks) -- match what the user sees.
-        // Showing every already-cleared task confused the router about what was
-        // actually on the board to clear / act on.
-        tasks: ctx.taskStore.sorted().filter((t) => !t.acknowledged).map(asPromptTask),
+        // Only the live board -- match what the user sees. Hide tasks that are
+        // BOTH acknowledged AND finished; an active (running/waiting) task always
+        // shows even if somehow acked, so nothing can ever go invisible mid-work.
+        tasks: ctx.taskStore
+            .sorted()
+            .filter((t) => !(t.acknowledged && isTerminal(t.status)))
+            .map(asPromptTask),
         instruction,
     });
     const backend = ctx.backend();

package/dist/chat-client.js

@@ -227,7 +227,10 @@ function TaskRow(props) {
         expanded && task.status !== 'running' ? (React.createElement("div", { className: "task-notes", dangerouslySetInnerHTML: renderMarkdown(notes) })) : null));
 }
 function TaskBoard(props) {
-    const visible = props.tasks.filter((t) => !t.acknowledged);
+    // Hide only tasks that are acknowledged AND finished. An active
+    // (running/waiting) task always shows, even if somehow acked -- a running
+    // task must never go invisible while it (and its dependents) are live.
+    const visible = props.tasks.filter((t) => !(t.acknowledged && TERMINAL_TASK_STATUSES.includes(t.status)));
     if (visible.length === 0)
         return null;
     return (React.createElement("div", { id: "chat-strip" }, visible.map((task) => (React.createElement(TaskRow, { key: task.id, task: task, feed: props.feeds[task.id], onAck: props.onAck })))));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "castle-web-cli",
-  "version": "0.4.35",
+  "version": "0.4.36",
   "type": "module",
   "bin": {
     "castle-web": "./dist/index.js"