fix(opencode-go): green CI and address review nits (#172)

- Add missing opencodeGo translation keys across all 17 non-English locales
  (check-translations was failing).
- Include the opencode-go entry in expected routerModels for the ClineProvider
  and webviewMessageHandler requestRouterModels suites (unit tests were failing).
- Add a request timeout to the /models fetch so it can't hang.
- Fall back to the default ModelInfo in useSelectedModel when the /models list
  is empty, keeping capability-driven UI working.
- Validate the Opencode Go API key and add validate.spec + OpenCodeGo component
  tests.

Author: proyectoauraorg

src/api/providers/fetchers/__tests__/opencode-go.spec.ts

@@ -35,6 +35,7 @@ describe("Opencode Go Fetchers", () => {
 
 			expect(mockedAxios.get).toHaveBeenCalledWith("https://opencode.ai/zen/go/v1/models", {
 				headers: { Authorization: "Bearer test-key" },
+				timeout: 10_000,
 			})
 
 			expect(Object.keys(models).sort()).toEqual(["deepseek-v4-pro", "glm-5.1"])

src/api/providers/fetchers/opencode-go.ts

@@ -43,6 +43,7 @@ export async function getOpencodeGoModels(apiKey?: string): Promise<Record<strin
 	try {
 		const response = await axios.get(`${OPENCODE_GO_BASE_URL}/models`, {
 			headers: apiKey ? { Authorization: `Bearer ${apiKey}` } : undefined,
+			timeout: 10_000,
 		})
 
 		const result = opencodeGoModelsResponseSchema.safeParse(response.data)

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

@@ -2479,6 +2479,7 @@ describe("ClineProvider - Router Models", () => {
 				"vercel-ai-gateway": mockModels,
 				poe: {},
 				deepseek: {},
+				"opencode-go": {},
 			},
 			values: undefined,
 		})
@@ -2525,6 +2526,7 @@ describe("ClineProvider - Router Models", () => {
 				"vercel-ai-gateway": mockModels,
 				poe: {},
 				deepseek: {},
+				"opencode-go": {},
 			},
 			values: undefined,
 		})
@@ -2620,6 +2622,7 @@ describe("ClineProvider - Router Models", () => {
 				"vercel-ai-gateway": mockModels,
 				poe: {},
 				deepseek: {},
+				"opencode-go": {},
 			},
 			values: undefined,
 		})

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

@@ -371,6 +371,7 @@ describe("webviewMessageHandler - requestRouterModels", () => {
 				"vercel-ai-gateway": mockModels,
 				poe: {},
 				deepseek: {},
+				"opencode-go": {},
 			},
 			values: undefined,
 		})
@@ -457,6 +458,7 @@ describe("webviewMessageHandler - requestRouterModels", () => {
 				"vercel-ai-gateway": mockModels,
 				poe: {},
 				deepseek: {},
+				"opencode-go": {},
 			},
 			values: undefined,
 		})
@@ -512,6 +514,7 @@ describe("webviewMessageHandler - requestRouterModels", () => {
 				"vercel-ai-gateway": mockModels,
 				poe: {},
 				deepseek: {},
+				"opencode-go": {},
 			},
 			values: undefined,
 		})

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

@@ -0,0 +1,90 @@
+import { render, screen, fireEvent } from "@testing-library/react"
+
+import type { ProviderSettings, OrganizationAllowList } from "@roo-code/types"
+import { opencodeGoDefaultModelId } from "@roo-code/types"
+
+import { OpenCodeGo } from "../OpenCodeGo"
+
+vi.mock("@vscode/webview-ui-toolkit/react", () => ({
+	VSCodeTextField: ({ children, value, onInput, type }: any) => (
+		<div>
+			{children}
+			<input type={type} value={value} onChange={(e) => onInput(e)} data-testid="api-key-input" />
+		</div>
+	),
+}))
+
+vi.mock("@src/i18n/TranslationContext", () => ({
+	useAppTranslation: () => ({ t: (key: string) => key }),
+}))
+
+vi.mock("@src/components/common/VSCodeButtonLink", () => ({
+	VSCodeButtonLink: ({ children, href }: any) => (
+		<a href={href} data-testid="get-api-key-link">
+			{children}
+		</a>
+	),
+}))
+
+// Stub ModelPicker so we can assert the props it receives without pulling in its hooks.
+vi.mock("../../ModelPicker", () => ({
+	ModelPicker: ({ defaultModelId, modelIdKey, serviceName }: any) => (
+		<div
+			data-testid="model-picker"
+			data-default-model-id={defaultModelId}
+			data-model-id-key={modelIdKey}
+			data-service-name={serviceName}
+		/>
+	),
+}))
+
+describe("OpenCodeGo", () => {
+	const organizationAllowList: OrganizationAllowList = { allowAll: true, providers: {} }
+	const mockSetApiConfigurationField = vi.fn()
+
+	const renderComponent = (apiConfiguration: ProviderSettings) =>
+		render(
+			<OpenCodeGo
+				apiConfiguration={apiConfiguration}
+				setApiConfigurationField={mockSetApiConfigurationField}
+				organizationAllowList={organizationAllowList}
+			/>,
+		)
+
+	beforeEach(() => {
+		vi.clearAllMocks()
+	})
+
+	it("updates the API key via setApiConfigurationField on input", () => {
+		renderComponent({ opencodeGoApiKey: "" })
+
+		fireEvent.change(screen.getByTestId("api-key-input"), { target: { value: "secret-key" } })
+
+		expect(mockSetApiConfigurationField).toHaveBeenCalledWith("opencodeGoApiKey", "secret-key")
+	})
+
+	it("shows the get-API-key CTA only when no API key is set", () => {
+		const { rerender } = renderComponent({ opencodeGoApiKey: "" })
+		const link = screen.getByTestId("get-api-key-link")
+		expect(link).toBeInTheDocument()
+		expect(link).toHaveAttribute("href", "https://opencode.ai/docs/go/")
+
+		rerender(
+			<OpenCodeGo
+				apiConfiguration={{ opencodeGoApiKey: "already-set" }}
+				setApiConfigurationField={mockSetApiConfigurationField}
+				organizationAllowList={organizationAllowList}
+			/>,
+		)
+		expect(screen.queryByTestId("get-api-key-link")).not.toBeInTheDocument()
+	})
+
+	it("wires the ModelPicker with the Opencode Go defaults", () => {
+		renderComponent({ opencodeGoApiKey: "key" })
+
+		const picker = screen.getByTestId("model-picker")
+		expect(picker).toHaveAttribute("data-default-model-id", opencodeGoDefaultModelId)
+		expect(picker).toHaveAttribute("data-model-id-key", "opencodeGoModelId")
+		expect(picker).toHaveAttribute("data-service-name", "Opencode Go")
+	})
+})

webview-ui/src/components/ui/hooks/useSelectedModel.ts

@@ -27,6 +27,7 @@ import {
 	qwenCodeModels,
 	litellmDefaultModelInfo,
 	lMStudioDefaultModelInfo,
+	opencodeGoDefaultModelInfo,
 	BEDROCK_1M_CONTEXT_MODEL_IDS,
 	VERTEX_1M_CONTEXT_MODEL_IDS,
 	isDynamicProvider,
@@ -352,7 +353,9 @@ function getSelectedModel({
 				routerModels["opencode-go"],
 				defaultModelId,
 			)
-			const info = routerModels["opencode-go"]?.[id]
+			// Fall back to the provider's default ModelInfo so capability-driven UI
+			// keeps working when the /models list is empty or unavailable.
+			const info = routerModels["opencode-go"]?.[id] ?? opencodeGoDefaultModelInfo
 			return { id, info }
 		}
 		// case "anthropic":