Support V8 Inspector Integration for Node.js #893

New Issue

fergalmoran · 2025-08-09T17:18:05Z

fergalmoran commented

2025-08-09 17:18:05 +00:00

Originally created by @novascreen on 3/25/2017

Changes deprecated --debug to --inspect when using LaunchWithDebugging.

I have not implemented inspect-brk. Node 6 uses debug-brk so maybe this is enough for this feature and details like that might be better addressed by a separate feature which allows users to pass any parameter, as discussed in #796.

Node logs 4 separate messages for the debugger initially, so I wasn't sure what you would do to handle those. I've decided to catch any logs related to the debugger and log them as warnings.

One of the messages is a warning, but terminated the dotnet process, so I created a separate check for that (StartsWith("Warning") and log that as a warning as well.

Let me know if there is anything you'd like me to improve.

*Originally created by @novascreen on 3/25/2017* Changes deprecated `--debug` to `--inspect` when using `LaunchWithDebugging`. I have not implemented `inspect-brk`. Node 6 uses `debug-brk` so maybe this is enough for this feature and details like that might be better addressed by a separate feature which allows users to pass any parameter, as discussed in #796. Node logs 4 separate messages for the debugger initially, so I wasn't sure what you would do to handle those. I've decided to catch any logs related to the debugger and log them as warnings. One of the messages is a warning, but terminated the dotnet process, so I created a separate check for that (`StartsWith("Warning"`) and log that as a warning as well. Let me know if there is anything you'd like me to improve.

fergalmoran closed this issue

2025-08-09 17:18:05 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github/JavaScriptServices#893