lsd-pi 1.3.2 → 1.3.7

package/packages/pi-coding-agent/src/modes/interactive/controllers/extension-ui-controller.ts

@@ -53,7 +53,7 @@ export function createExtensionUIContext(host: any): ExtensionUIContext {
 			}
 			return result;
 		},
-		getToolsExpanded: () => host.toolOutputExpanded,
+		getToolsExpanded: () => host.collapsedToolCallsExpanded || host.toolOutputExpanded,
 		setToolsExpanded: (expanded) => host.setToolsExpanded(expanded),
 	};
 }

package/packages/pi-coding-agent/src/modes/interactive/controllers/input-controller.ts

@@ -54,6 +54,13 @@ export function setupEditorSubmitHandler(host: InteractiveModeStateHost & {
 		}
 
 		if (host.session.isStreaming) {
+			if (text === "/btw" || text.startsWith("/btw ")) {
+				const handled = await dispatchSlashCommand(text, host.getSlashCommandContext());
+				if (handled) {
+					host.editor.setText("");
+					return;
+				}
+			}
 			host.editor.addToHistory?.(text);
 			host.editor.setText("");
 			host.recordLastSentPrompt?.(text);

package/packages/pi-coding-agent/src/modes/interactive/interactive-mode-state.ts

@@ -11,8 +11,12 @@ export interface InteractiveModeStateHost {
 	chatContainer: any;
 	settingsManager: any;
 	pendingTools: Map<string, any>;
+	collapsedToolSummaryLine?: any;
+	collapsedToolCallsExpanded: boolean;
 	toolOutputExpanded: boolean;
 	hideThinkingBlock: boolean;
+	notificationSoundEnabled: boolean;
+	playNotificationSoundOnAgentEnd: boolean;
 	isBashMode: boolean;
 	onInputCallback?: (text: string) => void;
 	isInitialized: boolean;

package/packages/pi-coding-agent/src/modes/interactive/interactive-mode.ts

@@ -99,6 +99,7 @@ import { SessionSelectorComponent } from "./components/session-selector.js";
 import { SettingsSelectorComponent } from "./components/settings-selector.js";
 import { SkillInvocationMessageComponent } from "./components/skill-invocation-message.js";
 import { ToolExecutionComponent } from "./components/tool-execution.js";
+import { ToolSummaryLine } from "./components/tool-summary-line.js";
 import { TreeSelectorComponent } from "./components/tree-selector.js";
 import { UserMessageComponent } from "./components/user-message.js";
 import { UserMessageSelectorComponent } from "./components/user-message-selector.js";
@@ -320,14 +321,20 @@ export class InteractiveMode {
 
 	// Tool execution tracking: toolCallId -> component
 	private pendingTools = new Map<string, ToolExecutionComponent>();
+	private collapsedToolSummaryLine: ToolSummaryLine | undefined = undefined;
 	private agentPtyComponents = new Map<string, EmbeddedTerminalComponent>();
 
 	// Tool output expansion state
+	private collapsedToolCallsExpanded = false;
 	private toolOutputExpanded = false;
 
 	// Thinking block visibility state
 	private hideThinkingBlock = false;
 
+	// Notification sound (terminal bell on completed agent response)
+	notificationSoundEnabled = false;
+	playNotificationSoundOnAgentEnd = false;
+
 	// Pin last prompt feature
 	private pinLastPromptEnabled = false;
 	private lastSentPromptText: string | undefined = undefined;
@@ -435,6 +442,11 @@ export class InteractiveMode {
 		// Load pin last prompt setting
 		this.pinLastPromptEnabled = this.settingsManager.getPinLastPrompt();
 
+		// Load notification sound setting
+		this.notificationSoundEnabled = this.settingsManager.getNotificationSound();
+		this.footer.setNotificationSoundEnabled(this.notificationSoundEnabled);
+		this.footer.setVerboseFooterEnabled(this.settingsManager.getVerboseFooter());
+
 		// Register themes from resource loader and initialize
 		setRegisteredThemes(this.session.resourceLoader.getThemes().themes);
 		initTheme(this.settingsManager.getTheme(), true, this.settingsManager.getThemeAccent());
@@ -2327,6 +2339,7 @@ export class InteractiveMode {
 		this.defaultEditor.onAction("cyclePermissionMode", () => this.cyclePermissionMode());
 		this.defaultEditor.onAction("showHotkeys", () => showHotkeys(this.getSlashCommandContext()));
 		this.defaultEditor.onAction("terminalFocus", () => this.toggleEmbeddedTerminalFocus());
+		this.defaultEditor.onAction("toggleNotificationSound", () => this.toggleNotificationSound());
 
 		// Global debug handler on TUI (works regardless of focus)
 		this.ui.onDebug = () => this.handleDebugCommand();
@@ -2347,6 +2360,7 @@ export class InteractiveMode {
 			if (wasBashMode !== this.isBashMode) {
 				this.updateEditorBorderColor();
 			}
+			this.updateEditorExpandHint();
 		};
 
 		// Handle clipboard image paste (triggered on Ctrl+V)
@@ -3035,6 +3049,14 @@ export class InteractiveMode {
 		}
 	}
 
+	private toggleNotificationSound(): void {
+		this.notificationSoundEnabled = !this.notificationSoundEnabled;
+		this.settingsManager.setNotificationSound(this.notificationSoundEnabled);
+		this.footer.setNotificationSoundEnabled(this.notificationSoundEnabled);
+		this.ui.requestRender();
+		this.showStatus(`Notification sound: ${this.notificationSoundEnabled ? "enabled" : "disabled"}`);
+	}
+
 	private async cycleModel(direction: "forward" | "backward"): Promise<void> {
 		try {
 			const result = await this.session.cycleModel(direction);
@@ -3053,57 +3075,92 @@ export class InteractiveMode {
 		}
 	}
 
+	private hasCollapsedToolCallSummaries(): boolean {
+		return this.chatContainer.children.some((child) => child instanceof ToolSummaryLine);
+	}
+
+	private shouldUseCollapsedToolCallIntermediateState(): boolean {
+		return this.settingsManager.getCollapseToolCalls() && this.hasCollapsedToolCallSummaries();
+	}
+
 	private toggleToolOutputExpansion(): void {
-		this.setToolsExpanded(!this.toolOutputExpanded);
+		if (this.toolOutputExpanded) {
+			this.setToolOutputExpansionState(false, false);
+			return;
+		}
+		if (this.shouldUseCollapsedToolCallIntermediateState() && !this.collapsedToolCallsExpanded) {
+			this.setToolOutputExpansionState(true, false);
+			return;
+		}
+		this.setToolOutputExpansionState(true, true);
 	}
 
 	private setToolsExpanded(expanded: boolean): void {
-		this.toolOutputExpanded = expanded;
+		this.setToolOutputExpansionState(expanded, expanded);
+	}
+
+	private setToolOutputExpansionState(collapsedToolCallsExpanded: boolean, toolOutputExpanded: boolean): void {
+		this.collapsedToolCallsExpanded = collapsedToolCallsExpanded || toolOutputExpanded;
+		this.toolOutputExpanded = toolOutputExpanded;
+		const collapseToolCalls = this.settingsManager.getCollapseToolCalls();
+		const showCollapsedToolCalls = collapseToolCalls && this.collapsedToolCallsExpanded;
 		for (const child of this.chatContainer.children) {
 			if (isExpandable(child)) {
-				child.setExpanded(expanded);
+				child.setExpanded(toolOutputExpanded);
+			}
+			if (child instanceof ToolExecutionComponent) {
+				child.setHidden(!showCollapsedToolCalls && child.shouldHideWhenCollapsed(collapseToolCalls));
+			}
+			if (child instanceof ToolSummaryLine) {
+				child.setHidden(!collapseToolCalls || showCollapsedToolCalls);
 			}
 		}
 		if (this.bashComponent) {
-			this.bashComponent.setExpanded(expanded);
+			this.bashComponent.setExpanded(toolOutputExpanded);
 		}
 		for (const component of this.pendingBashComponents) {
-			component.setExpanded(expanded);
+			component.setExpanded(toolOutputExpanded);
 		}
 		this.updateEditorExpandHint();
 		this.ui.requestRender();
 	}
 
-	/** Append/remove the "ctrl+o to expand" hint in the editor bottom border. */
+	/** Append/remove right-aligned expand and streaming hints in editor bottom border. */
 	updateEditorExpandHint(): void {
 		const expandKey = appKey(this.keybindings, "expandTools");
 		const collapseHint = `${theme.fg("dim", expandKey)}${theme.fg("muted", " collapse")}`;
-		const expandHint = `${theme.fg("dim", expandKey)}${theme.fg("muted", " : verbose")}`;
-		// The base hint set during agent_start
-		const enterKey = theme.fg("dim", "↵");
-		const followUpKey = theme.fg("dim", appKey(this.keybindings, "followUp"));
-		const steerLabel = theme.fg("muted", " steer");
-		const queueLabel = theme.fg("muted", " queue");
-		const baseHint = `${enterKey}${steerLabel}  ${followUpKey}${queueLabel}`;
+		const expandToolCallsHint = `${theme.fg("dim", expandKey)}${theme.fg("muted", " : uncollapsed tools")}`;
+		const expandVerboseHint = `${theme.fg("dim", expandKey)}${theme.fg("muted", " : verbose")}`;
+		const editorHasText = this.defaultEditor.getText().trim().length > 0;
+		const streamingHints = editorHasText
+			? `${theme.fg("dim", "↵")}${theme.fg("muted", " steer")}  ${theme.fg("dim", appKey(this.keybindings, "followUp"))}${theme.fg("muted", " queue")}`
+			: "";
 
 		// Check if there are any expandable tool outputs in the chat
 		const hasToolOutputs =
 			this.chatContainer.children.some(isExpandable) ||
 			!!this.bashComponent ||
 			this.pendingBashComponents.length > 0;
+		const canExpandCollapsedToolCalls = this.shouldUseCollapsedToolCallIntermediateState();
 
-		const activeHint = this.toolOutputExpanded ? collapseHint : expandHint;
+		let activeHint = expandVerboseHint;
+		if (this.toolOutputExpanded) {
+			activeHint = collapseHint;
+		} else if (this.collapsedToolCallsExpanded && canExpandCollapsedToolCalls) {
+			activeHint = expandVerboseHint;
+		} else if (canExpandCollapsedToolCalls) {
+			activeHint = expandToolCallsHint;
+		}
 
-		if (this.loadingAnimation) {
-			// Agent is running - always show expand/collapse hint when there are tool outputs
-			this.defaultEditor.bottomHint = hasToolOutputs
-				? `${baseHint}  ${activeHint}`
-				: baseHint;
-		} else if (hasToolOutputs) {
-			// Idle - show expand/collapse hint so user knows ctrl+o works
-			this.defaultEditor.bottomHint = activeHint;
+		const rightHints: string[] = [];
+		if (this.loadingAnimation && streamingHints) {
+			rightHints.push(streamingHints);
+		}
+		if (hasToolOutputs) {
+			rightHints.push(activeHint);
 		}
-		// If no tool outputs and idle, leave bottomHint as-is (cleared by agent_end)
+
+		this.defaultEditor.bottomHint = rightHints.join("  ");
 	}
 
 	private toggleThinkingBlockVisibility(): void {
@@ -3454,7 +3511,9 @@ export class InteractiveMode {
 					enableSkillCommands: this.settingsManager.getEnableSkillCommands(),
 					toolProfile: this.settingsManager.getToolProfile(),
 					codexRotate: this.settingsManager.getCodexRotate(),
+					fastMode: this.settingsManager.getFastMode(),
 					cacheTimer: this.settingsManager.getCacheTimer(),
+					verboseFooter: this.settingsManager.getVerboseFooter(),
 					pinLastPrompt: this.settingsManager.getPinLastPrompt(),
 					steeringMode: this.session.steeringMode,
 					followUpMode: this.session.followUpMode,
@@ -3479,10 +3538,12 @@ export class InteractiveMode {
 					clearOnShrink: this.settingsManager.getClearOnShrink(),
 					timestampFormat: this.settingsManager.getTimestampFormat(),
 					toolOutputMode: this.settingsManager.getToolOutputMode(),
+					collapseToolCalls: this.settingsManager.getCollapseToolCalls(),
 					rtk: this.settingsManager.getRtk(),
 					editorScheme: this.settingsManager.getEditorScheme(),
 					autoDream: this.settingsManager.getAutoDream(),
 					autoMemory: this.settingsManager.getAutoMemory(),
+					notificationSound: this.notificationSoundEnabled,
 					telegramLiveRelayAutoConnect: this.settingsManager.getTelegramLiveRelayAutoConnect(),
 					sandboxEnabled: this.settingsManager.getSandboxSettings().enabled ?? true,
 					sandboxNetworkMode: this.settingsManager.getSandboxSettings().networkMode
@@ -3653,10 +3714,20 @@ export class InteractiveMode {
 						this.settingsManager.setCodexRotate(enabled);
 						this.showStatus(`Codex rotate: ${enabled ? "enabled" : "disabled"} (restart required)`);
 					},
+					onFastModeChange: (enabled) => {
+						this.settingsManager.setFastMode(enabled);
+						this.showStatus(`Fast mode: ${enabled ? "enabled" : "disabled"}`);
+					},
 					onCacheTimerChange: (enabled) => {
 						this.settingsManager.setCacheTimer(enabled);
 						this.showStatus(`Cache timer: ${enabled ? "enabled" : "disabled"}`);
 					},
+					onVerboseFooterChange: (enabled) => {
+						this.settingsManager.setVerboseFooter(enabled);
+						this.footer.setVerboseFooterEnabled(enabled);
+						this.ui.requestRender();
+						this.showStatus(`Verbose footer: ${enabled ? "enabled" : "disabled"}`);
+					},
 					onPinLastPromptChange: (enabled) => {
 						this.settingsManager.setPinLastPrompt(enabled);
 						this.pinLastPromptEnabled = enabled;
@@ -3675,6 +3746,13 @@ export class InteractiveMode {
 						this.settingsManager.setAutoMemory(enabled);
 						this.showStatus(`Auto memory: ${enabled ? "enabled" : "disabled"}`);
 					},
+					onNotificationSoundChange: (enabled) => {
+						this.notificationSoundEnabled = enabled;
+						this.settingsManager.setNotificationSound(enabled);
+						this.footer.setNotificationSoundEnabled(enabled);
+						this.ui.requestRender();
+						this.showStatus(`Notification sound: ${enabled ? "enabled" : "disabled"}`);
+					},
 					onTelegramLiveRelayAutoConnectChange: (enabled) => {
 						this.settingsManager.setTelegramLiveRelayAutoConnect(enabled);
 						this.showStatus(`Telegram autoconnect: ${enabled ? "enabled" : "disabled"}`);
@@ -3789,6 +3867,14 @@ export class InteractiveMode {
 						this.settingsManager.setEditorScheme(scheme);
 						this.showStatus(`Editor link scheme: ${scheme}`);
 					},
+					onCollapseToolCallsChange: (enabled) => {
+						this.settingsManager.setCollapseToolCalls(enabled);
+						this.setToolOutputExpansionState(enabled && (this.collapsedToolCallsExpanded || this.toolOutputExpanded), this.toolOutputExpanded);
+						if (!enabled) {
+							this.collapsedToolSummaryLine = undefined;
+						}
+						this.showStatus(`Collapse tool calls: ${enabled ? "enabled" : "disabled"}`);
+					},
 					onToolOutputModeChange: (mode) => {
 						this.settingsManager.setToolOutputMode(mode);
 						for (const child of this.chatContainer.children) {

package/packages/pi-coding-agent/src/modes/interactive/slash-command-handlers.ts

@@ -17,6 +17,9 @@ import type {
 	EditorComponent,
 	MarkdownTheme,
 } from "@gsd/pi-tui";
+import {
+	streamSimple,
+} from "@gsd/pi-ai";
 import {
 	type Component,
 	Container,
@@ -32,17 +35,19 @@ import type { AgentSession } from "../../core/agent-session.js";
 import type { AppAction, KeybindingsManager } from "../../core/keybindings.js";
 import type { SessionManager } from "../../core/session-manager.js";
 import type { SettingsManager } from "../../core/settings-manager.js";
+import { convertToLlm } from "../../core/messages.js";
 import { copyToClipboard } from "../../utils/clipboard.js";
 import { getChangelogPath, parseChangelog } from "../../utils/changelog.js";
 import { ArminComponent } from "./components/armin.js";
 import { BorderedLoader } from "./components/bordered-loader.js";
+import { BtwOverlayComponent } from "./components/btw-overlay.js";
 import { DynamicBorder } from "./components/dynamic-border.js";
 import { BUILTIN_SLASH_COMMANDS } from "../../core/slash-commands.js";
 import { appKey, editorKey, formatKeyForDisplay } from "./components/keybinding-hints.js";
 import { SelectSubmenu, THINKING_DESCRIPTIONS } from "./components/settings-selector.js";
 import { theme } from "./theme/theme.js";
 
-import type { TUI } from "@gsd/pi-tui";
+import type { OverlayHandle, TUI } from "@gsd/pi-tui";
 
 // ---------------------------------------------------------------------------
 // Context interface — the subset of InteractiveMode needed by slash commands
@@ -173,6 +178,15 @@ export async function dispatchSlashCommand(
 		showHotkeys(ctx);
 		return true;
 	}
+	if (text === "/btw" || text.startsWith("/btw ")) {
+		const question = text.startsWith("/btw ") ? text.slice(5).trim() : "";
+		if (!question) {
+			ctx.showWarning("Usage: /btw <question>");
+			return true;
+		}
+		await handleBtwCommand(question, ctx);
+		return true;
+	}
 	if (text === "/fork") {
 		ctx.showUserMessageSelector();
 		return true;
@@ -704,6 +718,7 @@ export function showHotkeys(ctx: SlashCommandContext): void {
 	const externalEditor = getAppKeyDisplay(ctx.keybindings, "externalEditor");
 	const followUp = getAppKeyDisplay(ctx.keybindings, "followUp");
 	const dequeue = getAppKeyDisplay(ctx.keybindings, "dequeue");
+	const toggleNotificationSound = getAppKeyDisplay(ctx.keybindings, "toggleNotificationSound");
 
 	let hotkeys = `
 **Navigation**
@@ -746,6 +761,7 @@ export function showHotkeys(ctx: SlashCommandContext): void {
 | \`${externalEditor}\` | Edit message in external editor |
 | \`${followUp}\` | Queue follow-up message |
 | \`${dequeue}\` | Restore queued messages |
+| \`${toggleNotificationSound}\` | Toggle notification sound |
 | \`Ctrl+V\` | Paste image from clipboard |
 | \`/\` | Slash commands |
 | \`!\` | Run bash command |
@@ -779,6 +795,49 @@ export function showHotkeys(ctx: SlashCommandContext): void {
 	ctx.requestRender();
 }
 
+async function handleBtwCommand(question: string, ctx: SlashCommandContext): Promise<void> {
+	const model = ctx.session.model;
+	if (!model) {
+		ctx.showWarning("No model selected. Cannot use /btw.");
+		return;
+	}
+
+	if (!ctx.session.modelRegistry.isProviderRequestReady(model.provider)) {
+		ctx.showWarning(`No API key available for provider: ${model.provider}`);
+		return;
+	}
+
+	const systemPrompt = ctx.session.systemPrompt;
+	const messages = convertToLlm(ctx.session.messages);
+	const sessionId = ctx.session.sessionId;
+	const apiKey = await ctx.session.modelRegistry.getApiKey(model, sessionId);
+	let handle: OverlayHandle | undefined;
+
+	const overlay = new BtwOverlayComponent(
+		question,
+		model,
+		systemPrompt,
+		messages,
+		ctx.getMarkdownThemeWithSettings(),
+		ctx.ui,
+		() => {
+			handle?.hide();
+		},
+		() => {
+			ctx.requestRender();
+		},
+		streamSimple,
+		{ apiKey, sessionId },
+	);
+
+	handle = ctx.ui.showOverlay(overlay, {
+		width: "80%",
+		maxHeight: "70%",
+		anchor: "center",
+		margin: { top: 2, bottom: 2, left: 4, right: 4 },
+	});
+}
+
 async function handleCompactCommand(customInstructions: string | undefined, ctx: SlashCommandContext): Promise<void> {
 	const entries = ctx.sessionManager.getEntries();
 	const messageCount = entries.filter((e) => e.type === "message").length;

package/packages/pi-coding-agent/src/modes/interactive/theme/themes.ts

@@ -78,8 +78,8 @@ const dark: ThemeJson = {
 		mdHr: "gray",
 		mdListBullet: "accent",
 
-		toolDiffAdded: "green",
-		toolDiffRemoved: "red",
+		toolDiffAdded: "#4ade80",
+		toolDiffRemoved: "#fb7185",
 		toolDiffContext: "gray",
 
 		syntaxComment: "#6A9955",
@@ -174,8 +174,8 @@ const light: ThemeJson = {
 		mdHr: "mediumGray",
 		mdListBullet: "green",
 
-		toolDiffAdded: "green",
-		toolDiffRemoved: "red",
+		toolDiffAdded: "#15803d",
+		toolDiffRemoved: "#b91c1c",
 		toolDiffContext: "mediumGray",
 
 		syntaxComment: "#008000",

package/packages/pi-tui/dist/components/editor.js

@@ -343,15 +343,15 @@ export class Editor {
             result.push(this.borderColor(indicator + "─".repeat(Math.max(0, remaining))));
         }
         else if (this.bottomHint) {
-            // Embed hint right-aligned in the bottom border: ─────── hint ─
+            // Embed hint at far right in bottom border: ─────────────── hint ─
             // Apply borderColor only to the dashes so the hint's own styling is preserved.
             const hintVisible = visibleWidth(this.bottomHint);
             const minDashes = 1;
             const separatorWidth = 1; // single space on each side of hint
             const totalFixed = hintVisible + separatorWidth * 2 + minDashes * 2;
             if (width >= totalFixed) {
-                const leftDashes = Math.floor((width - hintVisible - separatorWidth * 2) * 0.75);
-                const rightDashes = Math.max(minDashes, width - hintVisible - separatorWidth * 2 - leftDashes);
+                const rightDashes = minDashes;
+                const leftDashes = Math.max(minDashes, width - hintVisible - separatorWidth * 2 - rightDashes);
                 const line = this.borderColor("─".repeat(leftDashes)) +
                     " " +
                     this.bottomHint +