fix(desktop): route prevent-sleep writes through the side-effecting IPC

The generic store-set/store-delete/store-clear IPCs could write the
preventSleep key in the settings store without starting/stopping the
powerSaveBlocker, desyncing the persisted setting from the running
blocker (a string "false" even resolved back to enabled on next
launch).

- store-set: reject writes to the preventSleep key; the dedicated
  set-prevent-sleep IPC is now the single write path
- store-delete/store-clear: after touching the settings store,
  re-sync the blocker via syncPowerSaveBlocker() so the reset to the
  default-on behavior takes effect immediately

Verified with the macOS prevent-sleep smoke (4/4) plus a Playwright
guard check: rejected store-set leaves the assertion running, and
store-delete/store-clear restore the NoIdleSleepAssertion.

Signed-off-by: thomas-yanga <odie_majere@outlook.com>

Author: thomas-yanga

packages/desktop/src/main/ipc.ts

@@ -12,7 +12,8 @@ import { archivePath, copyPath, extractPath, guardFileOpCall, movePath, removePa
 import { fileLog, isTracked } from "./git"
 import { getStore } from "./store"
 import { getPinchZoomEnabled, setPinchZoomEnabled, setTitlebar, updateTitlebar } from "./windows"
-import { getPreventSleepEnabled, setPreventSleepEnabled } from "./power"
+import { getPreventSleepEnabled, setPreventSleepEnabled, syncPowerSaveBlocker } from "./power"
+import { PREVENT_SLEEP_KEY, SETTINGS_STORE } from "./store-keys"
 import { browserView } from "./browser-view"
 import type { UpdaterController } from "./updater-controller"
 import { createUpdaterSubscriptions } from "./updater-subscriptions"
@@ -93,13 +94,21 @@ export function registerIpcHandlers(deps: Deps) {
     }
   })
   ipcMain.handle("store-set", (_event: IpcMainInvokeEvent, name: string, key: string, value: string) => {
+    // preventSleep drives a live powerSaveBlocker side effect; routing writes through the
+    // dedicated set-prevent-sleep IPC keeps the blocker in sync with the persisted value.
+    if (name === SETTINGS_STORE && key === PREVENT_SLEEP_KEY) {
+      throw new Error(`"${PREVENT_SLEEP_KEY}" must be written via the set-prevent-sleep IPC`)
+    }
     getStore(name).set(key, value)
   })
   ipcMain.handle("store-delete", (_event: IpcMainInvokeEvent, name: string, key: string) => {
     getStore(name).delete(key)
+    // Deleting the key resets it to the default-on behavior; re-sync so the blocker matches.
+    if (name === SETTINGS_STORE && key === PREVENT_SLEEP_KEY) syncPowerSaveBlocker()
   })
   ipcMain.handle("store-clear", (_event: IpcMainInvokeEvent, name: string) => {
     getStore(name).clear()
+    if (name === SETTINGS_STORE) syncPowerSaveBlocker()
   })
   ipcMain.handle("store-keys", (_event: IpcMainInvokeEvent, name: string) => {
     const store = getStore(name)

packages/desktop/src/main/power.ts

@@ -14,7 +14,13 @@ export function getPreventSleepEnabled(): boolean {
 
 export function setPreventSleepEnabled(enabled: boolean): void {
   getStore().set(PREVENT_SLEEP_KEY, enabled)
-  if (enabled) {
+  syncPowerSaveBlocker()
+}
+
+// Reconciles the running blocker with the persisted setting. Must be called after any write path
+// that can change the stored value outside setPreventSleepEnabled (generic store delete/clear).
+export function syncPowerSaveBlocker(): void {
+  if (getPreventSleepEnabled()) {
     startPowerSaveBlocker()
     return
   }