Skip to content

feat: ramp sequencer header gasLimit toward --builder.gaslimit - #163

Open
panos-xyz wants to merge 1 commit into
mainfrom
feat/sequencer-gas-limit-156
Open

feat: ramp sequencer header gasLimit toward --builder.gaslimit#163
panos-xyz wants to merge 1 commit into
mainfrom
feat/sequencer-gas-limit-156

Conversation

@panos-xyz

Copy link
Copy Markdown
Contributor

Summary

  • Sequencer assemble no longer copies parent gasLimit forever. --builder.gaslimit / --miner.gaslimit is now the header GasCeil.
  • Each assembled block ramps toward that target by at most ~1/1024 of the parent (same elasticity validators already enforce).
  • newSafeL2Block / attributes gas_limit overrides still win, so derivation keeps the imported header value.
  • Unset flag keeps current behavior (copy parent). Runtime miner_setGasLimit is not in this PR.

Fixes #156

Test plan

  • cargo nextest run -p morph-payload-builder
  • cargo check -p morph-node -p morph-payload-builder
  • Start a sequencer with --builder.gaslimit 60000000 and confirm successive assembled headers increase toward 60M within the 1/1024 bound

Wire reth's --builder.gaslimit / --miner.gaslimit into Morph assemble.
When set, each new block moves toward the target by at most ~1/1024 of
the parent (Ethereum CalcGasLimit). Explicit payload gas_limit overrides
are unchanged so derivation still imports the header as given.

Fixes #156
@github-actions github-actions Bot added the bug Something isn't working label Aug 14, 2026
@coderabbitai

coderabbitai Bot commented Aug 14, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@panos-xyz, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 116 minutes

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 039d8220-cee1-46cc-b670-d3ca24ceea6f

📥 Commits

Reviewing files that changed from the base of the PR and between 1494463 and 8ae2dd4.

📒 Files selected for processing (4)
  • README.md
  • crates/node/src/components/payload.rs
  • crates/payload/builder/src/builder.rs
  • crates/payload/builder/src/config.rs

Warning

Billing warning: we have not been able to collect payment for this subscription for more than 72 hours. Please update the payment method or pay any pending invoices in Billing to avoid service interruption.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sequencer cannot change block gasLimit; header stays at parent/genesis

1 participant