fix: clear progress reporters on successful completion

Progress reporters were not being cleared after successful operations,
leaving progress bars visible in the terminal output. Now they are
properly cleared when all operations complete without errors.

2 files changed, 8 insertions, 2 deletions

diff --git a/src/bin/git_remote_nostr/main.rs b/src/bin/git_remote_nostr/main.rs
index 618aff7..b26981d 100644
--- a/src/bin/git_remote_nostr/main.rs
+++ b/src/bin/git_remote_nostr/main.rs
@@ -157,7 +157,7 @@ async fn fetching_with_report_for_helper(
 ) -> Result<()> {
     let term = console::Term::stderr();
     term.write_line("nostr: fetching...")?;
-    let (relay_reports, _progress_reporter) = client
+    let (relay_reports, progress_reporter) = client
         .fetch_all(
             Some(git_repo_path),
             Some(trusted_maintainer_coordinate),
@@ -170,5 +170,6 @@ async fn fetching_with_report_for_helper(
     } else {
         term.write_line(&format!("nostr updates: {report}"))?;
     }
+    progress_reporter.clear()?;
     Ok(())
 }
diff --git a/src/lib/list.rs b/src/lib/list.rs
index cb26c19..ce8737c 100644
--- a/src/lib/list.rs
+++ b/src/lib/list.rs
@@ -333,18 +333,23 @@ pub async fn list_from_remotes(
         .await;
 
     let mut remote_states = HashMap::new();
+    let mut all_succeeded = true;
     for result in results {
         match result {
             Ok((url, state, is_grasp_server)) => {
                 remote_states.insert(url, (state, is_grasp_server));
             }
             Err((url, error)) => {
-                // Errors are already displayed in progress bars
+                all_succeeded = false;
                 let _ = term.write_line(&format!("failed to list from {}: {}", url, error));
             }
         }
     }
 
+    if all_succeeded {
+        let _ = progress_reporter.clear();
+    }
+
     remote_states
 }