@aztec/sequencer-client 0.0.1-commit.2448fdb → 0.0.1-commit.2606882

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 (118) hide show
  1. package/README.md +281 -21
  2. package/dest/client/sequencer-client.d.ts +4 -3
  3. package/dest/client/sequencer-client.d.ts.map +1 -1
  4. package/dest/client/sequencer-client.js +11 -11
  5. package/dest/config.d.ts +4 -1
  6. package/dest/config.d.ts.map +1 -1
  7. package/dest/config.js +35 -14
  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 +4 -10
  15. package/dest/global_variable_builder/global_builder.d.ts.map +1 -1
  16. package/dest/global_variable_builder/global_builder.js +5 -40
  17. package/dest/global_variable_builder/index.d.ts +3 -1
  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 +19 -3
  24. package/dest/publisher/config.d.ts.map +1 -1
  25. package/dest/publisher/config.js +25 -5
  26. package/dest/publisher/index.d.ts +2 -1
  27. package/dest/publisher/index.d.ts.map +1 -1
  28. package/dest/publisher/l1_tx_failed_store/factory.d.ts +11 -0
  29. package/dest/publisher/l1_tx_failed_store/factory.d.ts.map +1 -0
  30. package/dest/publisher/l1_tx_failed_store/factory.js +22 -0
  31. package/dest/publisher/l1_tx_failed_store/failed_tx_store.d.ts +58 -0
  32. package/dest/publisher/l1_tx_failed_store/failed_tx_store.d.ts.map +1 -0
  33. package/dest/publisher/l1_tx_failed_store/failed_tx_store.js +1 -0
  34. package/dest/publisher/l1_tx_failed_store/file_store_failed_tx_store.d.ts +15 -0
  35. package/dest/publisher/l1_tx_failed_store/file_store_failed_tx_store.d.ts.map +1 -0
  36. package/dest/publisher/l1_tx_failed_store/file_store_failed_tx_store.js +34 -0
  37. package/dest/publisher/l1_tx_failed_store/index.d.ts +4 -0
  38. package/dest/publisher/l1_tx_failed_store/index.d.ts.map +1 -0
  39. package/dest/publisher/l1_tx_failed_store/index.js +2 -0
  40. package/dest/publisher/sequencer-bundle-simulator.d.ts +96 -0
  41. package/dest/publisher/sequencer-bundle-simulator.d.ts.map +1 -0
  42. package/dest/publisher/sequencer-bundle-simulator.js +198 -0
  43. package/dest/publisher/sequencer-publisher-factory.d.ts +3 -5
  44. package/dest/publisher/sequencer-publisher-factory.d.ts.map +1 -1
  45. package/dest/publisher/sequencer-publisher-factory.js +16 -3
  46. package/dest/publisher/sequencer-publisher.d.ts +77 -66
  47. package/dest/publisher/sequencer-publisher.d.ts.map +1 -1
  48. package/dest/publisher/sequencer-publisher.js +410 -398
  49. package/dest/sequencer/automine/automine_factory.d.ts +54 -0
  50. package/dest/sequencer/automine/automine_factory.d.ts.map +1 -0
  51. package/dest/sequencer/automine/automine_factory.js +84 -0
  52. package/dest/sequencer/automine/automine_sequencer.d.ts +151 -0
  53. package/dest/sequencer/automine/automine_sequencer.d.ts.map +1 -0
  54. package/dest/sequencer/automine/automine_sequencer.js +509 -0
  55. package/dest/sequencer/chain_state_overrides.d.ts +61 -0
  56. package/dest/sequencer/chain_state_overrides.d.ts.map +1 -0
  57. package/dest/sequencer/chain_state_overrides.js +98 -0
  58. package/dest/sequencer/checkpoint_proposal_job.d.ts +58 -10
  59. package/dest/sequencer/checkpoint_proposal_job.d.ts.map +1 -1
  60. package/dest/sequencer/checkpoint_proposal_job.js +736 -191
  61. package/dest/sequencer/checkpoint_proposal_job_metrics.d.ts +34 -0
  62. package/dest/sequencer/checkpoint_proposal_job_metrics.d.ts.map +1 -0
  63. package/dest/sequencer/checkpoint_proposal_job_metrics.js +72 -0
  64. package/dest/sequencer/events.d.ts +48 -2
  65. package/dest/sequencer/events.d.ts.map +1 -1
  66. package/dest/sequencer/index.d.ts +3 -1
  67. package/dest/sequencer/index.d.ts.map +1 -1
  68. package/dest/sequencer/index.js +2 -0
  69. package/dest/sequencer/metrics.d.ts +13 -10
  70. package/dest/sequencer/metrics.d.ts.map +1 -1
  71. package/dest/sequencer/metrics.js +45 -20
  72. package/dest/sequencer/sequencer.d.ts +59 -12
  73. package/dest/sequencer/sequencer.d.ts.map +1 -1
  74. package/dest/sequencer/sequencer.js +344 -105
  75. package/dest/sequencer/timetable.d.ts +11 -1
  76. package/dest/sequencer/timetable.d.ts.map +1 -1
  77. package/dest/sequencer/timetable.js +41 -36
  78. package/dest/sequencer/types.d.ts +2 -2
  79. package/dest/sequencer/types.d.ts.map +1 -1
  80. package/dest/test/mock_checkpoint_builder.d.ts +1 -1
  81. package/dest/test/mock_checkpoint_builder.d.ts.map +1 -1
  82. package/dest/test/mock_checkpoint_builder.js +0 -2
  83. package/dest/test/utils.d.ts +1 -1
  84. package/dest/test/utils.d.ts.map +1 -1
  85. package/dest/test/utils.js +7 -6
  86. package/package.json +27 -28
  87. package/src/client/sequencer-client.ts +19 -20
  88. package/src/config.ts +40 -11
  89. package/src/global_variable_builder/README.md +44 -0
  90. package/src/global_variable_builder/fee_predictor.ts +172 -0
  91. package/src/global_variable_builder/fee_provider.ts +75 -0
  92. package/src/global_variable_builder/global_builder.ts +10 -46
  93. package/src/global_variable_builder/index.ts +2 -0
  94. package/src/index.ts +10 -1
  95. package/src/publisher/config.ts +49 -6
  96. package/src/publisher/index.ts +3 -0
  97. package/src/publisher/l1_tx_failed_store/factory.ts +32 -0
  98. package/src/publisher/l1_tx_failed_store/failed_tx_store.ts +57 -0
  99. package/src/publisher/l1_tx_failed_store/file_store_failed_tx_store.ts +46 -0
  100. package/src/publisher/l1_tx_failed_store/index.ts +3 -0
  101. package/src/publisher/sequencer-bundle-simulator.ts +254 -0
  102. package/src/publisher/sequencer-publisher-factory.ts +18 -6
  103. package/src/publisher/sequencer-publisher.ts +485 -474
  104. package/src/sequencer/README.md +162 -450
  105. package/src/sequencer/automine/README.md +46 -0
  106. package/src/sequencer/automine/automine_factory.ts +145 -0
  107. package/src/sequencer/automine/automine_sequencer.ts +595 -0
  108. package/src/sequencer/chain_state_overrides.ts +162 -0
  109. package/src/sequencer/checkpoint_proposal_job.ts +857 -201
  110. package/src/sequencer/checkpoint_proposal_job_metrics.ts +128 -0
  111. package/src/sequencer/events.ts +51 -2
  112. package/src/sequencer/index.ts +2 -0
  113. package/src/sequencer/metrics.ts +57 -24
  114. package/src/sequencer/sequencer.ts +421 -118
  115. package/src/sequencer/timetable.ts +49 -44
  116. package/src/sequencer/types.ts +1 -1
  117. package/src/test/mock_checkpoint_builder.ts +0 -2
  118. 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,26 +1020,42 @@ _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
1060
  // Final validation: per-block limits are only checked if the operator set them explicitly.
699
1061
  // Otherwise, checkpoint-level budgets were already enforced by the redistribution logic.
@@ -706,68 +1068,95 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
706
1068
  maxTxsPerCheckpoint: this.config.maxTxsPerCheckpoint
707
1069
  });
708
1070
  } catch (err) {
709
- 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',
710
1083
  checkpoint: checkpoint.header.toInspect()
711
1084
  });
712
1085
  return undefined;
713
1086
  }
714
1087
  // Record checkpoint-level build metrics
715
- this.metrics.recordCheckpointBuild(checkpointBuildTimer.ms(), blocksInCheckpoint.length, checkpoint.getStats().txCount, Number(checkpoint.header.totalManaUsed.toBigInt()));
716
- // 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
717
1101
  if (this.config.fishermanMode) {
718
- this.log.info(`Built checkpoint for slot ${this.slot} with ${blocksInCheckpoint.length} blocks. ` + `Skipping proposal in fisherman mode.`, {
719
- 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,
720
1104
  checkpoint: checkpoint.header.toInspect(),
721
1105
  blocksBuilt: blocksInCheckpoint.length
722
1106
  });
723
1107
  this.metrics.recordCheckpointSuccess();
724
- 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
+ };
725
1114
  }
726
- // Include the block pending broadcast in the checkpoint proposal if any
727
- const lastBlock = blockPendingBroadcast && {
728
- blockHeader: blockPendingBroadcast.block.header,
729
- indexWithinCheckpoint: blockPendingBroadcast.block.indexWithinCheckpoint,
730
- txs: blockPendingBroadcast.txs
731
- };
732
- // Create the checkpoint proposal and broadcast it
733
- const proposal = await this.validatorClient.createCheckpointProposal(checkpoint.header, checkpoint.archive.root, feeAssetPriceModifier, lastBlock, this.proposer, checkpointProposalOptions);
734
- const blockProposedAt = this.dateProvider.now();
735
- await this.p2pClient.broadcastCheckpointProposal(proposal);
736
- this.setStateFn(SequencerState.COLLECTING_ATTESTATIONS, this.slot);
737
- const attestations = await this.waitForAttestations(proposal);
738
- const blockAttestedAt = this.dateProvider.now();
739
- this.metrics.recordCheckpointAttestationDelay(blockAttestedAt - blockProposedAt);
740
- // Proposer must sign over the attestations before pushing them to L1
741
- const signer = this.proposer ?? this.publisher.getSenderAddress();
742
- 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.
743
1118
  try {
744
- attestationsSignature = await this.validatorClient.signAttestationsAndSigners(attestations, signer, this.slot, this.checkpointNumber);
1119
+ await this.publisher.validateBlockHeader(checkpoint.header, this.checkpointSimulationOverridesPlan);
745
1120
  } catch (err) {
746
- // We shouldn't really get here since we yield to another HA node
747
- // as soon as we see these errors when creating block or checkpoint proposals.
748
- if (this.handleHASigningError(err, 'Attestations signature')) {
749
- return undefined;
750
- }
751
- 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;
752
1132
  }
753
- // Enqueue publishing the checkpoint to L1
754
- this.setStateFn(SequencerState.PUBLISHING_CHECKPOINT, this.slot);
755
- const aztecSlotDuration = this.l1Constants.slotDuration;
756
- const slotStartBuildTimestamp = this.getSlotStartBuildTimestamp();
757
- const txTimeoutAt = new Date((slotStartBuildTimestamp + aztecSlotDuration) * 1000);
758
- // If we have been configured to potentially skip publishing checkpoint then roll the dice here
759
- if (this.config.skipPublishingCheckpointsPercent !== undefined && this.config.skipPublishingCheckpointsPercent > 0) {
760
- const result = Math.max(0, randomInt(100));
761
- if (result < this.config.skipPublishingCheckpointsPercent) {
762
- this.log.warn(`Skipping publishing proposal for checkpoint ${checkpoint.number}. Configured percentage: ${this.config.skipPublishingCheckpointsPercent}, generated value: ${result}`);
763
- 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);
764
1149
  }
1150
+ } else if (!this.config.skipBroadcastProposals) {
1151
+ await this.p2pClient.broadcastCheckpointProposal(proposal);
1152
+ this.checkpointMetrics.noteCheckpointBroadcast(this.dateProvider.now());
765
1153
  }
766
- await this.publisher.enqueueProposeCheckpoint(checkpoint, attestations, attestationsSignature, {
767
- txTimeoutAt,
768
- forcePendingCheckpointNumber: this.invalidateCheckpoint?.forcePendingCheckpointNumber
769
- });
770
- return checkpoint;
1154
+ // Return immediately after broadcast — attestation collection happens in the background
1155
+ return {
1156
+ checkpoint,
1157
+ proposal,
1158
+ blockProposedAt
1159
+ };
771
1160
  } catch (e) {
772
1161
  env.error = e;
773
1162
  env.hasError = true;
@@ -780,7 +1169,7 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
780
1169
  // swallow this error. It's already been logged by a function deeper in the stack
781
1170
  return undefined;
782
1171
  }
783
- this.log.error(`Error building checkpoint at slot ${this.slot}`, err);
1172
+ this.log.error(`Error building checkpoint at slot ${this.targetSlot}`, err);
784
1173
  return undefined;
785
1174
  }
786
1175
  }
@@ -790,18 +1179,25 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
790
1179
  const blocksInCheckpoint = [];
791
1180
  const txHashesAlreadyIncluded = new Set();
792
1181
  const initialBlockNumber = BlockNumber(this.syncedToBlockNumber + 1);
793
- const slot = this.slot;
794
1182
  // Last block in the checkpoint will usually be flagged as pending broadcast, so we send it along with the checkpoint proposal
795
1183
  let blockPendingBroadcast = undefined;
796
1184
  while(true){
797
1185
  const blocksBuilt = blocksInCheckpoint.length;
798
1186
  const indexWithinCheckpoint = IndexWithinCheckpoint(blocksBuilt);
799
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
+ }
800
1196
  const secondsIntoSlot = this.getSecondsIntoSlot();
801
1197
  const timingInfo = this.timetable.canStartNextBlock(secondsIntoSlot);
802
1198
  if (!timingInfo.canStart) {
803
1199
  this.log.debug(`Not enough time left in slot to start another block`, {
804
- slot,
1200
+ slot: this.targetSlot,
805
1201
  blocksBuilt,
806
1202
  secondsIntoSlot
807
1203
  });
@@ -818,20 +1214,22 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
818
1214
  indexWithinCheckpoint,
819
1215
  txHashesAlreadyIncluded
820
1216
  });
821
- // TODO(palla/mbps): Review these conditions. We may want to keep trying in some scenarios.
822
- if (!buildResult && timingInfo.isLastBlock) {
823
- break;
824
- } else if (!buildResult && timingInfo.deadline !== undefined) {
825
- // 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
826
1224
  await this.waitUntilNextSubslot(timingInfo.deadline);
827
1225
  continue;
828
- } else if (!buildResult) {
829
- break;
830
- } else if ('error' in buildResult) {
831
- // 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) {
832
1230
  if (!(buildResult.error instanceof SequencerInterruptedError)) {
833
- this.log.warn(`Halting block building for slot ${slot}`, {
834
- slot,
1231
+ this.log.warn(`Halting block building for slot ${this.targetSlot}`, {
1232
+ slot: this.targetSlot,
835
1233
  blocksBuilt,
836
1234
  error: buildResult.error
837
1235
  });
@@ -839,39 +1237,41 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
839
1237
  break;
840
1238
  }
841
1239
  const { block, usedTxs } = buildResult;
1240
+ this.checkpointMetrics.noteCheckpointBlockBuilt(this.dateProvider.now(), {
1241
+ isFirstBlock: blocksBuilt === 0,
1242
+ isLastBlock: timingInfo.isLastBlock
1243
+ });
842
1244
  blocksInCheckpoint.push(block);
843
1245
  usedTxs.forEach((tx)=>txHashesAlreadyIncluded.add(tx.txHash.toString()));
844
- // If this is the last block, send the proposed block to the archiver,
845
- // and exit the loop now so we can build the checkpoint and 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.
846
1257
  if (timingInfo.isLastBlock) {
847
- await this.syncProposedBlockToArchiver(block);
848
- this.log.verbose(`Completed final block ${blockNumber} for slot ${slot}`, {
849
- 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
- // Broadcast the block proposal (unless we're in fisherman mode) unless the block is the last one,
860
- // in which case we'll broadcast it along with the checkpoint at the end of the loop.
861
- // Note that we only send the block to the archiver if we manage to create the proposal, so if there's
862
- // a HA error we don't pollute our archiver with a block that won't make it to the chain.
863
- const proposal = await this.createBlockProposal(block, inHash, usedTxs, blockProposalOptions);
864
- // Sync the proposed block to the archiver to make it available, only after we've managed to sign the proposal.
865
- // We wait for the sync to succeed, as this helps catch consistency errors, even if it means we lose some time for block-building.
866
- // If this throws, we abort the entire checkpoint.
867
- await this.syncProposedBlockToArchiver(block);
868
1266
  // Once we have a signed proposal and the archiver agreed with our proposed block, then we broadcast it.
869
- proposal && await this.p2pClient.broadcastProposal(proposal);
1267
+ if (proposal && !this.config.skipBroadcastProposals) {
1268
+ await this.p2pClient.broadcastProposal(proposal);
1269
+ }
870
1270
  // Wait until the next block's start time
871
1271
  await this.waitUntilNextSubslot(timingInfo.deadline);
872
1272
  }
873
- this.log.verbose(`Block building loop completed for slot ${this.slot}`, {
874
- slot: this.slot,
1273
+ this.log.verbose(`Block building loop completed for slot ${this.targetSlot}`, {
1274
+ slot: this.targetSlot,
875
1275
  blocksBuilt: blocksInCheckpoint.length
876
1276
  });
877
1277
  return {
@@ -884,18 +1284,18 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
884
1284
  this.log.info(`Skipping block proposal for block ${block.number} in fisherman mode`);
885
1285
  return Promise.resolve(undefined);
886
1286
  }
887
- return this.validatorClient.createBlockProposal(block.header, block.indexWithinCheckpoint, inHash, block.archive.root, usedTxs, this.proposer, blockProposalOptions);
1287
+ return this.validatorClient.createBlockProposal(block.header, this.checkpointNumber, block.indexWithinCheckpoint, inHash, block.archive.root, usedTxs, this.proposer, blockProposalOptions);
888
1288
  }
889
1289
  /** Sleeps until it is time to produce the next block in the slot */ async waitUntilNextSubslot(nextSubslotStart) {
890
- this.setStateFn(SequencerState.WAITING_UNTIL_NEXT_BLOCK, this.slot);
1290
+ this.setStateFn(SequencerState.WAITING_UNTIL_NEXT_BLOCK, this.targetSlot, this.slotNow);
891
1291
  this.log.verbose(`Waiting until time for the next block at ${nextSubslotStart}s into slot`, {
892
- slot: this.slot
1292
+ slot: this.targetSlot
893
1293
  });
894
1294
  await this.waitUntilTimeInSlot(nextSubslotStart);
895
1295
  }
896
1296
  /** Builds a single block. Called from the main block building loop. */ async buildSingleBlock(checkpointBuilder, opts) {
897
1297
  const { blockTimestamp, forceCreate, blockNumber, indexWithinCheckpoint, buildDeadline, txHashesAlreadyIncluded } = opts;
898
- 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}`, {
899
1299
  ...checkpointBuilder.getConstantData(),
900
1300
  ...opts
901
1301
  });
@@ -903,28 +1303,43 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
903
1303
  // Wait until we have enough txs to build the block
904
1304
  const { availableTxs, canStartBuilding, minTxs } = await this.waitForMinTxs(opts);
905
1305
  if (!canStartBuilding) {
906
- 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',
907
1308
  blockNumber,
908
- slot: this.slot,
909
- indexWithinCheckpoint
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',
1317
+ blockNumber,
1318
+ slot: this.targetSlot,
1319
+ checkpointNumber: this.checkpointNumber,
1320
+ indexWithinCheckpoint,
1321
+ availableTxs,
1322
+ minTxs
910
1323
  });
911
1324
  this.eventEmitter.emit('block-tx-count-check-failed', {
912
1325
  minTxs,
913
1326
  availableTxs,
914
- slot: this.slot
1327
+ slot: this.targetSlot
915
1328
  });
916
1329
  this.metrics.recordBlockProposalFailed('insufficient_txs');
917
- return undefined;
1330
+ return {
1331
+ failure: 'insufficient-txs'
1332
+ };
918
1333
  }
919
1334
  // Create iterator to pending txs. We filter out txs already included in previous blocks in the checkpoint
920
1335
  // just in case p2p failed to sync the provisional block and didn't get to remove those txs from the mempool yet.
921
1336
  const pendingTxs = filter(this.p2pClient.iterateEligiblePendingTxs(), (tx)=>!txHashesAlreadyIncluded.has(tx.txHash.toString()));
922
- this.log.debug(`Building block ${blockNumber} at index ${indexWithinCheckpoint} for slot ${this.slot} with ${availableTxs} available txs`, {
923
- 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,
924
1339
  blockNumber,
925
1340
  indexWithinCheckpoint
926
1341
  });
927
- this.setStateFn(SequencerState.CREATING_BLOCK, this.slot);
1342
+ this.setStateFn(SequencerState.CREATING_BLOCK, this.targetSlot, this.slotNow);
928
1343
  // Per-block limits are operator overrides (from SEQ_MAX_L2_BLOCK_GAS etc.) further capped
929
1344
  // by remaining checkpoint-level budgets inside CheckpointBuilder before each block is built.
930
1345
  // minValidTxs is passed into the builder so it can reject the block *before* updating state.
@@ -945,8 +1360,19 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
945
1360
  // If any txs failed during execution, drop them from the mempool so we don't pick them up again
946
1361
  await this.dropFailedTxsFromP2P(buildResult.failedTxs);
947
1362
  if (buildResult.status === 'insufficient-valid-txs') {
948
- this.log.warn(`Block ${blockNumber} at index ${indexWithinCheckpoint} on slot ${this.slot} has too few valid txs to be proposed`, {
949
- 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,
950
1376
  blockNumber,
951
1377
  numTxs: buildResult.processedCount,
952
1378
  indexWithinCheckpoint,
@@ -954,10 +1380,12 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
954
1380
  });
955
1381
  this.eventEmitter.emit('block-build-failed', {
956
1382
  reason: `Insufficient valid txs`,
957
- slot: this.slot
1383
+ slot: this.targetSlot
958
1384
  });
959
1385
  this.metrics.recordBlockProposalFailed('insufficient_valid_txs');
960
- return undefined;
1386
+ return {
1387
+ failure: 'insufficient-valid-txs'
1388
+ };
961
1389
  }
962
1390
  // Block creation succeeded, emit stats and metrics
963
1391
  const { block, publicProcessorDuration, usedTxs, blockBuildDuration, numTxs } = buildResult;
@@ -970,17 +1398,23 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
970
1398
  const blockHash = await block.hash();
971
1399
  const txHashes = block.body.txEffects.map((tx)=>tx.txHash);
972
1400
  const manaPerSec = block.header.totalManaUsed.toNumberUnsafe() / (blockBuildDuration / 1000);
973
- 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`, {
974
1402
  blockHash,
975
1403
  txHashes,
976
1404
  manaPerSec,
977
1405
  ...blockStats
978
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.
979
1409
  this.eventEmitter.emit('block-proposed', {
980
1410
  blockNumber: block.number,
981
- slot: this.slot
1411
+ blockHash,
1412
+ checkpointNumber: this.checkpointNumber,
1413
+ indexWithinCheckpoint: block.indexWithinCheckpoint,
1414
+ slot: this.targetSlot,
1415
+ buildSlot: this.slotNow
982
1416
  });
983
- this.metrics.recordBuiltBlock(blockBuildDuration, block.header.totalManaUsed.toNumberUnsafe());
1417
+ this.metrics.recordBuiltBlock(blockBuildDuration, block.header.totalManaUsed.toNumberUnsafe(), this.targetSlot);
984
1418
  return {
985
1419
  block,
986
1420
  usedTxs
@@ -988,11 +1422,19 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
988
1422
  } catch (err) {
989
1423
  this.eventEmitter.emit('block-build-failed', {
990
1424
  reason: err.message,
991
- 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
992
1432
  });
993
1433
  this.log.error(`Error building block`, err, {
994
- blockNumber,
995
- slot: this.slot
1434
+ reason: err instanceof Error ? err.message : String(err),
1435
+ slot: this.targetSlot,
1436
+ checkpointNumber: this.checkpointNumber,
1437
+ blockNumber
996
1438
  });
997
1439
  this.metrics.recordBlockProposalFailed(err.name || 'unknown_error');
998
1440
  this.metrics.recordFailedBlock();
@@ -1040,10 +1482,10 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1040
1482
  };
1041
1483
  }
1042
1484
  // Wait a bit before checking again
1043
- this.setStateFn(SequencerState.WAITING_FOR_TXS, this.slot);
1044
- 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})`, {
1045
1487
  blockNumber,
1046
- slot: this.slot,
1488
+ slot: this.targetSlot,
1047
1489
  indexWithinCheckpoint
1048
1490
  });
1049
1491
  await this.waitForTxsPollingInterval();
@@ -1055,13 +1497,37 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1055
1497
  minTxs
1056
1498
  };
1057
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
+ }
1058
1524
  /**
1059
1525
  * Waits for enough attestations to be collected via p2p.
1060
1526
  * This is run after all blocks for the checkpoint have been built.
1061
1527
  */ async waitForAttestations(proposal) {
1062
1528
  if (this.config.fishermanMode) {
1063
1529
  this.log.debug('Skipping attestation collection in fisherman mode');
1064
- return CommitteeAttestationsAndSigners.empty();
1530
+ return CommitteeAttestationsAndSigners.empty(this.getSignatureContext());
1065
1531
  }
1066
1532
  const slotNumber = proposal.slotNumber;
1067
1533
  const { committee, seed, epoch } = await this.epochCache.getCommittee(slotNumber);
@@ -1069,25 +1535,28 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1069
1535
  throw new Error('No committee when collecting attestations');
1070
1536
  } else if (committee.length === 0) {
1071
1537
  this.log.verbose(`Attesting committee is empty`);
1072
- return CommitteeAttestationsAndSigners.empty();
1538
+ return CommitteeAttestationsAndSigners.empty(this.getSignatureContext());
1073
1539
  } else {
1074
1540
  this.log.debug(`Attesting committee length is ${committee.length}`, {
1075
1541
  committee
1076
1542
  });
1077
1543
  }
1078
- const numberOfRequiredAttestations = Math.floor(committee.length * 2 / 3) + 1;
1544
+ const numberOfRequiredAttestations = computeQuorum(committee.length);
1079
1545
  if (this.config.skipCollectingAttestations) {
1080
1546
  this.log.warn('Skipping attestation collection as per config (attesting with own keys only)');
1081
- const attestations = await this.validatorClient?.collectOwnAttestations(proposal);
1082
- 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());
1083
1552
  }
1084
- 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;
1085
1554
  const attestationDeadline = new Date((this.getSlotStartBuildTimestamp() + attestationTimeAllowed) * 1000);
1086
1555
  this.metrics.recordRequiredAttestations(numberOfRequiredAttestations, attestationTimeAllowed);
1087
1556
  const collectAttestationsTimer = new Timer();
1088
1557
  let collectedAttestationsCount = 0;
1089
1558
  try {
1090
- const attestations = await this.validatorClient.collectAttestations(proposal, numberOfRequiredAttestations, attestationDeadline);
1559
+ const attestations = await this.validatorClient.collectAttestations(proposal, numberOfRequiredAttestations, attestationDeadline, this.checkpointNumber);
1091
1560
  collectedAttestationsCount = attestations.length;
1092
1561
  // Trim attestations to minimum required to save L1 calldata gas
1093
1562
  const localAddresses = this.validatorClient.getValidatorAddresses();
@@ -1097,20 +1566,55 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1097
1566
  }
1098
1567
  // Rollup contract requires that the signatures are provided in the order of the committee
1099
1568
  const sorted = orderAttestations(trimmed, committee);
1569
+ this.logCheckpointAttestations('collected', committee, attestations, numberOfRequiredAttestations, {
1570
+ submittedCount: trimmed.length
1571
+ });
1100
1572
  // Manipulate the attestations if we've been configured to do so
1101
1573
  if (this.config.injectFakeAttestation || this.config.injectHighSValueAttestation || this.config.injectUnrecoverableSignatureAttestation || this.config.shuffleAttestationOrdering) {
1102
1574
  return this.manipulateAttestations(proposal.slotNumber, epoch, seed, committee, sorted);
1103
1575
  }
1104
- return new CommitteeAttestationsAndSigners(sorted);
1576
+ return new CommitteeAttestationsAndSigners(sorted, this.getSignatureContext());
1105
1577
  } catch (err) {
1106
1578
  if (err && err instanceof AttestationTimeoutError) {
1107
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);
1108
1591
  }
1109
- throw err;
1592
+ return undefined;
1110
1593
  } finally{
1111
1594
  this.metrics.recordCollectedAttestations(collectedAttestationsCount, collectAttestationsTimer.ms());
1112
1595
  }
1113
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
+ }
1114
1618
  /** Breaks the attestations before publishing based on attack configs */ manipulateAttestations(slotNumber, epoch, seed, committee, attestations) {
1115
1619
  // Compute the proposer index in the committee, since we dont want to tweak it.
1116
1620
  // Otherwise, the L1 rollup contract will reject the block outright.
@@ -1136,25 +1640,36 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1136
1640
  unfreeze(attestations[targetIndex]).signature = generateRecoverableSignature();
1137
1641
  }
1138
1642
  }
1139
- return new CommitteeAttestationsAndSigners(attestations);
1643
+ return new CommitteeAttestationsAndSigners(attestations, this.getSignatureContext());
1140
1644
  }
1141
1645
  if (this.config.shuffleAttestationOrdering) {
1142
1646
  this.log.warn(`Shuffling attestation ordering in checkpoint for slot ${slotNumber} (proposer #${proposerIndex})`);
1143
1647
  const shuffled = [
1144
1648
  ...attestations
1145
1649
  ];
1146
- const [i, j] = [
1147
- (proposerIndex + 1) % shuffled.length,
1148
- (proposerIndex + 2) % shuffled.length
1149
- ];
1150
- const valueI = shuffled[i];
1151
- const valueJ = shuffled[j];
1152
- shuffled[i] = valueJ;
1153
- shuffled[j] = valueI;
1154
- const signers = new CommitteeAttestationsAndSigners(attestations).getSigners();
1155
- return new MaliciousCommitteeAttestationsAndSigners(shuffled, signers);
1650
+ // Find two non-proposer positions that both have non-empty signatures to swap.
1651
+ // This ensures the bitmap doesn't change, so the MaliciousCommitteeAttestationsAndSigners
1652
+ // signers array stays correctly aligned with L1's committee reconstruction.
1653
+ const swappable = [];
1654
+ for(let k = 0; k < shuffled.length; k++){
1655
+ if (!shuffled[k].signature.isEmpty() && k !== proposerIndex) {
1656
+ swappable.push(k);
1657
+ }
1658
+ }
1659
+ if (swappable.length >= 2) {
1660
+ const [i, j] = [
1661
+ swappable[0],
1662
+ swappable[1]
1663
+ ];
1664
+ [shuffled[i], shuffled[j]] = [
1665
+ shuffled[j],
1666
+ shuffled[i]
1667
+ ];
1668
+ }
1669
+ const signers = new CommitteeAttestationsAndSigners(attestations, this.getSignatureContext()).getSigners();
1670
+ return new MaliciousCommitteeAttestationsAndSigners(shuffled, signers, this.getSignatureContext());
1156
1671
  }
1157
- return new CommitteeAttestationsAndSigners(attestations);
1672
+ return new CommitteeAttestationsAndSigners(attestations, this.getSignatureContext());
1158
1673
  }
1159
1674
  async dropFailedTxsFromP2P(failedTxs) {
1160
1675
  if (failedTxs.length === 0) {
@@ -1174,7 +1689,7 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1174
1689
  * and fee analysis only, and pushing them to the archiver causes spurious reorg cascades
1175
1690
  * whenever the real proposer's block arrives from L1.
1176
1691
  */ async syncProposedBlockToArchiver(block) {
1177
- if (this.config.skipPushProposedBlocksToArchiver !== false || this.config.fishermanMode) {
1692
+ if (this.config.skipPushProposedBlocksToArchiver || this.config.fishermanMode) {
1178
1693
  this.log.warn(`Skipping push of proposed block ${block.number} to archiver`, {
1179
1694
  blockNumber: block.number,
1180
1695
  slot: block.header.globalVariables.slotNumber
@@ -1187,19 +1702,48 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1187
1702
  });
1188
1703
  await this.blockSink.addBlock(block);
1189
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
+ }
1190
1734
  /** Runs fee analysis and logs checkpoint outcome as fisherman */ async handleCheckpointEndAsFisherman(checkpoint) {
1191
1735
  // Perform L1 fee analysis before clearing requests
1192
1736
  // The callback is invoked asynchronously after the next block is mined
1193
- 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));
1194
1738
  if (checkpoint) {
1195
- this.log.info(`Validation checkpoint building SUCCEEDED for slot ${this.slot}`, {
1739
+ this.log.info(`Validation checkpoint building SUCCEEDED for slot ${this.targetSlot}`, {
1196
1740
  ...checkpoint.toCheckpointInfo(),
1197
1741
  ...checkpoint.getStats(),
1198
1742
  feeAnalysisId: feeAnalysis?.id
1199
1743
  });
1200
1744
  } else {
1201
- this.log.warn(`Validation block building FAILED for slot ${this.slot}`, {
1202
- slot: this.slot,
1745
+ this.log.warn(`Validation block building FAILED for slot ${this.targetSlot}`, {
1746
+ slot: this.targetSlot,
1203
1747
  feeAnalysisId: feeAnalysis?.id
1204
1748
  });
1205
1749
  this.metrics.recordCheckpointProposalFailed('block_build_failed');
@@ -1210,15 +1754,15 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1210
1754
  * Helper to handle HA double-signing errors. Returns true if the error was handled (caller should yield).
1211
1755
  */ handleHASigningError(err, errorContext) {
1212
1756
  if (err instanceof DutyAlreadySignedError) {
1213
- this.log.info(`${errorContext} for slot ${this.slot} already signed by another HA node, yielding`, {
1214
- slot: this.slot,
1757
+ this.log.info(`${errorContext} for slot ${this.targetSlot} already signed by another HA node, yielding`, {
1758
+ slot: this.targetSlot,
1215
1759
  signedByNode: err.signedByNode
1216
1760
  });
1217
1761
  return true;
1218
1762
  }
1219
1763
  if (err instanceof SlashingProtectionError) {
1220
- this.log.info(`${errorContext} for slot ${this.slot} blocked by slashing protection, yielding`, {
1221
- slot: this.slot,
1764
+ this.log.info(`${errorContext} for slot ${this.targetSlot} blocked by slashing protection, yielding`, {
1765
+ slot: this.targetSlot,
1222
1766
  existingMessageHash: err.existingMessageHash,
1223
1767
  attemptedMessageHash: err.attemptedMessageHash
1224
1768
  });
@@ -1229,13 +1773,14 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1229
1773
  /** Waits until a specific time within the current slot */ async waitUntilTimeInSlot(targetSecondsIntoSlot) {
1230
1774
  const slotStartTimestamp = this.getSlotStartBuildTimestamp();
1231
1775
  const targetTimestamp = slotStartTimestamp + targetSecondsIntoSlot;
1232
- await sleepUntil(new Date(targetTimestamp * 1000), this.dateProvider.nowAsDate());
1776
+ const delayMs = targetTimestamp * 1000 - this.dateProvider.nowAsDate().getTime();
1777
+ await this.awaitInterruptibleSleep(delayMs);
1233
1778
  }
1234
- /** Waits the polling interval for transactions. Extracted for test overriding. */ async waitForTxsPollingInterval() {
1235
- 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);
1236
1781
  }
1237
1782
  getSlotStartBuildTimestamp() {
1238
- return getSlotStartBuildTimestamp(this.slot, this.l1Constants);
1783
+ return getSlotStartBuildTimestamp(this.slotNow, this.l1Constants);
1239
1784
  }
1240
1785
  getSecondsIntoSlot() {
1241
1786
  const slotStartTimestamp = this.getSlotStartBuildTimestamp();