Fusion - the Fusion 360 Bridge

@@ -9194,6 +9194,17 @@ def _orchestrate_bind_library_3d(args: dict) -> dict:                       "fresh mint) - EDIT %s.dev then screenshot to verify the render." % deviceset)}  +def _lbr_has_package3d(lbr_path: str):+    """Does this .lbr carry a bound package3d? Returns True/False, or None when the file+    cannot be read (unknown is NOT False - a caller must be able to tell them apart)."""+    try:+        txt = open(lbr_path, encoding="utf-8", errors="replace").read()+    except Exception:+        return None+    m = re.search(r'<package3d[^>]*(?:\burn|\bwip_urn)="([^"]+)"', txt)+    return bool(m and m.group(1).strip())++ def _orchestrate_show_library(args: dict) -> dict:     """Eight-surfaces contract (ab SDK, issue #30): the LIBRARY surface by its smallest     name. libraryPath opens it (open_lbr); name alone handles the already-open case."""@@ -9214,7 +9225,61 @@ def _orchestrate_show_library(args: dict) -> dict:                     "_hint": "Pass libraryPath (WINDOWS .lbr path) to open it; name alone only handles the already-open case."}     else:         return {"success": False, "error": "libraryPath or name required."}-    out = {"success": True, "view": "library"}+    # ── FINDING 2 (#58): ASSERT THE LIBRARY IS ACTUALLY THE ACTIVE DOCUMENT ────────────+    # One call returned success:true with Fusion sitting on an empty Untitled(1) DESIGN+    # tab, no Electronics panels at all, and nothing in the response distinguished it from+    # the nine that worked. Worse, the reporter's blank-frame smoke alarm RATED IT BEST:+    # 99.5% ink coverage vs 94.5-94.7% for the real ones, because a uniform dark canvas is+    # "inked" everywhere. A screenshot cannot be the proof here; the document name can.+    want = (name or (lbr.rsplit("/", 1)[-1].rsplit(".", 1)[0] if lbr else "")).strip()+    active = ""+    try:+        st = _proxy_to_addin("get_app_state", {}, timeout=12)+        active = str((st.get("data") or {}).get("activeDocument", "") or "")+    except Exception:+        pass+    if want and active and want.lower() not in active.lower():+        return {"success": False, "errorCode": "library_not_active",+                "error": "opened %r but the ACTIVE document is %r - the view you would "+                         "screenshot is not the library." % (want, active),+                "activeDocument": active, "expected": want,+                "_hint": ("Fusion switched or left another tab in front. Re-call "+                          "fusion_show_library {libraryPath} once; if it repeats, close the "+                          "stray document (fusion_close_document) and retry.")}++    out = {"success": True, "view": "library", "activeDocument": active or None}++    # ── THE HINT REQUEST (#58): say when there is no 3D to show ────────────────────────+    # The reporter called this verb alone, got a grey Placeholder in the 3D pane on 10/10+    # parts, and filed it as a bridge bug before finding their own documentation saying+    # fusion_bind_library_3d must run first. They are right that the bridge KNOWS this and+    # said nothing. threeDBound is null when unknowable (name-only call), never False.+    bound = _lbr_has_package3d(lbr) if lbr else None+    out["threeDBound"] = bound+    if bound is False:+        out["_hint"] = ("This library has NO package3d, so Fusion renders a grey Placeholder "+                        "slab in the 3D pane - that is not a captured 3D artifact. Run "+                        "fusion_bind_library_3d {libraryPath, stepPath} FIRST, then re-open. "+                        "(fusion_create_library does the whole chain for you.)")+    elif bound is None and lbr:+        out["_hint"] = ("Could not read the .lbr to tell whether a package3d is bound; if the "+                        "3D pane shows a grey Placeholder, run fusion_bind_library_3d first.")++    # ── FINDING 3 (#58): ten calls left ten tabs open ──────────────────────────────────+    # Not closed automatically: showing a library is a "leave it on screen" verb, so+    # closing it would defeat the call. Reported instead, so a batch caller can see the+    # accumulation it is causing rather than discovering it ten tabs later.+    try:+        _docs = _open_doc_names()+        if len(_docs) > 3:+            out["openDocuments"] = len(_docs)+            out["_hintDocs"] = ("%d documents are open in Fusion. show_library deliberately "+                                "leaves the library on screen, so a batch of calls accumulates "+                                "tabs - close them with fusion_close_all_documents between "+                                "runs." % len(_docs))+    except Exception:+        pass+     try:         shot = _handle_screenshot_fusion({}, {})         p = shot.get("savedTo") or shot.get("localSafePath")