@songsid/agend 2.1.2 → 2.1.3-beta.2

package/dist/daemon.js

@@ -1145,7 +1145,6 @@ export class Daemon extends EventEmitter {
                         this.emitSupervisionEnded("the CLI exited normally (code 0)", "Nothing crashed — start it again when you need it.");
                         return;
                     }
-                    this.setProcessStatus("crashed");
                     // Distinguish tmux server crash from single window crash.
                     // nullReason records *why* getPaneStatus returned null (for diagnosing
                     // whether this was a real window loss or a transient query failure).
@@ -1182,8 +1181,17 @@ export class Daemon extends EventEmitter {
                             nullReason = "no_window";
                             try {
                                 const windows = await TmuxManager.listWindows(this.tmuxSessionName);
+                                const currentWindowId = this.tmux.getWindowId();
+                                if (windows.some(w => w.id === currentWindowId)) {
+                                    // The exact window still exists, so `list-panes` was the query
+                                    // that glitched. Keep the process/state intact and retry on
+                                    // the next health tick instead of killing a live Kiro TUI.
+                                    this.logger.warn({ windowId: currentWindowId }, `${cliLabel} pane status unavailable but window is present — deferring crash recovery`);
+                                    scheduleNext();
+                                    return;
+                                }
                                 if (windows.some(w => w.name === this.name))
-                                    nullReason = "window_present_query_glitch";
+                                    nullReason = "same_name_other_window";
                             }
                             catch {
                                 nullReason = "query_error";
@@ -1194,6 +1202,7 @@ export class Daemon extends EventEmitter {
                     else {
                         this.logger.warn({ exitCode }, `${cliLabel} process exited`);
                     }
+                    this.setProcessStatus("crashed");
                     // Capture last output before killing. Best-effort even when the pane is
                     // gone (paneStatus null) — gives the crash record something to diagnose
                     // from instead of an empty lastOutput.
@@ -2433,17 +2442,23 @@ export class Daemon extends EventEmitter {
             await new Promise(r => setTimeout(r, enterSettleMs));
             let enterAt = Date.now();
             await this.tmux.sendSpecialKey("Enter");
-            // Kiro may expose its ready prompt before its final startup redraw. If
-            // that redraw swallows Enter, the redraw's own output makes the normal
-            // busy confirmation return true and suppresses the conditional retry.
-            // Kiro has no native input queue, so one defensive retry on the first
-            // post-ready delivery is safe and cannot mutate a queued turn.
-            if (enterSettleMs > NORMAL_ENTER_SETTLE_MS
-                && this.backend?.requiresFirstDeliveryEnterRetry?.() === true) {
+            // Kiro's legacy TUI can swallow Enter while it is still processing a large
+            // paste — not only during the post-ready redraw (#479): on slower hosts it
+            // happens on ordinary deliveries too (v2.1.2 stable, WSL2 + tmux 3.4).
+            // The busy confirmation below cannot be trusted to catch that: it accepts
+            // ANY output after Enter, and the paste's own late render satisfies it, so
+            // the message is confirmed ✅ while the text sits unsubmitted. Kiro has no
+            // native input queue and a bare Enter is a no-op both at an empty prompt
+            // and during generation (both verified live on kiro-cli 2.16.1), so the
+            // defensive retry runs on EVERY delivery — which is also what the legacy
+            // pasteText path has always done for queue-less backends. enterAt is
+            // re-baselined to the second Enter so leftover paste-render output between
+            // the two cannot be what "confirms" the submission.
+            if (this.backend?.requiresDeliveryEnterRetry?.() === true) {
                 await new Promise(r => setTimeout(r, 1_000));
                 enterAt = Date.now();
                 await this.tmux.sendSpecialKey("Enter");
-                this.logger.debug("First delivery after ready — sent defensive Enter retry");
+                this.logger.debug("Sent defensive Enter retry (queue-less TUI can swallow the first)");
             }
             if (status)
                 this.emit("message_delivered", status); // 👀