Fusion - the Fusion 360 Bridge

@@ -9029,86 +9029,84 @@ COMMAND_HANDLERS["prove_library"] = _handle_prove_library   +def _find_adom_lbr() -> str:+    """Locate adom-lbr on THIS box (the canonical KiCad->EAGLE converter, adom/adom-lbr)."""+    import shutil as _sh+    hit = _sh.which("adom-lbr") or _sh.which("adom-lbr.exe")+    if hit:+        return hit+    la = os.environ.get("LOCALAPPDATA", "")+    for c in ([os.path.join(la, "Programs", "adom-lbr", "adom-lbr.exe"),+               os.path.join(la, "adom-lbr", "adom-lbr.exe")] if la else []):+        if os.path.exists(c):+            return c+    return ""++ def _handle_import_kicad(fusion_info: dict, args: dict) -> dict:-    """Ingest KiCad symbol/footprint files and BUILD or PATCH a valid .lbr (issue #43).--    KiCad is the FUNDAMENTAL symbol/footprint representation at Adom; the .lbr is derived,-    and a derived file can be missing, stale, or half-generated. This verb makes the .lbr an-    optimization instead of a prerequisite - the same posture bind_library_3d already takes-    for STEP.--    args:-      kicadFootprintPath - Windows path to a .kicad_mod   (optional)-      kicadSymbolPath    - Windows path to a .kicad_sym   (optional)-      libraryPath        - .lbr to PATCH if it exists, else where to WRITE the new library-                           (optional; defaults next to the footprint/symbol file)-      name, prefix       - optional overrides for the created deviceset--    PATCH fills only what is EMPTY (<packages></packages>, <connects/>) and never overwrites-    existing content. The concrete case: cr0603-fx-5601elf's .lbr had an empty <packages>-    while a correct .kicad_mod sat beside it, so every downstream verb refused honestly and-    uselessly. After this verb, that part just works.+    """Ingest KiCad symbol/footprint files and produce a valid .lbr (issue #43) - by+    DELEGATING to adom-lbr, the ecosystem's canonical converter.++    John, 2026-08-21: "just use adom-lbr as your converter" - instead of this bridge+    growing its own KiCad parser. adom-lbr owns the conversion, ships a linter whose+    checks name the exact failure this issue was filed about (empty-packages /+    empty-connects -> Fusion "No Components"), and round-trips KiCad, EAGLE and Altium.+    One converter, one set of bugs, one place to fix them.++    args: kicadSymbolPath, kicadFootprintPath (Windows paths), libraryPath (output,+    optional), name (optional override).     """-    import os as _os-    import kicad_lbr as _kl-    mod_p = (args.get("kicadFootprintPath") or "").replace("\\", "/")+    import subprocess as _sp     sym_p = (args.get("kicadSymbolPath") or "").replace("\\", "/")+    mod_p = (args.get("kicadFootprintPath") or "").replace("\\", "/")     lbr_p = (args.get("libraryPath") or "").replace("\\", "/")-    if not mod_p and not sym_p:+    if not sym_p and not mod_p:         return {"success": False, "errorCode": "kicad_source_required",-                "error": "Pass kicadFootprintPath and/or kicadSymbolPath.",-                "_hint": "Both are wiki first-class artifacts; stage them with send_files, "-                         "then call this with their Windows paths."}-    def _read(p, what):-        if not p:-            return None, None-        try:-            return open(p, "r", encoding="utf-8", errors="replace").read(), None-        except Exception as e:-            return None, {"success": False, "errorCode": "kicad_source_unreadable",-                          "error": "Could not read %s: %s (%s)" % (what, p, str(e)[:80])}-    mod_text, err = _read(mod_p, ".kicad_mod")-    if err:-        return err-    sym_text, err = _read(sym_p, ".kicad_sym")-    if err:-        return err+                "error": "Pass kicadSymbolPath and/or kicadFootprintPath."}+    tool = _find_adom_lbr()+    if not tool:+        return {"success": False, "errorCode": "adom_lbr_unavailable",+                "error": "adom-lbr (the canonical KiCad->EAGLE converter) is not installed "+                         "on this box, and this bridge deliberately does NOT convert KiCad "+                         "formats itself.",+                "_hint": ("Conversion is owned by adom/adom-lbr - one converter for the whole "+                          "ecosystem. From a CONTAINER: `adom-lbr generate --sym <.kicad_sym> "+                          "--fp <.kicad_mod> -o <out.lbr>` (auto-linted), then stage the .lbr "+                          "to this box (send_files, dest under %TEMP%) and use the fusion_show_* "+                          "verbs. A windows release asset of adom-lbr has been requested so this "+                          "verb can run it on-box.")}     if not lbr_p:-        base = (mod_p or sym_p).rsplit("/", 1)+        base = (sym_p or mod_p).rsplit("/", 1)         stem = base[-1].rsplit(".", 1)[0]         lbr_p = (base[0] + "/" if len(base) > 1 else "") + stem + ".lbr"-    mode = "patched" if _os.path.exists(lbr_p) else "created"+    cmd = [tool, "generate"]+    if sym_p:+        cmd += ["--sym", sym_p]+    if mod_p:+        cmd += ["--fp", mod_p]+    if args.get("name"):+        cmd += ["--name", str(args["name"])]+    cmd += ["-o", lbr_p]     try:-        if mode == "patched":-            original = open(lbr_p, "r", encoding="utf-8", errors="replace").read()-            new_text, report = _kl.patch_lbr(original, kicad_mod_text=mod_text,-                                             kicad_sym_text=sym_text, name=args.get("name"))-            if new_text == original:-                report["unchanged"] = True-            else:-                try:-                    open(lbr_p + ".bak", "w", encoding="utf-8").write(original)-                except Exception:-                    pass-                open(lbr_p, "w", encoding="utf-8").write(new_text)-        else:-            new_text, report = _kl.build_lbr(kicad_sym_text=sym_text, kicad_mod_text=mod_text,-                                             name=args.get("name"),-                                             prefix=args.get("prefix") or "U")-            open(lbr_p, "w", encoding="utf-8").write(new_text)+        r = _sp.run(cmd, capture_output=True, text=True, timeout=120)     except Exception as e:-        return {"success": False, "errorCode": "kicad_conversion_failed",-                "error": "Conversion failed: %s" % str(e)[:200]}+        return {"success": False, "errorCode": "adom_lbr_failed",+                "error": "adom-lbr did not run: %s" % str(e)[:150]}+    if r.returncode != 0:+        return {"success": False, "errorCode": "adom_lbr_failed",+                "error": "adom-lbr generate failed (exit %d): %s"+                         % (r.returncode, (r.stderr or r.stdout)[-300:]),+                "_hint": "adom-lbr auto-lints its output and exits 2 on lint failure - the "+                         "message above names the exact check that failed."}     names = _library_artifact_names(lbr_p)-    return {"success": True, "libraryPath": lbr_p, "mode": mode, "report": report,+    return {"success": True, "libraryPath": lbr_p, "converter": "adom-lbr",             "symbols": names.get("sym", []), "packages": names.get("pac", []),             "devicesets": names.get("dev", []), "has3d": bool(names.get("p3d")),-            "sources": {"kicadFootprint": mod_p or None, "kicadSymbol": sym_p or None},-            "_hint": ("%s %s from KiCad sources. fusion_show_symbol / show_footprint consume "-                      "it directly; bind the 3D with fusion_bind_library_3d {libraryPath, "-                      "stepPath} (has3d is %s), then fusion_prove_library captures the four "-                      "evidence frames. A .bak sits beside a patched file."-                      % (mode.capitalize(), lbr_p, bool(names.get("p3d"))))}+            "toolOutput": (r.stdout or "").strip()[-300:],+            "_hint": ("Generated by adom-lbr (lint-clean by construction). fusion_show_symbol / "+                      "show_footprint consume it directly; bind the 3D with "+                      "fusion_bind_library_3d {libraryPath, stepPath}, then "+                      "fusion_prove_library captures the four evidence frames.")}   COMMAND_HANDLERS["import_kicad"] = _handle_import_kicad