KiCad - the KiCad Bridge

@@ -133,6 +133,69 @@ def is_kicad_window(hwnd) -> bool:     return _hwnd_exe_name(hwnd) in _KICAD_EXES  +# ── PREVENTION, not reaction (v0.9.101, John: "i sure see a lot of windows+# coming to fg"). The polling guardian REACTED: KiCad took focus, we yanked it+# back 250ms later — the user saw every steal as a flash and lost keystrokes.+# WS_EX_NOACTIVATE makes Windows REFUSE the window's self-activation outright:+# no flash, no focus churn. We apply it to a KiCad window the first time it+# misbehaves (or when a guardian arms over it), and CLEAR it when the last+# guardian exits / on bring_to_user / on close — so after bridge work ends the+# user can click into KiCad normally again.+WS_EX_NOACTIVATE = 0x08000000+GWL_EXSTYLE = -20++_noactivated: set = set()+_guard_count = 0+_guard_lock = None+try:+    import threading as _thr+    _guard_lock = _thr.Lock()+except Exception:+    pass+++def _get_exstyle(hwnd):+    fn = getattr(_user32, "GetWindowLongPtrW", None) or _user32.GetWindowLongW+    return fn(hwnd, GWL_EXSTYLE)+++def _set_exstyle(hwnd, style):+    fn = getattr(_user32, "SetWindowLongPtrW", None) or _user32.SetWindowLongW+    return fn(hwnd, GWL_EXSTYLE, style)+++def suppress_activation(hwnd) -> None:+    """Make a KiCad window unable to grab the foreground (WS_EX_NOACTIVATE)."""+    try:+        cur = _get_exstyle(hwnd)+        if not (cur & WS_EX_NOACTIVATE):+            _set_exstyle(hwnd, cur | WS_EX_NOACTIVATE)+        _noactivated.add(hwnd)+    except Exception:+        pass+++def allow_activation(hwnd) -> None:+    """Undo suppress_activation for one window (user wants it / bridge done)."""+    try:+        if _user32.IsWindow(hwnd):+            cur = _get_exstyle(hwnd)+            if cur & WS_EX_NOACTIVATE:+                _set_exstyle(hwnd, cur & ~WS_EX_NOACTIVATE)+    except Exception:+        pass+    _noactivated.discard(hwnd)+++def allow_activation_all() -> int:+    """Clear NOACTIVATE from every window we altered. Returns count cleared."""+    n = 0+    for h in list(_noactivated):+        allow_activation(h)+        n += 1+    return n++ def _demote_below(kicad_hwnd, user_hwnd) -> None:     """Slide a KiCad window directly UNDER the user's window without activating     either — the window stays visible in the taskbar and fully screenshotable,@@ -145,7 +208,7 @@ def _demote_below(kicad_hwnd, user_hwnd) -> None:         pass  -def restore_foreground_persistent(hwnd, seconds: float = 6.0, interval: float = 0.25) -> None:+def restore_foreground_persistent(hwnd, seconds: float = 6.0, interval: float = 0.1) -> None:     """Focus guardian: for `seconds` after a window-spawning verb, make sure NO     KiCad window holds the user's foreground. KiCad activates windows     ASYNCHRONOUSLY (a 3D viewer self-raises when its render completes — sometimes@@ -167,23 +230,41 @@ def restore_foreground_persistent(hwnd, seconds: float = 6.0, interval: float =     import time as _t      def _loop():+        global _guard_count+        if _guard_lock:+            with _guard_lock:+                _guard_count += 1         protect = hwnd         deadline = _t.monotonic() + seconds-        while _t.monotonic() < deadline:-            try:-                fg = _user32.GetForegroundWindow()-                if fg and fg != protect:-                    if is_kicad_window(fg):-                        # KiCad stole the foreground: user back on top, thief tucked under.-                        if _user32.IsWindow(protect):-                            _set_foreground_robust(protect)-                            _demote_below(fg, protect)-                    else:-                        # The USER moved to another window — protect that instead.-                        protect = fg-            except Exception:-                pass-            _t.sleep(interval)+        try:+            while _t.monotonic() < deadline:+                try:+                    fg = _user32.GetForegroundWindow()+                    if fg and fg != protect:+                        if is_kicad_window(fg):+                            # PREVENT the next steal (NOACTIVATE), then repair this+                            # one: user back on top, thief tucked under.+                            suppress_activation(fg)+                            if _user32.IsWindow(protect):+                                _set_foreground_robust(protect)+                                _demote_below(fg, protect)+                        else:+                            # The USER moved to another window — protect that instead.+                            protect = fg+                except Exception:+                    pass+                _t.sleep(interval)+        finally:+            # last guardian out clears the NOACTIVATE styles so the user can+            # click into KiCad normally once bridge work has quieted down.+            if _guard_lock:+                with _guard_lock:+                    _guard_count -= 1+                    last = _guard_count <= 0+            else:+                last = True+            if last:+                allow_activation_all()      try:         threading.Thread(target=_loop, daemon=True).start()@@ -207,6 +288,7 @@ def bring_to_user(hwnd) -> bool:     if not IS_WINDOWS or not hwnd:         return False     try:+        allow_activation(hwnd)       # a suppressed window can't be activated         _user32.ShowWindow(hwnd, 9)  # SW_RESTORE (in case minimized)         return _set_foreground_robust(hwnd)     except Exception: