@aztec/sequencer-client 0.0.1-commit.b33fc05d0 → 0.0.1-commit.b3d3157a

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (105) hide show
  1. package/README.md +281 -21
  2. package/dest/client/sequencer-client.d.ts +8 -14
  3. package/dest/client/sequencer-client.d.ts.map +1 -1
  4. package/dest/client/sequencer-client.js +33 -83
  5. package/dest/config.d.ts +5 -1
  6. package/dest/config.d.ts.map +1 -1
  7. package/dest/config.js +42 -15
  8. package/dest/global_variable_builder/fee_predictor.d.ts +37 -0
  9. package/dest/global_variable_builder/fee_predictor.d.ts.map +1 -0
  10. package/dest/global_variable_builder/fee_predictor.js +128 -0
  11. package/dest/global_variable_builder/fee_provider.d.ts +21 -0
  12. package/dest/global_variable_builder/fee_provider.d.ts.map +1 -0
  13. package/dest/global_variable_builder/fee_provider.js +58 -0
  14. package/dest/global_variable_builder/global_builder.d.ts +14 -14
  15. package/dest/global_variable_builder/global_builder.d.ts.map +1 -1
  16. package/dest/global_variable_builder/global_builder.js +16 -50
  17. package/dest/global_variable_builder/index.d.ts +4 -2
  18. package/dest/global_variable_builder/index.d.ts.map +1 -1
  19. package/dest/global_variable_builder/index.js +2 -0
  20. package/dest/index.d.ts +2 -2
  21. package/dest/index.d.ts.map +1 -1
  22. package/dest/index.js +1 -1
  23. package/dest/publisher/config.d.ts +15 -3
  24. package/dest/publisher/config.d.ts.map +1 -1
  25. package/dest/publisher/config.js +19 -4
  26. package/dest/publisher/l1_tx_failed_store/failed_tx_store.d.ts +3 -4
  27. package/dest/publisher/l1_tx_failed_store/failed_tx_store.d.ts.map +1 -1
  28. package/dest/publisher/sequencer-bundle-simulator.d.ts +96 -0
  29. package/dest/publisher/sequencer-bundle-simulator.d.ts.map +1 -0
  30. package/dest/publisher/sequencer-bundle-simulator.js +198 -0
  31. package/dest/publisher/sequencer-publisher-factory.d.ts +3 -5
  32. package/dest/publisher/sequencer-publisher-factory.d.ts.map +1 -1
  33. package/dest/publisher/sequencer-publisher-factory.js +2 -3
  34. package/dest/publisher/sequencer-publisher.d.ts +73 -65
  35. package/dest/publisher/sequencer-publisher.d.ts.map +1 -1
  36. package/dest/publisher/sequencer-publisher.js +318 -523
  37. package/dest/sequencer/automine/automine_factory.d.ts +54 -0
  38. package/dest/sequencer/automine/automine_factory.d.ts.map +1 -0
  39. package/dest/sequencer/automine/automine_factory.js +84 -0
  40. package/dest/sequencer/automine/automine_sequencer.d.ts +151 -0
  41. package/dest/sequencer/automine/automine_sequencer.d.ts.map +1 -0
  42. package/dest/sequencer/automine/automine_sequencer.js +509 -0
  43. package/dest/sequencer/chain_state_overrides.d.ts +61 -0
  44. package/dest/sequencer/chain_state_overrides.d.ts.map +1 -0
  45. package/dest/sequencer/chain_state_overrides.js +98 -0
  46. package/dest/sequencer/checkpoint_proposal_job.d.ts +60 -10
  47. package/dest/sequencer/checkpoint_proposal_job.d.ts.map +1 -1
  48. package/dest/sequencer/checkpoint_proposal_job.js +748 -200
  49. package/dest/sequencer/checkpoint_proposal_job_metrics.d.ts +34 -0
  50. package/dest/sequencer/checkpoint_proposal_job_metrics.d.ts.map +1 -0
  51. package/dest/sequencer/checkpoint_proposal_job_metrics.js +72 -0
  52. package/dest/sequencer/checkpoint_voter.d.ts +1 -2
  53. package/dest/sequencer/checkpoint_voter.d.ts.map +1 -1
  54. package/dest/sequencer/checkpoint_voter.js +2 -5
  55. package/dest/sequencer/events.d.ts +48 -2
  56. package/dest/sequencer/events.d.ts.map +1 -1
  57. package/dest/sequencer/index.d.ts +3 -1
  58. package/dest/sequencer/index.d.ts.map +1 -1
  59. package/dest/sequencer/index.js +2 -0
  60. package/dest/sequencer/metrics.d.ts +13 -10
  61. package/dest/sequencer/metrics.d.ts.map +1 -1
  62. package/dest/sequencer/metrics.js +45 -20
  63. package/dest/sequencer/sequencer.d.ts +63 -13
  64. package/dest/sequencer/sequencer.d.ts.map +1 -1
  65. package/dest/sequencer/sequencer.js +349 -107
  66. package/dest/sequencer/timetable.d.ts +11 -1
  67. package/dest/sequencer/timetable.d.ts.map +1 -1
  68. package/dest/sequencer/timetable.js +41 -36
  69. package/dest/sequencer/types.d.ts +2 -2
  70. package/dest/sequencer/types.d.ts.map +1 -1
  71. package/dest/test/mock_checkpoint_builder.d.ts +4 -4
  72. package/dest/test/mock_checkpoint_builder.d.ts.map +1 -1
  73. package/dest/test/utils.d.ts +1 -1
  74. package/dest/test/utils.d.ts.map +1 -1
  75. package/dest/test/utils.js +7 -6
  76. package/package.json +27 -28
  77. package/src/client/sequencer-client.ts +53 -114
  78. package/src/config.ts +48 -12
  79. package/src/global_variable_builder/README.md +44 -0
  80. package/src/global_variable_builder/fee_predictor.ts +172 -0
  81. package/src/global_variable_builder/fee_provider.ts +75 -0
  82. package/src/global_variable_builder/global_builder.ts +25 -62
  83. package/src/global_variable_builder/index.ts +3 -1
  84. package/src/index.ts +10 -1
  85. package/src/publisher/config.ts +40 -6
  86. package/src/publisher/l1_tx_failed_store/failed_tx_store.ts +3 -1
  87. package/src/publisher/sequencer-bundle-simulator.ts +254 -0
  88. package/src/publisher/sequencer-publisher-factory.ts +3 -6
  89. package/src/publisher/sequencer-publisher.ts +377 -568
  90. package/src/sequencer/README.md +162 -450
  91. package/src/sequencer/automine/README.md +46 -0
  92. package/src/sequencer/automine/automine_factory.ts +145 -0
  93. package/src/sequencer/automine/automine_sequencer.ts +595 -0
  94. package/src/sequencer/chain_state_overrides.ts +162 -0
  95. package/src/sequencer/checkpoint_proposal_job.ts +894 -220
  96. package/src/sequencer/checkpoint_proposal_job_metrics.ts +128 -0
  97. package/src/sequencer/checkpoint_voter.ts +1 -12
  98. package/src/sequencer/events.ts +51 -2
  99. package/src/sequencer/index.ts +2 -0
  100. package/src/sequencer/metrics.ts +57 -24
  101. package/src/sequencer/sequencer.ts +429 -121
  102. package/src/sequencer/timetable.ts +49 -44
  103. package/src/sequencer/types.ts +1 -1
  104. package/src/test/mock_checkpoint_builder.ts +3 -3
  105. package/src/test/utils.ts +28 -10
@@ -436,33 +436,36 @@ function _apply_decs_2203_r(targetClass, memberDecs, classDecs, parentClass) {
436
436
  return (_apply_decs_2203_r = applyDecs2203RFactory())(targetClass, memberDecs, classDecs, parentClass);
437
437
  }
438
438
  var _dec, _dec1, _dec2, _dec3, _dec4, _dec5, _dec6, _dec7, _initProto;
439
- import { BlockNumber, IndexWithinCheckpoint } from '@aztec/foundation/branded-types';
439
+ import { BlockNumber, CheckpointNumber, IndexWithinCheckpoint } from '@aztec/foundation/branded-types';
440
440
  import { randomInt } from '@aztec/foundation/crypto/random';
441
441
  import { flipSignature, generateRecoverableSignature, generateUnrecoverableSignature } from '@aztec/foundation/crypto/secp256k1-signer';
442
+ import { InterruptError, TimeoutError } from '@aztec/foundation/error';
442
443
  import { filter } from '@aztec/foundation/iterator';
443
444
  import { createLogger } from '@aztec/foundation/log';
444
- import { sleep, sleepUntil } from '@aztec/foundation/sleep';
445
+ import { InterruptibleSleep } from '@aztec/foundation/sleep';
445
446
  import { Timer } from '@aztec/foundation/timer';
446
447
  import { isErrorClass, unfreeze } from '@aztec/foundation/types';
447
448
  import { CommitteeAttestationsAndSigners, MaliciousCommitteeAttestationsAndSigners } from '@aztec/stdlib/block';
448
- import { validateCheckpoint } from '@aztec/stdlib/checkpoint';
449
- import { getSlotStartBuildTimestamp } from '@aztec/stdlib/epoch-helpers';
449
+ import { getPreviousCheckpointOutHashes, validateCheckpoint } from '@aztec/stdlib/checkpoint';
450
+ import { computeQuorum, getSlotStartBuildTimestamp, getTimestampForSlot } from '@aztec/stdlib/epoch-helpers';
450
451
  import { Gas } from '@aztec/stdlib/gas';
451
- import { NoValidTxsError } from '@aztec/stdlib/interfaces/server';
452
+ import { InsufficientValidTxsError } from '@aztec/stdlib/interfaces/server';
452
453
  import { computeInHashFromL1ToL2Messages } from '@aztec/stdlib/messaging';
453
454
  import { orderAttestations, trimAttestations } from '@aztec/stdlib/p2p';
454
455
  import { AttestationTimeoutError } from '@aztec/stdlib/validators';
455
456
  import { Attributes, trackSpan } from '@aztec/telemetry-client';
456
457
  import { DutyAlreadySignedError, SlashingProtectionError } from '@aztec/validator-ha-signer/errors';
458
+ import { buildCheckpointSimulationOverridesPlan } from './chain_state_overrides.js';
457
459
  import { CheckpointVoter } from './checkpoint_voter.js';
458
460
  import { SequencerInterruptedError } from './errors.js';
459
461
  import { SequencerState } from './utils.js';
460
462
  /** How much time to sleep while waiting for min transactions to accumulate for a block */ const TXS_POLLING_MS = 500;
463
+ const ARCHIVER_SYNC_POLLING_MS = 200;
461
464
  _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('CheckpointProposalJob.proposeCheckpoint', function() {
462
465
  return {
463
466
  // nullish operator needed for tests
464
467
  [Attributes.COINBASE]: this.validatorClient.getCoinbaseForAttestor(this.attestorAddress)?.toString(),
465
- [Attributes.SLOT_NUMBER]: this.slot
468
+ [Attributes.SLOT_NUMBER]: this.targetSlot
466
469
  };
467
470
  }), _dec2 = trackSpan('CheckpointProposalJob.buildBlocksForCheckpoint'), _dec3 = trackSpan('CheckpointProposalJob.waitUntilNextSubslot'), _dec4 = trackSpan('CheckpointProposalJob.buildSingleBlock'), _dec5 = trackSpan('CheckpointProposalJob.waitForMinTxs'), _dec6 = trackSpan('CheckpointProposalJob.waitForAttestations'), _dec7 = trackSpan('CheckpointProposalJob.waitUntilTimeInSlot');
468
471
  /**
@@ -471,10 +474,12 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
471
474
  * as well as enqueueing votes for slashing and governance proposals. This class is created from
472
475
  * the Sequencer once the check for being the proposer for the slot has succeeded.
473
476
  */ export class CheckpointProposalJob {
474
- epoch;
475
- slot;
477
+ slotNow;
478
+ targetSlot;
479
+ targetEpoch;
476
480
  checkpointNumber;
477
481
  syncedToBlockNumber;
482
+ checkpointedCheckpointNumber;
478
483
  proposer;
479
484
  publisher;
480
485
  attestorAddress;
@@ -488,15 +493,18 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
488
493
  checkpointsBuilder;
489
494
  blockSink;
490
495
  l1Constants;
496
+ signatureContext;
491
497
  config;
492
498
  timetable;
493
499
  slasherClient;
494
500
  epochCache;
495
501
  dateProvider;
496
502
  metrics;
503
+ checkpointMetrics;
497
504
  eventEmitter;
498
505
  setStateFn;
499
506
  tracer;
507
+ proposedCheckpointData;
500
508
  static{
501
509
  ({ e: [_initProto] } = _apply_decs_2203_r(this, [
502
510
  [
@@ -542,12 +550,27 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
542
550
  ], []));
543
551
  }
544
552
  log;
545
- constructor(epoch, slot, checkpointNumber, syncedToBlockNumber, // TODO(palla/mbps): Can we remove the proposer in favor of attestorAddress? Need to check fisherman-node flows.
546
- proposer, publisher, attestorAddress, invalidateCheckpoint, validatorClient, globalsBuilder, p2pClient, worldState, l1ToL2MessageSource, l2BlockSource, checkpointsBuilder, blockSink, l1Constants, config, timetable, slasherClient, epochCache, dateProvider, metrics, eventEmitter, setStateFn, tracer, bindings){
547
- this.epoch = epoch;
548
- this.slot = slot;
553
+ checkpointEventLog;
554
+ /** Tracks the fire-and-forget L1 submission promise so it can be awaited during shutdown. */ pendingL1Submission;
555
+ interruptibleSleep;
556
+ interrupted;
557
+ /**
558
+ * Chain state overrides built once per slot in proposeCheckpoint after the checkpoint is
559
+ * complete. Carries the pending parent override (archive + slot + fee header) for pipelining,
560
+ * or the invalidation pending override when rolling back. Consumed by
561
+ * publisher.validateBlockHeader before broadcast.
562
+ */ checkpointSimulationOverridesPlan;
563
+ getSignatureContext() {
564
+ return this.signatureContext;
565
+ }
566
+ constructor(slotNow, targetSlot, targetEpoch, checkpointNumber, syncedToBlockNumber, checkpointedCheckpointNumber, // TODO(palla/mbps): Can we remove the proposer in favor of attestorAddress? Need to check fisherman-node flows.
567
+ proposer, publisher, attestorAddress, invalidateCheckpoint, validatorClient, globalsBuilder, p2pClient, worldState, l1ToL2MessageSource, l2BlockSource, checkpointsBuilder, blockSink, l1Constants, signatureContext, config, timetable, slasherClient, epochCache, dateProvider, metrics, checkpointMetrics, eventEmitter, setStateFn, tracer, bindings, proposedCheckpointData){
568
+ this.slotNow = slotNow;
569
+ this.targetSlot = targetSlot;
570
+ this.targetEpoch = targetEpoch;
549
571
  this.checkpointNumber = checkpointNumber;
550
572
  this.syncedToBlockNumber = syncedToBlockNumber;
573
+ this.checkpointedCheckpointNumber = checkpointedCheckpointNumber;
551
574
  this.proposer = proposer;
552
575
  this.publisher = publisher;
553
576
  this.attestorAddress = attestorAddress;
@@ -561,58 +584,337 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
561
584
  this.checkpointsBuilder = checkpointsBuilder;
562
585
  this.blockSink = blockSink;
563
586
  this.l1Constants = l1Constants;
587
+ this.signatureContext = signatureContext;
564
588
  this.config = config;
565
589
  this.timetable = timetable;
566
590
  this.slasherClient = slasherClient;
567
591
  this.epochCache = epochCache;
568
592
  this.dateProvider = dateProvider;
569
593
  this.metrics = metrics;
594
+ this.checkpointMetrics = checkpointMetrics;
570
595
  this.eventEmitter = eventEmitter;
571
596
  this.setStateFn = setStateFn;
572
597
  this.tracer = tracer;
573
- _initProto(this);
598
+ this.proposedCheckpointData = proposedCheckpointData;
599
+ this.interruptibleSleep = (_initProto(this), new InterruptibleSleep());
600
+ this.interrupted = false;
574
601
  this.log = createLogger('sequencer:checkpoint-proposal', {
575
602
  ...bindings,
576
- instanceId: `slot-${slot}`
603
+ instanceId: `slot-${this.slotNow}`
604
+ });
605
+ this.checkpointEventLog = createLogger('sequencer:checkpoint-events', {
606
+ ...bindings,
607
+ instanceId: `slot-${this.slotNow}`
608
+ });
609
+ }
610
+ /** Awaits the pending L1 submission if one is in progress. Call during shutdown. */ async awaitPendingSubmission() {
611
+ this.log.info('Awaiting pending L1 payload submission');
612
+ await this.pendingL1Submission;
613
+ }
614
+ /** Interrupts job-owned waits, including the publisher's send-at-slot sleep, so shutdown can finish. */ interrupt() {
615
+ this.interrupted = true;
616
+ this.interruptibleSleep.interrupt(true);
617
+ this.publisher.interrupt();
618
+ }
619
+ async awaitInterruptibleSleep(ms) {
620
+ if (this.interrupted) {
621
+ throw new SequencerInterruptedError();
622
+ }
623
+ if (ms <= 0) {
624
+ return;
625
+ }
626
+ try {
627
+ await this.interruptibleSleep.sleep(ms);
628
+ } catch (err) {
629
+ if (err instanceof InterruptError) {
630
+ throw new SequencerInterruptedError();
631
+ }
632
+ throw err;
633
+ }
634
+ }
635
+ logCheckpointEvent(eventName, message, fields) {
636
+ this.checkpointEventLog.debug(message, {
637
+ eventName: `sequencer-checkpoint-${eventName}`,
638
+ ...fields
577
639
  });
578
640
  }
579
641
  /**
580
642
  * Executes the checkpoint proposal job.
581
- * Returns the published checkpoint if successful, undefined otherwise.
643
+ * Builds blocks, assembles checkpoint, and broadcasts the proposal (blocking).
644
+ * Attestation collection, signing, and L1 submission are backgrounded so the
645
+ * work loop can return to IDLE immediately for consecutive slot proposals.
646
+ * Returns the built checkpoint if successful, undefined otherwise.
582
647
  */ async execute() {
583
648
  // Enqueue governance and slashing votes (returns promises that will be awaited later)
584
649
  // In fisherman mode, we simulate slashing but don't actually publish to L1
585
650
  // These are constant for the whole slot, so we only enqueue them once
586
- const votesPromises = new CheckpointVoter(this.slot, this.publisher, this.attestorAddress, this.validatorClient, this.slasherClient, this.l1Constants, this.config, this.metrics, this.log).enqueueVotes();
587
- // Build and propose the checkpoint. This will enqueue the request on the publisher if a checkpoint is built.
588
- const checkpoint = await this.proposeCheckpoint();
589
- // Wait until the voting promises have resolved, so all requests are enqueued (not sent)
590
- await Promise.all(votesPromises);
591
- if (checkpoint) {
592
- this.metrics.recordCheckpointProposalSuccess();
651
+ const votesPromises = new CheckpointVoter(this.targetSlot, this.publisher, this.attestorAddress, this.validatorClient, this.slasherClient, this.l1Constants, this.config, this.metrics, this.log).enqueueVotes();
652
+ // Build blocks, assemble checkpoint, and broadcast proposal (BLOCKING).
653
+ // Returns after broadcast — attestation collection is deferred.
654
+ const broadcast = await this.proposeCheckpoint();
655
+ if (!broadcast) {
656
+ await Promise.all(votesPromises);
657
+ // Still submit votes even without a checkpoint.
658
+ // Under proposer pipelining, vote-offenses signatures are EIP-712-bound to `targetSlot`
659
+ // (the pipelined slot in which the multicall is expected to mine). Submitting at the
660
+ // wall-clock time would let the multicall mine in a different L2 slot, causing
661
+ // signature verification to fail silently inside Multicall3. Delay submission to the
662
+ // start of `targetSlot` so the tx mines in the slot the vote was signed for.
663
+ if (!this.config.fishermanMode) {
664
+ this.pendingL1Submission = this.publisher.sendRequestsAt(this.targetSlot).then(()=>{});
665
+ }
666
+ return undefined;
593
667
  }
668
+ const { checkpoint } = broadcast;
669
+ this.metrics.recordCheckpointProposalSuccess();
594
670
  // Do not post anything to L1 if we are fishermen, but do perform L1 fee analysis
595
671
  if (this.config.fishermanMode) {
596
672
  await this.handleCheckpointEndAsFisherman(checkpoint);
597
- return;
673
+ return checkpoint;
598
674
  }
599
- // Then send everything to L1
600
- const l1Response = await this.publisher.sendRequests();
601
- const proposedAction = l1Response?.successfulActions.find((a)=>a === 'propose');
602
- if (proposedAction) {
603
- this.eventEmitter.emit('checkpoint-published', {
604
- checkpoint: this.checkpointNumber,
605
- slot: this.slot
675
+ // Background the attestation signing → L1 pipeline so the work loop is unblocked
676
+ this.pendingL1Submission = this.waitForAttestationsAndEnqueueSubmissionAsync(broadcast, votesPromises);
677
+ // Return the built checkpoint immediately — the work loop is now unblocked
678
+ return checkpoint;
679
+ }
680
+ /**
681
+ * Background pipeline: collects attestations, signs them, enqueues the checkpoint, and submits to L1.
682
+ * Runs as a fire-and-forget task stored in `pendingL1Submission` so the work loop is unblocked.
683
+ */ async waitForAttestationsAndEnqueueSubmissionAsync(broadcast, votesPromises) {
684
+ const { checkpoint } = broadcast;
685
+ try {
686
+ // Wait for all votes actions, enqueued at the beginning, to resolve
687
+ await Promise.all(votesPromises);
688
+ // Try to collect attestations from the committee
689
+ const signedAttestations = await this.getSignedCommitteeAttestations(broadcast);
690
+ // Wait for the previous checkpoint to land on L1 before submitting, so we can check it
691
+ // matches the proposed checkpoint we used as parent, and has valid attestations.
692
+ if (signedAttestations && await this.waitForValidParentCheckpointOnL1()) {
693
+ await this.enqueueCheckpointForSubmission({
694
+ checkpoint,
695
+ ...signedAttestations
696
+ });
697
+ }
698
+ // If we failed to collect attestations, at least check if we need to issue an invalidation
699
+ if (!signedAttestations && await this.waitForSyncedL2SlotNumber(this.slotNow)) {
700
+ const validationStatus = await this.l2BlockSource.getPendingChainValidationStatus();
701
+ if (!validationStatus.valid) {
702
+ this.log.warn(`Checkpoint ${validationStatus.checkpoint.checkpointNumber} has invalid attestations, enqueuing invalidation in spite of attestation collection failure`, {
703
+ checkpoint: validationStatus.checkpoint,
704
+ reason: validationStatus.reason
705
+ });
706
+ await this.enqueueInvalidation(validationStatus);
707
+ }
708
+ }
709
+ // Send whatever was enqueued: votes + (propose | invalidation | nothing).
710
+ const l1Response = await this.publisher.sendRequestsAt(this.targetSlot);
711
+ const proposedAction = l1Response?.successfulActions.find((a)=>a === 'propose');
712
+ if (proposedAction) {
713
+ this.logCheckpointEvent('published', `Checkpoint published for slot ${this.targetSlot}`, {
714
+ slot: this.targetSlot,
715
+ checkpointNumber: this.checkpointNumber,
716
+ successfulActions: l1Response?.successfulActions,
717
+ sentActions: l1Response?.sentActions
718
+ });
719
+ this.eventEmitter.emit('checkpoint-published', {
720
+ checkpoint: this.checkpointNumber,
721
+ slot: this.targetSlot
722
+ });
723
+ const coinbase = checkpoint.header.coinbase;
724
+ await this.metrics.incFilledSlot(this.publisher.getSenderAddress().toString(), coinbase);
725
+ } else {
726
+ this.logCheckpointEvent('publish-failed', `Checkpoint publish failed for slot ${this.targetSlot}`, {
727
+ slot: this.targetSlot,
728
+ checkpointNumber: this.checkpointNumber,
729
+ successfulActions: l1Response?.successfulActions,
730
+ failedActions: l1Response?.failedActions,
731
+ sentActions: l1Response?.sentActions,
732
+ expiredActions: l1Response?.expiredActions,
733
+ reason: 'propose_action_not_successful'
734
+ });
735
+ this.log.warn(`Checkpoint publish failed for slot ${this.targetSlot}`, {
736
+ slot: this.targetSlot,
737
+ checkpointNumber: this.checkpointNumber,
738
+ successfulActions: l1Response?.successfulActions,
739
+ failedActions: l1Response?.failedActions,
740
+ sentActions: l1Response?.sentActions,
741
+ expiredActions: l1Response?.expiredActions,
742
+ reason: 'propose_action_not_successful'
743
+ });
744
+ this.eventEmitter.emit('checkpoint-publish-failed', {
745
+ ...l1Response,
746
+ slot: this.targetSlot
747
+ });
748
+ this.metrics.recordPipelineDiscard();
749
+ }
750
+ } catch (err) {
751
+ if (err instanceof SequencerInterruptedError) {
752
+ return;
753
+ }
754
+ this.logCheckpointEvent('publish-failed', `Checkpoint publish failed for slot ${this.targetSlot}`, {
755
+ slot: this.targetSlot,
756
+ checkpointNumber: this.checkpointNumber,
757
+ reason: err instanceof Error ? err.message : String(err)
758
+ });
759
+ this.log.error(`Background attestation/L1 pipeline failed for slot ${this.targetSlot}`, err, {
760
+ slot: this.targetSlot,
761
+ checkpointNumber: this.checkpointNumber,
762
+ reason: err instanceof Error ? err.message : String(err)
606
763
  });
607
- const coinbase = checkpoint?.header.coinbase;
608
- await this.metrics.incFilledSlot(this.publisher.getSenderAddress().toString(), coinbase);
609
- return checkpoint;
610
- } else if (checkpoint) {
611
764
  this.eventEmitter.emit('checkpoint-publish-failed', {
612
- ...l1Response,
613
- slot: this.slot
765
+ slot: this.targetSlot
766
+ });
767
+ this.metrics.recordPipelineDiscard();
768
+ }
769
+ }
770
+ /** Enqueues the checkpoint for L1 submission. Called after pipeline sleep in execute(). */ async enqueueCheckpointForSubmission(result) {
771
+ const { checkpoint, attestations, attestationsSignature } = result;
772
+ // Build-frame deadlines are measured against `slotNow`; observers still see `targetSlot`.
773
+ this.setStateFn(SequencerState.PUBLISHING_CHECKPOINT, this.targetSlot, this.slotNow);
774
+ const aztecSlotDuration = this.l1Constants.slotDuration;
775
+ const submissionSlotStart = Number(getTimestampForSlot(this.targetSlot, this.l1Constants));
776
+ const txTimeoutAt = new Date((submissionSlotStart + aztecSlotDuration) * 1000);
777
+ // If we have been configured to potentially skip publishing checkpoint then roll the dice here
778
+ if (this.config.skipPublishingCheckpointsPercent !== undefined && this.config.skipPublishingCheckpointsPercent > 0) {
779
+ const roll = Math.max(0, randomInt(100));
780
+ if (roll < this.config.skipPublishingCheckpointsPercent) {
781
+ this.log.warn(`Skipping publishing proposal for checkpoint ${checkpoint.number}. Configured percentage: ${this.config.skipPublishingCheckpointsPercent}, generated value: ${roll}`);
782
+ return;
783
+ }
784
+ }
785
+ await this.publisher.enqueueProposeCheckpoint(checkpoint, attestations, attestationsSignature, {
786
+ txTimeoutAt
787
+ });
788
+ }
789
+ /**
790
+ * Wait until the archiver syncs past the given L2 slot number.
791
+ * The deadline is the end of `this.targetSlot`, beyond which any pipelined work would miss its
792
+ * L1 submission window and is no longer useful.
793
+ */ async waitForSyncedL2SlotNumber(waitForSlot) {
794
+ const targetSlotStart = Number(getTimestampForSlot(this.targetSlot, this.l1Constants));
795
+ const targetSlotEndMs = (targetSlotStart + this.l1Constants.slotDuration) * 1000;
796
+ const syncDelayTolerance = this.l1Constants.ethereumSlotDuration * 2 * 1000;
797
+ const timeoutSeconds = Math.max(0.1, (targetSlotEndMs + syncDelayTolerance - this.dateProvider.now()) / 1000);
798
+ try {
799
+ const timer = new Timer();
800
+ while(true){
801
+ const syncedSlot = await this.l2BlockSource.getSyncedL2SlotNumber();
802
+ if (syncedSlot !== undefined && syncedSlot >= waitForSlot) {
803
+ return true;
804
+ }
805
+ if (timeoutSeconds && timer.s() > timeoutSeconds) {
806
+ throw new TimeoutError(`Timeout awaiting archiver sync past slot ${waitForSlot}`);
807
+ }
808
+ await this.awaitInterruptibleSleep(ARCHIVER_SYNC_POLLING_MS);
809
+ }
810
+ } catch (err) {
811
+ if (err instanceof SequencerInterruptedError) {
812
+ throw err;
813
+ }
814
+ this.log.warn(`Archiver did not sync L1 past slot ${waitForSlot} before slot ${this.targetSlot} expired, discarding pipelined work`, {
815
+ checkpointNumber: this.checkpointNumber
816
+ });
817
+ this.emitPipelinedCheckpointDiscarded('archiver-sync-timeout');
818
+ return false;
819
+ }
820
+ }
821
+ /**
822
+ * Waits for the parent checkpoint to land on L1 before submitting a pipelined checkpoint.
823
+ * Polls until the archiver has synced L1 past the parent's slot, then verifies:
824
+ * - If we built on a proposed parent: it must have landed on L1 with matching hash and valid attestations.
825
+ * - If we built without a proposed parent: no new checkpoint must have appeared for that slot.
826
+ * If the parent has invalid attestations, enqueues an invalidation. Returns whether to proceed with the proposal.
827
+ */ async waitForValidParentCheckpointOnL1() {
828
+ if (this.config.skipWaitForValidParentCheckpointOnL1) {
829
+ this.log.warn(`Skipping waitForValidParentCheckpointOnL1 due to test configuration`, {
830
+ checkpointNumber: this.checkpointNumber
831
+ });
832
+ return true;
833
+ }
834
+ const parentCheckpointNumber = CheckpointNumber(this.checkpointNumber - 1);
835
+ // Wait until archiver has synced L1 past the parent's slot (slotNow)
836
+ if (!await this.waitForSyncedL2SlotNumber(this.slotNow)) {
837
+ return false;
838
+ }
839
+ const tips = await this.l2BlockSource.getL2Tips();
840
+ const checkpointedNumber = tips.checkpointed.checkpoint.number;
841
+ // We built on top of a proposed checkpoint. Verify it landed on L1 as expected.
842
+ if (this.proposedCheckpointData) {
843
+ // After syncing from L1 we see the chain tip has invalid attestations. This means the parent checkpoint was posted
844
+ // with invalid attestations, or it built on top of something with invalid attestations and didnt invalidate them.
845
+ // Either way, we thought our parent would be valid, so we have to throw away our work. But at least we'll try and
846
+ // invalidate on L1 so we clean up the chain for the next proposer. And we'll slash them, but that's handled elsewhere.
847
+ const validationStatus = await this.l2BlockSource.getPendingChainValidationStatus();
848
+ if (!validationStatus.valid) {
849
+ this.log.warn(`Parent checkpoint ${parentCheckpointNumber} has invalid attestations, discarding pipelined work`, {
850
+ checkpointNumber: this.checkpointNumber,
851
+ reason: validationStatus.reason
852
+ });
853
+ this.emitPipelinedCheckpointDiscarded('parent-invalid-attestations');
854
+ await this.enqueueInvalidation(validationStatus);
855
+ return false;
856
+ }
857
+ // The pending chain is valid. But did the parent checkpoint land on L1 at all?
858
+ if (checkpointedNumber < parentCheckpointNumber) {
859
+ this.log.warn(`Parent checkpoint ${parentCheckpointNumber} did not land on L1, discarding pipelined work`, {
860
+ checkpointNumber: this.checkpointNumber,
861
+ checkpointedNumber
862
+ });
863
+ this.emitPipelinedCheckpointDiscarded('parent-not-on-l1');
864
+ return false;
865
+ }
866
+ // It landed. But is it the one we were expecting?
867
+ const expectedHash = this.proposedCheckpointData.header.hash().toString();
868
+ if (tips.checkpointed.checkpoint.hash !== expectedHash) {
869
+ this.log.warn(`Parent checkpoint ${parentCheckpointNumber} hash mismatch on L1, discarding pipelined work`, {
870
+ checkpointNumber: this.checkpointNumber,
871
+ expectedHash,
872
+ actualHash: tips.checkpointed.checkpoint.hash
873
+ });
874
+ this.emitPipelinedCheckpointDiscarded('parent-hash-mismatch');
875
+ return false;
876
+ }
877
+ return true;
878
+ } else {
879
+ // We didn't see a proposed checkpoint at build time, so we built on checkpointed parent from two slots ago.
880
+ // But if a new checkpoint for the previous slot appeared on L1 in the meantime, our checkpoint assumed the wrong parent,
881
+ // so we have to discard our work. This can happen if we're somehow cut off from p2p and fail to see the checkpoint
882
+ // proposal for the previous slot.
883
+ if (checkpointedNumber > parentCheckpointNumber) {
884
+ this.log.warn(`Unexpected checkpoint ${checkpointedNumber} landed on L1 after we built on top of parent ${parentCheckpointNumber}, discarding pipelined work`, {
885
+ checkpointNumber: this.checkpointNumber,
886
+ checkpointedNumber
887
+ });
888
+ this.emitPipelinedCheckpointDiscarded('unexpected-parent-appeared');
889
+ return false;
890
+ }
891
+ return true;
892
+ }
893
+ }
894
+ /** Emits the pipelined-checkpoint-discarded event and records the metric. */ emitPipelinedCheckpointDiscarded(reason) {
895
+ this.metrics.recordPipelineParentCheckpointMismatch(reason);
896
+ this.eventEmitter.emit('pipelined-checkpoint-discarded', {
897
+ slot: this.targetSlot,
898
+ checkpointNumber: this.checkpointNumber,
899
+ reason
900
+ });
901
+ }
902
+ /** Simulates and enqueues an invalidation request for the invalid parent checkpoint. */ async enqueueInvalidation(validationStatus) {
903
+ if (this.config.skipInvalidateBlockAsProposer) {
904
+ this.log.warn(`Skipping checkpoint invalidation as proposer due to test configuration`);
905
+ return;
906
+ }
907
+ const invalidateRequest = await this.publisher.simulateInvalidateCheckpoint(validationStatus);
908
+ if (invalidateRequest) {
909
+ const submissionSlotStart = Number(getTimestampForSlot(this.targetSlot, this.l1Constants));
910
+ const txTimeoutAt = new Date((submissionSlotStart + this.l1Constants.slotDuration) * 1000);
911
+ this.publisher.enqueueInvalidateCheckpoint(invalidateRequest, {
912
+ txTimeoutAt
913
+ });
914
+ } else {
915
+ this.log.info(`Invalidation simulation returned undefined, checkpoint may have been removed already`, {
916
+ checkpointNumber: this.checkpointNumber
614
917
  });
615
- return undefined;
616
918
  }
617
919
  }
618
920
  async proposeCheckpoint() {
@@ -623,25 +925,70 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
623
925
  hasError: false
624
926
  };
625
927
  try {
928
+ const now = this.dateProvider.now();
929
+ if (this.proposedCheckpointData) {
930
+ // Measure against the wall-clock slot whose build window we are currently using.
931
+ // In pipelining mode `targetSlot` is intentionally one slot ahead, which makes the
932
+ // target-slot boundary a full slot away from the actual build start time.
933
+ const slotBoundaryMs = Number(getTimestampForSlot(this.slotNow, this.l1Constants)) * 1000;
934
+ this.checkpointMetrics.recordPipelinedCheckpointBuildStartOffsetFromSlotBoundary(now - slotBoundaryMs);
935
+ }
936
+ this.checkpointMetrics.startCheckpointTiming(now);
626
937
  // Get operator configured coinbase and fee recipient for this attestor
627
938
  const coinbase = this.validatorClient.getCoinbaseForAttestor(this.attestorAddress);
628
939
  const feeRecipient = this.validatorClient.getFeeRecipientForAttestor(this.attestorAddress);
629
940
  // Start the checkpoint
630
- this.setStateFn(SequencerState.INITIALIZING_CHECKPOINT, this.slot);
631
- this.metrics.incOpenSlot(this.slot, this.proposer?.toString() ?? 'unknown');
941
+ this.setStateFn(SequencerState.INITIALIZING_CHECKPOINT, this.targetSlot, this.slotNow);
942
+ this.logCheckpointEvent('slot-started', `Starting checkpoint proposal for slot ${this.targetSlot}`, {
943
+ buildSlot: this.slotNow,
944
+ submissionSlot: this.targetSlot,
945
+ slot: this.targetSlot,
946
+ checkpointNumber: this.checkpointNumber,
947
+ proposer: this.proposer?.toString(),
948
+ attestorAddress: this.attestorAddress.toString(),
949
+ publisherAddress: this.publisher.getSenderAddress().toString(),
950
+ coinbase: coinbase.toString()
951
+ });
952
+ this.metrics.incOpenSlot(this.targetSlot, this.proposer?.toString() ?? 'unknown');
632
953
  // Enqueues checkpoint invalidation (constant for the whole slot)
633
954
  if (this.invalidateCheckpoint && !this.config.skipInvalidateBlockAsProposer) {
634
955
  this.publisher.enqueueInvalidateCheckpoint(this.invalidateCheckpoint);
635
956
  }
636
- // Create checkpoint builder for the slot
637
- const checkpointGlobalVariables = await this.globalsBuilder.buildCheckpointGlobalVariables(coinbase, feeRecipient, this.slot);
957
+ // Build the simulation plan for this slot. When pipelining, this overrides L1's view of
958
+ // pending/archive/fee-header to "as if the proposed parent had landed", so both the
959
+ // mana-min-fee simulation (in the globals builder) and the pre-broadcast
960
+ // validateBlockHeader see the chain tip the eventual L1 send will see.
961
+ this.checkpointSimulationOverridesPlan = await buildCheckpointSimulationOverridesPlan({
962
+ checkpointNumber: this.checkpointNumber,
963
+ proposedCheckpointData: this.proposedCheckpointData,
964
+ invalidateToPendingCheckpointNumber: this.invalidateCheckpoint?.forcePendingCheckpointNumber,
965
+ checkpointedCheckpointNumber: this.checkpointedCheckpointNumber,
966
+ rollup: this.publisher.rollupContract,
967
+ signatureContext: this.signatureContext,
968
+ log: this.log
969
+ });
970
+ const checkpointGlobalVariables = await this.globalsBuilder.buildCheckpointGlobalVariables(coinbase, feeRecipient, this.targetSlot, this.checkpointSimulationOverridesPlan);
638
971
  // Collect L1 to L2 messages for the checkpoint and compute their hash
639
972
  const l1ToL2Messages = await this.l1ToL2MessageSource.getL1ToL2Messages(this.checkpointNumber);
640
973
  const inHash = computeInHashFromL1ToL2Messages(l1ToL2Messages);
641
- // Collect the out hashes of all the checkpoints before this one in the same epoch
642
- const previousCheckpointOutHashes = (await this.l2BlockSource.getCheckpointsDataForEpoch(this.epoch)).filter((c)=>c.checkpointNumber < this.checkpointNumber).map((c)=>c.checkpointOutHash);
643
- // Get the fee asset price modifier from the oracle
644
- const feeAssetPriceModifier = await this.publisher.getFeeAssetPriceModifier();
974
+ // Collect the out hashes of all the checkpoints before this one in the same epoch.
975
+ // Under pipelining the parent checkpoint may not be on L1 yet at build time, so the helper
976
+ // splices in the parent's checkpointOutHash from the locally-known proposed checkpoint so
977
+ // the resulting `epochOutHash` matches what validators (and L1) compute once the parent
978
+ // lands on L1.
979
+ const previousCheckpointOutHashes = await getPreviousCheckpointOutHashes({
980
+ blockSource: this.l2BlockSource,
981
+ epoch: this.targetEpoch,
982
+ checkpointNumber: this.checkpointNumber,
983
+ l1Constants: this.epochCache.getL1Constants(),
984
+ pipeliningEnabled: true,
985
+ proposedCheckpointData: this.proposedCheckpointData,
986
+ log: this.log
987
+ });
988
+ // Anchor the modifier to the predicted parent fee header: L1 will apply it against
989
+ // that, not against the latest published checkpoint (which lags by one under pipelining).
990
+ const predictedParentEthPerFeeAssetE12 = this.checkpointSimulationOverridesPlan?.pendingCheckpointState?.feeHeader?.ethPerFeeAsset;
991
+ const feeAssetPriceModifier = await this.publisher.getFeeAssetPriceModifier(predictedParentEthPerFeeAssetE12);
645
992
  const fork = _ts_add_disposable_resource(env, await this.worldState.fork(this.syncedToBlockNumber, {
646
993
  closeDelayMs: 12_000
647
994
  }), true);
@@ -654,7 +1001,7 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
654
1001
  };
655
1002
  const checkpointProposalOptions = {
656
1003
  publishFullTxs: !!this.config.publishTxsWithProposals,
657
- broadcastInvalidCheckpointProposal: this.config.broadcastInvalidBlockProposal
1004
+ broadcastInvalidCheckpointProposal: this.config.broadcastInvalidCheckpointProposalOnly || this.config.broadcastInvalidBlockProposal
658
1005
  };
659
1006
  let blocksInCheckpoint = [];
660
1007
  let blockPendingBroadcast = undefined;
@@ -674,28 +1021,45 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
674
1021
  throw err;
675
1022
  }
676
1023
  if (blocksInCheckpoint.length === 0) {
677
- this.log.warn(`No blocks were built for slot ${this.slot}`, {
678
- slot: this.slot
1024
+ this.logCheckpointEvent('build-failed', `Checkpoint build failed for slot ${this.targetSlot}`, {
1025
+ slot: this.targetSlot,
1026
+ checkpointNumber: this.checkpointNumber,
1027
+ reason: 'no_blocks_built'
1028
+ });
1029
+ this.log.warn(`No blocks were built for slot ${this.targetSlot}`, {
1030
+ slot: this.targetSlot,
1031
+ checkpointNumber: this.checkpointNumber,
1032
+ reason: 'no_blocks_built'
679
1033
  });
680
1034
  this.eventEmitter.emit('checkpoint-empty', {
681
- slot: this.slot
1035
+ slot: this.targetSlot
682
1036
  });
683
1037
  return undefined;
684
1038
  }
685
1039
  const minBlocksForCheckpoint = this.config.minBlocksForCheckpoint;
686
1040
  if (minBlocksForCheckpoint !== undefined && blocksInCheckpoint.length < minBlocksForCheckpoint) {
1041
+ this.logCheckpointEvent('build-failed', `Checkpoint build failed for slot ${this.targetSlot}`, {
1042
+ slot: this.targetSlot,
1043
+ checkpointNumber: this.checkpointNumber,
1044
+ blocksBuilt: blocksInCheckpoint.length,
1045
+ minBlocksForCheckpoint,
1046
+ reason: 'min_blocks_not_met'
1047
+ });
687
1048
  this.log.warn(`Checkpoint has fewer blocks than minimum (${blocksInCheckpoint.length} < ${minBlocksForCheckpoint}), skipping proposal`, {
688
- slot: this.slot,
1049
+ slot: this.targetSlot,
1050
+ checkpointNumber: this.checkpointNumber,
689
1051
  blocksBuilt: blocksInCheckpoint.length,
690
- minBlocksForCheckpoint
1052
+ minBlocksForCheckpoint,
1053
+ reason: 'min_blocks_not_met'
691
1054
  });
692
1055
  return undefined;
693
1056
  }
694
1057
  // Assemble and broadcast the checkpoint proposal, including the last block that was not
695
1058
  // broadcasted yet, and wait to collect the committee attestations.
696
- this.setStateFn(SequencerState.ASSEMBLING_CHECKPOINT, this.slot);
1059
+ this.setStateFn(SequencerState.ASSEMBLING_CHECKPOINT, this.targetSlot, this.slotNow);
697
1060
  const checkpoint = await checkpointBuilder.completeCheckpoint();
698
- // Final validation round for the checkpoint before we propose it, just for safety
1061
+ // Final validation: per-block limits are only checked if the operator set them explicitly.
1062
+ // Otherwise, checkpoint-level budgets were already enforced by the redistribution logic.
699
1063
  try {
700
1064
  validateCheckpoint(checkpoint, {
701
1065
  rollupManaLimit: this.l1Constants.rollupManaLimit,
@@ -705,68 +1069,95 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
705
1069
  maxTxsPerCheckpoint: this.config.maxTxsPerCheckpoint
706
1070
  });
707
1071
  } catch (err) {
708
- this.log.error(`Built an invalid checkpoint at slot ${this.slot} (skipping proposal)`, err, {
1072
+ this.logCheckpointEvent('build-failed', `Checkpoint build failed for slot ${this.targetSlot}`, {
1073
+ slot: this.targetSlot,
1074
+ checkpointNumber: this.checkpointNumber,
1075
+ blocksBuilt: blocksInCheckpoint.length,
1076
+ reason: 'invalid_checkpoint',
1077
+ checkpoint: checkpoint.header.toInspect()
1078
+ });
1079
+ this.log.error(`Built an invalid checkpoint at slot ${this.slotNow} (skipping proposal)`, err, {
1080
+ slot: this.targetSlot,
1081
+ checkpointNumber: this.checkpointNumber,
1082
+ blocksBuilt: blocksInCheckpoint.length,
1083
+ reason: 'invalid_checkpoint',
709
1084
  checkpoint: checkpoint.header.toInspect()
710
1085
  });
711
1086
  return undefined;
712
1087
  }
713
1088
  // Record checkpoint-level build metrics
714
- this.metrics.recordCheckpointBuild(checkpointBuildTimer.ms(), blocksInCheckpoint.length, checkpoint.getStats().txCount, Number(checkpoint.header.totalManaUsed.toBigInt()));
715
- // Do not collect attestations nor publish to L1 in fisherman mode
1089
+ this.checkpointMetrics.recordCheckpointBuild(checkpointBuildTimer.ms(), blocksInCheckpoint.length, checkpoint.getStats().txCount, Number(checkpoint.header.totalManaUsed.toBigInt()));
1090
+ this.logCheckpointEvent('built', `Checkpoint built for slot ${this.targetSlot}`, {
1091
+ slot: this.targetSlot,
1092
+ buildSlot: this.slotNow,
1093
+ checkpointNumber: this.checkpointNumber,
1094
+ proposer: this.proposer?.toString(),
1095
+ attestorAddress: this.attestorAddress.toString(),
1096
+ publisherAddress: this.publisher.getSenderAddress().toString(),
1097
+ blocksBuilt: blocksInCheckpoint.length,
1098
+ txCount: checkpoint.getStats().txCount,
1099
+ totalMana: Number(checkpoint.header.totalManaUsed.toBigInt())
1100
+ });
1101
+ // In fisherman mode, return the checkpoint without broadcasting or collecting attestations
716
1102
  if (this.config.fishermanMode) {
717
- this.log.info(`Built checkpoint for slot ${this.slot} with ${blocksInCheckpoint.length} blocks. ` + `Skipping proposal in fisherman mode.`, {
718
- slot: this.slot,
1103
+ this.log.info(`Built checkpoint for slot ${this.targetSlot} with ${blocksInCheckpoint.length} blocks. ` + `Skipping proposal in fisherman mode.`, {
1104
+ slot: this.targetSlot,
719
1105
  checkpoint: checkpoint.header.toInspect(),
720
1106
  blocksBuilt: blocksInCheckpoint.length
721
1107
  });
722
1108
  this.metrics.recordCheckpointSuccess();
723
- return checkpoint;
1109
+ // Return a broadcast result with a dummy proposal — fisherman mode skips attestation collection
1110
+ return {
1111
+ checkpoint,
1112
+ proposal: undefined,
1113
+ blockProposedAt: this.dateProvider.now()
1114
+ };
724
1115
  }
725
- // Include the block pending broadcast in the checkpoint proposal if any
726
- const lastBlock = blockPendingBroadcast && {
727
- blockHeader: blockPendingBroadcast.block.header,
728
- indexWithinCheckpoint: blockPendingBroadcast.block.indexWithinCheckpoint,
729
- txs: blockPendingBroadcast.txs
730
- };
731
- // Create the checkpoint proposal and broadcast it
732
- const proposal = await this.validatorClient.createCheckpointProposal(checkpoint.header, checkpoint.archive.root, feeAssetPriceModifier, lastBlock, this.proposer, checkpointProposalOptions);
733
- const blockProposedAt = this.dateProvider.now();
734
- await this.p2pClient.broadcastCheckpointProposal(proposal);
735
- this.setStateFn(SequencerState.COLLECTING_ATTESTATIONS, this.slot);
736
- const attestations = await this.waitForAttestations(proposal);
737
- const blockAttestedAt = this.dateProvider.now();
738
- this.metrics.recordCheckpointAttestationDelay(blockAttestedAt - blockProposedAt);
739
- // Proposer must sign over the attestations before pushing them to L1
740
- const signer = this.proposer ?? this.publisher.getSenderAddress();
741
- let attestationsSignature;
1116
+ // Validate the header against L1 state before broadcasting.
1117
+ // If this fails the slot is aborted before any gossip work; state drift between here
1118
+ // and the eventual L1 send is caught by the bundle simulate at send time.
742
1119
  try {
743
- attestationsSignature = await this.validatorClient.signAttestationsAndSigners(attestations, signer, this.slot, this.checkpointNumber);
1120
+ await this.publisher.validateBlockHeader(checkpoint.header, this.checkpointSimulationOverridesPlan);
744
1121
  } catch (err) {
745
- // We shouldn't really get here since we yield to another HA node
746
- // as soon as we see these errors when creating block or checkpoint proposals.
747
- if (this.handleHASigningError(err, 'Attestations signature')) {
748
- return undefined;
749
- }
750
- throw err;
1122
+ this.log.error(`Pre-broadcast header validation failed for slot ${this.targetSlot}; aborting`, err, {
1123
+ slot: this.targetSlot,
1124
+ checkpointNumber: this.checkpointNumber
1125
+ });
1126
+ this.metrics.recordCheckpointProposalFailed('header_validation_failed');
1127
+ this.eventEmitter.emit('header-validation-failed', {
1128
+ slot: this.targetSlot,
1129
+ checkpointNumber: this.checkpointNumber,
1130
+ reason: err instanceof Error ? err.message : String(err)
1131
+ });
1132
+ return undefined;
751
1133
  }
752
- // Enqueue publishing the checkpoint to L1
753
- this.setStateFn(SequencerState.PUBLISHING_CHECKPOINT, this.slot);
754
- const aztecSlotDuration = this.l1Constants.slotDuration;
755
- const slotStartBuildTimestamp = this.getSlotStartBuildTimestamp();
756
- const txTimeoutAt = new Date((slotStartBuildTimestamp + aztecSlotDuration) * 1000);
757
- // If we have been configured to potentially skip publishing checkpoint then roll the dice here
758
- if (this.config.skipPublishingCheckpointsPercent !== undefined && this.config.skipPublishingCheckpointsPercent > 0) {
759
- const result = Math.max(0, randomInt(100));
760
- if (result < this.config.skipPublishingCheckpointsPercent) {
761
- this.log.warn(`Skipping publishing proposal for checkpoint ${checkpoint.number}. Configured percentage: ${this.config.skipPublishingCheckpointsPercent}, generated value: ${result}`);
762
- return checkpoint;
1134
+ // Create the checkpoint proposal and broadcast it
1135
+ const proposal = await this.validatorClient.createCheckpointProposal(checkpoint.header, checkpoint.archive.root, this.checkpointNumber, feeAssetPriceModifier, blockPendingBroadcast, this.proposer, checkpointProposalOptions);
1136
+ // Advance our own optimistic proposed-checkpoint tip locally before gossiping. Gossipsub
1137
+ // doesn't echo our own messages back, so this is how the proposer makes its own proposed
1138
+ // checkpoint visible for pipelining the next slot. Built from local checkpoint data — never
1139
+ // from the broadcast proposal archive, which may be deliberately corrupted under test flags.
1140
+ // Fail closed: if this throws, the outer catch aborts the slot before gossiping.
1141
+ await this.syncProposedCheckpointToArchiver(checkpoint, blocksInCheckpoint.length, feeAssetPriceModifier);
1142
+ const blockProposedAt = this.dateProvider.now();
1143
+ if (this.config.skipBroadcastCheckpointProposal) {
1144
+ // Test-only: suppress the CheckpointProposal so peers never see a proposed checkpoint for
1145
+ // this slot, but still broadcast the held last block standalone so peers' archivers ingest
1146
+ // it as a proposed-but-uncheckpointed tip — the exact orphan-block state that
1147
+ // pruneOrphanProposedBlocks / checkSync must handle.
1148
+ if (blockPendingBroadcast && !this.config.skipBroadcastProposals) {
1149
+ await this.p2pClient.broadcastProposal(blockPendingBroadcast);
763
1150
  }
1151
+ } else if (!this.config.skipBroadcastProposals) {
1152
+ await this.p2pClient.broadcastCheckpointProposal(proposal);
1153
+ this.checkpointMetrics.noteCheckpointBroadcast(this.dateProvider.now());
764
1154
  }
765
- await this.publisher.enqueueProposeCheckpoint(checkpoint, attestations, attestationsSignature, {
766
- txTimeoutAt,
767
- forcePendingCheckpointNumber: this.invalidateCheckpoint?.forcePendingCheckpointNumber
768
- });
769
- return checkpoint;
1155
+ // Return immediately after broadcast — attestation collection happens in the background
1156
+ return {
1157
+ checkpoint,
1158
+ proposal,
1159
+ blockProposedAt
1160
+ };
770
1161
  } catch (e) {
771
1162
  env.error = e;
772
1163
  env.hasError = true;
@@ -779,7 +1170,7 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
779
1170
  // swallow this error. It's already been logged by a function deeper in the stack
780
1171
  return undefined;
781
1172
  }
782
- this.log.error(`Error building checkpoint at slot ${this.slot}`, err);
1173
+ this.log.error(`Error building checkpoint at slot ${this.targetSlot}`, err);
783
1174
  return undefined;
784
1175
  }
785
1176
  }
@@ -795,11 +1186,19 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
795
1186
  const blocksBuilt = blocksInCheckpoint.length;
796
1187
  const indexWithinCheckpoint = IndexWithinCheckpoint(blocksBuilt);
797
1188
  const blockNumber = BlockNumber(initialBlockNumber + blocksBuilt);
1189
+ if (blocksBuilt >= this.config.maxBlocksPerCheckpoint) {
1190
+ this.log.debug(`Reached max blocks per checkpoint`, {
1191
+ slot: this.targetSlot,
1192
+ blocksBuilt,
1193
+ maxBlocksPerCheckpoint: this.config.maxBlocksPerCheckpoint
1194
+ });
1195
+ break;
1196
+ }
798
1197
  const secondsIntoSlot = this.getSecondsIntoSlot();
799
1198
  const timingInfo = this.timetable.canStartNextBlock(secondsIntoSlot);
800
1199
  if (!timingInfo.canStart) {
801
1200
  this.log.debug(`Not enough time left in slot to start another block`, {
802
- slot: this.slot,
1201
+ slot: this.targetSlot,
803
1202
  blocksBuilt,
804
1203
  secondsIntoSlot
805
1204
  });
@@ -816,20 +1215,22 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
816
1215
  indexWithinCheckpoint,
817
1216
  txHashesAlreadyIncluded
818
1217
  });
819
- // TODO(palla/mbps): Review these conditions. We may want to keep trying in some scenarios.
820
- if (!buildResult && timingInfo.isLastBlock) {
821
- break;
822
- } else if (!buildResult && timingInfo.deadline !== undefined) {
823
- // But if there is still time for more blocks, wait until the next subslot and try again
1218
+ // If we failed to build the block due to insufficient txs, we try again if there is still time left in the slot
1219
+ if ('failure' in buildResult) {
1220
+ // If this was the last subslot, or we're running with a single block per slot, we're done
1221
+ if (timingInfo.isLastBlock || timingInfo.deadline === undefined) {
1222
+ break;
1223
+ }
1224
+ // Otherwise, if there is still time for more blocks, we wait until the next subslot and try again
824
1225
  await this.waitUntilNextSubslot(timingInfo.deadline);
825
1226
  continue;
826
- } else if (!buildResult) {
827
- break;
828
- } else if ('error' in buildResult) {
829
- // If there was an error building the block, just exit the loop and give up the rest of the slot
1227
+ }
1228
+ // If there was an error building the block, we just exit the loop and give up the rest of the slot.
1229
+ // We don't want to risk building more blocks if something went wrong.
1230
+ if ('error' in buildResult) {
830
1231
  if (!(buildResult.error instanceof SequencerInterruptedError)) {
831
- this.log.warn(`Halting block building for slot ${this.slot}`, {
832
- slot: this.slot,
1232
+ this.log.warn(`Halting block building for slot ${this.targetSlot}`, {
1233
+ slot: this.targetSlot,
833
1234
  blocksBuilt,
834
1235
  error: buildResult.error
835
1236
  });
@@ -837,42 +1238,41 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
837
1238
  break;
838
1239
  }
839
1240
  const { block, usedTxs } = buildResult;
840
- blocksInCheckpoint.push(block);
841
- // Sync the proposed block to the archiver to make it available
842
- // Note that the checkpoint builder uses its own fork so it should not need to wait for this syncing
843
- // Eventually we should refactor the checkpoint builder to not need a separate long-lived fork
844
- // Fire and forget - don't block the critical path, but log errors
845
- this.syncProposedBlockToArchiver(block).catch((err)=>{
846
- this.log.error(`Failed to sync proposed block ${block.number} to archiver`, {
847
- blockNumber: block.number,
848
- err
849
- });
1241
+ this.checkpointMetrics.noteCheckpointBlockBuilt(this.dateProvider.now(), {
1242
+ isFirstBlock: blocksBuilt === 0,
1243
+ isLastBlock: timingInfo.isLastBlock
850
1244
  });
1245
+ blocksInCheckpoint.push(block);
851
1246
  usedTxs.forEach((tx)=>txHashesAlreadyIncluded.add(tx.txHash.toString()));
852
- // If this is the last block, exit the loop now so we start collecting attestations
1247
+ // Sign the block proposal. This will throw if HA signing fails.
1248
+ const proposal = await this.createBlockProposal(block, inHash, usedTxs, {
1249
+ ...blockProposalOptions,
1250
+ broadcastInvalidBlockProposal: blockProposalOptions.broadcastInvalidBlockProposal || block.indexWithinCheckpoint === this.config.invalidBlockProposalIndexWithinCheckpoint
1251
+ });
1252
+ // Sync the proposed block to the archiver to make it available, only after we've managed to sign the proposal,
1253
+ // so we avoid polluting our archive with a block that would fail.
1254
+ // We wait for the sync to succeed, as this helps catch consistency errors, even if it means we lose some time for block-building.
1255
+ // If this throws, we abort the entire checkpoint.
1256
+ await this.syncProposedBlockToArchiver(block);
1257
+ // If this is the last block, do not broadcast it, since it will be included in the checkpoint proposal.
853
1258
  if (timingInfo.isLastBlock) {
854
- this.log.verbose(`Completed final block ${blockNumber} for slot ${this.slot}`, {
855
- slot: this.slot,
1259
+ this.log.verbose(`Completed final block ${blockNumber} for slot ${this.targetSlot}`, {
1260
+ slot: this.targetSlot,
856
1261
  blockNumber,
857
1262
  blocksBuilt
858
1263
  });
859
- blockPendingBroadcast = {
860
- block,
861
- txs: usedTxs
862
- };
1264
+ blockPendingBroadcast = proposal;
863
1265
  break;
864
1266
  }
865
- // For non-last blocks, broadcast the block proposal (unless we're in fisherman mode)
866
- // If the block is the last one, we'll broadcast it along with the checkpoint at the end of the loop
867
- if (!this.config.fishermanMode) {
868
- const proposal = await this.validatorClient.createBlockProposal(block.header, block.indexWithinCheckpoint, inHash, block.archive.root, usedTxs, this.proposer, blockProposalOptions);
1267
+ // Once we have a signed proposal and the archiver agreed with our proposed block, then we broadcast it.
1268
+ if (proposal && !this.config.skipBroadcastProposals) {
869
1269
  await this.p2pClient.broadcastProposal(proposal);
870
1270
  }
871
1271
  // Wait until the next block's start time
872
1272
  await this.waitUntilNextSubslot(timingInfo.deadline);
873
1273
  }
874
- this.log.verbose(`Block building loop completed for slot ${this.slot}`, {
875
- slot: this.slot,
1274
+ this.log.verbose(`Block building loop completed for slot ${this.targetSlot}`, {
1275
+ slot: this.targetSlot,
876
1276
  blocksBuilt: blocksInCheckpoint.length
877
1277
  });
878
1278
  return {
@@ -880,16 +1280,23 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
880
1280
  blockPendingBroadcast
881
1281
  };
882
1282
  }
1283
+ /** Creates a block proposal for a given block via the validator client (unless in fisherman mode) */ createBlockProposal(block, inHash, usedTxs, blockProposalOptions) {
1284
+ if (this.config.fishermanMode) {
1285
+ this.log.info(`Skipping block proposal for block ${block.number} in fisherman mode`);
1286
+ return Promise.resolve(undefined);
1287
+ }
1288
+ return this.validatorClient.createBlockProposal(block.header, this.checkpointNumber, block.indexWithinCheckpoint, inHash, block.archive.root, usedTxs, this.proposer, blockProposalOptions);
1289
+ }
883
1290
  /** Sleeps until it is time to produce the next block in the slot */ async waitUntilNextSubslot(nextSubslotStart) {
884
- this.setStateFn(SequencerState.WAITING_UNTIL_NEXT_BLOCK, this.slot);
1291
+ this.setStateFn(SequencerState.WAITING_UNTIL_NEXT_BLOCK, this.targetSlot, this.slotNow);
885
1292
  this.log.verbose(`Waiting until time for the next block at ${nextSubslotStart}s into slot`, {
886
- slot: this.slot
1293
+ slot: this.targetSlot
887
1294
  });
888
1295
  await this.waitUntilTimeInSlot(nextSubslotStart);
889
1296
  }
890
1297
  /** Builds a single block. Called from the main block building loop. */ async buildSingleBlock(checkpointBuilder, opts) {
891
1298
  const { blockTimestamp, forceCreate, blockNumber, indexWithinCheckpoint, buildDeadline, txHashesAlreadyIncluded } = opts;
892
- this.log.verbose(`Preparing block ${blockNumber} index ${indexWithinCheckpoint} at checkpoint ${this.checkpointNumber} for slot ${this.slot}`, {
1299
+ this.log.verbose(`Preparing block ${blockNumber} index ${indexWithinCheckpoint} at checkpoint ${this.checkpointNumber} for slot ${this.targetSlot}`, {
893
1300
  ...checkpointBuilder.getConstantData(),
894
1301
  ...opts
895
1302
  });
@@ -897,62 +1304,92 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
897
1304
  // Wait until we have enough txs to build the block
898
1305
  const { availableTxs, canStartBuilding, minTxs } = await this.waitForMinTxs(opts);
899
1306
  if (!canStartBuilding) {
900
- this.log.warn(`Not enough txs to build block ${blockNumber} at index ${indexWithinCheckpoint} in slot ${this.slot} (got ${availableTxs} txs but needs ${minTxs})`, {
1307
+ this.logCheckpointEvent('block-build-failed', `Block build failed for slot ${this.targetSlot}`, {
1308
+ reason: 'insufficient_txs',
901
1309
  blockNumber,
902
- slot: this.slot,
903
- indexWithinCheckpoint
1310
+ slot: this.targetSlot,
1311
+ checkpointNumber: this.checkpointNumber,
1312
+ indexWithinCheckpoint,
1313
+ availableTxs,
1314
+ minTxs
1315
+ });
1316
+ this.log.warn(`Not enough txs to build block ${blockNumber} at index ${indexWithinCheckpoint} in slot ${this.targetSlot} (got ${availableTxs} txs but needs ${minTxs})`, {
1317
+ reason: 'insufficient_txs',
1318
+ blockNumber,
1319
+ slot: this.targetSlot,
1320
+ checkpointNumber: this.checkpointNumber,
1321
+ indexWithinCheckpoint,
1322
+ availableTxs,
1323
+ minTxs
904
1324
  });
905
1325
  this.eventEmitter.emit('block-tx-count-check-failed', {
906
1326
  minTxs,
907
1327
  availableTxs,
908
- slot: this.slot
1328
+ slot: this.targetSlot
909
1329
  });
910
1330
  this.metrics.recordBlockProposalFailed('insufficient_txs');
911
- return undefined;
1331
+ return {
1332
+ failure: 'insufficient-txs'
1333
+ };
912
1334
  }
913
1335
  // Create iterator to pending txs. We filter out txs already included in previous blocks in the checkpoint
914
1336
  // just in case p2p failed to sync the provisional block and didn't get to remove those txs from the mempool yet.
915
1337
  const pendingTxs = filter(this.p2pClient.iterateEligiblePendingTxs(), (tx)=>!txHashesAlreadyIncluded.has(tx.txHash.toString()));
916
- this.log.debug(`Building block ${blockNumber} at index ${indexWithinCheckpoint} for slot ${this.slot} with ${availableTxs} available txs`, {
917
- slot: this.slot,
1338
+ this.log.debug(`Building block ${blockNumber} at index ${indexWithinCheckpoint} for slot ${this.targetSlot} with ${availableTxs} available txs`, {
1339
+ slot: this.targetSlot,
918
1340
  blockNumber,
919
1341
  indexWithinCheckpoint
920
1342
  });
921
- this.setStateFn(SequencerState.CREATING_BLOCK, this.slot);
922
- // Per-block limits derived at startup by computeBlockLimits(), further capped
1343
+ this.setStateFn(SequencerState.CREATING_BLOCK, this.targetSlot, this.slotNow);
1344
+ // Per-block limits are operator overrides (from SEQ_MAX_L2_BLOCK_GAS etc.) further capped
923
1345
  // by remaining checkpoint-level budgets inside CheckpointBuilder before each block is built.
1346
+ // minValidTxs is passed into the builder so it can reject the block *before* updating state.
1347
+ const minValidTxs = forceCreate ? 0 : this.config.minValidTxsPerBlock ?? minTxs;
924
1348
  const blockBuilderOptions = {
925
1349
  maxTransactions: this.config.maxTxsPerBlock,
926
1350
  maxBlockGas: this.config.maxL2BlockGas !== undefined || this.config.maxDABlockGas !== undefined ? new Gas(this.config.maxDABlockGas ?? Infinity, this.config.maxL2BlockGas ?? Infinity) : undefined,
927
1351
  deadline: buildDeadline,
928
- isBuildingProposal: true
1352
+ isBuildingProposal: true,
1353
+ minValidTxs,
1354
+ maxBlocksPerCheckpoint: this.timetable.maxNumberOfBlocks,
1355
+ perBlockAllocationMultiplier: this.config.perBlockAllocationMultiplier
929
1356
  };
930
- // Actually build the block by executing txs
1357
+ // Actually build the block by executing txs. The builder throws InsufficientValidTxsError
1358
+ // if the number of successfully processed txs is below minValidTxs, ensuring state is not
1359
+ // updated for blocks that will be discarded.
931
1360
  const buildResult = await this.buildSingleBlockWithCheckpointBuilder(checkpointBuilder, pendingTxs, blockNumber, blockTimestamp, blockBuilderOptions);
932
1361
  // If any txs failed during execution, drop them from the mempool so we don't pick them up again
933
1362
  await this.dropFailedTxsFromP2P(buildResult.failedTxs);
934
- // Check if we have created a block with enough txs. If there were invalid txs in the pool, or if execution took
935
- // too long, then we may not get to minTxsPerBlock after executing public functions.
936
- const minValidTxs = this.config.minValidTxsPerBlock ?? minTxs;
937
- const numTxs = buildResult.status === 'no-valid-txs' ? 0 : buildResult.numTxs;
938
- if (buildResult.status === 'no-valid-txs' || !forceCreate && numTxs < minValidTxs) {
939
- this.log.warn(`Block ${blockNumber} at index ${indexWithinCheckpoint} on slot ${this.slot} has too few valid txs to be proposed`, {
940
- slot: this.slot,
1363
+ if (buildResult.status === 'insufficient-valid-txs') {
1364
+ this.logCheckpointEvent('block-build-failed', `Block build failed for slot ${this.targetSlot}`, {
1365
+ reason: 'insufficient_valid_txs',
1366
+ slot: this.targetSlot,
1367
+ checkpointNumber: this.checkpointNumber,
1368
+ blockNumber,
1369
+ numTxs: buildResult.processedCount,
1370
+ indexWithinCheckpoint,
1371
+ minValidTxs
1372
+ });
1373
+ this.log.warn(`Block ${blockNumber} at index ${indexWithinCheckpoint} on slot ${this.targetSlot} has too few valid txs to be proposed`, {
1374
+ reason: 'insufficient_valid_txs',
1375
+ slot: this.targetSlot,
1376
+ checkpointNumber: this.checkpointNumber,
941
1377
  blockNumber,
942
- numTxs,
1378
+ numTxs: buildResult.processedCount,
943
1379
  indexWithinCheckpoint,
944
- minValidTxs,
945
- buildResult: buildResult.status
1380
+ minValidTxs
946
1381
  });
947
1382
  this.eventEmitter.emit('block-build-failed', {
948
1383
  reason: `Insufficient valid txs`,
949
- slot: this.slot
1384
+ slot: this.targetSlot
950
1385
  });
951
1386
  this.metrics.recordBlockProposalFailed('insufficient_valid_txs');
952
- return undefined;
1387
+ return {
1388
+ failure: 'insufficient-valid-txs'
1389
+ };
953
1390
  }
954
1391
  // Block creation succeeded, emit stats and metrics
955
- const { block, publicProcessorDuration, usedTxs, blockBuildDuration } = buildResult;
1392
+ const { block, publicProcessorDuration, usedTxs, blockBuildDuration, numTxs } = buildResult;
956
1393
  const blockStats = {
957
1394
  eventName: 'l2-block-built',
958
1395
  duration: blockBuildDuration,
@@ -962,17 +1399,23 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
962
1399
  const blockHash = await block.hash();
963
1400
  const txHashes = block.body.txEffects.map((tx)=>tx.txHash);
964
1401
  const manaPerSec = block.header.totalManaUsed.toNumberUnsafe() / (blockBuildDuration / 1000);
965
- this.log.info(`Built block ${block.number} at checkpoint ${this.checkpointNumber} for slot ${this.slot} with ${numTxs} txs`, {
1402
+ this.log.info(`Built block ${block.number} at checkpoint ${this.checkpointNumber} for slot ${this.targetSlot} with ${numTxs} txs`, {
966
1403
  blockHash,
967
1404
  txHashes,
968
1405
  manaPerSec,
969
1406
  ...blockStats
970
1407
  });
1408
+ // `slot` is the target/submission slot (may be one ahead when pipelining),
1409
+ // `buildSlot` is the wall-clock slot during which the block was actually built.
971
1410
  this.eventEmitter.emit('block-proposed', {
972
1411
  blockNumber: block.number,
973
- slot: this.slot
1412
+ blockHash,
1413
+ checkpointNumber: this.checkpointNumber,
1414
+ indexWithinCheckpoint: block.indexWithinCheckpoint,
1415
+ slot: this.targetSlot,
1416
+ buildSlot: this.slotNow
974
1417
  });
975
- this.metrics.recordBuiltBlock(blockBuildDuration, block.header.totalManaUsed.toNumberUnsafe());
1418
+ this.metrics.recordBuiltBlock(blockBuildDuration, block.header.totalManaUsed.toNumberUnsafe(), this.targetSlot);
976
1419
  return {
977
1420
  block,
978
1421
  usedTxs
@@ -980,11 +1423,19 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
980
1423
  } catch (err) {
981
1424
  this.eventEmitter.emit('block-build-failed', {
982
1425
  reason: err.message,
983
- slot: this.slot
1426
+ slot: this.targetSlot
1427
+ });
1428
+ this.logCheckpointEvent('block-build-failed', `Block build failed for slot ${this.targetSlot}`, {
1429
+ reason: err instanceof Error ? err.message : String(err),
1430
+ slot: this.targetSlot,
1431
+ checkpointNumber: this.checkpointNumber,
1432
+ blockNumber
984
1433
  });
985
1434
  this.log.error(`Error building block`, err, {
986
- blockNumber,
987
- slot: this.slot
1435
+ reason: err instanceof Error ? err.message : String(err),
1436
+ slot: this.targetSlot,
1437
+ checkpointNumber: this.checkpointNumber,
1438
+ blockNumber
988
1439
  });
989
1440
  this.metrics.recordBlockProposalFailed(err.name || 'unknown_error');
990
1441
  this.metrics.recordFailedBlock();
@@ -993,7 +1444,7 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
993
1444
  };
994
1445
  }
995
1446
  }
996
- /** Uses the checkpoint builder to build a block, catching specific txs */ async buildSingleBlockWithCheckpointBuilder(checkpointBuilder, pendingTxs, blockNumber, blockTimestamp, blockBuilderOptions) {
1447
+ /** Uses the checkpoint builder to build a block, catching InsufficientValidTxsError. */ async buildSingleBlockWithCheckpointBuilder(checkpointBuilder, pendingTxs, blockNumber, blockTimestamp, blockBuilderOptions) {
997
1448
  try {
998
1449
  const workTimer = new Timer();
999
1450
  const result = await checkpointBuilder.buildBlock(pendingTxs, blockNumber, blockTimestamp, blockBuilderOptions);
@@ -1004,10 +1455,11 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1004
1455
  status: 'success'
1005
1456
  };
1006
1457
  } catch (err) {
1007
- if (isErrorClass(err, NoValidTxsError)) {
1458
+ if (isErrorClass(err, InsufficientValidTxsError)) {
1008
1459
  return {
1009
1460
  failedTxs: err.failedTxs,
1010
- status: 'no-valid-txs'
1461
+ processedCount: err.processedCount,
1462
+ status: 'insufficient-valid-txs'
1011
1463
  };
1012
1464
  }
1013
1465
  throw err;
@@ -1031,10 +1483,10 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1031
1483
  };
1032
1484
  }
1033
1485
  // Wait a bit before checking again
1034
- this.setStateFn(SequencerState.WAITING_FOR_TXS, this.slot);
1035
- this.log.verbose(`Waiting for enough txs to build block ${blockNumber} at index ${indexWithinCheckpoint} in slot ${this.slot} (have ${availableTxs} but need ${minTxs})`, {
1486
+ this.setStateFn(SequencerState.WAITING_FOR_TXS, this.targetSlot, this.slotNow);
1487
+ this.log.verbose(`Waiting for enough txs to build block ${blockNumber} at index ${indexWithinCheckpoint} in slot ${this.targetSlot} (have ${availableTxs} but need ${minTxs})`, {
1036
1488
  blockNumber,
1037
- slot: this.slot,
1489
+ slot: this.targetSlot,
1038
1490
  indexWithinCheckpoint
1039
1491
  });
1040
1492
  await this.waitForTxsPollingInterval();
@@ -1046,13 +1498,37 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1046
1498
  minTxs
1047
1499
  };
1048
1500
  }
1501
+ async getSignedCommitteeAttestations(broadcast) {
1502
+ const { proposal, blockProposedAt } = broadcast;
1503
+ this.setStateFn(SequencerState.COLLECTING_ATTESTATIONS, this.targetSlot, this.slotNow);
1504
+ const attestations = await this.waitForAttestations(proposal);
1505
+ if (!attestations) {
1506
+ return undefined;
1507
+ }
1508
+ this.checkpointMetrics.recordCheckpointAttestationDelay(this.dateProvider.now() - blockProposedAt);
1509
+ // Proposer must sign over the attestations before pushing them to L1
1510
+ const signer = this.proposer ?? this.publisher.getSenderAddress();
1511
+ try {
1512
+ const attestationsSignature = await this.validatorClient.signAttestationsAndSigners(attestations, signer, this.targetSlot, this.checkpointNumber);
1513
+ return {
1514
+ attestations,
1515
+ attestationsSignature
1516
+ };
1517
+ } catch (err) {
1518
+ if (this.handleHASigningError(err, 'Attestations signature')) {
1519
+ return;
1520
+ }
1521
+ this.log.error(`Error signing attestations for checkpoint proposal at slot ${proposal.slotNumber}`, err);
1522
+ return undefined;
1523
+ }
1524
+ }
1049
1525
  /**
1050
1526
  * Waits for enough attestations to be collected via p2p.
1051
1527
  * This is run after all blocks for the checkpoint have been built.
1052
1528
  */ async waitForAttestations(proposal) {
1053
1529
  if (this.config.fishermanMode) {
1054
1530
  this.log.debug('Skipping attestation collection in fisherman mode');
1055
- return CommitteeAttestationsAndSigners.empty();
1531
+ return CommitteeAttestationsAndSigners.empty(this.getSignatureContext());
1056
1532
  }
1057
1533
  const slotNumber = proposal.slotNumber;
1058
1534
  const { committee, seed, epoch } = await this.epochCache.getCommittee(slotNumber);
@@ -1060,25 +1536,28 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1060
1536
  throw new Error('No committee when collecting attestations');
1061
1537
  } else if (committee.length === 0) {
1062
1538
  this.log.verbose(`Attesting committee is empty`);
1063
- return CommitteeAttestationsAndSigners.empty();
1539
+ return CommitteeAttestationsAndSigners.empty(this.getSignatureContext());
1064
1540
  } else {
1065
1541
  this.log.debug(`Attesting committee length is ${committee.length}`, {
1066
1542
  committee
1067
1543
  });
1068
1544
  }
1069
- const numberOfRequiredAttestations = Math.floor(committee.length * 2 / 3) + 1;
1545
+ const numberOfRequiredAttestations = computeQuorum(committee.length);
1070
1546
  if (this.config.skipCollectingAttestations) {
1071
1547
  this.log.warn('Skipping attestation collection as per config (attesting with own keys only)');
1072
- const attestations = await this.validatorClient?.collectOwnAttestations(proposal);
1073
- return new CommitteeAttestationsAndSigners(orderAttestations(attestations ?? [], committee));
1548
+ const attestations = await this.validatorClient?.collectOwnAttestations(proposal, this.checkpointNumber);
1549
+ this.logCheckpointAttestations('collected', committee, attestations ?? [], numberOfRequiredAttestations, {
1550
+ reason: 'collect_own_only'
1551
+ });
1552
+ return new CommitteeAttestationsAndSigners(orderAttestations(attestations ?? [], committee), this.getSignatureContext());
1074
1553
  }
1075
- const attestationTimeAllowed = this.config.enforceTimeTable ? this.timetable.getMaxAllowedTime(SequencerState.PUBLISHING_CHECKPOINT) : this.l1Constants.slotDuration;
1554
+ const attestationTimeAllowed = this.config.enforceTimeTable ? this.timetable.getCheckpointAttestationDeadline() : this.l1Constants.slotDuration;
1076
1555
  const attestationDeadline = new Date((this.getSlotStartBuildTimestamp() + attestationTimeAllowed) * 1000);
1077
1556
  this.metrics.recordRequiredAttestations(numberOfRequiredAttestations, attestationTimeAllowed);
1078
1557
  const collectAttestationsTimer = new Timer();
1079
1558
  let collectedAttestationsCount = 0;
1080
1559
  try {
1081
- const attestations = await this.validatorClient.collectAttestations(proposal, numberOfRequiredAttestations, attestationDeadline);
1560
+ const attestations = await this.validatorClient.collectAttestations(proposal, numberOfRequiredAttestations, attestationDeadline, this.checkpointNumber);
1082
1561
  collectedAttestationsCount = attestations.length;
1083
1562
  // Trim attestations to minimum required to save L1 calldata gas
1084
1563
  const localAddresses = this.validatorClient.getValidatorAddresses();
@@ -1088,20 +1567,55 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1088
1567
  }
1089
1568
  // Rollup contract requires that the signatures are provided in the order of the committee
1090
1569
  const sorted = orderAttestations(trimmed, committee);
1570
+ this.logCheckpointAttestations('collected', committee, attestations, numberOfRequiredAttestations, {
1571
+ submittedCount: trimmed.length
1572
+ });
1091
1573
  // Manipulate the attestations if we've been configured to do so
1092
1574
  if (this.config.injectFakeAttestation || this.config.injectHighSValueAttestation || this.config.injectUnrecoverableSignatureAttestation || this.config.shuffleAttestationOrdering) {
1093
1575
  return this.manipulateAttestations(proposal.slotNumber, epoch, seed, committee, sorted);
1094
1576
  }
1095
- return new CommitteeAttestationsAndSigners(sorted);
1577
+ return new CommitteeAttestationsAndSigners(sorted, this.getSignatureContext());
1096
1578
  } catch (err) {
1097
1579
  if (err && err instanceof AttestationTimeoutError) {
1098
1580
  collectedAttestationsCount = err.collectedCount;
1581
+ this.logCheckpointAttestations('failed', committee, undefined, numberOfRequiredAttestations, {
1582
+ collectedCount: collectedAttestationsCount,
1583
+ reason: 'timeout'
1584
+ });
1585
+ this.log.error(`Timeout while waiting for attestations for checkpoint proposal at slot ${proposal.slotNumber} (collected ${collectedAttestationsCount}/${numberOfRequiredAttestations})`, err);
1586
+ } else {
1587
+ this.logCheckpointAttestations('failed', committee, undefined, numberOfRequiredAttestations, {
1588
+ collectedCount: collectedAttestationsCount,
1589
+ reason: err instanceof Error ? err.message : String(err)
1590
+ });
1591
+ this.log.error(`Error collecting attestations for checkpoint proposal at slot ${proposal.slotNumber}`, err);
1099
1592
  }
1100
- throw err;
1593
+ return undefined;
1101
1594
  } finally{
1102
1595
  this.metrics.recordCollectedAttestations(collectedAttestationsCount, collectAttestationsTimer.ms());
1103
1596
  }
1104
1597
  }
1598
+ logCheckpointAttestations(status, committee, attestations, requiredAttestations, opts = {}) {
1599
+ const signedValidators = attestations?.map((attestation)=>attestation.getSender()?.toString()).filter((address)=>address !== undefined) ?? [];
1600
+ const collectedCount = opts.collectedCount ?? new Set(signedValidators).size;
1601
+ const missingValidatorCount = status === 'failed' ? Math.max(0, requiredAttestations - collectedCount) : undefined;
1602
+ this.logCheckpointEvent(`attestations-${status}`, `Checkpoint attestations ${status} for slot ${this.targetSlot}`, {
1603
+ slot: this.targetSlot,
1604
+ checkpointNumber: this.checkpointNumber,
1605
+ committeeSize: committee.length,
1606
+ requiredAttestations,
1607
+ collectedAttestations: collectedCount,
1608
+ ...opts.submittedCount !== undefined && {
1609
+ submittedAttestations: opts.submittedCount
1610
+ },
1611
+ ...missingValidatorCount !== undefined && {
1612
+ missingValidatorCount
1613
+ },
1614
+ ...opts.reason !== undefined && {
1615
+ reason: opts.reason
1616
+ }
1617
+ });
1618
+ }
1105
1619
  /** Breaks the attestations before publishing based on attack configs */ manipulateAttestations(slotNumber, epoch, seed, committee, attestations) {
1106
1620
  // Compute the proposer index in the committee, since we dont want to tweak it.
1107
1621
  // Otherwise, the L1 rollup contract will reject the block outright.
@@ -1127,7 +1641,7 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1127
1641
  unfreeze(attestations[targetIndex]).signature = generateRecoverableSignature();
1128
1642
  }
1129
1643
  }
1130
- return new CommitteeAttestationsAndSigners(attestations);
1644
+ return new CommitteeAttestationsAndSigners(attestations, this.getSignatureContext());
1131
1645
  }
1132
1646
  if (this.config.shuffleAttestationOrdering) {
1133
1647
  this.log.warn(`Shuffling attestation ordering in checkpoint for slot ${slotNumber} (proposer #${proposerIndex})`);
@@ -1153,10 +1667,10 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1153
1667
  shuffled[i]
1154
1668
  ];
1155
1669
  }
1156
- const signers = new CommitteeAttestationsAndSigners(attestations).getSigners();
1157
- return new MaliciousCommitteeAttestationsAndSigners(shuffled, signers);
1670
+ const signers = new CommitteeAttestationsAndSigners(attestations, this.getSignatureContext()).getSigners();
1671
+ return new MaliciousCommitteeAttestationsAndSigners(shuffled, signers, this.getSignatureContext());
1158
1672
  }
1159
- return new CommitteeAttestationsAndSigners(attestations);
1673
+ return new CommitteeAttestationsAndSigners(attestations, this.getSignatureContext());
1160
1674
  }
1161
1675
  async dropFailedTxsFromP2P(failedTxs) {
1162
1676
  if (failedTxs.length === 0) {
@@ -1171,8 +1685,12 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1171
1685
  * Adds the proposed block to the archiver so it's available via P2P.
1172
1686
  * Gossip doesn't echo messages back to the sender, so the proposer's archiver/world-state
1173
1687
  * would never receive its own block without this explicit sync.
1688
+ *
1689
+ * In fisherman mode we skip this push: the fisherman builds blocks locally for validation
1690
+ * and fee analysis only, and pushing them to the archiver causes spurious reorg cascades
1691
+ * whenever the real proposer's block arrives from L1.
1174
1692
  */ async syncProposedBlockToArchiver(block) {
1175
- if (this.config.skipPushProposedBlocksToArchiver !== false) {
1693
+ if (this.config.skipPushProposedBlocksToArchiver || this.config.fishermanMode) {
1176
1694
  this.log.warn(`Skipping push of proposed block ${block.number} to archiver`, {
1177
1695
  blockNumber: block.number,
1178
1696
  slot: block.header.globalVariables.slotNumber
@@ -1185,19 +1703,48 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1185
1703
  });
1186
1704
  await this.blockSink.addBlock(block);
1187
1705
  }
1706
+ /**
1707
+ * Adds the proposed checkpoint to the archiver so the proposer's optimistic proposed-checkpoint
1708
+ * tip advances locally. Gossip doesn't echo our own messages back, so without this the proposer
1709
+ * would never see its own proposed checkpoint and couldn't pipeline the next slot.
1710
+ *
1711
+ * Skipped whenever proposed blocks aren't pushed (`skipPushProposedBlocksToArchiver`, fisherman
1712
+ * mode): the archiver derives the checkpoint archive from its stored blocks, so without them the
1713
+ * push would fail. All blocks were already added (and awaited) during block building, so this
1714
+ * needs no retry — they are guaranteed present by the time we get here.
1715
+ */ async syncProposedCheckpointToArchiver(checkpoint, blockCount, feeAssetPriceModifier) {
1716
+ if (this.config.skipPushProposedBlocksToArchiver || this.config.fishermanMode) {
1717
+ return;
1718
+ }
1719
+ const startBlock = BlockNumber(this.syncedToBlockNumber + 1);
1720
+ this.log.debug(`Syncing proposed checkpoint ${this.checkpointNumber} to archiver`, {
1721
+ checkpointNumber: this.checkpointNumber,
1722
+ slot: this.targetSlot,
1723
+ startBlock,
1724
+ blockCount
1725
+ });
1726
+ await this.blockSink.addProposedCheckpoint({
1727
+ header: checkpoint.header,
1728
+ checkpointNumber: this.checkpointNumber,
1729
+ startBlock,
1730
+ blockCount,
1731
+ totalManaUsed: checkpoint.header.totalManaUsed.toBigInt(),
1732
+ feeAssetPriceModifier
1733
+ });
1734
+ }
1188
1735
  /** Runs fee analysis and logs checkpoint outcome as fisherman */ async handleCheckpointEndAsFisherman(checkpoint) {
1189
1736
  // Perform L1 fee analysis before clearing requests
1190
1737
  // The callback is invoked asynchronously after the next block is mined
1191
- const feeAnalysis = await this.publisher.analyzeL1Fees(this.slot, (analysis)=>this.metrics.recordFishermanFeeAnalysis(analysis));
1738
+ const feeAnalysis = await this.publisher.analyzeL1Fees(this.targetSlot, (analysis)=>this.metrics.recordFishermanFeeAnalysis(analysis));
1192
1739
  if (checkpoint) {
1193
- this.log.info(`Validation checkpoint building SUCCEEDED for slot ${this.slot}`, {
1740
+ this.log.info(`Validation checkpoint building SUCCEEDED for slot ${this.targetSlot}`, {
1194
1741
  ...checkpoint.toCheckpointInfo(),
1195
1742
  ...checkpoint.getStats(),
1196
1743
  feeAnalysisId: feeAnalysis?.id
1197
1744
  });
1198
1745
  } else {
1199
- this.log.warn(`Validation block building FAILED for slot ${this.slot}`, {
1200
- slot: this.slot,
1746
+ this.log.warn(`Validation block building FAILED for slot ${this.targetSlot}`, {
1747
+ slot: this.targetSlot,
1201
1748
  feeAnalysisId: feeAnalysis?.id
1202
1749
  });
1203
1750
  this.metrics.recordCheckpointProposalFailed('block_build_failed');
@@ -1208,15 +1755,15 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1208
1755
  * Helper to handle HA double-signing errors. Returns true if the error was handled (caller should yield).
1209
1756
  */ handleHASigningError(err, errorContext) {
1210
1757
  if (err instanceof DutyAlreadySignedError) {
1211
- this.log.info(`${errorContext} for slot ${this.slot} already signed by another HA node, yielding`, {
1212
- slot: this.slot,
1758
+ this.log.info(`${errorContext} for slot ${this.targetSlot} already signed by another HA node, yielding`, {
1759
+ slot: this.targetSlot,
1213
1760
  signedByNode: err.signedByNode
1214
1761
  });
1215
1762
  return true;
1216
1763
  }
1217
1764
  if (err instanceof SlashingProtectionError) {
1218
- this.log.info(`${errorContext} for slot ${this.slot} blocked by slashing protection, yielding`, {
1219
- slot: this.slot,
1765
+ this.log.info(`${errorContext} for slot ${this.targetSlot} blocked by slashing protection, yielding`, {
1766
+ slot: this.targetSlot,
1220
1767
  existingMessageHash: err.existingMessageHash,
1221
1768
  attemptedMessageHash: err.attemptedMessageHash
1222
1769
  });
@@ -1227,13 +1774,14 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1227
1774
  /** Waits until a specific time within the current slot */ async waitUntilTimeInSlot(targetSecondsIntoSlot) {
1228
1775
  const slotStartTimestamp = this.getSlotStartBuildTimestamp();
1229
1776
  const targetTimestamp = slotStartTimestamp + targetSecondsIntoSlot;
1230
- await sleepUntil(new Date(targetTimestamp * 1000), this.dateProvider.nowAsDate());
1777
+ const delayMs = targetTimestamp * 1000 - this.dateProvider.nowAsDate().getTime();
1778
+ await this.awaitInterruptibleSleep(delayMs);
1231
1779
  }
1232
- /** Waits the polling interval for transactions. Extracted for test overriding. */ async waitForTxsPollingInterval() {
1233
- await sleep(TXS_POLLING_MS);
1780
+ /** Pause between mempool polls in waitForMinTxs. Extracted for test overriding. */ async waitForTxsPollingInterval() {
1781
+ await this.awaitInterruptibleSleep(TXS_POLLING_MS);
1234
1782
  }
1235
1783
  getSlotStartBuildTimestamp() {
1236
- return getSlotStartBuildTimestamp(this.slot, this.l1Constants);
1784
+ return getSlotStartBuildTimestamp(this.slotNow, this.l1Constants);
1237
1785
  }
1238
1786
  getSecondsIntoSlot() {
1239
1787
  const slotStartTimestamp = this.getSlotStartBuildTimestamp();