@code-yeongyu/senpi 2026.8.26 → 2026.8.28

package/dist/modes/interactive/interactive-mode.js

@@ -17,7 +17,7 @@ import { envValue } from "../../core/brand.js";
 import { CACHE_TTL_MS, collectCacheMisses, computeCacheWaste, detectCacheMiss, } from "../../core/cache-stats.js";
 import { buildNoticeBox } from "../../core/extensions/notice/index.js";
 import { FooterDataProvider } from "../../core/footer-data-provider.js";
-import { appendHiddenTuiStdout } from "../../core/hidden-stdout-log.js";
+import { appendHiddenTuiStdout, appendUncaughtCrashLog } from "../../core/hidden-stdout-log.js";
 import { buildHighReasoningWarning } from "../../core/high-reasoning-warning.js";
 import { configureHttpDispatcher, formatHttpIdleTimeoutMs } from "../../core/http-dispatcher.js";
 import { KeybindingsManager } from "../../core/keybindings.js";
@@ -143,6 +143,8 @@ function isCompactionCostNotice(item) {
     return "type" in item && item.type === "compaction_cost";
 }
 const DEAD_TERMINAL_ERROR_CODES = new Set(["EIO", "EPIPE", "ENOTCONN"]);
+// Bun's macOS tty shim can report a dead terminal as raw positive errno 5 without a string code.
+const EIO_ERRNO = 5;
 const DEFAULT_RETRY_STATUS_REFRESH_INTERVAL_MS = 80;
 const LARGE_SESSION_RETRY_STATUS_REFRESH_INTERVAL_MS = 60_000;
 const DEFAULT_WORKING_STATUS_REFRESH_INTERVAL_MS = 600;
@@ -207,11 +209,20 @@ function formatWorkingStatusShimmerText(text, intensity) {
     return `\x1b[1m\x1b[38;2;${color.r};${color.g};${color.b}m${text}\x1b[39m\x1b[22m`;
 }
 function isDeadTerminalError(error) {
-    if (!error || typeof error !== "object" || !("code" in error)) {
+    if (!error || typeof error !== "object") {
         return false;
     }
-    const code = error.code;
-    return code !== undefined && DEAD_TERMINAL_ERROR_CODES.has(code);
+    if ("code" in error) {
+        const code = error.code;
+        if (code !== undefined && DEAD_TERMINAL_ERROR_CODES.has(code)) {
+            return true;
+        }
+    }
+    if ("errno" in error) {
+        const errno = error.errno;
+        return errno === EIO_ERRNO || errno === -EIO_ERRNO;
+    }
+    return false;
 }
 const ANTHROPIC_SUBSCRIPTION_AUTH_WARNING = "Anthropic subscription auth is active. Third-party harness usage draws from extra usage and is billed per token, not your Claude plan limits. Manage extra usage at https://claude.ai/settings/usage. Disable this warning in /settings.";
 function isAnthropicSubscriptionAuthKey(apiKey) {
@@ -479,6 +490,8 @@ export class InteractiveMode {
         this.ui.setMaxRenderFps(fps);
     }
     // Convenience accessors
+    // The session may be the local AgentSession or the shared-host RPC proxy; the
+    // four reads widened on InteractiveSession must be awaited at every call site.
     get session() {
         return this.runtimeHost.session;
     }
@@ -3474,6 +3487,7 @@ export class InteractiveMode {
             case "thinking_level_changed":
                 this.footer.invalidate();
                 this.updateEditorBorderColor();
+                this.showStatus(`Thinking level: ${event.level}`);
                 break;
             case "high_reasoning_warning":
                 this.showHighReasoningWarning(event);
@@ -4032,8 +4046,12 @@ export class InteractiveMode {
         this.ui.requestRender();
         const removePending = () => {
             const componentIndex = this.chatContainer.children.indexOf(component);
-            const insertionIndex = spacer ? this.chatContainer.children.indexOf(spacer) : componentIndex;
-            if (spacer)
+            if (componentIndex === -1) {
+                return this.chatContainer.children.length;
+            }
+            const spacerIndex = spacer ? this.chatContainer.children.indexOf(spacer) : -1;
+            const insertionIndex = spacerIndex >= 0 ? spacerIndex : componentIndex;
+            if (spacerIndex >= 0 && spacer)
                 this.chatContainer.removeChild(spacer);
             this.chatContainer.removeChild(component);
             return insertionIndex;
@@ -4479,8 +4497,17 @@ export class InteractiveMode {
         }
         process.exit(0);
     }
-    emergencyTerminalExit() {
+    emergencyTerminalExit(crash) {
         this.isShuttingDown = true;
+        // This exit is silent by design (the terminal is gone, so a banner would go
+        // nowhere), which makes the debug log the ONLY surface that can record this
+        // crash class — exactly the EIO case that left the 2026-08-26 diagnosis with no
+        // evidence. Write before the cleanup below, which is unguarded and could throw.
+        // A logging failure must never alter the exit path.
+        try {
+            appendUncaughtCrashLog(crash.origin, crash.error);
+        }
+        catch { }
         this.unregisterSignalHandlers();
         killTrackedDetachedChildren();
         // The terminal is gone. Do not run normal shutdown because TUI and
@@ -4502,6 +4529,13 @@ export class InteractiveMode {
         if (this.isShuttingDown) {
             process.exit(1);
         }
+        if (isDeadTerminalError(error)) {
+            // The terminal died under the session (e.g. an stdin read EIO after
+            // the controlling terminal vanished or this pgrp lost the tty
+            // foreground). Same handling as terminal write errors: exit silently
+            // instead of printing a crash banner to a terminal that is gone.
+            this.emergencyTerminalExit({ origin: `dead-terminal ${origin}`, error });
+        }
         if (isRecoverableInspectorVmImportError(error, origin)) {
             this.showWarning(INSPECTOR_VM_IMPORT_WARNING);
             return;
@@ -4519,6 +4553,13 @@ export class InteractiveMode {
             this.ui.stop();
         }
         catch { }
+        // Record the crash before the terminal handoff: the banner below only reaches
+        // terminal scrollback, which is gone when the terminal is closed or is itself
+        // the thing that failed. A logging failure must never alter the crash path.
+        try {
+            appendUncaughtCrashLog(origin, error);
+        }
+        catch { }
         restoreInteractiveStderr();
         console.error(`${APP_NAME} exiting due to uncaughtException:`);
         console.error(error);
@@ -4552,7 +4593,7 @@ export class InteractiveMode {
         }
         const terminalErrorHandler = (error) => {
             if (isDeadTerminalError(error)) {
-                this.emergencyTerminalExit();
+                this.emergencyTerminalExit({ origin: "dead-terminal stdio error", error });
             }
             throw error;
         };
@@ -4700,16 +4741,15 @@ export class InteractiveMode {
         }
         this.ui.requestRender();
     }
-    cycleThinkingLevel() {
-        const newLevel = this.session.cycleThinkingLevel();
+    async cycleThinkingLevel() {
+        // The shared-host proxy answers this over RPC. The level itself is rendered
+        // from the thinking_level_changed event (single path for local and remote,
+        // and for changes made by OTHER attached clients); the awaited value only
+        // distinguishes "model does not support thinking".
+        const newLevel = await this.session.cycleThinkingLevel();
         if (newLevel === undefined) {
             this.showStatus("Current model does not support thinking");
         }
-        else {
-            this.footer.invalidate();
-            this.updateEditorBorderColor();
-            this.showStatus(`Thinking level: ${newLevel}`);
-        }
     }
     async cycleModel(direction) {
         try {
@@ -5191,7 +5231,9 @@ export class InteractiveMode {
         this.ui.setFocus(created.focus);
         this.ui.requestRender();
     }
-    showSettingsSelector() {
+    async showSettingsSelector() {
+        // Awaited at the boundary: the shared-host proxy answers this over RPC.
+        const availableThinkingLevels = await this.session.getAvailableThinkingLevels();
         this.showSelector((done) => {
             let selector;
             selector = new SettingsSelectorComponent({
@@ -5206,7 +5248,7 @@ export class InteractiveMode {
                 transport: this.settingsManager.getTransport(),
                 httpIdleTimeoutMs: this.settingsManager.getHttpIdleTimeoutMs(),
                 thinkingLevel: this.session.thinkingLevel,
-                availableThinkingLevels: this.session.getAvailableThinkingLevels(),
+                availableThinkingLevels,
                 currentTheme: this.themeController.getThemeSelection() || "dark",
                 terminalTheme: this.themeController.getTerminalTheme(),
                 availableThemes: getAvailableThemes(),
@@ -5794,8 +5836,9 @@ export class InteractiveMode {
             };
         });
     }
-    showUserMessageSelector() {
-        const userMessages = this.session.getUserMessagesForForking();
+    async showUserMessageSelector() {
+        // Awaited at the boundary: the shared-host proxy answers this over RPC.
+        const userMessages = await this.session.getUserMessagesForForking();
         if (userMessages.length === 0) {
             this.showStatus("No messages to fork from");
             return;
@@ -6786,8 +6829,9 @@ export class InteractiveMode {
         this.chatContainer.addChild(new Text(theme.fg("dim", `Session name set: ${sessionName ?? name}`), 1, 0));
         this.ui.requestRender();
     }
-    handleSessionCommand() {
-        const stats = this.session.getSessionStats();
+    async handleSessionCommand() {
+        // Awaited at the boundary: the shared-host proxy answers this over RPC.
+        const stats = await this.session.getSessionStats();
         const sessionName = this.sessionManager.getSessionName();
         const entries = this.sessionManager.getEntries();
         const cacheWaste = computeCacheWaste(entries, this.session.modelRuntime);