refactor: move Claude Code built-in tools filtering to CLI level

As suggested by @juliettefournier-econ, this change moves the filtering
of Claude Code built-in tools (ExitPlanMode, BashOutput, KillBash) from
the handler level to the CLI configuration level by adding them to the
claudeCodeTools disallowed list in run.ts.

This is a cleaner approach as it prevents these tools from being used
at the source rather than filtering them after the fact.

Author: roomote

src/api/providers/__tests__/claude-code.spec.ts

@@ -562,137 +562,4 @@ describe("ClaudeCodeHandler", () => {
 
 		consoleSpy.mockRestore()
 	})
-
-	test("should silently ignore Claude Code built-in tools", async () => {
-		const systemPrompt = "You are a helpful assistant"
-		const messages = [{ role: "user" as const, content: "Hello" }]
-		const consoleSpy = vi.spyOn(console, "error").mockImplementation(() => {})
-
-		// Mock async generator that yields Claude Code built-in tool_use content
-		const mockGenerator = async function* (): AsyncGenerator<ClaudeCodeMessage | string> {
-			yield {
-				type: "assistant" as const,
-				message: {
-					id: "msg_123",
-					type: "message",
-					role: "assistant",
-					model: "claude-3-5-sonnet-20241022",
-					content: [
-						{
-							type: "tool_use",
-							id: "tool_123",
-							name: "ExitPlanMode",
-							input: {},
-						},
-						{
-							type: "tool_use",
-							id: "tool_124",
-							name: "BashOutput",
-							input: { command: "ls" },
-						},
-						{
-							type: "tool_use",
-							id: "tool_125",
-							name: "KillBash",
-							input: { pid: 1234 },
-						},
-						{
-							type: "text",
-							text: "Here's the response",
-						},
-					],
-					stop_reason: null,
-					stop_sequence: null,
-					usage: {
-						input_tokens: 10,
-						output_tokens: 20,
-					},
-				} as any,
-				session_id: "session_123",
-			}
-		}
-
-		mockRunClaudeCode.mockReturnValue(mockGenerator())
-
-		const stream = handler.createMessage(systemPrompt, messages)
-		const results = []
-
-		for await (const chunk of stream) {
-			results.push(chunk)
-		}
-
-		// Should NOT log errors for Claude Code built-in tools
-		expect(consoleSpy).not.toHaveBeenCalled()
-
-		// Should only have the text content in results
-		expect(results).toHaveLength(1)
-		expect(results[0]).toEqual({
-			type: "text",
-			text: "Here's the response",
-		})
-
-		consoleSpy.mockRestore()
-	})
-
-	test("should log warning for non-Claude Code tools but not for built-in ones", async () => {
-		const systemPrompt = "You are a helpful assistant"
-		const messages = [{ role: "user" as const, content: "Hello" }]
-		const consoleSpy = vi.spyOn(console, "error").mockImplementation(() => {})
-
-		// Mock async generator that yields mixed tool_use content
-		const mockGenerator = async function* (): AsyncGenerator<ClaudeCodeMessage | string> {
-			yield {
-				type: "assistant" as const,
-				message: {
-					id: "msg_123",
-					type: "message",
-					role: "assistant",
-					model: "claude-3-5-sonnet-20241022",
-					content: [
-						{
-							type: "tool_use",
-							id: "tool_123",
-							name: "ExitPlanMode", // Built-in tool - should be ignored silently
-							input: {},
-						},
-						{
-							type: "tool_use",
-							id: "tool_124",
-							name: "custom_tool", // Non-built-in tool - should log error
-							input: { data: "test" },
-						},
-						{
-							type: "tool_use",
-							id: "tool_125",
-							name: "BashOutput", // Built-in tool - should be ignored silently
-							input: { command: "pwd" },
-						},
-					],
-					stop_reason: null,
-					stop_sequence: null,
-					usage: {
-						input_tokens: 10,
-						output_tokens: 20,
-					},
-				} as any,
-				session_id: "session_123",
-			}
-		}
-
-		mockRunClaudeCode.mockReturnValue(mockGenerator())
-
-		const stream = handler.createMessage(systemPrompt, messages)
-		const results = []
-
-		for await (const chunk of stream) {
-			results.push(chunk)
-		}
-
-		// Should only log error for the non-built-in tool
-		expect(consoleSpy).toHaveBeenCalledTimes(1)
-		expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining("tool_use is not supported yet"))
-		expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining("custom_tool"))
-
-		consoleSpy.mockRestore()
-	})
 })

src/api/providers/claude-code.ts

@@ -16,8 +16,6 @@ import { ApiHandlerOptions } from "../../shared/api"
 
 export class ClaudeCodeHandler extends BaseProvider implements ApiHandler {
 	private options: ApiHandlerOptions
-	// Claude Code built-in tools that should be filtered out
-	private static readonly CLAUDE_CODE_BUILTIN_TOOLS = new Set(["ExitPlanMode", "BashOutput", "KillBash"])
 
 	constructor(options: ApiHandlerOptions) {
 		super()
@@ -118,11 +116,7 @@ export class ClaudeCodeHandler extends BaseProvider implements ApiHandler {
 							}
 							break
 						case "tool_use":
-							// Filter out Claude Code built-in tools
-							if (!ClaudeCodeHandler.CLAUDE_CODE_BUILTIN_TOOLS.has(content.name)) {
-								console.error(`tool_use is not supported yet. Received: ${JSON.stringify(content)}`)
-							}
-							// Silently ignore Claude Code built-in tools
+							console.error(`tool_use is not supported yet. Received: ${JSON.stringify(content)}`)
 							break
 					}
 				}

src/integrations/claude-code/run.ts

@@ -139,6 +139,9 @@ const claudeCodeTools = [
 	"TodoRead",
 	"TodoWrite",
 	"WebSearch",
+	"ExitPlanMode",
+	"BashOutput",
+	"KillBash",
 ].join(",")
 
 const CLAUDE_CODE_TIMEOUT = 600000 // 10 minutes