jozefizso / apimonitor

Other
17 stars 6 forks source link

Can this be open sourced? #1

Open amithegde opened 1 month ago

amithegde commented 1 month ago

Hi @rohitab, can this tool be made open source? Or a commercial tool?

rohitab commented 1 month ago

@amithegde, it's been a while since I've had time to work on API Monitor. But, I do have plans to resume development and add additional features. As for making it open source, I haven't made a decision yet.

amithegde commented 1 month ago

This tool saves so much time while debugging large applications and to get a birds eye view. I have few feature suggestions if you consider resuming development!

rohitab commented 1 month ago

@amithegde of course. I'm very open to suggestions and would be happy to add new features in once I start working on it again.

Thanks for the feedback too. Glad to know you're finding it useful.

amithegde commented 1 month ago

Here are few that help:

rohitab commented 1 month ago

Great ideas. I have most of these on my todo list, so they will be implemented for sure. I had been holding of on developing some features, like exporting to a variety of file formats, since I wanted to implement them via plugins, either mine or 3rd party. I also plan to add scripting, so API Monitor can be fully automated.

amithegde commented 1 month ago

thank you @rohitab plugins and ability to script are great.

Additionally, I think currenlty there is no way to provide symbols band because of which the stack doesn't resolve properly. Even when I have private symbols, it is kind of hard to view full call stack.

A good example to look at would be perfview which allows folding callstack and expanding call stack and also allows loading symbols: image

May be if calls collected can be formatted to make it load on JS version of Perfview which would save the effort required to write the UI - https://github.com/microsoft/perfview/issues/903

If we have these capabilities, the tool can be extremely useful to debug various scenarios which are tedios to debug through.

rohitab commented 1 month ago

@amithegde, loading symbols is definitely required and is already on the roadmap. I'm not familiar with PerfViewJS, but I've bookmarked this page, so when I start implementing these features, I'll get back to you and we can discuss it at that time.