fix(desktop): allow updater-controlled relaunch

[0x4bs3nt]

What Changed

• Listen for Electron’s native before-quit-for-update event.
• Allow the subsequent updater-controlled quit to proceed without calling preventDefault().
• Preserve the existing graceful-shutdown behavior for ordinary quits.
• Add regression coverage for macOS, Windows, and Linux.

Why

T3 Code’s lifecycle handler cancelled every before-quit event, including quits initiated by Electron’s updater. On macOS, this interrupted the native install-and-relaunch sequence, causing “Restart to update” to close the app without reopening it.

Recognizing Electron’s dedicated update-quit signal keeps normal shutdown safeguards intact while allowing the updater to own its install and relaunch sequence.

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes — not applicable
• I included a video for animation/interaction changes — not applicable

---

Note

Low Risk
Small, targeted change to quit handling with platform regression tests; normal user-initiated quits still use the existing graceful shutdown path.

Overview
Fixes “Restart to update” closing the app without relaunching (especially on macOS) by not treating updater-driven quits like user quits that need graceful shutdown.

The desktop app was calling preventDefault() on every before-quit unless a normal quit had already been allowed. Electron’s updater first emits before-quit-for-update, then before-quit as part of install/relaunch—blocking that second event breaks the updater’s sequence.

ElectronApp now exposes onBeforeQuitForUpdate, a scoped subscription to autoUpdater’s before-quit-for-update (register on acquire, remove on release). DesktopLifecycle sets an updaterQuitAllowed flag when that fires so the next before-quit is allowed through without preventDefault() or the shutdown-and-retry-quit path. Ordinary quits are unchanged.

Regression tests cover scoped updater listeners, lifecycle behavior on darwin/win32/linux, and test doubles for the new service method.

^{Reviewed by Cursor Bugbot for commit bed368e. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Allow updater-controlled relaunch by not blocking before-quit after before-quit-for-update

• Adds onBeforeQuitForUpdate to the ElectronApp service interface, implemented via Effect.acquireRelease to scope a listener on Electron.autoUpdater's before-quit-for-update event in ElectronApp.ts.
• Introduces an updaterQuitAllowed flag in DesktopLifecycle.ts that is set when the updater fires before-quit-for-update, allowing the subsequent before-quit event to proceed rather than being cancelled.
• Behavioral Change: previously, all before-quit events were blocked unless quitAllowed was set via the normal quit flow; now updater-initiated quits bypass that block.

^{Macroscope summarized bed368e.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: c0584438-597a-4206-a0f4-b5694f405017

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Approvability

Verdict: Approved

Straightforward bug fix that allows Electron's auto-updater to complete its relaunch sequence by adding a flag to bypass quit-prevention when the updater triggers the quit. Changes are self-contained with comprehensive test coverage.

^{You can customize Macroscope's approvability policy. Learn more.}