fix(settings): reload settings on system resume to prevent reset after sleep

When the computer goes to sleep and wakes up, some settings like
MAX OUTPUT LINES PER RESPONSE were resetting to default values.

This adds Electron's powerMonitor to detect system resume events and
notify the renderer to reload settings from persistent storage:

- Add powerMonitor.on('resume') listener in main process
- Add onSystemResume IPC handler in preload API
- Refactor loadSettings to useCallback for reuse
- Register system resume listener in useSettings hook

Closes #269

Author: pedramamini

src/__tests__/renderer/hooks/useSettings.test.ts

@@ -1295,6 +1295,61 @@ describe('useSettings', () => {
 		});
 	});
 
+	describe('system resume behavior', () => {
+		it('should register onSystemResume listener on mount', async () => {
+			renderHook(() => useSettings());
+
+			expect(window.maestro.app.onSystemResume).toHaveBeenCalled();
+		});
+
+		it('should reload settings when system resumes from sleep', async () => {
+			// Capture the callback passed to onSystemResume
+			let resumeCallback: (() => void) | undefined;
+			vi.mocked(window.maestro.app.onSystemResume).mockImplementation((cb) => {
+				resumeCallback = cb;
+				return () => {};
+			});
+
+			// Initial load with default settings
+			vi.mocked(window.maestro.settings.getAll).mockResolvedValue({
+				maxOutputLines: 25,
+			});
+
+			const { result } = renderHook(() => useSettings());
+			await waitForSettingsLoaded(result);
+
+			expect(result.current.maxOutputLines).toBe(25);
+
+			// Simulate settings change while asleep (user may have changed via another method)
+			// In the real bug, the setting was being reset, so simulate that by changing
+			// the mock to return a different value on next load
+			vi.mocked(window.maestro.settings.getAll).mockResolvedValue({
+				maxOutputLines: 0, // 0 means "ALL" in the UI
+			});
+
+			// Trigger system resume
+			await act(async () => {
+				resumeCallback?.();
+				// Allow async operations to complete
+				await new Promise((resolve) => setTimeout(resolve, 0));
+			});
+
+			// Settings should be reloaded with the new value
+			expect(result.current.maxOutputLines).toBe(0);
+		});
+
+		it('should cleanup onSystemResume listener on unmount', async () => {
+			const cleanupFn = vi.fn();
+			vi.mocked(window.maestro.app.onSystemResume).mockReturnValue(cleanupFn);
+
+			const { unmount } = renderHook(() => useSettings());
+
+			unmount();
+
+			expect(cleanupFn).toHaveBeenCalled();
+		});
+	});
+
 	describe('edge cases', () => {
 		it('should handle undefined values from settings.getAll gracefully', async () => {
 			// All settings return empty object (uses defaults)

src/__tests__/setup.ts

@@ -480,6 +480,12 @@ const mockMaestro = {
 		onUpdated: vi.fn().mockReturnValue(() => {}),
 		onContributionStarted: vi.fn().mockReturnValue(() => {}),
 	},
+	app: {
+		onQuitConfirmationRequest: vi.fn().mockReturnValue(() => {}),
+		confirmQuit: vi.fn(),
+		cancelQuit: vi.fn(),
+		onSystemResume: vi.fn().mockReturnValue(() => {}),
+	},
 };
 
 // Only mock window.maestro if window exists (jsdom environment)

src/main/index.ts

@@ -1,4 +1,4 @@
-import { app, BrowserWindow } from 'electron';
+import { app, BrowserWindow, powerMonitor } from 'electron';
 import path from 'path';
 import os from 'os';
 import crypto from 'crypto';
@@ -358,6 +358,15 @@ app.whenReady().then(async () => {
 			createWindow();
 		}
 	});
+
+	// Listen for system resume (after sleep/suspend) and notify renderer
+	// This allows the renderer to refresh settings that may have been reset
+	powerMonitor.on('resume', () => {
+		logger.info('System resumed from sleep/suspend', 'PowerMonitor');
+		if (isWebContentsAvailable(mainWindow)) {
+			mainWindow.webContents.send('app:systemResume');
+		}
+	});
 });
 
 app.on('window-all-closed', () => {

src/main/preload/system.ts

@@ -191,6 +191,15 @@ export function createAppApi() {
 		cancelQuit: () => {
 			ipcRenderer.send('app:quitCancelled');
 		},
+		/**
+		 * Listen for system resume event (after sleep/suspend)
+		 * Used to refresh settings that may have been reset during sleep
+		 */
+		onSystemResume: (callback: () => void) => {
+			const handler = () => callback();
+			ipcRenderer.on('app:systemResume', handler);
+			return () => ipcRenderer.removeListener('app:systemResume', handler);
+		},
 	};
 }
 

src/renderer/global.d.ts

@@ -954,6 +954,7 @@ interface MaestroAPI {
 		onQuitConfirmationRequest: (callback: () => void) => () => void;
 		confirmQuit: () => void;
 		cancelQuit: () => void;
+		onSystemResume: (callback: () => void) => () => void;
 	};
 	logger: {
 		log: (

src/renderer/hooks/settings/useSettings.ts

@@ -1296,10 +1296,10 @@ export function useSettings(): UseSettingsReturn {
 		window.maestro.settings.set('sshRemoteHonorGitignore', value);
 	}, []);
 
-	// Load settings from electron-store on mount
+	// Load settings from electron-store
+	// This function is called on mount and on system resume (after sleep/suspend)
 	// PERF: Use batch loading to reduce IPC calls from ~60 to 3
-	useEffect(() => {
-		const loadSettings = async () => {
+	const loadSettings = useCallback(async () => {
 			try {
 				// Batch load all settings in a single IPC call
 				const allSettings = (await window.maestro.settings.getAll()) as Record<string, unknown>;
@@ -1725,9 +1725,22 @@ export function useSettings(): UseSettingsReturn {
 				// Mark settings as loaded even if there was an error (use defaults)
 				setSettingsLoaded(true);
 			}
-		};
+		}, []);
+
+	// Load settings on mount
+	useEffect(() => {
 		loadSettings();
-	}, []);
+	}, [loadSettings]);
+
+	// Reload settings when system resumes from sleep/suspend
+	// This ensures settings like maxOutputLines aren't reset to defaults
+	useEffect(() => {
+		const cleanup = window.maestro.app.onSystemResume(() => {
+			console.log('[Settings] System resumed from sleep, reloading settings');
+			loadSettings();
+		});
+		return cleanup;
+	}, [loadSettings]);
 
 	// Apply font size to HTML root element so rem-based Tailwind classes scale
 	// Only apply after settings are loaded to prevent layout shift from default->saved font size