Merge pull request #87 from trypear/agent/model-provider-refresh-2026-06-12

Author: nang-dev

jest.config.js

@@ -36,6 +36,7 @@ module.exports = {
 		"^default-shell$": "<rootDir>/src/__mocks__/default-shell.js",
 		"^os-name$": "<rootDir>/src/__mocks__/os-name.js",
 		"^strip-bom$": "<rootDir>/src/__mocks__/strip-bom.js",
+		"^execa$": "<rootDir>/src/__mocks__/execa.js",
 		"^@roo/(.*)$": "<rootDir>/src/$1",
 		"^@src/(.*)$": "<rootDir>/webview-ui/src/$1",
 	},

package-lock.json

@@ -409,7 +409,7 @@
 		"monaco-vscode-textmate-theme-converter": "^0.1.7",
 		"node-cache": "^5.1.2",
 		"node-ipc": "^12.0.0",
-		"openai": "^4.78.1",
+		"openai": "^6.42.0",
 		"os-name": "^6.0.0",
 		"p-wait-for": "^5.0.2",
 		"pdf-parse": "^1.1.1",
@@ -435,7 +435,7 @@
 		"vscode-material-icons": "^0.1.1",
 		"web-tree-sitter": "^0.22.6",
 		"workerpool": "^9.2.0",
-		"zod": "^3.23.8"
+		"zod": "^3.25.76"
 	},
 	"devDependencies": {
 		"@changesets/cli": "^2.27.10",

package.json

@@ -0,0 +1,38 @@
+class ExecaError extends Error {
+	constructor(message = "Command failed", options = {}) {
+		super(message)
+		this.name = "ExecaError"
+		this.exitCode = options.exitCode ?? 1
+		this.signal = options.signal
+	}
+}
+
+const mockFn = (implementation) => (typeof jest === "undefined" ? implementation : jest.fn(implementation))
+
+const createSubprocess = () => {
+	const subprocess = Promise.resolve({
+		exitCode: 0,
+		stdout: "",
+		stderr: "",
+		all: "",
+	})
+
+	subprocess.pid = 1234
+	subprocess.iterable = mockFn(async function* () {})
+	subprocess.kill = mockFn(() => true)
+
+	return subprocess
+}
+
+const execa = mockFn((firstArg) => {
+	if (firstArg && typeof firstArg === "object" && !Array.isArray(firstArg)) {
+		return mockFn(() => createSubprocess())
+	}
+
+	return createSubprocess()
+})
+
+module.exports = {
+	execa,
+	ExecaError,
+}

src/__mocks__/execa.js

@@ -60,6 +60,10 @@ const vscode = {
 		Workspace: 2,
 		WorkspaceFolder: 3,
 	},
+	CodeActionKind: {
+		QuickFix: { value: "quickfix" },
+		RefactorRewrite: { value: "refactor.rewrite" },
+	},
 	Position: class {
 		constructor(line, character) {
 			this.line = line

src/__mocks__/vscode.js

@@ -165,13 +165,17 @@ describe("DeepSeekHandler", () => {
 				...mockOptions,
 				apiModelId: "invalid-model",
 			})
+			const defaultHandler = new DeepSeekHandler({
+				...mockOptions,
+				apiModelId: undefined,
+			})
 			const model = handlerWithInvalidModel.getModel()
 			expect(model.id).toBe("invalid-model") // Returns provided ID
 			expect(model.info).toBeDefined()
 			// With the current implementation, it's the same object reference when using default model info
-			expect(model.info).toBe(handler.getModel().info)
+			expect(model.info).toBe(defaultHandler.getModel().info)
 			// Should have the same base properties
-			expect(model.info.contextWindow).toBe(handler.getModel().info.contextWindow)
+			expect(model.info.contextWindow).toBe(defaultHandler.getModel().info.contextWindow)
 			// And should have supportsPromptCache set to true
 			expect(model.info.supportsPromptCache).toBe(true)
 		})

src/api/providers/__tests__/deepseek.test.ts

@@ -383,7 +383,7 @@ describe("OpenAiNativeHandler", () => {
 				openAiNativeApiKey: "test-api-key",
 			})
 			const modelInfo = handlerWithoutModel.getModel()
-			expect(modelInfo.id).toBe("gpt-4.1") // Default model
+			expect(modelInfo.id).toBe("gpt-5.5") // Default model
 			expect(modelInfo.info).toBeDefined()
 		})
 	})

src/api/providers/__tests__/openai-native.test.ts

@@ -61,17 +61,17 @@ describe("XAIHandler", () => {
 	})
 
 	test("should return specified model when valid model is provided", () => {
-		const testModelId = "grok-2-latest"
+		const testModelId = "grok-build-0.1"
 		const handlerWithModel = new XAIHandler({ apiModelId: testModelId })
 		const model = handlerWithModel.getModel()
 
 		expect(model.id).toBe(testModelId)
 		expect(model.info).toEqual(xaiModels[testModelId])
 	})
 
-	test("should include reasoning_effort parameter for mini models", async () => {
-		const miniModelHandler = new XAIHandler({
-			apiModelId: "grok-3-mini-beta",
+	test("should include reasoning_effort parameter for Grok 4.3", async () => {
+		const reasoningModelHandler = new XAIHandler({
+			apiModelId: "grok-4.3",
 			reasoningEffort: "high",
 		})
 
@@ -87,7 +87,7 @@ describe("XAIHandler", () => {
 		})
 
 		// Start generating a message
-		const messageGenerator = miniModelHandler.createMessage("test prompt", [])
+		const messageGenerator = reasoningModelHandler.createMessage("test prompt", [])
 		await messageGenerator.next() // Start the generator
 
 		// Check that reasoning_effort was included
@@ -98,9 +98,9 @@ describe("XAIHandler", () => {
 		)
 	})
 
-	test("should not include reasoning_effort parameter for non-mini models", async () => {
+	test("should not include reasoning_effort parameter for non-reasoning models", async () => {
 		const regularModelHandler = new XAIHandler({
-			apiModelId: "grok-2-latest",
+			apiModelId: "grok-build-0.1",
 			reasoningEffort: "high",
 		})
 
@@ -254,7 +254,7 @@ describe("XAIHandler", () => {
 
 	test("createMessage should pass correct parameters to OpenAI client", async () => {
 		// Setup a handler with specific model
-		const modelId = "grok-2-latest"
+		const modelId = "grok-build-0.1"
 		const modelInfo = xaiModels[modelId]
 		const handlerWithModel = new XAIHandler({ apiModelId: modelId })
 

src/api/providers/__tests__/xai.test.ts

@@ -13,6 +13,12 @@ import { BaseProvider } from "./base-provider"
 import { ANTHROPIC_DEFAULT_MAX_TOKENS } from "./constants"
 import { SingleCompletionHandler, getModelParams } from "../index"
 
+const ANTHROPIC_MODELS_WITHOUT_SAMPLING_PARAMS = new Set(["claude-fable-5", "claude-opus-4-8", "claude-sonnet-4-6"])
+
+function omitsSamplingParams(modelId: string) {
+	return ANTHROPIC_MODELS_WITHOUT_SAMPLING_PARAMS.has(modelId) || modelId.startsWith("claude-opus-4-7")
+}
+
 export class AnthropicHandler extends BaseProvider implements SingleCompletionHandler {
 	private options: ApiHandlerOptions
 	private client: Anthropic
@@ -34,8 +40,16 @@ export class AnthropicHandler extends BaseProvider implements SingleCompletionHa
 		let stream: AnthropicStream<Anthropic.Messages.RawMessageStreamEvent>
 		const cacheControl: CacheControlEphemeral = { type: "ephemeral" }
 		let { id: modelId, maxTokens, thinking, temperature, virtualId } = this.getModel()
+		let requestTemperature: number | undefined = temperature
+		if (omitsSamplingParams(modelId)) {
+			requestTemperature = undefined
+		}
 
 		switch (modelId) {
+			case "claude-fable-5":
+			case "claude-opus-4-8":
+			case "claude-sonnet-4-6":
+			case "claude-haiku-4-5-20251001":
 			case "claude-3-7-sonnet-20250219":
 			case "claude-3-5-sonnet-20241022":
 			case "claude-3-5-haiku-20241022":
@@ -63,7 +77,7 @@ export class AnthropicHandler extends BaseProvider implements SingleCompletionHa
 					{
 						model: modelId,
 						max_tokens: maxTokens ?? ANTHROPIC_DEFAULT_MAX_TOKENS,
-						temperature,
+						temperature: requestTemperature,
 						thinking,
 						// Setting cache breakpoint for system prompt so new tasks can reuse it.
 						system: [{ text: systemPrompt, type: "text", cache_control: cacheControl }],
@@ -129,7 +143,7 @@ export class AnthropicHandler extends BaseProvider implements SingleCompletionHa
 				stream = (await this.client.messages.create({
 					model: modelId,
 					max_tokens: maxTokens ?? ANTHROPIC_DEFAULT_MAX_TOKENS,
-					temperature,
+					temperature: requestTemperature,
 					system: [{ text: systemPrompt, type: "text" }],
 					messages,
 					stream: true,
@@ -273,12 +287,16 @@ export class AnthropicHandler extends BaseProvider implements SingleCompletionHa
 
 	async completePrompt(prompt: string) {
 		let { id: model, temperature } = this.getModel()
+		let requestTemperature: number | undefined = temperature
+		if (omitsSamplingParams(model)) {
+			requestTemperature = undefined
+		}
 
 		const message = await this.client.messages.create({
 			model,
 			max_tokens: ANTHROPIC_DEFAULT_MAX_TOKENS,
 			thinking: undefined,
-			temperature,
+			temperature: requestTemperature,
 			messages: [{ role: "user", content: prompt }],
 			stream: false,
 		})