@aztec/sequencer-client 0.0.1-commit.001888fc → 0.0.1-commit.017a351

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