Skip to content

Comments

add maxUseGas field to eth_simulateV1#746

Open
Rhovian wants to merge 1 commit intoethereum:mainfrom
Rhovian:eth-simulate-max-gas
Open

add maxUseGas field to eth_simulateV1#746
Rhovian wants to merge 1 commit intoethereum:mainfrom
Rhovian:eth-simulate-max-gas

Conversation

@Rhovian
Copy link

@Rhovian Rhovian commented Jan 31, 2026

motivation for this PR can be found here:

ethereum/go-ethereum#32741

summary of changes:

  • Added maxUsedGas field to CallResultFailure
  • Added maxUsedGas field to CallResultSuccess
  • Added MaxUsedGas hexutil.Uint64 to the test generator's callResult Go struct

I didn't regenerate the test fixtures to keep the PR digestible, happy to make any adjustments as needed.

more context:

ethereum/go-ethereum#32789

@s1na
Copy link
Contributor

s1na commented Feb 10, 2026

Makes sense to me!

gakonst added a commit to alloy-rs/alloy that referenced this pull request Feb 17, 2026
Add optional `maxUsedGas` field to `SimCallResult`, mirroring
ethereum/execution-apis#746 and ethereum/go-ethereum#32789.

This field represents the maximum gas consumed during execution
before refunds are applied, enabling more accurate gas limit
estimation for complex transaction sequences.

Amp-Thread-ID: https://ampcode.com/threads/T-019c6b14-bfd0-71df-bce9-533057940778
Co-authored-by: Amp <[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.

2 participants