@aztec/sequencer-client 0.0.1-commit.9ef841308 → 0.0.1-commit.a4600f49

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 (114) hide show
  1. package/README.md +281 -21
  2. package/dest/client/sequencer-client.d.ts +20 -5
  3. package/dest/client/sequencer-client.d.ts.map +1 -1
  4. package/dest/client/sequencer-client.js +26 -22
  5. package/dest/config.d.ts +11 -4
  6. package/dest/config.d.ts.map +1 -1
  7. package/dest/config.js +55 -26
  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 +7 -3
  21. package/dest/publisher/config.d.ts.map +1 -1
  22. package/dest/publisher/config.js +13 -3
  23. package/dest/publisher/l1_to_l2_messaging.d.ts +21 -0
  24. package/dest/publisher/l1_to_l2_messaging.d.ts.map +1 -0
  25. package/dest/publisher/l1_to_l2_messaging.js +70 -0
  26. package/dest/publisher/l1_tx_failed_store/failed_tx_store.d.ts +50 -11
  27. package/dest/publisher/l1_tx_failed_store/failed_tx_store.d.ts.map +1 -1
  28. package/dest/publisher/l1_tx_failed_store/failed_tx_store.js +68 -1
  29. package/dest/publisher/l1_tx_failed_store/file_store_failed_tx_store.d.ts +2 -2
  30. package/dest/publisher/l1_tx_failed_store/file_store_failed_tx_store.d.ts.map +1 -1
  31. package/dest/publisher/l1_tx_failed_store/file_store_failed_tx_store.js +4 -2
  32. package/dest/publisher/l1_tx_failed_store/index.d.ts +2 -2
  33. package/dest/publisher/l1_tx_failed_store/index.d.ts.map +1 -1
  34. package/dest/publisher/l1_tx_failed_store/index.js +1 -0
  35. package/dest/publisher/sequencer-bundle-simulator.d.ts +96 -0
  36. package/dest/publisher/sequencer-bundle-simulator.d.ts.map +1 -0
  37. package/dest/publisher/sequencer-bundle-simulator.js +198 -0
  38. package/dest/publisher/sequencer-publisher-factory.d.ts +1 -3
  39. package/dest/publisher/sequencer-publisher-factory.d.ts.map +1 -1
  40. package/dest/publisher/sequencer-publisher-factory.js +0 -1
  41. package/dest/publisher/sequencer-publisher.d.ts +97 -68
  42. package/dest/publisher/sequencer-publisher.d.ts.map +1 -1
  43. package/dest/publisher/sequencer-publisher.js +603 -536
  44. package/dest/publisher/write_json.d.ts +11 -0
  45. package/dest/publisher/write_json.d.ts.map +1 -0
  46. package/dest/publisher/write_json.js +57 -0
  47. package/dest/sequencer/automine/automine_factory.d.ts +56 -0
  48. package/dest/sequencer/automine/automine_factory.d.ts.map +1 -0
  49. package/dest/sequencer/automine/automine_factory.js +85 -0
  50. package/dest/sequencer/automine/automine_sequencer.d.ts +189 -0
  51. package/dest/sequencer/automine/automine_sequencer.d.ts.map +1 -0
  52. package/dest/sequencer/automine/automine_sequencer.js +690 -0
  53. package/dest/sequencer/automine/index.d.ts +3 -0
  54. package/dest/sequencer/automine/index.d.ts.map +1 -0
  55. package/dest/sequencer/automine/index.js +2 -0
  56. package/dest/sequencer/checkpoint_proposal_job.d.ts +71 -21
  57. package/dest/sequencer/checkpoint_proposal_job.d.ts.map +1 -1
  58. package/dest/sequencer/checkpoint_proposal_job.js +724 -212
  59. package/dest/sequencer/checkpoint_proposal_job_metrics.d.ts +34 -0
  60. package/dest/sequencer/checkpoint_proposal_job_metrics.d.ts.map +1 -0
  61. package/dest/sequencer/checkpoint_proposal_job_metrics.js +72 -0
  62. package/dest/sequencer/events.d.ts +61 -5
  63. package/dest/sequencer/events.d.ts.map +1 -1
  64. package/dest/sequencer/metrics.d.ts +9 -10
  65. package/dest/sequencer/metrics.d.ts.map +1 -1
  66. package/dest/sequencer/metrics.js +34 -20
  67. package/dest/sequencer/requests_tracker.d.ts +22 -0
  68. package/dest/sequencer/requests_tracker.d.ts.map +1 -0
  69. package/dest/sequencer/requests_tracker.js +33 -0
  70. package/dest/sequencer/sequencer.d.ts +155 -33
  71. package/dest/sequencer/sequencer.d.ts.map +1 -1
  72. package/dest/sequencer/sequencer.js +543 -158
  73. package/dest/sequencer/types.d.ts +2 -2
  74. package/dest/sequencer/types.d.ts.map +1 -1
  75. package/dest/test/index.d.ts +3 -3
  76. package/dest/test/index.d.ts.map +1 -1
  77. package/dest/test/utils.d.ts +15 -1
  78. package/dest/test/utils.d.ts.map +1 -1
  79. package/dest/test/utils.js +25 -7
  80. package/package.json +30 -28
  81. package/src/client/sequencer-client.ts +37 -27
  82. package/src/config.ts +64 -26
  83. package/src/global_variable_builder/README.md +44 -0
  84. package/src/global_variable_builder/fee_predictor.ts +182 -0
  85. package/src/global_variable_builder/fee_provider.ts +97 -0
  86. package/src/global_variable_builder/global_builder.ts +12 -80
  87. package/src/global_variable_builder/index.ts +2 -0
  88. package/src/publisher/config.ts +30 -7
  89. package/src/publisher/l1_to_l2_messaging.ts +85 -0
  90. package/src/publisher/l1_tx_failed_store/failed_tx_store.ts +117 -8
  91. package/src/publisher/l1_tx_failed_store/file_store_failed_tx_store.ts +4 -3
  92. package/src/publisher/l1_tx_failed_store/index.ts +1 -1
  93. package/src/publisher/sequencer-bundle-simulator.ts +254 -0
  94. package/src/publisher/sequencer-publisher-factory.ts +0 -3
  95. package/src/publisher/sequencer-publisher.ts +702 -598
  96. package/src/publisher/write_json.ts +78 -0
  97. package/src/sequencer/README.md +162 -450
  98. package/src/sequencer/automine/README.md +60 -0
  99. package/src/sequencer/automine/automine_factory.ts +152 -0
  100. package/src/sequencer/automine/automine_sequencer.ts +796 -0
  101. package/src/sequencer/automine/index.ts +6 -0
  102. package/src/sequencer/checkpoint_proposal_job.ts +844 -241
  103. package/src/sequencer/checkpoint_proposal_job_metrics.ts +128 -0
  104. package/src/sequencer/events.ts +66 -5
  105. package/src/sequencer/metrics.ts +43 -24
  106. package/src/sequencer/requests_tracker.ts +43 -0
  107. package/src/sequencer/sequencer.ts +625 -175
  108. package/src/sequencer/types.ts +1 -1
  109. package/src/test/index.ts +2 -2
  110. package/src/test/utils.ts +61 -10
  111. package/dest/sequencer/timetable.d.ts +0 -88
  112. package/dest/sequencer/timetable.d.ts.map +0 -1
  113. package/dest/sequencer/timetable.js +0 -222
  114. 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, getTimestampForSlot } 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,25 +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
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
- slotNow;
475
477
  targetSlot;
476
- epochNow;
477
478
  targetEpoch;
478
479
  checkpointNumber;
479
480
  syncedToBlockNumber;
481
+ checkpointedCheckpointNumber;
480
482
  proposer;
481
483
  publisher;
482
484
  attestorAddress;
@@ -490,15 +492,19 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
490
492
  checkpointsBuilder;
491
493
  blockSink;
492
494
  l1Constants;
495
+ signatureContext;
493
496
  config;
494
497
  timetable;
495
498
  slasherClient;
496
499
  epochCache;
497
500
  dateProvider;
498
501
  metrics;
502
+ checkpointMetrics;
499
503
  eventEmitter;
504
+ pendingRequests;
500
505
  setStateFn;
501
506
  tracer;
507
+ proposedCheckpointData;
502
508
  static{
503
509
  ({ e: [_initProto] } = _apply_decs_2203_r(this, [
504
510
  [
@@ -535,23 +541,31 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
535
541
  _dec6,
536
542
  2,
537
543
  "waitForAttestations"
538
- ],
539
- [
540
- _dec7,
541
- 2,
542
- "waitUntilTimeInSlot"
543
544
  ]
544
545
  ], []));
545
546
  }
546
547
  log;
547
- constructor(slotNow, targetSlot, epochNow, targetEpoch, checkpointNumber, syncedToBlockNumber, // TODO(palla/mbps): Can we remove the proposer in favor of attestorAddress? Need to check fisherman-node flows.
548
- proposer, publisher, attestorAddress, invalidateCheckpoint, validatorClient, globalsBuilder, p2pClient, worldState, l1ToL2MessageSource, l2BlockSource, checkpointsBuilder, blockSink, l1Constants, config, timetable, slasherClient, epochCache, dateProvider, metrics, eventEmitter, setStateFn, tracer, bindings){
549
- this.slotNow = slotNow;
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){
550
564
  this.targetSlot = targetSlot;
551
- this.epochNow = epochNow;
552
565
  this.targetEpoch = targetEpoch;
553
566
  this.checkpointNumber = checkpointNumber;
554
567
  this.syncedToBlockNumber = syncedToBlockNumber;
568
+ this.checkpointedCheckpointNumber = checkpointedCheckpointNumber;
555
569
  this.proposer = proposer;
556
570
  this.publisher = publisher;
557
571
  this.attestorAddress = attestorAddress;
@@ -565,82 +579,351 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
565
579
  this.checkpointsBuilder = checkpointsBuilder;
566
580
  this.blockSink = blockSink;
567
581
  this.l1Constants = l1Constants;
582
+ this.signatureContext = signatureContext;
568
583
  this.config = config;
569
584
  this.timetable = timetable;
570
585
  this.slasherClient = slasherClient;
571
586
  this.epochCache = epochCache;
572
587
  this.dateProvider = dateProvider;
573
588
  this.metrics = metrics;
589
+ this.checkpointMetrics = checkpointMetrics;
574
590
  this.eventEmitter = eventEmitter;
591
+ this.pendingRequests = pendingRequests;
575
592
  this.setStateFn = setStateFn;
576
593
  this.tracer = tracer;
577
- _initProto(this);
594
+ this.proposedCheckpointData = proposedCheckpointData;
595
+ this.interruptibleSleep = (_initProto(this), new InterruptibleSleep());
596
+ this.interrupted = false;
578
597
  this.log = createLogger('sequencer:checkpoint-proposal', {
579
598
  ...bindings,
580
- instanceId: `slot-${this.slotNow}`
599
+ instanceId: `slot-${this.getBuildSlot()}`
600
+ });
601
+ this.checkpointEventLog = createLogger('sequencer:checkpoint-events', {
602
+ ...bindings,
603
+ instanceId: `slot-${this.getBuildSlot()}`
581
604
  });
582
605
  }
583
- /** The wall-clock slot during which the proposer builds. */ get slot() {
584
- return this.slotNow;
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);
585
618
  }
586
- /** The wall-clock epoch. */ get epoch() {
587
- return this.epochNow;
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
644
+ });
588
645
  }
589
646
  /**
590
647
  * Executes the checkpoint proposal job.
591
- * 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.
592
652
  */ async execute() {
593
653
  // Enqueue governance and slashing votes (returns promises that will be awaited later)
594
654
  // In fisherman mode, we simulate slashing but don't actually publish to L1
595
655
  // These are constant for the whole slot, so we only enqueue them once
596
656
  const votesPromises = new CheckpointVoter(this.targetSlot, this.publisher, this.attestorAddress, this.validatorClient, this.slasherClient, this.l1Constants, this.config, this.metrics, this.log).enqueueVotes();
597
- // Build and propose the checkpoint. This will enqueue the request on the publisher if a checkpoint is built.
598
- const checkpoint = await this.proposeCheckpoint();
599
- // Wait until the voting promises have resolved, so all requests are enqueued (not sent)
600
- await Promise.all(votesPromises);
601
- if (checkpoint) {
602
- this.metrics.recordCheckpointProposalSuccess();
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;
603
672
  }
673
+ const { checkpoint } = broadcast;
674
+ this.metrics.recordCheckpointProposalSuccess();
604
675
  // Do not post anything to L1 if we are fishermen, but do perform L1 fee analysis
605
676
  if (this.config.fishermanMode) {
606
677
  await this.handleCheckpointEndAsFisherman(checkpoint);
607
- return;
678
+ return checkpoint;
608
679
  }
609
- // If pipelining, wait until the submission slot so L1 recognizes the pipelined proposer
610
- if (this.epochCache.isProposerPipeliningEnabled()) {
611
- const submissionSlotTimestamp = getTimestampForSlot(this.targetSlot, this.l1Constants) - BigInt(this.l1Constants.ethereumSlotDuration);
612
- this.log.info(`Waiting until submission slot ${this.targetSlot} for L1 submission`, {
613
- slot: this.slot,
614
- submissionSlot: this.targetSlot,
615
- submissionSlotTimestamp
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)
616
763
  });
617
- await sleepUntil(new Date(Number(submissionSlotTimestamp) * 1000), this.dateProvider.nowAsDate());
618
- // After waking, verify the parent checkpoint wasn't pruned during the sleep.
619
- // We check L1's pending tip directly instead of canProposeAt, which also validates the proposer
620
- // identity and would fail because the timestamp resolves to a different slot's proposer.
621
- const l1Tips = await this.publisher.rollupContract.getTips();
622
- if (l1Tips.pending < this.checkpointNumber - 1) {
623
- this.log.warn(`Parent checkpoint was pruned during pipelining sleep (L1 pending=${l1Tips.pending}, expected>=${this.checkpointNumber - 1}), skipping L1 submission for checkpoint ${this.checkpointNumber}`);
624
- return undefined;
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)
768
+ });
769
+ this.eventEmitter.emit('checkpoint-publish-failed', {
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;
625
792
  }
626
793
  }
627
- // Then send everything to L1
628
- const l1Response = await this.publisher.sendRequests();
629
- const proposedAction = l1Response?.successfulActions.find((a)=>a === 'propose');
630
- if (proposedAction) {
631
- this.eventEmitter.emit('checkpoint-published', {
632
- checkpoint: this.checkpointNumber,
633
- slot: this.slot
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
634
825
  });
635
- const coinbase = checkpoint?.header.coinbase;
636
- await this.metrics.incFilledSlot(this.publisher.getSenderAddress().toString(), coinbase);
637
- return checkpoint;
638
- } else if (checkpoint) {
639
- this.eventEmitter.emit('checkpoint-publish-failed', {
640
- ...l1Response,
641
- slot: this.slot
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
642
926
  });
643
- return undefined;
644
927
  }
645
928
  }
646
929
  async proposeCheckpoint() {
@@ -651,16 +934,28 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
651
934
  hasError: false
652
935
  };
653
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);
654
946
  // Get operator configured coinbase and fee recipient for this attestor
655
947
  const coinbase = this.validatorClient.getCoinbaseForAttestor(this.attestorAddress);
656
948
  const feeRecipient = this.validatorClient.getFeeRecipientForAttestor(this.attestorAddress);
657
949
  // Start the checkpoint
658
- this.setStateFn(SequencerState.INITIALIZING_CHECKPOINT, this.targetSlot);
659
- this.log.info(`Starting checkpoint proposal`, {
660
- buildSlot: this.slot,
950
+ this.setState(SequencerState.INITIALIZING_CHECKPOINT);
951
+ this.logCheckpointEvent('slot-started', `Starting checkpoint proposal for slot ${this.targetSlot}`, {
952
+ buildSlot: this.getBuildSlot(),
661
953
  submissionSlot: this.targetSlot,
662
- pipelining: this.epochCache.isProposerPipeliningEnabled(),
954
+ slot: this.targetSlot,
955
+ checkpointNumber: this.checkpointNumber,
663
956
  proposer: this.proposer?.toString(),
957
+ attestorAddress: this.attestorAddress.toString(),
958
+ publisherAddress: this.publisher.getSenderAddress().toString(),
664
959
  coinbase: coinbase.toString()
665
960
  });
666
961
  this.metrics.incOpenSlot(this.targetSlot, this.proposer?.toString() ?? 'unknown');
@@ -668,15 +963,41 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
668
963
  if (this.invalidateCheckpoint && !this.config.skipInvalidateBlockAsProposer) {
669
964
  this.publisher.enqueueInvalidateCheckpoint(this.invalidateCheckpoint);
670
965
  }
671
- // Create checkpoint builder for the slot
672
- const checkpointGlobalVariables = await this.globalsBuilder.buildCheckpointGlobalVariables(coinbase, feeRecipient, this.targetSlot);
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);
673
980
  // Collect L1 to L2 messages for the checkpoint and compute their hash
674
981
  const l1ToL2Messages = await this.l1ToL2MessageSource.getL1ToL2Messages(this.checkpointNumber);
675
982
  const inHash = computeInHashFromL1ToL2Messages(l1ToL2Messages);
676
- // Collect the out hashes of all the checkpoints before this one in the same epoch
677
- const previousCheckpointOutHashes = (await this.l2BlockSource.getCheckpointsDataForEpoch(this.targetEpoch)).filter((c)=>c.checkpointNumber < this.checkpointNumber).map((c)=>c.checkpointOutHash);
678
- // Get the fee asset price modifier from the oracle
679
- 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);
680
1001
  const fork = _ts_add_disposable_resource(env, await this.worldState.fork(this.syncedToBlockNumber, {
681
1002
  closeDelayMs: 12_000
682
1003
  }), true);
@@ -689,7 +1010,7 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
689
1010
  };
690
1011
  const checkpointProposalOptions = {
691
1012
  publishFullTxs: !!this.config.publishTxsWithProposals,
692
- broadcastInvalidCheckpointProposal: this.config.broadcastInvalidBlockProposal
1013
+ broadcastInvalidCheckpointProposal: this.config.broadcastInvalidCheckpointProposalOnly || this.config.broadcastInvalidBlockProposal
693
1014
  };
694
1015
  let blocksInCheckpoint = [];
695
1016
  let blockPendingBroadcast = undefined;
@@ -709,8 +1030,15 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
709
1030
  throw err;
710
1031
  }
711
1032
  if (blocksInCheckpoint.length === 0) {
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
+ });
712
1038
  this.log.warn(`No blocks were built for slot ${this.targetSlot}`, {
713
- slot: this.targetSlot
1039
+ slot: this.targetSlot,
1040
+ checkpointNumber: this.checkpointNumber,
1041
+ reason: 'no_blocks_built'
714
1042
  });
715
1043
  this.eventEmitter.emit('checkpoint-empty', {
716
1044
  slot: this.targetSlot
@@ -719,16 +1047,25 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
719
1047
  }
720
1048
  const minBlocksForCheckpoint = this.config.minBlocksForCheckpoint;
721
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
+ });
722
1057
  this.log.warn(`Checkpoint has fewer blocks than minimum (${blocksInCheckpoint.length} < ${minBlocksForCheckpoint}), skipping proposal`, {
723
1058
  slot: this.targetSlot,
1059
+ checkpointNumber: this.checkpointNumber,
724
1060
  blocksBuilt: blocksInCheckpoint.length,
725
- minBlocksForCheckpoint
1061
+ minBlocksForCheckpoint,
1062
+ reason: 'min_blocks_not_met'
726
1063
  });
727
1064
  return undefined;
728
1065
  }
729
1066
  // Assemble and broadcast the checkpoint proposal, including the last block that was not
730
1067
  // broadcasted yet, and wait to collect the committee attestations.
731
- this.setStateFn(SequencerState.ASSEMBLING_CHECKPOINT, this.targetSlot);
1068
+ this.setState(SequencerState.ASSEMBLING_CHECKPOINT);
732
1069
  const checkpoint = await checkpointBuilder.completeCheckpoint();
733
1070
  // Final validation: per-block limits are only checked if the operator set them explicitly.
734
1071
  // Otherwise, checkpoint-level budgets were already enforced by the redistribution logic.
@@ -741,14 +1078,36 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
741
1078
  maxTxsPerCheckpoint: this.config.maxTxsPerCheckpoint
742
1079
  });
743
1080
  } catch (err) {
744
- 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',
745
1093
  checkpoint: checkpoint.header.toInspect()
746
1094
  });
747
1095
  return undefined;
748
1096
  }
749
1097
  // Record checkpoint-level build metrics
750
- this.metrics.recordCheckpointBuild(checkpointBuildTimer.ms(), blocksInCheckpoint.length, checkpoint.getStats().txCount, Number(checkpoint.header.totalManaUsed.toBigInt()));
751
- // 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
752
1111
  if (this.config.fishermanMode) {
753
1112
  this.log.info(`Built checkpoint for slot ${this.targetSlot} with ${blocksInCheckpoint.length} blocks. ` + `Skipping proposal in fisherman mode.`, {
754
1113
  slot: this.targetSlot,
@@ -756,53 +1115,58 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
756
1115
  blocksBuilt: blocksInCheckpoint.length
757
1116
  });
758
1117
  this.metrics.recordCheckpointSuccess();
759
- 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
+ };
760
1124
  }
761
- // Include the block pending broadcast in the checkpoint proposal if any
762
- const lastBlock = blockPendingBroadcast && {
763
- blockHeader: blockPendingBroadcast.block.header,
764
- indexWithinCheckpoint: blockPendingBroadcast.block.indexWithinCheckpoint,
765
- txs: blockPendingBroadcast.txs
766
- };
767
- // Create the checkpoint proposal and broadcast it
768
- const proposal = await this.validatorClient.createCheckpointProposal(checkpoint.header, checkpoint.archive.root, feeAssetPriceModifier, lastBlock, this.proposer, checkpointProposalOptions);
769
- const blockProposedAt = this.dateProvider.now();
770
- await this.p2pClient.broadcastCheckpointProposal(proposal);
771
- this.setStateFn(SequencerState.COLLECTING_ATTESTATIONS, this.targetSlot);
772
- const attestations = await this.waitForAttestations(proposal);
773
- const blockAttestedAt = this.dateProvider.now();
774
- this.metrics.recordCheckpointAttestationDelay(blockAttestedAt - blockProposedAt);
775
- // Proposer must sign over the attestations before pushing them to L1
776
- const signer = this.proposer ?? this.publisher.getSenderAddress();
777
- 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.
778
1128
  try {
779
- attestationsSignature = await this.validatorClient.signAttestationsAndSigners(attestations, signer, this.targetSlot, this.checkpointNumber);
1129
+ await this.publisher.validateBlockHeader(checkpoint.header, this.checkpointSimulationOverridesPlan);
780
1130
  } catch (err) {
781
- // We shouldn't really get here since we yield to another HA node
782
- // as soon as we see these errors when creating block or checkpoint proposals.
783
- if (this.handleHASigningError(err, 'Attestations signature')) {
784
- return undefined;
785
- }
786
- 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;
787
1142
  }
788
- // Enqueue publishing the checkpoint to L1
789
- this.setStateFn(SequencerState.PUBLISHING_CHECKPOINT, this.targetSlot);
790
- const aztecSlotDuration = this.l1Constants.slotDuration;
791
- const submissionSlotStart = Number(getTimestampForSlot(this.targetSlot, this.l1Constants));
792
- const txTimeoutAt = new Date((submissionSlotStart + aztecSlotDuration) * 1000);
793
- // If we have been configured to potentially skip publishing checkpoint then roll the dice here
794
- if (this.config.skipPublishingCheckpointsPercent !== undefined && this.config.skipPublishingCheckpointsPercent > 0) {
795
- const result = Math.max(0, randomInt(100));
796
- if (result < this.config.skipPublishingCheckpointsPercent) {
797
- this.log.warn(`Skipping publishing proposal for checkpoint ${checkpoint.number}. Configured percentage: ${this.config.skipPublishingCheckpointsPercent}, generated value: ${result}`);
798
- 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);
799
1159
  }
1160
+ } else if (!this.config.skipBroadcastProposals) {
1161
+ await this.p2pClient.broadcastCheckpointProposal(proposal);
1162
+ this.checkpointMetrics.noteCheckpointBroadcast(this.dateProvider.now());
800
1163
  }
801
- await this.publisher.enqueueProposeCheckpoint(checkpoint, attestations, attestationsSignature, {
802
- txTimeoutAt,
803
- forcePendingCheckpointNumber: this.invalidateCheckpoint?.forcePendingCheckpointNumber
804
- });
805
- return checkpoint;
1164
+ // Return immediately after broadcast — attestation collection happens in the background
1165
+ return {
1166
+ checkpoint,
1167
+ proposal,
1168
+ blockProposedAt
1169
+ };
806
1170
  } catch (e) {
807
1171
  env.error = e;
808
1172
  env.hasError = true;
@@ -815,7 +1179,7 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
815
1179
  // swallow this error. It's already been logged by a function deeper in the stack
816
1180
  return undefined;
817
1181
  }
818
- this.log.error(`Error building checkpoint at slot ${this.slot}`, err);
1182
+ this.log.error(`Error building checkpoint at slot ${this.targetSlot}`, err);
819
1183
  return undefined;
820
1184
  }
821
1185
  }
@@ -831,13 +1195,21 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
831
1195
  const blocksBuilt = blocksInCheckpoint.length;
832
1196
  const indexWithinCheckpoint = IndexWithinCheckpoint(blocksBuilt);
833
1197
  const blockNumber = BlockNumber(initialBlockNumber + blocksBuilt);
834
- const secondsIntoSlot = this.getSecondsIntoSlot();
835
- 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);
836
1208
  if (!timingInfo.canStart) {
837
1209
  this.log.debug(`Not enough time left in slot to start another block`, {
838
1210
  slot: this.targetSlot,
839
1211
  blocksBuilt,
840
- secondsIntoSlot
1212
+ nowSeconds
841
1213
  });
842
1214
  break;
843
1215
  }
@@ -846,23 +1218,24 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
846
1218
  blockTimestamp: timestamp,
847
1219
  // Create an empty block if we haven't already and this is the last one
848
1220
  forceCreate: timingInfo.isLastBlock && blocksBuilt === 0 && this.config.buildCheckpointIfEmpty,
849
- // Build deadline is only set if we are enforcing the timetable
850
- buildDeadline: timingInfo.deadline ? new Date((this.getSlotStartBuildTimestamp() + timingInfo.deadline) * 1000) : undefined,
1221
+ buildDeadline: new Date(timingInfo.deadline * 1000),
851
1222
  blockNumber,
852
1223
  indexWithinCheckpoint,
853
1224
  txHashesAlreadyIncluded
854
1225
  });
855
- // TODO(palla/mbps): Review these conditions. We may want to keep trying in some scenarios.
856
- if (!buildResult && timingInfo.isLastBlock) {
857
- break;
858
- } else if (!buildResult && timingInfo.deadline !== undefined) {
859
- // 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
860
1233
  await this.waitUntilNextSubslot(timingInfo.deadline);
861
1234
  continue;
862
- } else if (!buildResult) {
863
- break;
864
- } else if ('error' in buildResult) {
865
- // 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) {
866
1239
  if (!(buildResult.error instanceof SequencerInterruptedError)) {
867
1240
  this.log.warn(`Halting block building for slot ${this.targetSlot}`, {
868
1241
  slot: this.targetSlot,
@@ -873,34 +1246,36 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
873
1246
  break;
874
1247
  }
875
1248
  const { block, usedTxs } = buildResult;
1249
+ this.checkpointMetrics.noteCheckpointBlockBuilt(this.dateProvider.now(), {
1250
+ isFirstBlock: blocksBuilt === 0,
1251
+ isLastBlock: timingInfo.isLastBlock
1252
+ });
876
1253
  blocksInCheckpoint.push(block);
877
1254
  usedTxs.forEach((tx)=>txHashesAlreadyIncluded.add(tx.txHash.toString()));
878
- // If this is the last block, sync it to the archiver and exit the loop
879
- // 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.
880
1266
  if (timingInfo.isLastBlock) {
881
- await this.syncProposedBlockToArchiver(block);
882
1267
  this.log.verbose(`Completed final block ${blockNumber} for slot ${this.targetSlot}`, {
883
1268
  slot: this.targetSlot,
884
1269
  blockNumber,
885
1270
  blocksBuilt
886
1271
  });
887
- blockPendingBroadcast = {
888
- block,
889
- txs: usedTxs
890
- };
1272
+ blockPendingBroadcast = proposal;
891
1273
  break;
892
1274
  }
893
- // Broadcast the block proposal (unless we're in fisherman mode) unless the block is the last one,
894
- // in which case we'll broadcast it along with the checkpoint at the end of the loop.
895
- // Note that we only send the block to the archiver if we manage to create the proposal, so if there's
896
- // a HA error we don't pollute our archiver with a block that won't make it to the chain.
897
- const proposal = await this.createBlockProposal(block, inHash, usedTxs, blockProposalOptions);
898
- // Sync the proposed block to the archiver to make it available, only after we've managed to sign the proposal.
899
- // We wait for the sync to succeed, as this helps catch consistency errors, even if it means we lose some time for block-building.
900
- // If this throws, we abort the entire checkpoint.
901
- await this.syncProposedBlockToArchiver(block);
902
1275
  // Once we have a signed proposal and the archiver agreed with our proposed block, then we broadcast it.
903
- proposal && await this.p2pClient.broadcastProposal(proposal);
1276
+ if (proposal && !this.config.skipBroadcastProposals) {
1277
+ await this.p2pClient.broadcastProposal(proposal);
1278
+ }
904
1279
  // Wait until the next block's start time
905
1280
  await this.waitUntilNextSubslot(timingInfo.deadline);
906
1281
  }
@@ -918,14 +1293,17 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
918
1293
  this.log.info(`Skipping block proposal for block ${block.number} in fisherman mode`);
919
1294
  return Promise.resolve(undefined);
920
1295
  }
921
- 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);
922
1297
  }
923
- /** Sleeps until it is time to produce the next block in the slot */ async waitUntilNextSubslot(nextSubslotStart) {
924
- this.setStateFn(SequencerState.WAITING_UNTIL_NEXT_BLOCK, this.targetSlot);
925
- this.log.verbose(`Waiting until time for the next block at ${nextSubslotStart}s into 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`, {
926
1304
  slot: this.targetSlot
927
1305
  });
928
- await this.waitUntilTimeInSlot(nextSubslotStart);
1306
+ await this.awaitInterruptibleSleep(Math.max(0, nextSubslotStart * 1000 - this.dateProvider.now()));
929
1307
  }
930
1308
  /** Builds a single block. Called from the main block building loop. */ async buildSingleBlock(checkpointBuilder, opts) {
931
1309
  const { blockTimestamp, forceCreate, blockNumber, indexWithinCheckpoint, buildDeadline, txHashesAlreadyIncluded } = opts;
@@ -935,30 +1313,46 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
935
1313
  });
936
1314
  try {
937
1315
  // Wait until we have enough txs to build the block
938
- const { availableTxs, canStartBuilding, minTxs } = await this.waitForMinTxs(opts);
1316
+ const { canStartBuilding, minTxs } = await this.waitForMinTxs(opts);
939
1317
  if (!canStartBuilding) {
940
- this.log.warn(`Not enough txs to build block ${blockNumber} at index ${indexWithinCheckpoint} in slot ${this.targetSlot} (got ${availableTxs} txs but needs ${minTxs})`, {
1318
+ this.logCheckpointEvent('block-build-failed', `Block build failed for slot ${this.targetSlot}`, {
1319
+ reason: 'insufficient_txs',
941
1320
  blockNumber,
942
1321
  slot: this.targetSlot,
943
- indexWithinCheckpoint
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',
1328
+ blockNumber,
1329
+ slot: this.targetSlot,
1330
+ checkpointNumber: this.checkpointNumber,
1331
+ indexWithinCheckpoint,
1332
+ minTxs
944
1333
  });
945
1334
  this.eventEmitter.emit('block-tx-count-check-failed', {
946
1335
  minTxs,
947
- availableTxs,
948
1336
  slot: this.targetSlot
949
1337
  });
950
1338
  this.metrics.recordBlockProposalFailed('insufficient_txs');
951
- return undefined;
1339
+ return {
1340
+ failure: 'insufficient-txs'
1341
+ };
952
1342
  }
953
1343
  // Create iterator to pending txs. We filter out txs already included in previous blocks in the checkpoint
954
1344
  // just in case p2p failed to sync the provisional block and didn't get to remove those txs from the mempool yet.
955
- const pendingTxs = filter(this.p2pClient.iterateEligiblePendingTxs(), (tx)=>!txHashesAlreadyIncluded.has(tx.txHash.toString()));
956
- this.log.debug(`Building block ${blockNumber} at index ${indexWithinCheckpoint} for slot ${this.targetSlot} with ${availableTxs} available txs`, {
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}`, {
957
1351
  slot: this.targetSlot,
958
1352
  blockNumber,
959
1353
  indexWithinCheckpoint
960
1354
  });
961
- this.setStateFn(SequencerState.CREATING_BLOCK, this.targetSlot);
1355
+ this.setState(SequencerState.CREATING_BLOCK);
962
1356
  // Per-block limits are operator overrides (from SEQ_MAX_L2_BLOCK_GAS etc.) further capped
963
1357
  // by remaining checkpoint-level budgets inside CheckpointBuilder before each block is built.
964
1358
  // minValidTxs is passed into the builder so it can reject the block *before* updating state.
@@ -969,8 +1363,9 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
969
1363
  deadline: buildDeadline,
970
1364
  isBuildingProposal: true,
971
1365
  minValidTxs,
972
- maxBlocksPerCheckpoint: this.timetable.maxNumberOfBlocks,
973
- perBlockAllocationMultiplier: this.config.perBlockAllocationMultiplier
1366
+ maxBlocksPerCheckpoint: this.timetable.getMaxBlocksPerCheckpoint(),
1367
+ perBlockAllocationMultiplier: this.config.perBlockAllocationMultiplier,
1368
+ perBlockDAAllocationMultiplier: this.config.perBlockDAAllocationMultiplier
974
1369
  };
975
1370
  // Actually build the block by executing txs. The builder throws InsufficientValidTxsError
976
1371
  // if the number of successfully processed txs is below minValidTxs, ensuring state is not
@@ -979,8 +1374,19 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
979
1374
  // If any txs failed during execution, drop them from the mempool so we don't pick them up again
980
1375
  await this.dropFailedTxsFromP2P(buildResult.failedTxs);
981
1376
  if (buildResult.status === 'insufficient-valid-txs') {
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
+ });
982
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',
983
1388
  slot: this.targetSlot,
1389
+ checkpointNumber: this.checkpointNumber,
984
1390
  blockNumber,
985
1391
  numTxs: buildResult.processedCount,
986
1392
  indexWithinCheckpoint,
@@ -991,7 +1397,9 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
991
1397
  slot: this.targetSlot
992
1398
  });
993
1399
  this.metrics.recordBlockProposalFailed('insufficient_valid_txs');
994
- return undefined;
1400
+ return {
1401
+ failure: 'insufficient-valid-txs'
1402
+ };
995
1403
  }
996
1404
  // Block creation succeeded, emit stats and metrics
997
1405
  const { block, publicProcessorDuration, usedTxs, blockBuildDuration, numTxs } = buildResult;
@@ -1010,12 +1418,17 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1010
1418
  manaPerSec,
1011
1419
  ...blockStats
1012
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.
1013
1423
  this.eventEmitter.emit('block-proposed', {
1014
1424
  blockNumber: block.number,
1425
+ blockHash,
1426
+ checkpointNumber: this.checkpointNumber,
1427
+ indexWithinCheckpoint: block.indexWithinCheckpoint,
1015
1428
  slot: this.targetSlot,
1016
- buildSlot: this.slotNow
1429
+ buildSlot: this.getBuildSlot()
1017
1430
  });
1018
- this.metrics.recordBuiltBlock(blockBuildDuration, block.header.totalManaUsed.toNumberUnsafe());
1431
+ this.metrics.recordBuiltBlock(blockBuildDuration, block.header.totalManaUsed.toNumberUnsafe(), this.targetSlot);
1019
1432
  return {
1020
1433
  block,
1021
1434
  usedTxs
@@ -1025,9 +1438,17 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1025
1438
  reason: err.message,
1026
1439
  slot: this.targetSlot
1027
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
1446
+ });
1028
1447
  this.log.error(`Error building block`, err, {
1029
- blockNumber,
1030
- slot: this.targetSlot
1448
+ reason: err instanceof Error ? err.message : String(err),
1449
+ slot: this.targetSlot,
1450
+ checkpointNumber: this.checkpointNumber,
1451
+ blockNumber
1031
1452
  });
1032
1453
  this.metrics.recordBlockProposalFailed(err.name || 'unknown_error');
1033
1454
  this.metrics.recordFailedBlock();
@@ -1061,42 +1482,65 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1061
1482
  const { indexWithinCheckpoint, blockNumber, buildDeadline, forceCreate } = opts;
1062
1483
  // We only allow a block with 0 txs in the first block of the checkpoint
1063
1484
  const minTxs = indexWithinCheckpoint > 0 && this.config.minTxsPerBlock === 0 ? 1 : this.config.minTxsPerBlock;
1064
- // Deadline is undefined if we are not enforcing the timetable, meaning we'll exit immediately when out of time
1065
- const startBuildingDeadline = buildDeadline ? new Date(buildDeadline.getTime() - this.timetable.minExecutionTime * 1000) : undefined;
1066
- let availableTxs = await this.p2pClient.getPendingTxCount();
1067
- 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)){
1068
1490
  // If we're past deadline, or we have no deadline, give up
1069
1491
  const now = this.dateProvider.nowAsDate();
1070
1492
  if (startBuildingDeadline === undefined || now >= startBuildingDeadline) {
1071
1493
  return {
1072
1494
  canStartBuilding: false,
1073
- availableTxs,
1074
1495
  minTxs
1075
1496
  };
1076
1497
  }
1077
1498
  // Wait a bit before checking again
1078
- this.setStateFn(SequencerState.WAITING_FOR_TXS, this.targetSlot);
1079
- this.log.verbose(`Waiting for enough txs to build block ${blockNumber} at index ${indexWithinCheckpoint} in slot ${this.targetSlot} (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}`, {
1080
1501
  blockNumber,
1081
1502
  slot: this.targetSlot,
1082
- indexWithinCheckpoint
1503
+ indexWithinCheckpoint,
1504
+ minTxs
1083
1505
  });
1084
1506
  await this.waitForTxsPollingInterval();
1085
- availableTxs = await this.p2pClient.getPendingTxCount();
1086
1507
  }
1087
1508
  return {
1088
1509
  canStartBuilding: true,
1089
- availableTxs,
1090
1510
  minTxs
1091
1511
  };
1092
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
+ }
1093
1537
  /**
1094
1538
  * Waits for enough attestations to be collected via p2p.
1095
1539
  * This is run after all blocks for the checkpoint have been built.
1096
1540
  */ async waitForAttestations(proposal) {
1097
1541
  if (this.config.fishermanMode) {
1098
1542
  this.log.debug('Skipping attestation collection in fisherman mode');
1099
- return CommitteeAttestationsAndSigners.empty();
1543
+ return CommitteeAttestationsAndSigners.empty(this.getSignatureContext());
1100
1544
  }
1101
1545
  const slotNumber = proposal.slotNumber;
1102
1546
  const { committee, seed, epoch } = await this.epochCache.getCommittee(slotNumber);
@@ -1104,25 +1548,29 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1104
1548
  throw new Error('No committee when collecting attestations');
1105
1549
  } else if (committee.length === 0) {
1106
1550
  this.log.verbose(`Attesting committee is empty`);
1107
- return CommitteeAttestationsAndSigners.empty();
1551
+ return CommitteeAttestationsAndSigners.empty(this.getSignatureContext());
1108
1552
  } else {
1109
1553
  this.log.debug(`Attesting committee length is ${committee.length}`, {
1110
1554
  committee
1111
1555
  });
1112
1556
  }
1113
- const numberOfRequiredAttestations = Math.floor(committee.length * 2 / 3) + 1;
1557
+ const numberOfRequiredAttestations = computeQuorum(committee.length);
1114
1558
  if (this.config.skipCollectingAttestations) {
1115
1559
  this.log.warn('Skipping attestation collection as per config (attesting with own keys only)');
1116
- const attestations = await this.validatorClient?.collectOwnAttestations(proposal);
1117
- 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());
1118
1565
  }
1119
- const attestationTimeAllowed = this.config.enforceTimeTable ? this.timetable.getMaxAllowedTime(SequencerState.PUBLISHING_CHECKPOINT) : this.l1Constants.slotDuration;
1120
- const attestationDeadline = new Date((this.getSlotStartBuildTimestamp() + attestationTimeAllowed) * 1000);
1121
- 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()));
1122
1570
  const collectAttestationsTimer = new Timer();
1123
1571
  let collectedAttestationsCount = 0;
1124
1572
  try {
1125
- const attestations = await this.validatorClient.collectAttestations(proposal, numberOfRequiredAttestations, attestationDeadline);
1573
+ const attestations = await this.validatorClient.collectAttestations(proposal, numberOfRequiredAttestations, attestationDeadline, this.checkpointNumber);
1126
1574
  collectedAttestationsCount = attestations.length;
1127
1575
  // Trim attestations to minimum required to save L1 calldata gas
1128
1576
  const localAddresses = this.validatorClient.getValidatorAddresses();
@@ -1132,20 +1580,55 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1132
1580
  }
1133
1581
  // Rollup contract requires that the signatures are provided in the order of the committee
1134
1582
  const sorted = orderAttestations(trimmed, committee);
1583
+ this.logCheckpointAttestations('collected', committee, attestations, numberOfRequiredAttestations, {
1584
+ submittedCount: trimmed.length
1585
+ });
1135
1586
  // Manipulate the attestations if we've been configured to do so
1136
- 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) {
1137
1588
  return this.manipulateAttestations(proposal.slotNumber, epoch, seed, committee, sorted);
1138
1589
  }
1139
- return new CommitteeAttestationsAndSigners(sorted);
1590
+ return new CommitteeAttestationsAndSigners(sorted, this.getSignatureContext());
1140
1591
  } catch (err) {
1141
1592
  if (err && err instanceof AttestationTimeoutError) {
1142
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);
1143
1605
  }
1144
- throw err;
1606
+ return undefined;
1145
1607
  } finally{
1146
1608
  this.metrics.recordCollectedAttestations(collectedAttestationsCount, collectAttestationsTimer.ms());
1147
1609
  }
1148
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
+ }
1149
1632
  /** Breaks the attestations before publishing based on attack configs */ manipulateAttestations(slotNumber, epoch, seed, committee, attestations) {
1150
1633
  // Compute the proposer index in the committee, since we dont want to tweak it.
1151
1634
  // Otherwise, the L1 rollup contract will reject the block outright.
@@ -1171,7 +1654,15 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1171
1654
  unfreeze(attestations[targetIndex]).signature = generateRecoverableSignature();
1172
1655
  }
1173
1656
  }
1174
- 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());
1175
1666
  }
1176
1667
  if (this.config.shuffleAttestationOrdering) {
1177
1668
  this.log.warn(`Shuffling attestation ordering in checkpoint for slot ${slotNumber} (proposer #${proposerIndex})`);
@@ -1197,10 +1688,10 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1197
1688
  shuffled[i]
1198
1689
  ];
1199
1690
  }
1200
- const signers = new CommitteeAttestationsAndSigners(attestations).getSigners();
1201
- return new MaliciousCommitteeAttestationsAndSigners(shuffled, signers);
1691
+ const signers = new CommitteeAttestationsAndSigners(attestations, this.getSignatureContext()).getSigners();
1692
+ return new MaliciousCommitteeAttestationsAndSigners(shuffled, signers, this.getSignatureContext());
1202
1693
  }
1203
- return new CommitteeAttestationsAndSigners(attestations);
1694
+ return new CommitteeAttestationsAndSigners(attestations, this.getSignatureContext());
1204
1695
  }
1205
1696
  async dropFailedTxsFromP2P(failedTxs) {
1206
1697
  if (failedTxs.length === 0) {
@@ -1215,8 +1706,12 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1215
1706
  * Adds the proposed block to the archiver so it's available via P2P.
1216
1707
  * Gossip doesn't echo messages back to the sender, so the proposer's archiver/world-state
1217
1708
  * would never receive its own block without this explicit sync.
1709
+ *
1710
+ * In fisherman mode we skip this push: the fisherman builds blocks locally for validation
1711
+ * and fee analysis only, and pushing them to the archiver causes spurious reorg cascades
1712
+ * whenever the real proposer's block arrives from L1.
1218
1713
  */ async syncProposedBlockToArchiver(block) {
1219
- if (this.config.skipPushProposedBlocksToArchiver) {
1714
+ if (this.config.skipPushProposedBlocksToArchiver || this.config.fishermanMode) {
1220
1715
  this.log.warn(`Skipping push of proposed block ${block.number} to archiver`, {
1221
1716
  blockNumber: block.number,
1222
1717
  slot: block.header.globalVariables.slotNumber
@@ -1229,6 +1724,35 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1229
1724
  });
1230
1725
  await this.blockSink.addBlock(block);
1231
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
+ }
1232
1756
  /** Runs fee analysis and logs checkpoint outcome as fisherman */ async handleCheckpointEndAsFisherman(checkpoint) {
1233
1757
  // Perform L1 fee analysis before clearing requests
1234
1758
  // The callback is invoked asynchronously after the next block is mined
@@ -1268,20 +1792,8 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1268
1792
  }
1269
1793
  return false;
1270
1794
  }
1271
- /** Waits until a specific time within the current slot */ async waitUntilTimeInSlot(targetSecondsIntoSlot) {
1272
- const slotStartTimestamp = this.getSlotStartBuildTimestamp();
1273
- const targetTimestamp = slotStartTimestamp + targetSecondsIntoSlot;
1274
- await sleepUntil(new Date(targetTimestamp * 1000), this.dateProvider.nowAsDate());
1275
- }
1276
1795
  /** Waits the polling interval for transactions. Extracted for test overriding. */ async waitForTxsPollingInterval() {
1277
- await sleep(TXS_POLLING_MS);
1278
- }
1279
- getSlotStartBuildTimestamp() {
1280
- return getSlotStartBuildTimestamp(this.slot, this.l1Constants);
1281
- }
1282
- getSecondsIntoSlot() {
1283
- const slotStartTimestamp = this.getSlotStartBuildTimestamp();
1284
- return Number((this.dateProvider.now() / 1000 - slotStartTimestamp).toFixed(3));
1796
+ await this.awaitInterruptibleSleep(TXS_POLLING_MS);
1285
1797
  }
1286
1798
  getPublisher() {
1287
1799
  return this.publisher;