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: eth_simulateV1 Request / Response types #1042

Conversation

DoTheBestToGetTheBest
Copy link
Contributor

I'am going to first convert this to draft, i need some help on some type, like error one and back & stateDb should i use Revm and foundry backend here or i do have to manually implemnt those ? Thank you

This pr should relate #901

@DoTheBestToGetTheBest DoTheBestToGetTheBest changed the title Eth simulate v1 request / response types Feat: eth_simulateV1 Request / Response types Jul 12, 2024
@DoTheBestToGetTheBest DoTheBestToGetTheBest marked this pull request as draft July 12, 2024 00:07
Copy link
Member

@mattsse mattsse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cool great start

like error one and back & stateDb should i use Revm and foundry backend here or i do have to manually implemnt those

those will be implemented by the nodes itself

crates/rpc-types-trace/src/simulate.rs Outdated Show resolved Hide resolved
crates/rpc-types-trace/src/simulate.rs Outdated Show resolved Hide resolved
crates/rpc-types-trace/src/simulate.rs Outdated Show resolved Hide resolved
@mattsse mattsse marked this pull request as ready for review July 15, 2024 16:13
@DoTheBestToGetTheBest
Copy link
Contributor Author

cool great start

ty!

those will be implemented by the nodes itself

ty but i'm not sure to understand wdym by this ?

Copy link
Member

@mattsse mattsse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in this pr we're only interested in the types

could you perhaps add some sanity serde json tests with sample requests from

https://github.com/ethereum/execution-apis/pull/484/files#diff-358634b5754d6188b1888d3bb89dbe4c8ba62ea941f2afa4d58e2a8a3bdf0f41

and we should move this file to the rpc-types-eth crate

@mattsse
Copy link
Member

mattsse commented Jul 19, 2024

@DoTheBestToGetTheBest do you have the bandwidth to finalize this pr?

we'd need a few tests that serialize data from the examples
https://github.com/ethereum/execution-apis/pull/484/files

@DoTheBestToGetTheBest
Copy link
Contributor Author

@DoTheBestToGetTheBest do you have the bandwidth to finalize this pr?

we'd need a few tests that serialize data from the examples https://github.com/ethereum/execution-apis/pull/484/files

sorry, it was already done and i though i commited change

Copy link
Member

@mattsse mattsse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cool,

this is likely not entirely complete what will unblock reth work to figure out what's missing

@mattsse mattsse merged commit 8ff415d into alloy-rs:main Jul 25, 2024
22 checks passed
ben186 pushed a commit to ben186/alloy that referenced this pull request Jul 27, 2024
* Feat: eth_simulateV1 Request / Response types

* Update lib.rs

* Update simulate.rs

* fmt

* Update simulate.rs

* Update simulate.rs

* Delete crates/rpc-types-trace/src/simulate.rs

* Update lib.rs

* Create simulate.rs

* Update lib.rs

* fmt

* fmt

* Update lib.rs

* Update lib.rs

* touchups

* fix type

---------

Co-authored-by: Matthias Seitz <[email protected]>
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

Successfully merging this pull request may close these issues.

3 participants