Altium - the Altium Bridge

@@ -1399,29 +1399,53 @@ def h_read_schematic(args):   def h_cross_check(args):-    """Compare the current schematic sheet against the current board.+    """Compare a schematic sheet against a board: what is drawn but not placed, and placed but not+    drawn. Neither read answers this alone, which is the point. -    Neither read answers this alone, which is the point: what is drawn but not placed, and what is-    placed but not drawn. Two caveats travel WITH the answer because both make it wrong to act on:-    the schematic side is one sheet, and Altium's SchServer and PCBServer each hold their own-    "current document" — so nothing stops the two belonging to different projects."""+    Altium CANNOT hold both current at once — `GetCurrentPCBBoard` follows the focused document, so+    focusing the sheet clears the board (measured 2026-09-01, which is why the first cut of this+    verb always failed on the second read). It therefore reads them in TURN: activate the board,+    read it, activate the sheet, read it, then put back whatever the user had in front of them.++    Pass `board` and `sheet` paths, or leave them out to use whatever is already open.+    """     g = _need_licensed()     if g:         return g-    sr = agent.call("read_schematic", {"sections": "components"}, timeout=float(args.get("timeout", 90)))-    sd = sr.get("data") or {}-    if not sd.get("ok"):-        return {"success": False, "stage": "schematic", "error": sd.get("error"),-                "_hint": "Activate a leaf .SchDoc (altium_activate_document), then retry."}-    br = agent.call("read_board", {"sections": "components"}, timeout=float(args.get("timeout", 90)))-    bd = br.get("data") or {}-    if not bd.get("ok"):-        return {"success": False, "stage": "board", "error": bd.get("error"),-                "_hint": "A .PcbDoc must be current — altium_activate_document."}-    sch = sorted({c.get("designator") for c in ((sd.get("components") or {}).get("components") or [])-                  if c.get("designator")}, key=_natural_key)+    was = altium_detect.licence_state().get("document")  # restore the user's view at the end+    board_path, sheet_path = args.get("board"), args.get("sheet")+    timeout = float(args.get("timeout", 45))++    def _read(kind, path):+        if path:+            bad = _validate_openable(path)+            if bad:+                return None, bad+            o = agent.call("open", {"path": path}, timeout=timeout)+            if not (o.get("data") or {}).get("ok", True):+                return None, {"success": False, "stage": kind, "error": (o.get("data") or {}).get("error"),+                              "_hint": f"could not open {path}"}+        cmd = "read_board" if kind == "board" else "read_schematic"+        r = agent.call(cmd, {"sections": "components"}, timeout=timeout)+        d = r.get("data") or {}+        if not d.get("ok"):+            return None, {"success": False, "stage": kind, "error": d.get("error"),+                          "_hint": (f"No current {'PCB board' if kind == 'board' else 'schematic sheet'}. "+                                    f"Pass {kind}:<path>, or open one first — Altium can only hold ONE "+                                    "document current, so cross_check must switch between them.")}+        return d, None++    bd, err = _read("board", board_path)+    if err:+        return err+    sd, err = _read("sheet", sheet_path)+    if err:+        return err+     pcb = sorted({c.get("designator") for c in ((bd.get("components") or {}).get("components") or [])                   if c.get("designator")}, key=_natural_key)+    sch = sorted({c.get("designator") for c in ((sd.get("components") or {}).get("components") or [])+                  if c.get("designator")}, key=_natural_key)     ss, ps = set(sch), set(pcb)     doc, board = sd.get("document"), bd.get("board") @@ -1439,13 +1463,13 @@ def h_cross_check(args):            "onSheetNotOnBoard": sorted(ss - ps, key=_natural_key),            "onBoardNotOnSheet": sorted(ps - ss, key=_natural_key),            "schematicComponents": len(sch), "boardComponents": len(pcb),-           "roundtripMs": (sr.get("roundtripMs") or 0) + (br.get("roundtripMs") or 0),-           "_scope": "The schematic side is the CURRENT SHEET only. On a hierarchical design, read every "-                     "leaf sheet before calling anything 'missing from the schematic'."}+           "_scope": "The schematic side is ONE sheet. On a hierarchical design, run this per leaf sheet "+                     "before calling anything 'missing from the schematic'."}     if same is False:-        out["_warning"] = ("The current sheet and the current board are in DIFFERENT folders, so they are "-                           "probably not the same project — which makes every difference above meaningless. "-                           "Activate the right documents and re-run.")+        out["_warning"] = ("The sheet and the board are in DIFFERENT folders, so they are probably not the "+                           "same project — which makes every difference above meaningless.")+    if was:+        out["restoredDocument"] = was  # best effort: leave the user looking at what they were looking at     return out