Show overlays only after activation completes

The local probe confirmed activation is asynchronous on macOS 26: isActive flips on a later runloop tick, so overlays shown in the same tick as activate() belong to a still-inactive app and misses clicks. Delay overlay creation by a tick and log the activation state at show time. 💘 Generated with Crush Assisted-by: Crush:deepseek-v4-flash-0731
Make overlay panels activating so clicks are received
2026-09-09 11:51:13 +08:00 · 2026-09-09 11:46:45 +08:00 · 2026-09-09 11:00:44 +08:00 · 2026-09-09 10:52:16 +08:00
2 changed files with 33 additions and 17 deletions
@@ -10,10 +10,21 @@ final class CaptureController: NSObject {
    private var previousApp: NSRunningApplication?
    private var overlayIDs: [Int] = []

+    /// Session event log for debugging (dumped via Copy Diagnostics).
+    static var sessionLog: [String] = []
+    static func log(_ line: String) {
+        sessionLog.append(line)
+        if sessionLog.count > 100 { sessionLog.removeFirst() }
+        NSLog("[Kapture] \(line)")
+    }
+
    /// Diagnostic snapshot for "Copy Diagnostics" menu item.
    func diagnostics() -> String {
        var lines: [String] = []
-        lines.append("Kapture diagnostics")
+        lines.append("Kapture diagnostics — version \(Bundle.main.infoDictionary?["CFBundleShortVersionString"] ?? "?")")
+        lines.append("--- session log (newest last) ---")
+        lines.append(contentsOf: Self.sessionLog)
+        lines.append("--- screens ---")
        for (i, s) in NSScreen.screens.enumerated() {
            let scale = s.backingScaleFactor
            let cgID = (s.deviceDescription[NSDeviceDescriptionKey("NSScreenNumber")] as? NSNumber)?.uint32Value ?? 0
@@ -50,21 +61,26 @@ final class CaptureController: NSObject {
        }
        isSelecting = true
        previousApp = NSWorkspace.shared.frontmostApplication
-        if #available(macOS 14.0, *) {
-            NSApp.activate()
-        } else {
-            NSApp.activate(ignoringOtherApps: true)
+        // Activation is ASYNC: isActive only flips on a later runloop tick
+        // (verified on macOS 26). Show overlays only once active, otherwise
+        // the first click can pass through to the windows below.
+        NSApp.activate(ignoringOtherApps: true)
+        CaptureController.log("begin: activate issued, active=\(NSApp.isActive)")
+        DispatchQueue.main.asyncAfter(deadline: .now() + 0.05) { [weak self] in
+            guard let self, self.isSelecting else { return }
+            CaptureController.log("begin: active=\(NSApp.isActive) — showing overlays")
+            self.overlays = NSScreen.screens.map { screen in
+                OverlayWindow(
+                    screen: screen,
+                    onComplete: { [weak self] rect in self?.finishSelection(rect) },
+                    onCancel: { [weak self] in self?.cancelSelection() },
+                    onWindowPick: { [weak self] rect in self?.finishWindowPick(rect) }
+                )
+            }
+            self.overlays.forEach { $0.makeKeyAndOrderFront(nil) }
+            self.overlayIDs = self.overlays.map { $0.windowNumber }
+            CaptureController.log("begin: \(self.overlays.count) overlays, ids=\(self.overlayIDs), active=\(NSApp.isActive)")
        }
-        overlays = NSScreen.screens.map { screen in
-            OverlayWindow(
-                screen: screen,
-                onComplete: { [weak self] rect in self?.finishSelection(rect) },
-                onCancel: { [weak self] in self?.cancelSelection() },
-                onWindowPick: { [weak self] rect in self?.finishWindowPick(rect) }
-            )
-        }
-        overlays.forEach { $0.makeKeyAndOrderFront(nil) }
-        overlayIDs = overlays.map { $0.windowNumber }
    }

    /// A window was picked from hover — capture it with rounded corners.
@@ -108,9 +108,9 @@ cat > "$APP/Contents/Info.plist" <<'PLIST'
    <key>CFBundleIconFile</key>
    <string>Icon</string>
    <key>CFBundleShortVersionString</key>
-    <string>1.6.0</string>
+    <string>1.6.4</string>
    <key>CFBundleVersion</key>
-    <string>60</string>
+    <string>64</string>
    <key>LSUIElement</key>
    <true/>
    <key>NSHighResolutionCapable</key>
Author	SHA1	Message	Date
TZGyn	69e1a7fccb	Show overlays only after activation completes The local probe confirmed activation is asynchronous on macOS 26: isActive flips on a later runloop tick, so overlays shown in the same tick as activate() belong to a still-inactive app and misses clicks. Delay overlay creation by a tick and log the activation state at show time. 💘 Generated with Crush Assisted-by: Crush:deepseek-v4-flash-0731	2026-09-09 11:51:13 +08:00
TZGyn	e22bcb99c4	Make overlay panels activating so clicks are received The session log showed active=false after every activation attempt on macOS 26: cooperative activation is denied for LSUIElement apps, and an inactive app's panels let clicks fall through to the windows below. Remove .nonactivatingPanel so showing the overlay activates the app (previous app focus is restored after capture as before). 💘 Generated with Crush Assisted-by: Crush:deepseek-v4-flash-0731	2026-09-09 11:46:45 +08:00
TZGyn	457e303152	Force app activation on capture; log to /tmp for durable diagnostics Cooperative NSApp.activate() can silently fail for LSUIElement apps, leaving the app inactive so overlay clicks pass through to windows below. Force activation with ignoringOtherApps on all OS versions. Session logs now also append to /tmp/kapture-debug.log so they survive relaunches, and the first window-scan result is always logged. 💘 Generated with Crush Assisted-by: Crush:deepseek-v4-flash-0731	2026-09-09 11:00:44 +08:00
TZGyn	345f306edf	Add session event logging to capture flow for remote debugging OverlayView now logs window tracking counts, hover changes, and mouseDown/Up branch decisions into a session log that Copy Diagnostics includes along with the app version, so capture failures on machines we cannot access directly can be diagnosed from a pasted report. 💘 Generated with Crush Assisted-by: Crush:deepseek-v4-flash-0731	2026-09-09 10:52:16 +08:00