v8_inspector: url field is missed

[develar]

• Version:
  master
• Platform:
  OS X
• Subsystem:
  inspector

node --inspect=9229 --debug-brk foo.js

Actual:

{
  "description": "node.js instance",
  "devtoolsFrontendUrl": "https://chrome-devtools-frontend.appspot.com/serve_file/@521e5b7e2b7cc66b4006a8a54cb9c4e57494a5ef/inspector.html?experiments=true&v8only=true&ws=localhost:9229/node",
  "faviconUrl": "https://nodejs.org/static/favicon.ico",
  "id": "44439",
  "title": "/Volumes/hdd/node/out/Release/node",
  "type": "node",
  "webSocketDebuggerUrl": "ws://localhost:9229/node"
}

Expected: url is missed — should be file path.

NPE in the JetBrains Debugger (Chromium Remote run configuration in the JetBrains IDEs can be used to debug node).

[bnoordhuis]

Isn't that a JetBrains bug? node --inspect=9229 --debug-brk -e 'require("./")' won't have a file path. It should be prepared to deal with that.

[develar]

Yes, it is a JetBrains bug and it will be fixed. But to be consistent, url should be set to some path. And for -e case maybe we can set it to <eval>.

For example, Google Chrome set url to "chrome://newtab/" for new tab (e.g. if you run javascript: some code).

[eugeneo]

I made several fixes and can see basic debug working in the WebStorm - https://github.com/eugeneo/node/tree/webstorm

I will need to test it on Windows though. There are also some issues that we do not want to "fix" - I will try to contact JetBrains to describe incompatibilities.

[eugeneo]

I opened WebStorm bug here - https://youtrack.jetbrains.com/issue/WEB-21991

I described 2 incompatibilities that are WebStorm issues.

[eugeneo]

#8243 added the URL field. This issue can be closed now.