Fusion - the Fusion 360 Bridge

@@ -4007,6 +4007,37 @@ def _orchestrate_make_3d_package(args: dict) -> dict:   _BUILD_IN_FLIGHT = {}          # lbrPath -> epoch started; page issue #62+_BUILD_FLIGHT_FILE = os.path.join(os.path.expanduser("~"), ".adom", "fusion-bridge",+                                  "builds_in_flight.json")+++def _flight_load() -> dict:+    """In-flight builds recorded on DISK, because AD reaps and respawns this bridge.++    Peer lesson (pup thread, 2026-08-28): a guard that reads its own in-memory bookkeeping+    "passes vacuously" the moment that record is reset while the real condition persists.+    A multi-part 3D build runs for minutes and a respawn in the middle emptied this dict,+    so the build_in_progress refusal silently stopped firing. (The wrong_editor_captured+    check still caught the resulting frames, so this was wasted work rather than a false+    pass - but a guard that only sometimes fires is not a guard.)+    """+    try:+        import json as _js+        with open(_BUILD_FLIGHT_FILE, encoding="utf-8") as f:+            return {k: float(v) for k, v in (_js.load(f) or {}).items()}+    except Exception:+        return {}+++def _flight_save():+    try:+        import json as _js, pathlib as _pl+        _pl.Path(_BUILD_FLIGHT_FILE).parent.mkdir(parents=True, exist_ok=True)+        with open(_BUILD_FLIGHT_FILE, "w", encoding="utf-8") as f:+            _js.dump(_BUILD_IN_FLIGHT, f)+    except Exception as e:+        print("[Fusion Bridge] could not persist in-flight builds (%s) - the "+              "build_in_progress guard will not survive a respawn" % e)   def _build_in_flight():@@ -4020,6 +4051,9 @@ def _build_in_flight():     """     import time as _t     now = _t.time()+    # Merge the on-disk record so a respawn does not silently disarm this.+    for k, v in _flight_load().items():+        _BUILD_IN_FLIGHT.setdefault(k, v)     # A build that has run absurdly long is treated as finished rather than blocking     # captures forever - a stuck marker must not become a permanent refusal.     return {k: v for k, v in _BUILD_IN_FLIGHT.items() if now - v < 3600}@@ -4074,6 +4108,7 @@ def _orchestrate_build_library_3d(args: dict) -> dict:          import time as _tt, uuid as _uu         _BUILD_IN_FLIGHT[out_path] = _tt.time()+        _flight_save()         _parts = args.get("parts") or []         _run_id = _uu.uuid4().hex[:12]         _bstate = {"runId": _run_id, "verb": "fusion_build_library_3d",@@ -4102,6 +4137,7 @@ def _orchestrate_build_library_3d(args: dict) -> dict:                 # ALWAYS clear, even on failure: a marker that outlives its build would                 # refuse every later capture (page issue #62).                 _BUILD_IN_FLIGHT.pop(out_path, None)+                _flight_save()                 _publish_run_state(_run_id, _bstate)         _th.Thread(target=_run, daemon=True).start()         return {@@ -4552,13 +4588,26 @@ def _orchestrate_capture_library_views(args: dict) -> dict:                 if editor_ws and "electronics library" not in str(editor_ws).lower():                     editor_ok = False             except Exception:-                editor_ws = None          # unknown, not a pass and not a failure+                # "unknown, not a pass and not a failure" is what the comment USED to say+                # while the code left editor_ok True, i.e. it passed. A frame we cannot+                # attribute to the right editor is not evidence: if the add-in did not+                # answer, we do not know what was on screen, and ok:true would be a claim+                # we cannot support. Reported as unverified instead (peer lesson, pup+                # 2026-08-28: never default an unreadable condition to "fine").+                editor_ws, editor_ok = None, False             entry = {"package": pkg, "view": label_of.get(ext, ext), "path": shot,                      "dialogDismissed": dismissed or None,                      "capturedInWorkspace": editor_ws,                      "ok": (bool(er.get("success", True)) and bool(shot)                             and not dismissed and editor_ok)}-            if not editor_ok:+            if not editor_ok and editor_ws is None:+                entry["errorCode"] = "editor_unverified"+                entry["error"] = ("Could not read which editor was active at capture time "+                                  "(the add-in did not answer), so this frame cannot be "+                                  "confirmed to show the view it is labelled with.")+                entry["_hint"] = ("Not necessarily a bad frame - it is an unverified one. "+                                  "Check fusion_addin_status and re-capture.")+            elif not editor_ok:                 entry["errorCode"] = "wrong_editor_captured"                 entry["error"] = ("Captured while %r was active, not the Electronics Library "                                   "editor - the frame does not show the %s view it is "