update coderabbit

Author: plyght

Cargo.lock

@@ -45,6 +45,12 @@ function Component() {
   const isDraggingRef = useRef(false);
   const dragOffsetRef = useRef({ x: 0, y: 0 });
 
+  // Interaction tracking (lifted to component scope)
+  const lastInteractionRef = React.useRef(Date.now());
+  const trackInteraction = React.useCallback(() => {
+    lastInteractionRef.current = Date.now();
+  }, []);
+  
   // Update refs whenever state changes
   useEffect(() => {
     isDraggingRef.current = isDragging;
@@ -262,15 +268,9 @@ function Component() {
       updateOverlayBounds();
     };
 
-    // Lightweight recovery: only trigger on actual user interaction
-    let lastInteractionTime = Date.now();
-    const trackInteraction = () => {
-      lastInteractionTime = Date.now();
-    };
-    
     // Only do aggressive reset if it's been a while since last interaction
     const smartRecovery = () => {
-      const timeSinceInteraction = Date.now() - lastInteractionTime;
+      const timeSinceInteraction = Date.now() - lastInteractionRef.current;
       if (timeSinceInteraction > 10000) { // 10 seconds of no interaction
         windowsCommands.removeFakeWindow("control").then(() => {
           setTimeout(updateOverlayBounds, 100);

apps/desktop/src/routes/app.control.tsx

@@ -33,3 +33,4 @@ tauri-plugin-auth = { workspace = true }
 thiserror = { workspace = true }
 tokio = { workspace = true, features = ["time"] }
 tracing = { workspace = true }
+once_cell = { workspace = true }

plugins/windows/Cargo.toml

@@ -225,6 +225,8 @@ impl HyprWindow {
     fn close(&self, app: &AppHandle<tauri::Wry>) -> Result<(), crate::Error> {
         match self {
             HyprWindow::Control => {
+                crate::abort_overlay_join_handle();
+                
                 #[cfg(target_os = "macos")]
                 {
                     use tauri_nspanel::ManagerExt;
@@ -437,12 +439,12 @@ impl HyprWindow {
                 }
 
                 let join_handle = crate::spawn_overlay_listener(app.clone(), window.clone());
+                crate::set_overlay_join_handle(join_handle);
 
                 // Cancel the overlay listener when the window is closed
-                let window_clone = window.clone();
                 window.on_window_event(move |event| {
                     if let tauri::WindowEvent::CloseRequested { .. } = event {
-                        join_handle.abort();
+                        crate::abort_overlay_join_handle();
                     }
                 });
 

plugins/windows/src/ext.rs

@@ -13,9 +13,30 @@ pub use overlay::{FakeWindowBounds, OverlayBound};
 const PLUGIN_NAME: &str = "windows";
 
 use tauri::Manager;
+use std::sync::Mutex;
+use once_cell::sync::Lazy;
 
 pub type ManagedState = std::sync::Mutex<State>;
 
+static OVERLAY_JOIN_HANDLE: Lazy<Mutex<Option<tokio::task::JoinHandle<()>>>> = Lazy::new(|| Mutex::new(None));
+
+pub fn set_overlay_join_handle(handle: tokio::task::JoinHandle<()>) {
+    if let Ok(mut guard) = OVERLAY_JOIN_HANDLE.lock() {
+        if let Some(old_handle) = guard.take() {
+            old_handle.abort();
+        }
+        *guard = Some(handle);
+    }
+}
+
+pub fn abort_overlay_join_handle() {
+    if let Ok(mut guard) = OVERLAY_JOIN_HANDLE.lock() {
+        if let Some(handle) = guard.take() {
+            handle.abort();
+        }
+    }
+}
+
 #[derive(Default)]
 pub struct WindowState {
     floating: bool,

plugins/windows/src/lib.rs

@@ -69,7 +69,7 @@ pub fn spawn_overlay_listener(app: AppHandle, window: WebviewWindow) -> tokio::t
 
             let mut ignore = true;
 
-            for (name, bounds) in windows.iter() {
+            for (_name, bounds) in windows.iter() {
                 let x_min = (window_position.x as f64) + bounds.x * scale_factor;
                 let x_max = (window_position.x as f64) + (bounds.x + bounds.width) * scale_factor;
                 let y_min = (window_position.y as f64) + bounds.y * scale_factor;