fix: prevent duplicate ADD captures by making on_navigation the only board action handler

src-tauri/src/lib.rs

@@ -34,36 +34,10 @@ pub fn run() {
             let query = uri.split('?').nth(1).unwrap_or("");
             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();
             match action.as_str() {
-                "library" => {
-                    let url = params.get("url").cloned().unwrap_or_default();
-                    if !url.is_empty() {
-                        let source = params.get("source").cloned();
-                        let title = params.get("title").cloned();
-                        tauri::async_runtime::spawn(async move { let _ = crate::library::library_add_item(app, url, source, title).await; });
-                    }
-                }
-                "board" => {
-                    let url = params.get("url").cloned().unwrap_or_default();
-                    if !url.is_empty() {
-                        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);
-                        tauri::async_runtime::spawn(async move {
-                            match crate::library::library_add_item(app.clone(), url.clone(), Some(source.clone()), Some(title.clone())).await {
-                                Ok(item) => {
-                                    let _ = crate::board::board_add_image(app.clone(), Some(item.id.clone()), item.data_url.clone(), 120.0, 120.0, 300.0, 200.0);
-                                    let _ = app.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: still add visible remote image to the canvas.
-                                    // This mirrors snip/webclip behavior: user action must produce a board item.
-                                    let _ = app.emit("board://image_added", serde_json::json!({"url": url, "dataUrl": url, "sourceUrl": source, "title": title, "width": w, "height": h, "colors": [], "downloadError": e}));
-                                }
-                            }
-                        });
-                    }
-                }
+                // IMPORTANT: board/library ADD is intentionally NOT processed here.
+                // The hover overlay uses top-level navigation and browser::commands::on_navigation
+                // is the single canonical handler. Processing it here too caused duplicate images.
+                "board" | "library" => {}
                 "vault" => {
                     let va = params.get("action").cloned().unwrap_or_default();
                     match va.as_str() {