module/eth: add support for eth_simulateV1 #287
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Implements SimulateV1 function and SimulateV1Response type to support the eth_simulateV1 RPC method. This provides comprehensive simulation results including return data, gas used, and logs in a single call.
Owner
|
Have you tested this? A PR to fix #240 is verry welcome, but this does not look quite right yet. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR adds support for the
eth_simulateV1RPC method, which provides comprehensive simulation results in a single call. Unlikeeth_calloreth_estimateGas,eth_simulateV1returns return data, gas used, and logs together, making it ideal for debugging and transaction simulation.Changes
SimulateV1function that follows the same pattern asCallandAccessListSimulateV1Responsestruct with proper JSON unmarshalingoverridesparameter for state overridesblockNumberparameter (defaults to latest if nil)Implementation Details
The implementation follows the existing codebase patterns:
module.NewFactorywithmsgArgsWrapperfor consistent message handlingUnmarshalJSONmethod handles hex-encoded JSON fields (hexutil.Bytes,hexutil.Uint64)ReturnData,GasUsed, andLogsfieldsFixes #240
@lmittmann