Fusion - the Fusion 360 Bridge

@@ -3405,9 +3405,19 @@ def _orchestrate_open_lbr(args: dict) -> dict:     # Remember it so a later capture/EDIT can pre-check names offline instead of letting     # Fusion raise "Create new footprint 'X'?" and dismissing it after the fact.     _remember_open_library(file_path)+    # BUBBLE THE PART-NUMBER GAP UP WITH THE IMPORT (John, 2026-08-28). The import is NOT+    # blocked by missing part numbers - a library without them is still a real library and+    # the user asked for it - but the gap has to reach the top (the wiki's quick actions+    # UI, or web control) so they see "2 of my 5 components had no Mouser part number" and+    # can go fix it where the data lives. We are the only thing that has read the file.+    try:+        _pm = _part_metadata_report(file_path)+    except Exception:+        _pm = None     response = {         "success": True,         "output": f"Opened library: {os.path.basename(file_path)}",+        "partMetadata": _pm,         "data": {"results": results, "filePath": file_path},         "_hint": (             "Library opened in the Electronics Library editor (Content Manager). "@@ -11280,6 +11290,82 @@ def _lint_part_metadata(path: str) -> dict:     return {"ok": not warns, "warnings": warns, "totals": md["totals"]}  +def _part_metadata_report(path: str) -> dict:+    """A SHORT, user-facing metadata report an import can carry all the way up the chain.++    John, 2026-08-28: importing must NOT be blocked by missing part numbers, but the gap has+    to "bubble up all the way to the top, like in the wiki quick actions UI or via web+    control so the user sees 'oh, 2 of my 5 components had no mouser part number' so they+    can go fix it". The bridge is the end of the chain and cannot invent the data - but it+    is the only thing that has actually read the file, so it is the only thing that can say+    what is missing. Silence here means the user finds out at ordering time.++    Counts are per COMPONENT (deviceset), because that is the unit a user thinks in, with+    per-variant detail underneath for the automotive-vs-baseline case.+    """+    md = _library_part_metadata(path)+    if not md.get("ok"):+        return {"ok": None, "unavailable": md.get("error", "could not read the library")}+    lint = _lint_part_metadata(path)+    # TWO SEVERITIES, and conflating them made a GOOD library look broken (caught by my own+    # test): a part needs an MPN and a manufacturer, and it needs AT LEAST ONE distributor -+    # not all three. A library with DigiKey on one variant and Mouser on another is+    # perfectly orderable, and reporting "no LCSC part number" as a deficiency there is+    # noise that trains people to ignore the whole report.+    #   missing            -> actionable: cannot order this component+    #   absentDistributors -> informational: which catalogues lack a number, John's+    #                         "2 of my 5 components had no mouser part number"+    dist = ("MOUSER_PN", "DIGIKEY_PN", "LCSC_PN")+    fields = ("MPN", "MANUFACTURER") + dist+    comps, missing_counts = [], {f: 0 for f in fields}+    n_no_distributor = 0+    for ds in md["devicesets"]:+        vs = ds["variants"] or []+        # A component is missing a field when NO variant of it carries that field.+        absent = [f for f in fields if not any(v["attributes"].get(f) for v in vs)]+        for f in absent:+            missing_counts[f] += 1+        blocking = [f for f in ("MPN", "MANUFACTURER") if f in absent]+        if all(d in absent for d in dist):+            blocking.append("a distributor part number")+            n_no_distributor += 1+        if blocking or [d for d in dist if d in absent]:+            comps.append({"component": ds["name"], "variants": len(vs),+                          "missing": blocking,+                          "absentDistributors": [d for d in dist if d in absent],+                          "variantNames": [v["fullName"] for v in vs]})+    total = len(md["devicesets"])+    label = {"MPN": "no manufacturer part number", "MANUFACTURER": "no manufacturer",+             "MOUSER_PN": "no Mouser part number", "DIGIKEY_PN": "no DigiKey part number",+             "LCSC_PN": "no LCSC part number"}++    def _n(n, what):+        return "%d of %d component%s ha%s %s" % (n, total, "" if total == 1 else "s",+                                                 "s" if n == 1 else "ve", what)+    bits = [_n(missing_counts[f], label[f]) for f in ("MPN", "MANUFACTURER")+            if missing_counts[f]]+    if n_no_distributor:+        bits.append(_n(n_no_distributor, "no distributor part number at all"))+    return {+        "ok": lint["ok"],+        "componentsTotal": total,+        "variantsTotal": md["totals"]["variants"],+        "missingCounts": missing_counts,+        "componentsWithoutAnyDistributor": n_no_distributor,+        "distributorCoverage": {d: total - missing_counts[d] for d in dist},+        "components": comps[:40],+        "warnings": lint["warnings"],+        "summary": ("; ".join(bits) + ". The library imported fine - this is about ordering, "+                    "not geometry." ) if bits else+                   "All %d component(s) carry a manufacturer part number and at least one "+                   "distributor part number." % total,+        "distributorNote": ("; ".join(_n(missing_counts[d], label[d]) for d in dist+                                      if missing_counts[d]) or None),+        "userAction": ("Add the missing part numbers where the component metadata lives "+                       "(upstream of this bridge), then re-export." if bits else None),+    }++ def _handle_library_parts(fusion_info: dict, args: dict) -> dict:     """Every ORDERABLE variant in a .lbr with its part numbers. Offline, no Fusion needed. @@ -11887,6 +11973,16 @@ def _handle_create_library(fusion_info: dict, args: dict) -> dict:                 state.update(stage="done", success=bool(prove.get("proven")),                              verdict=("proven" if prove.get("proven") else "not_proven"),                              evidenceCount=_frames, prove=prove)+            # The part-number gap rides the RUN STATE, so web control sees it on the same+            # poll it already makes rather than needing a second call (John, 2026-08-28).+            # Attached on the failure path too: a chain that could not prove the geometry+            # still read the file, and the user still wants to know what is missing.+            try:+                _lbr_for_meta = state.get("libraryPath") or target_lbr+                if _lbr_for_meta:+                    state["partMetadata"] = _part_metadata_report(_lbr_for_meta)+            except Exception:+                pass             _save()         except Exception as e:             state.update(stage="failed", success=False, verdict="not_proven",