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

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

@@ -7,8 +7,8 @@ import { DynamicBorder } from "./dynamic-border";
  * Themes must be pre-loaded and passed to the constructor.
  */
 export class ThemeSelectorComponent extends Container {
-	private selectList: SelectList;
-	private onPreview: (themeName: string) => void;
+	#selectList: SelectList;
+	#onPreview: (themeName: string) => void;
 
 	constructor(
 		currentTheme: string,
@@ -18,7 +18,7 @@ export class ThemeSelectorComponent extends Container {
 		onPreview: (themeName: string) => void,
 	) {
 		super();
-		this.onPreview = onPreview;
+		this.#onPreview = onPreview;
 
 		// Create select items from provided themes
 		const themeItems: SelectItem[] = themes.map(name => ({
@@ -31,33 +31,33 @@ export class ThemeSelectorComponent extends Container {
 		this.addChild(new DynamicBorder());
 
 		// Create selector
-		this.selectList = new SelectList(themeItems, 10, getSelectListTheme());
+		this.#selectList = new SelectList(themeItems, 10, getSelectListTheme());
 
 		// Preselect current theme
 		const currentIndex = themes.indexOf(currentTheme);
 		if (currentIndex !== -1) {
-			this.selectList.setSelectedIndex(currentIndex);
+			this.#selectList.setSelectedIndex(currentIndex);
 		}
 
-		this.selectList.onSelect = item => {
+		this.#selectList.onSelect = item => {
 			onSelect(item.value);
 		};
 
-		this.selectList.onCancel = () => {
+		this.#selectList.onCancel = () => {
 			onCancel();
 		};
 
-		this.selectList.onSelectionChange = item => {
-			this.onPreview(item.value);
+		this.#selectList.onSelectionChange = item => {
+			this.#onPreview(item.value);
 		};
 
-		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/thinking-selector.ts

@@ -16,7 +16,7 @@ const LEVEL_DESCRIPTIONS: Record<ThinkingLevel, string> = {
  * Component that renders a thinking level selector with borders
  */
 export class ThinkingSelectorComponent extends Container {
-	private selectList: SelectList;
+	#selectList: SelectList;
 
 	constructor(
 		currentLevel: ThinkingLevel,
@@ -36,29 +36,29 @@ export class ThinkingSelectorComponent extends Container {
 		this.addChild(new DynamicBorder());
 
 		// Create selector
-		this.selectList = new SelectList(thinkingLevels, thinkingLevels.length, getSelectListTheme());
+		this.#selectList = new SelectList(thinkingLevels, thinkingLevels.length, getSelectListTheme());
 
 		// Preselect current level
 		const currentIndex = thinkingLevels.findIndex(item => item.value === currentLevel);
 		if (currentIndex !== -1) {
-			this.selectList.setSelectedIndex(currentIndex);
+			this.#selectList.setSelectedIndex(currentIndex);
 		}
 
-		this.selectList.onSelect = item => {
+		this.#selectList.onSelect = item => {
 			onSelect(item.value as ThinkingLevel);
 		};
 
-		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/todo-display.ts

@@ -27,17 +27,17 @@ async function loadTodoFile(filePath: string): Promise<TodoFile | null> {
 }
 
 export class TodoDisplayComponent {
-	public todos: TodoItem[] = [];
-	private expanded = false;
-	private visible = false;
-	private cached: RenderCache | undefined;
+	todos: TodoItem[] = [];
+	#expanded = false;
+	#visible = false;
+	#cached: RenderCache | undefined;
 
 	constructor(private readonly sessionFile: string | null) {}
 
 	async loadTodos(): Promise<void> {
 		if (!this.sessionFile) {
 			this.todos = [];
-			this.visible = false;
+			this.#visible = false;
 			return;
 		}
 
@@ -45,36 +45,36 @@ export class TodoDisplayComponent {
 		const todoPath = path.join(artifactsDir, TODO_FILE_NAME);
 		const data = await loadTodoFile(todoPath);
 		this.todos = data?.todos ?? [];
-		this.visible = this.todos.length > 0;
-		this.cached = undefined;
+		this.#visible = this.todos.length > 0;
+		this.#cached = undefined;
 	}
 
 	setTodos(todos: TodoItem[]): void {
 		this.todos = todos;
-		this.visible = this.todos.length > 0;
-		this.cached = undefined;
+		this.#visible = this.todos.length > 0;
+		this.#cached = undefined;
 	}
 
 	setExpanded(expanded: boolean): void {
-		this.expanded = expanded;
-		this.cached = undefined;
+		this.#expanded = expanded;
+		this.#cached = undefined;
 	}
 
 	isVisible(): boolean {
-		return this.visible;
+		return this.#visible;
 	}
 
 	render(width: number): string[] {
-		if (!this.visible || this.todos.length === 0) {
+		if (!this.#visible || this.todos.length === 0) {
 			return [];
 		}
 
-		const key = new Hasher().bool(this.expanded).u32(width).digest();
-		if (this.cached?.key === key) return this.cached.lines;
+		const key = new Hasher().bool(this.#expanded).u32(width).digest();
+		if (this.#cached?.key === key) return this.#cached.lines;
 
 		const lines: string[] = [];
-		const maxItems = this.expanded ? this.todos.length : Math.min(5, this.todos.length);
-		const hasMore = !this.expanded && this.todos.length > 5;
+		const maxItems = this.#expanded ? this.todos.length : Math.min(5, this.todos.length);
+		const hasMore = !this.#expanded && this.todos.length > 5;
 
 		for (let i = 0; i < maxItems; i++) {
 			const todo = this.todos[i];
@@ -102,12 +102,12 @@ export class TodoDisplayComponent {
 		}
 
 		const result = lines.map(l => truncateToWidth(l, width, Ellipsis.Omit));
-		this.cached = { key, lines: result };
+		this.#cached = { key, lines: result };
 		return result;
 	}
 
 	getRenderedComponent(): Text | null {
-		if (!this.visible) return null;
+		if (!this.#visible) return null;
 		const lines = this.render(80);
 		return new Text(lines.join("\n"), 0, 0);
 	}

package/src/modes/components/todo-reminder.ts

@@ -7,7 +7,7 @@ import type { TodoItem } from "../../tools/todo-write";
  * Shows when the agent stops with incomplete todos.
  */
 export class TodoReminderComponent extends Container {
-	private box: Box;
+	#box: Box;
 
 	constructor(
 		private readonly todos: TodoItem[],
@@ -18,23 +18,23 @@ export class TodoReminderComponent extends Container {
 
 		this.addChild(new Spacer(1));
 
-		this.box = new Box(1, 1, t => theme.inverse(theme.fg("warning", t)));
-		this.addChild(this.box);
+		this.#box = new Box(1, 1, t => theme.inverse(theme.fg("warning", t)));
+		this.addChild(this.#box);
 
-		this.rebuild();
+		this.#rebuild();
 	}
 
-	private rebuild(): void {
-		this.box.clear();
+	#rebuild(): void {
+		this.#box.clear();
 
 		const count = this.todos.length;
 		const label = count === 1 ? "todo" : "todos";
 		const header = `${theme.icon.warning} ${count} incomplete ${label} - reminder ${this.attempt}/${this.maxAttempts}`;
 
-		this.box.addChild(new Text(header, 0, 0));
-		this.box.addChild(new Spacer(1));
+		this.#box.addChild(new Text(header, 0, 0));
+		this.#box.addChild(new Spacer(1));
 
 		const todoList = this.todos.map(t => `  ${theme.checkbox.unchecked} ${t.content}`).join("\n");
-		this.box.addChild(new Text(theme.italic(todoList), 0, 0));
+		this.#box.addChild(new Text(theme.italic(todoList), 0, 0));
 	}
 }