Fusion - the Fusion 360 Bridge

@@ -81,6 +81,13 @@ fn remember_method(state_dir: &std::path::Path, method: &str, account: Option<&s     let _ = std::fs::write(method_pref_path(state_dir), json!({"method": method, "account": account}).to_string()); } +/// nbrowser_eval answers {data:{result}} or a JSON string in `output` ({ok, result}); read either.+fn eval_result(r: &Value) -> Value {+    for c in [&r["data"]["result"], &r["result"], &r["data"]["value"], &r["value"]] { if !c.is_null() { return c.clone(); } }+    if let Some(o) = r["output"].as_str() { if let Ok(v) = serde_json::from_str::<Value>(o) { if !v["result"].is_null() { return v["result"].clone(); } } }+    Value::Null+}+ /// Read the sign-in page through the browser extension (a window the BRIDGE opened, so it is /// drivable): what stage it is at, which methods it offers, which accounts a chooser lists. fn probe_page(ctx: &Ctx, session: &str) -> Value {@@ -96,8 +103,8 @@ fn probe_page(ctx: &Ctx, session: &str) -> Value {                signedIn: /you're signed in|youre signed in/i.test(document.body.innerText || '') };     })()"#;     let r = ab_call(ctx, "nbrowser_eval", json!({"sessionId": session, "expression": js}), 30);-    let d = if r["data"].is_object() { r["data"].clone() } else { r.clone() };-    d["result"].clone().take_if_object().unwrap_or(d["value"].clone())+    let v = eval_result(&r);+    if v.is_object() { v } else { Value::Null } }  trait TakeIfObject { fn take_if_object(self) -> Option<Value>; }@@ -105,13 +112,21 @@ impl TakeIfObject for Value { fn take_if_object(self) -> Option<Value> { if self  /// Click an element by its visible TEXT in the extension-driven window (no coordinates, no OCR). fn click_text(ctx: &Ctx, session: &str, needle: &str) -> bool {+    // Actionable elements only (a matched wrapper <div> swallows the click: the 1.11.1 loop clicked+    // one ten times on AdomLapper and the page never moved). Innermost match wins, then the click+    // goes to its closest button/link/row, and the answer says whether the page actually changed.     let js = format!(r#"(() => {{ const n = {needle:?}.toLowerCase();-      const els = [...document.querySelectorAll('button,a,[role=button],[data-identifier],li,div')];-      const hit = els.find(e => {{ const t = (e.innerText || e.textContent || '').trim().toLowerCase(); return t && t.includes(n) && t.length < 200 && e.offsetParent !== null; }});-      if (!hit) return false; hit.click(); return true; }})()"#);+      const vis = e => e.offsetParent !== null || e.getClientRects().length > 0;+      const txt = e => (e.innerText || e.textContent || '').trim().toLowerCase();+      const exact = n.includes('@') ? document.querySelector('[data-identifier="' + n + '"]') : null;+      const cands = exact ? [exact] : [...document.querySelectorAll('button,a,[role=button],[role=link],[data-identifier],li')].filter(e => vis(e) && txt(e).includes(n));+      if (!cands.length) return {{ clicked: false }};+      cands.sort((a, b) => txt(a).length - txt(b).length);+      const el = cands[0].closest('button,a,[role=button],[role=link],li') || cands[0];+      el.click();+      return {{ clicked: true, tag: el.tagName }}; }})()"#);     let r = ab_call(ctx, "nbrowser_eval", json!({"sessionId": session, "expression": js}), 30);-    let d = if r["data"].is_object() { r["data"].clone() } else { r.clone() };-    d["result"].as_bool().or(d["value"].as_bool()).unwrap_or(false)+    eval_result(&r)["clicked"].as_bool().unwrap_or(false) }  /// Drive the Autodesk sign-in page with the METHOD the user signs in with. Returns a typed@@ -155,7 +170,11 @@ pub fn drive_with_method(ctx: &Ctx, session: &str, method: Option<&str>, account             continue;         }         let label = format!("continue with {m}");-        if click_text(ctx, session, &label) { steps.push(format!("clicked '{label}'")); std::thread::sleep(std::time::Duration::from_secs(5)); continue; }+        if steps.iter().filter(|x| x.as_str() == format!("clicked '{label}'")).count() >= 2 {+            return json!({"stage": "click_did_not_advance", "methods": methods, "wanted": m, "steps": steps,+                "_hint": "The provider button was clicked twice and the page did not move. Read the window (fusion_state / a screenshot); the browser may be showing its own prompt."});+        }+        if click_text(ctx, session, &label) { steps.push(format!("clicked '{label}'")); std::thread::sleep(std::time::Duration::from_secs(4)); continue; }         return json!({"stage": "method_not_on_page", "methods": methods, "wanted": m, "steps": steps,             "_hint": "That method is not offered on this page. methods[] is what Autodesk shows; ask the user which one to use."});     }@@ -395,6 +414,17 @@ pub fn signin(ctx: &Ctx, args: &Value, env: &Env) -> Value {             "_hint": format!("The newest Fusion sign-in URL is {age}s old - Fusion expires the request in ~2 min, so completing it yields 'Sign-in request expired' and NO handoff. Get a FRESH request: fusion_stop then fusion_start (resets Fusion to a clean 'Welcome to Fusion'), click Fusion's Sign In, and call fusion_signin again PROMPTLY. Completion is near-instant when the target profile is already signed into Autodesk (auto-consents). Skill: fusion-multiprofile-signin."), "statusVerb": "fusion_signin"});     }     let mut reason = String::new();+    // The account the user asked for decides the browser profile: the profile SIGNED INTO that+    // identity already holds its Google/Microsoft session, so the provider step needs no password.+    // Picking "a profile with an Autodesk session" instead chose John's personal Chrome on+    // AdomLapper, where the work account's Google session had lapsed (2026-09-23).+    if target.is_none() {+        if let Some(a) = &account_arg {+            if let Some(c) = choices.iter().find(|c| c["email"].as_str().map(|e| e.eq_ignore_ascii_case(a)).unwrap_or(false)) {+                target = c["profile"].as_str().map(str::to_string); reason = format!("its browser profile is signed into {a}");+            }+        }+    }     if target.is_none() {         for c in choices.iter_mut() {             if c["live"].as_bool() != Some(true) { continue; }@@ -449,7 +479,7 @@ pub fn signin(ctx: &Ctx, args: &Value, env: &Env) -> Value {         let out = drive_with_method(ctx, &sess, method.as_deref(), acct.as_deref());         let stage = out["stage"].as_str().unwrap_or("").to_string();         // AI-answerable stages come back WITHOUT a toast; only a real human step toasts-        let human = matches!(stage.as_str(), "password_required" | "gave_up" | "unreadable");+        let human = stage == "password_required";         let notified = if human { notify_signin_wall(ctx, if stage == "password_required" { "password" } else { "credentials" }) } else { false };         if stage != "unreadable" {             let mut r = out.clone();