fix(api): route setConfiguration through ClineProvider.setValues

Route the CS API setConfiguration write through ClineProvider.setValues
instead of the raw context proxy so the view-local subset stays in sync
with the sidebar view's state. A direct context-proxy write leaves the
view-local apiConfiguration stale; the mode-switch / profile-activation
paths re-apply the current provider settings to the focused task on
every user message, so a task started after an API setConfiguration runs
with the stale configuration (wrong model / reasoning flags / endpoint).
That is the deterministic 13-test e2e provider-suite failure (72/13) on
this head: the provider suites configure the provider via setConfiguration
and the mock serves fixtures by the model the request actually carries.

Ported from the F5 head (vps2 F5) where the e2e provider suites pass;
includes the unit spec asserting the routing.

Upstream: #982 (vps2 F3)

Author: easonliang28

src/extension/__tests__/api-set-configuration.spec.ts

@@ -0,0 +1,55 @@
+import { describe, expect, it, vi } from "vitest"
+
+import { providerIdentifiers } from "@roo-code/types"
+
+import { API } from "../api"
+import type { ClineProvider } from "../../core/webview/ClineProvider"
+import type { OutputChannel } from "vscode"
+
+vi.mock("@roo-code/ipc", () => ({
+	IpcServer: class {},
+}))
+
+vi.mock("../../integrations/terminal/Terminal", () => ({
+	Terminal: {
+		getTerminalProfile: vi.fn(),
+		setTerminalProfile: vi.fn(),
+	},
+}))
+
+vi.mock("../../integrations/terminal/TerminalRegistry", () => ({
+	TerminalRegistry: {
+		closeIdleTerminals: vi.fn(),
+	},
+}))
+
+describe("API.setConfiguration", () => {
+	it("routes configuration through ClineProvider.setValues so view-local state stays in sync", async () => {
+		const provider = {
+			context: {},
+			on: vi.fn(),
+			setValues: vi.fn().mockResolvedValue(undefined),
+			contextProxy: {
+				setValues: vi.fn().mockResolvedValue(undefined),
+			},
+			providerSettingsManager: {
+				saveConfig: vi.fn().mockResolvedValue("default-id"),
+			},
+			postStateToWebview: vi.fn().mockResolvedValue(undefined),
+		} as unknown as ClineProvider
+		const api = new API({ appendLine: vi.fn() } as unknown as OutputChannel, provider)
+		const configuration = {
+			apiProvider: providerIdentifiers.bedrock,
+			currentApiConfigName: "default",
+			awsRegion: "us-east-1",
+			apiModelId: "us.anthropic.claude-haiku-4-5-20251001-v1:0",
+		}
+
+		await api.setConfiguration(configuration)
+
+		expect(provider.setValues).toHaveBeenCalledWith(configuration)
+		expect(provider.contextProxy.setValues).not.toHaveBeenCalled()
+		expect(provider.providerSettingsManager.saveConfig).toHaveBeenCalledWith("default", configuration)
+		expect(provider.postStateToWebview).toHaveBeenCalled()
+	})
+})

src/extension/api.ts

@@ -565,7 +565,7 @@ export class API extends EventEmitter<RooCodeEvents> implements RooCodeAPI {
 	}
 
 	public async setConfiguration(values: RooCodeSettings) {
-		await this.sidebarProvider.contextProxy.setValues(values)
+		await this.sidebarProvider.setValues(values)
 		await this.sidebarProvider.providerSettingsManager.saveConfig(values.currentApiConfigName || "default", values)
 		if (values.modeApiConfigs) {
 			await Promise.all(