Node: [Feature request] Add node-report to cdp and add Visual Panel like React or Vue

Created on 24 May 2019  路  8Comments  路  Source: nodejs/node

I found node-report is very usefully (tbh I am a little surprised that Node.js has this level of clarity, really amazing job 馃憤 ), in some cases more usefully than profiler :)
I would like two thing:

  • Introduce it to CDP panel
    kind of like this (pic from https://github.com/vuejs/vue-devtools), but for node-report

screenshot

cc @nodejs/inspector @nodejs/report @richardlau
(Not sure I ping the group right, I don't know the Node.js group category )

feature request inspector report

Most helpful comment

Took over a week to get the update pushed thru the Chrome Web Store but it is done. https://blog.brakecode.com/nodejs-diagnostic-reports/

All 8 comments

cc @nodejs/diagnostics

  • adding report to the CDP panel of Vuejs: makes perfect sense, as one could generate an in-memory report upon user selection, and then render the relevant sections in the view.
  • adding report to the CDP protocol: Not very much sure about how does it work / help. Can you please elaborate a little on that?

Also in both cases, where does the feature be developed? in core, or in chrome? can the report.getReport() api can be consumed as is by the panel / protocol implementors?

node-report largely is used to refer the original npm module https://www.npmjs.com/package/node-report and diagnostic-report or simply report is used to refer to the one that is available in core. It may be great if you change the title to reflect the right report meant in this case.

adding report to the CDP panel of Vuejs

Nope, my thought is a new panel, like opened in chrome://inspect. May need to change chrome front-end code https://github.com/ChromeDevTools/devtools-frontend/blob/master/front_end/node_main/NodeConnectionsPanel.js.

image

image

node-report largely is used to refer the original npm module npmjs.com/package/node-report.

Hopefully work on Node.js with node-report installed too.

TBH I don't know how much value it is to show the JSON data that node-report or diagnostic-report provides in a DevTools panel, visually..

TBH I don't know how much value it is to show the JSON data that node-report or diagnostic-report provides in a DevTools panel, visually..

Kind of like DOM in front end (Elements Panel in Devtools).

@gengjiawen Will likely drop in the next version of NiM(see screenshot from the DevTools Panel)...

Will have the ability to save reports and quickly access reports from multiple node instances.

image

Stay tuned.

Took over a week to get the update pushed thru the Chrome Web Store but it is done. https://blog.brakecode.com/nodejs-diagnostic-reports/

Was this page helpful?
0 / 5 - 0 ratings