feat: Reintroduce geminiFreeTier configuration in provider settings and update related tests

Author: d-oit

evals/packages/types/src/roo-code.ts

@@ -357,8 +357,7 @@ export const providerSettingsSchema = z.object({
 	lmStudioSpeculativeDecodingEnabled: z.boolean().optional(),
 	// Gemini
 	geminiApiKey: z.string().optional(),
-	// geminiFreeTier: z.boolean().optional(), // Moved to globalSettingsSchema
-	geminiModelInfo: modelInfoSchema.optional(), // Keep this uncommented
+	geminiFreeTier: z.boolean().optional(),
 	googleGeminiBaseUrl: z.string().optional(),
 	// OpenAI Native
 	openAiNativeApiKey: z.string().optional(),
@@ -446,7 +445,7 @@ const providerSettingsRecord: ProviderSettingsRecord = {
 	lmStudioSpeculativeDecodingEnabled: undefined,
 	// Gemini
 	geminiApiKey: undefined,
-	// geminiFreeTier: undefined, // Moved to globalSettingsRecord
+	geminiFreeTier: undefined,
 	geminiModelInfo: undefined, // Keep this uncommented
 	googleGeminiBaseUrl: undefined,
 	// OpenAI Native

src/core/config/__tests__/ProviderSettingsManager.test.ts

@@ -300,6 +300,61 @@ describe("ProviderSettingsManager", () => {
 				"Failed to save config: Error: Failed to write provider profiles to secrets: Error: Storage failed",
 			)
 		})
+
+		it("should save boolean values correctly, even when true", async () => {
+			// Create a clean initial state
+			const initialState = {
+				currentApiConfigName: "default",
+				apiConfigs: {
+					default: { id: "default-id" },
+				},
+				modeApiConfigs: {
+					code: "default",
+					architect: "default",
+					ask: "default",
+				},
+				migrations: {
+					rateLimitSecondsMigrated: true,
+				},
+			}
+
+			// Mock the initial state
+			mockSecrets.get.mockResolvedValueOnce(JSON.stringify(initialState))
+
+			// Create a config with geminiFreeTier set to true
+			const configWithTrueBoolean: ProviderSettings = {
+				apiProvider: "gemini",
+				geminiFreeTier: true,
+			}
+
+			// Save the config
+			await providerSettingsManager.saveConfig("test", configWithTrueBoolean)
+
+			// Verify store was called
+			expect(mockSecrets.store).toHaveBeenCalled()
+
+			// Get what was stored
+			const storeCall = mockSecrets.store.mock.calls[0]
+			expect(storeCall[0]).toBe("roo_cline_config_api_config")
+
+			const storedData = JSON.parse(storeCall[1])
+
+			// Verify the structure
+			expect(storedData).toHaveProperty("apiConfigs.test")
+			expect(storedData.apiConfigs.test).toHaveProperty("apiProvider", "gemini")
+			expect(storedData.apiConfigs.test).toHaveProperty("geminiFreeTier", true)
+
+			// Now test loading the config
+			mockSecrets.get.mockReset()
+			mockSecrets.get.mockResolvedValueOnce(JSON.stringify(storedData))
+
+			// Load the config
+			const loadedConfig = await providerSettingsManager.loadConfig("test")
+
+			// Verify the loaded config
+			expect(loadedConfig).toHaveProperty("apiProvider", "gemini")
+			expect(loadedConfig).toHaveProperty("geminiFreeTier", true)
+		})
 	})
 
 	describe("DeleteConfig", () => {

src/core/webview/webviewMessageHandler.ts

@@ -645,10 +645,6 @@ export const webviewMessageHandler = async (provider: ClineProvider, message: We
 			await updateGlobalState("diffEnabled", diffEnabled)
 			await provider.postStateToWebview()
 			break
-		// case "geminiFreeTier": // Removed - Now part of apiConfiguration
-		// 	await provider.setValue("geminiFreeTier", message.bool ?? false) // Use provider.setValue
-		// 	await provider.postStateToWebview()
-		// 	break
 		case "showGreeting":
 			const showGreeting = message.bool ?? true
 			await updateGlobalState("showGreeting", showGreeting)