@oh-my-pi/pi-coding-agent 13.9.3 → 13.9.5

package/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 ## [Unreleased]
 
+## [13.9.4] - 2026-03-07
+### Added
+
+- Automatic detection of Ollama model capabilities including reasoning/thinking support and vision input via the `/api/show` endpoint
+- Improved Kagi API error handling with extraction of detailed error messages from JSON and plain text responses
+
+### Changed
+
+- Updated Kagi provider description to clarify requirement for Kagi Search API beta access
+
 ## [13.9.3] - 2026-03-07
 
 ### Breaking Changes
@@ -54,6 +64,7 @@
 - Fixed model registry to preserve explicit thinking configuration on runtime-registered models
 - Fixed usage limit reset time calculation to use absolute `resetsAt` timestamps instead of deprecated `resetInMs` field
 - Fixed compaction summary message creation to no longer be automatically added to chat during compaction (now handled by session manager)
+- Fixed Kagi web search errors to surface the provider's beta-access message and clarified that Kagi search requires Search API beta access
 
 ## [13.9.2] - 2026-03-05
 

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "13.9.3",
+	"version": "13.9.5",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://github.com/can1357/oh-my-pi",
 	"author": "Can Boluk",
@@ -41,12 +41,12 @@
 	},
 	"dependencies": {
 		"@mozilla/readability": "^0.6",
-		"@oh-my-pi/omp-stats": "13.9.3",
-		"@oh-my-pi/pi-agent-core": "13.9.3",
-		"@oh-my-pi/pi-ai": "13.9.3",
-		"@oh-my-pi/pi-natives": "13.9.3",
-		"@oh-my-pi/pi-tui": "13.9.3",
-		"@oh-my-pi/pi-utils": "13.9.3",
+		"@oh-my-pi/omp-stats": "13.9.5",
+		"@oh-my-pi/pi-agent-core": "13.9.5",
+		"@oh-my-pi/pi-ai": "13.9.5",
+		"@oh-my-pi/pi-natives": "13.9.5",
+		"@oh-my-pi/pi-tui": "13.9.5",
+		"@oh-my-pi/pi-utils": "13.9.5",
 		"@sinclair/typebox": "^0.34",
 		"@xterm/headless": "^6.0",
 		"ajv": "^8.18",

package/src/cli/args.ts

@@ -108,7 +108,10 @@ export function parseArgs(args: string[], extensionFlags?: Map<string, { type: "
 		} else if (arg === "--no-pty") {
 			result.noPty = true;
 		} else if (arg === "--tools" && i + 1 < args.length) {
-			const toolNames = args[++i].split(",").map(s => s.trim());
+			const toolNames = args[++i]
+				.split(",")
+				.map(s => s.trim().toLowerCase())
+				.filter(Boolean);
 			const validTools: string[] = [];
 			for (const name of toolNames) {
 				if (name in BUILTIN_TOOLS) {

package/src/config/model-registry.ts

@@ -23,7 +23,7 @@ import {
 	unregisterCustomApis,
 	unregisterOAuthProviders,
 } from "@oh-my-pi/pi-ai";
-import { logger } from "@oh-my-pi/pi-utils";
+import { isRecord, logger } from "@oh-my-pi/pi-utils";
 import { type Static, Type } from "@sinclair/typebox";
 import { type ConfigError, ConfigFile } from "../config";
 import type { ThemeColor } from "../modes/theme/theme";
@@ -862,12 +862,57 @@ export class ModelRegistry {
 		}
 	}
 
+	async #discoverOllamaModelMetadata(
+		endpoint: string,
+		modelId: string,
+		headers: Record<string, string> | undefined,
+	): Promise<{ reasoning: boolean; input: ("text" | "image")[] } | null> {
+		const showUrl = `${endpoint}/api/show`;
+		try {
+			const response = await fetch(showUrl, {
+				method: "POST",
+				headers: { ...(headers ?? {}), "Content-Type": "application/json" },
+				body: JSON.stringify({ model: modelId }),
+				signal: AbortSignal.timeout(1500),
+			});
+			if (!response.ok) {
+				return null;
+			}
+			const payload = (await response.json()) as unknown;
+			if (!isRecord(payload)) {
+				return null;
+			}
+			const capabilities = payload.capabilities;
+			if (Array.isArray(capabilities)) {
+				const normalized = new Set(
+					capabilities.flatMap(capability => (typeof capability === "string" ? [capability.toLowerCase()] : [])),
+				);
+				const supportsVision = normalized.has("vision") || normalized.has("image");
+				return {
+					reasoning: normalized.has("thinking"),
+					input: supportsVision ? ["text", "image"] : ["text"],
+				};
+			}
+			if (!isRecord(capabilities)) {
+				return null;
+			}
+			const supportsVision = capabilities.vision === true || capabilities.image === true;
+			return {
+				reasoning: capabilities.thinking === true,
+				input: supportsVision ? ["text", "image"] : ["text"],
+			};
+		} catch {
+			return null;
+		}
+	}
+
 	async #discoverOllamaModels(providerConfig: DiscoveryProviderConfig): Promise<Model<Api>[]> {
 		const endpoint = this.#normalizeOllamaBaseUrl(providerConfig.baseUrl);
 		const tagsUrl = `${endpoint}/api/tags`;
+		const headers = { ...(providerConfig.headers ?? {}) };
 		try {
 			const response = await fetch(tagsUrl, {
-				headers: { ...(providerConfig.headers ?? {}) },
+				headers,
 				signal: AbortSignal.timeout(3000),
 			});
 			if (!response.ok) {
@@ -879,27 +924,34 @@ export class ModelRegistry {
 				return [];
 			}
 			const payload = (await response.json()) as { models?: Array<{ name?: string; model?: string }> };
-			const models = payload.models ?? [];
-			const discovered: Model<Api>[] = [];
-			for (const item of models) {
+			const entries = (payload.models ?? []).flatMap(item => {
 				const id = item.model || item.name;
-				if (!id) continue;
-				discovered.push(
-					enrichModelThinking({
-						id,
-						name: item.name || id,
-						api: providerConfig.api,
-						provider: providerConfig.provider,
-						baseUrl: `${endpoint}/v1`,
-						reasoning: false,
-						input: ["text"],
-						cost: { input: 0, output: 0, cacheRead: 0, cacheWrite: 0 },
-						contextWindow: 128000,
-						maxTokens: 8192,
-						headers: providerConfig.headers,
-					}),
-				);
-			}
+				return id ? [{ id, name: item.name || id }] : [];
+			});
+			const metadataById = new Map(
+				await Promise.all(
+					entries.map(
+						async entry =>
+							[entry.id, await this.#discoverOllamaModelMetadata(endpoint, entry.id, headers)] as const,
+					),
+				),
+			);
+			const discovered = entries.map(entry => {
+				const metadata = metadataById.get(entry.id);
+				return enrichModelThinking({
+					id: entry.id,
+					name: entry.name,
+					api: providerConfig.api,
+					provider: providerConfig.provider,
+					baseUrl: `${endpoint}/v1`,
+					reasoning: metadata?.reasoning ?? false,
+					input: metadata?.input ?? ["text"],
+					cost: { input: 0, output: 0, cacheRead: 0, cacheWrite: 0 },
+					contextWindow: 128000,
+					maxTokens: 8192,
+					headers: providerConfig.headers,
+				});
+			});
 			return this.#applyProviderModelOverrides(providerConfig.provider, discovered);
 		} catch (error) {
 			logger.warn("model discovery failed for provider", {

package/src/discovery/helpers.ts

@@ -206,7 +206,7 @@ export function parseAgentFields(frontmatter: Record<string, unknown>): ParsedAg
 		return null;
 	}
 
-	let tools = parseArrayOrCSV(frontmatter.tools);
+	let tools = parseArrayOrCSV(frontmatter.tools)?.map(tool => tool.toLowerCase());
 
 	// Subagents with explicit tool lists always need submit_result
 	if (tools && !tools.includes("submit_result")) {

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

@@ -234,7 +234,7 @@ const OPTION_PROVIDERS: Partial<Record<SettingPath, OptionProvider>> = {
 		{ value: "perplexity", label: "Perplexity", description: "Requires PERPLEXITY_COOKIES or PERPLEXITY_API_KEY" },
 		{ value: "anthropic", label: "Anthropic", description: "Uses Anthropic web search" },
 		{ value: "zai", label: "Z.AI", description: "Calls Z.AI webSearchPrime MCP" },
-		{ value: "kagi", label: "Kagi", description: "Requires KAGI_API_KEY" },
+		{ value: "kagi", label: "Kagi", description: "Requires KAGI_API_KEY and Kagi Search API beta access" },
 		{ value: "synthetic", label: "Synthetic", description: "Requires SYNTHETIC_API_KEY" },
 	],
 	"providers.image": [

package/src/modes/components/status-line.ts

@@ -7,6 +7,7 @@ import { settings } from "../../config/settings";
 import type { StatusLinePreset, StatusLineSegmentId, StatusLineSeparatorStyle } from "../../config/settings-schema";
 import { theme } from "../../modes/theme/theme";
 import type { AgentSession } from "../../session/agent-session";
+import { calculatePromptTokens } from "../../session/compaction/compaction";
 import { findGitHeadPathSync, sanitizeStatusText } from "../shared";
 import {
 	canReuseCachedPr,
@@ -365,12 +366,7 @@ export class StatusLineComponent implements Component {
 			.reverse()
 			.find(m => m.role === "assistant" && m.stopReason !== "aborted") as AssistantMessage | undefined;
 
-		const contextTokens = lastAssistantMessage
-			? lastAssistantMessage.usage.input +
-				lastAssistantMessage.usage.output +
-				lastAssistantMessage.usage.cacheRead +
-				lastAssistantMessage.usage.cacheWrite
-			: 0;
+		const contextTokens = lastAssistantMessage ? calculatePromptTokens(lastAssistantMessage.usage) : 0;
 		const contextWindow = state.model?.contextWindow || 0;
 		const contextPercent = contextWindow > 0 ? (contextTokens / contextWindow) * 100 : 0;
 

package/src/sdk.ts

@@ -1251,9 +1251,9 @@ export async function createAgentSession(options: CreateAgentSessionOptions = {}
 	};
 
 	const toolNamesFromRegistry = Array.from(toolRegistry.keys());
-	const requestedToolNames = options.toolNames ?? toolNamesFromRegistry;
+	const requestedToolNames = options.toolNames?.map(name => name.toLowerCase()) ?? toolNamesFromRegistry;
 	const normalizedRequested = requestedToolNames.filter(name => toolRegistry.has(name));
-	const includeExitPlanMode = options.toolNames?.includes("exit_plan_mode") ?? false;
+	const includeExitPlanMode = requestedToolNames.includes("exit_plan_mode");
 	const initialToolNames = includeExitPlanMode
 		? normalizedRequested
 		: normalizedRequested.filter(name => name !== "exit_plan_mode");

package/src/session/agent-session.ts

@@ -108,6 +108,7 @@ import { extractFileMentions, generateFileMentionMessages } from "../utils/file-
 import {
 	type CompactionResult,
 	calculateContextTokens,
+	calculatePromptTokens,
 	collectEntriesForBranchSummary,
 	compact,
 	estimateTokens,
@@ -5076,7 +5077,7 @@ Be thorough - include exact file paths, function names, error messages, and tech
 			};
 		}
 
-		const usageTokens = calculateContextTokens(lastUsage);
+		const usageTokens = calculatePromptTokens(lastUsage);
 		let trailingTokens = 0;
 		for (let i = lastUsageIndex + 1; i < messages.length; i++) {
 			trailingTokens += estimateTokens(messages[i]);

package/src/tools/index.ts

@@ -227,7 +227,8 @@ function getPythonModeFromEnv(): PythonToolMode | null {
 export async function createTools(session: ToolSession, toolNames?: string[]): Promise<Tool[]> {
 	const includeSubmitResult = session.requireSubmitResultTool === true;
 	const enableLsp = session.enableLsp ?? true;
-	const requestedTools = toolNames && toolNames.length > 0 ? [...new Set(toolNames)] : undefined;
+	const requestedTools =
+		toolNames && toolNames.length > 0 ? [...new Set(toolNames.map(name => name.toLowerCase()))] : undefined;
 	if (requestedTools && !requestedTools.includes("exit_plan_mode")) {
 		requestedTools.push("exit_plan_mode");
 	}

package/src/web/kagi.ts

@@ -28,15 +28,23 @@ interface KagiRelatedSearchesObject {
 
 type KagiSearchObject = KagiSearchResultObject | KagiRelatedSearchesObject;
 
+interface KagiErrorEntry {
+	code?: number;
+	msg?: string;
+}
+
 interface KagiSearchResponse {
 	meta: {
 		id: string;
 	};
 	data: KagiSearchObject[];
-	error?: Array<{
-		code: number;
-		msg: string;
-	}>;
+	error?: KagiErrorEntry[];
+}
+
+interface KagiErrorResponse {
+	error?: string | KagiErrorEntry[];
+	message?: string;
+	detail?: string;
 }
 
 export class KagiApiError extends Error {
@@ -49,6 +57,54 @@ export class KagiApiError extends Error {
 	}
 }
 
+function extractKagiErrorMessage(payload: unknown): string | null {
+	if (!payload || typeof payload !== "object") return null;
+	const record = payload as Record<string, unknown>;
+
+	for (const value of [record.message, record.detail]) {
+		if (typeof value === "string" && value.trim().length > 0) {
+			return value.trim();
+		}
+	}
+
+	if (typeof record.error === "string" && record.error.trim().length > 0) {
+		return record.error.trim();
+	}
+
+	if (Array.isArray(record.error)) {
+		for (const entry of record.error) {
+			if (!entry || typeof entry !== "object") continue;
+			const message = (entry as Record<string, unknown>).msg;
+			if (typeof message === "string" && message.trim().length > 0) {
+				return message.trim();
+			}
+		}
+	}
+
+	return null;
+}
+
+function createKagiApiError(statusCode: number, detail?: string): KagiApiError {
+	return new KagiApiError(
+		detail ? `Kagi API error (${statusCode}): ${detail}` : `Kagi API error (${statusCode})`,
+		statusCode,
+	);
+}
+
+function parseKagiErrorResponse(statusCode: number, responseText: string): KagiApiError {
+	const trimmedResponseText = responseText.trim();
+	if (trimmedResponseText.length === 0) {
+		return createKagiApiError(statusCode);
+	}
+
+	try {
+		const payload = JSON.parse(trimmedResponseText) as KagiErrorResponse;
+		return createKagiApiError(statusCode, extractKagiErrorMessage(payload) ?? trimmedResponseText);
+	} catch {
+		return createKagiApiError(statusCode, trimmedResponseText);
+	}
+}
+
 export interface KagiSummarizeOptions {
 	engine?: string;
 	summaryType?: string;
@@ -127,14 +183,13 @@ export async function searchWithKagi(query: string, options: KagiSearchOptions =
 		signal: options.signal,
 	});
 	if (!response.ok) {
-		const errorText = await response.text();
-		throw new KagiApiError(`Kagi API error (${response.status}): ${errorText}`, response.status);
+		throw parseKagiErrorResponse(response.status, await response.text());
 	}
 
 	const payload = (await response.json()) as KagiSearchResponse;
 	if (payload.error && payload.error.length > 0) {
 		const firstError = payload.error[0];
-		throw new KagiApiError(`Kagi API error: ${firstError.msg}`, firstError.code);
+		throw createKagiApiError(firstError.code ?? response.status, extractKagiErrorMessage(payload) ?? undefined);
 	}
 
 	const sources: KagiSearchSource[] = [];