@oh-my-pi/pi-coding-agent 1.340.0 → 2.0.1337

package/src/modes/interactive/components/session-selector.ts

@@ -11,10 +11,10 @@ import {
 	Text,
 	truncateToWidth,
 } from "@oh-my-pi/pi-tui";
-import type { SessionInfo } from "../../../core/session-manager.js";
-import { fuzzyFilter } from "../../../utils/fuzzy.js";
-import { theme } from "../theme/theme.js";
-import { DynamicBorder } from "./dynamic-border.js";
+import type { SessionInfo } from "../../../core/session-manager";
+import { fuzzyFilter } from "../../../utils/fuzzy";
+import { theme } from "../theme/theme";
+import { DynamicBorder } from "./dynamic-border";
 
 /**
  * Custom session list component with multi-line items and search
@@ -90,7 +90,7 @@ class SessionList implements Component {
 		);
 		const endIndex = Math.min(startIndex + this.maxVisible, this.filteredSessions.length);
 
-		// Render visible sessions (2 lines per session + blank line)
+		// Render visible sessions (2-3 lines per session + blank line)
 		for (let i = startIndex; i < endIndex; i++) {
 			const session = this.filteredSessions[i];
 			const isSelected = i === this.selectedIndex;
@@ -98,19 +98,32 @@ class SessionList implements Component {
 			// Normalize first message to single line
 			const normalizedMessage = session.firstMessage.replace(/\n/g, " ").trim();
 
-			// First line: cursor + message (truncate to visible width)
+			// First line: cursor + title (or first message if no title)
 			const cursor = isSelected ? theme.fg("accent", "› ") : "  ";
-			const maxMsgWidth = width - 2; // Account for cursor (2 visible chars)
-			const truncatedMsg = truncateToWidth(normalizedMessage, maxMsgWidth, "...");
-			const messageLine = cursor + (isSelected ? theme.bold(truncatedMsg) : truncatedMsg);
+			const maxWidth = width - 2; // Account for cursor (2 visible chars)
+
+			if (session.title) {
+				// Has title: show title on first line, dimmed first message on second line
+				const truncatedTitle = truncateToWidth(session.title, maxWidth, "...");
+				const titleLine = cursor + (isSelected ? theme.bold(truncatedTitle) : truncatedTitle);
+				lines.push(titleLine);
+
+				// Second line: dimmed first message preview
+				const truncatedPreview = truncateToWidth(normalizedMessage, maxWidth, "...");
+				lines.push(`  ${theme.fg("dim", truncatedPreview)}`);
+			} else {
+				// No title: show first message as main line
+				const truncatedMsg = truncateToWidth(normalizedMessage, maxWidth, "...");
+				const messageLine = cursor + (isSelected ? theme.bold(truncatedMsg) : truncatedMsg);
+				lines.push(messageLine);
+			}
 
-			// Second line: metadata (dimmed) - also truncate for safety
+			// Metadata line: date + message count
 			const modified = formatDate(session.modified);
 			const msgCount = `${session.messageCount} message${session.messageCount !== 1 ? "s" : ""}`;
 			const metadata = `  ${modified} · ${msgCount}`;
 			const metadataLine = theme.fg("dim", truncateToWidth(metadata, width, ""));
 
-			lines.push(messageLine);
 			lines.push(metadataLine);
 			lines.push(""); // Blank line between sessions
 		}

package/src/modes/interactive/components/settings-defs.ts

@@ -10,7 +10,7 @@
 
 import type { ThinkingLevel } from "@oh-my-pi/pi-agent-core";
 import { getCapabilities } from "@oh-my-pi/pi-tui";
-import type { SettingsManager } from "../../../core/settings-manager.js";
+import type { SettingsManager } from "../../../core/settings-manager";
 
 // Setting value types
 export type SettingValue = boolean | string;
@@ -20,7 +20,7 @@ interface BaseSettingDef {
 	id: string;
 	label: string;
 	description: string;
-	tab: "config" | "exa";
+	tab: string;
 }
 
 // Boolean toggle setting
@@ -99,6 +99,16 @@ export const SETTINGS_DEFS: SettingDef[] = [
 		get: (sm) => sm.getQueueMode(),
 		set: (sm, v) => sm.setQueueMode(v as "all" | "one-at-a-time"), // Also handled in session
 	},
+	{
+		id: "interruptMode",
+		tab: "config",
+		type: "enum",
+		label: "Interrupt mode",
+		description: "When to process queued messages: immediately (interrupt tools) or wait for turn to complete",
+		values: ["immediate", "wait"],
+		get: (sm) => sm.getInterruptMode(),
+		set: (sm, v) => sm.setInterruptMode(v as "immediate" | "wait"), // Also handled in session
+	},
 	{
 		id: "hideThinking",
 		tab: "config",
@@ -135,6 +145,15 @@ export const SETTINGS_DEFS: SettingDef[] = [
 		get: (sm) => sm.getMCPProjectConfigEnabled(),
 		set: (sm, v) => sm.setMCPProjectConfigEnabled(v),
 	},
+	{
+		id: "editFuzzyMatch",
+		tab: "config",
+		type: "boolean",
+		label: "Edit fuzzy match",
+		description: "Accept high-confidence fuzzy matches for whitespace/indentation differences",
+		get: (sm) => sm.getEditFuzzyMatch(),
+		set: (sm, v) => sm.setEditFuzzyMatch(v),
+	},
 	{
 		id: "thinkingLevel",
 		tab: "config",
@@ -161,6 +180,35 @@ export const SETTINGS_DEFS: SettingDef[] = [
 		getOptions: () => [], // Filled dynamically from context
 	},
 
+	// LSP tab
+	{
+		id: "lspFormatOnWrite",
+		tab: "lsp",
+		type: "boolean",
+		label: "Format on write",
+		description: "Automatically format code files using LSP after writing",
+		get: (sm) => sm.getLspFormatOnWrite(),
+		set: (sm, v) => sm.setLspFormatOnWrite(v),
+	},
+	{
+		id: "lspDiagnosticsOnWrite",
+		tab: "lsp",
+		type: "boolean",
+		label: "Diagnostics on write",
+		description: "Return LSP diagnostics (errors/warnings) after writing code files",
+		get: (sm) => sm.getLspDiagnosticsOnWrite(),
+		set: (sm, v) => sm.setLspDiagnosticsOnWrite(v),
+	},
+	{
+		id: "lspDiagnosticsOnEdit",
+		tab: "lsp",
+		type: "boolean",
+		label: "Diagnostics on edit",
+		description: "Return LSP diagnostics (errors/warnings) after editing code files",
+		get: (sm) => sm.getLspDiagnosticsOnEdit(),
+		set: (sm, v) => sm.setLspDiagnosticsOnEdit(v),
+	},
+
 	// Exa tab
 	{
 		id: "exaEnabled",
@@ -219,7 +267,7 @@ export const SETTINGS_DEFS: SettingDef[] = [
 ];
 
 /** Get settings for a specific tab */
-export function getSettingsForTab(tab: "config" | "exa"): SettingDef[] {
+export function getSettingsForTab(tab: string): SettingDef[] {
 	return SETTINGS_DEFS.filter((def) => def.tab === tab);
 }
 

package/src/modes/interactive/components/settings-selector.ts

@@ -16,11 +16,11 @@ import {
 	type TabBarTheme,
 	Text,
 } from "@oh-my-pi/pi-tui";
-import type { SettingsManager } from "../../../core/settings-manager.js";
-import { getSelectListTheme, getSettingsListTheme, theme } from "../theme/theme.js";
-import { DynamicBorder } from "./dynamic-border.js";
-import { PluginSettingsComponent } from "./plugin-settings.js";
-import { getSettingsForTab, type SettingDef } from "./settings-defs.js";
+import type { SettingsManager } from "../../../core/settings-manager";
+import { getSelectListTheme, getSettingsListTheme, theme } from "../theme/theme";
+import { DynamicBorder } from "./dynamic-border";
+import { PluginSettingsComponent } from "./plugin-settings";
+import { getSettingsForTab, type SettingDef } from "./settings-defs";
 
 function getTabBarTheme(): TabBarTheme {
 	return {
@@ -93,10 +93,11 @@ class SelectSubmenu extends Container {
 	}
 }
 
-type TabId = "config" | "exa" | "plugins";
+type TabId = string;
 
 const SETTINGS_TABS: Tab[] = [
 	{ id: "config", label: "Config" },
+	{ id: "lsp", label: "LSP" },
 	{ id: "exa", label: "Exa" },
 	{ id: "plugins", label: "Plugins" },
 ];
@@ -189,14 +190,10 @@ export class SettingsSelectorComponent extends Container {
 		const bottomBorder = this.children[this.children.length - 1];
 		this.removeChild(bottomBorder);
 
-		switch (tabId) {
-			case "config":
-			case "exa":
-				this.showSettingsTab(tabId);
-				break;
-			case "plugins":
-				this.showPluginsTab();
-				break;
+		if (tabId === "plugins") {
+			this.showPluginsTab();
+		} else {
+			this.showSettingsTab(tabId);
 		}
 
 		// Re-add bottom border
@@ -301,9 +298,9 @@ export class SettingsSelectorComponent extends Container {
 	}
 
 	/**
-	 * Show a settings tab (config or exa) using definitions.
+	 * Show a settings tab using definitions.
 	 */
-	private showSettingsTab(tabId: "config" | "exa"): void {
+	private showSettingsTab(tabId: string): void {
 		const defs = getSettingsForTab(tabId);
 		const items: SettingItem[] = [];
 

package/src/modes/interactive/components/show-images-selector.ts

@@ -1,6 +1,6 @@
 import { Container, type SelectItem, SelectList } from "@oh-my-pi/pi-tui";
-import { getSelectListTheme } from "../theme/theme.js";
-import { DynamicBorder } from "./dynamic-border.js";
+import { getSelectListTheme } from "../theme/theme";
+import { DynamicBorder } from "./dynamic-border";
 
 /**
  * Component that renders a show images selector with borders

package/src/modes/interactive/components/theme-selector.ts

@@ -1,6 +1,6 @@
 import { Container, type SelectItem, SelectList } from "@oh-my-pi/pi-tui";
-import { getAvailableThemes, getSelectListTheme } from "../theme/theme.js";
-import { DynamicBorder } from "./dynamic-border.js";
+import { getAvailableThemes, getSelectListTheme } from "../theme/theme";
+import { DynamicBorder } from "./dynamic-border";
 
 /**
  * Component that renders a theme selector

package/src/modes/interactive/components/thinking-selector.ts

@@ -1,7 +1,7 @@
 import type { ThinkingLevel } from "@oh-my-pi/pi-agent-core";
 import { Container, type SelectItem, SelectList } from "@oh-my-pi/pi-tui";
-import { getSelectListTheme } from "../theme/theme.js";
-import { DynamicBorder } from "./dynamic-border.js";
+import { getSelectListTheme } from "../theme/theme";
+import { DynamicBorder } from "./dynamic-border";
 
 const LEVEL_DESCRIPTIONS: Record<ThinkingLevel, string> = {
 	off: "No reasoning",

package/src/modes/interactive/components/tool-execution.ts

@@ -11,14 +11,14 @@ import {
 	type TUI,
 } from "@oh-my-pi/pi-tui";
 import stripAnsi from "strip-ansi";
-import type { CustomTool } from "../../../core/custom-tools/types.js";
-import { computeEditDiff, type EditDiffError, type EditDiffResult } from "../../../core/tools/edit-diff.js";
-import { toolRenderers } from "../../../core/tools/renderers.js";
-import { DEFAULT_MAX_BYTES, DEFAULT_MAX_LINES, formatSize } from "../../../core/tools/truncate.js";
-import { sanitizeBinaryOutput } from "../../../utils/shell.js";
-import { getLanguageFromPath, highlightCode, theme } from "../theme/theme.js";
-import { renderDiff } from "./diff.js";
-import { truncateToVisualLines } from "./visual-truncate.js";
+import type { CustomTool } from "../../../core/custom-tools/types";
+import { computeEditDiff, type EditDiffError, type EditDiffResult } from "../../../core/tools/edit-diff";
+import { toolRenderers } from "../../../core/tools/renderers";
+import { DEFAULT_MAX_BYTES, DEFAULT_MAX_LINES, formatSize } from "../../../core/tools/truncate";
+import { sanitizeBinaryOutput } from "../../../utils/shell";
+import { getLanguageFromPath, highlightCode, theme } from "../theme/theme";
+import { renderDiff } from "./diff";
+import { truncateToVisualLines } from "./visual-truncate";
 
 // Preview line limit for bash when not expanded
 const BASH_PREVIEW_LINES = 5;
@@ -501,6 +501,24 @@ export class ToolExecutionComponent extends Container {
 					text += theme.fg("toolOutput", `\n... (${remaining} more lines, ${totalLines} total)`);
 				}
 			}
+
+			// Show LSP diagnostics if available
+			if (this.result?.details?.diagnostics?.available) {
+				const diag = this.result.details.diagnostics;
+				if (diag.diagnostics.length > 0) {
+					const icon = diag.hasErrors ? theme.fg("error", "●") : theme.fg("warning", "●");
+					text += `\n\n${icon} ${theme.fg("toolTitle", "LSP Diagnostics")} ${theme.fg("dim", `(${diag.summary})`)}`;
+					const maxDiags = this.expanded ? diag.diagnostics.length : 5;
+					const displayDiags = diag.diagnostics.slice(0, maxDiags);
+					for (const d of displayDiags) {
+						const color = d.includes("[error]") ? "error" : d.includes("[warning]") ? "warning" : "dim";
+						text += `\n  ${theme.fg(color, d)}`;
+					}
+					if (diag.diagnostics.length > maxDiags) {
+						text += theme.fg("dim", `\n  ... (${diag.diagnostics.length - maxDiags} more)`);
+					}
+				}
+			}
 		} else if (this.toolName === "edit") {
 			const rawPath = this.args?.file_path || this.args?.path || "";
 			const path = shortenPath(rawPath);
@@ -532,6 +550,24 @@ export class ToolExecutionComponent extends Container {
 					text += `\n\n${renderDiff(this.editDiffPreview.diff, { filePath: rawPath })}`;
 				}
 			}
+
+			// Show LSP diagnostics if available
+			if (this.result?.details?.diagnostics?.available) {
+				const diag = this.result.details.diagnostics;
+				if (diag.diagnostics.length > 0) {
+					const icon = diag.hasErrors ? theme.fg("error", "●") : theme.fg("warning", "●");
+					text += `\n\n${icon} ${theme.fg("toolTitle", "LSP Diagnostics")} ${theme.fg("dim", `(${diag.summary})`)}`;
+					const maxDiags = this.expanded ? diag.diagnostics.length : 5;
+					const displayDiags = diag.diagnostics.slice(0, maxDiags);
+					for (const d of displayDiags) {
+						const color = d.includes("[error]") ? "error" : d.includes("[warning]") ? "warning" : "dim";
+						text += `\n  ${theme.fg(color, d)}`;
+					}
+					if (diag.diagnostics.length > maxDiags) {
+						text += theme.fg("dim", `\n  ... (${diag.diagnostics.length - maxDiags} more)`);
+					}
+				}
+			}
 		} else if (this.toolName === "ls") {
 			const path = shortenPath(this.args?.path || ".");
 			const limit = this.args?.limit;

package/src/modes/interactive/components/tree-selector.ts

@@ -17,9 +17,9 @@ import {
 	TruncatedText,
 	truncateToWidth,
 } from "@oh-my-pi/pi-tui";
-import type { SessionTreeNode } from "../../../core/session-manager.js";
-import { theme } from "../theme/theme.js";
-import { DynamicBorder } from "./dynamic-border.js";
+import type { SessionTreeNode } from "../../../core/session-manager";
+import { theme } from "../theme/theme";
+import { DynamicBorder } from "./dynamic-border";
 
 /** Gutter info: position (displayIndent where connector was) and whether to show │ */
 interface GutterInfo {
@@ -356,7 +356,7 @@ class TreeList implements Component {
 				parts.push("branch summary", entry.summary);
 				break;
 			case "model_change":
-				parts.push("model", entry.modelId);
+				parts.push("model", entry.model);
 				break;
 			case "thinking_level_change":
 				parts.push("thinking", entry.thinkingLevel);
@@ -558,7 +558,7 @@ class TreeList implements Component {
 				result = theme.fg("warning", `[branch summary]: `) + normalize(entry.summary);
 				break;
 			case "model_change":
-				result = theme.fg("dim", `[model: ${entry.modelId}]`);
+				result = theme.fg("dim", `[model: ${entry.model}]`);
 				break;
 			case "thinking_level_change":
 				result = theme.fg("dim", `[thinking: ${entry.thinkingLevel}]`);

package/src/modes/interactive/components/user-message-selector.ts

@@ -10,8 +10,8 @@ import {
 	Text,
 	truncateToWidth,
 } from "@oh-my-pi/pi-tui";
-import { theme } from "../theme/theme.js";
-import { DynamicBorder } from "./dynamic-border.js";
+import { theme } from "../theme/theme";
+import { DynamicBorder } from "./dynamic-border";
 
 interface UserMessageItem {
 	id: string; // Entry ID in the session

package/src/modes/interactive/components/user-message.ts

@@ -1,5 +1,5 @@
 import { Container, Markdown, Spacer } from "@oh-my-pi/pi-tui";
-import { getMarkdownTheme, theme } from "../theme/theme.js";
+import { getMarkdownTheme, theme } from "../theme/theme";
 
 /**
  * Component that renders a user message

package/src/modes/interactive/components/welcome.ts

@@ -1,12 +1,18 @@
 import { type Component, visibleWidth } from "@oh-my-pi/pi-tui";
-import { APP_NAME } from "../../../config.js";
-import { theme } from "../theme/theme.js";
+import { APP_NAME } from "../../../config";
+import { theme } from "../theme/theme";
 
 export interface RecentSession {
 	name: string;
 	timeAgo: string;
 }
 
+export interface LspServerInfo {
+	name: string;
+	status: "ready" | "error" | "connecting";
+	fileTypes: string[];
+}
+
 /**
  * Premium welcome screen with block-based Pi logo and two-column layout.
  */
@@ -15,12 +21,20 @@ export class WelcomeComponent implements Component {
 	private modelName: string;
 	private providerName: string;
 	private recentSessions: RecentSession[];
-
-	constructor(version: string, modelName: string, providerName: string, recentSessions: RecentSession[] = []) {
+	private lspServers: LspServerInfo[];
+
+	constructor(
+		version: string,
+		modelName: string,
+		providerName: string,
+		recentSessions: RecentSession[] = [],
+		lspServers: LspServerInfo[] = [],
+	) {
 		this.version = version;
 		this.modelName = modelName;
 		this.providerName = providerName;
 		this.recentSessions = recentSessions;
+		this.lspServers = lspServers;
 	}
 
 	invalidate(): void {}
@@ -34,6 +48,10 @@ export class WelcomeComponent implements Component {
 		this.recentSessions = sessions;
 	}
 
+	setLspServers(servers: LspServerInfo[]): void {
+		this.lspServers = servers;
+	}
+
 	render(termWidth: number): string[] {
 		// Box dimensions - responsive with min/max
 		const minWidth = 80;
@@ -76,13 +94,32 @@ export class WelcomeComponent implements Component {
 			}
 		}
 
+		// LSP servers content
+		const lspLines: string[] = [];
+		if (this.lspServers.length === 0) {
+			lspLines.push(` ${theme.fg("dim", "No LSP servers")}`);
+		} else {
+			for (const server of this.lspServers) {
+				const icon =
+					server.status === "ready"
+						? theme.fg("success", "●")
+						: server.status === "connecting"
+							? theme.fg("warning", "○")
+							: theme.fg("error", "●");
+				const exts = server.fileTypes.slice(0, 3).join(" ");
+				lspLines.push(` ${icon} ${theme.fg("muted", server.name)} ${theme.fg("dim", exts)}`);
+			}
+		}
+
 		// Right column
 		const rightLines = [
 			` ${theme.bold(theme.fg("accent", "Tips"))}`,
 			` ${theme.fg("dim", "?")}${theme.fg("muted", " for keyboard shortcuts")}`,
 			` ${theme.fg("dim", "/")}${theme.fg("muted", " for commands")}`,
 			` ${theme.fg("dim", "!")}${theme.fg("muted", " to run bash")}`,
-			` ${theme.fg("dim", "/status")}${theme.fg("muted", " for loaded extensions")}`,
+			separator,
+			` ${theme.bold(theme.fg("accent", "LSP Servers"))}`,
+			...lspLines,
 			separator,
 			` ${theme.bold(theme.fg("accent", "Recent sessions"))}`,
 			...sessionLines,