@aztec/sequencer-client 0.0.1-commit.a89ec08 → 0.0.1-commit.aa0c64f

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