test: bumping coverage

Author: edelauna

apps/cli/src/commands/cli/__tests__/list.test.ts

@@ -1,4 +1,5 @@
 import { readWorkspaceTaskSessions } from "@/lib/task-history/index.js"
+import { isRecord } from "@/lib/utils/guards.js"
 
 import { listSessions, parseFormat } from "../list.js"
 
@@ -38,6 +39,33 @@ describe("parseFormat", () => {
 	})
 })
 
+describe("router model extraction", () => {
+	// This mirrors the extraction logic in requestOpenRouterModels (list.ts:226-228)
+	const extractOpenRouterModels = (routerModelsRaw: unknown) => {
+		const routerModels = isRecord(routerModelsRaw) ? routerModelsRaw : {}
+		const openRouterModels = routerModels.openrouter
+		return isRecord(openRouterModels) ? openRouterModels : {}
+	}
+
+	it("extracts openrouter models from valid routerModels", () => {
+		const models = { "openai/gpt-4.1": { contextWindow: 128000, supportsPromptCache: false } }
+		const result = extractOpenRouterModels({ openrouter: models })
+		expect(result).toEqual(models)
+	})
+
+	it("returns empty object when routerModels is null", () => {
+		expect(extractOpenRouterModels(null)).toEqual({})
+	})
+
+	it("returns empty object when openrouter key is missing", () => {
+		expect(extractOpenRouterModels({ requesty: {} })).toEqual({})
+	})
+
+	it("returns empty object when openrouter value is not a record", () => {
+		expect(extractOpenRouterModels({ openrouter: "invalid" })).toEqual({})
+	})
+})
+
 describe("listSessions", () => {
 	const workspacePath = process.cwd()
 

apps/cli/src/types/__tests__/types.test.ts

@@ -0,0 +1,46 @@
+import { isSupportedProvider, supportedProviders } from "../types.js"
+
+describe("isSupportedProvider", () => {
+	it.each(supportedProviders)("returns true for supported provider '%s'", (provider) => {
+		expect(isSupportedProvider(provider)).toBe(true)
+	})
+
+	it("returns false for 'roo' (retired provider)", () => {
+		expect(isSupportedProvider("roo")).toBe(false)
+	})
+
+	it("returns false for unknown provider", () => {
+		expect(isSupportedProvider("not-a-provider")).toBe(false)
+	})
+
+	it("returns false for empty string", () => {
+		expect(isSupportedProvider("")).toBe(false)
+	})
+})
+
+describe("provider resolution fallback", () => {
+	it("defaults to openrouter when no flag or setting is provided", () => {
+		const flagProvider = undefined
+		const settingsProvider = undefined
+		const effectiveProvider = flagProvider ?? settingsProvider ?? "openrouter"
+
+		expect(effectiveProvider).toBe("openrouter")
+		expect(isSupportedProvider(effectiveProvider)).toBe(true)
+	})
+
+	it("uses flag provider over settings and default", () => {
+		const flagProvider = "anthropic"
+		const settingsProvider = "gemini"
+		const effectiveProvider = flagProvider ?? settingsProvider ?? "openrouter"
+
+		expect(effectiveProvider).toBe("anthropic")
+	})
+
+	it("uses settings provider when flag is not provided", () => {
+		const flagProvider = undefined
+		const settingsProvider = "gemini"
+		const effectiveProvider = flagProvider ?? settingsProvider ?? "openrouter"
+
+		expect(effectiveProvider).toBe("gemini")
+	})
+})

apps/vscode-e2e/src/types/global.d.ts

@@ -1,6 +1,7 @@
 import type { RooCodeAPI } from "@roo-code/types"
 
 declare global {
+	// eslint-disable-next-line no-var -- var is required in declare global
 	var api: RooCodeAPI
 }