fix(tui): let idle local stop finish

steipete · steipete · commit 4b4c4d01ba5d · 2026-05-27T03:06:25.000+01:00
diff --git a/src/tui/embedded-backend.test.ts b/src/tui/embedded-backend.test.ts
@@ -818,6 +818,46 @@ describe("EmbeddedTuiBackend", () => {
     await flushMicrotasks();
   });
 
+  it("sends idle slash stop as a normal prompt so the TUI receives a terminal event", async () => {
+    const { EmbeddedTuiBackend } = await import("./embedded-backend.js");
+    const pending = deferred<{
+      payloads: Array<{ text: string }>;
+      meta: Record<string, unknown>;
+    }>();
+    agentCommandFromIngressMock.mockReturnValueOnce(pending.promise);
+
+    const backend = new EmbeddedTuiBackend();
+    const events: Array<{ event: string; payload: unknown }> = [];
+    backend.onEvent = (evt) => {
+      events.push({ event: evt.event, payload: evt.payload });
+    };
+    backend.start();
+    await backend.sendChat({
+      sessionKey: "agent:main:main",
+      message: "/stop",
+      runId: "run-local-idle-stop",
+    });
+
+    expect(agentCommandFromIngressMock).toHaveBeenCalledTimes(1);
+
+    pending.resolve({ payloads: [{ text: "idle stop prompt" }], meta: {} });
+    await flushMicrotasks();
+
+    expect(events).toContainEqual({
+      event: "chat",
+      payload: {
+        runId: "run-local-idle-stop",
+        sessionKey: "agent:main:main",
+        state: "final",
+        message: {
+          role: "assistant",
+          content: [{ type: "text", text: "idle stop prompt" }],
+          timestamp: embeddedEventTimestamp,
+        },
+      },
+    });
+  });
+
   it("queues same-session sends behind terminal local runs until maintenance settles", async () => {
     const { EmbeddedTuiBackend } = await import("./embedded-backend.js");
     const first = deferred<{
diff --git a/src/tui/embedded-backend.ts b/src/tui/embedded-backend.ts
@@ -135,11 +135,6 @@ function resolveBtwQuestion(message: string): string | undefined {
   return question ? question : undefined;
 }
 
-function isSlashStopCommand(text: string): boolean {
-  const trimmed = text.trim();
-  return trimmed.startsWith("/") && isChatStopCommandText(trimmed);
-}
-
 function payloadText(parts: unknown): string {
   if (!Array.isArray(parts)) {
     return "";
@@ -331,9 +326,7 @@ export class EmbeddedTuiBackend implements TuiBackend {
     const runId = opts.runId ?? randomUUID();
     const question = resolveBtwQuestion(opts.message);
     const abortableSessionRun = this.hasAbortableSessionRun(opts.sessionKey);
-    const stopCommand =
-      isChatStopCommandText(opts.message) &&
-      (isSlashStopCommand(opts.message) || abortableSessionRun);
+    const stopCommand = abortableSessionRun && isChatStopCommandText(opts.message);
     const queuedAfter =
       question || stopCommand ? undefined : this.findQueuedSessionRunPromise(opts.sessionKey);
     if (stopCommand) {
