fix(opencode-go): fetch models unconditionally — the /models endpoint is public (#437)

* fix(opencode-go): fetch models unconditionally — the /models endpoint is public

The Opencode Go model fetch in `requestRouterModels` was gated behind
`if (opencodeGoApiKey)` on the assumption that its `/models` endpoint requires
auth. It does not: the endpoint returns the full model list (HTTP 200) with no
Authorization header. Gating meant `routerModels["opencode-go"]` stayed `{}`
whenever the key wasn't present in `apiConfiguration` at fetch time, so the
model picker showed an empty list, fell back to the hardcoded default
(`glm-5.1`), and offered no model to select.

Fetch Opencode Go unconditionally like the other public routers
(`openrouter`, `vercel-ai-gateway`), forwarding the API key when present and
still flushing the cache when a new key is supplied. Updates the affected
`requestRouterModels` tests and adds a regression test for the keyless path.

* test: update ClineProvider.spec.ts assertions for unconditional opencode-go fetch

Mirror changes already made in webviewMessageHandler.spec.ts:
- successful responses: expect opencode-go call, assert mockModels
- individual provider failures: add mockResolvedValueOnce for opencode-go
- skips LiteLLM: assert mockModels instead of empty object

* add refresh button and update coverage

---------

Co-authored-by: Armando Vaquera <263793884+proyectoauraorg@users.noreply.github.com>
Co-authored-by: Naved  Merchant <naved.merchant@gmail.com>

Author: proyectoauraorg

src/core/webview/__tests__/ClineProvider.spec.ts

@@ -2580,6 +2580,8 @@ describe("ClineProvider - Router Models", () => {
 			apiKey: "litellm-key",
 			baseUrl: "http://localhost:4000",
 		})
+		// Opencode Go's /models endpoint is public, so it is fetched like the other no-auth routers.
+		expect(getModels).toHaveBeenCalledWith(expect.objectContaining({ provider: "opencode-go" }))
 
 		// Verify response was sent
 		expect(mockPostMessage).toHaveBeenCalledWith({
@@ -2595,7 +2597,7 @@ describe("ClineProvider - Router Models", () => {
 				lmstudio: {},
 				poe: {},
 				deepseek: {},
-				"opencode-go": {},
+				"opencode-go": mockModels,
 			},
 			values: undefined,
 		})
@@ -2627,6 +2629,7 @@ describe("ClineProvider - Router Models", () => {
 			.mockResolvedValueOnce(mockModels) // vercel-ai-gateway success
 			.mockResolvedValueOnce(mockModels) // zoo-gateway success
 			.mockRejectedValueOnce(new Error("LiteLLM connection failed")) // litellm fail
+			.mockResolvedValueOnce(mockModels) // opencode-go (public endpoint)
 
 		await messageHandler({ type: "requestRouterModels" })
 
@@ -2644,7 +2647,7 @@ describe("ClineProvider - Router Models", () => {
 				litellm: {},
 				poe: {},
 				deepseek: {},
-				"opencode-go": {},
+				"opencode-go": mockModels,
 			},
 			values: undefined,
 		})
@@ -2741,7 +2744,7 @@ describe("ClineProvider - Router Models", () => {
 				lmstudio: {},
 				poe: {},
 				deepseek: {},
-				"opencode-go": {},
+				"opencode-go": mockModels,
 			},
 			values: undefined,
 		})

src/core/webview/__tests__/webviewMessageHandler.spec.ts

@@ -366,6 +366,8 @@ describe("webviewMessageHandler - requestRouterModels", () => {
 			apiKey: "litellm-key",
 			baseUrl: "http://localhost:4000",
 		})
+		// Opencode Go's /models endpoint is public, so it is fetched like the other no-auth routers.
+		expect(mockGetModels).toHaveBeenCalledWith(expect.objectContaining({ provider: "opencode-go" }))
 
 		// Verify response was sent
 		expect(mockClineProvider.postMessageToWebview).toHaveBeenCalledWith({
@@ -381,12 +383,41 @@ describe("webviewMessageHandler - requestRouterModels", () => {
 				lmstudio: {},
 				poe: {},
 				deepseek: {},
-				"opencode-go": {},
+				"opencode-go": mockModels,
 			},
 			values: undefined,
 		})
 	})
 
+	it("fetches Opencode Go models without an API key (public /models endpoint, regression for empty picker)", async () => {
+		mockClineProvider.getState = vi.fn().mockResolvedValue({
+			apiConfiguration: {
+				openRouterApiKey: "openrouter-key",
+				// Deliberately no opencodeGoApiKey — the endpoint is public.
+			},
+		})
+
+		const mockModels: ModelRecord = {
+			"glm-5.1": {
+				maxTokens: 4096,
+				contextWindow: 8192,
+				supportsPromptCache: false,
+				description: "GLM 5.1",
+			},
+		}
+		mockGetModels.mockResolvedValue(mockModels)
+
+		await webviewMessageHandler(mockClineProvider, { type: "requestRouterModels" })
+
+		// Must be fetched despite no configured key, forwarding apiKey: undefined.
+		expect(mockGetModels).toHaveBeenCalledWith({ provider: "opencode-go", apiKey: undefined })
+
+		const routerModelsCall = (mockClineProvider.postMessageToWebview as any).mock.calls.find(
+			([msg]: [{ type: string }]) => msg.type === "routerModels",
+		)
+		expect(routerModelsCall?.[0].routerModels["opencode-go"]).toEqual(mockModels)
+	})
+
 	it("handles LiteLLM models with values from message when config is missing", async () => {
 		mockClineProvider.getState = vi.fn().mockResolvedValue({
 			apiConfiguration: {
@@ -469,7 +500,7 @@ describe("webviewMessageHandler - requestRouterModels", () => {
 				lmstudio: {},
 				poe: {},
 				deepseek: {},
-				"opencode-go": {},
+				"opencode-go": mockModels,
 			},
 			values: undefined,
 		})
@@ -493,6 +524,7 @@ describe("webviewMessageHandler - requestRouterModels", () => {
 			.mockResolvedValueOnce(mockModels) // vercel-ai-gateway
 			.mockResolvedValueOnce(mockModels) // zoo-gateway
 			.mockRejectedValueOnce(new Error("LiteLLM connection failed")) // litellm
+			.mockResolvedValueOnce(mockModels) // opencode-go
 
 		await webviewMessageHandler(mockClineProvider, {
 			type: "requestRouterModels",
@@ -527,7 +559,7 @@ describe("webviewMessageHandler - requestRouterModels", () => {
 				lmstudio: {},
 				poe: {},
 				deepseek: {},
-				"opencode-go": {},
+				"opencode-go": mockModels,
 			},
 			values: undefined,
 		})

src/core/webview/webviewMessageHandler.ts

@@ -1034,20 +1034,23 @@ export const webviewMessageHandler = async (
 				})
 			}
 
-			// Opencode Go is conditional on apiKey (its /models endpoint requires auth)
+			// Opencode Go's /models endpoint is public — it returns the full model list with no
+			// Authorization header — so it's fetched unconditionally like openrouter/vercel-ai-gateway
+			// above. Gating it behind a key meant the picker stayed empty (and fell back to the default
+			// model) whenever the key wasn't yet in apiConfiguration at fetch time. The key is still
+			// forwarded when present.
 			const opencodeGoApiKey = message?.values?.opencodeGoApiKey ?? apiConfiguration.opencodeGoApiKey
 
-			if (opencodeGoApiKey) {
-				if (message?.values?.opencodeGoApiKey) {
-					await flushModels({ provider: "opencode-go", apiKey: opencodeGoApiKey }, true)
-				}
-
-				candidates.push({
-					key: "opencode-go",
-					options: { provider: "opencode-go", apiKey: opencodeGoApiKey },
-				})
+			// Refresh the cache when a new key is explicitly provided (e.g. the Refresh Models button).
+			if (message?.values?.opencodeGoApiKey) {
+				await flushModels({ provider: "opencode-go", apiKey: opencodeGoApiKey }, true)
 			}
 
+			candidates.push({
+				key: "opencode-go",
+				options: { provider: "opencode-go", apiKey: opencodeGoApiKey },
+			})
+
 			// Apply single provider filter if specified
 			const modelFetchPromises = providerFilter
 				? candidates.filter(({ key }) => key === providerFilter)

webview-ui/src/components/settings/providers/OpenCodeGo.tsx

@@ -1,15 +1,20 @@
-import { useCallback } from "react"
+import { useCallback, useState, useEffect, useRef } from "react"
 import { VSCodeTextField } from "@vscode/webview-ui-toolkit/react"
 
 import {
 	type ProviderSettings,
 	type OrganizationAllowList,
 	type RouterModels,
+	type ExtensionMessage,
 	opencodeGoDefaultModelId,
 } from "@roo-code/types"
 
+import type { RouterName } from "@roo/api"
+
+import { vscode } from "@src/utils/vscode"
 import { useAppTranslation } from "@src/i18n/TranslationContext"
 import { VSCodeButtonLink } from "@src/components/common/VSCodeButtonLink"
+import { Button } from "@src/components/ui"
 
 import { inputEventTransform } from "../transforms"
 import { ModelPicker } from "../ModelPicker"
@@ -32,6 +37,34 @@ export const OpenCodeGo = ({
 	simplifySettings,
 }: OpenCodeGoProps) => {
 	const { t } = useAppTranslation()
+	const [refreshStatus, setRefreshStatus] = useState<"idle" | "loading" | "success" | "error">("idle")
+	const [refreshError, setRefreshError] = useState<string | undefined>()
+	const errorJustReceived = useRef(false)
+
+	useEffect(() => {
+		const handleMessage = (event: MessageEvent<ExtensionMessage>) => {
+			const message = event.data
+			if (message.type === "singleRouterModelFetchResponse" && !message.success) {
+				const providerName = message.values?.provider as RouterName
+				if (providerName === "opencode-go") {
+					errorJustReceived.current = true
+					setRefreshStatus("error")
+					setRefreshError(message.error)
+				}
+			} else if (message.type === "routerModels") {
+				if (refreshStatus === "loading") {
+					if (!errorJustReceived.current) {
+						setRefreshStatus("success")
+					}
+				}
+			}
+		}
+
+		window.addEventListener("message", handleMessage)
+		return () => {
+			window.removeEventListener("message", handleMessage)
+		}
+	}, [refreshStatus])
 
 	const handleInputChange = useCallback(
 		<K extends keyof ProviderSettings, E>(
@@ -44,6 +77,16 @@ export const OpenCodeGo = ({
 		[setApiConfigurationField],
 	)
 
+	const handleRefreshModels = useCallback(() => {
+		errorJustReceived.current = false
+		setRefreshStatus("loading")
+		setRefreshError(undefined)
+		vscode.postMessage({
+			type: "requestRouterModels",
+			values: { provider: "opencode-go", refresh: true, opencodeGoApiKey: apiConfiguration.opencodeGoApiKey },
+		})
+	}, [apiConfiguration.opencodeGoApiKey])
+
 	return (
 		<>
 			<VSCodeTextField
@@ -62,6 +105,33 @@ export const OpenCodeGo = ({
 					{t("settings:providers.getOpencodeGoApiKey")}
 				</VSCodeButtonLink>
 			)}
+			<Button
+				variant="outline"
+				onClick={handleRefreshModels}
+				disabled={refreshStatus === "loading"}
+				className="w-full">
+				<div className="flex items-center gap-2">
+					{refreshStatus === "loading" ? (
+						<span className="codicon codicon-loading codicon-modifier-spin" />
+					) : (
+						<span className="codicon codicon-refresh" />
+					)}
+					{t("settings:providers.refreshModels.label")}
+				</div>
+			</Button>
+			{refreshStatus === "loading" && (
+				<div className="text-sm text-vscode-descriptionForeground">
+					{t("settings:providers.refreshModels.loading")}
+				</div>
+			)}
+			{refreshStatus === "success" && (
+				<div className="text-sm text-vscode-foreground">{t("settings:providers.refreshModels.success")}</div>
+			)}
+			{refreshStatus === "error" && (
+				<div className="text-sm text-vscode-errorForeground">
+					{refreshError || t("settings:providers.refreshModels.error")}
+				</div>
+			)}
 			<ModelPicker
 				apiConfiguration={apiConfiguration}
 				setApiConfigurationField={setApiConfigurationField}