@oh-my-pi/pi-coding-agent 3.13.1337 → 3.15.0

package/src/core/compaction/compaction.ts

@@ -8,6 +8,9 @@
 import type { AgentMessage } from "@oh-my-pi/pi-agent-core";
 import type { AssistantMessage, Model, Usage } from "@oh-my-pi/pi-ai";
 import { complete, completeSimple } from "@oh-my-pi/pi-ai";
+import compactionSummaryPrompt from "../../prompts/compaction-summary.md" with { type: "text" };
+import compactionTurnPrefixPrompt from "../../prompts/compaction-turn-prefix.md" with { type: "text" };
+import compactionUpdateSummaryPrompt from "../../prompts/compaction-update-summary.md" with { type: "text" };
 import { convertToLlm, createBranchSummaryMessage, createHookMessage } from "../messages";
 import type { CompactionEntry, SessionEntry } from "../session-manager";
 import {
@@ -383,77 +386,9 @@ export function findCutPoint(
 // Summarization
 // ============================================================================
 
-const SUMMARIZATION_PROMPT = `The messages above are a conversation to summarize. Create a structured context checkpoint summary that another LLM will use to continue the work.
+const SUMMARIZATION_PROMPT = compactionSummaryPrompt;
 
-Use this EXACT format:
-
-## Goal
-[What is the user trying to accomplish? Can be multiple items if the session covers different tasks.]
-
-## Constraints & Preferences
-- [Any constraints, preferences, or requirements mentioned by user]
-- [Or "(none)" if none were mentioned]
-
-## Progress
-### Done
-- [x] [Completed tasks/changes]
-
-### In Progress
-- [ ] [Current work]
-
-### Blocked
-- [Issues preventing progress, if any]
-
-## Key Decisions
-- **[Decision]**: [Brief rationale]
-
-## Next Steps
-1. [Ordered list of what should happen next]
-
-## Critical Context
-- [Any data, examples, or references needed to continue]
-- [Or "(none)" if not applicable]
-
-Keep each section concise. Preserve exact file paths, function names, and error messages.`;
-
-const UPDATE_SUMMARIZATION_PROMPT = `The messages above are NEW conversation messages to incorporate into the existing summary provided in <previous-summary> tags.
-
-Update the existing structured summary with new information. RULES:
-- PRESERVE all existing information from the previous summary
-- ADD new progress, decisions, and context from the new messages
-- UPDATE the Progress section: move items from "In Progress" to "Done" when completed
-- UPDATE "Next Steps" based on what was accomplished
-- PRESERVE exact file paths, function names, and error messages
-- If something is no longer relevant, you may remove it
-
-Use this EXACT format:
-
-## Goal
-[Preserve existing goals, add new ones if the task expanded]
-
-## Constraints & Preferences
-- [Preserve existing, add new ones discovered]
-
-## Progress
-### Done
-- [x] [Include previously done items AND newly completed items]
-
-### In Progress
-- [ ] [Current work - update based on progress]
-
-### Blocked
-- [Current blockers - remove if resolved]
-
-## Key Decisions
-- **[Decision]**: [Brief rationale] (preserve all previous, add new)
-
-## Next Steps
-1. [Update based on current state]
-
-## Critical Context
-- [Preserve important context, add new if needed]
-
-Keep each section concise. Preserve exact file paths, function names, and error messages.`;
+const UPDATE_SUMMARIZATION_PROMPT = compactionUpdateSummaryPrompt;
 
 /**
  * Generate a summary of the conversation using the LLM.
@@ -617,20 +552,7 @@ export function prepareCompaction(
 // Main compaction function
 // ============================================================================
 
-const TURN_PREFIX_SUMMARIZATION_PROMPT = `This is the PREFIX of a turn that was too large to keep. The SUFFIX (recent work) is retained.
-
-Summarize the prefix to provide context for the retained suffix:
-
-## Original Request
-[What did the user ask for in this turn?]
-
-## Early Progress
-- [Key decisions and work done in the prefix]
-
-## Context for Suffix
-- [Information needed to understand the retained recent work]
-
-Be concise. Focus on what's needed to understand the kept suffix.`;
+const TURN_PREFIX_SUMMARIZATION_PROMPT = compactionTurnPrefixPrompt;
 
 /**
  * Generate summaries for compaction using prepared data.

package/src/core/compaction/utils.ts

@@ -4,6 +4,7 @@
 
 import type { AgentMessage } from "@oh-my-pi/pi-agent-core";
 import type { Message } from "@oh-my-pi/pi-ai";
+import summarizationSystemPrompt from "../../prompts/summarization-system.md" with { type: "text" };
 
 // ============================================================================
 // File Operation Tracking
@@ -149,6 +150,4 @@ export function serializeConversation(messages: Message[]): string {
 // Summarization System Prompt
 // ============================================================================
 
-export const SUMMARIZATION_SYSTEM_PROMPT = `You are a context summarization assistant. Your task is to read a conversation between a user and an AI coding assistant, then produce a structured summary following the exact format specified.
-
-Do NOT continue the conversation. Do NOT respond to any questions in the conversation. ONLY output the structured summary.`;
+export const SUMMARIZATION_SYSTEM_PROMPT = summarizationSystemPrompt;

package/src/core/custom-tools/types.ts

@@ -76,6 +76,8 @@ export interface RenderResultOptions {
 	expanded: boolean;
 	/** Whether this is a partial/streaming result */
 	isPartial: boolean;
+	/** Current spinner frame index for animated elements (0-9, only provided during partial results) */
+	spinnerFrame?: number;
 }
 
 export type CustomToolResult<TDetails = any> = AgentToolResult<TDetails>;

package/src/core/export-html/index.ts

@@ -144,7 +144,7 @@ export async function exportFromFile(inputPath: string, options?: ExportOptions
 
 	if (!existsSync(inputPath)) throw new Error(`File not found: ${inputPath}`);
 
-	const sm = SessionManager.open(inputPath);
+	const sm = await SessionManager.open(inputPath);
 	const sessionData: SessionData = {
 		header: sm.getHeader(),
 		entries: sm.getEntries(),

package/src/core/hooks/index.ts

@@ -13,4 +13,4 @@ export {
 export { execCommand, HookRunner, type HookErrorListener } from "./runner";
 export { wrapToolsWithHooks, wrapToolWithHooks } from "./tool-wrapper";
 export * from "./types";
-export type { ReadonlySessionManager } from "../session-manager";
+export type { UsageStatistics, ReadonlySessionManager } from "../session-manager";

package/src/core/hooks/tool-wrapper.ts

@@ -59,7 +59,6 @@ export function wrapToolWithHooks<T>(tool: AgentTool<any, T>, hookRunner: HookRu
 						input: params,
 						content: result.content,
 						details: result.details,
-						isError: false,
 					})) as ToolResultEventResult | undefined;
 
 					// Apply modifications if any

package/src/core/hooks/types.ts

@@ -129,7 +129,7 @@ export interface HookUIContext {
 	 *
 	 * @example
 	 * const theme = ctx.ui.theme;
-	 * ctx.ui.setStatus("my-hook", theme.fg("success", "✓") + " Ready");
+	 * ctx.ui.setStatus("my-hook", theme.fg("success", theme.status.success) + " Ready");
 	 */
 	readonly theme: Theme;
 }
@@ -410,7 +410,7 @@ interface ToolResultEventBase {
 	/** Full content array (text and images) */
 	content: (TextContent | ImageContent)[];
 	/** Whether the tool execution was an error */
-	isError: boolean;
+	isError?: boolean;
 }
 
 /** Tool result event for bash tool */

package/src/core/plugins/doctor.ts

@@ -1,3 +1,4 @@
+import { theme } from "../../modes/interactive/theme/theme";
 import type { DoctorCheck } from "./types";
 
 export async function runDoctorChecks(): Promise<DoctorCheck[]> {
@@ -42,10 +43,17 @@ export async function runDoctorChecks(): Promise<DoctorCheck[]> {
 }
 
 export function formatDoctorResults(checks: DoctorCheck[]): string {
+	// Note: This function returns plain text without theming as it may be called outside TUI context.
+	// For TUI usage, the plugin CLI handler applies theme colors.
 	const lines: string[] = ["System Health Check", "=".repeat(40), ""];
 
 	for (const check of checks) {
-		const icon = check.status === "ok" ? "✓" : check.status === "warning" ? "!" : "✗";
+		const icon =
+			check.status === "ok"
+				? theme.status.success
+				: check.status === "warning"
+					? theme.status.warning
+					: theme.status.error;
 		lines.push(`${icon} ${check.name}: ${check.message}`);
 	}
 

package/src/core/sdk.ts

@@ -32,6 +32,7 @@
 import { join } from "node:path";
 import { Agent, type ThinkingLevel } from "@oh-my-pi/pi-agent-core";
 import type { Model } from "@oh-my-pi/pi-ai";
+import chalk from "chalk";
 // Import discovery to register all providers on startup
 import "../discovery";
 import { loadSync as loadCapability } from "../capability/index";
@@ -684,7 +685,7 @@ export async function createAgentSession(options: CreateAgentSessionOptions = {}
 		const mcpResult = await discoverAndLoadMCPTools(cwd, {
 			onConnecting: (serverNames) => {
 				if (options.hasUI && serverNames.length > 0) {
-					process.stderr.write(`\x1b[90mConnecting to MCP servers: ${serverNames.join(", ")}...\x1b[0m\n`);
+					process.stderr.write(chalk.gray(`Connecting to MCP servers: ${serverNames.join(", ")}...\n`));
 				}
 			},
 			enableProjectConfig: settingsManager.getMCPProjectConfigEnabled(),