nodejs / node

Node.js JavaScript runtime ✨🐢🚀✨
https://nodejs.org
Other
108.11k stars 29.86k forks source link

Switching to REPL in inspect is cumbersome #30078

Closed justrajeshkp closed 7 months ago

justrajeshkp commented 5 years ago

Hi, When all the debugging functionalities are given with keyboard shortcut (like for reset, next etc), Why is the REPL mode given with typing of a word to get into the debug mode. Any ctrl-key toggle combination to get into and out of debug-mode will be great help as typing a word is not only cumbersome for developers alike and also a 'fox in the herd' phenomenon

Looking forward for a change in this procedure Thanks, with regards, Rajesh KP

bnoordhuis commented 5 years ago

Your bug report (feature request?) is unclear to me. Can you post steps describing what you do from the moment on you start node and then explain what you would like to see changed?

justrajeshkp commented 5 years ago

Ok, I will do as suggested

Requirement: I want to debug a node application. I have different variables which I need to debug

Steps to debug: I add let x = 2, y = x + 10; debugger; let x = 10;

node inspect index.js

Now application stops at debugger point. I can step through the code line by line with keyshortcut 'n'. But if i want to inspect any variable I should use the REPL mode ( i.e. to type 'repl' ). My point of suggestion is that rather than typing the word 'repl', it will be great if node team provide a short-cut-key (possibly with ctrl-shift combination key) to toggle to and off from the repl-mode

Hope I am clear now

Thanks n regards, Rajesh Kp

On Wed, Oct 23, 2019 at 1:28 PM Ben Noordhuis notifications@github.com wrote:

Your bug report (feature request?) is unclear to me. Can you post steps describing what you do from the moment on you start node and then explain what you would like to see changed?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/nodejs/node/issues/30078?email_source=notifications&email_token=ANPVXMMSFOYAN3URWW3NDIDQP77ZZA5CNFSM4JD4QF52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECAOOAI#issuecomment-545318657, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANPVXMOPORLWIHXLNZ4CMWLQP77ZZANCNFSM4JD4QF5Q .

tniessen commented 5 years ago

rather than typing the word 'repl', it will be great if node team provide a short-cut-key (possibly with ctrl-shift combination key)

So if I understand you correctly, you would prefer a three-key combination (ctrl-shift + something) over a four letter word which does not require any special keys? That does not seem like a huge improvement to me.

to toggle to and off from the repl-mode

I think you can already use CTRL+C to exit the repl mode.

justrajeshkp commented 5 years ago

Hi Ben, You are right that ctrl+shift+ key and 'repl' has a difference of 1 key only (quantity wise).. But to type repl I need to look the keyboard to type (which is not user friendly). With ctrl+shift+key press finger mindmaps the key position. Only on UX basis, it applies. Else its fine

Thanks, Rajesh Kp

On Thu, Oct 24, 2019 at 1:29 PM rajesh kp just.rajeshkp@gmail.com wrote:

Ok, I will do as suggested

Requirement: I want to debug a node application. I have different variables which I need to debug

Steps to debug: I add let x = 2, y = x + 10; debugger; let x = 10;

node inspect index.js

Now application stops at debugger point. I can step through the code line by line with keyshortcut 'n'. But if i want to inspect any variable I should use the REPL mode ( i.e. to type 'repl' ). My point of suggestion is that rather than typing the word 'repl', it will be great if node team provide a short-cut-key (possibly with ctrl-shift combination key) to toggle to and off from the repl-mode

Hope I am clear now

Thanks n regards, Rajesh Kp

On Wed, Oct 23, 2019 at 1:28 PM Ben Noordhuis notifications@github.com wrote:

Your bug report (feature request?) is unclear to me. Can you post steps describing what you do from the moment on you start node and then explain what you would like to see changed?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/nodejs/node/issues/30078?email_source=notifications&email_token=ANPVXMMSFOYAN3URWW3NDIDQP77ZZA5CNFSM4JD4QF52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECAOOAI#issuecomment-545318657, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANPVXMOPORLWIHXLNZ4CMWLQP77ZZANCNFSM4JD4QF5Q .

RedYetiDev commented 7 months ago

(Closing, as it seems that this is not going to be changed)