fix(tui): clarify web search provider labels (#43155)

Co-authored-by: Aiden Cline <rekram1-node@users.noreply.github.com>

packages/tui/src/util/tool-display.ts

@@ -20,11 +20,8 @@ export function primitiveInputSummary(input: Record<string, unknown>, omit: read
 }
 
 export function webSearchProviderLabel(provider: unknown) {
-  if (provider === "parallel") return "Parallel Web Search"
-  if (provider === "exa") return "Exa Web Search"
-  if (provider === "firecrawl") return "Firecrawl Web Search"
-  if (provider === "tavily") return "Tavily Web Search"
-  return "Web Search"
+  if (typeof provider !== "string" || !provider) return "Web Search"
+  return `Web Search via ${provider[0].toUpperCase()}${provider.slice(1)}`
 }
 
 export function toolDisplayMetadata(state: unknown): Record<string, unknown> {

packages/tui/test/mini/permission.shared.test.ts

@@ -147,7 +147,7 @@ describe("run permission shared", () => {
         }),
       ),
     ).toMatchObject({
-      title: 'Parallel Web Search "current releases"',
+      title: 'Web Search via Parallel "current releases"',
       lines: ["Query: current releases"],
     })
   })

packages/tui/test/util/tool-display.test.ts

@@ -21,9 +21,14 @@ test("normalizes shared tool primitives", () => {
 
 describe("webSearchProviderLabel", () => {
   test("labels known providers", () => {
-    expect(webSearchProviderLabel("parallel")).toBe("Parallel Web Search")
-    expect(webSearchProviderLabel("exa")).toBe("Exa Web Search")
-    expect(webSearchProviderLabel("tavily")).toBe("Tavily Web Search")
+    expect(webSearchProviderLabel("parallel")).toBe("Web Search via Parallel")
+    expect(webSearchProviderLabel("exa")).toBe("Web Search via Exa")
+    expect(webSearchProviderLabel("firecrawl")).toBe("Web Search via Firecrawl")
+    expect(webSearchProviderLabel("tavily")).toBe("Web Search via Tavily")
+  })
+
+  test("labels providers dynamically", () => {
+    expect(webSearchProviderLabel("other")).toBe("Web Search via Other")
   })
 
   for (const [name, provider] of [
@@ -32,7 +37,7 @@ describe("webSearchProviderLabel", () => {
     ["an object", {}],
     ["an array", []],
     ["a number", 1],
-    ["an unexpected string", "other"],
+    ["an empty string", ""],
   ] as const) {
     test(`uses the generic label for ${name}`, () => {
       expect(webSearchProviderLabel(provider)).toBe("Web Search")