@oh-my-pi/pi-coding-agent 14.4.1 → 14.4.4

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" },
@@ -353,6 +348,7 @@ const OPTION_PROVIDERS: Partial<Record<SettingPath, OptionProvider>> = {
 		{ value: "kagi", label: "Kagi", description: "Requires KAGI_API_KEY and Kagi Search API beta access" },
 		{ value: "synthetic", label: "Synthetic", description: "Requires SYNTHETIC_API_KEY" },
 		{ value: "parallel", label: "Parallel", description: "Requires PARALLEL_API_KEY" },
+		{ value: "searxng", label: "SearXNG", description: "Self-hosted metasearch; set searxng.endpoint" },
 	],
 	"providers.image": [
 		{

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;

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

@@ -28,6 +28,7 @@ import { buildHotkeysMarkdown } from "../../modes/utils/hotkeys-markdown";
 import { buildToolsMarkdown } from "../../modes/utils/tools-markdown";
 import type { AsyncJobSnapshotItem } from "../../session/agent-session";
 import type { AuthStorage } from "../../session/auth-storage";
+import type { NewSessionOptions } from "../../session/session-manager";
 import { outputMeta } from "../../tools/output-meta";
 import { resolveToCwd, stripOuterDoubleQuotes } from "../../tools/path-utils";
 import { replaceTabs } from "../../tools/render-utils";
@@ -573,7 +574,7 @@ export class CommandController {
 		this.ctx.showError("Usage: /memory <view|clear|reset|enqueue|rebuild>");
 	}
 
-	async handleClearCommand(): Promise<void> {
+	async #runNewSessionFlow(options?: NewSessionOptions, label: string = "New session started"): Promise<void> {
 		if (this.ctx.loadingAnimation) {
 			this.ctx.loadingAnimation.stop();
 			this.ctx.loadingAnimation = undefined;
@@ -586,7 +587,7 @@ export class CommandController {
 				await Bun.sleep(10);
 			}
 		}
-		await this.ctx.session.newSession();
+		if (!(await this.ctx.session.newSession(options))) return;
 		this.ctx.resetObserverRegistry();
 		setSessionTerminalTitle(
 			this.ctx.sessionManager.getSessionName(),
@@ -608,13 +609,23 @@ export class CommandController {
 		this.ctx.pendingTools.clear();
 
 		this.ctx.chatContainer.addChild(new Spacer(1));
-		this.ctx.chatContainer.addChild(
-			new Text(`${theme.fg("accent", `${theme.status.success} New session started`)}`, 1, 1),
-		);
+		this.ctx.chatContainer.addChild(new Text(`${theme.fg("accent", `${theme.status.success} ${label}`)}`, 1, 1));
 		await this.ctx.reloadTodos();
 		this.ctx.ui.requestRender();
 	}
 
+	async handleClearCommand(): Promise<void> {
+		await this.#runNewSessionFlow();
+	}
+
+	async handleDropCommand(): Promise<void> {
+		if (!this.ctx.sessionManager.getSessionFile()) {
+			this.ctx.showError("Nothing to drop (in-memory session)");
+			return;
+		}
+		await this.#runNewSessionFlow({ drop: true }, "Session dropped");
+	}
+
 	async handleForkCommand(): Promise<void> {
 		if (this.ctx.session.isStreaming) {
 			this.ctx.showWarning("Wait for the current response to finish or abort it before forking.");

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

@@ -1,14 +1,17 @@
+import * as os from "node:os";
+import * as path from "node:path";
 import { ThinkingLevel } from "@oh-my-pi/pi-agent-core";
 import { getOAuthProviders, type OAuthProvider } from "@oh-my-pi/pi-ai";
-import type { Component } from "@oh-my-pi/pi-tui";
+import type { Component, OverlayHandle } from "@oh-my-pi/pi-tui";
 import { Input, Loader, Spacer, Text } from "@oh-my-pi/pi-tui";
-import { getAgentDbPath, getProjectDir } from "@oh-my-pi/pi-utils";
+import { getAgentDbPath, getConfigDirName, getProjectDir } from "@oh-my-pi/pi-utils";
+import { invalidate as invalidateFsCache } from "../../capability/fs";
 import { getRoleInfo } from "../../config/model-registry";
 import { formatModelSelectorValue } from "../../config/model-resolver";
 import { settings } from "../../config/settings";
 import { DebugSelectorComponent } from "../../debug";
 import { disableProvider, enableProvider } from "../../discovery";
-import { clearPluginRootsAndCaches, resolveActiveProjectRegistryPath } from "../../discovery/helpers";
+import { clearClaudePluginRootsCache, resolveActiveProjectRegistryPath } from "../../discovery/helpers";
 import {
 	getInstalledPluginsRegistryPath,
 	getMarketplacesCacheDir,
@@ -440,7 +443,13 @@ export class SelectorController {
 			projectInstalledRegistryPath: (await resolveActiveProjectRegistryPath(getProjectDir())) ?? undefined,
 			marketplacesCacheDir: getMarketplacesCacheDir(),
 			pluginsCacheDir: getPluginsCacheDir(),
-			clearPluginRootsCache: clearPluginRootsAndCaches,
+			clearPluginRootsCache: (extraPaths?: readonly string[]) => {
+				const home = os.homedir();
+				invalidateFsCache(path.join(home, ".claude", "plugins", "installed_plugins.json"));
+				invalidateFsCache(path.join(home, getConfigDirName(), "plugins", "installed_plugins.json"));
+				for (const p of extraPaths ?? []) invalidateFsCache(p);
+				clearClaudePluginRootsCache();
+			},
 		});
 
 		const [marketplaces, installed] = await Promise.all([mgr.listMarketplaces(), mgr.listInstalledPlugins()]);
@@ -969,25 +978,29 @@ export class SelectorController {
 
 	showSessionObserver(registry: SessionObserverRegistry): void {
 		const observeKeys = this.ctx.keybindings.getKeys("app.session.observe");
+		let cleanup: (() => void) | undefined;
+		let overlayHandle: OverlayHandle | undefined;
 
-		this.showSelector(done => {
-			let cleanup: (() => void) | undefined;
+		const done = () => {
+			cleanup?.();
+			overlayHandle?.hide();
+			this.ctx.ui.requestRender();
+		};
 
-			const selector = new SessionObserverOverlayComponent(
-				registry,
-				() => {
-					cleanup?.();
-					done();
-				},
-				observeKeys,
-			);
+		const selector = new SessionObserverOverlayComponent(registry, done, observeKeys);
 
-			cleanup = registry.onChange(() => {
-				selector.refreshFromRegistry();
-				this.ctx.ui.requestRender();
-			});
+		cleanup = registry.onChange(() => {
+			selector.refreshFromRegistry();
+			this.ctx.ui.requestRender();
+		});
 
-			return { component: selector, focus: selector };
+		overlayHandle = this.ctx.ui.showOverlay(selector, {
+			anchor: "bottom-center",
+			width: "100%",
+			maxHeight: "100%",
+			margin: 0,
 		});
+		this.ctx.ui.setFocus(selector);
+		this.ctx.ui.requestRender();
 	}
 }