Throttle event handling

Author: intitni

OverlayWindow/Sources/OverlayWindow/IDEWorkspaceWindowOverlayWindowController.swift

@@ -1,6 +1,7 @@
 import AppKit
 import AXExtension
 import AXNotificationStream
+import DebounceFunction
 import Foundation
 import Perception
 import SwiftUI
@@ -31,6 +32,7 @@ final class IDEWorkspaceWindowOverlayWindowController {
     let maskPanel: OverlayPanel
     var windowElement: AXUIElement
     private var axNotificationTask: Task<Void, Never>?
+    let updateFrameThrottler = ThrottleRunner(duration: 0.2)
 
     init(
         inspector: WorkspaceXcodeWindowInspector,
@@ -123,7 +125,9 @@ final class IDEWorkspaceWindowOverlayWindowController {
                 if Task.isCancelled { return }
                 switch notification.name {
                 case kAXMovedNotification, kAXResizedNotification:
-                    self.updateFrame()
+                    await self.updateFrameThrottler.throttle { [weak self] in
+                        await self?.updateFrame()
+                    }
                 case kAXWindowMiniaturizedNotification:
                     self.hide()
                 default: continue

OverlayWindow/Sources/OverlayWindow/OverlayWindowController.swift

@@ -1,4 +1,5 @@
 import AppKit
+import DebounceFunction
 import Foundation
 import Perception
 import XcodeInspector
@@ -18,6 +19,8 @@ public final class OverlayWindowController {
         [ObjectIdentifier: IDEWorkspaceWindowOverlayWindowController]()
     var updateWindowStateTask: Task<Void, Error>?
 
+    let windowUpdateThrottler = ThrottleRunner(duration: 0.2)
+
     lazy var fullscreenDetector = {
         let it = NSWindow(
             contentRect: .zero,
@@ -95,53 +98,8 @@ private extension OverlayWindowController {
             guard let self else { return }
             Task { @MainActor in
                 defer { self.observeWindowChange() }
-
-                guard XcodeInspector.shared.activeApplication?.isXcode ?? false else {
-                    var closedControllers: [ObjectIdentifier] = []
-                    for (id, controller) in self.ideWindowOverlayWindowControllers {
-                        if controller.isWindowClosed {
-                            controller.dim()
-                            closedControllers.append(id)
-                        } else {
-                            controller.dim()
-                        }
-                    }
-                    for id in closedControllers {
-                        self.removeIDEOverlayWindowController(for: id)
-                    }
-                    return
-                }
-
-                guard let app = XcodeInspector.shared.activeXcode else {
-                    for (_, controller) in self.ideWindowOverlayWindowControllers {
-                        controller.hide()
-                    }
-                    return
-                }
-
-                let windowInspector = XcodeInspector.shared.focusedWindow
-                if let ideWindowInspector = windowInspector as? WorkspaceXcodeWindowInspector {
-                    let objectID = ObjectIdentifier(ideWindowInspector)
-                    // Workspace window is active
-                    // Hide all controllers first
-                    for (id, controller) in self.ideWindowOverlayWindowControllers {
-                        if id != objectID {
-                            controller.hide()
-                        }
-                    }
-                    if let controller = self.ideWindowOverlayWindowControllers[objectID] {
-                        controller.access()
-                    } else {
-                        self.createNewIDEOverlayWindowController(
-                            inspector: ideWindowInspector,
-                            application: app.runningApplication
-                        )
-                    }
-                } else {
-                    // Not a workspace window, dim all controllers
-                    for (_, controller) in self.ideWindowOverlayWindowControllers {
-                        controller.dim()
-                    }
+                await self.windowUpdateThrottler.throttle { [weak self] in
+                    await self?.handleOverlayStatusChange()
                 }
             }
         }
@@ -194,5 +152,55 @@ private extension OverlayWindowController {
             activeWindowController.maskPanel.orderFrontRegardless()
         }
     }
+
+    func handleOverlayStatusChange() {
+        guard XcodeInspector.shared.activeApplication?.isXcode ?? false else {
+            var closedControllers: [ObjectIdentifier] = []
+            for (id, controller) in ideWindowOverlayWindowControllers {
+                if controller.isWindowClosed {
+                    controller.dim()
+                    closedControllers.append(id)
+                } else {
+                    controller.dim()
+                }
+            }
+            for id in closedControllers {
+                removeIDEOverlayWindowController(for: id)
+            }
+            return
+        }
+
+        guard let app = XcodeInspector.shared.activeXcode else {
+            for (_, controller) in ideWindowOverlayWindowControllers {
+                controller.hide()
+            }
+            return
+        }
+
+        let windowInspector = XcodeInspector.shared.focusedWindow
+        if let ideWindowInspector = windowInspector as? WorkspaceXcodeWindowInspector {
+            let objectID = ObjectIdentifier(ideWindowInspector)
+            // Workspace window is active
+            // Hide all controllers first
+            for (id, controller) in ideWindowOverlayWindowControllers {
+                if id != objectID {
+                    controller.hide()
+                }
+            }
+            if let controller = ideWindowOverlayWindowControllers[objectID] {
+                controller.access()
+            } else {
+                createNewIDEOverlayWindowController(
+                    inspector: ideWindowInspector,
+                    application: app.runningApplication
+                )
+            }
+        } else {
+            // Not a workspace window, dim all controllers
+            for (_, controller) in ideWindowOverlayWindowControllers {
+                controller.dim()
+            }
+        }
+    }
 }