repl: simplify internal repl

[BridgeAR]

repl: always check for NODE_REPL_MODE environment variable

This makes sure all REPL instances check for the NODE_REPL_MODE
environment variable in case the replMode is not passed through
as option.

At the same time this simplifies the internal REPL code significantly
and color detection is improved.

##### doc: document the --use-strict flag for the REPL

Add information that this flag is going to evaluate all code in strict
mode. Also use the regular default description as it's used in other
options.

@nodejs/repl @nodejs/documentation PTAL

Checklist

• make -j4 test (UNIX), or vcbuild test (Windows) passes
• tests and/or benchmarks are included
• documentation is changed or added
• commit message follows commit guidelines

[nodejs-github-bot]

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

[nodejs-github-bot]

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

[nodejs-github-bot]

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

[BridgeAR]

@nodejs/repl this could use some reviews. PTAL.

[nodejs-github-bot]

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

[BridgeAR]

This could use some reviews. I am not sure whom to ping though.

[benjamingr]

This generally LGTM. Maybe @addaleax since she had an opinion on the older repl PR?

I'll try checking this out and reviewing it today/tomorrow.

[addaleax]

Yeah idk, I’ve seen this PR but I don’t really know how to feel about it? The commit message says what this PR does, but not why, and that might be helpful here.

To be honest, I think that’s mostly because I don’t know if this would be more helpful to – the probably very low number of – external REPL module users or not. For other modules I would consider relying on environment variables for config an antipattern, but given that the REPL is probably more often used as the main application than as part of a larger application, it might make sense here?

[BridgeAR]

@addaleax my main goal here is to unify the behavior of our "internal" (standalone) REPL instances and instances created otherwise. The improved color detection could theoretically be handled separately.

[nodejs-github-bot]

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

[nodejs-github-bot]

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

[nodejs-github-bot]

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

[benjamingr]

Actual code LGTM, not sure I understand the use case well enough but I trust you on this

[nodejs-github-bot]

Running CI again since the last one was 27 days ago

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

[nodejs-github-bot]

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

[jasnell]

Landed in b831b08

[addaleax]

Revert PR because this breaks the github actions build: #34058