Altium - the Altium Bridge

@@ -1386,6 +1386,7 @@ def h_read_schematic(args):     if g:         return g     r = agent.call("read_schematic", {"sections": args.get("sections") or "components",+                                      "params": "1" if args.get("params", True) else "0",                                       "limit": args.get("limit", 500)},                    timeout=float(args.get("timeout", 90)))     data = r.get("data") or {}@@ -1630,6 +1631,89 @@ def _chk_components_near_board_edge(board, sheet, margin_mil=20):     return out  ++# Adom house rule (John, standing): Mouser is the PREFERRED vendor, and every component carries+# Mouser + DigiKey + LCSC part numbers. Encoded as a check so a board can be held to it, rather+# than the rule living only in people's heads.+VENDOR_PATTERNS = {+    "Mouser": ("mouser",),+    "DigiKey": ("digikey", "digi-key", "digi key"),+    "LCSC": ("lcsc", "jlcpcb"),+}+# Designator prefixes that legitimately carry no vendor part number.+NON_PURCHASED_PREFIXES = ("TP", "FID", "MH", "H", "LOGO", "NET", "PWR", "GND")+++def _params_of(comp):+    return {(p.get("name") or "").strip().lower(): (p.get("value") or "").strip()+            for p in (comp.get("params") or []) if p.get("name")}+++def _is_purchased(designator):+    d = (designator or "").upper()+    prefix = "".join(ch for ch in d if ch.isalpha())+    return prefix not in NON_PURCHASED_PREFIXES+++def _chk_vendor_part_numbers(board, sheet):+    """warning — a purchased component missing Mouser / DigiKey / LCSC part numbers.++    Mouser first in the message because it is the preferred vendor: if only one is going to be+    filled in, that is the one. Test points, fiducials and mounting holes are exempt — they are not+    purchased parts and flagging them would train people to ignore the check."""+    if not sheet:+        return []+    out = []+    for c in (sheet.get("components") or {}).get("components") or []:+        des = c.get("designator")+        if not _is_purchased(des):+            continue+        params = _params_of(c)+        blob = " ".join(f"{k} {v}" for k, v in params.items())+        missing = [vendor for vendor, keys in VENDOR_PATTERNS.items()+                   if not any(k in blob for k in keys)]+        if missing:+            out.append({"check": "vendor_part_numbers", "severity": "warning", "designator": des,+                        "missing": missing,+                        "message": f"{des} has no {' / '.join(missing)} part number "+                                   f"(house rule: Mouser preferred, all three required)."})+    return out+++def _chk_missing_datasheets(board, sheet):+    """info — a purchased component with no datasheet link."""+    if not sheet:+        return []+    out = []+    for c in (sheet.get("components") or {}).get("components") or []:+        des = c.get("designator")+        if not _is_purchased(des):+            continue+        params = _params_of(c)+        if not any(k in ("datasheet", "helpurl", "datasheet link", "datasheeturl") or "datasheet" in k+                   for k in params):+            out.append({"check": "missing_datasheets", "severity": "info", "designator": des,+                        "message": f"{des} has no Datasheet parameter."})+    return out+++def _chk_schematic_placeholders(board, sheet):+    """error — a schematic component still carrying default text in Comment.++    The board-side check only sees placed parts; this catches it at the source, where it is+    cheaper to fix and where the BOM is actually generated from."""+    if not sheet:+        return []+    out = []+    for c in (sheet.get("components") or {}).get("components") or []:+        com = (c.get("comment") or "").strip()+        if com.lower() in PLACEHOLDER_VALUES:+            out.append({"check": "schematic_placeholder_values", "severity": "error",+                        "designator": c.get("designator"),+                        "message": f"{c.get('designator')} on the sheet has placeholder Comment {com!r}."})+    return out++ # (id, fn). The ID is what appears in a finding's `check` field, so `checks:["..."]` filters on the # name the caller actually sees. Deriving it from the Python function name instead was a trap: the # function is `_chk_components_outside_board`, the finding says `components_outside_board_outline`,@@ -1642,6 +1726,10 @@ BOARD_CHECKS = [     ("duplicate_designators", _chk_duplicate_designators),     ("components_outside_board_outline", _chk_components_outside_board),     ("components_near_board_edge", _chk_components_near_board_edge),+    # schematic-side: need includeSchematic:true, and quietly return nothing without it+    ("vendor_part_numbers", _chk_vendor_part_numbers),+    ("missing_datasheets", _chk_missing_datasheets),+    ("schematic_placeholder_values", _chk_schematic_placeholders), ]  # Named so a clean result is not mistaken for a complete one.@@ -1674,7 +1762,8 @@ def h_review_board(args):         return r     sheet = None     if args.get("includeSchematic"):-        sr = agent.call("read_schematic", {"sections": "components"}, timeout=float(args.get("timeout", 90)))+        sr = agent.call("read_schematic", {"sections": "components", "params": "1"},+                        timeout=float(args.get("timeout", 90)))         sd = sr.get("data") or {}         sheet = sd if sd.get("ok") else None     wanted = args.get("checks")