fix: process muse-action:// inside on_navigation (scheme protocol doesn't fire for blocked navs)

src-tauri/src/browser/commands.rs

@@ -1,5 +1,5 @@
 use tauri::webview::{PageLoadEvent, WebviewBuilder};
-use tauri::{AppHandle, LogicalPosition, LogicalSize, Manager, Url, WebviewUrl};
+use tauri::{AppHandle, Emitter, LogicalPosition, LogicalSize, Manager, Url, WebviewUrl};
 
 use super::layout::{hide_tab, resize_active, show_tab};
 use super::navigation::{navigation_blocked, resolve_url};
@@ -113,7 +113,13 @@ pub(crate) async fn create_tab_inner(app: &AppHandle, url: &str, _layout: &Viewp
         .on_document_title_changed(move |_wv, title| { update_tab_field(&app_for_title, &id_for_title, |t| { if !title.trim().is_empty() { t.title = title.clone(); } }); })
         .on_navigation({ let app_nav = app.clone(); move |url| {
             let s = url.as_str();
-            if s.starts_with("muse-action://") { return false; }
+            // Process muse-action:// HERE because register_uri_scheme_protocol does NOT fire
+            // for navigations that are blocked (returning false prevents the request from
+            // reaching the network layer where the scheme protocol lives).
+            if s.starts_with("muse-action://") {
+                handle_muse_action(&app_nav, s);
+                return false; // Block navigation — page stays on current URL
+            }
             if navigation_blocked(&app_nav, s) { return false; }
             true
         }});
@@ -130,4 +136,44 @@ pub(crate) async fn create_tab_inner(app: &AppHandle, url: &str, _layout: &Viewp
     emit_snapshot(app)?; Ok(id)
 }
 
+/// Process muse-action:// URLs from the hover overlay.
+/// Called from on_navigation (the only reliable interception point for child webview navigations).
+fn handle_muse_action(app: &AppHandle, raw: &str) {
+    let rest = raw.trim_start_matches("muse-action://");
+    let (action, query) = rest.split_once('?').unwrap_or((rest, ""));
+    let params: std::collections::HashMap<String, String> = query.split('&').filter_map(|pair| { let (k, v) = pair.split_once('=')?; Some((percent_decode(k), percent_decode(v))) }).collect();
+    let url = params.get("url").cloned().unwrap_or_default();
+    if url.is_empty() { return; }
+    let source = params.get("source").cloned().unwrap_or_default();
+    let title = params.get("title").cloned().unwrap_or_else(|| "Web Reference".to_string());
+    let w = params.get("w").and_then(|s| s.parse::<u32>().ok()).unwrap_or(300);
+    let h = params.get("h").and_then(|s| s.parse::<u32>().ok()).unwrap_or(200);
+    let action = action.trim_end_matches('/').to_string();
+    let app2 = app.clone();
+
+    tauri::async_runtime::spawn(async move {
+        match crate::library::library_add_item(app2.clone(), url.clone(), Some(source.clone()), Some(title.clone())).await {
+            Ok(item) => {
+                if action == "board" {
+                    let _ = crate::board::board_add_image(app2.clone(), Some(item.id.clone()), item.data_url.clone(), 120.0, 120.0, 300.0, 200.0);
+                }
+                let _ = app2.emit("board://image_added", serde_json::json!({
+                    "libraryId": item.id, "dataUrl": item.data_url, "url": item.url,
+                    "sourceUrl": item.source_url, "title": item.title,
+                    "width": item.width, "height": item.height, "colors": item.colors
+                }));
+            }
+            Err(_e) => {
+                // Fallback: emit with raw URL so image still appears on canvas (as remote img)
+                let _ = app2.emit("board://image_added", serde_json::json!({
+                    "url": url, "dataUrl": url, "sourceUrl": source, "title": title,
+                    "width": w, "height": h, "colors": []
+                }));
+            }
+        }
+    });
+}
+
+fn percent_decode(s: &str) -> String { let bytes = s.as_bytes(); let mut out = Vec::with_capacity(bytes.len()); let mut i = 0; while i < bytes.len() { if bytes[i] == b'%' && i + 2 < bytes.len() { if let Ok(hex) = std::str::from_utf8(&bytes[i+1..i+3]) { if let Ok(v) = u8::from_str_radix(hex, 16) { out.push(v); i += 3; continue; } } } out.push(if bytes[i] == b'+' { b' ' } else { bytes[i] }); i += 1; } String::from_utf8_lossy(&out).to_string() }
+
 const FAVICON_SCRIPT: &str = r#"(function(){window.__muse_report_favicon=function(){const link=document.querySelector('link[rel~="icon"],link[rel="shortcut icon"],link[rel="apple-touch-icon"]');if(link&&link.href){try{window.__TAURI_INTERNALS__.invoke('__tab_favicon',{favicon:link.href});}catch{}}};if(document.readyState==='loading'){document.addEventListener('DOMContentLoaded',window.__muse_report_favicon);}else{window.__muse_report_favicon();}})();"#;