feat(flashblocks): add UnifiedReceiptBuilder for seamless deposit receipt handling #345
+380
−57
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
Collaborator
✅ Heimdall Review Status
|
refcell
reviewed
Jan 11, 2026
refcell
reviewed
Jan 11, 2026
refcell
reviewed
Jan 11, 2026
refcell
reviewed
Jan 11, 2026
Contributor
Author
|
@refcell Ready to review |
refcell
reviewed
Jan 12, 2026
refcell
reviewed
Jan 12, 2026
refcell
reviewed
Jan 12, 2026
refcell
reviewed
Jan 12, 2026
…eipt handling Add a new UnifiedReceiptBuilder that handles both deposit and non-deposit transactions seamlessly without requiring error handling at the call site. The builder wraps OpRethReceiptBuilder and automatically handles the deposit receipt case internally, eliminating the need for callers to implement the try-catch pattern typically required when using build_receipt followed by build_deposit_receipt. Changes: - Add receipt_builder module with UnifiedReceiptBuilder - Update PendingStateBuilder to use UnifiedReceiptBuilder - Simplify execute_with_evm method by removing manual deposit handling - Remove receipt_builder parameter from PendingStateBuilder::new() Closes base#309
Removed wrapper around OpRethReceiptBuilder and implemented direct receipt building logic. This fixes type mismatches between OpTxEnvelope and OpTransactionSigned that prevented compilation. - Use direct OpTxType matching instead of build_receipt/build_deposit_receipt pattern - Accept ExecutionResult<E::HaltReason> for proper generic type handling - Remove unused alloy-op-evm dependency
- Add From<ReceiptBuildError> for ExecutionError and StateProcessorError - Remove unused _state parameter from build() - Replace .unwrap() with proper error handling - Add comprehensive tests for receipt building logic
- Add EVM-based tests that directly test build() method - Test legacy receipt building - Test deposit receipt building - Test Canyon hardfork activation (deposit_receipt_version) - Test failed transaction receipt
- Use RpcReceiptBuild error variant in From<ReceiptBuildError> impl - Remove no-op tests (test_legacy_receipt_type, test_deposit_receipt_type, test_tx_type_to_receipt_mapping)
Himess
force-pushed
the
feat/unified-receipt-builder
branch
from
090c618 to
a24dbfe
Compare
Contributor
Author
|
@refcell I think it's ready now |
Contributor
Author
|
Fixed formatting issues. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 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.
Summary
Introduces
UnifiedReceiptBuilder- a wrapper that handles both deposit and non-deposit transaction receipts seamlessly, eliminating the try-catch pattern required byOpReceiptBuilder.Changes
receipt_builder.rswithUnifiedReceiptBuilder<C>structbuild()method handles both tx types internallystate_builder.rs- removed ~40 lines of manual deposit handlingalloy-op-evmdependencyCloses #309