feat: loading spinner

cli/src/main.rs

@@ -2,14 +2,14 @@ extern crate log;
 
 mod cli;
 mod error;
+mod help_strings;
 mod modules;
 mod register;
 mod utils;
-mod help_strings;
 
+use colored::*;
 use log::error;
 use register::register;
-use colored::*;
 
 #[tokio::main]
 async fn main() {

cli/src/modules/connections.rs

@@ -37,7 +37,6 @@ pub enum ConnectionSubcommands {
     },
 }
 
-// TODO: we should implement `from` so we can use todo and have a cleaner api
 pub async fn parse_connection_args(
     options: &ConnectionOptions,
     agent: impl ConnectionModule,
@@ -47,7 +46,7 @@ pub async fn parse_connection_args(
         return agent
             .get_connection_by_id(id.to_string())
             .await
-            .map(|connection| pretty_print_obj(connection));
+            .map(pretty_print_obj);
     }
     if options.all {
         return agent

cli/src/modules/credential_definition.rs

@@ -1,11 +1,12 @@
 use agent_controller::modules::credential_definition::CredentialDefinitionModule;
 use clap::{Args, Subcommand};
-use serde_json::json;
 use log::{debug, info};
+use serde_json::json;
 
 use crate::{
     error::{Error, Result},
-    utils::logger::{pretty_stringify_obj},
+    utils::loader::{start_loader, Loader},
+    utils::logger::pretty_stringify_obj,
 };
 
 #[derive(Args)]
@@ -32,6 +33,7 @@ pub async fn parse_credential_definition_args(
     options: &CredentialDefinitionOptions,
     agent: impl CredentialDefinitionModule,
 ) -> Result<()> {
+    start_loader(Loader::Spinner);
     if let Some(id) = &options.id {
         return agent.get_by_id(id.to_string()).await.map(|cred_def| {
             let loggable = json!({

cli/src/modules/credentials.rs

@@ -3,7 +3,10 @@ use clap::{Args, Subcommand};
 use log::{debug, info};
 
 use crate::error::{Error, Result};
-use crate::utils::logger::pretty_stringify_obj;
+use crate::{
+    utils::loader::{start_loader, Loader},
+    utils::logger::pretty_stringify_obj,
+};
 use colored::*;
 
 #[derive(Args)]
@@ -34,6 +37,7 @@ pub async fn parse_credentials_args(
     commands: &CredentialSubcommands,
     agent: impl CredentialsModule,
 ) -> Result<()> {
+    start_loader(Loader::Spinner);
     match commands {
         CredentialSubcommands::Offer {
             connection_id,

cli/src/modules/features.rs

@@ -3,12 +3,14 @@ use clap::Args;
 use log::{debug, info};
 
 use crate::error::Result;
+use crate::utils::loader::{start_loader, Loader};
 use crate::utils::logger::pretty_stringify_obj;
 
 #[derive(Args)]
 pub struct FeaturesOptions {}
 
 pub async fn parse_features_args(agent: impl FeaturesModule) -> Result<()> {
+    start_loader(Loader::Spinner);
     agent.discover_features().await.map(|features| {
         debug!("{}", pretty_stringify_obj(&features));
         features

cli/src/modules/message.rs

@@ -4,6 +4,7 @@ use colored::*;
 use log::info;
 
 use crate::error::Result;
+use crate::utils::loader::{start_loader, Loader};
 
 #[derive(Args)]
 pub struct MessageOptions {
@@ -14,6 +15,7 @@ pub struct MessageOptions {
 }
 
 pub async fn parse_message_args(options: &MessageOptions, agent: impl MessageModule) -> Result<()> {
+    start_loader(Loader::Spinner);
     let send_options = SendMessageOptions {
         id: options.id.to_owned(),
         message: options.message.to_owned(),

cli/src/modules/schema.rs

@@ -4,6 +4,7 @@ use log::info;
 
 use crate::{
     error::{Error, Result},
+    utils::loader::{start_loader, Loader},
     utils::logger::pretty_print_obj,
 };
 
@@ -32,6 +33,7 @@ pub enum SchemaSubcommands {
 }
 
 pub async fn parse_schema_args(options: &SchemaOptions, agent: impl SchemaModule) -> Result<()> {
+    start_loader(Loader::Spinner);
     if let Some(id) = &options.id {
         return agent
             .get_by_id(id.to_string())

cli/src/utils/loader.rs

@@ -0,0 +1,41 @@
+use core::time;
+use std::io::{self, Write};
+
+/// All the types of loaders
+pub enum Loader {
+    Spinner,
+}
+
+/// prints a char to stdout with a delay and whether it should be inplace or
+/// not
+pub fn print_char(c: impl Into<String>, timeout: u64, inplace: bool) {
+    if inplace {
+        print!("{}\r", c.into());
+    } else {
+        print!("{}", c.into());
+    }
+    io::stdout().flush().unwrap();
+    std::thread::sleep(time::Duration::from_millis(timeout));
+}
+
+/// Start the loader
+pub fn start_loader(loader: Loader) {
+    match loader {
+        Loader::Spinner => spinner_loader(),
+    }
+}
+
+/// Spinning loader. Does inplace replacement.
+fn spinner_loader() {
+    let time_between = 50;
+
+    std::thread::spawn(move || loop {
+        print_char('|', time_between, true);
+        print_char('/', time_between * 2, true);
+        print_char('-', time_between * 3, true);
+        print_char('\\', time_between * 4, true);
+        print_char('/', time_between * 5, true);
+        print_char('-', time_between * 6, true);
+        print_char('\\', time_between * 7, true);
+    });
+}

cli/src/utils/mod.rs

@@ -2,3 +2,4 @@
 pub mod logger;
 pub mod config;
 pub mod qr;
+pub mod loader;