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

feat(anvil): add support for JS tracer in Anvil #6764

Open
Tracked by #8269
mattsse opened this issue Jan 11, 2024 · 6 comments
Open
Tracked by #8269

feat(anvil): add support for JS tracer in Anvil #6764

mattsse opened this issue Jan 11, 2024 · 6 comments
Assignees
Labels
C-anvil Command: anvil good first issue Good for newcomers T-feature Type: feature
Milestone

Comments

@mattsse
Copy link
Member

mattsse commented Jan 11, 2024

Component

Anvil

Describe the feature you would like

with new inspector crate we can now add JS tracing

reference impl

https://github.com/paradigmxyz/reth/blob/6430731fff06bfde15318667240d5738505fd6f8/crates/rpc/rpc/src/debug.rs#L551-L551

Note: because JS needs to access the DB we need to spawn a task that handles various db requests:

https://github.com/paradigmxyz/reth/blob/6430731fff06bfde15318667240d5738505fd6f8/crates/rpc/rpc/src/debug.rs#L645-L645

Additional context

No response

@mattsse mattsse added good first issue Good for newcomers T-feature Type: feature C-anvil Command: anvil labels Jan 11, 2024
@krisdimitrov
Copy link

Would love to have this feature as well! Hope it gets to Anvil.

@1010adigupta
Copy link
Contributor

I would like to work on this

@mattsse
Copy link
Member Author

mattsse commented Jan 16, 2024

@1010adigupta great! lmk when you need pointers

@zerosnacks
Copy link
Member

Hi @1010adigupta, do you have any updates on this ticket or need help? We are happy to give pointers

@zerosnacks zerosnacks self-assigned this Jul 9, 2024
@zerosnacks zerosnacks changed the title Add support for js tracer in anvil feat(anvil): add support for JS tracer in Anvil Jul 9, 2024
@zerosnacks zerosnacks added this to the v1.0.0 milestone Jul 26, 2024
@EdwardJES
Copy link
Contributor

Hey @zerosnacks, if this is still available for dev, I'd be happy to take it. I would appreciate any pointers to get started since the reth code linked in the issue has changed. Ty 🙏

@zerosnacks
Copy link
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-anvil Command: anvil good first issue Good for newcomers T-feature Type: feature
Projects
No open projects
Status: Todo
Development

No branches or pull requests

5 participants