@oh-my-pi/pi-coding-agent 14.4.0 → 14.4.3

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

@@ -10,6 +10,7 @@ import {
 	truncateToWidth,
 	visibleWidth,
 } from "@oh-my-pi/pi-tui";
+import { formatBytes } from "@oh-my-pi/pi-utils";
 import { theme } from "../../modes/theme/theme";
 import { matchesAppInterrupt } from "../../modes/utils/keybinding-matchers";
 import type { SessionInfo } from "../../session/session-manager";
@@ -157,10 +158,9 @@ class SessionList implements Component {
 				lines.push(messageLine);
 			}
 
-			// Metadata line: date + message count
+			// Metadata line: date + file size
 			const modified = formatDate(session.modified);
-			const msgCount = `${session.messageCount} message${session.messageCount !== 1 ? "s" : ""}`;
-			const metadata = `  ${modified} ${theme.sep.dot} ${msgCount}`;
+			const metadata = `  ${modified} ${theme.sep.dot} ${formatBytes(session.size)}`;
 			const metadataLine = theme.fg("dim", truncateToWidth(metadata, width));
 
 			lines.push(metadataLine);

package/src/modes/components/settings-defs.ts

@@ -298,11 +298,6 @@ const OPTION_PROVIDERS: Partial<Record<SettingPath, OptionProvider>> = {
 		{ value: "1000", label: "1000 lines" },
 		{ value: "5000", label: "5000 lines" },
 	],
-	"read.anchorstyle": [
-		{ value: "full", label: "Full", description: "Show the kind prefix and identifier" },
-		{ value: "kind", label: "Kind", description: "Show only the kind prefix plus checksum" },
-		{ value: "bare", label: "Bare", description: "Show only the checksum" },
-	],
 	// Todo auto-clear delay
 	"tasks.todoClearDelay": [
 		{ value: "0", label: "Instant" },

package/src/modes/components/tool-execution.ts

@@ -109,7 +109,7 @@ export class ToolExecutionComponent extends Container {
 		isError?: boolean;
 		details?: any;
 	};
-	// Edit preview state (single-file for legacy modes, multi-file for chunk)
+	// Edit preview state
 	#editMode?: EditMode;
 	#editDiffPreview?: PerFileDiffPreview[];
 	#editDiffScheduleTimer?: NodeJS.Timeout;
@@ -639,10 +639,7 @@ export class ToolExecutionComponent extends Container {
 			return this.#args;
 		}
 		// Single-file previews feed the existing `previewDiff` channel consumed
-		// by `formatStreamingDiff` in the renderer. Multi-file previews are
-		// piped via `renderContext.perFileDiffPreview`, so the args we hand to
-		// `renderCall` only need the first file's diff to preserve prior
-		// single-file behavior.
+		// by `formatStreamingDiff` in the renderer.
 		const first = previews[0];
 		if (!first?.diff) {
 			return this.#args;

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

@@ -1,8 +1,5 @@
-import type { AgentMessage } from "@oh-my-pi/pi-agent-core";
-import { type AssistantMessage, type Context, streamSimple } from "@oh-my-pi/pi-ai";
 import { prompt } from "@oh-my-pi/pi-utils";
 import btwUserPrompt from "../../prompts/system/btw-user.md" with { type: "text" };
-import { toReasoningEffort } from "../../thinking";
 import { BtwPanelComponent } from "../components/btw-panel";
 import type { InteractiveModeContext } from "../types";
 
@@ -14,14 +11,8 @@ interface BtwRequest {
 
 export class BtwController {
 	#activeRequest: BtwRequest | undefined;
-	readonly #streamFn: typeof streamSimple;
 
-	constructor(
-		private readonly ctx: InteractiveModeContext,
-		options?: { streamFn?: typeof streamSimple },
-	) {
-		this.#streamFn = options?.streamFn ?? streamSimple;
-	}
+	constructor(private readonly ctx: InteractiveModeContext) {}
 
 	hasActiveRequest(): boolean {
 		return this.#activeRequest !== undefined;
@@ -61,64 +52,29 @@ export class BtwController {
 		this.ctx.btwContainer.addChild(request.component);
 		this.ctx.ui.requestRender();
 		this.#activeRequest = request;
-		void this.#runRequest(request, model);
+		void this.#runRequest(request);
 	}
 
-	async #runRequest(
-		request: BtwRequest,
-		model: NonNullable<InteractiveModeContext["session"]["model"]>,
-	): Promise<void> {
+	async #runRequest(request: BtwRequest): Promise<void> {
 		try {
-			const apiKey = await this.ctx.session.modelRegistry.getApiKey(model, this.ctx.session.sessionId);
-			if (!apiKey) {
-				throw new Error(`No API key for provider: ${model.provider}`);
-			}
-
-			const llmMessages = await this.ctx.session.convertMessagesToLlm(
-				[...this.#buildMessageSnapshot(), this.#buildQuestionMessage(request.question)],
-				request.abortController.signal,
-			);
-			const context: Context = {
-				systemPrompt: this.ctx.session.systemPrompt,
-				messages: llmMessages,
-			};
-			const options = this.ctx.session.prepareSimpleStreamOptions({
-				apiKey,
-				sessionId: this.ctx.session.sessionId,
-				reasoning: toReasoningEffort(this.ctx.session.thinkingLevel),
-				serviceTier: this.ctx.session.serviceTier,
+			const promptText = prompt.render(btwUserPrompt, { question: request.question });
+			const { replyText } = await this.ctx.session.runEphemeralTurn({
+				promptText,
+				onTextDelta: delta => {
+					if (this.#isActiveRequest(request)) {
+						request.component.appendText(delta);
+					}
+				},
 				signal: request.abortController.signal,
-				toolChoice: "none",
 			});
-			const stream = this.#streamFn(model, context, options);
 
-			for await (const event of stream) {
-				if (!this.#isActiveRequest(request)) {
-					return;
-				}
-				if (event.type === "text_delta") {
-					request.component.appendText(event.delta);
-					continue;
-				}
-				if (event.type === "done") {
-					const finalText = this.#assistantText(event.message);
-					if (finalText) {
-						request.component.setAnswer(finalText);
-					}
-					request.component.markComplete();
-					return;
-				}
-				if (event.type === "error") {
-					if (event.reason === "aborted" || request.abortController.signal.aborted) {
-						request.component.markAborted();
-					} else {
-						request.component.markError(
-							this.#assistantText(event.error) || event.error.errorMessage || "BTW request failed.",
-						);
-					}
-					return;
-				}
+			if (!this.#isActiveRequest(request)) {
+				return;
 			}
+			if (replyText) {
+				request.component.setAnswer(replyText);
+			}
+			request.component.markComplete();
 		} catch (error) {
 			if (!this.#isActiveRequest(request)) {
 				return;
@@ -131,50 +87,6 @@ export class BtwController {
 		}
 	}
 
-	#buildQuestionMessage(question: string): AgentMessage {
-		return {
-			role: "user",
-			content: [
-				{
-					type: "text",
-					text: prompt.render(btwUserPrompt, { question }),
-				},
-			],
-			attribution: "user",
-			timestamp: Date.now(),
-		};
-	}
-
-	#buildMessageSnapshot(): AgentMessage[] {
-		const messages = this.ctx.session.messages.slice();
-		if (!this.ctx.session.isStreaming || !this.ctx.streamingMessage) {
-			return messages;
-		}
-		const streamingText = this.ctx.extractAssistantText(this.ctx.streamingMessage);
-		const lastMessage = messages.at(-1);
-		if (!streamingText) {
-			return lastMessage?.role === "assistant" ? messages.slice(0, -1) : messages;
-		}
-		const normalizedStreamingMessage: AssistantMessage = {
-			...this.ctx.streamingMessage,
-			content: [{ type: "text", text: streamingText }],
-		};
-		if (lastMessage?.role === "assistant") {
-			return [...messages.slice(0, -1), normalizedStreamingMessage];
-		}
-		return [...messages, normalizedStreamingMessage];
-	}
-
-	#assistantText(message: AssistantMessage): string {
-		let text = "";
-		for (const content of message.content) {
-			if (content.type === "text") {
-				text += content.text;
-			}
-		}
-		return text.trim();
-	}
-
 	#closeActiveRequest(options: { abort: boolean }): void {
 		const request = this.#activeRequest;
 		if (!request) return;