refactor(web-search): simplified web-search API by removing provider param

- Removed provider parameter from web search tool schema; provider selection now handled internally.
- Removed deprecated no_fallback option from search parameters; fallback behavior is now automatic.
- Renamed SearchParams type to SearchToolParams and introduced SearchQueryParams for CLI queries.
- Updated executeSearch() to accept SearchQueryParams with optional provider selection.

packages/coding-agent/CHANGELOG.md

@@ -1,6 +1,16 @@
 # Changelog
 
 ## [Unreleased]
+### Breaking Changes
+
+- Web search tool no longer accepts `provider` parameter in tool calls; use internal provider resolution instead
+- Removed `no_fallback` option from search parameters
+
+### Changed
+
+- Removed `provider` parameter from web search tool schema; provider selection now handled internally
+- Removed `no_fallback` parameter from web search parameters; fallback behavior now automatic based on provider availability
+- Renamed `SearchParams` type to `SearchToolParams` for tool execution; introduced `SearchQueryParams` for CLI queries with optional provider selection
 
 ## [13.9.15] - 2026-03-10
 ### Added

packages/coding-agent/src/cli/web-search-cli.ts

@@ -7,7 +7,7 @@
 import { APP_NAME } from "@oh-my-pi/pi-utils";
 import chalk from "chalk";
 import { initTheme, theme } from "../modes/theme/theme";
-import { runSearchQuery, type SearchParams } from "../web/search/index";
+import { runSearchQuery, type SearchQueryParams } from "../web/search/index";
 import { SEARCH_PROVIDER_ORDER } from "../web/search/provider";
 import { renderSearchResult } from "../web/search/render";
 import type { SearchProviderId } from "../web/search/types";
@@ -87,18 +87,16 @@ export async function runSearchCommand(cmd: SearchCommandArgs): Promise<void> {
 
 	await initTheme();
 
-	const params: SearchParams = {
+	const params: SearchQueryParams = {
 		query: cmd.query,
 		provider: cmd.provider,
 		recency: cmd.recency,
 		limit: cmd.limit,
-		no_fallback: cmd.provider !== undefined && cmd.provider !== "auto",
 	};
 
 	const result = await runSearchQuery(params);
 	const component = renderSearchResult(result, { expanded: cmd.expanded, isPartial: false }, theme, {
 		query: cmd.query,
-		provider: cmd.provider,
 		allowLongAnswer: true,
 		maxAnswerLines: cmd.expanded ? undefined : 6,
 	});

packages/coding-agent/src/web/search/index.ts

@@ -26,34 +26,12 @@ import type { ToolSession } from "../../tools";
 import { formatAge } from "../../tools/render-utils";
 import { getSearchProvider, resolveProviderChain, type SearchProvider } from "./provider";
 import { renderSearchCall, renderSearchResult, type SearchRenderDetails } from "./render";
-import type { SearchResponse } from "./types";
+import type { SearchProviderId, SearchResponse } from "./types";
 import { SearchProviderError } from "./types";
 
-/** Web search parameters schema */
+/** Web search tool parameters schema */
 export const webSearchSchema = Type.Object({
 	query: Type.String({ description: "Search query" }),
-	provider: Type.Optional(
-		StringEnum(
-			[
-				"auto",
-				"exa",
-				"brave",
-				"jina",
-				"kimi",
-				"zai",
-				"anthropic",
-				"perplexity",
-				"gemini",
-				"codex",
-				"tavily",
-				"kagi",
-				"synthetic",
-			],
-			{
-				description: "Search provider (default: auto)",
-			},
-		),
-	),
 	recency: Type.Optional(
 		StringEnum(["day", "week", "month", "year"], {
 			description: "Recency filter (Brave, Perplexity)",
@@ -65,22 +43,8 @@ export const webSearchSchema = Type.Object({
 	num_search_results: Type.Optional(Type.Number({ description: "Number of search results to retrieve" })),
 });
 
-export type SearchParams = {
+export type SearchToolParams = {
 	query: string;
-	provider?:
-		| "auto"
-		| "exa"
-		| "brave"
-		| "jina"
-		| "kimi"
-		| "zai"
-		| "anthropic"
-		| "perplexity"
-		| "gemini"
-		| "codex"
-		| "tavily"
-		| "kagi"
-		| "synthetic";
 	recency?: "day" | "week" | "month" | "year";
 	limit?: number;
 	/** Maximum output tokens. Defaults to 4096. */
@@ -89,10 +53,12 @@ export type SearchParams = {
 	temperature?: number;
 	/** Number of search results to retrieve. Defaults to 10. */
 	num_search_results?: number;
-	/** Deprecated CLI flag; explicit provider fallback now happens only when provider is unavailable. */
-	no_fallback?: boolean;
 };
 
+export interface SearchQueryParams extends SearchToolParams {
+	provider?: SearchProviderId | "auto";
+}
+
 function formatProviderList(providers: SearchProvider[]): string {
 	return providers.map(provider => provider.label).join(", ");
 }
@@ -178,14 +144,14 @@ function formatForLLM(response: SearchResponse): string {
 /** Execute web search */
 async function executeSearch(
 	_toolCallId: string,
-	params: SearchParams,
+	params: SearchQueryParams,
 ): Promise<{ content: Array<{ type: "text"; text: string }>; details: SearchRenderDetails }> {
 	const providers =
 		params.provider && params.provider !== "auto"
 			? (await getSearchProvider(params.provider).isAvailable())
 				? [getSearchProvider(params.provider)]
 				: await resolveProviderChain("auto")
-			: await resolveProviderChain(params.provider);
+			: await resolveProviderChain();
 	if (providers.length === 0) {
 		const message = "No web search provider configured.";
 		return {
@@ -237,7 +203,7 @@ async function executeSearch(
  * Execute a web search query for CLI/testing workflows.
  */
 export async function runSearchQuery(
-	params: SearchParams,
+	params: SearchQueryParams,
 ): Promise<{ content: Array<{ type: "text"; text: string }>; details: SearchRenderDetails }> {
 	return executeSearch("cli-web-search", params);
 }
@@ -261,7 +227,7 @@ export class SearchTool implements AgentTool<typeof webSearchSchema, SearchRende
 
 	async execute(
 		_toolCallId: string,
-		params: SearchParams,
+		params: SearchToolParams,
 		_signal?: AbortSignal,
 		_onUpdate?: AgentToolUpdateCallback<SearchRenderDetails>,
 		_context?: AgentToolContext,
@@ -277,11 +243,17 @@ export const webSearchCustomTool: CustomTool<typeof webSearchSchema, SearchRende
 	description: renderPromptTemplate(webSearchDescription),
 	parameters: webSearchSchema,
 
-	async execute(toolCallId: string, params: SearchParams, _onUpdate, _ctx: CustomToolContext, _signal?: AbortSignal) {
+	async execute(
+		toolCallId: string,
+		params: SearchToolParams,
+		_onUpdate,
+		_ctx: CustomToolContext,
+		_signal?: AbortSignal,
+	) {
 		return executeSearch(toolCallId, params);
 	},
 
-	renderCall(args: SearchParams, options: RenderResultOptions, theme: Theme) {
+	renderCall(args: SearchToolParams, options: RenderResultOptions, theme: Theme) {
 		return renderSearchCall(args, options, theme);
 	},
 

packages/coding-agent/src/web/search/render.ts

@@ -75,7 +75,6 @@ export function renderSearchResult(
 	theme: Theme,
 	args?: {
 		query?: string;
-		provider?: string;
 		allowLongAnswer?: boolean;
 		maxAnswerLines?: number;
 	},
@@ -282,13 +281,12 @@ export function renderSearchResult(
 
 /** Render web search call (query preview) */
 export function renderSearchCall(
-	args: { query?: string; provider?: string; [key: string]: unknown },
+	args: { query?: string; [key: string]: unknown },
 	_options: RenderResultOptions,
 	theme: Theme,
 ): Component {
-	const provider = args.provider ?? "auto";
 	const query = truncateToWidth(args.query ?? "", 80);
-	const text = renderStatusLine({ icon: "pending", title: "Web Search", description: query, meta: [provider] }, theme);
+	const text = renderStatusLine({ icon: "pending", title: "Web Search", description: query }, theme);
 	return new Text(text, 0, 0);
 }