refactor: remove debug logging from configuration checks in CodeIndexConfigManager and webviewMessageHandler

Author: daniel-lxs

src/core/webview/webviewMessageHandler.ts

@@ -1830,15 +1830,6 @@ export const webviewMessageHandler = async (
 			const settings = message.codeIndexSettings
 
 			try {
-				console.log(`[DEBUG WebviewHandler] Starting atomic save for code index settings`)
-				console.log(`[DEBUG WebviewHandler] Settings received:`, {
-					enabled: settings.codebaseIndexEnabled,
-					provider: settings.codebaseIndexEmbedderProvider,
-					openAiKey: settings.codeIndexOpenAiKey
-						? `"${settings.codeIndexOpenAiKey.substring(0, 4)}..."`
-						: "undefined",
-				})
-
 				// Save global state settings atomically (without codebaseIndexEnabled which is now in global settings)
 				const currentConfig = getGlobalState("codebaseIndexConfig") || {}
 				const globalStateConfig = {
@@ -1867,12 +1858,15 @@ export const webviewMessageHandler = async (
 						settings.codebaseIndexOpenAiCompatibleApiKey,
 					)
 				}
+				if (settings.codebaseIndexGeminiApiKey !== undefined) {
+					await provider.contextProxy.storeSecret(
+						"codebaseIndexGeminiApiKey",
+						settings.codebaseIndexGeminiApiKey,
+					)
+				}
 
 				// Verify secrets are actually stored
 				const storedOpenAiKey = provider.contextProxy.getSecret("codeIndexOpenAiKey")
-				console.log(
-					`[DEBUG WebviewHandler] Verification - stored OpenAI key: ${storedOpenAiKey ? `"${storedOpenAiKey.substring(0, 4)}..."` : "undefined"}`,
-				)
 
 				// Notify code index manager of changes
 				if (provider.codeIndexManager) {
@@ -1922,13 +1916,15 @@ export const webviewMessageHandler = async (
 			const hasOpenAiCompatibleApiKey = !!(await provider.context.secrets.get(
 				"codebaseIndexOpenAiCompatibleApiKey",
 			))
+			const hasGeminiApiKey = !!(await provider.context.secrets.get("codebaseIndexGeminiApiKey"))
 
 			provider.postMessageToWebview({
 				type: "codeIndexSecretStatus",
 				values: {
 					hasOpenAiKey,
 					hasQdrantApiKey,
 					hasOpenAiCompatibleApiKey,
+					hasGeminiApiKey,
 				},
 			})
 			break

src/services/code-index/config-manager.ts

@@ -183,17 +183,11 @@ export class CodeIndexConfigManager {
 			const apiKey = this.openAiCompatibleOptions?.apiKey
 			const qdrantUrl = this.qdrantUrl
 			const isConfigured = !!(baseUrl && apiKey && qdrantUrl)
-			console.log(
-				`[DEBUG ConfigManager] OpenAI Compatible config check: baseUrl=${baseUrl}, key=${apiKey ? `"${apiKey.substring(0, 4)}..."` : "undefined"}, url=${qdrantUrl}, configured=${isConfigured}`,
-			)
 			return isConfigured
 		} else if (this.embedderProvider === "gemini") {
 			const apiKey = this.geminiOptions?.apiKey
 			const qdrantUrl = this.qdrantUrl
 			const isConfigured = !!(apiKey && qdrantUrl)
-			console.log(
-				`[DEBUG ConfigManager] Gemini config check: key=${apiKey ? `"${apiKey.substring(0, 4)}..."` : "undefined"}, url=${qdrantUrl}, configured=${isConfigured}`,
-			)
 			return isConfigured
 		}
 		return false // Should not happen if embedderProvider is always set correctly