Fix install bugs: friendly Gemini errors, install button states, mic button after install

Author: TechyCSR

main.js

@@ -1366,14 +1366,35 @@ class ApplicationController {
         }
       }
 
-      // If provider changed, reinitialize speech service so the new provider
-      // is picked up immediately without a restart.
-      if (settings.speechProvider && speechService.provider !== settings.speechProvider) {
+      // Reinitialize speech service when provider OR whisper command
+      // changes. Without the second check, the install flow (which
+      // writes a new whisperCommand after install but keeps the same
+      // provider) would leave the speech service pointing at a stale
+      // (or non-existent) binary, and the main overlay's mic button
+      // would stay hidden / non-functional.
+      const providerChanged = settings.speechProvider && speechService.provider !== settings.speechProvider;
+      const whisperCommandChanged = settings.whisperCommand !== undefined &&
+        (process.env.WHISPER_COMMAND || '') !== String(settings.whisperCommand || '');
+      if (providerChanged || whisperCommandChanged) {
         try {
           speechService.initializeClient();
           this.speechAvailable = speechService.isAvailable
             ? speechService.isAvailable()
             : false;
+          // Broadcast so any open window (settings, overlay, chat)
+          // can react immediately — especially the main overlay's
+          // mic button, which queries availability on load.
+          const { BrowserWindow } = require("electron");
+          BrowserWindow.getAllWindows().forEach((win) => {
+            if (!win.isDestroyed()) {
+              win.webContents.send("speech-availability", { available: this.speechAvailable });
+            }
+          });
+          logger.info('Speech service reinitialized after settings change', {
+            providerChanged,
+            whisperCommandChanged,
+            speechAvailable: this.speechAvailable,
+          });
         } catch (e) {
           logger.warn("Failed to reinitialize speech service after settings change", {
             error: e.message

onboarding.js

@@ -265,10 +265,20 @@
   }
 
   async function runWhisperInstall() {
+    const btn = document.getElementById('installWhisperBtn');
     installLog.textContent = '';
     setDetectStatus('testing', 'Installing');
     appendLog('Starting install…');
 
+    // Lock the button while installing so the user can't double-click
+    // and spawn parallel installs. Change the label to "Installing…"
+    // with a spinner so they see real progress.
+    if (btn) {
+      btn.disabled = true;
+      btn.dataset.originalHtml = btn.dataset.originalHtml || btn.innerHTML;
+      btn.innerHTML = '<span class="spinner"></span> Installing…';
+    }
+
     // Subscribe to streamed progress lines from the main process.
     // `installWhisper()` only returns once install completes; live
     // output comes through `onInstallProgress` events.
@@ -286,13 +296,29 @@
         detectCmd.textContent = r.command;
         setDetectStatus('success', 'Installed');
         appendLog(`\n✓ ${r.message}`);
+        if (btn) {
+          // Keep button disabled — install is done. Show a checkmark
+          // so the user sees the final state at a glance.
+          btn.innerHTML = '<i class="fas fa-check-circle"></i> Installed';
+          btn.classList.remove('primary');
+          btn.classList.add('success');
+        }
       } else {
         setDetectStatus('error', 'Install failed');
         appendLog(`\n✗ ${r.message}`);
+        // Restore the button so the user can retry.
+        if (btn) {
+          btn.disabled = false;
+          btn.innerHTML = btn.dataset.originalHtml || '<i class="fas fa-download"></i> Install Whisper now';
+        }
       }
     } catch (e) {
       setDetectStatus('error', 'Install error');
       appendLog(`\n! ${e.message || e}`);
+      if (btn) {
+        btn.disabled = false;
+        btn.innerHTML = btn.dataset.originalHtml || '<i class="fas fa-download"></i> Install Whisper now';
+      }
     } finally {
       if (progressHandler && window.electronAPI.removeAllListeners) {
         try { window.electronAPI.removeAllListeners('install-progress'); } catch (_) { /* ignore */ }

src/services/llm.service.js

@@ -1054,20 +1054,52 @@ Remember: Be intelligent about filtering - only provide detailed responses when
       };
     } catch (error) {
       const errorAnalysis = this.analyzeError(error);
-      logger.error('Connection test failed', { 
+      logger.error('Connection test failed', {
         error: error.message,
         errorAnalysis
       });
-      
-      return { 
-        success: false, 
-        error: error.message,
+
+      // Map raw SDK errors to user-friendly messages. The wizard only
+      // surfaces `error`, so any raw "[GoogleGenerativeAI Error]: ..."
+      // string would land in the UI verbatim.
+      const friendlyError = this._friendlyTestError(error, errorAnalysis);
+
+      return {
+        success: false,
+        error: friendlyError,
+        errorType: errorAnalysis?.type || 'UNKNOWN',
         errorAnalysis,
         networkConnectivity: await this.checkNetworkConnectivity().catch(() => null)
       };
     }
   }
 
+  /**
+   * Translate raw SDK / network errors into something a user can act on.
+   */
+  _friendlyTestError(error, analysis) {
+    const type = analysis?.type;
+    const raw = (error?.message || '').toLowerCase();
+
+    if (type === 'NETWORK_ERROR' || raw.includes('fetch failed') || raw.includes('enotfound')) {
+      return 'Cannot reach Google servers. Check your internet connection, firewall, or VPN settings.';
+    }
+    if (type === 'AUTH_ERROR' || raw.includes('api key') || raw.includes('401') || raw.includes('403')) {
+      return 'Invalid API key or insufficient permissions. Double-check the key at aistudio.google.com/apikey.';
+    }
+    if (type === 'RATE_LIMIT_ERROR' || raw.includes('429') || raw.includes('quota')) {
+      return 'Rate limit or quota exceeded. Wait a moment or check your Google Cloud billing.';
+    }
+    if (type === 'TIMEOUT_ERROR') {
+      return 'Request timed out. The Google API may be slow or unreachable right now.';
+    }
+    if (type === 'MODEL_ERROR' || raw.includes('model') || raw.includes('404')) {
+      return 'The configured Gemini model is unavailable. Try a different model in Settings.';
+    }
+    // Fall back to a stripped-down raw message (no SDK prefix noise)
+    return (error?.message || 'Connection failed').replace(/^\[GoogleGenerativeAI Error\]:\s*/i, '');
+  }
+
   updateApiKey(newApiKey) {
     process.env.GEMINI_API_KEY = newApiKey;
     this.isInitialized = false;