Skip to content

refactor: remove op-revm/op-alloy dependencies#312

Merged
klkvr merged 2 commits intoalloy-rs:mainfrom
theochap:theo/remove-op-deps
Mar 13, 2026
Merged

refactor: remove op-revm/op-alloy dependencies#312
klkvr merged 2 commits intoalloy-rs:mainfrom
theochap:theo/remove-op-deps

Conversation

@theochap
Copy link
Contributor

Summary

  • Remove all OP-specific code from alloy-evm (src/op/ directory, OP feature flags, OP dependencies)
  • Remove op-revm, op-alloy, and alloy-op-hardforks from workspace and crate dependencies
  • Remove op feature and all cfg-gated OP implementations (InvalidTxError for OpTransactionError, IntoTxEnv for OpTransaction<T>, env constructors, spec ID mapping, RPC conversions)

This decouples alloy-evm from Optimism-specific types, making it a general-purpose EVM crate. All removed OP-specific code is being re-implemented in alloy-op-evm using newtype wrappers (OpTx, OpTxError) to satisfy Rust's orphan rule.

Test plan

  • cargo check --all-features passes
  • cargo test passes (all non-OP tests unaffected)
  • Downstream optimism workspace compiles with alloy-op-evm providing the moved implementations

🤖 Generated with Claude Code

Move all OP-specific implementations (OpTransaction conversions, spec ID
mapping, env constructors, RPC support) out of alloy-evm into
alloy-op-evm. This decouples alloy-evm from Optimism-specific types,
making it a general-purpose EVM crate.

The removed code is being re-implemented in alloy-op-evm using newtype
wrappers (OpTx, OpTxError) to satisfy Rust's orphan rule.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Arsenii Kulikov <62447812+klkvr@users.noreply.github.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019ce847-4f8d-76a9-966a-91b53f67a971
@klkvr klkvr merged commit b8bcae0 into alloy-rs:main Mar 13, 2026
27 checks passed
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