fix(workspace): correct clone progress math and persist task dismissal

- parse_git_clone_progress computed phase progress in u8, so
  percent.saturating_mul(80) clamped to 255 for any percent >= 4 and the
  receiving phase reported 5-7% for the whole download; widen to u16
  (with a 100 cap) before scaling, and cover the parser with unit tests
- split clone stderr chunks on both \r and \n before applying them:
  newline-terminated lines (Cloning into/remote:) used to merge with the
  next progress line into one multi-line detail string
- dismissing a finished task card now also removes it from the desktop
  registry via clone_dismiss (new Tauri command + gateway git action,
  write-gated in guard.go like clone_start/clone_cancel); previously the
  dismissed-set lived only in page state, so a WebUI refresh resurrected
  every historical card and terminal tasks leaked until app exit;
  in-flight tasks refuse dismissal
- relax the WebUI clone poll from 250ms to 750ms: each tick crosses the
  WS gateway into a desktop spawn_blocking round-trip, too chatty for
  remote/mobile sessions (desktop stays at 250ms over local IPC)

Author: su-fen

crates/agent-gateway/internal/protocol/pbws/guard.go

@@ -108,7 +108,7 @@ func vetAgentRequest(sm *session.Manager, env *gatewayv1.GatewayEnvelope) error
 // enable_web_git 门控，读操作（status/log/diff 等）始终放行。
 func gitActionIsWrite(action string) bool {
 	switch action {
-	case "clone", "clone_start", "clone_cancel", "init", "switch_branch", "create_branch", "stage", "stage_all", "unstage", "unstage_all", "discard", "discard_all", "add_to_gitignore", "commit", "fetch", "pull", "set_remote", "push", "delete_branch", "rename_branch", "stash_push", "stash_pop":
+	case "clone", "clone_start", "clone_cancel", "clone_dismiss", "init", "switch_branch", "create_branch", "stage", "stage_all", "unstage", "unstage_all", "discard", "discard_all", "add_to_gitignore", "commit", "fetch", "pull", "set_remote", "push", "delete_branch", "rename_branch", "stash_push", "stash_pop":
 		return true
 	default:
 		return false

crates/agent-gateway/web/src/app/GatewayApp.tsx

@@ -1205,7 +1205,7 @@ export default function GatewayApp() {
       void refreshWorkspaceCloneTasks().catch(() => {
         // Keep the last task snapshot visible while transport reconnects.
       });
-    }, 250);
+    }, 750);
     return () => window.clearInterval(timer);
   }, [hasActiveWorkspaceCloneTask, refreshWorkspaceCloneTasks]);
 
@@ -1226,10 +1226,26 @@ export default function GatewayApp() {
     [api],
   );
 
-  const handleDismissWorkspaceCloneTask = useCallback((taskId: string) => {
-    dismissedWorkspaceCloneTaskIds.current.add(taskId);
-    setWorkspaceCloneTasks((tasks) => tasks.filter((task) => task.id !== taskId));
-  }, []);
+  const handleDismissWorkspaceCloneTask = useCallback(
+    (taskId: string) => {
+      dismissedWorkspaceCloneTaskIds.current.add(taskId);
+      setWorkspaceCloneTasks((tasks) => tasks.filter((task) => task.id !== taskId));
+      if (!api) return;
+      // 服务端同步移除终态任务，否则刷新页面后快照会让卡片重现。
+      void api
+        .gitRequest<WorkspaceCloneTask[]>("clone_dismiss", "", { taskId })
+        .then((tasks) => {
+          dismissedWorkspaceCloneTaskIds.current.delete(taskId);
+          setWorkspaceCloneTasks(
+            tasks.filter((task) => !dismissedWorkspaceCloneTaskIds.current.has(task.id)),
+          );
+        })
+        .catch(() => {
+          // 本地 dismissed 集合已隐藏该卡片；服务端移除失败留待下次快照。
+        });
+    },
+    [api],
+  );
 
   const handleOpenClonedWorkspace = useCallback(
     (path: string) => {

crates/agent-gui/src-tauri/src/commands/workspace/git.rs

@@ -376,6 +376,24 @@ impl GitCloneTaskRegistry {
             .ok_or_else(|| "找不到克隆任务。".to_string())
     }
 
+    /// 终态任务的唯一清理路径：用户关闭任务卡时从注册表移除，
+    /// 否则刷新/重连后快照会让已关闭的卡片重现。运行中的任务拒绝移除。
+    pub fn dismiss(&self, id: String) -> Result<(), String> {
+        let mut tasks = self
+            .tasks
+            .lock()
+            .map_err(|_| "克隆任务注册表不可用。".to_string())?;
+        let id = id.trim();
+        let Some(entry) = tasks.get(id) else {
+            return Ok(());
+        };
+        if entry.task.status == "running" || entry.task.status == "cancelling" {
+            return Err("克隆任务仍在进行，无法移除。".to_string());
+        }
+        tasks.remove(id);
+        Ok(())
+    }
+
     fn run(
         self: Arc<Self>,
         id: String,
@@ -457,20 +475,25 @@ impl GitCloneTaskRegistry {
     }
 
     fn apply_output(&self, id: &str, chunk: &str) {
-        let detail = chunk.trim().to_string();
-        if detail.is_empty() {
-            return;
-        }
-        self.update(id, |task| {
-            if task.status != "running" {
-                return;
-            }
-            task.detail = detail.clone();
-            if let Some((phase, progress)) = parse_git_clone_progress(&detail) {
-                task.phase = phase.to_string();
-                task.progress = Some(progress);
+        // 读取线程按 \r 切块，但 git 的非进度输出（Cloning into/remote: 等）
+        // 以 \n 结尾，会与下一条进度行合并进同一 chunk——逐行处理防止
+        // detail 混入多行文本。
+        for line in chunk.split(['\r', '\n']) {
+            let detail = line.trim();
+            if detail.is_empty() {
+                continue;
             }
-        });
+            self.update(id, |task| {
+                if task.status != "running" {
+                    return;
+                }
+                task.detail = detail.to_string();
+                if let Some((phase, progress)) = parse_git_clone_progress(detail) {
+                    task.phase = phase.to_string();
+                    task.progress = Some(progress);
+                }
+            });
+        }
     }
 
     fn fail(&self, id: &str, error: String, target: &Path) {
@@ -525,15 +548,16 @@ fn parse_git_clone_progress(line: &str) -> Option<(&'static str, u8)> {
             .trim()
             .parse::<u8>()
             .ok()
+            .map(|percent| percent.min(100) as u16)
     };
     if let Some(percent) = parse_percent("Receiving objects:") {
-        return Some(("receiving", 5 + percent.saturating_mul(80) / 100));
+        return Some(("receiving", (5 + percent * 80 / 100) as u8));
     }
     if let Some(percent) = parse_percent("Resolving deltas:") {
-        return Some(("resolving", 85 + percent.saturating_mul(15) / 100));
+        return Some(("resolving", (85 + percent * 15 / 100) as u8));
     }
     if let Some(percent) = parse_percent("Checking out files:") {
-        return Some(("finalizing", 95 + percent.saturating_mul(5) / 100));
+        return Some(("finalizing", (95 + percent * 5 / 100) as u8));
     }
     if line.starts_with("remote:") || line.starts_with("Cloning into") {
         return Some(("preparing", 5));
@@ -3088,6 +3112,11 @@ pub(crate) fn git_gateway_clone_task_action_sync(
             .map_err(|error| format!("序列化 Git 响应失败：{error}")),
         "clone_cancel" => serde_json::to_value(registry.cancel(args.task_id.unwrap_or_default())?)
             .map_err(|error| format!("序列化 Git 响应失败：{error}")),
+        "clone_dismiss" => {
+            registry.dismiss(args.task_id.unwrap_or_default())?;
+            serde_json::to_value(registry.snapshot()?)
+                .map_err(|error| format!("序列化 Git 响应失败：{error}"))
+        }
         _ => git_gateway_action_sync(action, workdir, args_json),
     }
 }
@@ -3196,6 +3225,15 @@ pub fn git_clone_repository_cancel(
     registry.cancel(task_id)
 }
 
+#[tauri::command(rename_all = "snake_case")]
+pub fn git_clone_repository_dismiss(
+    registry: tauri::State<'_, Arc<GitCloneTaskRegistry>>,
+    task_id: String,
+) -> Result<Vec<GitCloneTask>, String> {
+    registry.dismiss(task_id)?;
+    registry.snapshot()
+}
+
 #[tauri::command(rename_all = "snake_case")]
 pub async fn git_list_remote_branches(
     remote_url: String,
@@ -3689,8 +3727,12 @@ mod tests {
             },
         );
 
-        let cancelled = registry.cancel(task.id).expect("cancel clone task");
+        let cancelled = registry.cancel(task.id.clone()).expect("cancel clone task");
         assert_eq!(cancelled.status, "cancelling");
+        assert!(
+            registry.dismiss(task.id).is_err(),
+            "in-flight clone task must refuse dismissal"
+        );
         let deadline = Instant::now() + Duration::from_secs(1);
         loop {
             if child.try_wait().expect("read clone process").is_some() {
@@ -3796,6 +3838,53 @@ mod tests {
             .expect("task snapshot")
             .iter()
             .any(|task| task["id"] == task_id));
+        let dismissed = git_gateway_clone_task_action_sync(
+            "clone_dismiss".to_string(),
+            String::new(),
+            json!({ "taskId": task_id }).to_string(),
+            &registry,
+        )
+        .expect("dismiss gateway clone task");
+        assert!(
+            dismissed.as_array().expect("dismissed snapshot").is_empty(),
+            "dismissed task must leave the registry"
+        );
+    }
+
+    #[test]
+    fn parses_git_clone_progress_lines() {
+        assert_eq!(
+            parse_git_clone_progress("Cloning into '.'..."),
+            Some(("preparing", 5))
+        );
+        assert_eq!(
+            parse_git_clone_progress("remote: Enumerating objects: 1553, done."),
+            Some(("preparing", 5))
+        );
+        assert_eq!(
+            parse_git_clone_progress("Receiving objects:   0% (1/1553)"),
+            Some(("receiving", 5))
+        );
+        assert_eq!(
+            parse_git_clone_progress("Receiving objects:  50% (777/1553), 10.20 MiB | 5.00 MiB/s"),
+            Some(("receiving", 45))
+        );
+        assert_eq!(
+            parse_git_clone_progress("Receiving objects: 100% (1553/1553), done."),
+            Some(("receiving", 85))
+        );
+        assert_eq!(
+            parse_git_clone_progress("Resolving deltas: 100% (900/900), done."),
+            Some(("resolving", 100))
+        );
+        assert_eq!(
+            parse_git_clone_progress("Checking out files:  40% (200/500)"),
+            Some(("finalizing", 97))
+        );
+        assert_eq!(
+            parse_git_clone_progress("fatal: repository not found"),
+            None
+        );
     }
 
     #[test]

crates/agent-gui/src-tauri/src/lib.rs

@@ -196,6 +196,7 @@ macro_rules! app_invoke_handler {
             commands::git::git_clone_repository_start,
             commands::git::git_clone_repository_tasks,
             commands::git::git_clone_repository_cancel,
+            commands::git::git_clone_repository_dismiss,
             commands::git::git_list_remote_branches,
             commands::git::git_switch_branch,
             commands::git::git_create_branch,

crates/agent-gui/src/pages/chat/workspace/cloneTasks.ts

@@ -88,6 +88,15 @@ export async function cancelWorkspaceCloneTask(taskId: string) {
 export function dismissWorkspaceCloneTask(taskId: string) {
   dismissedTaskIds.add(taskId);
   replaceTasks(tasks.filter((task) => task.id !== taskId));
+  // 服务端同步移除终态任务，否则重新挂载后快照会让卡片重现。
+  void invoke<WorkspaceCloneTask[]>("git_clone_repository_dismiss", { task_id: taskId })
+    .then((nextTasks) => {
+      dismissedTaskIds.delete(taskId);
+      replaceTasks(nextTasks);
+    })
+    .catch(() => {
+      // 本地 dismissedTaskIds 已隐藏该卡片；服务端移除失败留待下次快照。
+    });
 }
 
 function subscribe(listener: () => void) {