Overhaul native menus on macOS and Linux

- Build all menus from scratch, no Menu::default() patching
- Strip system-injected Edit junk (Writing Tools, AutoFill, Dictation, Emoji & Symbols) via objc2 cleanup pass
- Register Help menu via NSApplication.setHelpMenu for search
- Route ⌘Q, ⌘H, ⌥⌘H through PredefinedMenuItems (native-only)
- Add all file operations to File menu (Copy, Move, Delete, etc.)
- Dedicated Tab submenu, shared Sort by submenu
- Rebuild viewer menu from scratch (minimal, no file ops)
- Align Linux menus: same structure, GTK mnemonics, no F-key accelerators (GTK intercepts them)
- Unify dispatch: single "execute-command" event replaces ~15 individual Tauri listeners
- Focus guard on all file-scoped commands (skip when main window not focused)
- Context-aware menus: file ops gray out when Settings/viewer has focus
- Extend accelerator sync to all ~28 menu commands (was only 2)
- Add app.licenseKey command, migrate MCP emitters to unified path
- Update CLAUDE.md files for commands, shortcuts, and architecture

Author: vdavid

apps/desktop/src-tauri/Cargo.lock

@@ -114,13 +114,14 @@ core-services = "1.0.0"
 icns = "0.3.1"
 plist = "1.8.0"
 urlencoding = "2.1.3"
-objc2 = { version = "0.6", features = ["std"] }
+objc2 = { version = "0.6", features = ["std", "exception"] }
 objc2-foundation = { version = "0.3", features = [
     "NSURL", "NSString", "NSDictionary", "NSDate", "NSArray", "NSValue", "NSError",
     "NSFileManager", "NSNotification",
 ] }
 objc2-app-kit = { version = "0.3", features = [
     "NSDragging", "NSDraggingItem", "NSImage", "NSColor", "NSColorSpace",
+    "NSApplication", "NSMenu", "NSMenuItem", "NSRunningApplication",
 ] }
 block2 = "0.6"
 security-framework = "3.2"

apps/desktop/src-tauri/Cargo.toml

@@ -17,7 +17,7 @@ immediately to business-logic modules. No significant logic lives here.
 | `icons.rs` | File icons | `get_icons`, `refresh_directory_icons`, cache clear |
 | `rename.rs` | Rename / trash | `move_to_trash` (delegates to `write_operations::trash::move_to_trash_sync`), `check_rename_permission`, `check_rename_validity`, `rename_file` |
 | `file_viewer.rs` | File viewer | Session lifecycle, line search, word wrap, menu state |
-| `ui.rs` | UI / menu | Context menu, Finder reveal, clipboard, Quick Look, Get Info, view mode |
+| `ui.rs` | UI / menu | Context menu, Finder reveal, clipboard, Quick Look, Get Info, view mode, `set_menu_context` (enables/disables file-scoped menu items based on window focus) |
 | `settings.rs` | Settings | Port availability check, watcher debounce setting, menu accelerator updates |
 | `licensing.rs` | Licensing | Status query, activation, expiry, reminder, key validation |
 | `indexing.rs` | Drive index | `start_drive_index`, `stop_drive_index`, `get_index_status`, `get_dir_stats`, `get_dir_stats_batch`, `prioritize_dir`, `cancel_nav_priority`, `clear_drive_index`, `set_indexing_enabled`. Uses `State<IndexManagerState>`. |

apps/desktop/src-tauri/src/commands/CLAUDE.md

@@ -6,7 +6,10 @@ use tauri::{AppHandle, Manager};
 
 use crate::file_system::update_debounce_ms;
 use crate::ignore_poison::IgnorePoison;
-use crate::menu::{MenuState, frontend_shortcut_to_accelerator, update_view_mode_accelerator};
+use crate::menu::{
+    MenuState, command_id_to_menu_id, frontend_shortcut_to_accelerator, update_menu_item_accelerator,
+    update_view_mode_accelerator,
+};
 #[cfg(target_os = "macos")]
 use crate::network::mdns_discovery::update_resolve_timeout;
 
@@ -53,7 +56,6 @@ pub fn update_service_resolve_timeout(_timeout_ms: u64) {
 
 /// Update menu accelerator for a command.
 /// Called from frontend when keyboard shortcuts are changed.
-/// Currently supports: view.fullMode, view.briefMode
 #[tauri::command]
 pub fn update_menu_accelerator(app: AppHandle, command_id: &str, shortcut: &str) -> Result<(), String> {
     let menu_state = app.state::<MenuState<tauri::Wry>>();
@@ -62,8 +64,8 @@ pub fn update_menu_accelerator(app: AppHandle, command_id: &str, shortcut: &str)
     let accelerator = frontend_shortcut_to_accelerator(shortcut);
 
     match command_id {
+        // View mode CheckMenuItems need special handling to preserve checked state
         "view.fullMode" => {
-            // Get current checked state before updating
             let is_checked = menu_state
                 .view_mode_full
                 .lock_ignore_poison()
@@ -74,12 +76,10 @@ pub fn update_menu_accelerator(app: AppHandle, command_id: &str, shortcut: &str)
             let new_item = update_view_mode_accelerator(&app, &menu_state, true, accelerator.as_deref(), is_checked)
                 .map_err(|e| format!("Failed to update Full view accelerator: {e}"))?;
 
-            // Update the reference in MenuState
             *menu_state.view_mode_full.lock_ignore_poison() = Some(new_item);
             Ok(())
         }
         "view.briefMode" => {
-            // Get current checked state before updating
             let is_checked = menu_state
                 .view_mode_brief
                 .lock_ignore_poison()
@@ -90,13 +90,16 @@ pub fn update_menu_accelerator(app: AppHandle, command_id: &str, shortcut: &str)
             let new_item = update_view_mode_accelerator(&app, &menu_state, false, accelerator.as_deref(), is_checked)
                 .map_err(|e| format!("Failed to update Brief view accelerator: {e}"))?;
 
-            // Update the reference in MenuState
             *menu_state.view_mode_brief.lock_ignore_poison() = Some(new_item);
             Ok(())
         }
+        // All other commands: use the generic HashMap-based update
         _ => {
+            if let Some(menu_id) = command_id_to_menu_id(command_id) {
+                update_menu_item_accelerator(&app, &menu_state, menu_id, accelerator.as_deref())
+                    .map_err(|e| format!("Failed to update {command_id} accelerator: {e}"))?;
+            }
             // Silently succeed for commands that don't have menu items
-            // This allows the frontend to call this for all shortcuts without errors
             Ok(())
         }
     }

apps/desktop/src-tauri/src/commands/settings.rs

@@ -1,11 +1,10 @@
 use crate::ignore_poison::IgnorePoison;
-use crate::menu::{MenuState, build_context_menu, build_tab_context_menu};
+use crate::menu::{CommandScope, MenuState, build_context_menu, build_tab_context_menu, menu_id_to_command};
 #[cfg(any(target_os = "macos", target_os = "linux"))]
 use std::process::Command;
 use tauri::menu::ContextMenu;
 use tauri::{AppHandle, Emitter, Manager, Runtime, Window};
 use tauri_plugin_clipboard_manager::ClipboardExt;
-use tauri_plugin_opener::OpenerExt;
 
 #[tauri::command]
 pub fn update_menu_context<R: Runtime>(app: AppHandle<R>, path: String, filename: String) {
@@ -230,49 +229,34 @@ pub fn update_pin_tab_menu<R: Runtime>(app: AppHandle<R>, is_pinned: bool) -> Re
     item.set_text(label).map_err(|e| e.to_string())
 }
 
-/// Executes a menu action for the current context.
-pub fn execute_menu_action<R: Runtime>(app: &AppHandle<R>, id: &str) {
-    let state = app.state::<MenuState<R>>();
-    let context = state.context.lock_ignore_poison().clone();
+/// Enables or disables file-scoped menu items based on the current context.
+/// - `"explorer"`: all menu items enabled (main file explorer has focus)
+/// - `"other"`: file-scoped items disabled (Settings, file viewer, or other window has focus)
+#[tauri::command]
+pub fn set_menu_context<R: Runtime>(app: AppHandle<R>, context: String) -> Result<(), String> {
+    let enabled = context == "explorer";
+    let menu_state = app.state::<MenuState<R>>();
+    let items = menu_state.items.lock_ignore_poison();
 
-    if context.path.is_empty() {
-        return;
+    for (id, entry) in items.iter() {
+        if let Some((_, CommandScope::FileScoped)) = menu_id_to_command(id) {
+            let _ = entry.item.set_enabled(enabled);
+        }
     }
 
-    match id {
-        crate::menu::OPEN_ID => {
-            let _ = app.opener().open_path(&context.path, None::<&str>);
-        }
-        crate::menu::EDIT_ID => {
-            #[cfg(any(target_os = "macos", target_os = "linux"))]
-            {
-                let _ = open_in_editor(context.path);
-            }
-        }
-        crate::menu::SHOW_IN_FINDER_ID => {
-            #[cfg(any(target_os = "macos", target_os = "linux"))]
-            {
-                let _ = show_in_finder(context.path);
-            }
-        }
-        crate::menu::COPY_PATH_ID => {
-            let _ = app.clipboard().write_text(context.path);
-        }
-        crate::menu::COPY_FILENAME_ID => {
-            let _ = app.clipboard().write_text(context.filename);
-        }
-        crate::menu::QUICK_LOOK_ID => {
-            #[cfg(target_os = "macos")]
-            {
-                let _ = quick_look(context.path);
-            }
-        }
-        crate::menu::GET_INFO_ID => {
-            #[cfg(target_os = "macos")]
-            {
-                let _ = get_info(context.path);
-            }
-        }
-        _ => {}
+    // Items stored in separate MenuState fields (not in the HashMap) also need toggling
+    if let Some(ref item) = *menu_state.pin_tab.lock_ignore_poison() {
+        let _ = item.set_enabled(enabled);
+    }
+    if let Some(ref item) = *menu_state.show_hidden_files.lock_ignore_poison() {
+        let _ = item.set_enabled(enabled);
     }
+    if let Some(ref item) = *menu_state.view_mode_full.lock_ignore_poison() {
+        let _ = item.set_enabled(enabled);
+    }
+    if let Some(ref item) = *menu_state.view_mode_brief.lock_ignore_poison() {
+        let _ = item.set_enabled(enabled);
+    }
+
+    Ok(())
 }