Skip to content

Fix update-related issues: macOS crash, duplicate notifications, version display - #37

Merged
nwparker merged 1 commit into
mainfrom
nwparker/update-fixes
Mar 22, 2026
Merged

nwparker merged 1 commit into
mainfrom
nwparker/update-fixes

Conversation

@nwparker

Copy link
Copy Markdown
Contributor

Summary

  • macOS crash on restart: Use app.relaunch() + app.quit() instead of autoUpdater.quitAndInstall() on macOS, which calls app.exit() and bypasses the normal quit lifecycle causing the "quit unexpectedly" dialog
  • Duplicate notifications: Remove redundant sendStatus({ state: 'checking' }) calls from checkForUpdates() and checkForUpdatesFromMenu() — the checking-for-update event handler already emits this status, so it was firing twice
  • Version display: Show current app version in the Settings > Updates section via new updater:getVersion IPC channel

Fixes #35

Test plan

  • On macOS, trigger "Restart to Update" and verify no crash dialog appears
  • Check for updates from Settings and verify only one "Checking for updates..." toast appears
  • Check for updates from menu bar and verify only one toast appears
  • Verify current version number displays in Settings > Updates section
  • Verify update flow still works end-to-end on Windows/Linux (NSIS path unchanged)

🤖 Generated with Claude Code

…ion display

- Fix macOS "quit unexpectedly" crash on update-and-restart by using
  app.relaunch() + app.quit() instead of autoUpdater.quitAndInstall()
  which bypasses the normal quit lifecycle
- Fix duplicate "checking for updates" notifications by removing
  redundant sendStatus calls (the electron-updater event handler
  already emits the checking state)
- Show current app version in the Settings update section

Fixes #35

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@nwparker
nwparker force-pushed the nwparker/update-fixes branch from 3069f83 to 096a78e Compare March 22, 2026 22:54
@nwparker
nwparker merged commit c17a985 into main Mar 22, 2026
Jinwoo-H added a commit that referenced this pull request Sep 5, 2026
enrick-dev pushed a commit to enrick-dev/orca that referenced this pull request Oct 15, 2026
…ion display (stablyai#37)

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[P2] Duplicated "checking for updates"

1 participant