Altium - the Altium Bridge

@@ -4,43 +4,57 @@ use crate::ab; use serde_json::{json, Value}; -/// Window titles Altium uses for script/engine modals, plus the generic ones worth surfacing.-const MODAL_TITLES: [&str; 6] = ["Error", "Information", "Confirm", "Warning", "Altium Designer", "Script"];+/// Dialog window classes, from the Python bridge's measured list. NOTE `#32770` is the generic+/// Win32 dialog class — half the desktop uses it — so class alone is NEVER enough to decide a+/// window is Altium's. See `scan()`.+const DIALOG_CLASSES: [&str; 5] = ["TMessageForm", "#32770", "TForm", "TMessageDialog", "TForm_RunScriptParameters"]; -/// Windows that merely CONTAIN these are the app itself / our own tooling, never a modal.-fn is_main_window(title: &str) -> bool {-    crate::health::parse_title(title).is_some()-}--/// Enumerate blocking dialogs. Returns [] when the screen is clean.+/// Every dialog owned by an Altium process.+///+/// Scoped by PID, not by title. The first cut of this matched window titles containing "Error" /+/// "Warning" and immediately picked up four **"RemoteApp security warning"** windows that had+/// nothing to do with Altium (2026-09-01, on John's desktop) — which `dismiss_dialogs` would then+/// have clicked OK on. Pressing buttons in someone else's application is exactly the kind of thing+/// this bridge must never do, so: ask ab which pids are Altium (`process_list`), enumerate only+/// those windows (`desktop_enum_windows {pids}`), and let class + "not the main window" decide. pub fn scan() -> Vec<Value> {-    let mut out: Vec<Value> = Vec::new();-    let mut seen: Vec<i64> = Vec::new();-    for t in MODAL_TITLES {-        let r = ab::call("desktop_find_window", json!({"titleContains": t}), &[], 10);-        let wins = r-            .get("windows")-            .and_then(|a| a.as_array())-            .cloned()-            .unwrap_or_default();-        for w in wins {-            let hwnd = match w.get("hwnd").and_then(|h| h.as_i64()) {-                Some(h) => h,-                None => continue,-            };-            let title = w.get("title").and_then(|s| s.as_str()).unwrap_or("").to_string();-            if seen.contains(&hwnd) || is_main_window(&title) || title.is_empty() {-                continue;-            }-            seen.push(hwnd);-            out.push(json!({"hwnd": hwnd, "title": title,-                            "text": dialog_text(hwnd),-                            "buttons": buttons(hwnd)}));+    let pids: Vec<u64> = ab::call("process_list", json!({"nameContains": "X2"}), &[], 15)+        .get("processes")+        .and_then(|a| a.as_array())+        .map(|a| a.iter().filter_map(|p| p.get("pid").and_then(|v| v.as_u64())).collect())+        .unwrap_or_default();+    if pids.is_empty() {+        return Vec::new(); // Altium is not running — nothing of ours can be on screen+    }+    let r = ab::call("desktop_enum_windows", json!({"pids": pids}), &[], 20);+    let wins = r.get("windows").and_then(|a| a.as_array()).cloned().unwrap_or_default();+    let mut out = Vec::new();+    for w in wins {+        let title = w.get("title").and_then(|s| s.as_str()).unwrap_or("").to_string();+        let class = w.get("className").and_then(|s| s.as_str()).unwrap_or("").to_string();+        let hwnd = match w.get("hwnd").and_then(|h| h.as_i64()) {+            Some(h) => h,+            None => continue,+        };+        let visible = w.get("visible").and_then(|b| b.as_bool()).unwrap_or(true);+        if !visible || title.is_empty() || is_main_window(&title) {+            continue;         }+        if !DIALOG_CLASSES.iter().any(|c| c.eq_ignore_ascii_case(&class)) {+            continue;+        }+        out.push(json!({"hwnd": hwnd, "title": title, "className": class,+                        "pid": w.get("pid").cloned().unwrap_or(Value::Null),+                        "text": dialog_text(hwnd), "buttons": buttons(hwnd)}));     }     out } +/// Windows that merely CONTAIN the app name are the app itself, never a modal.+fn is_main_window(title: &str) -> bool {+    crate::health::parse_title(title).is_some()+}+ /// The message body — this is what tells a save prompt apart from a compile error. fn dialog_text(hwnd: i64) -> Value {     let r = ab::call("desktop_ui_text", json!({"hwnd": hwnd}), &[], 10);