fix: keep stdin unpaused after ora completes

[clavin]

• I have read the contribution documentation for this project.
• I agree to follow the code of conduct that this project follows, as appropriate.
• The changes are appropriately documented (if applicable).
• The changes have sufficient test coverage (if applicable).
• The testsuite passes successfully on my local machine (if applicable).

Summarize your changes:

As a side-effect of using node:readline, when ora completes it may pause stdin. Once it is in this paused state, it is impossible to truly revert to the original "unpaused" state as calling resume on stdin has its own side-effect of keeping the node process alive as long as stdin remains unpaused (which is different than the default, where stdin is both unpaused but will not keep the process alive).

To get around this, I added a flag on asyncOra that will maintain stdin's unpaused state across ora's completion, but only for cases where that behavior is desirable, like the electron-forge start which both reads from stdin and is meant to stay alive "forever".

Fixes #2319.

[malept]

Have you tested this with both Webpack and non-Webpack templates?

[codecov]

Codecov Report

Merging #2904 (2265971) into master (345250c) will decrease coverage by 0.12%.
The diff coverage is 22.22%.

@@            Coverage Diff             @@
##           master    #2904      +/-   ##
==========================================
- Coverage   71.82%   71.69%   -0.13%     
==========================================
  Files          79       79              
  Lines        2385     2392       +7     
  Branches      449      450       +1     
==========================================
+ Hits         1713     1715       +2     
- Misses        450      454       +4     
- Partials      222      223       +1     

Impacted Files	Coverage Δ
packages/api/core/src/api/start.ts	64.04% <16.66%> (-1.84%)	⬇️
packages/utils/async-ora/src/ora-handler.ts	81.81% <33.33%> (-4.85%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 345250c...2265971. Read the comment docs.

[clavin]

I have certainly tested this with the webpack template, will test with a non-webpack one too rn just to be safe!

[clavin]

It is restarting the server in both cases 👍 I tested on the webpack template, once with and once without the plugin enabled in the config.

[malept]

Minor suggestion but otherwise