From 18fa9d1955779bbf22ab2f8af485c16efd571455 Mon Sep 17 00:00:00 2001 From: Dallin Romney Date: Sun, 17 May 2026 21:21:20 -0700 Subject: [PATCH] fix(tui): drop unnecessary comments from watchdog dismiss helpers --- src/tui/components/chat-log.ts | 3 --- src/tui/tui-event-handlers.ts | 3 --- 2 files changed, 6 deletions(-) diff --git a/src/tui/components/chat-log.ts b/src/tui/components/chat-log.ts index 138001f72a9..e8e8a7b15b7 100644 --- a/src/tui/components/chat-log.ts +++ b/src/tui/components/chat-log.ts @@ -109,9 +109,6 @@ export class ChatLog extends Container { this.append(this.createSystemMessage(text)); } - // Tag a system notice with a runId so a later chat event for the same run - // can dismiss it. Used by the streaming watchdog so the "taking longer than - // expected" notice goes away if the response actually arrives afterwards. addPendingSystem(runId: string, text: string) { const existing = this.pendingSystemNotices.get(runId); if (existing) { diff --git a/src/tui/tui-event-handlers.ts b/src/tui/tui-event-handlers.ts index 7ac707d349b..7880686be63 100644 --- a/src/tui/tui-event-handlers.ts +++ b/src/tui/tui-event-handlers.ts @@ -393,9 +393,6 @@ export function createEventHandlers(context: EventHandlerContext) { if (reconnectPendingRunId === evt.runId) { reconnectPendingRunId = null; } - // Any chat event for a run that previously tripped the streaming watchdog - // is proof the response is no longer stuck — drop the stale notice so it - // doesn't sit alongside the recovered content. chatLog.dismissPendingSystem(evt.runId); noteSessionRun(evt.runId); if (!state.activeChatRunId && !isLocalBtwRunId?.(evt.runId)) {