@oh-my-pi/pi-coding-agent 12.7.6 → 12.8.0

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "12.7.6",
+	"version": "12.8.0",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"bin": {
@@ -84,12 +84,12 @@
 	},
 	"dependencies": {
 		"@mozilla/readability": "0.6.0",
-		"@oh-my-pi/omp-stats": "12.7.6",
-		"@oh-my-pi/pi-agent-core": "12.7.6",
-		"@oh-my-pi/pi-ai": "12.7.6",
-		"@oh-my-pi/pi-natives": "12.7.6",
-		"@oh-my-pi/pi-tui": "12.7.6",
-		"@oh-my-pi/pi-utils": "12.7.6",
+		"@oh-my-pi/omp-stats": "12.8.0",
+		"@oh-my-pi/pi-agent-core": "12.8.0",
+		"@oh-my-pi/pi-ai": "12.8.0",
+		"@oh-my-pi/pi-natives": "12.8.0",
+		"@oh-my-pi/pi-tui": "12.8.0",
+		"@oh-my-pi/pi-utils": "12.8.0",
 		"@sinclair/typebox": "^0.34.48",
 		"@xterm/headless": "^6.0.0",
 		"ajv": "^8.18.0",

package/src/cli/args.ts

@@ -213,6 +213,7 @@ export function getExtraHelpText(): string {
 
   ${chalk.dim("# Search & Tools")}
   EXA_API_KEY                - Exa web search
+  BRAVE_API_KEY              - Brave web search
   PERPLEXITY_API_KEY         - Perplexity web search
   ANTHROPIC_SEARCH_API_KEY   - Anthropic search provider
 

package/src/cli/update-cli.ts

@@ -80,7 +80,7 @@ async function getLatestRelease(): Promise<ReleaseInfo> {
 	return {
 		tag,
 		version,
-		assets: [makeAsset(getBinaryName()), makeAsset(getNativeAddonName())],
+		assets: [makeAsset(getBinaryName()), ...getNativeAddonNames().map(makeAsset)],
 	};
 }
 
@@ -143,13 +143,11 @@ function getBinaryName(): string {
 }
 
 /**
- * Get the appropriate native addon name for this platform.
- * Uses process.platform directly (linux, darwin, win32).
+ * Get native addon names for this platform, ordered by preference.
  */
-function getNativeAddonName(): string {
+function getNativeAddonNames(): string[] {
 	const platform = process.platform;
 	const arch = process.arch;
-
 	if (!["linux", "darwin", "win32"].includes(platform)) {
 		throw new Error(`Unsupported platform: ${platform}`);
 	}
@@ -157,7 +155,12 @@ function getNativeAddonName(): string {
 		throw new Error(`Unsupported architecture: ${arch}`);
 	}
 
-	return `pi_natives.${platform}-${arch}.node`;
+	const baseName = `pi_natives.${platform}-${arch}.node`;
+	if (arch !== "x64") {
+		return [baseName];
+	}
+
+	return [`pi_natives.${platform}-${arch}-modern.node`, `pi_natives.${platform}-${arch}-baseline.node`];
 }
 
 /**
@@ -197,50 +200,58 @@ async function updateViaBun(expectedVersion: string): Promise<void> {
  */
 async function updateViaBinary(release: ReleaseInfo): Promise<void> {
 	const binaryName = getBinaryName();
-	const nativeAddonName = getNativeAddonName();
-
+	const nativeAddonNames = getNativeAddonNames();
 	const asset = release.assets.find(a => a.name === binaryName);
-	const nativeAsset = release.assets.find(a => a.name === nativeAddonName);
-
 	if (!asset) {
 		throw new Error(`No binary found for ${binaryName}`);
 	}
-	if (!nativeAsset) {
-		throw new Error(`No native addon found for ${nativeAddonName}`);
-	}
-
 	const execPath = process.execPath;
 	const execDir = path.dirname(execPath);
 	const tempPath = `${execPath}.new`;
 	const backupPath = `${execPath}.bak`;
-	const nativePath = path.join(execDir, nativeAddonName);
-	const nativeTempPath = `${nativePath}.new`;
+	const nativeDownloads: Array<{ name: string; tempPath: string; finalPath: string }> = [];
+
+	const downloadNativeAsset = async (name: string, required: boolean): Promise<boolean> => {
+		const nativeAsset = release.assets.find(assetEntry => assetEntry.name === name);
+		if (!nativeAsset) {
+			if (required) throw new Error(`No native addon found for ${name}`);
+			return false;
+		}
+
+		console.log(chalk.dim(`Downloading ${name}…`));
+		try {
+			const nativeResponse = await fetch(nativeAsset.url, { redirect: "follow" });
+			if (!nativeResponse.ok || !nativeResponse.body) {
+				if (required) throw new Error(`Native addon download failed for ${name}: ${nativeResponse.statusText}`);
+				return false;
+			}
+
+			const nativeFinalPath = path.join(execDir, name);
+			const nativeTempPath = `${nativeFinalPath}.new`;
+			const nativeFileStream = fs.createWriteStream(nativeTempPath, { mode: 0o755 });
+			await pipeline(nativeResponse.body, nativeFileStream);
+			nativeDownloads.push({ name, tempPath: nativeTempPath, finalPath: nativeFinalPath });
+			return true;
+		} catch (err) {
+			if (required) throw err;
+			return false;
+		}
+	};
 
 	console.log(chalk.dim(`Downloading ${binaryName}…`));
 
-	// Download to temp file
+	// Download binary to temp file
 	const response = await fetch(asset.url, { redirect: "follow" });
 	if (!response.ok || !response.body) {
 		throw new Error(`Download failed: ${response.statusText}`);
 	}
-
 	const fileStream = fs.createWriteStream(tempPath, { mode: 0o755 });
 	await pipeline(response.body, fileStream);
-
-	// Download native addon
-	console.log(chalk.dim(`Downloading ${nativeAddonName}…`));
-
-	const nativeResponse = await fetch(nativeAsset.url, { redirect: "follow" });
-	if (!nativeResponse.ok || !nativeResponse.body) {
-		throw new Error(`Native addon download failed: ${nativeResponse.statusText}`);
+	for (const nativeAddonName of nativeAddonNames) {
+		await downloadNativeAsset(nativeAddonName, true);
 	}
-
-	const nativeFileStream = fs.createWriteStream(nativeTempPath, { mode: 0o755 });
-	await pipeline(nativeResponse.body, nativeFileStream);
-
 	// Replace current binary
 	console.log(chalk.dim("Installing update..."));
-
 	try {
 		try {
 			await fs.promises.unlink(backupPath);
@@ -251,10 +262,11 @@ async function updateViaBinary(release: ReleaseInfo): Promise<void> {
 		await fs.promises.rename(tempPath, execPath);
 		await fs.promises.unlink(backupPath);
 
-		// Replace native addon (no backup needed, just overwrite)
-		await fs.promises.rename(nativeTempPath, nativePath);
-
+		for (const nativeDownload of nativeDownloads) {
+			await fs.promises.rename(nativeDownload.tempPath, nativeDownload.finalPath);
+		}
 		console.log(chalk.green(`\n${theme.status.success} Updated to ${release.version}`));
+		console.log(chalk.dim(`Installed ${nativeDownloads.length} native addon file(s)`));
 		console.log(chalk.dim(`Restart ${APP_NAME} to use the new version`));
 	} catch (err) {
 		if (fs.existsSync(backupPath) && !fs.existsSync(execPath)) {
@@ -263,8 +275,10 @@ async function updateViaBinary(release: ReleaseInfo): Promise<void> {
 		if (fs.existsSync(tempPath)) {
 			await fs.promises.unlink(tempPath);
 		}
-		if (fs.existsSync(nativeTempPath)) {
-			await fs.promises.unlink(nativeTempPath);
+		for (const nativeDownload of nativeDownloads) {
+			if (fs.existsSync(nativeDownload.tempPath)) {
+				await fs.promises.unlink(nativeDownload.tempPath);
+			}
 		}
 		throw err;
 	}

package/src/cli/web-search-cli.ts

@@ -24,6 +24,7 @@ const PROVIDERS: Array<SearchProviderId | "auto"> = [
 	"anthropic",
 	"perplexity",
 	"exa",
+	"brave",
 	"jina",
 	"zai",
 	"gemini",
@@ -131,14 +132,14 @@ ${chalk.bold("Arguments:")}
 
 ${chalk.bold("Options:")}
   --provider <name>   Provider: ${PROVIDERS.join(", ")}
-  --recency <value>   Recency filter (Perplexity only): ${RECENCY_OPTIONS.join(", ")}
+  --recency <value>   Recency filter (Brave/Perplexity): ${RECENCY_OPTIONS.join(", ")}
   -l, --limit <n>     Max results to return
   --compact           Render condensed output
   -h, --help          Show this help
 
 ${chalk.bold("Examples:")}
   ${APP_NAME} q --provider=exa "what's the color of the sky"
-  ${APP_NAME} q --provider=perplexity --recency=week "latest TypeScript 5.7 changes"
+  ${APP_NAME} q --provider=brave --recency=week "latest TypeScript 5.7 changes"
 `);
 }
 

package/src/commands/web-search.ts

@@ -10,6 +10,7 @@ const PROVIDERS: Array<SearchProviderId | "auto"> = [
 	"anthropic",
 	"perplexity",
 	"exa",
+	"brave",
 	"jina",
 	"zai",
 	"gemini",

package/src/config/model-registry.ts

@@ -93,6 +93,7 @@ const ModelDefinitionSchema = Type.Object({
 	maxTokens: Type.Optional(Type.Number()),
 	headers: Type.Optional(Type.Record(Type.String(), Type.String())),
 	compat: Type.Optional(OpenAICompatSchema),
+	contextPromotionTarget: Type.Optional(Type.String({ minLength: 1 })),
 });
 
 // Schema for per-model overrides (all fields optional, merged with built-in model)
@@ -112,6 +113,7 @@ const ModelOverrideSchema = Type.Object({
 	maxTokens: Type.Optional(Type.Number()),
 	headers: Type.Optional(Type.Record(Type.String(), Type.String())),
 	compat: Type.Optional(OpenAICompatSchema),
+	contextPromotionTarget: Type.Optional(Type.String({ minLength: 1 })),
 });
 
 type ModelOverride = Static<typeof ModelOverrideSchema>;
@@ -274,6 +276,7 @@ function applyModelOverride(model: Model<Api>, override: ModelOverride): Model<A
 	if (override.input !== undefined) result.input = override.input as ("text" | "image")[];
 	if (override.contextWindow !== undefined) result.contextWindow = override.contextWindow;
 	if (override.maxTokens !== undefined) result.maxTokens = override.maxTokens;
+	if (override.contextPromotionTarget !== undefined) result.contextPromotionTarget = override.contextPromotionTarget;
 	if (override.cost) {
 		result.cost = {
 			input: override.cost.input ?? model.cost.input,
@@ -652,6 +655,7 @@ export class ModelRegistry {
 					maxTokens: modelDef.maxTokens ?? 16384,
 					headers,
 					compat: modelDef.compat,
+					contextPromotionTarget: modelDef.contextPromotionTarget,
 				} as Model<Api>);
 			}
 		}
@@ -816,6 +820,7 @@ export class ModelRegistry {
 					maxTokens: modelDef.maxTokens,
 					headers,
 					compat: modelDef.compat,
+					contextPromotionTarget: modelDef.contextPromotionTarget,
 				} as Model<Api>);
 			}
 
@@ -872,5 +877,6 @@ export interface ProviderConfigInput {
 		maxTokens: number;
 		headers?: Record<string, string>;
 		compat?: Model<Api>["compat"];
+		contextPromotionTarget?: string;
 	}>;
 }

package/src/config/settings-schema.ts

@@ -299,6 +299,24 @@ export const SETTINGS_SCHEMA = {
 	disabledProviders: { type: "array", default: [] as string[] },
 	disabledExtensions: { type: "array", default: [] as string[] },
 	modelRoles: { type: "record", default: {} as Record<string, string> },
+	"contextPromotion.enabled": {
+		type: "boolean",
+		default: true,
+		ui: {
+			tab: "agent",
+			label: "Auto-promote context",
+			description: "Promote to a larger-context model on context overflow instead of compacting",
+		},
+	},
+
+	// ─────────────────────────────────────────────────────────────────────────
+	// Secrets settings
+	// ─────────────────────────────────────────────────────────────────────────
+	"secrets.enabled": {
+		type: "boolean",
+		default: false,
+		ui: { tab: "config", label: "Hide secrets", description: "Obfuscate secrets before sending to AI providers" },
+	},
 
 	// ─────────────────────────────────────────────────────────────────────────
 	// Compaction settings
@@ -409,7 +427,7 @@ export const SETTINGS_SCHEMA = {
 	},
 	"grep.contextBefore": {
 		type: "number",
-		default: 1,
+		default: 0,
 		ui: {
 			tab: "tools",
 			label: "Grep context before",
@@ -419,7 +437,7 @@ export const SETTINGS_SCHEMA = {
 	},
 	"grep.contextAfter": {
 		type: "number",
-		default: 3,
+		default: 0,
 		ui: {
 			tab: "tools",
 			label: "Grep context after",
@@ -617,7 +635,7 @@ export const SETTINGS_SCHEMA = {
 	// ─────────────────────────────────────────────────────────────────────────
 	"providers.webSearch": {
 		type: "enum",
-		values: ["auto", "exa", "jina", "zai", "perplexity", "anthropic"] as const,
+		values: ["auto", "exa", "brave", "jina", "zai", "perplexity", "anthropic"] as const,
 		default: "auto",
 		ui: { tab: "services", label: "Web search provider", description: "Provider for web search tool", submenu: true },
 	},
@@ -1027,6 +1045,9 @@ export interface CompactionSettings {
 	remoteEndpoint: string | undefined;
 }
 
+export interface ContextPromotionSettings {
+	enabled: boolean;
+}
 export interface RetrySettings {
 	enabled: boolean;
 	maxRetries: number;
@@ -1134,6 +1155,7 @@ export interface BashInterceptorRule {
 /** Map group prefix -> typed settings interface */
 export interface GroupTypeMap {
 	compaction: CompactionSettings;
+	contextPromotion: ContextPromotionSettings;
 	retry: RetrySettings;
 	memories: MemoriesSettings;
 	branchSummary: BranchSummarySettings;

package/src/config/settings.ts

@@ -39,6 +39,7 @@ export type {
 	BranchSummarySettings,
 	CommitSettings,
 	CompactionSettings,
+	ContextPromotionSettings,
 	ExaSettings,
 	GroupPrefix,
 	GroupTypeMap,

package/src/extensibility/extensions/wrapper.ts

@@ -18,11 +18,31 @@ export class RegisteredToolAdapter implements AgentTool<any, any, any> {
 	declare parameters: any;
 	declare label: string;
 
+	renderCall?: (args: any, theme: any) => any;
+	renderResult?: (result: any, options: any, theme: any, args?: any) => any;
+
 	constructor(
 		private registeredTool: RegisteredTool,
 		private runner: ExtensionRunner,
 	) {
 		applyToolProxy(registeredTool.definition, this);
+
+		// Only define render methods when the underlying definition provides them.
+		// If these exist unconditionally on the prototype, ToolExecutionComponent
+		// enters the custom-renderer path, gets undefined back, and silently
+		// discards tool result text (extensions without renderers show blank).
+		if (registeredTool.definition.renderCall) {
+			this.renderCall = (args: any, theme: any) => registeredTool.definition.renderCall!(args, theme as Theme);
+		}
+		if (registeredTool.definition.renderResult) {
+			this.renderResult = (result: any, options: any, theme: any, args?: any) =>
+				registeredTool.definition.renderResult!(
+					result,
+					{ expanded: options.expanded, isPartial: options.isPartial, spinnerFrame: options.spinnerFrame },
+					theme as Theme,
+					args,
+				);
+		}
 	}
 
 	async execute(
@@ -34,19 +54,6 @@ export class RegisteredToolAdapter implements AgentTool<any, any, any> {
 	) {
 		return this.registeredTool.definition.execute(toolCallId, params, signal, onUpdate, this.runner.createContext());
 	}
-
-	renderCall?(args: any, theme: any) {
-		return this.registeredTool.definition.renderCall?.(args, theme as Theme);
-	}
-
-	renderResult?(result: any, options: any, theme: any, args?: any) {
-		return this.registeredTool.definition.renderResult?.(
-			result,
-			{ expanded: options.expanded, isPartial: options.isPartial, spinnerFrame: options.spinnerFrame },
-			theme as Theme,
-			args,
-		);
-	}
 }
 
 /**

package/src/extensibility/slash-commands.ts

@@ -1,4 +1,15 @@
 import type { AutocompleteItem } from "@oh-my-pi/pi-tui";
+import { slashCommandCapability } from "../capability/slash-command";
+import { renderPromptTemplate } from "../config/prompt-templates";
+import type { SlashCommand } from "../discovery";
+import { loadCapability } from "../discovery";
+import {
+	BUILTIN_SLASH_COMMAND_DEFS,
+	type BuiltinSlashCommand,
+	type SubcommandDef,
+} from "../slash-commands/builtin-registry";
+import { EMBEDDED_COMMAND_TEMPLATES } from "../task/commands";
+import { parseFrontmatter } from "../utils/frontmatter";
 
 export type SlashCommandSource = "extension" | "prompt" | "skill";
 
@@ -12,22 +23,7 @@ export interface SlashCommandInfo {
 	path?: string;
 }
 
-/** Declarative subcommand definition for commands like /mcp */
-export interface SubcommandDef {
-	name: string;
-	description: string;
-	/** Usage hint shown as dim ghost text, e.g. "<name> [--scope project|user]" */
-	usage?: string;
-}
-
-export interface BuiltinSlashCommand {
-	name: string;
-	description: string;
-	/** Subcommands for dropdown completion (e.g. /mcp add, /mcp list) */
-	subcommands?: SubcommandDef[];
-	/** Static inline hint when command takes a simple argument (no subcommands) */
-	inlineHint?: string;
-}
+export type { BuiltinSlashCommand, SubcommandDef } from "../slash-commands/builtin-registry";
 
 /**
  * Build getArgumentCompletions from declarative subcommand definitions.
@@ -93,74 +89,6 @@ function buildStaticInlineHint(hint: string): (argumentText: string) => string |
 	return (argumentText: string) => (argumentText.trim().length === 0 ? hint : null);
 }
 
-const BUILTIN_SLASH_COMMAND_DEFS: ReadonlyArray<BuiltinSlashCommand> = [
-	{ name: "settings", description: "Open settings menu" },
-	{ name: "plan", description: "Toggle plan mode (agent plans before executing)" },
-	{ name: "model", description: "Select model (opens selector UI)" },
-	{ name: "export", description: "Export session to HTML file", inlineHint: "[path]" },
-	{ name: "dump", description: "Copy session transcript to clipboard" },
-	{ name: "share", description: "Share session as a secret GitHub gist" },
-	{
-		name: "browser",
-		description: "Toggle browser headless vs visible mode",
-		subcommands: [
-			{ name: "headless", description: "Switch to headless mode" },
-			{ name: "visible", description: "Switch to visible mode" },
-		],
-	},
-	{ name: "copy", description: "Copy last agent message to clipboard" },
-	{ name: "session", description: "Show session info and stats" },
-	{ name: "usage", description: "Show provider usage and limits" },
-	{ name: "changelog", description: "Show changelog entries" },
-	{ name: "hotkeys", description: "Show all keyboard shortcuts" },
-	{ name: "extensions", description: "Open Extension Control Center dashboard" },
-	{ name: "branch", description: "Create a new branch from a previous message" },
-	{ name: "fork", description: "Create a new fork from a previous message" },
-	{ name: "tree", description: "Navigate session tree (switch branches)" },
-	{ name: "login", description: "Login with OAuth provider" },
-	{ name: "logout", description: "Logout from OAuth provider" },
-	{
-		name: "mcp",
-		description: "Manage MCP servers (add, list, remove, test)",
-		subcommands: [
-			{
-				name: "add",
-				description: "Add a new MCP server",
-				usage: "<name> [--scope project|user] [--url <url>] [-- <command...>]",
-			},
-			{ name: "list", description: "List all configured MCP servers" },
-			{ name: "remove", description: "Remove an MCP server", usage: "<name> [--scope project|user]" },
-			{ name: "test", description: "Test connection to a server", usage: "<name>" },
-			{ name: "reauth", description: "Reauthorize OAuth for a server", usage: "<name>" },
-			{ name: "unauth", description: "Remove OAuth auth from a server", usage: "<name>" },
-			{ name: "enable", description: "Enable an MCP server", usage: "<name>" },
-			{ name: "disable", description: "Disable an MCP server", usage: "<name>" },
-			{ name: "reload", description: "Force reload MCP runtime tools" },
-			{ name: "help", description: "Show help message" },
-		],
-	},
-	{ name: "new", description: "Start a new session" },
-	{ name: "compact", description: "Manually compact the session context", inlineHint: "[focus instructions]" },
-	{ name: "handoff", description: "Hand off session context to a new session", inlineHint: "[focus instructions]" },
-	{ name: "resume", description: "Resume a different session" },
-	{ name: "background", description: "Detach UI and continue running in background" },
-	{ name: "debug", description: "Write debug log (TUI state and messages)" },
-	{
-		name: "memory",
-		description: "Inspect and operate memory maintenance",
-		subcommands: [
-			{ name: "view", description: "Show current memory injection payload" },
-			{ name: "clear", description: "Clear persisted memory data and artifacts" },
-			{ name: "reset", description: "Alias for clear" },
-			{ name: "enqueue", description: "Enqueue memory consolidation maintenance" },
-			{ name: "rebuild", description: "Alias for enqueue" },
-		],
-	},
-	{ name: "move", description: "Move session to a different working directory", inlineHint: "<path>" },
-	{ name: "exit", description: "Exit the application" },
-	{ name: "quit", description: "Quit the application" },
-];
-
 /**
  * Materialized builtin slash commands with completion functions derived from
  * declarative subcommand/hint definitions.
@@ -187,13 +115,6 @@ export const BUILTIN_SLASH_COMMANDS: ReadonlyArray<
 	return cmd;
 });
 
-import { slashCommandCapability } from "../capability/slash-command";
-import { renderPromptTemplate } from "../config/prompt-templates";
-import type { SlashCommand } from "../discovery";
-import { loadCapability } from "../discovery";
-import { EMBEDDED_COMMAND_TEMPLATES } from "../task/commands";
-import { parseFrontmatter } from "../utils/frontmatter";
-
 /**
  * Represents a custom slash command loaded from a file
  */