Terminate stale Kapture instances at launch; log topmost windows

A stale copy surviving an update keeps its own hotkey and overlays on screen, silently eating every click meant for the new instance (matching the empty session logs). New copies now force-terminate older instances, and the first window scan logs the topmost on-screen windows so anything covering the overlays is visible in diagnostics. 💘 Generated with Crush Assisted-by: Crush:deepseek-v4-flash-0731
Show overlays only after activation completes
2026-09-09 11:59:03 +08:00 · 2026-09-09 11:51:13 +08:00 · 2026-09-09 11:46:45 +08:00 · 2026-09-09 11:00:44 +08:00
3 changed files with 32 additions and 16 deletions
@@ -21,6 +21,17 @@ final class AppDelegate: NSObject, NSApplicationDelegate {
            runOCRSelfTest()
            return
        }
+        // Single instance: stale copies keep their own hotkey + overlays alive
+        // after app updates and eat clicks meant for the new instance.
+        let myPid = ProcessInfo.processInfo.processIdentifier
+        let others = NSRunningApplication.runningApplications(withBundleIdentifier: "dev.tzgyn.kapture")
+            .filter { $0.processIdentifier != myPid }
+        if !others.isEmpty {
+            CaptureController.log("launch: other Kapture instances pids=\(others.map { String($0.processIdentifier) }.joined(separator: ",")) — terminating them")
+            for other in others {
+                other.forceTerminate()
+            }
+        }
        HotkeyManager.shared.onTrigger = { CaptureController.shared.begin() }
        HotkeyManager.shared.register()
    }
@@ -61,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.1</string>
+    <string>1.6.5</string>
    <key>CFBundleVersion</key>
-    <string>61</string>
+    <string>65</string>
    <key>LSUIElement</key>
    <true/>
    <key>NSHighResolutionCapable</key>
Author	SHA1	Message	Date
TZGyn	a7cad2fc2f	Terminate stale Kapture instances at launch; log topmost windows A stale copy surviving an update keeps its own hotkey and overlays on screen, silently eating every click meant for the new instance (matching the empty session logs). New copies now force-terminate older instances, and the first window scan logs the topmost on-screen windows so anything covering the overlays is visible in diagnostics. 💘 Generated with Crush Assisted-by: Crush:deepseek-v4-flash-0731	2026-09-09 11:59:03 +08:00
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