resets ledger ui status between errors (#5490)

* resets ledger ui status between errors

removes the error status (e.g., 'Open Ledger App Ironfish') after the user has
resolved the cause of the error

* uses timer to clear status ledger ux action status

sets a timer to clear the status instead of clearing on each iteration. prevents
error status message from flashing if the user has not fixed the cause between
iterations

Author: hughy

ironfish-cli/src/ui/ledger.ts

@@ -49,6 +49,8 @@ export async function ledger<TResult>({
     ux.action.start(message)
   }
 
+  let clearStatusTimer
+
   try {
     // eslint-disable-next-line no-constant-condition
     while (true) {
@@ -57,6 +59,7 @@ export async function ledger<TResult>({
         ux.action.stop()
         return result
       } catch (e) {
+        clearTimeout(clearStatusTimer)
         if (e instanceof LedgerAppLocked) {
           // If an app is running and it's locked, trying to poll the device
           // will cause the Ledger device to hide the pin screen as the user
@@ -101,13 +104,15 @@ export async function ledger<TResult>({
         }
 
         statusAdded = true
+        clearStatusTimer = setTimeout(() => (ux.action.status = undefined), 2000)
         await PromiseUtils.sleep(1000)
         continue
       }
     }
   } finally {
     // Don't interrupt an existing status outside of ledgerAction()
     if (!wasRunning && statusAdded) {
+      clearTimeout(clearStatusTimer)
       ux.action.stop()
     }
   }