@oh-my-pi/pi-coding-agent 4.6.0 → 4.8.0

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

@@ -3,6 +3,7 @@ import type { AgentSessionEvent } from "../../../core/agent-session";
 import { detectNotificationProtocol, isNotificationSuppressed, sendNotification } from "../../../core/terminal-notify";
 import { AssistantMessageComponent } from "../components/assistant-message";
 import { ReadToolGroupComponent } from "../components/read-tool-group";
+import { TodoReminderComponent } from "../components/todo-reminder";
 import { ToolExecutionComponent } from "../components/tool-execution";
 import { TtsrNotificationComponent } from "../components/ttsr-notification";
 import { getSymbolTheme, theme } from "../theme/theme";
@@ -374,6 +375,13 @@ export class EventController {
 				this.ctx.ui.requestRender();
 				break;
 			}
+
+			case "todo_reminder": {
+				const component = new TodoReminderComponent(event.todos, event.attempt, event.maxAttempts);
+				this.ctx.chatContainer.addChild(component);
+				this.ctx.ui.requestRender();
+				break;
+			}
 		}
 	}
 

package/src/modes/interactive/controllers/extension-ui-controller.ts

@@ -25,7 +25,7 @@ export class ExtensionUiController {
 	async initHooksAndCustomTools(): Promise<void> {
 		// Create and set hook & tool UI context
 		const uiContext: ExtensionUIContext = {
-			select: (title, options, _dialogOptions) => this.showHookSelector(title, options),
+			select: (title, options, dialogOptions) => this.showHookSelector(title, options, dialogOptions?.initialIndex),
 			confirm: (title, message, _dialogOptions) => this.showHookConfirm(title, message),
 			input: (title, placeholder, _dialogOptions) => this.showHookInput(title, placeholder),
 			notify: (message, type) => this.showHookNotify(message, type),
@@ -141,6 +141,7 @@ export class ExtensionUiController {
 				this.ctx.chatContainer.addChild(
 					new Text(`${theme.fg("accent", `${theme.status.success} New session started`)}`, 1, 1),
 				);
+				await this.ctx.reloadTodos();
 				this.ctx.ui.requestRender();
 
 				return { cancelled: false };
@@ -154,6 +155,7 @@ export class ExtensionUiController {
 				// Update UI
 				this.ctx.chatContainer.clear();
 				this.ctx.renderInitialMessages();
+				await this.ctx.reloadTodos();
 				this.ctx.editor.setText(result.selectedText);
 				this.ctx.showStatus("Branched to new session");
 
@@ -168,6 +170,7 @@ export class ExtensionUiController {
 				// Update UI
 				this.ctx.chatContainer.clear();
 				this.ctx.renderInitialMessages();
+				await this.ctx.reloadTodos();
 				if (result.editorText) {
 					this.ctx.editor.setText(result.editorText);
 				}
@@ -289,6 +292,7 @@ export class ExtensionUiController {
 				this.ctx.chatContainer.addChild(
 					new Text(`${theme.fg("accent", `${theme.status.success} New session started`)}`, 1, 1),
 				);
+				await this.ctx.reloadTodos();
 				this.ctx.ui.requestRender();
 
 				return { cancelled: false };
@@ -305,6 +309,7 @@ export class ExtensionUiController {
 				// Update UI
 				this.ctx.chatContainer.clear();
 				this.ctx.renderInitialMessages();
+				await this.ctx.reloadTodos();
 				this.ctx.editor.setText(result.selectedText);
 				this.ctx.showStatus("Branched to new session");
 
@@ -322,6 +327,7 @@ export class ExtensionUiController {
 				// Update UI
 				this.ctx.chatContainer.clear();
 				this.ctx.renderInitialMessages();
+				await this.ctx.reloadTodos();
 				if (result.editorText) {
 					this.ctx.editor.setText(result.editorText);
 				}
@@ -425,7 +431,7 @@ export class ExtensionUiController {
 	/**
 	 * Show a selector for hooks.
 	 */
-	showHookSelector(title: string, options: string[]): Promise<string | undefined> {
+	showHookSelector(title: string, options: string[], initialIndex?: number): Promise<string | undefined> {
 		return new Promise((resolve) => {
 			this.ctx.hookSelector = new HookSelectorComponent(
 				title,
@@ -438,6 +444,7 @@ export class ExtensionUiController {
 					this.hideHookSelector();
 					resolve(undefined);
 				},
+				{ initialIndex },
 			);
 
 			this.ctx.editorContainer.clear();

package/src/modes/interactive/controllers/input-controller.ts

@@ -309,7 +309,7 @@ export class InputController {
 					.then(async (title) => {
 						if (title) {
 							await this.ctx.sessionManager.setSessionTitle(title);
-							setTerminalTitle(`omp: ${title}`);
+							setTerminalTitle(`π: ${title}`);
 						}
 					})
 					.catch(() => {});

package/src/modes/interactive/controllers/selector-controller.ts

@@ -428,6 +428,7 @@ export class SelectorController {
 						// Update UI
 						this.ctx.chatContainer.clear();
 						this.ctx.renderInitialMessages();
+						await this.ctx.reloadTodos();
 						if (result.editorText) {
 							this.ctx.editor.setText(result.editorText);
 						}
@@ -500,6 +501,7 @@ export class SelectorController {
 		// Clear and re-render the chat
 		this.ctx.chatContainer.clear();
 		this.ctx.renderInitialMessages();
+		await this.ctx.reloadTodos();
 		this.ctx.showStatus("Resumed session");
 	}
 

package/src/modes/interactive/interactive-mode.ts

@@ -446,7 +446,7 @@ export class InteractiveMode implements InteractiveModeContext {
 
 		if (!this.todoExpanded && visibleTodos.length < this.todoItems.length) {
 			const remaining = this.todoItems.length - visibleTodos.length;
-			lines.push(theme.fg("muted", `${indent}${hook} +${remaining} more (Ctrl+T to expand)`));
+			lines.push(theme.fg("muted", `${indent}  ${hook} +${remaining} more (Ctrl+T to expand)`));
 		}
 
 		this.todoContainer.addChild(new Text(lines.join("\n"), 1, 0));
@@ -757,6 +757,11 @@ export class InteractiveMode implements InteractiveModeContext {
 		this.ui.requestRender();
 	}
 
+	async reloadTodos(): Promise<void> {
+		await this.loadTodoList();
+		this.ui.requestRender();
+	}
+
 	openExternalEditor(): void {
 		this.inputController.openExternalEditor();
 	}
@@ -810,8 +815,8 @@ export class InteractiveMode implements InteractiveModeContext {
 		this.extensionUiController.setHookStatus(key, text);
 	}
 
-	showHookSelector(title: string, options: string[]): Promise<string | undefined> {
-		return this.extensionUiController.showHookSelector(title, options);
+	showHookSelector(title: string, options: string[], initialIndex?: number): Promise<string | undefined> {
+		return this.extensionUiController.showHookSelector(title, options, initialIndex);
 	}
 
 	hideHookSelector(): void {

package/src/modes/interactive/types.ts

@@ -124,6 +124,7 @@ export interface InteractiveModeContext {
 	updateEditorBorderColor(): void;
 	rebuildChatFromMessages(): void;
 	setTodos(todos: TodoItem[]): void;
+	reloadTodos(): Promise<void>;
 	toggleTodoExpansion(): void;
 
 	// Command handling
@@ -184,7 +185,7 @@ export interface InteractiveModeContext {
 	): Promise<void>;
 	setHookWidget(key: string, content: unknown): void;
 	setHookStatus(key: string, text: string | undefined): void;
-	showHookSelector(title: string, options: string[]): Promise<string | undefined>;
+	showHookSelector(title: string, options: string[], initialIndex?: number): Promise<string | undefined>;
 	hideHookSelector(): void;
 	showHookInput(title: string, placeholder?: string): Promise<string | undefined>;
 	hideHookInput(): void;

package/src/prompts/tools/ask.md

@@ -13,11 +13,25 @@ Tips:
 - 2-5 concise, distinct options
 - Users can always select "Other" for custom input
 
+**Do NOT include an "Other" option in your options array.** The UI automatically adds "Other (type your own)" to every question. Adding your own creates duplicate "Other" options.
+
 <example>
 question: "Which authentication method should this API use?"
 options: [{"label": "JWT (Recommended)"}, {"label": "OAuth2"}, {"label": "Session cookies"}]
 </example>
 
+## Multi-part questions
+
+When you have multiple related questions, use the `questions` array instead of asking one at a time. Each question has its own id, options, and optional `multi` flag.
+
+<example>
+questions: [
+  {"id": "auth", "question": "Which auth method?", "options": [{"label": "JWT"}, {"label": "OAuth2"}]},
+  {"id": "cache", "question": "Enable caching?", "options": [{"label": "Yes"}, {"label": "No"}]},
+  {"id": "features", "question": "Which features to include?", "options": [{"label": "Logging"}, {"label": "Metrics"}, {"label": "Tracing"}], "multi": true}
+]
+</example>
+
 ## Critical: Resolve before asking
 
 **Exhaust all other options before asking.** Questions interrupt user flow.