Skip to content

feat(sequencer): set block building limits from checkpoint limits (backport #20974)#21055

Merged
spalladino merged 12 commits intov4from
claudebox/2433a99083562757-1
Mar 4, 2026
Merged

feat(sequencer): set block building limits from checkpoint limits (backport #20974)#21055
spalladino merged 12 commits intov4from
claudebox/2433a99083562757-1

Conversation

@AztecBot
Copy link
Collaborator

@AztecBot AztecBot commented Mar 3, 2026

Summary

Backport of #20974 to v4.

The checkpoint builder now tracks remaining L2 gas, DA gas, and blob fields in a checkpoint while building each block, and forwards them to the public processor. This means that a proposer will not propose blocks that overall exceed checkpoint limits, and validators will properly reject them.

Cherry-picked commits:

  • feat(sequencer): set block building limits from checkpoint limits
  • feat(sequencer): only skip txs due to gas/blob limits during proposal building
  • fix(gas): saner defaults for da gas limit per tx
  • fix: fix spread when computing processor limits
  • fix(processor): check gas limits using used gas on reexecution
  • test: check that validators reject exceeded gas limits
  • chore: re-validate checkpoint limits before proposal and attestation
  • test: fix tests related to avm gas limits
  • fix(validator): do not check block-level limits on validation
  • chore: add inline comments

Conflict resolution:

  • timetable.ts: Resolved log level conflict — v4 uses createLogger (always defined) so kept this.log.info( without optional chaining (matching PR's intent to upgrade from verbose to info).

ClaudeBox log: http://ci.aztec-labs.com/2433a99083562757-1

@AztecBot AztecBot added the claudebox Owned by claudebox. it can push to this PR. label Mar 3, 2026
@AztecBot AztecBot force-pushed the claudebox/2433a99083562757-1 branch from df66cea to 60ff1cb Compare March 3, 2026 21:07
@spalladino spalladino marked this pull request as ready for review March 4, 2026 12:52
The logger parameter is optional but was accessed without optional
chaining, causing a TS2532 error after merging v4.
@spalladino spalladino merged commit 46943b6 into v4 Mar 4, 2026
9 checks passed
@spalladino spalladino deleted the claudebox/2433a99083562757-1 branch March 4, 2026 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants