Fusion - the Fusion 360 Bridge

@@ -1251,16 +1251,31 @@ def _capture_labeled(label) -> str | None:   def _inject_package3d_bindings(lbr_text: str, bindings: list) -> str:-    """Inject EAGLE <packages3d> + per-device <package3dinstances> into an .lbr in ONE pass.+    """Inject EAGLE <packages3d> + per-device <package3dinstances> into an .lbr - MERGE-AWARE+    and IDEMPOTENT (safe to re-run with any subset of parts). -    bindings: [{package: <pkg name>, wip_urn: <urn>}]. Adds a library-level <package3d> for-    each (between </packages> and <symbols>) and, into every <device> that uses that package,-    a <package3dinstances> right after </connects>. This is the exact XML surgery the AI used-    to hand-write per part - now one deterministic call. Returns the new .lbr text."""+    bindings: [{package: <pkg name>, wip_urn: <urn>}]. The function reads any bindings ALREADY in+    the .lbr, merges the new ones on top (new wins on conflict), strips all prior <packages3d> ++    <package3dinstances>, then re-emits the full merged set: a library-level <package3d> per part+    (between </packages> and <symbols>) and a <package3dinstances> inside every <device> that uses+    that package (right after </connects>). So calling it again with just the parts that failed last+    time ACCUMULATES instead of wiping the parts that already succeeded - the fix for the+    'a re-run dropped the earlier bindings' trap. Returns the new .lbr text."""     import re as _re-    blocks = []+    # 1. read existing bindings already in the file; new bindings override+    merged = {}+    em = _re.search(r"<packages3d>(.*?)</packages3d>", lbr_text, _re.S)+    if em:+        for pm in _re.finditer(r'<package3d name="([^"]+)"[^>]*wip_urn="([^"]+)"', em.group(1)):+            merged[pm.group(1)] = pm.group(2)     for b in bindings:-        pkg = b["package"]; urn = b["wip_urn"]+        merged[b["package"]] = b["wip_urn"]+    # 2. strip ALL prior package3d markup so re-injection is clean (idempotent)+    lbr_text = _re.sub(r"\s*<packages3d>.*?</packages3d>", "", lbr_text, flags=_re.S)+    lbr_text = _re.sub(r"\s*<package3dinstances>.*?</package3dinstances>", "", lbr_text, flags=_re.S)+    # 3. library-level <packages3d> block (all merged parts)+    blocks = []+    for pkg, urn in merged.items():         blocks.append(             f'<package3d name="{pkg}" urn="" wip_urn="{urn}" locally_modified="yes" type="model">'             f'<description>{pkg}</description>'@@ -1268,18 +1283,14 @@ def _inject_package3d_bindings(lbr_text: str, bindings: list) -> str:             f'</package3d>'         )     pkgs3d = "<packages3d>\n" + "\n".join(blocks) + "\n</packages3d>\n"-    if "<packages3d>" in lbr_text:-        lbr_text = _re.sub(r"<packages3d>.*?</packages3d>\s*", pkgs3d, lbr_text, flags=_re.S)-    else:-        lbr_text = lbr_text.replace("</packages>", "</packages>\n" + pkgs3d, 1)-    urn_by_pkg = {b["package"]: b["wip_urn"] for b in bindings}+    lbr_text = lbr_text.replace("</packages>", "</packages>\n" + pkgs3d, 1) +    # 4. per-device <package3dinstances>     def _dev_repl(m):         dev = m.group(0)         pm = _re.search(r'package="([^"]+)"', dev)-        if pm and pm.group(1) in urn_by_pkg and "<package3dinstances>" not in dev and "</connects>" in dev:-            urn = urn_by_pkg[pm.group(1)]-            inst = (f'<package3dinstances><package3dinstance package3d_urn="{urn}"/>'+        if pm and pm.group(1) in merged and "</connects>" in dev:+            inst = (f'<package3dinstances><package3dinstance package3d_urn="{merged[pm.group(1)]}"/>'                     f'</package3dinstances>')             dev = dev.replace("</connects>", "</connects>\n" + inst, 1)         return dev@@ -1450,12 +1461,19 @@ def _orchestrate_build_library_3d(args: dict) -> dict:             results.append({"package": pkg, "success": False,                             "error": "package, lbrPath (per-part .lbr) and modelPath (.step) all required"})             continue-        mk = _orchestrate_make_3d_package({+        mk_args = {             "lbrPath": per_lbr, "modelPath": step,             "projectId": project_id, "folderId": folder_id,             "fileName": f"{pkg}_3d",             "captureLabel": (pkg if capture else None),-        })+        }+        mk = _orchestrate_make_3d_package(mk_args)+        # Retry once: the FIRST part of a run often fails transiently while Fusion settles from a+        # prior view/library; a single retry recovers it (the dropped-first-part trap).+        if not mk.get("success"):+            import time as _t+            _t.sleep(2)+            mk = _orchestrate_make_3d_package(mk_args)         entry = {"package": pkg, "success": bool(mk.get("success"))}         if mk.get("success"):             entry["wip_urn"] = mk.get("wip_urn"); entry["dims_mm"] = mk.get("dims_mm")@@ -1467,11 +1485,14 @@ def _orchestrate_build_library_3d(args: dict) -> dict:         if mk.get("after"):             shots.append({"package": pkg, "stage": "after", "path": mk["after"]})         results.append(entry)-    # inject every binding in ONE pass, write the bound library+    # inject every binding in ONE pass, write the bound library. Read the ALREADY-BOUND output if it+    # exists so a re-run ACCUMULATES (merge-aware) onto prior successes instead of starting clean.     bound = None     if bindings:         try:-            with open(combined, "r", encoding="utf-8") as fh:+            import os as _os+            src = out_path if _os.path.exists(out_path) else combined+            with open(src, "r", encoding="utf-8") as fh:                 txt = fh.read()             txt = _inject_package3d_bindings(txt, bindings)             with open(out_path, "w", encoding="utf-8") as fh:@@ -1504,10 +1525,69 @@ def _orchestrate_build_library_3d(args: dict) -> dict:             "front. Each device's lower-right 3D preview should show footprint + chip (NOT 'Thumbnail "             "download failed'). The BEFORE (footprint) / AFTER (chip placed) PNGs are in screenshots[] "             "on the Windows box (C:/tmp/conduit-screenshots) - pull them with desktop_pull_file (or "-            "re-capture via desktop_screenshot_window) for the demo video. PITFALLS: needs Fusion "-            "running; on long runs Fusion can reset between parts (fusion_start, then re-run just the "-            "failed parts - each is independent); any failed part has its error in parts[]. Full recipe: "-            "the fusion-multipart-libraries skill."+            "re-capture via desktop_screenshot_window) for the demo video. "+            "AFTER (recommended): call fusion_capture_library_views per showcase part to grab the "+            "symbol / footprint / component (pin<->pad mapped) views - those are what make EEs trust "+            "the library, and the 3D-only shots miss them. "+            "PITFALLS: needs Fusion running. ⚠️ RELAY TIMEOUT: a many-part run takes minutes but the "+            "adom-desktop relay times out the REQUEST at ~60s - you may get 'Request timed out' even "+            "though the build KEEPS RUNNING server-side and finishes. Do NOT assume it failed: wait, "+            "then verify by reading the boundLbr (count <package3d name=) + the before/after PNGs in "+            "C:/tmp/conduit-screenshots. If a part is missing, just re-run build_library_3d for the "+            "missing parts pointing outLbrPath at the SAME file - binding injection is now MERGE-AWARE "+            "and idempotent, so it accumulates onto the existing bindings (it no longer wipes the "+            "parts that already succeeded). Each part also self-retries once on a transient first-part "+            "failure. Full recipe: the fusion-multipart-libraries skill."+        ),+    }+++def _orchestrate_capture_library_views(args: dict) -> dict:+    """Capture the LIBRARY-EDITOR views that make EEs trust a part: the schematic SYMBOL, the+    FOOTPRINT (pads + layer stack), and the COMPONENT/device view (Content Manager: the symbol ++    the package table with the footprint<->package Mapped check + pin/pad counts). These are what+    developers want to see - the 3D before/after shots alone don't show them.++    Requires the .lbr OPEN in the Electronics Library workspace (fusion_open_lbr first). For each+    package it runs the EAGLE/Electron EDIT <pkg>.sym / .pac / .dev, WINDOW FIT to frame, and a+    background hwnd screenshot (never fullscreen). Returns the saved PNG paths on the Windows box+    (C:/tmp/conduit-screenshots - pull them with desktop_pull_file).++    args: {packages: [<deviceset name>, ...] (or a single 'package'),+           views?: subset of ['component','symbol','footprint'] (default all three)}+    """+    import time as _t+    pkgs = args.get("packages") or ([args["package"]] if args.get("package") else [])+    if not pkgs:+        return {"success": False,+                "error": "packages: [<deviceset name>] (or package: <name>) required.",+                "_hint": "Open the .lbr first (fusion_open_lbr). Names are the deviceset names, e.g. ESP32-S3FN8."}+    view_ext = {"component": "dev", "symbol": "sym", "footprint": "pac",+                "dev": "dev", "sym": "sym", "pac": "pac"}+    label_of = {"dev": "component", "sym": "symbol", "pac": "footprint"}+    views = args.get("views") or ["component", "symbol", "footprint"]+    out = []+    for pkg in pkgs:+        for v in views:+            ext = view_ext.get(v, v)+            er = _proxy_to_addin("electron_run", {"command": f"EDIT {pkg}.{ext}"}, timeout=30)+            _proxy_to_addin("electron_run", {"command": "WINDOW FIT"}, timeout=20)+            _t.sleep(0.4)+            shot = _capture_labeled(f"{pkg}_{label_of.get(ext, ext)}")+            out.append({"package": pkg, "view": label_of.get(ext, ext), "path": shot,+                        "ok": bool(er.get("success", True)) and bool(shot)})+    n_ok = sum(1 for o in out if o.get("ok"))+    return {+        "success": n_ok > 0,+        "captured": out,+        "_hint": (+            f"Captured {n_ok}/{len(out)} library-editor views to C:/tmp/conduit-screenshots (pull with "+            "desktop_pull_file). SYMBOL = full pinout; FOOTPRINT = pads + layer stack; COMPONENT = the "+            "Content Manager device view with the footprint<->package Mapped check + pin/pad counts. "+            "PITFALLS: the .lbr must be OPEN in the Electronics Library workspace (fusion_open_lbr first; "+            "fusion_get_app_state activeWorkspace == 'Electronics Library'); EDIT uses the DEVICESET "+            "name (e.g. ESP32-S3FN8); a blank capture usually means a wrong name or the library is not "+            "the active tab."         ),     } @@ -1581,7 +1661,7 @@ def dispatch_command(command: str, args: dict) -> dict:     # The user should launch Fusion themselves; we just report the status.     # Commands that need Fusion running (add-in commands + orchestrated commands)     _OPEN_WITH_SCREENSHOT = {"open_schematic", "open_board", "show_3d_board", "show_2d_board", "show_schematic", "import_electronics"}-    if command in ADDIN_COMMANDS or command in ("open_lbr", "save_lbr", "attach_3d_package", "make_3d_package", "build_library_3d") or command in _OPEN_WITH_SCREENSHOT:+    if command in ADDIN_COMMANDS or command in ("open_lbr", "save_lbr", "attach_3d_package", "make_3d_package", "build_library_3d", "capture_library_views") or command in _OPEN_WITH_SCREENSHOT:         if not fusion_info.get("installed"):             return {                 "success": False,@@ -1689,6 +1769,8 @@ def dispatch_command(command: str, args: dict) -> dict:         return _orchestrate_make_3d_package(args)     if command == "build_library_3d":         return _orchestrate_build_library_3d(args)+    if command == "capture_library_views":+        return _orchestrate_capture_library_views(args)      # Add-in proxy commands     if command in ADDIN_COMMANDS: