cli: handle spinning states

[anonrig]

Fixes the annoying bug, when a question is asked through the client and a spinner is already taken place. This pull request, pauses and restarts the spinner after the answer.

Example:

[codecov]

Codecov Report

Base: 83.60% // Head: 83.41% // Decreases project coverage by -0.18% ⚠️

Coverage data is based on head (a6f996d) compared to base (3b5c232).
Patch coverage: 75.00% of modified lines in pull request are covered.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #657      +/-   ##
==========================================
- Coverage   83.60%   83.41%   -0.19%     
==========================================
  Files          37       37              
  Lines        4117     4131      +14     
==========================================
+ Hits         3442     3446       +4     
- Misses        675      685      +10     

Impacted Files	Coverage Δ
lib/cli.js	86.45% <75.00%> (-1.19%)	⬇️
lib/verbosity.js	73.07% <0.00%> (-7.70%)	⬇️
lib/ci/run_ci.js	75.00% <0.00%> (-3.34%)	⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[aduh95]

Can we add tests?

[anonrig]

@aduh95 @Trott Can you review the last 2 commits?

• I added the first commit, even though it was handled by the try catch, to remove the unnecessary debug error on the input.
• Secondly, mocha tests were not terminating, and mocha docs mentioned --exit.

[ShogunPanda]

Hello folks! This is very cool. When can we expect this to be released in NPM?

[targos]

This wasn't merged with a conventional commit message, so the release-please action didn't create a PR.
We would need to merge another fix/feat, or create an empty commit with a good message.

[ShogunPanda]

Do you mean a commit directly on the main branch?

[aduh95]

Sorry, I was fooled by the green CI. I've pushed 51a3b24 to fix it.

[anonrig]

For future references, what was missing from the pull request?

[Trott]

For future references, what was missing from the pull request?

I believe this repo is using release-please so the commit message needs to confirm to the expectations of release-please. Yup, the commit message requirements here are different than in the core repo. ¯\(ツ)/¯