Fusion - the Fusion 360 Bridge

@@ -6772,7 +6772,8 @@ def _library_artifact_names(lbr_path: str) -> dict:         return {}     def names(tag):         return [e.get("name") for e in root.iter(tag) if e.get("name")]-    return {"sym": names("symbol"), "pac": names("package"), "dev": names("deviceset")}+    return {"sym": names("symbol"), "pac": names("package"), "dev": names("deviceset"),+            "p3d": [e.get("name") for e in root.iter("package3d") if e.get("name")]}   def _resolve_artifact_name(name: str, suffix: str, lbr_path: str):@@ -6797,12 +6798,28 @@ def _resolve_artifact_name(name: str, suffix: str, lbr_path: str):     ci = [h for h in have if h.lower() == str(name).lower()]     if len(ci) == 1:         return ci[0], None+    # A CALLER SHOULD NEVER PARSE OUR FILE FORMAT TO TALK TO US (page issue #42). ab ended up+    # regexing <symbol name="..."> out of the .lbr because the trailing character differed+    # (file CL21B104KBCNFNC.lbr vs deviceset CL21B104KBCNFN). We have the library open and we+    # know the names, so resolve the near-miss ourselves and, when we cannot, put the names in+    # the error so the retry is obvious without reading XML.+    import difflib as _dl+    close = _dl.get_close_matches(str(name), have, n=5, cutoff=0.6)+    stem = str(name).rsplit("/", 1)[-1].rsplit(".", 1)[0]+    pref = [h for h in have if h.lower().startswith(stem.lower()[:max(4, len(stem) - 2)])]+    if len(close) == 1 and close[0].lower() != str(name).lower():+        print("[Fusion Bridge] resolved near-miss %r -> %r (issue #42)" % (name, close[0]))+        return close[0], None+    if len(pref) == 1:+        print("[Fusion Bridge] resolved by stem %r -> %r (issue #42)" % (name, pref[0]))+        return pref[0], None     return None, {         "success": False,         "error": "No %s named %r in that library, so the EDIT was NOT issued." % (             {"sym": "symbol", "pac": "footprint", "dev": "device"}.get(suffix, suffix), name),         "errorCode": "artifact_not_in_library",         "available": have[:40],+        "closestMatches": close,         "_hint": ("Deliberately refused BEFORE touching Fusion: EDIT on a missing name pops a "                   "'Create new ...?' modal that blocks the add-in's main thread and makes every "                   "later verb look like a crash. Pick a name from `available` (they are the "@@ -7012,6 +7029,24 @@ def _orchestrate_show_3d_package(args: dict) -> dict:     opened = _orchestrate_open_lbr({"filePath": library_path})     if not opened.get("success"):         return {"success": False, "error": "open_lbr failed: " + str(opened.get("error"))}+    # ⚠️ A PLACEHOLDER IS NOT A RESULT (page issue #42). With no <package3d> bound, Fusion+    # renders its grey Placeholder slab and this verb used to return success - ab's card+    # captured that, counted three artifacts, and reported the library PROVEN while a human+    # looking at the screenshot saw a featureless block where their capacitor should be.+    # Technically truthful, practically a false pass. Same class as a 237x39 "screenshot":+    # a non-result reported as success. The .lbr answers this offline, before we render.+    _p3d = _library_artifact_names(library_path).get("p3d", [])+    if not _p3d:+        return {"success": False,+                "error": "That library has no 3D package bound, so the only thing to show is "+                         "Fusion's grey Placeholder - refusing to report that as a 3D package.",+                "errorCode": "no_3d_package_bound",+                "libraryPath": library_path,+                "_hint": ("Bind a real model first: re-call fusion_show_3d_package with "+                          "stepPath pointing at the STEP file and the bridge attaches it onto "+                          "the footprint, then shows it. A Placeholder rendering LOOKS like a "+                          "successful 3D view in a screenshot, which is why this refuses "+                          "rather than letting a caller count it as proof.")}     cp = _proxy_to_addin("execute_text_command", {"command": "Electron.Create3DPackage %s" % library_path}, timeout=60)     if not cp.get("success"):         return {"success": False, "error": "Create3DPackage failed: " + str(cp.get("error")),@@ -8779,6 +8814,39 @@ COMMAND_HANDLERS["set_activity_indicator"] = _handle_set_activity_indicator COMMAND_HANDLERS["verb_times"] = _handle_verb_times  ++def _handle_list_library(fusion_info: dict, args: dict) -> dict:+    """What is actually IN a .lbr: symbols, packages, devicesets, package3d.++    ab asked for this (page issue #42) after resorting to regexing our XML from their client.+    A caller reading the bridge's own file format to work out what to ask the bridge for is a+    smell, and this is the cheap end of fixing it.+    """+    path = (args.get("libraryPath") or args.get("filePath") or "").replace("\\", "/")+    if not path:+        return {"success": False, "error": "libraryPath required.",+                "errorCode": "library_path_required"}+    names = _library_artifact_names(path)+    if not names:+        return {"success": False, "error": "Could not read or parse that .lbr: %s" % path,+                "errorCode": "library_unreadable",+                "_hint": "Path must be the WINDOWS path to an existing .lbr on the box."}+    return {"success": True,+            "libraryPath": path,+            "symbols": names.get("sym", []),+            "packages": names.get("pac", []),+            "devicesets": names.get("dev", []),+            "package3d": names.get("p3d", []),+            "has3d": bool(names.get("p3d")),+            "_hint": ("Use these EXACT names with fusion_show_symbol / show_footprint / "+                      "show_3d_package - no need to parse the .lbr yourself. has3d:false means "+                      "show_3d_package would only have a grey Placeholder to render, so pass "+                      "stepPath to bind a real model first.")}+++COMMAND_HANDLERS["list_library"] = _handle_list_library++ def dispatch_command(command: str, args: dict, caller_identity: dict = None) -> dict:     """Thin wrapper: run the verb, then guarantee the windowEtiquette annotation lands on     EVERY backgrounded window-raising result - the inner dispatcher's closure only covered