-
Notifications
You must be signed in to change notification settings - Fork 54
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: Remove L2 Execution Payload #542
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
Codecov ReportAttention: Patch coverage is
✅ All tests successful. No failed tests found. Additional details and impacted files☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Switch looks great to me at a high-level. Looks like there are a few tests that are failing, though. Glad to send through once CI is green :)
### Description Removes payload conversion utilities. Kona removed the need to fetch the payload by number, using the block instead in op-rs/kona#542. Instead of converting a payload into the `L2BlockInfo` and `SystemConfig`, the `OpBlock` type is now fetched and converted into those types using the conversion utilities: - [`L2BlockInfo::from_block_and_genesis`](https://github.com/alloy-rs/op-alloy/blob/main/crates/protocol/src/block.rs#L134) - `to_system_config` standalone method introduced in #135
Description
Removes the
L2ExecutionPayload
type, changing theL2ChainProvider
trait to returnOpBlock
by number instead of payload by number. This type was effectively just used as an intermediate type to convert between raw block fromdebug_getRawBlock
and transactions, l2 block ref, and system config used to validate span batches.Closes #517