KiCad - the KiCad Bridge

@@ -125,11 +125,50 @@ def _find_child_by_class_and_parent(parent_hwnd: int, child_class: str, parent_t     return result[0]  +SW_SHOWNOACTIVATE = 4++ def _force_foreground(hwnd: int):-    """Bring a window to the foreground."""-    user32.ShowWindow(hwnd, SW_RESTORE)-    user32.SetForegroundWindow(hwnd)-    user32.BringWindowToTop(hwnd)+    """DELIBERATELY does NOT foreground — the bridge must never steal the user's+    focus (John, 2026-07). Just make the window visible without activating it; the+    plugin WM_COMMAND path drives it without focus and the dispatcher restores the+    user's window afterward. Name kept to avoid churning the call sites."""+    try:+        user32.ShowWindow(hwnd, SW_SHOWNOACTIVATE)+    except Exception:+        pass+++def _get_symbol_editor_title() -> str:+    """Current Symbol Editor window title ('' if not found) — load ground truth."""+    try:+        hwnd = _find_symbol_editor()+        if not hwnd:+            return ""+        buf = ctypes.create_unicode_buffer(512)+        user32.GetWindowTextW(hwnd, buf, 512)+        return buf.value or ""+    except Exception:  # pylint: disable=broad-except+        return ""+++def _plugin_background_load(symbol_name: str) -> dict:+    """v0.9.74: background canvas-load via the in-process plugin (payload 0.9.1+).++    Two-hop filter->settle->activate inside the plugin, zero focus/keyboard+    theft (John's no-foreground mandate). Uses bridge_client's discovery to find+    the live eeschema instance. {'loaded': False, ...} on any failure so the+    caller can fall through to the legacy paths.+    """+    try:+        from .bridge_client import _resolve_plugin, call as _plugin_call+        info, err = _resolve_plugin("eeschema")+        if err is not None or not info:+            return {"loaded": False, "_error": "no live eeschema plugin"}+        return _plugin_call(info["port"], "select_and_load_symbol",+                            {"symbolName": symbol_name}, timeout=15.0) or {"loaded": False}+    except Exception as e:  # pylint: disable=broad-except+        return {"loaded": False, "_error": f"{type(e).__name__}: {e}"}   def _navigate_to_symbol(sym_hwnd: int, symbol_name: str) -> dict:@@ -292,7 +331,7 @@ def handle_open_symbol_editor(kicad_info: dict, args: dict) -> dict:                 else f"plugin-failed:{tier1.get('errorCode','')}"             )             if tier1.get("success"):-                for _ in range(20):+                for _ in range(12):                     time.sleep(0.5)                     sym_hwnd = _find_symbol_editor()                     if sym_hwnd:@@ -329,8 +368,15 @@ def handle_open_symbol_editor(kicad_info: dict, args: dict) -> dict:                 }              try:+                _si = None+                try:+                    from handlers import win_focus+                    _si = win_focus.background_startupinfo()  # open the manager behind the user+                except Exception:+                    _si = None                 subprocess.Popen(                     [kicad_exe],+                    startupinfo=_si,                     stdout=subprocess.DEVNULL,                     stderr=subprocess.DEVNULL,                 )@@ -341,7 +387,7 @@ def handle_open_symbol_editor(kicad_info: dict, args: dict) -> dict:                     "_hint": "Call close_kicad with {\"force\": true} to clear stuck processes, then retry.",                 } -            for _ in range(20):+            for _ in range(12):                 time.sleep(0.5)                 kicad_hwnd = _find_kicad_manager()                 if kicad_hwnd:@@ -359,7 +405,7 @@ def handle_open_symbol_editor(kicad_info: dict, args: dict) -> dict:         user32.PostMessageW(kicad_hwnd, WM_COMMAND, KICAD_SYMBOL_EDITOR_CMD_ID, 0)         pathways_tried.append("wm_command") -        for _ in range(20):+        for _ in range(12):             time.sleep(0.5)             sym_hwnd = _find_symbol_editor()             if sym_hwnd:@@ -395,6 +441,21 @@ def handle_open_symbol_editor(kicad_info: dict, args: dict) -> dict:     # users who want to experiment, but the default kicad_open_symbol_editor     # flow keeps using the proven PowerShell path. +    # v0.9.74: FIRST try the plugin's background canvas-load (payload 0.9.1's+    # two-hop filter->settle->activate) — zero focus/keyboard theft. The PS+    # navigate below is the fallback when no plugin instance is alive.+    _pl = _plugin_background_load(search_text)+    if str(_pl.get("loaded")) in ("True", "unverified"):+        time.sleep(0.6)+        _title = _get_symbol_editor_title() if "_get_symbol_editor_title" in globals() else ""+        if _pl.get("loaded") is True or (search_text.lower() in (_title or "").lower()):+            return {+                "success": True,+                "output": f"Symbol '{search_text}' loaded (background plugin path): {_title or _pl.get('frameTitle','')}",+                "via": "plugin-background",+            }+        # unverified and title doesn't confirm — fall through to PS nav+     nav_result = _navigate_to_symbol(sym_hwnd, search_text)      if nav_result["status"] == "ok":