[cli] Silence status messages and progress in non-live TTYs

Author: riken127

src/client/cli/cmd/animated_spinner.cpp

@@ -42,18 +42,18 @@ mp::AnimatedSpinner::~AnimatedSpinner()
 
 void mp::AnimatedSpinner::start(const std::string& start_message)
 {
+    if (!is_live)
+        return;
+
     std::unique_lock<decltype(mutex)> lock{mutex};
     if (!running)
     {
         current_message = start_message;
         running = true;
 
-        if (is_live)
-        {
-            clear_line(cout);
-            cout << start_message << "  " << std::flush;
-            t = std::thread(&AnimatedSpinner::draw, this);
-        }
+        clear_line(cout);
+        cout << start_message << "  " << std::flush;
+        t = std::thread(&AnimatedSpinner::draw, this);
     }
 }
 
@@ -65,6 +65,9 @@ void mp::AnimatedSpinner::start()
 
 void mp::AnimatedSpinner::stop()
 {
+    if(!is_live)
+        return;
+
     std::unique_lock<decltype(mutex)> lock{mutex};
     if (running)
     {
@@ -75,12 +78,15 @@ void mp::AnimatedSpinner::stop()
             t.join();
     }
 
-    if (is_live)
-        clear_line(cout);
+    
+    clear_line(cout);
 }
 
 void mp::AnimatedSpinner::print(std::ostream& stream, const std::string& message)
 {
+    if (!is_live)
+        return;
+    
     stop();
 
     stream << message;