Skip to content

fix(tauri): restore macOS quit shortcut - #633

Merged
shantur merged 1 commit into
NeuralNomadsAI:devfrom
pascalandr:fix/issue-590-macos-quit-shortcut
Aug 3, 2026
Merged

fix(tauri): restore macOS quit shortcut#633
shantur merged 1 commit into
NeuralNomadsAI:devfrom
pascalandr:fix/issue-590-macos-quit-shortcut

Conversation

@pascalandr

Copy link
Copy Markdown
Contributor

Summary

  • add the standard Cmd+Q accelerator to the Tauri macOS Quit menu item
  • keep the existing custom quit event so shutdown still flushes renderer state and stops the managed CLI
  • leave Windows Ctrl+W tab closing and Alt+F4 app closing unchanged

Why

The macOS menu used a plain text item with no accelerator. Tauri's predefined native Quit item would restore Cmd+Q, but it bypasses the custom menu event and could skip CodeNomad's coordinated shutdown. A regular menu item with CmdOrCtrl+Q restores the shortcut while preserving cleanup.

Validation

  • 84 Tauri tests pass
  • cargo check --locked passes

ustfmt check passes for main.rs

  • git diff --check passes
  • Gatekeeper review: zero findings
  • macOS runtime unavailable; accelerator and shutdown routing verified against the pinned Tauri/muda behavior

Closes #590

Attach CmdOrCtrl+Q to the existing custom quit menu item so macOS receives the standard Command+Q accelerator.

Keep the custom menu event instead of using Tauri's native predefined quit action, ensuring ExitRequested still reaches CodeNomad's coordinated renderer flush and CLI shutdown path. Windows Ctrl+W and Alt+F4 behavior is unchanged.

Validated with all 84 Tauri tests, cargo check --locked, rustfmt, diff checks, and repeated Gatekeeper review.

@pascalandr pascalandr left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gatekeeper review: zero findings. Verified Cmd+Q accelerator routing through the existing quit menu event and guarded ExitRequested shutdown flow, no frontend shortcut conflict, and unchanged Windows Ctrl+W/Alt+F4 behavior. All 84 Tauri tests, cargo check --locked, rustfmt for main.rs, and diff checks pass.

@pascalandr

Copy link
Copy Markdown
Contributor Author

@shantur LGTM

@shantur
shantur merged commit e7a27c9 into NeuralNomadsAI:dev Aug 3, 2026
4 of 6 checks passed
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.

Bug: Cmd+Q keyboard shortcut does not quit CodeNomad

2 participants