lib: avoid repl.setupHistory() blocks node exit after repl closed

[lucshi]

Fixes: #52386

[cola119]

@lucshi Can you add tests for the change?

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/58546/

[lucshi]

@lucshi Can you add tests for the change?

Sure I will add a test. Thanks!

[lucshi]

@cola119 Could you please take a look? Thank!

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/58628/

[cola119]

Can you fix the linter errors?

[lucshi]

Can you fix the linter errors?

Sure, just noticed the errors. Done.

[github-actions]

Failed to start CI

   ⚠  No approving reviews found
   ✘  Refusing to run CI on potentially unsafe PR

https://github.com/nodejs/node/actions/runs/8824871644

[lucshi]

@node/lib

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/58835/

[lucshi]

Hi @cola119, Could you please take a look at this? Thanks!

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/63400/

[BridgeAR]

AFAIK an alternative should be to only write sync. That would also make sure things are written properly and it is closed.

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/66983/

[richardlau]

CI: https://ci.nodejs.org/job/node-test-pull-request/66983/

This PR will need to be rebased. The CI detects this PR as being Node.js 22 because it was opened last year and is picking the compiler for that version of Node.js instead of the compiler that should be used for main.