relay-companion 0.1.86 → 0.1.87

package/overlay/main.cjs

@@ -20,7 +20,22 @@
 //   - Mutations (accept/reject/approve/decline/answer) call RelayClient with device-token
 //     auth. ack/mark-read writes state.json directly (atomic temp+rename).
 
-const { app, BrowserWindow, Menu, Tray, ipcMain, nativeImage, powerMonitor, shell, screen, systemPreferences } = require("electron");
+const { app, BrowserWindow, Menu, Tray, ipcMain, nativeImage, powerMonitor, powerSaveBlocker, shell, screen, systemPreferences } = require("electron");
+
+// The pill is an accessory window that spends most of its life idle behind other
+// apps. Both Chromium and macOS treat that as "safe to throttle", and the cost is
+// paid by the user: the FIRST click after an idle spell waits for the renderer to
+// spin back up (reported as "slow, then fast, then slow again"). These switches
+// stop the OS/engine from backgrounding a window the user can still see and click.
+// They must be set before app ready.
+app.commandLine.appendSwitch("disable-renderer-backgrounding");
+app.commandLine.appendSwitch("disable-background-timer-throttling");
+app.commandLine.appendSwitch("disable-backgrounding-occluded-windows");
+if (process.platform === "darwin") {
+  // macOS reports an always-on-top overlay as occluded far more eagerly than it
+  // deserves; Chromium then backgrounds the renderer behind the user's back.
+  app.commandLine.appendSwitch("disable-features", "MacWebContentsOcclusion");
+}
 
 // A closed stdout/stderr pipe must never kill the pill. When whatever launched
 // us goes away (launchd log rotation, a terminal that spawned `relay pill`, a
@@ -961,10 +976,32 @@ function idleSecondsSafe() {
 
 // Renderer timer throttling is disabled ONLY while a card's dwell runs: the
 // 7s fold timer must be honest, but an idle hidden overlay should coast.
-function setThrottlingForShow(showActive) {
+// Responsiveness policy: a renderer the user can SEE is never throttled. The
+// earlier rule (throttle unless a notification dwell was running) saved idle CPU
+// but made the first interaction after any quiet spell slow — the user feels a
+// dead click far more than a fraction of a percent of CPU. Throttling now applies
+// only while the window is genuinely hidden, where nobody can click it.
+let appSuspensionBlockerId = null;
+function applyThrottlingPolicy() {
+  const visible = Boolean(win && !win.isDestroyed() && win.isVisible());
   try {
-    if (win && !win.isDestroyed()) win.webContents.setBackgroundThrottling(!showActive);
+    if (win && !win.isDestroyed()) win.webContents.setBackgroundThrottling(!visible);
   } catch {}
+  // macOS App Nap suspends whole accessory processes; hold it off only while the
+  // pill is on screen, so an idle hidden pill still costs nothing.
+  try {
+    if (visible && appSuspensionBlockerId === null) {
+      appSuspensionBlockerId = powerSaveBlocker.start("prevent-app-suspension");
+    } else if (!visible && appSuspensionBlockerId !== null) {
+      powerSaveBlocker.stop(appSuspensionBlockerId);
+      appSuspensionBlockerId = null;
+    }
+  } catch {}
+}
+// Kept for the notification-dwell call sites: dwell timing still demands an
+// unthrottled renderer, and the visibility rule already guarantees it.
+function setThrottlingForShow() {
+  applyThrottlingPolicy();
 }
 
 function abortCurrentShow(reason) {
@@ -2340,6 +2377,9 @@ function showOverlayWindow({ force = false, reposition = true } = {}) {
     // window would silently flip it click-through while the renderer still thinks
     // the pointer handshake is live — a dead card until the pointer re-enters.
     win.setIgnoreMouseEvents(true, { forward: true });
+    // Visible again: unthrottle immediately, so the FIRST click after this is as
+    // fast as the tenth (the "slow, then fast, then slow again" report).
+    applyThrottlingPolicy();
     win.webContents.send("shown");
     // Becoming visible is when host-running freshness starts mattering for
     // click routing again; the hidden poll cadence is slow, so take one
@@ -3001,7 +3041,7 @@ function hideFromTray() {
   writeOverlayPrefs();
   ghostActive = false;
   trayForcedVisible = false;
-  if (win && !win.isDestroyed() && win.isVisible()) win.hide();
+  if (win && !win.isDestroyed() && win.isVisible()) { win.hide(); applyThrottlingPolicy(); }
   syncTray();
   writePillStatus();
 }
@@ -3291,7 +3331,7 @@ ipcMain.on("relay:dismiss", () => {
   writeOverlayPrefs();
   ghostActive = false;
   trayForcedVisible = false;
-  if (win && !win.isDestroyed() && win.isVisible()) win.hide();
+  if (win && !win.isDestroyed() && win.isVisible()) { win.hide(); applyThrottlingPolicy(); }
   writePillStatus();
 });
 // The renderer finished registering every IPC listener (last line of its

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "relay-companion",
-  "version": "0.1.86",
+  "version": "0.1.87",
   "description": "Relay companion for ordinary messages, with dormant coordination features available only by explicit opt-in.",
   "type": "module",
   "bin": {