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

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/model-registry.ts

@@ -992,6 +992,7 @@ export class ModelRegistry {
 				});
 				continue;
 			}
+			const configStale = this.#isDiscoveryCacheOlderThanModelsConfig(cache.updatedAt);
 			const models = this.#applyProviderModelOverrides(
 				providerConfig.provider,
 				this.#normalizeDiscoverableModels(
@@ -1007,7 +1008,7 @@ export class ModelRegistry {
 				provider: providerConfig.provider,
 				status: "cached",
 				optional: providerConfig.optional ?? false,
-				stale: !cache.fresh || !cache.authoritative,
+				stale: !cache.fresh || !cache.authoritative || configStale,
 				fetchedAt: cache.updatedAt,
 				models: models.map(model => model.id),
 			});
@@ -1259,12 +1260,19 @@ export class ModelRegistry {
 		return providerConfig.provider;
 	}
 
+	#isDiscoveryCacheOlderThanModelsConfig(cacheUpdatedAt: number): boolean {
+		const configMtime = this.#modelsConfigFile.getMtimeMs();
+		return configMtime !== null && cacheUpdatedAt < Math.floor(configMtime);
+	}
+
 	async #discoverProviderModels(
 		providerConfig: DiscoveryProviderConfig,
 		strategy: ModelRefreshStrategy,
 	): Promise<Model<Api>[]> {
 		const cacheProviderId = this.#configuredDiscoveryCacheProviderId(providerConfig);
 		const cached = readModelCache<Api>(cacheProviderId, 24 * 60 * 60 * 1000, Date.now, this.#cacheDbPath);
+		const cacheOlderThanConfig = cached !== null && this.#isDiscoveryCacheOlderThanModelsConfig(cached.updatedAt);
+		const effectiveStrategy = strategy === "online-if-uncached" && cacheOlderThanConfig ? "online" : strategy;
 		const requiresAuth = !this.#keylessProviders.has(providerConfig.provider);
 		if (requiresAuth) {
 			const apiKey = await this.#peekApiKeyForProvider(providerConfig.provider);
@@ -1311,12 +1319,12 @@ export class ModelRegistry {
 			cacheTtlMs: 24 * 60 * 60 * 1000,
 			fetchDynamicModels,
 		});
-		const result = await manager.refresh(strategy);
+		const result = await manager.refresh(effectiveStrategy);
 		const status = discoveryError
 			? result.models.length > 0
 				? "cached"
 				: "unavailable"
-			: strategy === "offline"
+			: effectiveStrategy === "offline"
 				? cached
 					? "cached"
 					: "idle"
@@ -1327,7 +1335,7 @@ export class ModelRegistry {
 			provider: providerId,
 			status,
 			optional: providerConfig.optional ?? false,
-			stale: result.stale || status === "cached",
+			stale: result.stale || status === "cached" || (cacheOlderThanConfig && status !== "ok"),
 			fetchedAt: discoveryError ? cached?.updatedAt : Date.now(),
 			models: result.models.map(model => model.id),
 			error: discoveryError,

package/src/config/settings-schema.ts

@@ -1592,6 +1592,30 @@ export const SETTINGS_SCHEMA = {
 		},
 	},
 
+	"share.store": {
+		type: "enum",
+		values: ["blob", "gist"] as const,
+		default: "blob",
+		ui: {
+			tab: "interaction",
+			group: "Collab",
+			label: "Share Store",
+			description: "Where /share uploads the encrypted session blob",
+			options: [
+				{
+					value: "blob",
+					label: "Encrypted Blob",
+					description: "Upload to the share server (no GitHub account needed; avoids gist API rate limits)",
+				},
+				{
+					value: "gist",
+					label: "GitHub Gist",
+					description: "Push to a secret gist (needs authenticated gh), falling back to the share server",
+				},
+			],
+		},
+	},
+
 	"share.redactSecrets": {
 		type: "boolean",
 		default: true,
@@ -3548,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.",
 		},
 	},
 
@@ -4455,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,
@@ -4761,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,