Add cls command to repl

[MrJithil]

add .cls command which will be very convinient for clearing the active screen

[cola119]

I understand the proposal for a new command, but given the existing ctrl+l to clear the screen, I'm curious about the unique use cases for it. Could you share the specific motivations and scenarios for this addition?

[MrJithil]

I understand the proposal for a new command, but given the existing ctrl+l to clear the screen, I'm curious about the unique use cases for it. Could you share the specific motivations and scenarios for this addition?

In most CLI and REPL having a command to clear the screen. People who is more using commands instead of shortcuts will a little hard to clear the screen while using Node.js REPL.
So, I though to provide .cls command to solve the issue.

[nodejs-github-bot]

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

[nodejs-github-bot]

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

[BridgeAR]

I looked at this twice at least and I have no strong opinion about adding the additional command.
Maybe it's useful, while it's also not required. I did not see requests for it before, so it's probably not often needed while people might also think about it more with the additional command existing.

Should we just accept this? I guess it won't hurt? @nodejs/repl

[BridgeAR]

Ah, I guess it's fine to add it.

[BridgeAR]

@MrJithil could you please rebase?

[MrJithil]

Sure.

[MrJithil]

@BridgeAR Rebased. Thanks.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.45%. Comparing base (4dc19ec) to head (43e1003).
⚠️ Report is 13 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #52206      +/-   ##
==========================================
- Coverage   88.49%   88.45%   -0.04%     
==========================================
  Files         703      703              
  Lines      207529   207545      +16     
  Branches    40011    40008       -3     
==========================================
- Hits       183643   183587      -56     
- Misses      15885    15944      +59     
- Partials     8001     8014      +13     

Files with missing lines	Coverage Δ
lib/repl.js	94.31% <100.00%> (+0.04%)	⬆️

... and 38 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[BridgeAR]

I think we could use assert.match here and have a concrete expectation for the output.

[nodejs-github-bot]

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

[dario-piotrowicz]

Just giving my two cents here 🙂

The command makes sense to me but I'm not a huge fan of the name, all the other commands are not in short form:

so .cls here would a bit stick out as a sore thumb in my opinion 😅

I was also not familiar of cls is that a microsoft command/convention?

I also feel like it's very unfortunate that repl has a .clear command which has a completely different meaning...
(and having both .clear and .cls feels a bit messy to me)

mh... would it be possible to, instead of introducing a .cls command to:

• rename .clear as .reset (as a semver major)
• later re-introduce .clear to clear the screen?

(or alternatively proceed with .cls for now with the plan of renaming it later on in the way I just mentioned?)
(PS: also if we do want to introduce it now what about maybe introducing it as .clearscreen or something like that?)

[jasnell]

I'm generally -1 on this. console.clear() works perfectly well for this.