@oh-my-pi/pi-coding-agent 16.1.15 → 16.1.16

package/dist/types/web/search/providers/exa.d.ts

@@ -10,6 +10,8 @@ import { type AuthStorage, type FetchImpl } from "@oh-my-pi/pi-ai";
 import type { SearchResponse } from "../../../web/search/types";
 import type { SearchParams } from "./base";
 import { SearchProvider } from "./base";
+/** Reset Exa request pacing state for isolated provider tests. */
+export declare function resetExaSearchThrottleForTest(): void;
 type ExaSearchType = "neural" | "fast" | "auto" | "deep";
 type ExaSearchParamType = ExaSearchType | "keyword";
 export interface ExaSearchParams {

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "16.1.15",
+	"version": "16.1.16",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://omp.sh",
 	"author": "Can Boluk",
@@ -48,17 +48,17 @@
 		"@agentclientprotocol/sdk": "0.25.0",
 		"@babel/parser": "^7.29.7",
 		"@mozilla/readability": "^0.6.0",
-		"@oh-my-pi/hashline": "16.1.15",
-		"@oh-my-pi/omp-stats": "16.1.15",
-		"@oh-my-pi/pi-agent-core": "16.1.15",
-		"@oh-my-pi/pi-ai": "16.1.15",
-		"@oh-my-pi/pi-catalog": "16.1.15",
-		"@oh-my-pi/pi-mnemopi": "16.1.15",
-		"@oh-my-pi/pi-natives": "16.1.15",
-		"@oh-my-pi/pi-tui": "16.1.15",
-		"@oh-my-pi/pi-utils": "16.1.15",
-		"@oh-my-pi/pi-wire": "16.1.15",
-		"@oh-my-pi/snapcompact": "16.1.15",
+		"@oh-my-pi/hashline": "16.1.16",
+		"@oh-my-pi/omp-stats": "16.1.16",
+		"@oh-my-pi/pi-agent-core": "16.1.16",
+		"@oh-my-pi/pi-ai": "16.1.16",
+		"@oh-my-pi/pi-catalog": "16.1.16",
+		"@oh-my-pi/pi-mnemopi": "16.1.16",
+		"@oh-my-pi/pi-natives": "16.1.16",
+		"@oh-my-pi/pi-tui": "16.1.16",
+		"@oh-my-pi/pi-utils": "16.1.16",
+		"@oh-my-pi/pi-wire": "16.1.16",
+		"@oh-my-pi/snapcompact": "16.1.16",
 		"@opentelemetry/api": "^1.9.1",
 		"@opentelemetry/context-async-hooks": "^2.7.1",
 		"@opentelemetry/exporter-trace-otlp-proto": "^0.218.0",

package/scripts/build-binary.ts

@@ -5,7 +5,15 @@ import * as path from "node:path";
 
 const packageDir = path.join(import.meta.dir, "..");
 const repoRoot = path.join(packageDir, "..", "..");
-const outputPath = path.join(packageDir, "dist", "omp");
+// Optional cross-compile target, e.g. CROSS_TARGET=linux-arm64 → bun build
+// --target=bun-linux-arm64, embeds the matching native, outputs dist/omp-<target>.
+const crossTarget = Bun.env.CROSS_TARGET || null;
+const [crossPlatform, crossArch] = crossTarget ? crossTarget.split("-") : [null, null];
+// x64 uses the baseline bun runtime so it runs under Rosetta / pre-AVX2 CPUs
+// (the modern bun-linux-x64 target SIGILLs under Apple-Silicon Rosetta).
+const bunTarget = crossTarget ? (crossTarget === "linux-x64" ? "bun-linux-x64-baseline" : `bun-${crossTarget}`) : null;
+const outName = crossTarget ? `omp-${crossTarget}` : "omp";
+const outputPath = path.join(packageDir, "dist", outName);
 
 // Transformers.js is an optional, native-heavy dependency that is never bundled
 // into the binary; the tiny-model worker `bun install`s it into a runtime cache
@@ -17,7 +25,7 @@ const transformersVersion = (
 ).version;
 
 function shouldAdhocSignDarwinBinary(): boolean {
-	return process.platform === "darwin";
+	return process.platform === "darwin" && !crossTarget;
 }
 
 async function runCommand(
@@ -43,7 +51,12 @@ async function main(): Promise<void> {
 	try {
 		await runCommand(["bun", "--cwd=../stats", "scripts/generate-client-bundle.ts", "--generate"]);
 		await runCommand(["bun", "scripts/generate-docs-index.ts", "--generate"]);
-		await runCommand(["bun", "--cwd=../natives", "run", "embed:native"]);
+		await runCommand(
+			["bun", "--cwd=../natives", "run", "embed:native"],
+			crossTarget
+				? { ...Bun.env, TARGET_PLATFORM: crossPlatform as string, TARGET_ARCH: crossArch as string }
+				: Bun.env,
+		);
 		await runCommand(["bun", "scripts/embed-mupdf-wasm.ts", "--generate"]);
 		try {
 			const buildEnv = shouldAdhocSignDarwinBinary() ? { ...Bun.env, BUN_NO_CODESIGN_MACHO_BINARY: "1" } : Bun.env;
@@ -52,6 +65,7 @@ async function main(): Promise<void> {
 					"bun",
 					"build",
 					"--compile",
+					...(bunTarget ? ["--target", bunTarget] : []),
 					"--no-compile-autoload-bunfig",
 					"--no-compile-autoload-dotenv",
 					"--no-compile-autoload-tsconfig",
@@ -85,7 +99,7 @@ async function main(): Promise<void> {
 					"./packages/coding-agent/src/extensibility/legacy-pi-ai-shim.ts",
 					"./packages/coding-agent/src/extensibility/legacy-pi-coding-agent-shim.ts",
 					"--outfile",
-					"packages/coding-agent/dist/omp",
+					`packages/coding-agent/dist/${outName}`,
 				],
 				buildEnv,
 				repoRoot,

package/src/cli/args.ts

@@ -1,10 +1,9 @@
 /**
  * CLI argument parsing and help display
  */
-import { type Effort, THINKING_EFFORTS } from "@oh-my-pi/pi-catalog/effort";
 import { APP_NAME, CONFIG_DIR_NAME, logger } from "@oh-my-pi/pi-utils";
 import chalk from "chalk";
-import { parseEffort } from "../thinking";
+import { CLI_THINKING_LEVELS, type ConfiguredThinkingLevel, parseCliThinkingLevel } from "../thinking";
 import { BUILTIN_TOOL_NAMES } from "../tools/builtin-names";
 import {
 	OPTIONAL_FLAGS,
@@ -32,7 +31,7 @@ export interface Args {
 	apiKey?: string;
 	systemPrompt?: string;
 	appendSystemPrompt?: string;
-	thinking?: Effort;
+	thinking?: ConfiguredThinkingLevel;
 	hideThinking?: boolean;
 	advisor?: boolean;
 	continue?: boolean;
@@ -89,9 +88,9 @@ export interface Args {
  */
 const PARSE_DEPS: ParseDeps = {
 	logger,
-	parseEffort,
+	parseThinking: parseCliThinkingLevel,
 	builtinToolNames: BUILTIN_TOOL_NAMES,
-	thinkingEfforts: THINKING_EFFORTS,
+	thinkingEfforts: CLI_THINKING_LEVELS,
 };
 
 export function parseArgs(inputArgs: string[], extensionFlags?: Map<string, { type: "boolean" | "string" }>): Args {

package/src/cli/flag-tables.ts

@@ -30,7 +30,7 @@
  * real implementations at the dispatch site.
  */
 
-import type { Effort } from "@oh-my-pi/pi-ai";
+import type { ConfiguredThinkingLevel } from "../thinking";
 import type { Args } from "./args";
 
 /**
@@ -44,7 +44,7 @@ import type { Args } from "./args";
  */
 export interface ParseDeps {
 	logger: { warn: (message: string, meta?: Record<string, unknown>) => void };
-	parseEffort: (value: string | null | undefined) => Effort | undefined;
+	parseThinking: (value: string | null | undefined) => ConfiguredThinkingLevel | undefined;
 	builtinToolNames: readonly string[];
 	thinkingEfforts: readonly string[];
 }
@@ -165,7 +165,7 @@ export const STRING_SETTERS: Record<string, StringSetter> = {
 		result.tools = valid;
 	},
 	"--thinking": (result, value, deps) => {
-		const thinking = deps.parseEffort(value);
+		const thinking = deps.parseThinking(value);
 		if (thinking !== undefined) {
 			result.thinking = thinking;
 		} else {

package/src/cli/gallery-fixtures/interaction.ts

@@ -5,17 +5,14 @@ export const interactionFixtures: Record<string, GalleryFixture> = {
 	todo: {
 		label: "Todo",
 		streamingArgs: {
-			ops: [{ op: "init", list: [{ phase: "Foundation", items: ["Scaffold crate"] }] }],
+			op: "init",
+			list: [{ phase: "Foundation", items: ["Scaffold crate"] }],
 		},
 		args: {
-			ops: [
-				{
-					op: "init",
-					list: [
-						{ phase: "Foundation", items: ["Scaffold crate", "Wire workspace"] },
-						{ phase: "Auth", items: ["Port credential store", "Wire OAuth providers"] },
-					],
-				},
+			op: "init",
+			list: [
+				{ phase: "Foundation", items: ["Scaffold crate", "Wire workspace"] },
+				{ phase: "Auth", items: ["Port credential store", "Wire OAuth providers"] },
 			],
 		},
 		result: {

package/src/cli/gallery-fixtures/shell.ts

@@ -59,31 +59,23 @@ export const shellFixtures: Record<string, GalleryFixture> = {
 	eval: {
 		label: "Eval",
 		streamingArgs: {
-			cells: [
-				{
-					language: "py",
-					code: 'import json\nfrom pathlib import Path\n\ndata = json.loads(Path("package.js',
-					title: "load config",
-				},
-			],
+			language: "py",
+			code: 'import json\nfrom pathlib import Path\n\ndata = json.loads(Path("package.js',
+			title: "load config",
 		},
 		args: {
-			cells: [
-				{
-					language: "py",
-					title: "load config",
-					code: [
-						"import json",
-						"from pathlib import Path",
-						"",
-						'data = json.loads(Path("package.json").read_text())',
-						'deps = data.get("dependencies", {})',
-						'print(f"{data[\\"name\\"]} v{data[\\"version\\"]}")',
-						'print(f"{len(deps)} dependencies")',
-						"display(sorted(deps)[:3])",
-					].join("\n"),
-				},
-			],
+			language: "py",
+			title: "load config",
+			code: [
+				"import json",
+				"from pathlib import Path",
+				"",
+				'data = json.loads(Path("package.json").read_text())',
+				'deps = data.get("dependencies", {})',
+				'print(f"{data[\\"name\\"]} v{data[\\"version\\"]}")',
+				'print(f"{len(deps)} dependencies")',
+				"display(sorted(deps)[:3])",
+			].join("\n"),
 		},
 		result: {
 			content: [

package/src/cli/session-picker.ts

@@ -8,8 +8,10 @@ import { FileSessionStorage } from "../session/session-storage";
 
 /**
  * Show the TUI session selector and return the selected session, or null if
- * cancelled. Tab toggles between current-folder and all-projects scope; the
- * all-projects list is loaded lazily via `SessionManager.listAll`.
+ * cancelled. Rendered as a fullscreen overlay on the terminal's alternate
+ * screen, so the list scrolls and rows are clickable with the mouse. Tab
+ * toggles between current-folder and all-projects scope; the all-projects list
+ * is loaded lazily via `SessionManager.listAll`.
  */
 export async function selectSession(
 	sessions: SessionInfo[],
@@ -65,6 +67,7 @@ export async function selectSession(
 				loadAllSessions: () => SessionManager.listAll(storage),
 				allSessions: options?.allSessions,
 				getTerminalRows: () => ui.terminal.rows,
+				fillHeight: true,
 			},
 		);
 		return selector;
@@ -72,7 +75,18 @@ export async function selectSession(
 
 	const selector = showSelector();
 	selector.setOnRequestRender(() => ui.requestRender());
-	ui.addChild(selector);
+	// Present as a fullscreen overlay so the picker borrows the terminal's
+	// alternate screen buffer (vim/less idiom): the list scrolls and rows are
+	// clickable via the mouse tracking the overlay enables for its lifetime.
+	// Anchored top-left at full size so a mouse row maps directly to a rendered
+	// line (the overlay paints from screen row 0).
+	ui.showOverlay(selector, {
+		anchor: "top-left",
+		width: "100%",
+		maxHeight: "100%",
+		margin: 0,
+		fullscreen: true,
+	});
 	ui.setFocus(selector);
 	ui.start();
 	return promise;

package/src/commands/launch.ts

@@ -2,12 +2,12 @@
  * Root command for the coding agent CLI.
  */
 
-import { THINKING_EFFORTS } from "@oh-my-pi/pi-catalog/effort";
 import { APP_NAME } from "@oh-my-pi/pi-utils";
 import { Args, Command, Flags } from "@oh-my-pi/pi-utils/cli";
 import { parseArgs } from "../cli/args";
 import { runRootCommand } from "../main";
 import { prepareAcpTerminalAuthArgs } from "../modes/acp/terminal-auth";
+import { CLI_THINKING_LEVELS } from "../thinking";
 
 export default class Index extends Command {
 	static description = "AI coding assistant";
@@ -100,8 +100,8 @@ export default class Index extends Command {
 			description: "Comma-separated list of tools to enable (default: all)",
 		}),
 		thinking: Flags.string({
-			description: `Set thinking level: ${THINKING_EFFORTS.join(", ")}`,
-			options: [...THINKING_EFFORTS],
+			description: `Set thinking level: ${CLI_THINKING_LEVELS.join(", ")}`,
+			options: [...CLI_THINKING_LEVELS],
 		}),
 		"hide-thinking": Flags.boolean({
 			description: "Hide thinking blocks in TUI output (display only, does not disable model thinking)",

package/src/config/settings-schema.ts

@@ -3572,7 +3572,7 @@ export const SETTINGS_SCHEMA = {
 			group: "Discovery & MCP",
 			label: "Essential Tools Override",
 			description:
-				"Override the always-loaded built-in tools (default: read, bash, edit). Leave empty to use defaults.",
+				"Override the always-loaded built-in tools (default: read, bash, edit, write, find, eval). Leave empty to use defaults.",
 		},
 	},
 
@@ -4479,6 +4479,17 @@ export const SETTINGS_SCHEMA = {
 		},
 	},
 
+	"exa.searchDelayMs": {
+		type: "number",
+		default: 1_000,
+		ui: {
+			tab: "providers",
+			group: "Services",
+			label: "Exa Search Delay",
+			description: "Minimum delay between Exa web search requests in milliseconds; set 0 to disable pacing",
+		},
+	},
+
 	"exa.enableResearcher": {
 		type: "boolean",
 		default: false,
@@ -4785,6 +4796,7 @@ export interface TtsrSettings {
 export interface ExaSettings {
 	enabled: boolean;
 	enableSearch: boolean;
+	searchDelayMs: number;
 	enableResearcher: boolean;
 	enableWebsets: boolean;
 }

package/src/edit/renderer.ts

@@ -22,6 +22,7 @@ import {
 	invalidateRenderedStringCache,
 	type LspBatchRequest,
 	PREVIEW_LIMITS,
+	previewWindowRows,
 	type RenderedStringCache,
 	replaceTabs,
 	shortenPath,
@@ -340,6 +341,7 @@ function renderPlainTextPreview(text: string, uiTheme: Theme, _filePath?: string
 function formatStreamingDiff(
 	diff: string,
 	rawPath: string,
+	width: number,
 	uiTheme: Theme,
 	expanded: boolean,
 	label = "streaming",
@@ -347,15 +349,32 @@ function formatStreamingDiff(
 	cache?: RenderedStringCache,
 ): string {
 	if (!diff) return "";
-	let text = cachedRenderedString(cache, uiTheme, expanded, rawPath, diff, () => {
-		// Collapsed uses a "Cursor" tail window: pin the last
-		// EDIT_STREAMING_PREVIEW_LINES rows to the bottom so freshly streamed changes
-		// stay on screen. The whole-file diff is recomputed on every streamed chunk
-		// and its Myers alignment is not monotonic in payload length, so a hunk-aware
-		// window stutters as rows move between hunks. Expanded deliberately lifts that
-		// cap for the approval-time full view.
+	// Clamp the collapsed tail to the viewport so a tall or fast-growing diff
+	// cannot outgrow the live window. Otherwise its mutating tail scrolls above
+	// the native-scrollback commit boundary and the engine re-commits a fresh
+	// snapshot every streamed frame, stacking duplicate "… more lines above"
+	// previews in history. The budget is VISUAL rows (a long wrapped line counts
+	// for more than one) at the framed block's inner width (border only —
+	// contentPaddingLeft is 0); only the visible suffix is syntax-colored, so the
+	// cheap raw-line wrap walk keeps the per-chunk cost bounded. innerWidth/budget
+	// are in the cache salt so a resize re-slices.
+	const innerWidth = Math.max(1, width - 2);
+	const budget = expanded ? Number.POSITIVE_INFINITY : Math.min(EDIT_STREAMING_PREVIEW_LINES, previewWindowRows());
+	let text = cachedRenderedString(cache, uiTheme, expanded, `${rawPath}:${innerWidth}:${budget}`, diff, () => {
+		// "Cursor" tail window: pin the last rows to the bottom so freshly streamed
+		// changes stay on screen. The whole-file diff is recomputed every chunk and
+		// its Myers alignment is not monotonic in payload length, so a hunk-aware
+		// window stutters as rows move between hunks. Expanded lifts the cap.
 		const allLines = diff.replace(/\n+$/u, "").split("\n");
-		const hiddenLines = expanded ? 0 : Math.max(0, allLines.length - EDIT_STREAMING_PREVIEW_LINES);
+		let visualUsed = 0;
+		let cut = allLines.length;
+		for (let i = allLines.length - 1; i >= 0; i--) {
+			const lineRows = Math.max(1, wrapTextWithAnsi(replaceTabs(allLines[i]!), innerWidth).length);
+			if (visualUsed + lineRows > budget && visualUsed > 0) break;
+			visualUsed += lineRows;
+			cut = i;
+		}
+		const hiddenLines = cut;
 		const visible = hiddenLines > 0 ? allLines.slice(hiddenLines) : allLines;
 		let rendered = "\n\n";
 		if (hiddenLines > 0) {
@@ -384,6 +403,7 @@ function formatStreamingDiff(
 
 function formatMultiFileStreamingDiff(
 	previews: PerFileDiffPreview[],
+	width: number,
 	uiTheme: Theme,
 	expanded: boolean,
 	spinnerFrame?: number,
@@ -405,7 +425,7 @@ function formatMultiFileStreamingDiff(
 			const isLast = index === previews.length - 1;
 			const cache = previewCacheAt(caches, index);
 			parts.push(
-				`${header}${formatStreamingDiff(preview.diff, preview.path, uiTheme, expanded, "preview", isLast ? spinnerFrame : undefined, cache)}`,
+				`${header}${formatStreamingDiff(preview.diff, preview.path, width, uiTheme, expanded, "preview", isLast ? spinnerFrame : undefined, cache)}`,
 			);
 		}
 	}
@@ -415,6 +435,7 @@ function formatMultiFileStreamingDiff(
 function getCallPreview(
 	args: EditRenderArgs,
 	rawPath: string,
+	width: number,
 	uiTheme: Theme,
 	renderContext: EditRenderContext | undefined,
 	expanded: boolean,
@@ -423,14 +444,14 @@ function getCallPreview(
 ): string {
 	const multi = renderContext?.perFileDiffPreview;
 	if (multi && multi.length > 1 && multi.some(p => p.diff || p.error)) {
-		return formatMultiFileStreamingDiff(multi, uiTheme, expanded, spinnerFrame, caches);
+		return formatMultiFileStreamingDiff(multi, width, uiTheme, expanded, spinnerFrame, caches);
 	}
 	const cache = previewCacheAt(caches, 0);
 	if (args.previewDiff) {
-		return formatStreamingDiff(args.previewDiff, rawPath, uiTheme, expanded, "preview", spinnerFrame, cache);
+		return formatStreamingDiff(args.previewDiff, rawPath, width, uiTheme, expanded, "preview", spinnerFrame, cache);
 	}
 	if (args.diff && args.op) {
-		return formatStreamingDiff(args.diff, rawPath, uiTheme, expanded, "streaming", spinnerFrame, cache);
+		return formatStreamingDiff(args.diff, rawPath, width, uiTheme, expanded, "streaming", spinnerFrame, cache);
 	}
 	if (args.diff) {
 		return renderPlainTextPreview(args.diff, uiTheme, rawPath);
@@ -628,6 +649,7 @@ export const editToolRenderer = {
 			let body = getCallPreview(
 				editArgs,
 				rawPath,
+				width,
 				uiTheme,
 				renderContext,
 				options.expanded,