@oh-my-pi/pi-coding-agent 14.9.2 → 14.9.5

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

@@ -539,6 +539,10 @@ class TreeList implements Component {
 					const msgWithContent = msg as { content?: unknown };
 					const content = normalize(this.#extractContent(msgWithContent.content));
 					result = theme.fg("accent", "user: ") + content;
+				} else if (role === "developer") {
+					const msgWithContent = msg as { content?: unknown };
+					const content = normalize(this.#extractContent(msgWithContent.content));
+					result = theme.fg("dim", "developer: ") + theme.fg("muted", content);
 				} else if (role === "assistant") {
 					const msgWithContent = msg as { content?: unknown; stopReason?: string; errorMessage?: string };
 					const textContent = normalize(this.#extractContent(msgWithContent.content));

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

@@ -1,6 +1,6 @@
 import { INTENT_FIELD } from "@oh-my-pi/pi-agent-core";
 import type { AssistantMessage, ImageContent } from "@oh-my-pi/pi-ai";
-import { Loader, TERMINAL, Text } from "@oh-my-pi/pi-tui";
+import { type Component, Loader, TERMINAL, Text } from "@oh-my-pi/pi-tui";
 import { settings } from "../../config/settings";
 import { AssistantMessageComponent } from "../../modes/components/assistant-message";
 import { ReadToolGroupComponent } from "../../modes/components/read-tool-group";
@@ -15,6 +15,8 @@ import type { ExitPlanModeDetails } from "../../tools";
 
 type AgentSessionEventKind = AgentSessionEvent["type"];
 
+const IRC_MESSAGE_VISIBLE_TTL_MS = 10_000;
+
 type AgentSessionEventHandlers = {
 	[E in AgentSessionEventKind]: (event: Extract<AgentSessionEvent, { type: E }>) => Promise<void>;
 };
@@ -29,6 +31,7 @@ export class EventController {
 	#readToolCallAssistantComponents = new Map<string, AssistantMessageComponent>();
 	#lastAssistantComponent: AssistantMessageComponent | undefined = undefined;
 	#idleCompactionTimer?: NodeJS.Timeout;
+	#ircExpiryTimers = new Map<string, NodeJS.Timeout>();
 	#handlers: AgentSessionEventHandlers;
 
 	constructor(private ctx: InteractiveModeContext) {
@@ -59,6 +62,10 @@ export class EventController {
 
 	dispose(): void {
 		this.#cancelIdleCompaction();
+		for (const timer of this.#ircExpiryTimers.values()) {
+			clearTimeout(timer);
+		}
+		this.#ircExpiryTimers.clear();
 	}
 
 	#resetReadGroup(): void {
@@ -222,10 +229,24 @@ export class EventController {
 		}
 		this.#renderedCustomMessages.add(signature);
 		this.#resetReadGroup();
-		this.ctx.addMessageToChat(event.message);
+		const components = this.ctx.addMessageToChat(event.message);
+		this.#scheduleIrcExpiry(signature, components);
 		this.ctx.ui.requestRender();
 	}
 
+	#scheduleIrcExpiry(signature: string, components: Component[]): void {
+		if (components.length === 0 || this.#ircExpiryTimers.has(signature)) return;
+		const timer = setTimeout(() => {
+			this.#ircExpiryTimers.delete(signature);
+			for (const component of components) {
+				this.ctx.chatContainer.removeChild(component);
+			}
+			this.ctx.ui.requestRender();
+		}, IRC_MESSAGE_VISIBLE_TTL_MS);
+		timer.unref?.();
+		this.#ircExpiryTimers.set(signature, timer);
+	}
+
 	async #handleNotice(event: Extract<AgentSessionEvent, { type: "notice" }>): Promise<void> {
 		const message = event.source ? `${event.source}: ${event.message}` : event.message;
 		if (event.level === "error") {

package/src/modes/controllers/mcp-command-controller.ts

@@ -1238,12 +1238,12 @@ export class MCPCommandController {
 								? theme.fg("muted", "Connecting")
 								: theme.fg("warning", "Not connected yet");
 					this.#showMessage(
-						["", theme.fg("success", `\u2713 Enabled "${name}"`), "", `  Status: ${status}`, ""].join("\n"),
+						["", theme.fg("success", `✓ Enabled "${name}"`), "", `  Status: ${status}`, ""].join("\n"),
 					);
 				} else {
 					await this.ctx.mcpManager?.disconnectServer(name);
 					await this.ctx.session.refreshMCPTools(this.ctx.mcpManager?.getTools() ?? []);
-					this.#showMessage(["", theme.fg("success", `\u2713 Disabled "${name}"`), ""].join("\n"));
+					this.#showMessage(["", theme.fg("success", `✓ Disabled "${name}"`), ""].join("\n"));
 				}
 				return;
 			}
@@ -1429,12 +1429,9 @@ export class MCPCommandController {
 				await this.ctx.session.refreshMCPTools(this.ctx.mcpManager.getTools());
 				const serverTools = this.ctx.mcpManager.getTools().filter(t => t.mcpServerName === name);
 				this.#showMessage(
-					[
-						"\n",
-						theme.fg("success", `\u2713 Reconnected to "${name}"`),
-						`  Tools: ${serverTools.length}`,
+					["\n", theme.fg("success", `✓ Reconnected to "${name}"`), `  Tools: ${serverTools.length}`, "\n"].join(
 						"\n",
-					].join("\n"),
+					),
 				);
 			} else {
 				this.ctx.showError(`Failed to reconnect to "${name}". Check server status and logs.`);
@@ -1589,8 +1586,8 @@ export class MCPCommandController {
 			hasAny = true;
 
 			lines.push(`${theme.fg("accent", name)}:`);
-			const check = theme.fg("success", "\u2713");
-			const cross = theme.fg("dim", "\u2717");
+			const check = theme.fg("success", "✓");
+			const cross = theme.fg("dim", "✗");
 			if (supportsToolsChanged) lines.push(`  ${check} tools/list_changed`);
 			if (supportsResourcesChanged) lines.push(`  ${check} resources/list_changed`);
 			if (supportsPromptsChanged) lines.push(`  ${check} prompts/list_changed`);
@@ -1607,7 +1604,7 @@ export class MCPCommandController {
 				lines.push(`  ${check} resources/subscribe  ${subStatus}`);
 				if (enabled && subscribedUris && subscribedUris.size > 0) {
 					for (const uri of subscribedUris) {
-						lines.push(`    ${theme.fg("success", "\u2713")} ${theme.fg("dim", uri)}`);
+						lines.push(`    ${theme.fg("success", "✓")} ${theme.fg("dim", uri)}`);
 					}
 				}
 			} else if (supportsResources) {

package/src/modes/interactive-mode.ts

@@ -1502,8 +1502,8 @@ export class InteractiveMode implements InteractiveModeContext {
 		return this.#uiHelpers.isKnownSlashCommand(text);
 	}
 
-	addMessageToChat(message: AgentMessage, options?: { populateHistory?: boolean }): void {
-		this.#uiHelpers.addMessageToChat(message, options);
+	addMessageToChat(message: AgentMessage, options?: { populateHistory?: boolean }): Component[] {
+		return this.#uiHelpers.addMessageToChat(message, options);
 	}
 
 	renderSessionContext(

package/src/modes/theme/theme.ts

@@ -387,51 +387,51 @@ const NERD_SYMBOLS: SymbolMap = {
 	"nav.back": "\uf060",
 	// Tree Connectors (same as unicode)
 	// pick: ├─ | alt: ├╴ ├╌ ╠═ ┣━
-	"tree.branch": "\u251c\u2500",
+	"tree.branch": "├─",
 	// pick: └─ | alt: └╴ └╌ ╚═ ┗━
-	"tree.last": "\u2514\u2500",
+	"tree.last": "└─",
 	// pick: │ | alt: ┃ ║ ▏ ▕
-	"tree.vertical": "\u2502",
+	"tree.vertical": "│",
 	// pick: ─ | alt: ━ ═ ╌ ┄
-	"tree.horizontal": "\u2500",
+	"tree.horizontal": "─",
 	// pick: └ | alt: ╰ ⎿ ↳
-	"tree.hook": "\u2514",
+	"tree.hook": "└",
 	// Box Drawing - Rounded (same as unicode)
 	// pick: ╭ | alt: ┌ ┏ ╔
-	"boxRound.topLeft": "\u256d",
+	"boxRound.topLeft": "╭",
 	// pick: ╮ | alt: ┐ ┓ ╗
-	"boxRound.topRight": "\u256e",
+	"boxRound.topRight": "╮",
 	// pick: ╰ | alt: └ ┗ ╚
-	"boxRound.bottomLeft": "\u2570",
+	"boxRound.bottomLeft": "╰",
 	// pick: ╯ | alt: ┘ ┛ ╝
-	"boxRound.bottomRight": "\u256f",
+	"boxRound.bottomRight": "╯",
 	// pick: ─ | alt: ━ ═ ╌
-	"boxRound.horizontal": "\u2500",
+	"boxRound.horizontal": "─",
 	// pick: │ | alt: ┃ ║ ▏
-	"boxRound.vertical": "\u2502",
+	"boxRound.vertical": "│",
 	// Box Drawing - Sharp (same as unicode)
 	// pick: ┌ | alt: ┏ ╭ ╔
-	"boxSharp.topLeft": "\u250c",
+	"boxSharp.topLeft": "┌",
 	// pick: ┐ | alt: ┓ ╮ ╗
-	"boxSharp.topRight": "\u2510",
+	"boxSharp.topRight": "┐",
 	// pick: └ | alt: ┗ ╰ ╚
-	"boxSharp.bottomLeft": "\u2514",
+	"boxSharp.bottomLeft": "└",
 	// pick: ┘ | alt: ┛ ╯ ╝
-	"boxSharp.bottomRight": "\u2518",
+	"boxSharp.bottomRight": "┘",
 	// pick: ─ | alt: ━ ═ ╌
-	"boxSharp.horizontal": "\u2500",
+	"boxSharp.horizontal": "─",
 	// pick: │ | alt: ┃ ║ ▏
-	"boxSharp.vertical": "\u2502",
+	"boxSharp.vertical": "│",
 	// pick: ┼ | alt: ╋ ╬ ┿
-	"boxSharp.cross": "\u253c",
+	"boxSharp.cross": "┼",
 	// pick: ┬ | alt: ╦ ┯ ┳
-	"boxSharp.teeDown": "\u252c",
+	"boxSharp.teeDown": "┬",
 	// pick: ┴ | alt: ╩ ┷ ┻
-	"boxSharp.teeUp": "\u2534",
+	"boxSharp.teeUp": "┴",
 	// pick: ├ | alt: ╠ ┝ ┣
-	"boxSharp.teeRight": "\u251c",
+	"boxSharp.teeRight": "├",
 	// pick: ┤ | alt: ╣ ┥ ┫
-	"boxSharp.teeLeft": "\u2524",
+	"boxSharp.teeLeft": "┤",
 	// Separators - Nerd Font specific
 	// pick:  | alt:   
 	"sep.powerline": "\ue0b0",
@@ -446,7 +446,7 @@ const NERD_SYMBOLS: SymbolMap = {
 	// pick:  | alt: 
 	"sep.powerlineThinRight": "\ue0b3",
 	// pick: █ | alt: ▓ ▒ ░ ▉ ▌
-	"sep.block": "\u2588",
+	"sep.block": "█",
 	// pick: space | alt: ␠ ·
 	"sep.space": " ",
 	// pick: > | alt: › » ▸
@@ -454,7 +454,7 @@ const NERD_SYMBOLS: SymbolMap = {
 	// pick: < | alt: ‹ « ◂
 	"sep.asciiRight": "<",
 	// pick: · | alt: • ⋅
-	"sep.dot": " \u00b7 ",
+	"sep.dot": " · ",
 	// pick:  | alt: / ∕ ⁄
 	"sep.slash": "\ue0bb",
 	// pick:  | alt: │ ┃ |
@@ -545,16 +545,16 @@ const NERD_SYMBOLS: SymbolMap = {
 	// pick:  | alt:   •
 	"format.bullet": "\uf111",
 	// pick: – | alt: — ― -
-	"format.dash": "\u2013",
+	"format.dash": "–",
 	// pick: ⟨ | alt: [ ⟦
 	"format.bracketLeft": "⟨",
 	// pick: ⟩ | alt: ] ⟧
 	"format.bracketRight": "⟩",
 	// Markdown-specific
 	// pick: │ | alt: ┃ ║
-	"md.quoteBorder": "\u2502",
+	"md.quoteBorder": "│",
 	// pick: ─ | alt: ━ ═
-	"md.hrChar": "\u2500",
+	"md.hrChar": "─",
 	// pick:  | alt:  •
 	"md.bullet": "\uf111",
 	// Language icons (nerd font devicons)

package/src/modes/types.ts

@@ -170,7 +170,7 @@ export interface InteractiveModeContext {
 	 */
 	withLocalSubmission<T>(text: string, fn: () => Promise<T>, options?: { imageCount?: number }): Promise<T>;
 	isKnownSlashCommand(text: string): boolean;
-	addMessageToChat(message: AgentMessage, options?: { populateHistory?: boolean }): void;
+	addMessageToChat(message: AgentMessage, options?: { populateHistory?: boolean }): Component[];
 	renderSessionContext(
 		sessionContext: SessionContext,
 		options?: { updateFooter?: boolean; populateHistory?: boolean },

package/src/modes/utils/ui-helpers.ts

@@ -1,6 +1,6 @@
 import type { AgentMessage } from "@oh-my-pi/pi-agent-core";
 import type { AssistantMessage, ImageContent, Message } from "@oh-my-pi/pi-ai";
-import { Spacer, Text, TruncatedText } from "@oh-my-pi/pi-tui";
+import { type Component, Spacer, Text, TruncatedText } from "@oh-my-pi/pi-tui";
 import { settings } from "../../config/settings";
 import { AssistantMessageComponent } from "../../modes/components/assistant-message";
 import { BashExecutionComponent } from "../../modes/components/bash-execution";
@@ -70,7 +70,7 @@ export class UiHelpers {
 		this.ctx.ui.requestRender();
 	}
 
-	addMessageToChat(message: AgentMessage, options?: { populateHistory?: boolean }): void {
+	addMessageToChat(message: AgentMessage, options?: { populateHistory?: boolean }): Component[] {
 		switch (message.role) {
 			case "bashExecution": {
 				const component = new BashExecutionComponent(message.command, this.ctx.ui, message.excludeFromContext);
@@ -147,26 +147,30 @@ export class UiHelpers {
 						if (message.customType === "irc:incoming") {
 							const peer = details?.from ?? "?";
 							body = details?.message ?? "";
-							arrow = `\u21e6 ${peer}`;
+							arrow = `⇦ ${peer}`;
 						} else if (message.customType === "irc:autoreply") {
 							const peer = details?.to ?? "?";
 							body = details?.reply ?? "";
-							arrow = `\u21e8 ${peer} (auto)`;
+							arrow = `⇨ ${peer}`;
 						} else {
 							const from = details?.from ?? "?";
 							const to = details?.to ?? "?";
 							body = details?.body ?? "";
-							const suffix = details?.kind === "reply" ? " (auto)" : "";
-							arrow = `${from} \u21e8 ${to}${suffix}`;
+							arrow = `${from} ⇨ ${to}`;
 						}
+						const components: Component[] = [];
 						const header = `${theme.fg("accent", `[IRC] ${arrow}`)}`;
-						this.ctx.chatContainer.addChild(new Text(header, 1, 0));
+						const headerComponent = new Text(header, 1, 0);
+						this.ctx.chatContainer.addChild(headerComponent);
+						components.push(headerComponent);
 						if (body) {
 							for (const line of body.split("\n")) {
-								this.ctx.chatContainer.addChild(new Text(theme.fg("muted", `  ${line}`), 0, 0));
+								const lineComponent = new Text(theme.fg("muted", `  ${line}`), 0, 0);
+								this.ctx.chatContainer.addChild(lineComponent);
+								components.push(lineComponent);
 							}
 						}
-						break;
+						return components;
 					}
 					const renderer = this.ctx.session.extensionRunner?.getMessageRenderer(message.customType);
 					// Both HookMessage and CustomMessage have the same structure, cast for compatibility
@@ -240,6 +244,7 @@ export class UiHelpers {
 				const _exhaustive: never = message;
 			}
 		}
+		return [];
 	}
 
 	/**

package/src/prompts/commands/orchestrate.md

@@ -26,6 +26,7 @@ You decompose, dispatch, verify, and iterate. You do **not** edit code. Every fi
 6. **Commit policy.** If the task asks for commits or the repo workflow expects them, commit after each green phase with a focused message. Never commit a red tree. Never commit work the user did not ask to commit.
 7. **Respawn, do not absorb.** If a subagent returns incomplete or wrong work, spawn a corrective subagent with the specific gap — do not silently fix it yourself.
 8. **No scope creep, no scope shrink.** Do not add work the user did not ask for. Do not relabel unfinished items as "follow-up", "v1", or "MVP" to imply completion.
+9. **Subagents do not verify, lint, or format.** Every `task` assignment **MUST** instruct the subagent to skip all gates and formatters. Their job is the edit only. You — the orchestrator — run verification and formatting **once** at the end of the phase across the union of changed files. Avoids redundant runs and racing formatter passes.
 </rules>
 
 <workflow>

package/src/prompts/system/custom-system-prompt.md

@@ -57,8 +57,6 @@ Rules are local constraints. You **MUST** read `rule://<name>` when working in t
 {{/list}}
 </rules>
 {{/if}}
-Current date: {{date}}
-Current working directory: {{cwd}}
 {{#if secretsEnabled}}
 <redacted-content>
 Some values in tool output are redacted for security. They appear as `#XXXX#` tokens (4 uppercase-alphanumeric characters wrapped in `#`). These are **not errors** — they are intentional placeholders for sensitive values (API keys, passwords, tokens). Treat them as opaque strings. Do not attempt to decode, fix, or report them as problems.

package/src/prompts/system/project-prompt.md

@@ -1,3 +1,4 @@
+[PROJECT]
 <workstation>
 {{#list environment prefix="- " join="\n"}}{{label}}: {{value}}{{/list}}
 </workstation>
@@ -31,6 +32,15 @@ Working directory layout (sorted by mtime, recent first; depth ≤ 3):
 </workspace-tree>
 {{/if}}
 
+Today is {{date}}, and the current working directory is '{{cwd}}'.
+
+<critical>
+- Each response **MUST** advance the task. There is no stopping condition other than completion.
+- You **MUST** default to informed action; do not ask for confirmation when tools or repo context can answer.
+- You **MUST** verify the effect of significant behavioral changes before yielding: run the specific test, command, or scenario that covers your change.
+</critical>
+
 {{#if appendPrompt}}
 {{appendPrompt}}
 {{/if}}
+[/PROJECT]

package/src/prompts/system/subagent-system-prompt.md

@@ -1,31 +1,40 @@
-{{base}}
-
-{{SECTION_SEPARATOR "Acting as"}}
+[ROLE]
 {{agent}}
+[/ROLE]
+
+{{#if context}}
+[CONTEXT]
+{{context}}
+[/CONTEXT]
+{{/if}}
+
+[COOP]
+You are operating on a piece of work assigned to you by the main agent.
 
-{{SECTION_SEPARATOR "Job"}}
-You are operating on a delegated sub-task.
 {{#if worktree}}
+# Working Tree
 You are working in an isolated working tree at `{{worktree}}` for this sub-task.
 You **MUST NOT** modify files outside this tree or in the original repository.
 {{/if}}
 
 {{#if contextFile}}
+# Conversation Context
 If you need additional information, you can find your conversation with the user in {{contextFile}} (`tail` or `grep` relevant terms).
 {{/if}}
 
 {{#if ircPeers}}
-{{SECTION_SEPARATOR "IRC Peers"}}
+# IRC Peers
 You can reach other live agents via the `irc` tool. Your id is `{{ircSelfId}}`. Currently visible peers:
 {{ircPeers}}
 
 Use `irc` only when you need a quick answer from a peer; do not use it for long-form content. Address peers by id or use `"all"` to broadcast.
 {{/if}}
+[/COOP]
 
-{{SECTION_SEPARATOR "Closure"}}
+[COMPLETION]
 No TODO tracking, no progress updates. Execute, call `yield`, done.
 
-Every turn **MUST** end with a tool call. A turn whose final block is plain text (or thinking only) is treated as a stop and you will be reminded to yield. While work remains, always continue with another tool call — investigate, edit, run, verify. Save narrative for the final `yield` payload.
+While work remains, always continue with another tool call — investigate, edit, run, verify. Save narrative for the final `yield` payload.
 
 When finished, you **MUST** call `yield` exactly once. This is like writing to a ticket: provide what is required and close it.
 
@@ -38,8 +47,8 @@ Your result **MUST** match this TypeScript interface:
 ```
 {{/if}}
 
-{{SECTION_SEPARATOR "Giving Up"}}
 Giving up is a last resort. If truly blocked, you **MUST** call `yield` exactly once with `result.error` describing what you tried and the exact blocker.
 You **MUST NOT** give up due to uncertainty, missing information obtainable via tools or repo context, or needing a design decision you can derive yourself.
 
 You **MUST** keep going until this ticket is closed. This matters.
+[/COMPLETION]

package/src/prompts/system/subagent-user-prompt.md

@@ -1,12 +1,3 @@
-{{#if context}}
-{{SECTION_SEPARATOR "Background"}}
-<context>
-{{context}}
-</context>
-{{/if}}
+Complete the assignment below, thoroughly:
 
-{{SECTION_SEPARATOR "Task"}}
-Your assignment is below. Your work begins now.
-<goal>
 {{assignment}}
-</goal>