fix: review — empty snapshot on toggle OFF, safe settings parse, replace on focus

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

Author: grimmerk

src/main.ts

@@ -1834,6 +1834,8 @@ ipcMain.on('set-session-status-hooks-enabled', async (_event, enabled: boolean)
       statusWatcherCleanup();
       statusWatcherCleanup = null;
     }
+    // Clear renderer dots immediately
+    switcherWindow?.webContents.send('session-statuses-updated', {});
   }
 });
 

src/session-status-hooks.ts

@@ -63,14 +63,15 @@ export const installHooks = (): void => {
   // Create status directory
   fs.mkdirSync(STATUS_DIR, { recursive: true });
 
-  // Read existing settings
+  // Read existing settings — abort if file exists but can't be parsed (don't overwrite)
   let settings: any = {};
   try {
     if (fs.existsSync(SETTINGS_PATH)) {
       settings = JSON.parse(fs.readFileSync(SETTINGS_PATH, 'utf-8'));
     }
   } catch {
-    settings = {};
+    // Can't parse existing file — don't risk overwriting user's settings
+    return;
   }
 
   if (!settings.hooks) settings.hooks = {};

src/switcher-ui.tsx

@@ -541,7 +541,7 @@ function SwitcherApp() {
       }
       // Refresh session statuses on window focus
       window.electronAPI.getSessionStatuses().then((statuses: Record<string, string | null>) => {
-        if (statuses) setSessionStatuses((prev) => ({ ...prev, ...statuses }));
+        if (statuses) setSessionStatuses(statuses);
       });
       // Refresh display mode setting
       window.electronAPI.getSessionDisplayMode().then((mode: string) => {