Skip to content

feat(anvil): add support for Isthmus hardfork (OP Stack) and make the OP spec configurable #10521

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

Open
zerosnacks opened this issue May 15, 2025 · 1 comment
Labels
A-reth-anvil Area: reth-anvil C-anvil Command: anvil T-feature Type: feature T-to-discuss Type: requires discussion

Comments

@zerosnacks
Copy link
Member

zerosnacks commented May 15, 2025

Component

Anvil

Describe the feature you would like

As highlighted in #10508 we currently pin to OpSpec::BEDROCK as we only really use the deposit transaction type, present in Bedrock.

// TODO: we currently pin to `OpSpecId::BEDROCK` as it is primarily used in the context of
// testing deposit transactions. We should make this configurable in the future.
let op_cfg = env.evm_env.cfg_env.clone().with_spec(op_revm::OpSpecId::BEDROCK);

Upgrading this to Isthmus (latest) currently fails with: Missing operator fee scalar for isthmus L1 Block.

It looks like up to the Holocene hardfork is supported but we need to explore this further..

Additional context

Ref: #10508 (comment)

#[test]
fn build_op_evm_with_extra_precompiles_default_spec() {
let (env, mut evm) = create_op_evm(
SpecId::default(),
// TODO: OpSpecId::ISTHMUS is not yet supported, fails with: `Missing operator fee
// scalar for isthmus L1 Block`.
OpSpecId::HOLOCENE,
);

@zerosnacks zerosnacks added T-feature Type: feature C-anvil Command: anvil A-reth-anvil Area: reth-anvil labels May 15, 2025
@github-project-automation github-project-automation bot moved this to Backlog in Foundry May 15, 2025
@zerosnacks zerosnacks added the T-to-discuss Type: requires discussion label May 15, 2025
@zerosnacks
Copy link
Member Author

Added to discuss label to indicate we didn't have a specific ask for this and we have to evaluate if there is a need / demand for it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-reth-anvil Area: reth-anvil C-anvil Command: anvil T-feature Type: feature T-to-discuss Type: requires discussion
Projects
Status: Backlog
Development

No branches or pull requests

1 participant