fix: guard shortcut cleanup on tab re-click, show after bootstrap

- Skip shortcut resume when re-clicking active Shortcuts tab
- Move showSwitcherWindow() to after bootstrap() + loadUserSettings()
  (was using setTimeout(100) which didn't guarantee readiness)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: grimmerk

src/main.ts

@@ -1081,11 +1081,6 @@ const trayToggleEvtHandler = async () => {
   }
 
   switcherWindow = createSwitcherWindow();
-  // Normal mode: show window after bootstrap (server must be ready for API calls)
-  if (appMode === 'normal') {
-    // Delay show to ensure bootstrap() has completed (runs earlier in this block)
-    setTimeout(() => showSwitcherWindow(), 100);
-  }
   if (isDebug) {
     console.log('when ready');
   }
@@ -1280,6 +1275,11 @@ const trayToggleEvtHandler = async () => {
   // Load user settings
   await loadUserSettings();
 
+  // Normal mode: show window after bootstrap + settings loaded (server ready for API calls)
+  if (appMode === 'normal') {
+    showSwitcherWindow();
+  }
+
   let title = '';
   if (!isDebug) {
     title = ``;

src/popup.tsx

@@ -328,6 +328,7 @@ const PopupDefaultExample = ({
               <button
                 key={tab}
                 onClick={() => {
+                  if (tab === settingsTab) return;
                   // Resume any paused shortcut when switching away from Shortcuts tab
                   if (editingShortcut) {
                     window.electronAPI.resumeShortcut(editingShortcut);