Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tool integration support (for napalm maybe 🔥) #47

Open
JoranHonig opened this issue Feb 16, 2024 · 2 comments
Open

Tool integration support (for napalm maybe 🔥) #47

JoranHonig opened this issue Feb 16, 2024 · 2 comments

Comments

@JoranHonig
Copy link

Hello I just published https://github.com/ConsenSysDiligence/napalm the other week.

I think it'd be cool to integrate 4naly3er in napalm, however I believe some small things are missing that are needed to make this possible.

  1. computer friendly output: parsing a markdown file is possible ofc, but very painful. Having JSON would make things much easier! I recommend checking out SARIF, which is increasingly gaining adoption for static analysis issue reports.
  2. plugability: if I understand it correctly people currently clone this project and drop in their own detectors, it'd be very nice if there was some way of packaging detectors separate from the main code base.

I understand that both of these features might not be high priority, buut if you do decide to add them I'd be a happy dev and add 4naly3er support to napalm immediately!

@izcoser
Copy link
Contributor

izcoser commented Mar 16, 2024

This is pretty cool. Napalm is a cool idea. I'm building myself a web app which integrates several solidity analyzers, does forge tests and AI analysis, maybe I should use Napalm.

@JoranHonig
Copy link
Author

You should check it out! Though, tbf Napalms' main goal is to help people work with custom detectors!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants