@aztec/sequencer-client 0.0.1-commit.c31f2472 → 0.0.1-commit.c52d6e7

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 (133) hide show
  1. package/README.md +281 -21
  2. package/dest/client/sequencer-client.d.ts +34 -11
  3. package/dest/client/sequencer-client.d.ts.map +1 -1
  4. package/dest/client/sequencer-client.js +77 -39
  5. package/dest/config.d.ts +33 -6
  6. package/dest/config.d.ts.map +1 -1
  7. package/dest/config.js +100 -46
  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 +13 -28
  15. package/dest/global_variable_builder/global_builder.d.ts.map +1 -1
  16. package/dest/global_variable_builder/global_builder.js +9 -66
  17. package/dest/global_variable_builder/index.d.ts +4 -2
  18. package/dest/global_variable_builder/index.d.ts.map +1 -1
  19. package/dest/global_variable_builder/index.js +2 -0
  20. package/dest/publisher/config.d.ts +51 -17
  21. package/dest/publisher/config.d.ts.map +1 -1
  22. package/dest/publisher/config.js +131 -42
  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 +11 -5
  44. package/dest/publisher/sequencer-publisher-factory.d.ts.map +1 -1
  45. package/dest/publisher/sequencer-publisher-factory.js +27 -3
  46. package/dest/publisher/sequencer-publisher.d.ts +123 -68
  47. package/dest/publisher/sequencer-publisher.d.ts.map +1 -1
  48. package/dest/publisher/sequencer-publisher.js +775 -364
  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 +696 -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 +99 -20
  62. package/dest/sequencer/checkpoint_proposal_job.d.ts.map +1 -1
  63. package/dest/sequencer/checkpoint_proposal_job.js +926 -286
  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/checkpoint_voter.d.ts +1 -2
  68. package/dest/sequencer/checkpoint_voter.d.ts.map +1 -1
  69. package/dest/sequencer/checkpoint_voter.js +2 -5
  70. package/dest/sequencer/events.d.ts +62 -5
  71. package/dest/sequencer/events.d.ts.map +1 -1
  72. package/dest/sequencer/metrics.d.ts +23 -8
  73. package/dest/sequencer/metrics.d.ts.map +1 -1
  74. package/dest/sequencer/metrics.js +117 -21
  75. package/dest/sequencer/missing_committee.d.ts +72 -0
  76. package/dest/sequencer/missing_committee.d.ts.map +1 -0
  77. package/dest/sequencer/missing_committee.js +139 -0
  78. package/dest/sequencer/requests_tracker.d.ts +22 -0
  79. package/dest/sequencer/requests_tracker.d.ts.map +1 -0
  80. package/dest/sequencer/requests_tracker.js +33 -0
  81. package/dest/sequencer/sequencer.d.ts +182 -44
  82. package/dest/sequencer/sequencer.d.ts.map +1 -1
  83. package/dest/sequencer/sequencer.js +623 -216
  84. package/dest/sequencer/types.d.ts +2 -2
  85. package/dest/sequencer/types.d.ts.map +1 -1
  86. package/dest/test/index.d.ts +5 -7
  87. package/dest/test/index.d.ts.map +1 -1
  88. package/dest/test/mock_checkpoint_builder.d.ts +12 -12
  89. package/dest/test/mock_checkpoint_builder.d.ts.map +1 -1
  90. package/dest/test/mock_checkpoint_builder.js +45 -36
  91. package/dest/test/utils.d.ts +17 -3
  92. package/dest/test/utils.d.ts.map +1 -1
  93. package/dest/test/utils.js +28 -9
  94. package/package.json +29 -29
  95. package/src/client/sequencer-client.ts +110 -47
  96. package/src/config.ts +127 -55
  97. package/src/global_variable_builder/README.md +44 -0
  98. package/src/global_variable_builder/fee_predictor.ts +182 -0
  99. package/src/global_variable_builder/fee_provider.ts +97 -0
  100. package/src/global_variable_builder/global_builder.ts +20 -89
  101. package/src/global_variable_builder/index.ts +3 -1
  102. package/src/publisher/config.ts +173 -40
  103. package/src/publisher/index.ts +3 -0
  104. package/src/publisher/l1_to_l2_messaging.ts +85 -0
  105. package/src/publisher/l1_tx_failed_store/factory.ts +32 -0
  106. package/src/publisher/l1_tx_failed_store/failed_tx_store.ts +164 -0
  107. package/src/publisher/l1_tx_failed_store/file_store_failed_tx_store.ts +47 -0
  108. package/src/publisher/l1_tx_failed_store/index.ts +3 -0
  109. package/src/publisher/sequencer-bundle-simulator.ts +254 -0
  110. package/src/publisher/sequencer-publisher-factory.ts +38 -9
  111. package/src/publisher/sequencer-publisher.ts +885 -461
  112. package/src/publisher/write_json.ts +78 -0
  113. package/src/sequencer/README.md +162 -450
  114. package/src/sequencer/automine/README.md +60 -0
  115. package/src/sequencer/automine/automine_factory.ts +152 -0
  116. package/src/sequencer/automine/automine_sequencer.ts +800 -0
  117. package/src/sequencer/automine/index.ts +6 -0
  118. package/src/sequencer/checkpoint_proposal_job.ts +1131 -325
  119. package/src/sequencer/checkpoint_proposal_job_metrics.ts +128 -0
  120. package/src/sequencer/checkpoint_voter.ts +1 -12
  121. package/src/sequencer/events.ts +66 -5
  122. package/src/sequencer/metrics.ts +132 -25
  123. package/src/sequencer/missing_committee.ts +192 -0
  124. package/src/sequencer/requests_tracker.ts +43 -0
  125. package/src/sequencer/sequencer.ts +737 -241
  126. package/src/sequencer/types.ts +1 -1
  127. package/src/test/index.ts +4 -6
  128. package/src/test/mock_checkpoint_builder.ts +65 -53
  129. package/src/test/utils.ts +64 -10
  130. package/dest/sequencer/timetable.d.ts +0 -90
  131. package/dest/sequencer/timetable.d.ts.map +0 -1
  132. package/dest/sequencer/timetable.js +0 -226
  133. package/src/sequencer/timetable.ts +0 -282
@@ -435,22 +435,24 @@ 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 { NUM_CHECKPOINT_END_MARKER_FIELDS, getNumBlockEndBlobFields } from '@aztec/blob-lib/encoding';
440
- import { BLOBS_PER_CHECKPOINT, FIELDS_PER_BLOB } from '@aztec/constants';
441
- import { BlockNumber } 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';
442
441
  import { randomInt } from '@aztec/foundation/crypto/random';
443
- import { Signature } from '@aztec/foundation/eth-signature';
442
+ import { flipSignature, generateRecoverableSignature, generateUnrecoverableSignature } from '@aztec/foundation/crypto/secp256k1-signer';
443
+ import { InterruptError, TimeoutError } from '@aztec/foundation/error';
444
444
  import { filter } from '@aztec/foundation/iterator';
445
445
  import { createLogger } from '@aztec/foundation/log';
446
- import { sleep, sleepUntil } from '@aztec/foundation/sleep';
446
+ import { InterruptibleSleep } from '@aztec/foundation/sleep';
447
447
  import { Timer } from '@aztec/foundation/timer';
448
- import { unfreeze } from '@aztec/foundation/types';
449
- import { CommitteeAttestationsAndSigners, MaliciousCommitteeAttestationsAndSigners } from '@aztec/stdlib/block';
450
- import { getSlotStartBuildTimestamp } from '@aztec/stdlib/epoch-helpers';
448
+ import { isErrorClass, unfreeze } from '@aztec/foundation/types';
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';
451
452
  import { Gas } from '@aztec/stdlib/gas';
453
+ import { InsufficientValidTxsError } from '@aztec/stdlib/interfaces/server';
452
454
  import { computeInHashFromL1ToL2Messages } from '@aztec/stdlib/messaging';
453
- import { orderAttestations } from '@aztec/stdlib/p2p';
455
+ import { orderAttestations, trimAttestations } from '@aztec/stdlib/p2p';
454
456
  import { AttestationTimeoutError } from '@aztec/stdlib/validators';
455
457
  import { Attributes, trackSpan } from '@aztec/telemetry-client';
456
458
  import { DutyAlreadySignedError, SlashingProtectionError } from '@aztec/validator-ha-signer/errors';
@@ -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.recordBlockProposalSuccess();
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,29 +934,75 @@ _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 previousCheckpoints = (await this.l2BlockSource.getCheckpointsForEpoch(this.epoch)).filter((c)=>c.number < this.checkpointNumber);
643
- const previousCheckpointOutHashes = previousCheckpoints.map((c)=>c.getCheckpointOutHash());
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);
644
1001
  const fork = _ts_add_disposable_resource(env, await this.worldState.fork(this.syncedToBlockNumber, {
645
1002
  closeDelayMs: 12_000
646
- }), false);
1003
+ }), true);
647
1004
  // Create checkpoint builder for the entire slot
648
- const checkpointBuilder = await this.checkpointsBuilder.startCheckpoint(this.checkpointNumber, checkpointGlobalVariables, l1ToL2Messages, previousCheckpointOutHashes, fork, this.log.getBindings());
1005
+ const checkpointBuilder = await this.checkpointsBuilder.startCheckpoint(this.checkpointNumber, checkpointGlobalVariables, feeAssetPriceModifier, l1ToL2Messages, previousCheckpointOutHashes, fork, this.log.getBindings());
649
1006
  // Options for the validator client when creating block and checkpoint proposals
650
1007
  const blockProposalOptions = {
651
1008
  publishFullTxs: !!this.config.publishTxsWithProposals,
@@ -653,10 +1010,11 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
653
1010
  };
654
1011
  const checkpointProposalOptions = {
655
1012
  publishFullTxs: !!this.config.publishTxsWithProposals,
656
- broadcastInvalidCheckpointProposal: this.config.broadcastInvalidBlockProposal
1013
+ broadcastInvalidCheckpointProposal: this.config.broadcastInvalidCheckpointProposalOnly || this.config.broadcastInvalidBlockProposal
657
1014
  };
658
1015
  let blocksInCheckpoint = [];
659
1016
  let blockPendingBroadcast = undefined;
1017
+ const checkpointBuildTimer = new Timer();
660
1018
  try {
661
1019
  // Main loop: build blocks for the checkpoint
662
1020
  const result = await this.buildBlocksForCheckpoint(checkpointBuilder, checkpointGlobalVariables.timestamp, inHash, blockProposalOptions);
@@ -666,107 +1024,162 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
666
1024
  // These errors are expected in HA mode, so we yield and let another HA node handle the slot
667
1025
  // The only distinction between the 2 errors is SlashingProtectionError throws when the payload is different,
668
1026
  // which is normal for block building (may have picked different txs)
669
- if (err instanceof DutyAlreadySignedError) {
670
- this.log.info(`Checkpoint proposal for slot ${this.slot} already signed by another HA node, yielding`, {
671
- slot: this.slot,
672
- signedByNode: err.signedByNode
673
- });
674
- return undefined;
675
- }
676
- if (err instanceof SlashingProtectionError) {
677
- this.log.info(`Checkpoint proposal for slot ${this.slot} blocked by slashing protection, yielding`, {
678
- slot: this.slot,
679
- existingMessageHash: err.existingMessageHash,
680
- attemptedMessageHash: err.attemptedMessageHash
681
- });
1027
+ if (this.handleHASigningError(err, 'Block proposal')) {
682
1028
  return undefined;
683
1029
  }
684
1030
  throw err;
685
1031
  }
686
1032
  if (blocksInCheckpoint.length === 0) {
687
- this.log.warn(`No blocks were built for slot ${this.slot}`, {
688
- 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'
689
1042
  });
690
1043
  this.eventEmitter.emit('checkpoint-empty', {
691
- slot: this.slot
1044
+ slot: this.targetSlot
1045
+ });
1046
+ return undefined;
1047
+ }
1048
+ const minBlocksForCheckpoint = this.config.minBlocksForCheckpoint;
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
+ });
1057
+ this.log.warn(`Checkpoint has fewer blocks than minimum (${blocksInCheckpoint.length} < ${minBlocksForCheckpoint}), skipping proposal`, {
1058
+ slot: this.targetSlot,
1059
+ checkpointNumber: this.checkpointNumber,
1060
+ blocksBuilt: blocksInCheckpoint.length,
1061
+ minBlocksForCheckpoint,
1062
+ reason: 'min_blocks_not_met'
692
1063
  });
693
1064
  return undefined;
694
1065
  }
695
1066
  // Assemble and broadcast the checkpoint proposal, including the last block that was not
696
1067
  // broadcasted yet, and wait to collect the committee attestations.
697
- this.setStateFn(SequencerState.ASSEMBLING_CHECKPOINT, this.slot);
1068
+ this.setState(SequencerState.ASSEMBLING_CHECKPOINT);
698
1069
  const checkpoint = await checkpointBuilder.completeCheckpoint();
699
- // Do not collect attestations nor publish to L1 in fisherman mode
1070
+ // Final validation: per-block limits are only checked if the operator set them explicitly.
1071
+ // Otherwise, checkpoint-level budgets were already enforced by the redistribution logic.
1072
+ try {
1073
+ validateCheckpoint(checkpoint, {
1074
+ rollupManaLimit: this.l1Constants.rollupManaLimit,
1075
+ maxL2BlockGas: this.config.maxL2BlockGas,
1076
+ maxDABlockGas: this.config.maxDABlockGas,
1077
+ maxTxsPerBlock: this.config.maxTxsPerBlock,
1078
+ maxTxsPerCheckpoint: this.config.maxTxsPerCheckpoint
1079
+ });
1080
+ } catch (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',
1093
+ checkpoint: checkpoint.header.toInspect()
1094
+ });
1095
+ return undefined;
1096
+ }
1097
+ // Record checkpoint-level build metrics
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
700
1111
  if (this.config.fishermanMode) {
701
- this.log.info(`Built checkpoint for slot ${this.slot} with ${blocksInCheckpoint.length} blocks. ` + `Skipping proposal in fisherman mode.`, {
702
- 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,
703
1114
  checkpoint: checkpoint.header.toInspect(),
704
1115
  blocksBuilt: blocksInCheckpoint.length
705
1116
  });
706
1117
  this.metrics.recordCheckpointSuccess();
707
- 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
+ };
708
1124
  }
709
- // Include the block pending broadcast in the checkpoint proposal if any
710
- const lastBlock = blockPendingBroadcast && {
711
- blockHeader: blockPendingBroadcast.block.header,
712
- indexWithinCheckpoint: blockPendingBroadcast.block.indexWithinCheckpoint,
713
- txs: blockPendingBroadcast.txs
714
- };
715
- // Create the checkpoint proposal and broadcast it
716
- const proposal = await this.validatorClient.createCheckpointProposal(checkpoint.header, checkpoint.archive.root, lastBlock, this.proposer, checkpointProposalOptions);
717
- const blockProposedAt = this.dateProvider.now();
718
- await this.p2pClient.broadcastCheckpointProposal(proposal);
719
- this.setStateFn(SequencerState.COLLECTING_ATTESTATIONS, this.slot);
720
- const attestations = await this.waitForAttestations(proposal);
721
- const blockAttestedAt = this.dateProvider.now();
722
- this.metrics.recordCheckpointAttestationDelay(blockAttestedAt - blockProposedAt);
723
- // Proposer must sign over the attestations before pushing them to L1
724
- const signer = this.proposer ?? this.publisher.getSenderAddress();
725
- 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.
726
1128
  try {
727
- attestationsSignature = await this.validatorClient.signAttestationsAndSigners(attestations, signer, this.slot, this.checkpointNumber);
1129
+ await this.publisher.validateBlockHeader(checkpoint.header, this.checkpointSimulationOverridesPlan);
728
1130
  } catch (err) {
729
- // We shouldn't really get here since we yield to another HA node
730
- // as soon as we see these errors when creating block proposals.
731
- if (err instanceof DutyAlreadySignedError) {
732
- this.log.info(`Attestations signature for slot ${this.slot} already signed by another HA node, yielding`, {
733
- slot: this.slot,
734
- signedByNode: err.signedByNode
735
- });
736
- return undefined;
737
- }
738
- if (err instanceof SlashingProtectionError) {
739
- this.log.info(`Attestations signature for slot ${this.slot} blocked by slashing protection, yielding`, {
740
- slot: this.slot,
741
- existingMessageHash: err.existingMessageHash,
742
- attemptedMessageHash: err.attemptedMessageHash
743
- });
744
- return undefined;
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;
1142
+ }
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);
745
1159
  }
746
- throw err;
1160
+ } else if (!this.config.skipBroadcastProposals) {
1161
+ await this.p2pClient.broadcastCheckpointProposal(proposal);
1162
+ this.checkpointMetrics.noteCheckpointBroadcast(this.dateProvider.now());
747
1163
  }
748
- // Enqueue publishing the checkpoint to L1
749
- this.setStateFn(SequencerState.PUBLISHING_CHECKPOINT, this.slot);
750
- const aztecSlotDuration = this.l1Constants.slotDuration;
751
- const slotStartBuildTimestamp = this.getSlotStartBuildTimestamp();
752
- const txTimeoutAt = new Date((slotStartBuildTimestamp + aztecSlotDuration) * 1000);
753
- await this.publisher.enqueueProposeCheckpoint(checkpoint, attestations, attestationsSignature, {
754
- txTimeoutAt,
755
- forcePendingCheckpointNumber: this.invalidateCheckpoint?.forcePendingCheckpointNumber
756
- });
757
- return checkpoint;
1164
+ // Return immediately after broadcast attestation collection happens in the background
1165
+ return {
1166
+ checkpoint,
1167
+ proposal,
1168
+ blockProposedAt
1169
+ };
758
1170
  } catch (e) {
759
1171
  env.error = e;
760
1172
  env.hasError = true;
761
1173
  } finally{
762
- _ts_dispose_resources(env);
1174
+ const result = _ts_dispose_resources(env);
1175
+ if (result) await result;
763
1176
  }
764
1177
  } catch (err) {
765
1178
  if (err && (err instanceof DutyAlreadySignedError || err instanceof SlashingProtectionError)) {
766
1179
  // swallow this error. It's already been logged by a function deeper in the stack
767
1180
  return undefined;
768
1181
  }
769
- this.log.error(`Error building checkpoint at slot ${this.slot}`, err);
1182
+ this.log.error(`Error building checkpoint at slot ${this.targetSlot}`, err);
770
1183
  return undefined;
771
1184
  }
772
1185
  }
@@ -776,21 +1189,27 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
776
1189
  const blocksInCheckpoint = [];
777
1190
  const txHashesAlreadyIncluded = new Set();
778
1191
  const initialBlockNumber = BlockNumber(this.syncedToBlockNumber + 1);
779
- // Remaining blob fields available for blocks (checkpoint end marker already subtracted)
780
- let remainingBlobFields = BLOBS_PER_CHECKPOINT * FIELDS_PER_BLOB - NUM_CHECKPOINT_END_MARKER_FIELDS;
781
1192
  // Last block in the checkpoint will usually be flagged as pending broadcast, so we send it along with the checkpoint proposal
782
1193
  let blockPendingBroadcast = undefined;
783
1194
  while(true){
784
1195
  const blocksBuilt = blocksInCheckpoint.length;
785
- const indexWithinCheckpoint = blocksBuilt;
1196
+ const indexWithinCheckpoint = IndexWithinCheckpoint(blocksBuilt);
786
1197
  const blockNumber = BlockNumber(initialBlockNumber + blocksBuilt);
787
- const secondsIntoSlot = this.getSecondsIntoSlot();
788
- 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);
789
1208
  if (!timingInfo.canStart) {
790
1209
  this.log.debug(`Not enough time left in slot to start another block`, {
791
- slot: this.slot,
1210
+ slot: this.targetSlot,
792
1211
  blocksBuilt,
793
- secondsIntoSlot
1212
+ nowSeconds
794
1213
  });
795
1214
  break;
796
1215
  }
@@ -799,71 +1218,69 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
799
1218
  blockTimestamp: timestamp,
800
1219
  // Create an empty block if we haven't already and this is the last one
801
1220
  forceCreate: timingInfo.isLastBlock && blocksBuilt === 0 && this.config.buildCheckpointIfEmpty,
802
- // Build deadline is only set if we are enforcing the timetable
803
- buildDeadline: timingInfo.deadline ? new Date((this.getSlotStartBuildTimestamp() + timingInfo.deadline) * 1000) : undefined,
1221
+ buildDeadline: new Date(timingInfo.deadline * 1000),
804
1222
  blockNumber,
805
1223
  indexWithinCheckpoint,
806
- txHashesAlreadyIncluded,
807
- remainingBlobFields
1224
+ txHashesAlreadyIncluded
808
1225
  });
809
- if (!buildResult && timingInfo.isLastBlock) {
810
- break;
811
- } else if (!buildResult && timingInfo.deadline !== undefined) {
812
- // 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
813
1233
  await this.waitUntilNextSubslot(timingInfo.deadline);
814
1234
  continue;
815
- } else if (!buildResult) {
816
- break;
817
- } else if ('error' in buildResult) {
818
- // 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) {
819
1239
  if (!(buildResult.error instanceof SequencerInterruptedError)) {
820
- this.log.warn(`Halting block building for slot ${this.slot}`, {
821
- slot: this.slot,
1240
+ this.log.warn(`Halting block building for slot ${this.targetSlot}`, {
1241
+ slot: this.targetSlot,
822
1242
  blocksBuilt,
823
1243
  error: buildResult.error
824
1244
  });
825
1245
  }
826
1246
  break;
827
1247
  }
828
- const { block, usedTxs, remainingBlobFields: newRemainingBlobFields } = buildResult;
829
- blocksInCheckpoint.push(block);
830
- // Update remaining blob fields for the next block
831
- remainingBlobFields = newRemainingBlobFields;
832
- // Sync the proposed block to the archiver to make it available
833
- // Note that the checkpoint builder uses its own fork so it should not need to wait for this syncing
834
- // Eventually we should refactor the checkpoint builder to not need a separate long-lived fork
835
- // Fire and forget - don't block the critical path, but log errors
836
- this.syncProposedBlockToArchiver(block).catch((err)=>{
837
- this.log.error(`Failed to sync proposed block ${block.number} to archiver`, {
838
- blockNumber: block.number,
839
- err
840
- });
1248
+ const { block, usedTxs } = buildResult;
1249
+ this.checkpointMetrics.noteCheckpointBlockBuilt(this.dateProvider.now(), {
1250
+ isFirstBlock: blocksBuilt === 0,
1251
+ isLastBlock: timingInfo.isLastBlock
841
1252
  });
1253
+ blocksInCheckpoint.push(block);
842
1254
  usedTxs.forEach((tx)=>txHashesAlreadyIncluded.add(tx.txHash.toString()));
843
- // If this is the last block, exit the loop now so we 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.
844
1266
  if (timingInfo.isLastBlock) {
845
- this.log.verbose(`Completed final block ${blockNumber} for slot ${this.slot}`, {
846
- slot: this.slot,
1267
+ this.log.verbose(`Completed final block ${blockNumber} for slot ${this.targetSlot}`, {
1268
+ slot: this.targetSlot,
847
1269
  blockNumber,
848
1270
  blocksBuilt
849
1271
  });
850
- blockPendingBroadcast = {
851
- block,
852
- txs: usedTxs
853
- };
1272
+ blockPendingBroadcast = proposal;
854
1273
  break;
855
1274
  }
856
- // For non-last blocks, broadcast the block proposal (unless we're in fisherman mode)
857
- // If the block is the last one, we'll broadcast it along with the checkpoint at the end of the loop
858
- if (!this.config.fishermanMode) {
859
- const proposal = await this.validatorClient.createBlockProposal(block.header, block.indexWithinCheckpoint, inHash, block.archive.root, usedTxs, this.proposer, blockProposalOptions);
1275
+ // Once we have a signed proposal and the archiver agreed with our proposed block, then we broadcast it.
1276
+ if (proposal && !this.config.skipBroadcastProposals) {
860
1277
  await this.p2pClient.broadcastProposal(proposal);
861
1278
  }
862
1279
  // Wait until the next block's start time
863
1280
  await this.waitUntilNextSubslot(timingInfo.deadline);
864
1281
  }
865
- this.log.verbose(`Block building loop completed for slot ${this.slot}`, {
866
- slot: this.slot,
1282
+ this.log.verbose(`Block building loop completed for slot ${this.targetSlot}`, {
1283
+ slot: this.targetSlot,
867
1284
  blocksBuilt: blocksInCheckpoint.length
868
1285
  });
869
1286
  return {
@@ -871,115 +1288,167 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
871
1288
  blockPendingBroadcast
872
1289
  };
873
1290
  }
874
- /** Sleeps until it is time to produce the next block in the slot */ async waitUntilNextSubslot(nextSubslotStart) {
875
- this.setStateFn(SequencerState.WAITING_UNTIL_NEXT_BLOCK, this.slot);
876
- this.log.verbose(`Waiting until time for the next block at ${nextSubslotStart}s into slot`, {
877
- slot: this.slot
1291
+ /** Creates a block proposal for a given block via the validator client (unless in fisherman mode) */ createBlockProposal(block, inHash, usedTxs, blockProposalOptions) {
1292
+ if (this.config.fishermanMode) {
1293
+ this.log.info(`Skipping block proposal for block ${block.number} in fisherman mode`);
1294
+ return Promise.resolve(undefined);
1295
+ }
1296
+ return this.validatorClient.createBlockProposal(block.header, this.checkpointNumber, block.indexWithinCheckpoint, inHash, block.archive.root, usedTxs, this.proposer, blockProposalOptions);
1297
+ }
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
878
1305
  });
879
- await this.waitUntilTimeInSlot(nextSubslotStart);
1306
+ await this.awaitInterruptibleSleep(Math.max(0, nextSubslotStart * 1000 - this.dateProvider.now()));
880
1307
  }
881
1308
  /** Builds a single block. Called from the main block building loop. */ async buildSingleBlock(checkpointBuilder, opts) {
882
- const { blockTimestamp, forceCreate, blockNumber, indexWithinCheckpoint, buildDeadline, txHashesAlreadyIncluded, remainingBlobFields } = opts;
883
- this.log.verbose(`Preparing block ${blockNumber} index ${indexWithinCheckpoint} at checkpoint ${this.checkpointNumber} for slot ${this.slot}`, {
1309
+ const { blockTimestamp, forceCreate, blockNumber, indexWithinCheckpoint, buildDeadline, txHashesAlreadyIncluded } = opts;
1310
+ this.log.verbose(`Preparing block ${blockNumber} index ${indexWithinCheckpoint} at checkpoint ${this.checkpointNumber} for slot ${this.targetSlot}`, {
884
1311
  ...checkpointBuilder.getConstantData(),
885
1312
  ...opts
886
1313
  });
887
1314
  try {
888
1315
  // Wait until we have enough txs to build the block
889
- const minTxs = this.config.minTxsPerBlock;
890
- const { availableTxs, canStartBuilding } = await this.waitForMinTxs(opts);
1316
+ const { canStartBuilding, minTxs } = await this.waitForMinTxs(opts);
891
1317
  if (!canStartBuilding) {
892
- 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',
893
1320
  blockNumber,
894
- slot: this.slot,
895
- indexWithinCheckpoint
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',
1328
+ blockNumber,
1329
+ slot: this.targetSlot,
1330
+ checkpointNumber: this.checkpointNumber,
1331
+ indexWithinCheckpoint,
1332
+ minTxs
896
1333
  });
897
1334
  this.eventEmitter.emit('block-tx-count-check-failed', {
898
1335
  minTxs,
899
- availableTxs,
900
- slot: this.slot
1336
+ slot: this.targetSlot
901
1337
  });
902
1338
  this.metrics.recordBlockProposalFailed('insufficient_txs');
903
- return undefined;
1339
+ return {
1340
+ failure: 'insufficient-txs'
1341
+ };
904
1342
  }
905
1343
  // Create iterator to pending txs. We filter out txs already included in previous blocks in the checkpoint
906
1344
  // just in case p2p failed to sync the provisional block and didn't get to remove those txs from the mempool yet.
907
- const pendingTxs = filter(this.p2pClient.iteratePendingTxs(), (tx)=>!txHashesAlreadyIncluded.has(tx.txHash.toString()));
908
- this.log.debug(`Building block ${blockNumber} at index ${indexWithinCheckpoint} for slot ${this.slot} with ${availableTxs} available txs`, {
909
- 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,
910
1352
  blockNumber,
911
1353
  indexWithinCheckpoint
912
1354
  });
913
- this.setStateFn(SequencerState.CREATING_BLOCK, this.slot);
914
- // Calculate blob fields limit for txs (remaining capacity - this block's end overhead)
915
- const blockEndOverhead = getNumBlockEndBlobFields(indexWithinCheckpoint === 0);
916
- const maxBlobFieldsForTxs = remainingBlobFields - blockEndOverhead;
1355
+ this.setState(SequencerState.CREATING_BLOCK);
1356
+ // Per-block limits are operator overrides (from SEQ_MAX_L2_BLOCK_GAS etc.) further capped
1357
+ // by remaining checkpoint-level budgets inside CheckpointBuilder before each block is built.
1358
+ // minValidTxs is passed into the builder so it can reject the block *before* updating state.
1359
+ const minValidTxs = forceCreate ? 0 : this.config.minValidTxsPerBlock ?? minTxs;
917
1360
  const blockBuilderOptions = {
918
1361
  maxTransactions: this.config.maxTxsPerBlock,
919
- maxBlockSize: this.config.maxBlockSizeInBytes,
920
- maxBlockGas: new Gas(this.config.maxDABlockGas, this.config.maxL2BlockGas),
921
- maxBlobFields: maxBlobFieldsForTxs,
922
- deadline: buildDeadline
1362
+ maxBlockGas: this.config.maxL2BlockGas !== undefined || this.config.maxDABlockGas !== undefined ? new Gas(this.config.maxDABlockGas ?? Infinity, this.config.maxL2BlockGas ?? Infinity) : undefined,
1363
+ deadline: buildDeadline,
1364
+ isBuildingProposal: true,
1365
+ minValidTxs,
1366
+ maxBlocksPerCheckpoint: this.timetable.getMaxBlocksPerCheckpoint(),
1367
+ perBlockAllocationMultiplier: this.config.perBlockAllocationMultiplier,
1368
+ perBlockDAAllocationMultiplier: this.config.perBlockDAAllocationMultiplier
923
1369
  };
924
- // Actually build the block by executing txs
925
- const workTimer = new Timer();
926
- const { publicGas, block, publicProcessorDuration, numTxs, blockBuildingTimer, usedTxs, failedTxs, usedTxBlobFields } = await checkpointBuilder.buildBlock(pendingTxs, blockNumber, blockTimestamp, blockBuilderOptions);
927
- const blockBuildDuration = workTimer.ms();
1370
+ // Actually build the block by executing txs. The builder throws InsufficientValidTxsError
1371
+ // if the number of successfully processed txs is below minValidTxs, ensuring state is not
1372
+ // updated for blocks that will be discarded.
1373
+ const buildResult = await this.buildSingleBlockWithCheckpointBuilder(checkpointBuilder, pendingTxs, blockNumber, blockTimestamp, blockBuilderOptions);
928
1374
  // If any txs failed during execution, drop them from the mempool so we don't pick them up again
929
- await this.dropFailedTxsFromP2P(failedTxs);
930
- // Check if we have created a block with enough txs. If there were invalid txs in the pool, or if execution took
931
- // too long, then we may not get to minTxsPerBlock after executing public functions.
932
- const minValidTxs = this.config.minValidTxsPerBlock ?? minTxs;
933
- if (!forceCreate && numTxs < minValidTxs) {
934
- this.log.warn(`Block ${blockNumber} at index ${indexWithinCheckpoint} on slot ${this.slot} has too few valid txs to be proposed (got ${numTxs} but required ${minValidTxs})`, {
935
- slot: this.slot,
1375
+ await this.dropFailedTxsFromP2P(buildResult.failedTxs);
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,
936
1381
  blockNumber,
937
- numTxs,
938
- indexWithinCheckpoint
1382
+ numTxs: buildResult.processedCount,
1383
+ indexWithinCheckpoint,
1384
+ minValidTxs
939
1385
  });
940
- this.eventEmitter.emit('block-tx-count-check-failed', {
941
- minTxs: minValidTxs,
942
- availableTxs: numTxs,
943
- slot: this.slot
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,
1390
+ blockNumber,
1391
+ numTxs: buildResult.processedCount,
1392
+ indexWithinCheckpoint,
1393
+ minValidTxs
1394
+ });
1395
+ this.eventEmitter.emit('block-build-failed', {
1396
+ reason: `Insufficient valid txs`,
1397
+ slot: this.targetSlot
944
1398
  });
945
1399
  this.metrics.recordBlockProposalFailed('insufficient_valid_txs');
946
- return undefined;
1400
+ return {
1401
+ failure: 'insufficient-valid-txs'
1402
+ };
947
1403
  }
948
1404
  // Block creation succeeded, emit stats and metrics
1405
+ const { block, publicProcessorDuration, usedTxs, blockBuildDuration, numTxs } = buildResult;
949
1406
  const blockStats = {
950
1407
  eventName: 'l2-block-built',
951
1408
  duration: blockBuildDuration,
952
1409
  publicProcessDuration: publicProcessorDuration,
953
- rollupCircuitsDuration: blockBuildingTimer.ms(),
954
1410
  ...block.getStats()
955
1411
  };
956
1412
  const blockHash = await block.hash();
957
1413
  const txHashes = block.body.txEffects.map((tx)=>tx.txHash);
958
- const manaPerSec = publicGas.l2Gas / (blockBuildDuration / 1000);
959
- this.log.info(`Built block ${block.number} at checkpoint ${this.checkpointNumber} for slot ${this.slot} with ${numTxs} txs`, {
1414
+ const manaPerSec = block.header.totalManaUsed.toNumberUnsafe() / (blockBuildDuration / 1000);
1415
+ this.log.info(`Built block ${block.number} at checkpoint ${this.checkpointNumber} for slot ${this.targetSlot} with ${numTxs} txs`, {
960
1416
  blockHash,
961
1417
  txHashes,
962
1418
  manaPerSec,
963
1419
  ...blockStats
964
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.
965
1423
  this.eventEmitter.emit('block-proposed', {
966
1424
  blockNumber: block.number,
967
- slot: this.slot
1425
+ blockHash,
1426
+ checkpointNumber: this.checkpointNumber,
1427
+ indexWithinCheckpoint: block.indexWithinCheckpoint,
1428
+ slot: this.targetSlot,
1429
+ buildSlot: this.getBuildSlot()
968
1430
  });
969
- this.metrics.recordBuiltBlock(blockBuildDuration, publicGas.l2Gas);
1431
+ this.metrics.recordBuiltBlock(blockBuildDuration, block.header.totalManaUsed.toNumberUnsafe(), this.targetSlot);
970
1432
  return {
971
1433
  block,
972
- usedTxs,
973
- remainingBlobFields: maxBlobFieldsForTxs - usedTxBlobFields
1434
+ usedTxs
974
1435
  };
975
1436
  } catch (err) {
976
1437
  this.eventEmitter.emit('block-build-failed', {
977
1438
  reason: err.message,
978
- 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
979
1446
  });
980
1447
  this.log.error(`Error building block`, err, {
981
- blockNumber,
982
- slot: this.slot
1448
+ reason: err instanceof Error ? err.message : String(err),
1449
+ slot: this.targetSlot,
1450
+ checkpointNumber: this.checkpointNumber,
1451
+ blockNumber
983
1452
  });
984
1453
  this.metrics.recordBlockProposalFailed(err.name || 'unknown_error');
985
1454
  this.metrics.recordFailedBlock();
@@ -988,43 +1457,90 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
988
1457
  };
989
1458
  }
990
1459
  }
1460
+ /** Uses the checkpoint builder to build a block, catching InsufficientValidTxsError. */ async buildSingleBlockWithCheckpointBuilder(checkpointBuilder, pendingTxs, blockNumber, blockTimestamp, blockBuilderOptions) {
1461
+ try {
1462
+ const workTimer = new Timer();
1463
+ const result = await checkpointBuilder.buildBlock(pendingTxs, blockNumber, blockTimestamp, blockBuilderOptions);
1464
+ const blockBuildDuration = workTimer.ms();
1465
+ return {
1466
+ ...result,
1467
+ blockBuildDuration,
1468
+ status: 'success'
1469
+ };
1470
+ } catch (err) {
1471
+ if (isErrorClass(err, InsufficientValidTxsError)) {
1472
+ return {
1473
+ failedTxs: err.failedTxs,
1474
+ processedCount: err.processedCount,
1475
+ status: 'insufficient-valid-txs'
1476
+ };
1477
+ }
1478
+ throw err;
1479
+ }
1480
+ }
991
1481
  /** Waits until minTxs are available on the pool for building a block. */ async waitForMinTxs(opts) {
992
- const minTxs = this.config.minTxsPerBlock;
993
1482
  const { indexWithinCheckpoint, blockNumber, buildDeadline, forceCreate } = opts;
994
- // Deadline is undefined if we are not enforcing the timetable, meaning we'll exit immediately when out of time
995
- const startBuildingDeadline = buildDeadline ? new Date(buildDeadline.getTime() - this.timetable.minExecutionTime * 1000) : undefined;
996
- let availableTxs = await this.p2pClient.getPendingTxCount();
997
- while(!forceCreate && availableTxs < minTxs){
1483
+ // We only allow a block with 0 txs in the first block of the checkpoint
1484
+ const minTxs = indexWithinCheckpoint > 0 && this.config.minTxsPerBlock === 0 ? 1 : this.config.minTxsPerBlock;
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)){
998
1490
  // If we're past deadline, or we have no deadline, give up
999
1491
  const now = this.dateProvider.nowAsDate();
1000
1492
  if (startBuildingDeadline === undefined || now >= startBuildingDeadline) {
1001
1493
  return {
1002
1494
  canStartBuilding: false,
1003
- availableTxs: availableTxs
1495
+ minTxs
1004
1496
  };
1005
1497
  }
1006
1498
  // Wait a bit before checking again
1007
- this.setStateFn(SequencerState.WAITING_FOR_TXS, this.slot);
1008
- 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}`, {
1009
1501
  blockNumber,
1010
- slot: this.slot,
1011
- indexWithinCheckpoint
1502
+ slot: this.targetSlot,
1503
+ indexWithinCheckpoint,
1504
+ minTxs
1012
1505
  });
1013
- await sleep(TXS_POLLING_MS);
1014
- availableTxs = await this.p2pClient.getPendingTxCount();
1506
+ await this.waitForTxsPollingInterval();
1015
1507
  }
1016
1508
  return {
1017
1509
  canStartBuilding: true,
1018
- availableTxs
1510
+ minTxs
1019
1511
  };
1020
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
+ }
1021
1537
  /**
1022
1538
  * Waits for enough attestations to be collected via p2p.
1023
1539
  * This is run after all blocks for the checkpoint have been built.
1024
1540
  */ async waitForAttestations(proposal) {
1025
1541
  if (this.config.fishermanMode) {
1026
1542
  this.log.debug('Skipping attestation collection in fisherman mode');
1027
- return CommitteeAttestationsAndSigners.empty();
1543
+ return CommitteeAttestationsAndSigners.empty(this.getSignatureContext());
1028
1544
  }
1029
1545
  const slotNumber = proposal.slotNumber;
1030
1546
  const { committee, seed, epoch } = await this.epochCache.getCommittee(slotNumber);
@@ -1032,47 +1548,92 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1032
1548
  throw new Error('No committee when collecting attestations');
1033
1549
  } else if (committee.length === 0) {
1034
1550
  this.log.verbose(`Attesting committee is empty`);
1035
- return CommitteeAttestationsAndSigners.empty();
1551
+ return CommitteeAttestationsAndSigners.empty(this.getSignatureContext());
1036
1552
  } else {
1037
1553
  this.log.debug(`Attesting committee length is ${committee.length}`, {
1038
1554
  committee
1039
1555
  });
1040
1556
  }
1041
- const numberOfRequiredAttestations = Math.floor(committee.length * 2 / 3) + 1;
1557
+ const numberOfRequiredAttestations = computeQuorum(committee.length);
1042
1558
  if (this.config.skipCollectingAttestations) {
1043
1559
  this.log.warn('Skipping attestation collection as per config (attesting with own keys only)');
1044
- const attestations = await this.validatorClient?.collectOwnAttestations(proposal);
1045
- 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());
1046
1565
  }
1047
- const attestationTimeAllowed = this.config.enforceTimeTable ? this.timetable.getMaxAllowedTime(SequencerState.PUBLISHING_CHECKPOINT) : this.l1Constants.slotDuration;
1048
- const attestationDeadline = new Date((this.getSlotStartBuildTimestamp() + attestationTimeAllowed) * 1000);
1049
- 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()));
1050
1570
  const collectAttestationsTimer = new Timer();
1051
1571
  let collectedAttestationsCount = 0;
1052
1572
  try {
1053
- const attestations = await this.validatorClient.collectAttestations(proposal, numberOfRequiredAttestations, attestationDeadline);
1573
+ const attestations = await this.validatorClient.collectAttestations(proposal, numberOfRequiredAttestations, attestationDeadline, this.checkpointNumber);
1054
1574
  collectedAttestationsCount = attestations.length;
1575
+ // Trim attestations to minimum required to save L1 calldata gas
1576
+ const localAddresses = this.validatorClient.getValidatorAddresses();
1577
+ const trimmed = trimAttestations(attestations, numberOfRequiredAttestations, this.attestorAddress, localAddresses);
1578
+ if (trimmed.length < attestations.length) {
1579
+ this.log.debug(`Trimmed attestations from ${attestations.length} to ${trimmed.length} for L1 submission`);
1580
+ }
1055
1581
  // Rollup contract requires that the signatures are provided in the order of the committee
1056
- const sorted = orderAttestations(attestations, committee);
1582
+ const sorted = orderAttestations(trimmed, committee);
1583
+ this.logCheckpointAttestations('collected', committee, attestations, numberOfRequiredAttestations, {
1584
+ submittedCount: trimmed.length
1585
+ });
1057
1586
  // Manipulate the attestations if we've been configured to do so
1058
- if (this.config.injectFakeAttestation || this.config.shuffleAttestationOrdering) {
1587
+ if (this.config.injectFakeAttestation || this.config.injectHighSValueAttestation || this.config.injectUnrecoverableSignatureAttestation || this.config.injectYParityAttestation || this.config.shuffleAttestationOrdering) {
1059
1588
  return this.manipulateAttestations(proposal.slotNumber, epoch, seed, committee, sorted);
1060
1589
  }
1061
- return new CommitteeAttestationsAndSigners(sorted);
1590
+ return new CommitteeAttestationsAndSigners(sorted, this.getSignatureContext());
1062
1591
  } catch (err) {
1063
1592
  if (err && err instanceof AttestationTimeoutError) {
1064
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);
1065
1605
  }
1066
- throw err;
1606
+ return undefined;
1067
1607
  } finally{
1068
1608
  this.metrics.recordCollectedAttestations(collectedAttestationsCount, collectAttestationsTimer.ms());
1069
1609
  }
1070
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
+ }
1071
1632
  /** Breaks the attestations before publishing based on attack configs */ manipulateAttestations(slotNumber, epoch, seed, committee, attestations) {
1072
1633
  // Compute the proposer index in the committee, since we dont want to tweak it.
1073
1634
  // Otherwise, the L1 rollup contract will reject the block outright.
1074
1635
  const proposerIndex = Number(this.epochCache.computeProposerIndex(slotNumber, epoch, seed, BigInt(committee.length)));
1075
- if (this.config.injectFakeAttestation) {
1636
+ if (this.config.injectFakeAttestation || this.config.injectHighSValueAttestation || this.config.injectUnrecoverableSignatureAttestation) {
1076
1637
  // Find non-empty attestations that are not from the proposer
1077
1638
  const nonProposerIndices = [];
1078
1639
  for(let i = 0; i < attestations.length; i++){
@@ -1082,28 +1643,55 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1082
1643
  }
1083
1644
  if (nonProposerIndices.length > 0) {
1084
1645
  const targetIndex = nonProposerIndices[randomInt(nonProposerIndices.length)];
1085
- this.log.warn(`Injecting fake attestation in checkpoint for slot ${slotNumber} at index ${targetIndex}`);
1086
- unfreeze(attestations[targetIndex]).signature = Signature.random();
1646
+ if (this.config.injectHighSValueAttestation) {
1647
+ this.log.warn(`Injecting high-s value attestation in checkpoint for slot ${slotNumber} at index ${targetIndex}`);
1648
+ unfreeze(attestations[targetIndex]).signature = flipSignature(attestations[targetIndex].signature);
1649
+ } else if (this.config.injectUnrecoverableSignatureAttestation) {
1650
+ this.log.warn(`Injecting unrecoverable signature attestation in checkpoint for slot ${slotNumber} at index ${targetIndex}`);
1651
+ unfreeze(attestations[targetIndex]).signature = generateUnrecoverableSignature();
1652
+ } else {
1653
+ this.log.warn(`Injecting fake attestation in checkpoint for slot ${slotNumber} at index ${targetIndex}`);
1654
+ unfreeze(attestations[targetIndex]).signature = generateRecoverableSignature();
1655
+ }
1087
1656
  }
1088
- 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());
1089
1666
  }
1090
1667
  if (this.config.shuffleAttestationOrdering) {
1091
1668
  this.log.warn(`Shuffling attestation ordering in checkpoint for slot ${slotNumber} (proposer #${proposerIndex})`);
1092
1669
  const shuffled = [
1093
1670
  ...attestations
1094
1671
  ];
1095
- const [i, j] = [
1096
- (proposerIndex + 1) % shuffled.length,
1097
- (proposerIndex + 2) % shuffled.length
1098
- ];
1099
- const valueI = shuffled[i];
1100
- const valueJ = shuffled[j];
1101
- shuffled[i] = valueJ;
1102
- shuffled[j] = valueI;
1103
- const signers = new CommitteeAttestationsAndSigners(attestations).getSigners();
1104
- 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());
1105
1693
  }
1106
- return new CommitteeAttestationsAndSigners(attestations);
1694
+ return new CommitteeAttestationsAndSigners(attestations, this.getSignatureContext());
1107
1695
  }
1108
1696
  async dropFailedTxsFromP2P(failedTxs) {
1109
1697
  if (failedTxs.length === 0) {
@@ -1111,15 +1699,27 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1111
1699
  }
1112
1700
  const failedTxData = failedTxs.map((fail)=>fail.tx);
1113
1701
  const failedTxHashes = failedTxData.map((tx)=>tx.getTxHash());
1114
- this.log.verbose(`Dropping failed txs ${failedTxHashes.join(', ')}`);
1115
- await this.p2pClient.deleteTxs(failedTxHashes);
1702
+ const failures = failedTxs.map((fail)=>({
1703
+ txHash: fail.tx.getTxHash().toString(),
1704
+ reason: fail.error.message
1705
+ }));
1706
+ this.log.warn(`Dropping ${failedTxs.length} txs from mempool due to failures during block building for slot ${this.targetSlot}`, {
1707
+ slot: this.targetSlot,
1708
+ checkpointNumber: this.checkpointNumber,
1709
+ failures
1710
+ });
1711
+ await this.p2pClient.handleFailedExecution(failedTxHashes);
1116
1712
  }
1117
1713
  /**
1118
1714
  * Adds the proposed block to the archiver so it's available via P2P.
1119
1715
  * Gossip doesn't echo messages back to the sender, so the proposer's archiver/world-state
1120
1716
  * would never receive its own block without this explicit sync.
1717
+ *
1718
+ * In fisherman mode we skip this push: the fisherman builds blocks locally for validation
1719
+ * and fee analysis only, and pushing them to the archiver causes spurious reorg cascades
1720
+ * whenever the real proposer's block arrives from L1.
1121
1721
  */ async syncProposedBlockToArchiver(block) {
1122
- if (this.config.skipPushProposedBlocksToArchiver !== false) {
1722
+ if (this.config.skipPushProposedBlocksToArchiver || this.config.fishermanMode) {
1123
1723
  this.log.warn(`Skipping push of proposed block ${block.number} to archiver`, {
1124
1724
  blockNumber: block.number,
1125
1725
  slot: block.header.globalVariables.slotNumber
@@ -1132,36 +1732,76 @@ _dec = trackSpan('CheckpointProposalJob.execute'), _dec1 = trackSpan('Checkpoint
1132
1732
  });
1133
1733
  await this.blockSink.addBlock(block);
1134
1734
  }
1735
+ /**
1736
+ * Adds the proposed checkpoint to the archiver so the proposer's optimistic proposed-checkpoint
1737
+ * tip advances locally. Gossip doesn't echo our own messages back, so without this the proposer
1738
+ * would never see its own proposed checkpoint and couldn't pipeline the next slot.
1739
+ *
1740
+ * Skipped whenever proposed blocks aren't pushed (`skipPushProposedBlocksToArchiver`, fisherman
1741
+ * mode): the archiver derives the checkpoint archive from its stored blocks, so without them the
1742
+ * push would fail. All blocks were already added (and awaited) during block building, so this
1743
+ * needs no retry — they are guaranteed present by the time we get here.
1744
+ */ async syncProposedCheckpointToArchiver(checkpoint, blockCount, feeAssetPriceModifier) {
1745
+ if (this.config.skipPushProposedBlocksToArchiver || this.config.fishermanMode) {
1746
+ return;
1747
+ }
1748
+ const startBlock = BlockNumber(this.syncedToBlockNumber + 1);
1749
+ this.log.debug(`Syncing proposed checkpoint ${this.checkpointNumber} to archiver`, {
1750
+ checkpointNumber: this.checkpointNumber,
1751
+ slot: this.targetSlot,
1752
+ startBlock,
1753
+ blockCount
1754
+ });
1755
+ await this.blockSink.addProposedCheckpoint({
1756
+ header: checkpoint.header,
1757
+ checkpointNumber: this.checkpointNumber,
1758
+ startBlock,
1759
+ blockCount,
1760
+ totalManaUsed: checkpoint.header.totalManaUsed.toBigInt(),
1761
+ feeAssetPriceModifier
1762
+ });
1763
+ }
1135
1764
  /** Runs fee analysis and logs checkpoint outcome as fisherman */ async handleCheckpointEndAsFisherman(checkpoint) {
1136
1765
  // Perform L1 fee analysis before clearing requests
1137
1766
  // The callback is invoked asynchronously after the next block is mined
1138
- const feeAnalysis = await this.publisher.analyzeL1Fees(this.slot, (analysis)=>this.metrics.recordFishermanFeeAnalysis(analysis));
1767
+ const feeAnalysis = await this.publisher.analyzeL1Fees(this.targetSlot, (analysis)=>this.metrics.recordFishermanFeeAnalysis(analysis));
1139
1768
  if (checkpoint) {
1140
- this.log.info(`Validation checkpoint building SUCCEEDED for slot ${this.slot}`, {
1769
+ this.log.info(`Validation checkpoint building SUCCEEDED for slot ${this.targetSlot}`, {
1141
1770
  ...checkpoint.toCheckpointInfo(),
1142
1771
  ...checkpoint.getStats(),
1143
1772
  feeAnalysisId: feeAnalysis?.id
1144
1773
  });
1145
1774
  } else {
1146
- this.log.warn(`Validation block building FAILED for slot ${this.slot}`, {
1147
- slot: this.slot,
1775
+ this.log.warn(`Validation block building FAILED for slot ${this.targetSlot}`, {
1776
+ slot: this.targetSlot,
1148
1777
  feeAnalysisId: feeAnalysis?.id
1149
1778
  });
1150
- this.metrics.recordBlockProposalFailed('block_build_failed');
1779
+ this.metrics.recordCheckpointProposalFailed('block_build_failed');
1151
1780
  }
1152
1781
  this.publisher.clearPendingRequests();
1153
1782
  }
1154
- /** Waits until a specific time within the current slot */ async waitUntilTimeInSlot(targetSecondsIntoSlot) {
1155
- const slotStartTimestamp = this.getSlotStartBuildTimestamp();
1156
- const targetTimestamp = slotStartTimestamp + targetSecondsIntoSlot;
1157
- await sleepUntil(new Date(targetTimestamp * 1000), this.dateProvider.nowAsDate());
1158
- }
1159
- getSlotStartBuildTimestamp() {
1160
- return getSlotStartBuildTimestamp(this.slot, this.l1Constants);
1783
+ /**
1784
+ * Helper to handle HA double-signing errors. Returns true if the error was handled (caller should yield).
1785
+ */ handleHASigningError(err, errorContext) {
1786
+ if (err instanceof DutyAlreadySignedError) {
1787
+ this.log.info(`${errorContext} for slot ${this.targetSlot} already signed by another HA node, yielding`, {
1788
+ slot: this.targetSlot,
1789
+ signedByNode: err.signedByNode
1790
+ });
1791
+ return true;
1792
+ }
1793
+ if (err instanceof SlashingProtectionError) {
1794
+ this.log.info(`${errorContext} for slot ${this.targetSlot} blocked by slashing protection, yielding`, {
1795
+ slot: this.targetSlot,
1796
+ existingMessageHash: err.existingMessageHash,
1797
+ attemptedMessageHash: err.attemptedMessageHash
1798
+ });
1799
+ return true;
1800
+ }
1801
+ return false;
1161
1802
  }
1162
- getSecondsIntoSlot() {
1163
- const slotStartTimestamp = this.getSlotStartBuildTimestamp();
1164
- return Number((this.dateProvider.now() / 1000 - slotStartTimestamp).toFixed(3));
1803
+ /** Waits the polling interval for transactions. Extracted for test overriding. */ async waitForTxsPollingInterval() {
1804
+ await this.awaitInterruptibleSleep(TXS_POLLING_MS);
1165
1805
  }
1166
1806
  getPublisher() {
1167
1807
  return this.publisher;