Default to native tools for all models in the Roo provider

Author: mrubens

src/api/providers/__tests__/roo.spec.ts

@@ -101,6 +101,7 @@ vitest.mock("../../providers/fetchers/modelCache", () => ({
 					supportsPromptCache: true,
 					inputPrice: 0,
 					outputPrice: 0,
+					defaultToolProtocol: "native",
 				},
 				"minimax/minimax-m2:free": {
 					maxTokens: 32_768,
@@ -110,6 +111,7 @@ vitest.mock("../../providers/fetchers/modelCache", () => ({
 					supportsNativeTools: true,
 					inputPrice: 0.15,
 					outputPrice: 0.6,
+					defaultToolProtocol: "native",
 				},
 				"anthropic/claude-haiku-4.5": {
 					maxTokens: 8_192,
@@ -119,6 +121,7 @@ vitest.mock("../../providers/fetchers/modelCache", () => ({
 					supportsNativeTools: true,
 					inputPrice: 0.8,
 					outputPrice: 4,
+					defaultToolProtocol: "native",
 				},
 			}
 		}
@@ -425,28 +428,15 @@ describe("RooHandler", () => {
 			}
 		})
 
-		it("should apply defaultToolProtocol: native for minimax/minimax-m2:free", () => {
-			const handlerWithMinimax = new RooHandler({
-				apiModelId: "minimax/minimax-m2:free",
-			})
-			const modelInfo = handlerWithMinimax.getModel()
-			expect(modelInfo.id).toBe("minimax/minimax-m2:free")
-			expect((modelInfo.info as any).defaultToolProtocol).toBe("native")
-			// Verify cached model info is preserved
-			expect(modelInfo.info.maxTokens).toBe(32_768)
-			expect(modelInfo.info.contextWindow).toBe(1_000_000)
-		})
-
-		it("should apply defaultToolProtocol: native for anthropic/claude-haiku-4.5", () => {
-			const handlerWithHaiku = new RooHandler({
-				apiModelId: "anthropic/claude-haiku-4.5",
-			})
-			const modelInfo = handlerWithHaiku.getModel()
-			expect(modelInfo.id).toBe("anthropic/claude-haiku-4.5")
-			expect((modelInfo.info as any).defaultToolProtocol).toBe("native")
-			// Verify cached model info is preserved
-			expect(modelInfo.info.maxTokens).toBe(8_192)
-			expect(modelInfo.info.contextWindow).toBe(200_000)
+		it("should have defaultToolProtocol: native for all roo provider models", () => {
+			// Test that all models have defaultToolProtocol: native
+			const testModels = ["minimax/minimax-m2:free", "anthropic/claude-haiku-4.5", "xai/grok-code-fast-1"]
+			for (const modelId of testModels) {
+				const handlerWithModel = new RooHandler({ apiModelId: modelId })
+				const modelInfo = handlerWithModel.getModel()
+				expect(modelInfo.id).toBe(modelId)
+				expect((modelInfo.info as any).defaultToolProtocol).toBe("native")
+			}
 		})
 
 		it("should not override existing properties when applying MODEL_DEFAULTS", () => {

src/api/providers/fetchers/roo.ts

@@ -10,16 +10,9 @@ import { DEFAULT_HEADERS } from "../constants"
 // Exported so RooHandler.getModel() can also apply these for fallback cases
 export const MODEL_DEFAULTS: Record<string, Partial<ModelInfo>> = {
 	"minimax/minimax-m2:free": {
-		defaultToolProtocol: "native",
 		includedTools: ["search_and_replace"],
 		excludedTools: ["apply_diff"],
 	},
-	"anthropic/claude-haiku-4.5": {
-		defaultToolProtocol: "native",
-	},
-	"xai/grok-code-fast-1": {
-		defaultToolProtocol: "native",
-	},
 }
 
 /**
@@ -109,13 +102,8 @@ export async function getRooModels(baseUrl: string, apiKey?: string): Promise<Mo
 				// Determine if the model requires reasoning effort based on tags
 				const requiredReasoningEffort = tags.includes("reasoning-required")
 
-				// Determine if native tool calling should be the default protocol for this model
-				const hasDefaultNativeTools = tags.includes("default-native-tools")
-				const defaultToolProtocol = hasDefaultNativeTools ? ("native" as const) : undefined
-
 				// Determine if the model supports native tool calling based on tags
-				// default-native-tools implies tool-use support
-				const supportsNativeTools = tags.includes("tool-use") || hasDefaultNativeTools
+				const supportsNativeTools = tags.includes("tool-use")
 
 				// Determine if the model should hide vendor/company identity (stealth mode)
 				const isStealthModel = tags.includes("stealth")
@@ -143,7 +131,7 @@ export async function getRooModels(baseUrl: string, apiKey?: string): Promise<Mo
 					deprecated: model.deprecated || false,
 					isFree: tags.includes("free"),
 					defaultTemperature: model.default_temperature,
-					defaultToolProtocol,
+					defaultToolProtocol: "native" as const,
 					isStealthModel: isStealthModel || undefined,
 				}