fix(gemini): use own-property check for known model ids (#227)

modelId in geminiModels also matches inherited Object.prototype keys
(e.g. "toString"), resolving info to a function. Use Object.hasOwn so only
real model keys match; prototype-key ids fall through to the default.

Author: proyectoauraorg

src/api/providers/__tests__/gemini.spec.ts

@@ -194,6 +194,18 @@ describe("GeminiHandler", () => {
 			expect(modelInfo.info.tiers).toBeUndefined()
 		})
 
+		it("should not treat Object prototype keys as known models", () => {
+			// `"toString" in geminiModels` is true via the prototype chain, which would
+			// otherwise resolve `info` to a function. An own-property check avoids this.
+			const protoHandler = new GeminiHandler({
+				apiModelId: "toString",
+				geminiApiKey: "test-key",
+			})
+			const modelInfo = protoHandler.getModel()
+			expect(modelInfo.id).toBe(geminiDefaultModelId)
+			expect(modelInfo.info).toBeDefined()
+		})
+
 		it("should exclude apply_diff and include edit in tool preferences", () => {
 			const modelInfo = handler.getModel()
 			expect(modelInfo.info.excludedTools).toContain("apply_diff")

src/api/providers/gemini.ts

@@ -493,7 +493,7 @@ export class GeminiHandler extends BaseProvider implements SingleCompletionHandl
 		let id: string
 		let info: ModelInfo
 
-		if (modelId && modelId in geminiModels) {
+		if (modelId && Object.hasOwn(geminiModels, modelId)) {
 			id = modelId
 			info = geminiModels[modelId as GeminiModelId]
 		} else if (modelId && modelId.toLowerCase().startsWith("gemini-")) {