Skip to content

Make op-rbuilder compatible with upcoming op-node - #566

Closed
avalonche wants to merge 1 commit into
mainfrom
revert-545-revert-537-joshklop/v5-op-node
Closed

Make op-rbuilder compatible with upcoming op-node#566
avalonche wants to merge 1 commit into
mainfrom
revert-545-revert-537-joshklop/v5-op-node

Conversation

@avalonche

Copy link
Copy Markdown
Collaborator

Reverts #545

Copilot AI review requested due to automatic review settings July 29, 2026 17:50

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates op-rbuilder to align with upstream API changes in newer reth/op-reth versions (per PR description: revert of #545), so it remains compatible with an upcoming op-node.

Changes:

  • Bump reth git dependencies to v2.3.0, op-reth to op-reth/v2.3.2-rc.2, and update related crates (revm, alloy-evm).
  • Update transaction-pool delegation to match updated pool/blobstore interfaces.
  • Extend payload building config wiring (parent_block_info) and resolve a clap arg-id collision for max_uncompressed_block_size.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
crates/op-rbuilder/src/pool/delegate.rs Adjusts delegated TransactionPool surface (mempool retain APIs + blobstore accessor) to match newer upstream traits.
crates/op-rbuilder/src/builder/payload.rs Passes through parent_block_info into reth_basic_payload_builder::PayloadConfig.
crates/op-rbuilder/src/args/op.rs Avoids clap argument id collision by explicitly setting an id for max_uncompressed_block_size.
Cargo.toml Moves reth deps to v2.3.0, op-reth to v2.3.2-rc.2, and bumps revm/alloy-evm versions accordingly.
Cargo.lock Updates the resolved dependency graph to reflect the new upstream versions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@julio4

julio4 commented Jul 29, 2026

Copy link
Copy Markdown
Member

In favor of #567

@julio4 julio4 closed this Jul 29, 2026
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.

4 participants