fix(macos/tile-snap): switch release to panic=unwind so catch_unwind catches NSEvent handler panics

In v7.0.26 the bundled app crashed (SIGABRT in `GlobalObserverHandler`)
three times in field logs. Root cause: `panic = "abort"` in the release
profile makes `std::panic::catch_unwind` in the Tile Snap NSEvent global
monitor (tiling/macos.rs) inert — any panic in the ObjC block bypasses
the catch and aborts the process.

Fix:
- Set `panic = "unwind"` for release so catch_unwind actually catches.
- Defensive `state.displays.get(display_idx)` in handle_snap_event to
  avoid an out-of-bounds panic when displays change mid-drag.

Bumps version to 7.0.29.

Author: synle

src-tauri/Cargo.toml

@@ -57,8 +57,14 @@ windows = { version = "0.58", features = [
 ddc-winapi = "0.2"
 winapi = { version = "0.3", features = ["wingdi", "windef"] }
 
+# `panic = "unwind"` is required so that `std::panic::catch_unwind` in the
+# macOS Tile Snap NSEvent global monitor (tiling/macos.rs) actually catches
+# panics. With `panic = "abort"`, a panic in that ObjC-block handler bypasses
+# `catch_unwind` and calls `abort()`, which has crashed the bundled app
+# multiple times in the field (SIGABRT in `GlobalObserverHandler`). Slightly
+# larger binary in exchange for not aborting on a recoverable handler bug.
 [profile.release]
-panic = "abort"
+panic = "unwind"
 codegen-units = 1
 lto = true
 opt-level = "s"

src-tauri/src/tiling/macos.rs

@@ -2742,23 +2742,32 @@ fn handle_snap_event(ctx: &SnapContext, event_type: u64, cursor: CGPoint) {
 
             match zone {
                 Some((layout, display_idx)) => {
+                    // Defensive bounds check: `detect_snap_zone_macos` already
+                    // returns an index into `state.displays`, but `state.displays`
+                    // may have been refreshed mid-drag (monitor hotplug). An
+                    // out-of-bounds indexing here would panic inside the NSEvent
+                    // ObjC block, which on a `panic = "abort"` build aborts the
+                    // process. Skip rather than crash.
+                    let display = match state.displays.get(display_idx) {
+                        Some(d) => d.clone(),
+                        None => return,
+                    };
                     if state.current_layout != Some(layout) || state.current_display != display_idx
                     {
                         if let Some(dbg_state) = ctx.app.try_state::<crate::AppState>() {
-                            let d = &state.displays[display_idx];
                             crate::config::write_debug_log(
                                 &dbg_state,
                                 &format!(
                                     "tile_snap: zone detected — layout={:?}, display={} ({:.0},{:.0} {:.0}x{:.0}), cursor=({:.0},{:.0})",
-                                    layout, display_idx, d.x, d.y, d.width, d.height, cursor.x, cursor.y,
+                                    layout, display_idx, display.x, display.y, display.width, display.height, cursor.x, cursor.y,
                                 ),
                             );
                         }
                         state.current_layout = Some(layout);
                         state.current_display = display_idx;
                         let target = calculate_target_rect(
                             layout,
-                            &state.displays[display_idx],
+                            &display,
                             state.half_ratio,
                             state.third_ratio,
                             state.gap,

src-tauri/tauri.conf.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://raw.githubusercontent.com/nicoverbruggen/tauri-v2-docs/refs/heads/main/schemas/config.schema.json",
   "productName": "Display DJ",
-  "version": "7.0.28",
+  "version": "7.0.29",
   "identifier": "com.synle.display-dj",
   "build": {
     "beforeDevCommand": "npm run dev",