[CLI] Close handles with symbol instead of newline.

Otherwise they stop on whatever part of the loading
animation they happen to be.

cli/src/commands/invite/claim.rs

@@ -84,7 +84,7 @@ async fn step0(
 
     let ctx = claimer_retrieve_info(Arc::new(config.into()), addr, None).await?;
 
-    handle.stop_with_newline();
+    handle.stop_with_symbol(GREEN_CHECKMARK);
 
     Ok(ctx)
 }
@@ -100,7 +100,7 @@ async fn step1_user(ctx: UserClaimInitialCtx) -> anyhow::Result<UserClaimInProgr
 
     let ctx = ctx.do_wait_peer().await?;
 
-    handle.stop_with_newline();
+    handle.stop_with_symbol(GREEN_CHECKMARK);
 
     Ok(ctx)
 }
@@ -116,7 +116,7 @@ async fn step1_device(ctx: DeviceClaimInitialCtx) -> anyhow::Result<DeviceClaimI
 
     let ctx = ctx.do_wait_peer().await?;
 
-    handle.stop_with_newline();
+    handle.stop_with_symbol(GREEN_CHECKMARK);
 
     Ok(ctx)
 }
@@ -164,7 +164,7 @@ async fn step3_user(ctx: UserClaimInProgress2Ctx) -> anyhow::Result<UserClaimInP
 
     let ctx = ctx.do_wait_peer_trust().await?;
 
-    handle.stop_with_newline();
+    handle.stop_with_symbol(GREEN_CHECKMARK);
 
     Ok(ctx)
 }
@@ -180,7 +180,7 @@ async fn step3_device(ctx: DeviceClaimInProgress2Ctx) -> anyhow::Result<DeviceCl
 
     let ctx = ctx.do_wait_peer_trust().await?;
 
-    handle.stop_with_newline();
+    handle.stop_with_symbol(GREEN_CHECKMARK);
 
     Ok(ctx)
 }
@@ -195,7 +195,7 @@ async fn step4_user(ctx: UserClaimInProgress3Ctx) -> anyhow::Result<UserClaimFin
 
     let ctx = ctx.do_claim_user(device_label, human_handle).await?;
 
-    handle.stop_with_newline();
+    handle.stop_with_symbol(GREEN_CHECKMARK);
 
     Ok(ctx)
 }
@@ -209,7 +209,7 @@ async fn step4_device(ctx: DeviceClaimInProgress3Ctx) -> anyhow::Result<DeviceCl
 
     let ctx = ctx.do_claim_device(device_label).await?;
 
-    handle.stop_with_newline();
+    handle.stop_with_symbol(GREEN_CHECKMARK);
 
     Ok(ctx)
 }
@@ -237,30 +237,29 @@ async fn save_user(ctx: UserClaimFinalizeCtx, save_mode: SaveMode) -> anyhow::Re
     let key_file = ctx.get_default_key_file();
     let access = get_access_strategy(key_file, save_mode)?;
 
-    println!(
+    let mut handle = start_spinner(format!(
         "Saving device at: {YELLOW}{}{RESET}",
         access.key_file().display()
-    );
+    ));
 
     ctx.save_local_device(&access).await?;
 
-    println!("Saved");
+    handle.stop_with_symbol(GREEN_CHECKMARK);
 
     Ok(())
 }
 
 async fn save_device(ctx: DeviceClaimFinalizeCtx, save_mode: SaveMode) -> anyhow::Result<()> {
     let key_file = ctx.get_default_key_file();
     let access = get_access_strategy(key_file, save_mode)?;
-
-    println!(
+    let mut handle = start_spinner(format!(
         "Saving device at: {YELLOW}{}{RESET}",
         access.key_file().display()
-    );
+    ));
 
     ctx.save_local_device(&access).await?;
 
-    println!("Saved");
+    handle.stop_with_symbol(GREEN_CHECKMARK);
 
     Ok(())
 }

cli/src/commands/invite/greet.rs

@@ -75,7 +75,7 @@ async fn step0(
         None => return Err(anyhow::anyhow!("Invitation not found")),
     };
 
-    handle.stop_with_newline();
+    handle.stop_with_symbol(GREEN_CHECKMARK);
 
     Ok(invitation)
 }
@@ -86,7 +86,7 @@ async fn step1_user(ctx: UserGreetInitialCtx) -> anyhow::Result<UserGreetInProgr
 
     let ctx = ctx.do_wait_peer().await?;
 
-    handle.stop_with_newline();
+    handle.stop_with_symbol(GREEN_CHECKMARK);
 
     Ok(ctx)
 }
@@ -97,7 +97,7 @@ async fn step1_device(ctx: DeviceGreetInitialCtx) -> anyhow::Result<DeviceGreetI
 
     let ctx = ctx.do_wait_peer().await?;
 
-    handle.stop_with_newline();
+    handle.stop_with_symbol(GREEN_CHECKMARK);
 
     Ok(ctx)
 }
@@ -113,7 +113,7 @@ async fn step2_user(ctx: UserGreetInProgress1Ctx) -> anyhow::Result<UserGreetInP
 
     let ctx = ctx.do_wait_peer_trust().await?;
 
-    handle.stop_with_newline();
+    handle.stop_with_symbol(GREEN_CHECKMARK);
 
     Ok(ctx)
 }
@@ -129,7 +129,7 @@ async fn step2_device(ctx: DeviceGreetInProgress1Ctx) -> anyhow::Result<DeviceGr
 
     let ctx = ctx.do_wait_peer_trust().await?;
 
-    handle.stop_with_newline();
+    handle.stop_with_symbol(GREEN_CHECKMARK);
 
     Ok(ctx)
 }
@@ -189,7 +189,7 @@ async fn step5_user(ctx: UserGreetInProgress4Ctx) -> anyhow::Result<()> {
     ctx.do_create_new_user(device_label, human_handle, profile)
         .await?;
 
-    handle.stop_with_newline();
+    handle.stop_with_symbol(GREEN_CHECKMARK);
 
     Ok(())
 }
@@ -203,7 +203,7 @@ async fn step5_device(ctx: DeviceGreetInProgress4Ctx) -> anyhow::Result<()> {
 
     ctx.do_create_new_device(device_label).await?;
 
-    handle.stop_with_newline();
+    handle.stop_with_symbol(GREEN_CHECKMARK);
 
     Ok(())
 }

cli/src/commands/shared_recovery/create.rs

@@ -84,7 +84,7 @@ pub async fn create_shared_recovery(args: Args, client: &StartedClient) -> anyho
             .collect()
     };
     // we must stop the handle here to avoid messing up with the threshold choice
-    handle.stop_with_newline();
+    handle.stop_with_symbol("..."); // not green check mark because it's not finished
     let threshold = if let Some(t) = threshold {
         t
     } else {

cli/src/utils.rs

@@ -322,7 +322,7 @@ pub async fn start_client_with_config(
                 start_spinner("Device already used by another process, waiting...".into());
             let device_in_use_guard =
                 lock_device_for_use(&config.config_dir, device.device_id).await?;
-            handle.stop_with_newline();
+            handle.stop_with_message(format!("{GREEN_CHECKMARK} Device accessed"));
             device_in_use_guard
         }
         Err(TryLockDeviceForUseError::Internal(err)) => return Err(err),

newsfragments/9109.bugfix.rst

@@ -0,0 +1 @@
+[CLI] Stop spinners with check mark.