@oh-my-pi/pi-coding-agent 11.8.2 → 11.9.0

package/src/modes/components/queue-mode-selector.ts

@@ -6,7 +6,7 @@ import { DynamicBorder } from "./dynamic-border";
  * Component that renders a queue mode selector with borders
  */
 export class QueueModeSelectorComponent extends Container {
-	private selectList: SelectList;
+	#selectList: SelectList;
 
 	constructor(
 		currentMode: "all" | "one-at-a-time",
@@ -28,29 +28,29 @@ export class QueueModeSelectorComponent extends Container {
 		this.addChild(new DynamicBorder());
 
 		// Create selector
-		this.selectList = new SelectList(queueModes, 2, getSelectListTheme());
+		this.#selectList = new SelectList(queueModes, 2, getSelectListTheme());
 
 		// Preselect current mode
 		const currentIndex = queueModes.findIndex(item => item.value === currentMode);
 		if (currentIndex !== -1) {
-			this.selectList.setSelectedIndex(currentIndex);
+			this.#selectList.setSelectedIndex(currentIndex);
 		}
 
-		this.selectList.onSelect = item => {
+		this.#selectList.onSelect = item => {
 			onSelect(item.value as "all" | "one-at-a-time");
 		};
 
-		this.selectList.onCancel = () => {
+		this.#selectList.onCancel = () => {
 			onCancel();
 		};
 
-		this.addChild(this.selectList);
+		this.addChild(this.#selectList);
 
 		// Add bottom border
 		this.addChild(new DynamicBorder());
 	}
 
 	getSelectList(): SelectList {
-		return this.selectList;
+		return this.#selectList;
 	}
 }

package/src/modes/components/read-tool-group.ts

@@ -20,20 +20,20 @@ type ReadEntry = {
 };
 
 export class ReadToolGroupComponent extends Container implements ToolExecutionHandle {
-	private entries = new Map<string, ReadEntry>();
-	private text: Text;
+	#entries = new Map<string, ReadEntry>();
+	#text: Text;
 
 	constructor() {
 		super();
-		this.text = new Text("", 0, 0);
-		this.addChild(this.text);
-		this.updateDisplay();
+		this.#text = new Text("", 0, 0);
+		this.addChild(this.#text);
+		this.#updateDisplay();
 	}
 
 	updateArgs(args: ReadRenderArgs, toolCallId?: string): void {
 		if (!toolCallId) return;
 		const rawPath = args.file_path || args.path || "";
-		const entry: ReadEntry = this.entries.get(toolCallId) ?? {
+		const entry: ReadEntry = this.#entries.get(toolCallId) ?? {
 			toolCallId,
 			path: rawPath,
 			offset: args.offset,
@@ -43,8 +43,8 @@ export class ReadToolGroupComponent extends Container implements ToolExecutionHa
 		entry.path = rawPath;
 		entry.offset = args.offset;
 		entry.limit = args.limit;
-		this.entries.set(toolCallId, entry);
-		this.updateDisplay();
+		this.#entries.set(toolCallId, entry);
+		this.#updateDisplay();
 	}
 
 	updateResult(
@@ -53,38 +53,38 @@ export class ReadToolGroupComponent extends Container implements ToolExecutionHa
 		toolCallId?: string,
 	): void {
 		if (!toolCallId) return;
-		const entry = this.entries.get(toolCallId);
+		const entry = this.#entries.get(toolCallId);
 		if (!entry) return;
 		if (isPartial) return;
 		entry.status = result.isError ? "error" : "success";
-		this.updateDisplay();
+		this.#updateDisplay();
 	}
 
 	setArgsComplete(_toolCallId?: string): void {
-		this.updateDisplay();
+		this.#updateDisplay();
 	}
 
 	setExpanded(_expanded: boolean): void {
-		this.updateDisplay();
+		this.#updateDisplay();
 	}
 
 	getComponent(): Component {
 		return this;
 	}
 
-	private updateDisplay(): void {
-		const entries = [...this.entries.values()];
+	#updateDisplay(): void {
+		const entries = [...this.#entries.values()];
 
 		if (entries.length === 0) {
-			this.text.setText(` ${theme.format.bullet} ${theme.fg("toolTitle", theme.bold("Read"))}`);
+			this.#text.setText(` ${theme.format.bullet} ${theme.fg("toolTitle", theme.bold("Read"))}`);
 			return;
 		}
 
 		if (entries.length === 1) {
 			const entry = entries[0];
-			const statusSymbol = this.formatStatus(entry.status);
-			const pathDisplay = this.formatPath(entry);
-			this.text.setText(
+			const statusSymbol = this.#formatStatus(entry.status);
+			const pathDisplay = this.#formatPath(entry);
+			this.#text.setText(
 				` ${theme.format.bullet} ${theme.fg("toolTitle", theme.bold("Read"))} ${pathDisplay} ${statusSymbol}`.trimEnd(),
 			);
 			return;
@@ -95,15 +95,15 @@ export class ReadToolGroupComponent extends Container implements ToolExecutionHa
 		const total = entries.length;
 		for (const [index, entry] of entries.entries()) {
 			const connector = index === total - 1 ? theme.tree.last : theme.tree.branch;
-			const statusSymbol = this.formatStatus(entry.status);
-			const pathDisplay = this.formatPath(entry);
+			const statusSymbol = this.#formatStatus(entry.status);
+			const pathDisplay = this.#formatPath(entry);
 			lines.push(`   ${theme.fg("dim", connector)} ${statusSymbol} ${pathDisplay}`.trimEnd());
 		}
 
-		this.text.setText(lines.join("\n"));
+		this.#text.setText(lines.join("\n"));
 	}
 
-	private formatPath(entry: ReadEntry): string {
+	#formatPath(entry: ReadEntry): string {
 		const filePath = shortenPath(entry.path);
 		let pathDisplay = filePath ? theme.fg("accent", filePath) : theme.fg("toolOutput", "…");
 		if (entry.offset !== undefined || entry.limit !== undefined) {
@@ -114,7 +114,7 @@ export class ReadToolGroupComponent extends Container implements ToolExecutionHa
 		return pathDisplay;
 	}
 
-	private formatStatus(status: ReadEntry["status"]): string {
+	#formatStatus(status: ReadEntry["status"]): string {
 		if (status === "success") {
 			return theme.fg("success", theme.status.success);
 		}

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

@@ -18,25 +18,25 @@ import { DynamicBorder } from "./dynamic-border";
  * Custom session list component with multi-line items and search
  */
 class SessionList implements Component {
-	private filteredSessions: SessionInfo[] = [];
-	private selectedIndex: number = 0;
-	private readonly searchInput: Input;
-	public onSelect?: (sessionPath: string) => void;
-	public onCancel?: () => void;
-	public onExit: () => void = () => {};
-	private maxVisible: number = 5; // Max sessions visible (each session is 3 lines: msg + metadata + blank)
+	#filteredSessions: SessionInfo[] = [];
+	#selectedIndex: number = 0;
+	readonly #searchInput: Input;
+	onSelect?: (sessionPath: string) => void;
+	onCancel?: () => void;
+	onExit: () => void = () => {};
+	#maxVisible: number = 5; // Max sessions visible (each session is 3 lines: msg + metadata + blank)
 
 	constructor(
 		private readonly allSessions: SessionInfo[],
 		private readonly showCwd = false,
 	) {
-		this.filteredSessions = allSessions;
-		this.searchInput = new Input();
+		this.#filteredSessions = allSessions;
+		this.#searchInput = new Input();
 
 		// Handle Enter in search input - select current item
-		this.searchInput.onSubmit = () => {
-			if (this.filteredSessions[this.selectedIndex]) {
-				const selected = this.filteredSessions[this.selectedIndex];
+		this.#searchInput.onSubmit = () => {
+			if (this.#filteredSessions[this.#selectedIndex]) {
+				const selected = this.#filteredSessions[this.#selectedIndex];
 				if (this.onSelect) {
 					this.onSelect(selected.path);
 				}
@@ -44,8 +44,8 @@ class SessionList implements Component {
 		};
 	}
 
-	private filterSessions(query: string): void {
-		this.filteredSessions = fuzzyFilter(this.allSessions, query, session => {
+	#filterSessions(query: string): void {
+		this.#filteredSessions = fuzzyFilter(this.allSessions, query, session => {
 			const parts = [
 				session.id,
 				session.title ?? "",
@@ -56,7 +56,7 @@ class SessionList implements Component {
 			];
 			return parts.filter(Boolean).join(" ");
 		});
-		this.selectedIndex = Math.min(this.selectedIndex, Math.max(0, this.filteredSessions.length - 1));
+		this.#selectedIndex = Math.min(this.#selectedIndex, Math.max(0, this.#filteredSessions.length - 1));
 	}
 
 	invalidate(): void {
@@ -67,10 +67,10 @@ class SessionList implements Component {
 		const lines: string[] = [];
 
 		// Render search input
-		lines.push(...this.searchInput.render(width));
+		lines.push(...this.#searchInput.render(width));
 		lines.push(""); // Blank line after search
 
-		if (this.filteredSessions.length === 0) {
+		if (this.#filteredSessions.length === 0) {
 			if (this.showCwd) {
 				// "All" scope - no sessions anywhere that match filter
 				lines.push(truncateToWidth(theme.fg("muted", "  No sessions found"), width));
@@ -103,14 +103,17 @@ class SessionList implements Component {
 		// Calculate visible range with scrolling
 		const startIndex = Math.max(
 			0,
-			Math.min(this.selectedIndex - Math.floor(this.maxVisible / 2), this.filteredSessions.length - this.maxVisible),
+			Math.min(
+				this.#selectedIndex - Math.floor(this.#maxVisible / 2),
+				this.#filteredSessions.length - this.#maxVisible,
+			),
 		);
-		const endIndex = Math.min(startIndex + this.maxVisible, this.filteredSessions.length);
+		const endIndex = Math.min(startIndex + this.#maxVisible, this.#filteredSessions.length);
 
 		// 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;
+			const session = this.#filteredSessions[i];
+			const isSelected = i === this.#selectedIndex;
 
 			// Normalize first message to single line
 			const normalizedMessage = session.firstMessage.replace(/\n/g, " ").trim();
@@ -148,8 +151,8 @@ class SessionList implements Component {
 		}
 
 		// Add scroll indicator if needed
-		if (startIndex > 0 || endIndex < this.filteredSessions.length) {
-			const scrollText = `  (${this.selectedIndex + 1}/${this.filteredSessions.length})`;
+		if (startIndex > 0 || endIndex < this.#filteredSessions.length) {
+			const scrollText = `  (${this.#selectedIndex + 1}/${this.#filteredSessions.length})`;
 			const scrollInfo = theme.fg("muted", truncateToWidth(scrollText, width));
 			lines.push(scrollInfo);
 		}
@@ -160,23 +163,23 @@ class SessionList implements Component {
 	handleInput(keyData: string): void {
 		// Up arrow
 		if (matchesKey(keyData, "up")) {
-			this.selectedIndex = Math.max(0, this.selectedIndex - 1);
+			this.#selectedIndex = Math.max(0, this.#selectedIndex - 1);
 		}
 		// Down arrow
 		else if (matchesKey(keyData, "down")) {
-			this.selectedIndex = Math.min(this.filteredSessions.length - 1, this.selectedIndex + 1);
+			this.#selectedIndex = Math.min(this.#filteredSessions.length - 1, this.#selectedIndex + 1);
 		}
 		// Page up - jump up by maxVisible items
 		else if (matchesKey(keyData, "pageUp")) {
-			this.selectedIndex = Math.max(0, this.selectedIndex - this.maxVisible);
+			this.#selectedIndex = Math.max(0, this.#selectedIndex - this.#maxVisible);
 		}
 		// Page down - jump down by maxVisible items
 		else if (matchesKey(keyData, "pageDown")) {
-			this.selectedIndex = Math.min(this.filteredSessions.length - 1, this.selectedIndex + this.maxVisible);
+			this.#selectedIndex = Math.min(this.#filteredSessions.length - 1, this.#selectedIndex + this.#maxVisible);
 		}
 		// Enter
 		else if (matchesKey(keyData, "enter") || matchesKey(keyData, "return") || keyData === "\n") {
-			const selected = this.filteredSessions[this.selectedIndex];
+			const selected = this.#filteredSessions[this.#selectedIndex];
 			if (selected && this.onSelect) {
 				this.onSelect(selected.path);
 			}
@@ -193,8 +196,8 @@ class SessionList implements Component {
 		}
 		// Pass everything else to search input
 		else {
-			this.searchInput.handleInput(keyData);
-			this.filterSessions(this.searchInput.getValue());
+			this.#searchInput.handleInput(keyData);
+			this.#filterSessions(this.#searchInput.getValue());
 		}
 	}
 }
@@ -203,7 +206,7 @@ class SessionList implements Component {
  * Component that renders a session selector
  */
 export class SessionSelectorComponent extends Container {
-	private sessionList: SessionList;
+	#sessionList: SessionList;
 
 	constructor(
 		sessions: SessionInfo[],
@@ -221,12 +224,12 @@ export class SessionSelectorComponent extends Container {
 		this.addChild(new Spacer(1));
 
 		// Create session list
-		this.sessionList = new SessionList(sessions);
-		this.sessionList.onSelect = onSelect;
-		this.sessionList.onCancel = onCancel;
-		this.sessionList.onExit = onExit;
+		this.#sessionList = new SessionList(sessions);
+		this.#sessionList.onSelect = onSelect;
+		this.#sessionList.onCancel = onCancel;
+		this.#sessionList.onExit = onExit;
 
-		this.addChild(this.sessionList);
+		this.addChild(this.#sessionList);
 
 		// Add bottom border
 		this.addChild(new Spacer(1));
@@ -234,6 +237,6 @@ export class SessionSelectorComponent extends Container {
 	}
 
 	getSessionList(): SessionList {
-		return this.sessionList;
+		return this.#sessionList;
 	}
 }