fix: review — toggle OFF returns empty, persist scanned statuses to files

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

Author: grimmerk

src/main.ts

@@ -33,6 +33,7 @@ import {
   readAllStatuses,
   watchStatusDir,
   scanInitialStatuses,
+  writeStatusFile,
   SessionStatus,
 } from './session-status-hooks';
 import {
@@ -1836,6 +1837,10 @@ ipcMain.on('set-session-status-hooks-enabled', async (_event, enabled: boolean)
 });
 
 ipcMain.handle('get-session-statuses', async () => {
+  // Return empty when hooks disabled — all dots revert to purple
+  const enabled = (await settings.get('session-status-hooks')) !== false;
+  if (!enabled) return {};
+
   const fileStatuses = readAllStatuses();
   const obj: Record<string, SessionStatus> = {};
   fileStatuses.forEach((v, k) => { obj[k] = v; });
@@ -1854,7 +1859,11 @@ ipcMain.handle('get-session-statuses', async () => {
 
     if (sessionsWithoutStatus.length > 0) {
       const scanned = await scanInitialStatuses(sessionsWithoutStatus);
-      scanned.forEach((v, k) => { obj[k] = v; });
+      scanned.forEach((v, k) => {
+        obj[k] = v;
+        // Persist scanned status to file so fs.watch treats all statuses uniformly
+        writeStatusFile(k, v as string);
+      });
     }
   } catch {}
 

src/session-status-hooks.ts

@@ -296,4 +296,17 @@ export const scanInitialStatuses = async (
   return statuses;
 };
 
+/**
+ * Write a status file for a session (used to persist JSONL-scanned statuses).
+ */
+export const writeStatusFile = (sessionId: string, status: string): void => {
+  try {
+    fs.mkdirSync(STATUS_DIR, { recursive: true });
+    const tmpFile = path.join(STATUS_DIR, `.${sessionId}.tmp`);
+    const targetFile = path.join(STATUS_DIR, `${sessionId}.json`);
+    fs.writeFileSync(tmpFile, JSON.stringify({ status, timestamp: Math.floor(Date.now() / 1000), cwd: '' }));
+    fs.renameSync(tmpFile, targetFile);
+  } catch {}
+};
+
 export { STATUS_DIR, HOOK_SCRIPT_PATH };

src/switcher-ui.tsx

@@ -512,8 +512,8 @@ function SwitcherApp() {
       if (statuses) setSessionStatuses(statuses);
     });
     window.electronAPI.onSessionStatusesUpdated((_event: any, statuses: Record<string, string>) => {
-      // Merge instead of replace — preserve JSONL-scanned statuses for sessions without status files
-      setSessionStatuses((prev) => ({ ...prev, ...statuses }));
+      // Use disk snapshot as source of truth — clears removed sessions
+      setSessionStatuses(statuses);
     });
 
     window.electronAPI.onCheckTerminalAndHide(() => {