@gajae-code/coding-agent 0.12.1 → 0.12.4

package/src/modes/DESIGN.md

@@ -291,6 +291,59 @@ textual state without SGR. Korean, Japanese, Chinese, and mixed CJK/Latin prose
 must wrap at semantic phrase boundaries, never through an action/status label or
 short identifier; a semantic CJK break is a visual-QA failure.
 
+### Direct-root anatomy, IRC lane, and pin boundary
+
+The production root has one ordered, direct-child anatomy. Do not wrap, reorder,
+or independently pin these regions:
+
+1. `ircSplitView` is the viewport anchor and owns the transcript (and the IRC
+   sidebar when effective);
+2. `pendingMessagesContainer`, `statusContainer`, `todoContainer`, and
+   `btwContainer` follow the anchor **before** the pin boundary;
+3. `statusLine` is the pin boundary; and
+4. `hookWidgetContainerAbove`, `editorContainer`, `petFloorContainer`, and
+   `hookWidgetContainerBelow` are the later direct composer children in that
+   order.
+
+`statusContainer` is transient operation/status content in the pre-boundary
+application flow; it is not the persistent `statusLine` telemetry rail. The
+status line begins the fixed suffix. The focused editor, its cursor, and every
+later direct composer child remain below that boundary during manual history,
+streaming, and reflow. Pending messages, todo, and BTW therefore remain
+transcript-adjacent rather than becoming accidental fixed chrome.
+
+IRC has one shared work-lane geometry. At **64 cells or narrower** the IRC lane
+is ineffective and transcript/todo use the full width. At **65 cells** it is
+exactly **32 / 3 / 30** cells (left work lane / separator / IRC lane). At wider
+sizes the same split calculation applies; todo uses the left work lane whenever
+IRC is effective, including empty, streaming, and long IRC histories. Todo
+does not create a second sidebar, a separately rounded width, or a different
+collapse rule.
+
+Todo is absent when empty. Populated todos support long text, multiple phases,
+and collapsed/expanded views without crossing the separator or overflowing
+their work lane. The active phase is retained in collapsed view; expanded view
+keeps phase/task order. IRC empty, streaming, and long states preserve the
+same root order and pin boundary.
+
+Row reservation is content-priority based: reserve the focused composer and
+status line first, then normal hooks; when height is constrained, drop the
+manual-output notice first, then the decorative pet, then low-priority hooks.
+A zero-row transcript capacity is valid. Neither reservation nor degradation
+may hide focus/cursor geometry, move an anchor into pinned chrome, or turn a
+resize into a follow/manual state change. Manual and follow paths, streaming
+updates, width growth/shrink, and height growth/shrink preserve the anchor
+semantics; resize must recompute the shared IRC/todo lane before rendering.
+
+All lane measurement, clipping, and wrapping is ANSI-aware terminal-cell
+measurement. Terminal graphics may be shown only where the effective layout
+permits them and must degrade to the textual graphics fallback without changing
+row ownership. ANSI/color output preserves visible cursor/focus state; ASCII
+and no-color output retains textual selection/status affordances. CJK and mixed
+CJK/Latin todo, IRC, status, and BTW text break at semantic phrase boundaries,
+not inside a phase/action/status label, short identifier, or grapheme cluster.
+Those defects, width overflow, overlap, hidden focus/cursor, and lost anchors
+are blocking visual-QA failures.
 ### Sticky viewport deterministic visual QA
 
 `test/fixtures/tui/sticky-viewport-showcase.ts` is a fixed-clock, no-network,

package/src/modes/components/irc-sidebar.ts

@@ -82,6 +82,35 @@ export function computeIrcSplitWidths(width: number): {
 		rightWidth: sidebarWidth,
 	};
 }
+/**
+ * Resolves the single shared work-lane geometry. The left lane is used by both
+ * transcript and todo content whenever the IRC lane is effective.
+ */
+export function computeIrcWorkLaneWidths(
+	width: number,
+	sidebarVisible: boolean,
+): { leftWidth: number; separatorWidth: number; rightWidth: number } {
+	const fullWidth = Math.max(0, Math.floor(width));
+	return sidebarVisible
+		? computeIrcSplitWidths(fullWidth)
+		: { leftWidth: fullWidth, separatorWidth: 0, rightWidth: 0 };
+}
+
+export class IrcLeftLaneComponent implements Component {
+	constructor(
+		private readonly content: Component,
+		private readonly isSidebarVisible: (width: number) => boolean,
+	) {}
+
+	render(width: number): string[] {
+		const layout = computeIrcWorkLaneWidths(width, this.isSidebarVisible(width));
+		return this.content.render(layout.leftWidth);
+	}
+
+	invalidate(): void {
+		this.content.invalidate?.();
+	}
+}
 
 type SemanticLine = Readonly<{
 	kind: "header" | "body" | "elision" | "blank";
@@ -288,7 +317,7 @@ export class IrcSplitViewComponent implements ViewportAnchorProvider {
 	renderWithViewportAnchors(width: number): ViewportAnchorRender {
 		if (!this.#visible) return renderComponentWithViewportAnchors(this.leftPane, width);
 		const componentTheme = typeof this.componentTheme === "function" ? this.componentTheme() : this.componentTheme;
-		const { leftWidth, separatorWidth, rightWidth } = computeIrcSplitWidths(width);
+		const { leftWidth, separatorWidth, rightWidth } = computeIrcWorkLaneWidths(width, this.#visible);
 		if (rightWidth === 0) return renderComponentWithViewportAnchors(this.leftPane, width);
 		const separator = separatorWidth > 0 ? componentTheme.fg("dim", ` ${componentTheme.boxSharp.vertical} `) : "";
 		const leftRender = withTerminalGraphicsFallback(

package/src/modes/components/tool-status-header.ts

@@ -206,7 +206,6 @@ export class StatusLineComponent implements Component {
 			separator: settings.get("statusLine.separator"),
 			showHookStatus: settings.get("statusLine.showHookStatus"),
 			showSkillHud: settings.get("statusLine.showSkillHud"),
-			showActionHints: settings.get("statusLine.showActionHints"),
 			segmentOptions: settings.getGroup("statusLine").segmentOptions,
 			sessionAccent: settings.get("statusLine.sessionAccent"),
 			maxRows: settings.get("statusLine.maxRows"),

package/src/modes/controllers/command-controller.ts

@@ -34,7 +34,12 @@ import { BorderedLoader } from "../../modes/components/bordered-loader";
 import { DynamicBorder } from "../../modes/components/dynamic-border";
 import { EvalExecutionComponent } from "../../modes/components/eval-execution";
 import { getMarkdownTheme, getSymbolTheme, theme } from "../../modes/theme/theme";
-import type { InteractiveModeContext } from "../../modes/types";
+import {
+	clearInteractiveActivityLoaders,
+	type InteractiveModeContext,
+	stopInteractiveActivityIndicator,
+	suspendInteractiveActivityIndicator,
+} from "../../modes/types";
 import { computeContextBreakdown, renderContextUsage } from "../../modes/utils/context-usage";
 import { buildHotkeysMarkdown, formatHotkeyMarkdownCode } from "../../modes/utils/hotkeys-markdown";
 import { buildToolsMarkdown } from "../../modes/utils/tools-markdown";
@@ -965,12 +970,10 @@ export class CommandController {
 
 	async #runNewSessionFlow(options?: NewSessionOptions, label: string = "New session started"): Promise<boolean> {
 		if (!(await this.ctx.session.newSession(options))) return false;
+		if (this.ctx.isStopped?.()) return false;
+		clearInteractiveActivityLoaders(this.ctx);
 
-		if (this.ctx.loadingAnimation) {
-			this.ctx.loadingAnimation.stop();
-			this.ctx.loadingAnimation = undefined;
-		}
-		this.ctx.statusContainer.clear();
+		stopInteractiveActivityIndicator(this.ctx);
 		this.ctx.resetIrcSidebarSession();
 		this.ctx.resetObserverRegistry();
 		setSessionTerminalTitle(this.ctx.sessionManager.getSessionName(), this.ctx.sessionManager.getCwd());
@@ -991,6 +994,7 @@ export class CommandController {
 
 		this.ctx.chatContainer.addChild(new Text(`${theme.fg("accent", `${theme.status.success} ${label}`)}`, 1, 0));
 		await this.ctx.reloadTodos();
+		if (this.ctx.isStopped?.()) return false;
 		this.ctx.ui.requestRender();
 		return true;
 	}
@@ -1000,19 +1004,18 @@ export class CommandController {
 	}
 
 	async handleContextClearCommand(): Promise<void> {
-		if (this.ctx.loadingAnimation) {
-			this.ctx.loadingAnimation.stop();
-			this.ctx.loadingAnimation = undefined;
-		}
-		this.ctx.statusContainer.clear();
-
+		if (this.ctx.isStopped?.()) return;
+		clearInteractiveActivityLoaders(this.ctx);
+		stopInteractiveActivityIndicator(this.ctx);
 		if (this.ctx.session.isCompacting) {
 			this.ctx.session.abortCompaction();
 			while (this.ctx.session.isCompacting) {
 				await Bun.sleep(10);
 			}
 		}
+		if (this.ctx.isStopped?.()) return;
 		if (!(await this.ctx.session.clearContext())) return;
+		if (this.ctx.isStopped?.()) return;
 		setSessionTerminalTitle(this.ctx.sessionManager.getSessionName(), this.ctx.sessionManager.getCwd());
 
 		this.ctx.statusLine.invalidate();
@@ -1033,6 +1036,7 @@ export class CommandController {
 			new Text(`${theme.fg("accent", `${theme.status.success} Context cleared`)}`, 1, 0),
 		);
 		await this.ctx.reloadTodos();
+		if (this.ctx.isStopped?.()) return;
 		this.ctx.ui.requestRender();
 	}
 
@@ -1049,17 +1053,15 @@ export class CommandController {
 			this.ctx.showWarning("Wait for the current response to finish or abort it before forking.");
 			return;
 		}
-		if (this.ctx.loadingAnimation) {
-			this.ctx.loadingAnimation.stop();
-			this.ctx.loadingAnimation = undefined;
-		}
-		this.ctx.statusContainer.clear();
 
 		const success = await this.ctx.session.fork();
+		if (this.ctx.isStopped?.()) return;
 		if (!success) {
 			this.ctx.showError("Fork failed (session not persisted or cancelled)");
 			return;
 		}
+		clearInteractiveActivityLoaders(this.ctx);
+		stopInteractiveActivityIndicator(this.ctx);
 		this.ctx.resetIrcSidebarSession();
 
 		this.ctx.statusLine.invalidate();
@@ -1256,11 +1258,7 @@ export class CommandController {
 		customInstructionsOrOptions?: string | CompactOptions,
 		isAuto = false,
 	): Promise<CompactionOutcome> {
-		if (this.ctx.loadingAnimation) {
-			this.ctx.loadingAnimation.stop();
-			this.ctx.loadingAnimation = undefined;
-		}
-		this.ctx.statusContainer.clear();
+		const releaseActivityIndicator = suspendInteractiveActivityIndicator(this.ctx);
 
 		const originalOnEscape = this.ctx.editor.onEscape;
 		this.ctx.editor.onEscape = () => {
@@ -1287,24 +1285,28 @@ export class CommandController {
 					? customInstructionsOrOptions
 					: undefined;
 			await this.ctx.session.compact(instructions, options);
+			if (this.ctx.isStopped?.()) return outcome;
 
 			this.ctx.rebuildChatFromMessages("reconcile-same-transcript");
 
 			this.ctx.statusLine.invalidate();
 			this.ctx.updateEditorTopBorder();
 		} catch (error) {
-			if (error instanceof CompactionCancelledError) {
-				outcome = "cancelled";
+			outcome = error instanceof CompactionCancelledError ? "cancelled" : "failed";
+			if (this.ctx.isStopped?.()) return outcome;
+			if (outcome === "cancelled") {
 				this.ctx.showError("Compaction cancelled");
 			} else {
-				outcome = "failed";
 				const message = error instanceof Error ? error.message : String(error);
 				this.ctx.showError(`Compaction failed: ${message}`);
 			}
 		} finally {
 			compactingLoader.stop();
-			this.ctx.statusContainer.clear();
-			this.ctx.editor.onEscape = originalOnEscape;
+			if (!this.ctx.isStopped?.()) {
+				this.ctx.statusContainer.clear();
+				this.ctx.editor.onEscape = originalOnEscape;
+			}
+			releaseActivityIndicator();
 		}
 		await this.ctx.flushCompactionQueue({ willRetry: false });
 		return outcome;
@@ -1323,11 +1325,7 @@ export class CommandController {
 			return;
 		}
 
-		if (this.ctx.loadingAnimation) {
-			this.ctx.loadingAnimation.stop();
-			this.ctx.loadingAnimation = undefined;
-		}
-		this.ctx.statusContainer.clear();
+		const releaseActivityIndicator = suspendInteractiveActivityIndicator(this.ctx);
 
 		const originalOnEscape = this.ctx.editor.onEscape;
 		this.ctx.editor.onEscape = () => {
@@ -1347,6 +1345,7 @@ export class CommandController {
 		try {
 			// Handoff generation runs as a oneshot request; the new session is shown after it completes.
 			const result = await this.ctx.session.handoff(customInstructions);
+			if (this.ctx.isStopped?.()) return;
 
 			if (!result) {
 				this.ctx.showError("Handoff cancelled");
@@ -1361,6 +1360,7 @@ export class CommandController {
 			this.ctx.updateEditorTopBorder();
 			this.ctx.updateEditorBorderColor();
 			await this.ctx.reloadTodos();
+			if (this.ctx.isStopped?.()) return;
 
 			this.ctx.chatContainer.addChild(new Spacer(1));
 			this.ctx.chatContainer.addChild(
@@ -1370,6 +1370,7 @@ export class CommandController {
 				this.ctx.showStatus(`Handoff document saved to: ${result.savedPath}`);
 			}
 		} catch (error) {
+			if (this.ctx.isStopped?.()) return;
 			const message = error instanceof Error ? error.message : String(error);
 			if (message === "Handoff cancelled" || (error instanceof Error && error.name === "AbortError")) {
 				this.ctx.showError("Handoff cancelled");
@@ -1389,8 +1390,11 @@ export class CommandController {
 			}
 		} finally {
 			handoffLoader.stop();
-			this.ctx.statusContainer.clear();
-			this.ctx.editor.onEscape = originalOnEscape;
+			if (!this.ctx.isStopped?.()) {
+				this.ctx.statusContainer.clear();
+				this.ctx.editor.onEscape = originalOnEscape;
+			}
+			releaseActivityIndicator();
 		}
 	}
 

package/src/modes/controllers/event-controller.ts

@@ -15,7 +15,12 @@ import { TodoReminderComponent } from "../../modes/components/todo-reminder";
 import { ToolExecutionComponent, type ToolExecutionHandle } from "../../modes/components/tool-execution";
 import { TtsrNotificationComponent } from "../../modes/components/ttsr-notification";
 import { getSymbolTheme, theme } from "../../modes/theme/theme";
-import type { InteractiveModeContext, TodoPhase } from "../../modes/types";
+import {
+	type InteractiveModeContext,
+	stopInteractiveActivityIndicator,
+	syncInteractiveActivityIndicator,
+	type TodoPhase,
+} from "../../modes/types";
 import type { PlanApprovalDetails } from "../../plan-mode/approved-plan";
 import { completionNotifyDisabledByEnv } from "../../sdk/bus/config";
 import { summaryFromMessage } from "../../sdk/bus/helpers";
@@ -166,6 +171,14 @@ export class EventController {
 			this.ctx.retryEscapeHandler = undefined;
 		}
 		this.ctx.retryEscapePrimed = false;
+		if (this.ctx.autoCompactionEscapeHandler) {
+			this.ctx.editor.onEscape = this.ctx.autoCompactionEscapeHandler;
+			this.ctx.autoCompactionEscapeHandler = undefined;
+		}
+		if (this.ctx.autoCompactionLoader) {
+			this.ctx.autoCompactionLoader.stop();
+			this.ctx.autoCompactionLoader = undefined;
+		}
 		if (this.ctx.retryLoader) {
 			this.ctx.retryLoader.stop();
 			this.ctx.retryLoader = undefined;
@@ -266,7 +279,9 @@ export class EventController {
 	}
 
 	async #dispatchEvent(event: AgentSessionEvent): Promise<void> {
+		if (this.ctx.isStopped?.()) return;
 		if (!this.ctx.isInitialized) await this.ctx.init();
+		if (this.ctx.isStopped?.()) return;
 		this.#visibleTranscriptChanged = false;
 		this.#handlingEvent = true;
 		try {
@@ -274,6 +289,7 @@ export class EventController {
 			this.ctx.updateEditorTopBorder();
 			const run = this.#handlers[event.type] as (e: AgentSessionEvent) => Promise<void>;
 			await run(event);
+			if (this.ctx.isStopped?.()) return;
 			if (this.#visibleTranscriptChanged) this.ctx.recordVisibleTranscriptMutation?.();
 			if (this.ctx.isTranscriptViewerOpen?.()) this.ctx.refreshTranscriptViewer?.();
 		} finally {
@@ -878,23 +894,22 @@ export class EventController {
 				const planDetails = details.sourceResultDetails as PlanApprovalDetails | undefined;
 				if (planDetails) {
 					await this.ctx.planModeController.handleApproval(planDetails);
+					if (this.ctx.isStopped?.()) return;
 				}
 			}
 		}
 	}
 
 	async #handleAgentEnd(_event: Extract<AgentSessionEvent, { type: "agent_end" }>): Promise<void> {
-		if (this.ctx.loadingAnimation) {
-			this.ctx.loadingAnimation.stop();
-			this.ctx.loadingAnimation = undefined;
-			this.ctx.statusContainer.clear();
-		}
+		this.ctx.setWorkingMessage(undefined);
+		stopInteractiveActivityIndicator(this.ctx);
 		if (this.ctx.streamingComponent) {
 			this.ctx.chatContainer.removeChild(this.ctx.streamingComponent);
 			this.ctx.streamingComponent = undefined;
 			this.ctx.streamingMessage = undefined;
 		}
 		await this.ctx.planModeController.flushPendingModelSwitch();
+		if (this.ctx.isStopped?.()) return;
 		for (const toolCallId of Array.from(this.ctx.pendingTools.keys())) {
 			if (!this.#backgroundToolCallIds.has(toolCallId)) {
 				this.ctx.pendingTools.delete(toolCallId);
@@ -922,11 +937,7 @@ export class EventController {
 		this.ctx.editor.onEscape = () => {
 			this.ctx.session.abortCompaction();
 		};
-		if (this.ctx.loadingAnimation) {
-			this.ctx.loadingAnimation.stop();
-			this.ctx.loadingAnimation = undefined;
-		}
-		this.ctx.statusContainer.clear();
+		stopInteractiveActivityIndicator(this.ctx, { restoreBackground: false });
 		const reasonText =
 			event.reason === "overflow" ? "Context overflow detected, " : event.reason === "idle" ? "Idle " : "";
 		const actionLabel = event.action === "handoff" ? "Auto-handoff" : "Auto context-full maintenance";
@@ -982,6 +993,7 @@ export class EventController {
 			this.ctx.statusLine.invalidate();
 			this.ctx.updateEditorTopBorder();
 			await this.ctx.reloadTodos();
+			if (this.ctx.isStopped?.()) return;
 			this.ctx.showStatus("Auto-handoff completed");
 		} else if (event.skipped) {
 			// Benign skip: no model selected, no candidate models available, or nothing
@@ -995,6 +1007,8 @@ export class EventController {
 			this.ctx.showWarning("Auto context-full maintenance failed; continuing without maintenance");
 		}
 		await this.ctx.flushCompactionQueue({ willRetry: event.willRetry });
+		if (this.ctx.isStopped?.()) return;
+		syncInteractiveActivityIndicator(this.ctx);
 		this.ctx.ui.requestRender();
 	}
 
@@ -1024,11 +1038,7 @@ export class EventController {
 				this.ctx.session.abortRetry();
 			}
 		};
-		if (this.ctx.loadingAnimation) {
-			this.ctx.loadingAnimation.stop();
-			this.ctx.loadingAnimation = undefined;
-		}
-		this.ctx.statusContainer.clear();
+		stopInteractiveActivityIndicator(this.ctx, { restoreBackground: false });
 		// Stop any prior retry loader/timer before installing a new one.
 		this.ctx.retryLoader?.stop();
 		this.#clearRetryCountdown();
@@ -1071,6 +1081,7 @@ export class EventController {
 		if (!event.success) {
 			this.ctx.showError(`Retry failed after ${event.attempt} attempts: ${event.finalError || "Unknown error"}`);
 		}
+		syncInteractiveActivityIndicator(this.ctx);
 		this.ctx.ui.requestRender();
 	}