@aztec/sequencer-client 0.0.1-commit.181e2d196 → 0.0.1-commit.189eedb3

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 (106) hide show
  1. package/README.md +281 -21
  2. package/dest/client/sequencer-client.d.ts +8 -3
  3. package/dest/client/sequencer-client.d.ts.map +1 -1
  4. package/dest/client/sequencer-client.js +51 -29
  5. package/dest/config.d.ts +26 -3
  6. package/dest/config.d.ts.map +1 -1
  7. package/dest/config.js +51 -22
  8. package/dest/global_variable_builder/fee_predictor.d.ts +37 -0
  9. package/dest/global_variable_builder/fee_predictor.d.ts.map +1 -0
  10. package/dest/global_variable_builder/fee_predictor.js +128 -0
  11. package/dest/global_variable_builder/fee_provider.d.ts +21 -0
  12. package/dest/global_variable_builder/fee_provider.d.ts.map +1 -0
  13. package/dest/global_variable_builder/fee_provider.js +58 -0
  14. package/dest/global_variable_builder/global_builder.d.ts +14 -14
  15. package/dest/global_variable_builder/global_builder.d.ts.map +1 -1
  16. package/dest/global_variable_builder/global_builder.js +16 -50
  17. package/dest/global_variable_builder/index.d.ts +4 -2
  18. package/dest/global_variable_builder/index.d.ts.map +1 -1
  19. package/dest/global_variable_builder/index.js +2 -0
  20. package/dest/index.d.ts +2 -2
  21. package/dest/index.d.ts.map +1 -1
  22. package/dest/index.js +1 -1
  23. package/dest/publisher/config.d.ts +15 -3
  24. package/dest/publisher/config.d.ts.map +1 -1
  25. package/dest/publisher/config.js +19 -4
  26. package/dest/publisher/l1_tx_failed_store/failed_tx_store.d.ts +3 -4
  27. package/dest/publisher/l1_tx_failed_store/failed_tx_store.d.ts.map +1 -1
  28. package/dest/publisher/sequencer-bundle-simulator.d.ts +96 -0
  29. package/dest/publisher/sequencer-bundle-simulator.d.ts.map +1 -0
  30. package/dest/publisher/sequencer-bundle-simulator.js +198 -0
  31. package/dest/publisher/sequencer-publisher-factory.d.ts +3 -5
  32. package/dest/publisher/sequencer-publisher-factory.d.ts.map +1 -1
  33. package/dest/publisher/sequencer-publisher-factory.js +2 -3
  34. package/dest/publisher/sequencer-publisher.d.ts +73 -65
  35. package/dest/publisher/sequencer-publisher.d.ts.map +1 -1
  36. package/dest/publisher/sequencer-publisher.js +318 -523
  37. package/dest/sequencer/automine/automine_factory.d.ts +54 -0
  38. package/dest/sequencer/automine/automine_factory.d.ts.map +1 -0
  39. package/dest/sequencer/automine/automine_factory.js +84 -0
  40. package/dest/sequencer/automine/automine_sequencer.d.ts +151 -0
  41. package/dest/sequencer/automine/automine_sequencer.d.ts.map +1 -0
  42. package/dest/sequencer/automine/automine_sequencer.js +509 -0
  43. package/dest/sequencer/chain_state_overrides.d.ts +61 -0
  44. package/dest/sequencer/chain_state_overrides.d.ts.map +1 -0
  45. package/dest/sequencer/chain_state_overrides.js +98 -0
  46. package/dest/sequencer/checkpoint_proposal_job.d.ts +60 -12
  47. package/dest/sequencer/checkpoint_proposal_job.d.ts.map +1 -1
  48. package/dest/sequencer/checkpoint_proposal_job.js +774 -219
  49. package/dest/sequencer/checkpoint_proposal_job_metrics.d.ts +34 -0
  50. package/dest/sequencer/checkpoint_proposal_job_metrics.d.ts.map +1 -0
  51. package/dest/sequencer/checkpoint_proposal_job_metrics.js +72 -0
  52. package/dest/sequencer/checkpoint_voter.d.ts +1 -2
  53. package/dest/sequencer/checkpoint_voter.d.ts.map +1 -1
  54. package/dest/sequencer/checkpoint_voter.js +2 -5
  55. package/dest/sequencer/events.d.ts +48 -2
  56. package/dest/sequencer/events.d.ts.map +1 -1
  57. package/dest/sequencer/index.d.ts +3 -1
  58. package/dest/sequencer/index.d.ts.map +1 -1
  59. package/dest/sequencer/index.js +2 -0
  60. package/dest/sequencer/metrics.d.ts +13 -10
  61. package/dest/sequencer/metrics.d.ts.map +1 -1
  62. package/dest/sequencer/metrics.js +45 -20
  63. package/dest/sequencer/sequencer.d.ts +68 -17
  64. package/dest/sequencer/sequencer.d.ts.map +1 -1
  65. package/dest/sequencer/sequencer.js +359 -119
  66. package/dest/sequencer/timetable.d.ts +14 -3
  67. package/dest/sequencer/timetable.d.ts.map +1 -1
  68. package/dest/sequencer/timetable.js +47 -43
  69. package/dest/sequencer/types.d.ts +2 -2
  70. package/dest/sequencer/types.d.ts.map +1 -1
  71. package/dest/test/mock_checkpoint_builder.d.ts +7 -9
  72. package/dest/test/mock_checkpoint_builder.d.ts.map +1 -1
  73. package/dest/test/mock_checkpoint_builder.js +39 -30
  74. package/dest/test/utils.d.ts +1 -1
  75. package/dest/test/utils.d.ts.map +1 -1
  76. package/dest/test/utils.js +7 -6
  77. package/package.json +27 -28
  78. package/src/client/sequencer-client.ts +71 -33
  79. package/src/config.ts +61 -22
  80. package/src/global_variable_builder/README.md +44 -0
  81. package/src/global_variable_builder/fee_predictor.ts +172 -0
  82. package/src/global_variable_builder/fee_provider.ts +75 -0
  83. package/src/global_variable_builder/global_builder.ts +25 -62
  84. package/src/global_variable_builder/index.ts +3 -1
  85. package/src/index.ts +10 -1
  86. package/src/publisher/config.ts +40 -6
  87. package/src/publisher/l1_tx_failed_store/failed_tx_store.ts +3 -1
  88. package/src/publisher/sequencer-bundle-simulator.ts +254 -0
  89. package/src/publisher/sequencer-publisher-factory.ts +3 -6
  90. package/src/publisher/sequencer-publisher.ts +377 -568
  91. package/src/sequencer/README.md +162 -450
  92. package/src/sequencer/automine/README.md +46 -0
  93. package/src/sequencer/automine/automine_factory.ts +145 -0
  94. package/src/sequencer/automine/automine_sequencer.ts +595 -0
  95. package/src/sequencer/chain_state_overrides.ts +162 -0
  96. package/src/sequencer/checkpoint_proposal_job.ts +923 -251
  97. package/src/sequencer/checkpoint_proposal_job_metrics.ts +128 -0
  98. package/src/sequencer/checkpoint_voter.ts +1 -12
  99. package/src/sequencer/events.ts +51 -2
  100. package/src/sequencer/index.ts +2 -0
  101. package/src/sequencer/metrics.ts +57 -24
  102. package/src/sequencer/sequencer.ts +440 -134
  103. package/src/sequencer/timetable.ts +56 -51
  104. package/src/sequencer/types.ts +1 -1
  105. package/src/test/mock_checkpoint_builder.ts +51 -48
  106. package/src/test/utils.ts +28 -10
@@ -1,6 +1,5 @@
1
- import { NUM_CHECKPOINT_END_MARKER_FIELDS, getNumBlockEndBlobFields } from '@aztec/blob-lib/encoding';
2
- import { BLOBS_PER_CHECKPOINT, FIELDS_PER_BLOB } from '@aztec/constants';
3
1
  import type { EpochCache } from '@aztec/epoch-cache';
2
+ import type { SimulationOverridesPlan } from '@aztec/ethereum/contracts';
4
3
  import {
5
4
  BlockNumber,
6
5
  CheckpointNumber,
@@ -15,11 +14,12 @@ import {
15
14
  generateUnrecoverableSignature,
16
15
  } from '@aztec/foundation/crypto/secp256k1-signer';
17
16
  import { Fr } from '@aztec/foundation/curves/bn254';
17
+ import { InterruptError, TimeoutError } from '@aztec/foundation/error';
18
18
  import { EthAddress } from '@aztec/foundation/eth-address';
19
19
  import { Signature } from '@aztec/foundation/eth-signature';
20
20
  import { filter } from '@aztec/foundation/iterator';
21
21
  import { type Logger, type LoggerBindings, createLogger } from '@aztec/foundation/log';
22
- import { sleep, sleepUntil } from '@aztec/foundation/sleep';
22
+ import { InterruptibleSleep } from '@aztec/foundation/sleep';
23
23
  import { type DateProvider, Timer } from '@aztec/foundation/timer';
24
24
  import { type TypedEventEmitter, isErrorClass, unfreeze } from '@aztec/foundation/types';
25
25
  import type { P2P } from '@aztec/p2p';
@@ -31,18 +31,32 @@ import {
31
31
  type L2BlockSink,
32
32
  type L2BlockSource,
33
33
  MaliciousCommitteeAttestationsAndSigners,
34
+ type ProposedCheckpointSink,
35
+ type ValidateCheckpointResult,
34
36
  } from '@aztec/stdlib/block';
35
- import type { Checkpoint } from '@aztec/stdlib/checkpoint';
36
- import { getSlotStartBuildTimestamp } from '@aztec/stdlib/epoch-helpers';
37
+ import {
38
+ type Checkpoint,
39
+ type ProposedCheckpointData,
40
+ getPreviousCheckpointOutHashes,
41
+ validateCheckpoint,
42
+ } from '@aztec/stdlib/checkpoint';
43
+ import { computeQuorum, getSlotStartBuildTimestamp, getTimestampForSlot } from '@aztec/stdlib/epoch-helpers';
37
44
  import { Gas } from '@aztec/stdlib/gas';
38
45
  import {
39
- NoValidTxsError,
40
- type PublicProcessorLimits,
46
+ type BlockBuilderOptions,
47
+ InsufficientValidTxsError,
41
48
  type ResolvedSequencerConfig,
42
49
  type WorldStateSynchronizer,
43
50
  } from '@aztec/stdlib/interfaces/server';
44
51
  import { type L1ToL2MessageSource, computeInHashFromL1ToL2Messages } from '@aztec/stdlib/messaging';
45
- import type { BlockProposalOptions, CheckpointProposal, CheckpointProposalOptions } from '@aztec/stdlib/p2p';
52
+ import type {
53
+ BlockProposal,
54
+ BlockProposalOptions,
55
+ CheckpointAttestation,
56
+ CheckpointProposal,
57
+ CheckpointProposalOptions,
58
+ CoordinationSignatureContext,
59
+ } from '@aztec/stdlib/p2p';
46
60
  import { orderAttestations, trimAttestations } from '@aztec/stdlib/p2p';
47
61
  import type { L2BlockBuiltStats } from '@aztec/stdlib/stats';
48
62
  import { type FailedTx, Tx } from '@aztec/stdlib/tx';
@@ -53,6 +67,8 @@ import { DutyAlreadySignedError, SlashingProtectionError } from '@aztec/validato
53
67
 
54
68
  import type { GlobalVariableBuilder } from '../global_variable_builder/global_builder.js';
55
69
  import type { InvalidateCheckpointRequest, SequencerPublisher } from '../publisher/sequencer-publisher.js';
70
+ import { buildCheckpointSimulationOverridesPlan } from './chain_state_overrides.js';
71
+ import type { CheckpointProposalJobMetricsRecorder } from './checkpoint_proposal_job_metrics.js';
56
72
  import { CheckpointVoter } from './checkpoint_voter.js';
57
73
  import { SequencerInterruptedError } from './errors.js';
58
74
  import type { SequencerEvents } from './events.js';
@@ -63,6 +79,21 @@ import { SequencerState } from './utils.js';
63
79
 
64
80
  /** How much time to sleep while waiting for min transactions to accumulate for a block */
65
81
  const TXS_POLLING_MS = 500;
82
+ const ARCHIVER_SYNC_POLLING_MS = 200;
83
+
84
+ /** Result from proposeCheckpoint when a checkpoint was successfully built and broadcast. */
85
+ type CheckpointProposalBroadcast = {
86
+ checkpoint: Checkpoint;
87
+ proposal: CheckpointProposal;
88
+ blockProposedAt: number;
89
+ };
90
+
91
+ /** Result after attestation collection and signing, ready for L1 submission. */
92
+ type CheckpointProposalResult = {
93
+ checkpoint: Checkpoint;
94
+ attestations: CommitteeAttestationsAndSigners;
95
+ attestationsSignature: Signature;
96
+ };
66
97
 
67
98
  /**
68
99
  * Handles the execution of a checkpoint proposal after the initial preparation phase.
@@ -72,12 +103,32 @@ const TXS_POLLING_MS = 500;
72
103
  */
73
104
  export class CheckpointProposalJob implements Traceable {
74
105
  protected readonly log: Logger;
106
+ private readonly checkpointEventLog: Logger;
107
+
108
+ /** Tracks the fire-and-forget L1 submission promise so it can be awaited during shutdown. */
109
+ private pendingL1Submission: Promise<void> | undefined;
110
+ private readonly interruptibleSleep = new InterruptibleSleep();
111
+ private interrupted = false;
112
+
113
+ /**
114
+ * Chain state overrides built once per slot in proposeCheckpoint after the checkpoint is
115
+ * complete. Carries the pending parent override (archive + slot + fee header) for pipelining,
116
+ * or the invalidation pending override when rolling back. Consumed by
117
+ * publisher.validateBlockHeader before broadcast.
118
+ */
119
+ private checkpointSimulationOverridesPlan?: SimulationOverridesPlan;
120
+
121
+ private getSignatureContext(): CoordinationSignatureContext {
122
+ return this.signatureContext;
123
+ }
75
124
 
76
125
  constructor(
77
- private readonly epoch: EpochNumber,
78
- private readonly slot: SlotNumber,
126
+ private readonly slotNow: SlotNumber,
127
+ private readonly targetSlot: SlotNumber,
128
+ private readonly targetEpoch: EpochNumber,
79
129
  private readonly checkpointNumber: CheckpointNumber,
80
130
  private readonly syncedToBlockNumber: BlockNumber,
131
+ private readonly checkpointedCheckpointNumber: CheckpointNumber,
81
132
  // TODO(palla/mbps): Can we remove the proposer in favor of attestorAddress? Need to check fisherman-node flows.
82
133
  private readonly proposer: EthAddress | undefined,
83
134
  private readonly publisher: SequencerPublisher,
@@ -90,25 +141,75 @@ export class CheckpointProposalJob implements Traceable {
90
141
  private readonly l1ToL2MessageSource: L1ToL2MessageSource,
91
142
  private readonly l2BlockSource: L2BlockSource,
92
143
  private readonly checkpointsBuilder: FullNodeCheckpointsBuilder,
93
- private readonly blockSink: L2BlockSink,
144
+ private readonly blockSink: L2BlockSink & ProposedCheckpointSink,
94
145
  private readonly l1Constants: SequencerRollupConstants,
146
+ private readonly signatureContext: CoordinationSignatureContext,
95
147
  protected config: ResolvedSequencerConfig,
96
148
  protected timetable: SequencerTimetable,
97
149
  private readonly slasherClient: SlasherClientInterface | undefined,
98
150
  private readonly epochCache: EpochCache,
99
151
  private readonly dateProvider: DateProvider,
100
152
  private readonly metrics: SequencerMetrics,
101
- private readonly eventEmitter: TypedEventEmitter<SequencerEvents>,
102
- private readonly setStateFn: (state: SequencerState, slot?: SlotNumber) => void,
153
+ private readonly checkpointMetrics: CheckpointProposalJobMetricsRecorder,
154
+ protected readonly eventEmitter: TypedEventEmitter<SequencerEvents>,
155
+ private readonly setStateFn: (state: SequencerState, slot?: SlotNumber, timeReferenceSlot?: SlotNumber) => void,
103
156
  public readonly tracer: Tracer,
104
157
  bindings?: LoggerBindings,
158
+ private readonly proposedCheckpointData?: ProposedCheckpointData,
105
159
  ) {
106
- this.log = createLogger('sequencer:checkpoint-proposal', { ...bindings, instanceId: `slot-${slot}` });
160
+ this.log = createLogger('sequencer:checkpoint-proposal', {
161
+ ...bindings,
162
+ instanceId: `slot-${this.slotNow}`,
163
+ });
164
+ this.checkpointEventLog = createLogger('sequencer:checkpoint-events', {
165
+ ...bindings,
166
+ instanceId: `slot-${this.slotNow}`,
167
+ });
168
+ }
169
+
170
+ /** Awaits the pending L1 submission if one is in progress. Call during shutdown. */
171
+ public async awaitPendingSubmission(): Promise<void> {
172
+ this.log.info('Awaiting pending L1 payload submission');
173
+ await this.pendingL1Submission;
174
+ }
175
+
176
+ /** Interrupts job-owned waits, including the publisher's send-at-slot sleep, so shutdown can finish. */
177
+ public interrupt(): void {
178
+ this.interrupted = true;
179
+ this.interruptibleSleep.interrupt(true);
180
+ this.publisher.interrupt();
181
+ }
182
+
183
+ private async awaitInterruptibleSleep(ms: number): Promise<void> {
184
+ if (this.interrupted) {
185
+ throw new SequencerInterruptedError();
186
+ }
187
+ if (ms <= 0) {
188
+ return;
189
+ }
190
+ try {
191
+ await this.interruptibleSleep.sleep(ms);
192
+ } catch (err) {
193
+ if (err instanceof InterruptError) {
194
+ throw new SequencerInterruptedError();
195
+ }
196
+ throw err;
197
+ }
198
+ }
199
+
200
+ private logCheckpointEvent(eventName: string, message: string, fields: Record<string, unknown>): void {
201
+ this.checkpointEventLog.debug(message, {
202
+ eventName: `sequencer-checkpoint-${eventName}`,
203
+ ...fields,
204
+ });
107
205
  }
108
206
 
109
207
  /**
110
208
  * Executes the checkpoint proposal job.
111
- * Returns the published checkpoint if successful, undefined otherwise.
209
+ * Builds blocks, assembles checkpoint, and broadcasts the proposal (blocking).
210
+ * Attestation collection, signing, and L1 submission are backgrounded so the
211
+ * work loop can return to IDLE immediately for consecutive slot proposals.
212
+ * Returns the built checkpoint if successful, undefined otherwise.
112
213
  */
113
214
  @trackSpan('CheckpointProposalJob.execute')
114
215
  public async execute(): Promise<Checkpoint | undefined> {
@@ -116,7 +217,7 @@ export class CheckpointProposalJob implements Traceable {
116
217
  // In fisherman mode, we simulate slashing but don't actually publish to L1
117
218
  // These are constant for the whole slot, so we only enqueue them once
118
219
  const votesPromises = new CheckpointVoter(
119
- this.slot,
220
+ this.targetSlot,
120
221
  this.publisher,
121
222
  this.attestorAddress,
122
223
  this.validatorClient,
@@ -127,33 +228,302 @@ export class CheckpointProposalJob implements Traceable {
127
228
  this.log,
128
229
  ).enqueueVotes();
129
230
 
130
- // Build and propose the checkpoint. This will enqueue the request on the publisher if a checkpoint is built.
131
- const checkpoint = await this.proposeCheckpoint();
132
-
133
- // Wait until the voting promises have resolved, so all requests are enqueued (not sent)
134
- await Promise.all(votesPromises);
135
-
136
- if (checkpoint) {
137
- this.metrics.recordCheckpointProposalSuccess();
231
+ // Build blocks, assemble checkpoint, and broadcast proposal (BLOCKING).
232
+ // Returns after broadcast — attestation collection is deferred.
233
+ const broadcast = await this.proposeCheckpoint();
234
+
235
+ if (!broadcast) {
236
+ await Promise.all(votesPromises);
237
+ // Still submit votes even without a checkpoint.
238
+ // Under proposer pipelining, vote-offenses signatures are EIP-712-bound to `targetSlot`
239
+ // (the pipelined slot in which the multicall is expected to mine). Submitting at the
240
+ // wall-clock time would let the multicall mine in a different L2 slot, causing
241
+ // signature verification to fail silently inside Multicall3. Delay submission to the
242
+ // start of `targetSlot` so the tx mines in the slot the vote was signed for.
243
+ if (!this.config.fishermanMode) {
244
+ this.pendingL1Submission = this.publisher.sendRequestsAt(this.targetSlot).then(() => {});
245
+ }
246
+ return undefined;
138
247
  }
139
248
 
249
+ const { checkpoint } = broadcast;
250
+ this.metrics.recordCheckpointProposalSuccess();
251
+
140
252
  // Do not post anything to L1 if we are fishermen, but do perform L1 fee analysis
141
253
  if (this.config.fishermanMode) {
142
254
  await this.handleCheckpointEndAsFisherman(checkpoint);
143
- return;
255
+ return checkpoint;
144
256
  }
145
257
 
146
- // Then send everything to L1
147
- const l1Response = await this.publisher.sendRequests();
148
- const proposedAction = l1Response?.successfulActions.find(a => a === 'propose');
149
- if (proposedAction) {
150
- this.eventEmitter.emit('checkpoint-published', { checkpoint: this.checkpointNumber, slot: this.slot });
151
- const coinbase = checkpoint?.header.coinbase;
152
- await this.metrics.incFilledSlot(this.publisher.getSenderAddress().toString(), coinbase);
153
- return checkpoint;
154
- } else if (checkpoint) {
155
- this.eventEmitter.emit('checkpoint-publish-failed', { ...l1Response, slot: this.slot });
156
- return undefined;
258
+ // Background the attestation signing → L1 pipeline so the work loop is unblocked
259
+ this.pendingL1Submission = this.waitForAttestationsAndEnqueueSubmissionAsync(broadcast, votesPromises);
260
+
261
+ // Return the built checkpoint immediately — the work loop is now unblocked
262
+ return checkpoint;
263
+ }
264
+
265
+ /**
266
+ * Background pipeline: collects attestations, signs them, enqueues the checkpoint, and submits to L1.
267
+ * Runs as a fire-and-forget task stored in `pendingL1Submission` so the work loop is unblocked.
268
+ */
269
+ private async waitForAttestationsAndEnqueueSubmissionAsync(
270
+ broadcast: CheckpointProposalBroadcast,
271
+ votesPromises: Promise<unknown>[],
272
+ ): Promise<void> {
273
+ const { checkpoint } = broadcast;
274
+
275
+ try {
276
+ // Wait for all votes actions, enqueued at the beginning, to resolve
277
+ await Promise.all(votesPromises);
278
+
279
+ // Try to collect attestations from the committee
280
+ const signedAttestations = await this.getSignedCommitteeAttestations(broadcast);
281
+
282
+ // Wait for the previous checkpoint to land on L1 before submitting, so we can check it
283
+ // matches the proposed checkpoint we used as parent, and has valid attestations.
284
+ if (signedAttestations && (await this.waitForValidParentCheckpointOnL1())) {
285
+ await this.enqueueCheckpointForSubmission({ checkpoint, ...signedAttestations });
286
+ }
287
+
288
+ // If we failed to collect attestations, at least check if we need to issue an invalidation
289
+ if (!signedAttestations && (await this.waitForSyncedL2SlotNumber(this.slotNow))) {
290
+ const validationStatus = await this.l2BlockSource.getPendingChainValidationStatus();
291
+ if (!validationStatus.valid) {
292
+ this.log.warn(
293
+ `Checkpoint ${validationStatus.checkpoint.checkpointNumber} has invalid attestations, enqueuing invalidation in spite of attestation collection failure`,
294
+ { checkpoint: validationStatus.checkpoint, reason: validationStatus.reason },
295
+ );
296
+ await this.enqueueInvalidation(validationStatus);
297
+ }
298
+ }
299
+
300
+ // Send whatever was enqueued: votes + (propose | invalidation | nothing).
301
+ const l1Response = await this.publisher.sendRequestsAt(this.targetSlot);
302
+ const proposedAction = l1Response?.successfulActions.find(a => a === 'propose');
303
+ if (proposedAction) {
304
+ this.logCheckpointEvent('published', `Checkpoint published for slot ${this.targetSlot}`, {
305
+ slot: this.targetSlot,
306
+ checkpointNumber: this.checkpointNumber,
307
+ successfulActions: l1Response?.successfulActions,
308
+ sentActions: l1Response?.sentActions,
309
+ });
310
+ this.eventEmitter.emit('checkpoint-published', { checkpoint: this.checkpointNumber, slot: this.targetSlot });
311
+ const coinbase = checkpoint.header.coinbase;
312
+ await this.metrics.incFilledSlot(this.publisher.getSenderAddress().toString(), coinbase);
313
+ } else {
314
+ this.logCheckpointEvent('publish-failed', `Checkpoint publish failed for slot ${this.targetSlot}`, {
315
+ slot: this.targetSlot,
316
+ checkpointNumber: this.checkpointNumber,
317
+ successfulActions: l1Response?.successfulActions,
318
+ failedActions: l1Response?.failedActions,
319
+ sentActions: l1Response?.sentActions,
320
+ expiredActions: l1Response?.expiredActions,
321
+ reason: 'propose_action_not_successful',
322
+ });
323
+ this.log.warn(`Checkpoint publish failed for slot ${this.targetSlot}`, {
324
+ slot: this.targetSlot,
325
+ checkpointNumber: this.checkpointNumber,
326
+ successfulActions: l1Response?.successfulActions,
327
+ failedActions: l1Response?.failedActions,
328
+ sentActions: l1Response?.sentActions,
329
+ expiredActions: l1Response?.expiredActions,
330
+ reason: 'propose_action_not_successful',
331
+ });
332
+ this.eventEmitter.emit('checkpoint-publish-failed', { ...l1Response, slot: this.targetSlot });
333
+ this.metrics.recordPipelineDiscard();
334
+ }
335
+ } catch (err) {
336
+ if (err instanceof SequencerInterruptedError) {
337
+ return;
338
+ }
339
+ this.logCheckpointEvent('publish-failed', `Checkpoint publish failed for slot ${this.targetSlot}`, {
340
+ slot: this.targetSlot,
341
+ checkpointNumber: this.checkpointNumber,
342
+ reason: err instanceof Error ? err.message : String(err),
343
+ });
344
+ this.log.error(`Background attestation/L1 pipeline failed for slot ${this.targetSlot}`, err, {
345
+ slot: this.targetSlot,
346
+ checkpointNumber: this.checkpointNumber,
347
+ reason: err instanceof Error ? err.message : String(err),
348
+ });
349
+ this.eventEmitter.emit('checkpoint-publish-failed', { slot: this.targetSlot });
350
+ this.metrics.recordPipelineDiscard();
351
+ }
352
+ }
353
+
354
+ /** Enqueues the checkpoint for L1 submission. Called after pipeline sleep in execute(). */
355
+ private async enqueueCheckpointForSubmission(result: CheckpointProposalResult): Promise<void> {
356
+ const { checkpoint, attestations, attestationsSignature } = result;
357
+
358
+ // Build-frame deadlines are measured against `slotNow`; observers still see `targetSlot`.
359
+ this.setStateFn(SequencerState.PUBLISHING_CHECKPOINT, this.targetSlot, this.slotNow);
360
+ const aztecSlotDuration = this.l1Constants.slotDuration;
361
+ const submissionSlotStart = Number(getTimestampForSlot(this.targetSlot, this.l1Constants));
362
+ const txTimeoutAt = new Date((submissionSlotStart + aztecSlotDuration) * 1000);
363
+
364
+ // If we have been configured to potentially skip publishing checkpoint then roll the dice here
365
+ if (
366
+ this.config.skipPublishingCheckpointsPercent !== undefined &&
367
+ this.config.skipPublishingCheckpointsPercent > 0
368
+ ) {
369
+ const roll = Math.max(0, randomInt(100));
370
+ if (roll < this.config.skipPublishingCheckpointsPercent) {
371
+ this.log.warn(
372
+ `Skipping publishing proposal for checkpoint ${checkpoint.number}. Configured percentage: ${this.config.skipPublishingCheckpointsPercent}, generated value: ${roll}`,
373
+ );
374
+ return;
375
+ }
376
+ }
377
+
378
+ await this.publisher.enqueueProposeCheckpoint(checkpoint, attestations, attestationsSignature, {
379
+ txTimeoutAt,
380
+ });
381
+ }
382
+
383
+ /**
384
+ * Wait until the archiver syncs past the given L2 slot number.
385
+ * The deadline is the end of `this.targetSlot`, beyond which any pipelined work would miss its
386
+ * L1 submission window and is no longer useful.
387
+ */
388
+ private async waitForSyncedL2SlotNumber(waitForSlot: SlotNumber): Promise<boolean> {
389
+ const targetSlotStart = Number(getTimestampForSlot(this.targetSlot, this.l1Constants));
390
+ const targetSlotEndMs = (targetSlotStart + this.l1Constants.slotDuration) * 1000;
391
+ const syncDelayTolerance = this.l1Constants.ethereumSlotDuration * 2 * 1000;
392
+ const timeoutSeconds = Math.max(0.1, (targetSlotEndMs + syncDelayTolerance - this.dateProvider.now()) / 1000);
393
+
394
+ try {
395
+ const timer = new Timer();
396
+ while (true) {
397
+ const syncedSlot = await this.l2BlockSource.getSyncedL2SlotNumber();
398
+ if (syncedSlot !== undefined && syncedSlot >= waitForSlot) {
399
+ return true;
400
+ }
401
+ if (timeoutSeconds && timer.s() > timeoutSeconds) {
402
+ throw new TimeoutError(`Timeout awaiting archiver sync past slot ${waitForSlot}`);
403
+ }
404
+ await this.awaitInterruptibleSleep(ARCHIVER_SYNC_POLLING_MS);
405
+ }
406
+ } catch (err) {
407
+ if (err instanceof SequencerInterruptedError) {
408
+ throw err;
409
+ }
410
+ this.log.warn(
411
+ `Archiver did not sync L1 past slot ${waitForSlot} before slot ${this.targetSlot} expired, discarding pipelined work`,
412
+ { checkpointNumber: this.checkpointNumber },
413
+ );
414
+ this.emitPipelinedCheckpointDiscarded('archiver-sync-timeout');
415
+ return false;
416
+ }
417
+ }
418
+
419
+ /**
420
+ * Waits for the parent checkpoint to land on L1 before submitting a pipelined checkpoint.
421
+ * Polls until the archiver has synced L1 past the parent's slot, then verifies:
422
+ * - If we built on a proposed parent: it must have landed on L1 with matching hash and valid attestations.
423
+ * - If we built without a proposed parent: no new checkpoint must have appeared for that slot.
424
+ * If the parent has invalid attestations, enqueues an invalidation. Returns whether to proceed with the proposal.
425
+ */
426
+ protected async waitForValidParentCheckpointOnL1(): Promise<boolean> {
427
+ if (this.config.skipWaitForValidParentCheckpointOnL1) {
428
+ this.log.warn(`Skipping waitForValidParentCheckpointOnL1 due to test configuration`, {
429
+ checkpointNumber: this.checkpointNumber,
430
+ });
431
+ return true;
432
+ }
433
+
434
+ const parentCheckpointNumber = CheckpointNumber(this.checkpointNumber - 1);
435
+
436
+ // Wait until archiver has synced L1 past the parent's slot (slotNow)
437
+ if (!(await this.waitForSyncedL2SlotNumber(this.slotNow))) {
438
+ return false;
439
+ }
440
+
441
+ const tips = await this.l2BlockSource.getL2Tips();
442
+ const checkpointedNumber = tips.checkpointed.checkpoint.number;
443
+
444
+ // We built on top of a proposed checkpoint. Verify it landed on L1 as expected.
445
+ if (this.proposedCheckpointData) {
446
+ // After syncing from L1 we see the chain tip has invalid attestations. This means the parent checkpoint was posted
447
+ // with invalid attestations, or it built on top of something with invalid attestations and didnt invalidate them.
448
+ // Either way, we thought our parent would be valid, so we have to throw away our work. But at least we'll try and
449
+ // invalidate on L1 so we clean up the chain for the next proposer. And we'll slash them, but that's handled elsewhere.
450
+ const validationStatus = await this.l2BlockSource.getPendingChainValidationStatus();
451
+ if (!validationStatus.valid) {
452
+ this.log.warn(
453
+ `Parent checkpoint ${parentCheckpointNumber} has invalid attestations, discarding pipelined work`,
454
+ { checkpointNumber: this.checkpointNumber, reason: validationStatus.reason },
455
+ );
456
+ this.emitPipelinedCheckpointDiscarded('parent-invalid-attestations');
457
+ await this.enqueueInvalidation(validationStatus);
458
+ return false;
459
+ }
460
+
461
+ // The pending chain is valid. But did the parent checkpoint land on L1 at all?
462
+ if (checkpointedNumber < parentCheckpointNumber) {
463
+ this.log.warn(`Parent checkpoint ${parentCheckpointNumber} did not land on L1, discarding pipelined work`, {
464
+ checkpointNumber: this.checkpointNumber,
465
+ checkpointedNumber,
466
+ });
467
+ this.emitPipelinedCheckpointDiscarded('parent-not-on-l1');
468
+ return false;
469
+ }
470
+
471
+ // It landed. But is it the one we were expecting?
472
+ const expectedHash = this.proposedCheckpointData.header.hash().toString();
473
+ if (tips.checkpointed.checkpoint.hash !== expectedHash) {
474
+ this.log.warn(`Parent checkpoint ${parentCheckpointNumber} hash mismatch on L1, discarding pipelined work`, {
475
+ checkpointNumber: this.checkpointNumber,
476
+ expectedHash,
477
+ actualHash: tips.checkpointed.checkpoint.hash,
478
+ });
479
+ this.emitPipelinedCheckpointDiscarded('parent-hash-mismatch');
480
+ return false;
481
+ }
482
+
483
+ return true;
484
+ } else {
485
+ // We didn't see a proposed checkpoint at build time, so we built on checkpointed parent from two slots ago.
486
+ // But if a new checkpoint for the previous slot appeared on L1 in the meantime, our checkpoint assumed the wrong parent,
487
+ // so we have to discard our work. This can happen if we're somehow cut off from p2p and fail to see the checkpoint
488
+ // proposal for the previous slot.
489
+ if (checkpointedNumber > parentCheckpointNumber) {
490
+ this.log.warn(
491
+ `Unexpected checkpoint ${checkpointedNumber} landed on L1 after we built on top of parent ${parentCheckpointNumber}, discarding pipelined work`,
492
+ { checkpointNumber: this.checkpointNumber, checkpointedNumber },
493
+ );
494
+ this.emitPipelinedCheckpointDiscarded('unexpected-parent-appeared');
495
+ return false;
496
+ }
497
+
498
+ return true;
499
+ }
500
+ }
501
+
502
+ /** Emits the pipelined-checkpoint-discarded event and records the metric. */
503
+ private emitPipelinedCheckpointDiscarded(reason: string): void {
504
+ this.metrics.recordPipelineParentCheckpointMismatch(reason);
505
+ this.eventEmitter.emit('pipelined-checkpoint-discarded', {
506
+ slot: this.targetSlot,
507
+ checkpointNumber: this.checkpointNumber,
508
+ reason,
509
+ });
510
+ }
511
+
512
+ /** Simulates and enqueues an invalidation request for the invalid parent checkpoint. */
513
+ private async enqueueInvalidation(validationStatus: ValidateCheckpointResult): Promise<void> {
514
+ if (this.config.skipInvalidateBlockAsProposer) {
515
+ this.log.warn(`Skipping checkpoint invalidation as proposer due to test configuration`);
516
+ return;
517
+ }
518
+ const invalidateRequest = await this.publisher.simulateInvalidateCheckpoint(validationStatus);
519
+ if (invalidateRequest) {
520
+ const submissionSlotStart = Number(getTimestampForSlot(this.targetSlot, this.l1Constants));
521
+ const txTimeoutAt = new Date((submissionSlotStart + this.l1Constants.slotDuration) * 1000);
522
+ this.publisher.enqueueInvalidateCheckpoint(invalidateRequest, { txTimeoutAt });
523
+ } else {
524
+ this.log.info(`Invalidation simulation returned undefined, checkpoint may have been removed already`, {
525
+ checkpointNumber: this.checkpointNumber,
526
+ });
157
527
  }
158
528
  }
159
529
 
@@ -161,42 +531,89 @@ export class CheckpointProposalJob implements Traceable {
161
531
  return {
162
532
  // nullish operator needed for tests
163
533
  [Attributes.COINBASE]: this.validatorClient.getCoinbaseForAttestor(this.attestorAddress)?.toString(),
164
- [Attributes.SLOT_NUMBER]: this.slot,
534
+ [Attributes.SLOT_NUMBER]: this.targetSlot,
165
535
  };
166
536
  })
167
- private async proposeCheckpoint(): Promise<Checkpoint | undefined> {
537
+ private async proposeCheckpoint(): Promise<CheckpointProposalBroadcast | undefined> {
168
538
  try {
539
+ const now = this.dateProvider.now();
540
+ if (this.proposedCheckpointData) {
541
+ // Measure against the wall-clock slot whose build window we are currently using.
542
+ // In pipelining mode `targetSlot` is intentionally one slot ahead, which makes the
543
+ // target-slot boundary a full slot away from the actual build start time.
544
+ const slotBoundaryMs = Number(getTimestampForSlot(this.slotNow, this.l1Constants)) * 1000;
545
+ this.checkpointMetrics.recordPipelinedCheckpointBuildStartOffsetFromSlotBoundary(now - slotBoundaryMs);
546
+ }
547
+ this.checkpointMetrics.startCheckpointTiming(now);
548
+
169
549
  // Get operator configured coinbase and fee recipient for this attestor
170
550
  const coinbase = this.validatorClient.getCoinbaseForAttestor(this.attestorAddress);
171
551
  const feeRecipient = this.validatorClient.getFeeRecipientForAttestor(this.attestorAddress);
172
552
 
173
553
  // Start the checkpoint
174
- this.setStateFn(SequencerState.INITIALIZING_CHECKPOINT, this.slot);
175
- this.metrics.incOpenSlot(this.slot, this.proposer?.toString() ?? 'unknown');
554
+ this.setStateFn(SequencerState.INITIALIZING_CHECKPOINT, this.targetSlot, this.slotNow);
555
+ this.logCheckpointEvent('slot-started', `Starting checkpoint proposal for slot ${this.targetSlot}`, {
556
+ buildSlot: this.slotNow,
557
+ submissionSlot: this.targetSlot,
558
+ slot: this.targetSlot,
559
+ checkpointNumber: this.checkpointNumber,
560
+ proposer: this.proposer?.toString(),
561
+ attestorAddress: this.attestorAddress.toString(),
562
+ publisherAddress: this.publisher.getSenderAddress().toString(),
563
+ coinbase: coinbase.toString(),
564
+ });
565
+ this.metrics.incOpenSlot(this.targetSlot, this.proposer?.toString() ?? 'unknown');
176
566
 
177
567
  // Enqueues checkpoint invalidation (constant for the whole slot)
178
568
  if (this.invalidateCheckpoint && !this.config.skipInvalidateBlockAsProposer) {
179
569
  this.publisher.enqueueInvalidateCheckpoint(this.invalidateCheckpoint);
180
570
  }
181
571
 
182
- // Create checkpoint builder for the slot
572
+ // Build the simulation plan for this slot. When pipelining, this overrides L1's view of
573
+ // pending/archive/fee-header to "as if the proposed parent had landed", so both the
574
+ // mana-min-fee simulation (in the globals builder) and the pre-broadcast
575
+ // validateBlockHeader see the chain tip the eventual L1 send will see.
576
+ this.checkpointSimulationOverridesPlan = await buildCheckpointSimulationOverridesPlan({
577
+ checkpointNumber: this.checkpointNumber,
578
+ proposedCheckpointData: this.proposedCheckpointData,
579
+ invalidateToPendingCheckpointNumber: this.invalidateCheckpoint?.forcePendingCheckpointNumber,
580
+ checkpointedCheckpointNumber: this.checkpointedCheckpointNumber,
581
+ rollup: this.publisher.rollupContract,
582
+ signatureContext: this.signatureContext,
583
+ log: this.log,
584
+ });
585
+
183
586
  const checkpointGlobalVariables = await this.globalsBuilder.buildCheckpointGlobalVariables(
184
587
  coinbase,
185
588
  feeRecipient,
186
- this.slot,
589
+ this.targetSlot,
590
+ this.checkpointSimulationOverridesPlan,
187
591
  );
188
592
 
189
593
  // Collect L1 to L2 messages for the checkpoint and compute their hash
190
594
  const l1ToL2Messages = await this.l1ToL2MessageSource.getL1ToL2Messages(this.checkpointNumber);
191
595
  const inHash = computeInHashFromL1ToL2Messages(l1ToL2Messages);
192
596
 
193
- // Collect the out hashes of all the checkpoints before this one in the same epoch
194
- const previousCheckpointOutHashes = (await this.l2BlockSource.getCheckpointsDataForEpoch(this.epoch))
195
- .filter(c => c.checkpointNumber < this.checkpointNumber)
196
- .map(c => c.checkpointOutHash);
597
+ // Collect the out hashes of all the checkpoints before this one in the same epoch.
598
+ // Under pipelining the parent checkpoint may not be on L1 yet at build time, so the helper
599
+ // splices in the parent's checkpointOutHash from the locally-known proposed checkpoint so
600
+ // the resulting `epochOutHash` matches what validators (and L1) compute once the parent
601
+ // lands on L1.
602
+ const previousCheckpointOutHashes = await getPreviousCheckpointOutHashes({
603
+ blockSource: this.l2BlockSource,
604
+ epoch: this.targetEpoch,
605
+ checkpointNumber: this.checkpointNumber,
606
+ l1Constants: this.epochCache.getL1Constants(),
607
+ pipeliningEnabled: true,
608
+ proposedCheckpointData: this.proposedCheckpointData,
609
+ log: this.log,
610
+ });
197
611
 
198
- // Get the fee asset price modifier from the oracle
199
- const feeAssetPriceModifier = await this.publisher.getFeeAssetPriceModifier();
612
+ // Anchor the modifier to the predicted parent fee header: L1 will apply it against
613
+ // that, not against the latest published checkpoint (which lags by one under pipelining).
614
+ const predictedParentEthPerFeeAssetE12 =
615
+ this.checkpointSimulationOverridesPlan?.pendingCheckpointState?.feeHeader?.ethPerFeeAsset;
616
+ const feeAssetPriceModifier = await this.publisher.getFeeAssetPriceModifier(predictedParentEthPerFeeAssetE12);
200
617
 
201
618
  // Create a long-lived forked world state for the checkpoint builder
202
619
  await using fork = await this.worldState.fork(this.syncedToBlockNumber, { closeDelayMs: 12_000 });
@@ -220,11 +637,12 @@ export class CheckpointProposalJob implements Traceable {
220
637
 
221
638
  const checkpointProposalOptions: CheckpointProposalOptions = {
222
639
  publishFullTxs: !!this.config.publishTxsWithProposals,
223
- broadcastInvalidCheckpointProposal: this.config.broadcastInvalidBlockProposal,
640
+ broadcastInvalidCheckpointProposal:
641
+ this.config.broadcastInvalidCheckpointProposalOnly || this.config.broadcastInvalidBlockProposal,
224
642
  };
225
643
 
226
644
  let blocksInCheckpoint: L2Block[] = [];
227
- let blockPendingBroadcast: { block: L2Block; txs: Tx[] } | undefined = undefined;
645
+ let blockPendingBroadcast: BlockProposal | undefined = undefined;
228
646
  const checkpointBuildTimer = new Timer();
229
647
 
230
648
  try {
@@ -248,126 +666,170 @@ export class CheckpointProposalJob implements Traceable {
248
666
  }
249
667
 
250
668
  if (blocksInCheckpoint.length === 0) {
251
- this.log.warn(`No blocks were built for slot ${this.slot}`, { slot: this.slot });
252
- this.eventEmitter.emit('checkpoint-empty', { slot: this.slot });
669
+ this.logCheckpointEvent('build-failed', `Checkpoint build failed for slot ${this.targetSlot}`, {
670
+ slot: this.targetSlot,
671
+ checkpointNumber: this.checkpointNumber,
672
+ reason: 'no_blocks_built',
673
+ });
674
+ this.log.warn(`No blocks were built for slot ${this.targetSlot}`, {
675
+ slot: this.targetSlot,
676
+ checkpointNumber: this.checkpointNumber,
677
+ reason: 'no_blocks_built',
678
+ });
679
+ this.eventEmitter.emit('checkpoint-empty', { slot: this.targetSlot });
253
680
  return undefined;
254
681
  }
255
682
 
256
683
  const minBlocksForCheckpoint = this.config.minBlocksForCheckpoint;
257
684
  if (minBlocksForCheckpoint !== undefined && blocksInCheckpoint.length < minBlocksForCheckpoint) {
685
+ this.logCheckpointEvent('build-failed', `Checkpoint build failed for slot ${this.targetSlot}`, {
686
+ slot: this.targetSlot,
687
+ checkpointNumber: this.checkpointNumber,
688
+ blocksBuilt: blocksInCheckpoint.length,
689
+ minBlocksForCheckpoint,
690
+ reason: 'min_blocks_not_met',
691
+ });
258
692
  this.log.warn(
259
693
  `Checkpoint has fewer blocks than minimum (${blocksInCheckpoint.length} < ${minBlocksForCheckpoint}), skipping proposal`,
260
- { slot: this.slot, blocksBuilt: blocksInCheckpoint.length, minBlocksForCheckpoint },
694
+ {
695
+ slot: this.targetSlot,
696
+ checkpointNumber: this.checkpointNumber,
697
+ blocksBuilt: blocksInCheckpoint.length,
698
+ minBlocksForCheckpoint,
699
+ reason: 'min_blocks_not_met',
700
+ },
261
701
  );
262
702
  return undefined;
263
703
  }
264
704
 
265
705
  // Assemble and broadcast the checkpoint proposal, including the last block that was not
266
706
  // broadcasted yet, and wait to collect the committee attestations.
267
- this.setStateFn(SequencerState.ASSEMBLING_CHECKPOINT, this.slot);
707
+ this.setStateFn(SequencerState.ASSEMBLING_CHECKPOINT, this.targetSlot, this.slotNow);
268
708
  const checkpoint = await checkpointBuilder.completeCheckpoint();
269
709
 
710
+ // Final validation: per-block limits are only checked if the operator set them explicitly.
711
+ // Otherwise, checkpoint-level budgets were already enforced by the redistribution logic.
712
+ try {
713
+ validateCheckpoint(checkpoint, {
714
+ rollupManaLimit: this.l1Constants.rollupManaLimit,
715
+ maxL2BlockGas: this.config.maxL2BlockGas,
716
+ maxDABlockGas: this.config.maxDABlockGas,
717
+ maxTxsPerBlock: this.config.maxTxsPerBlock,
718
+ maxTxsPerCheckpoint: this.config.maxTxsPerCheckpoint,
719
+ });
720
+ } catch (err) {
721
+ this.logCheckpointEvent('build-failed', `Checkpoint build failed for slot ${this.targetSlot}`, {
722
+ slot: this.targetSlot,
723
+ checkpointNumber: this.checkpointNumber,
724
+ blocksBuilt: blocksInCheckpoint.length,
725
+ reason: 'invalid_checkpoint',
726
+ checkpoint: checkpoint.header.toInspect(),
727
+ });
728
+ this.log.error(`Built an invalid checkpoint at slot ${this.slotNow} (skipping proposal)`, err, {
729
+ slot: this.targetSlot,
730
+ checkpointNumber: this.checkpointNumber,
731
+ blocksBuilt: blocksInCheckpoint.length,
732
+ reason: 'invalid_checkpoint',
733
+ checkpoint: checkpoint.header.toInspect(),
734
+ });
735
+ return undefined;
736
+ }
737
+
270
738
  // Record checkpoint-level build metrics
271
- this.metrics.recordCheckpointBuild(
739
+ this.checkpointMetrics.recordCheckpointBuild(
272
740
  checkpointBuildTimer.ms(),
273
741
  blocksInCheckpoint.length,
274
742
  checkpoint.getStats().txCount,
275
743
  Number(checkpoint.header.totalManaUsed.toBigInt()),
276
744
  );
745
+ this.logCheckpointEvent('built', `Checkpoint built for slot ${this.targetSlot}`, {
746
+ slot: this.targetSlot,
747
+ buildSlot: this.slotNow,
748
+ checkpointNumber: this.checkpointNumber,
749
+ proposer: this.proposer?.toString(),
750
+ attestorAddress: this.attestorAddress.toString(),
751
+ publisherAddress: this.publisher.getSenderAddress().toString(),
752
+ blocksBuilt: blocksInCheckpoint.length,
753
+ txCount: checkpoint.getStats().txCount,
754
+ totalMana: Number(checkpoint.header.totalManaUsed.toBigInt()),
755
+ });
277
756
 
278
- // Do not collect attestations nor publish to L1 in fisherman mode
757
+ // In fisherman mode, return the checkpoint without broadcasting or collecting attestations
279
758
  if (this.config.fishermanMode) {
280
759
  this.log.info(
281
- `Built checkpoint for slot ${this.slot} with ${blocksInCheckpoint.length} blocks. ` +
760
+ `Built checkpoint for slot ${this.targetSlot} with ${blocksInCheckpoint.length} blocks. ` +
282
761
  `Skipping proposal in fisherman mode.`,
283
762
  {
284
- slot: this.slot,
763
+ slot: this.targetSlot,
285
764
  checkpoint: checkpoint.header.toInspect(),
286
765
  blocksBuilt: blocksInCheckpoint.length,
287
766
  },
288
767
  );
289
768
  this.metrics.recordCheckpointSuccess();
290
- return checkpoint;
769
+ // Return a broadcast result with a dummy proposal — fisherman mode skips attestation collection
770
+ return { checkpoint, proposal: undefined!, blockProposedAt: this.dateProvider.now() };
291
771
  }
292
772
 
293
- // Include the block pending broadcast in the checkpoint proposal if any
294
- const lastBlock = blockPendingBroadcast && {
295
- blockHeader: blockPendingBroadcast.block.header,
296
- indexWithinCheckpoint: blockPendingBroadcast.block.indexWithinCheckpoint,
297
- txs: blockPendingBroadcast.txs,
298
- };
773
+ // Validate the header against L1 state before broadcasting.
774
+ // If this fails the slot is aborted before any gossip work; state drift between here
775
+ // and the eventual L1 send is caught by the bundle simulate at send time.
776
+ try {
777
+ await this.publisher.validateBlockHeader(checkpoint.header, this.checkpointSimulationOverridesPlan);
778
+ } catch (err) {
779
+ this.log.error(`Pre-broadcast header validation failed for slot ${this.targetSlot}; aborting`, err, {
780
+ slot: this.targetSlot,
781
+ checkpointNumber: this.checkpointNumber,
782
+ });
783
+ this.metrics.recordCheckpointProposalFailed('header_validation_failed');
784
+ this.eventEmitter.emit('header-validation-failed', {
785
+ slot: this.targetSlot,
786
+ checkpointNumber: this.checkpointNumber,
787
+ reason: err instanceof Error ? err.message : String(err),
788
+ });
789
+ return undefined;
790
+ }
299
791
 
300
792
  // Create the checkpoint proposal and broadcast it
301
793
  const proposal = await this.validatorClient.createCheckpointProposal(
302
794
  checkpoint.header,
303
795
  checkpoint.archive.root,
796
+ this.checkpointNumber,
304
797
  feeAssetPriceModifier,
305
- lastBlock,
798
+ blockPendingBroadcast,
306
799
  this.proposer,
307
800
  checkpointProposalOptions,
308
801
  );
309
802
 
310
- const blockProposedAt = this.dateProvider.now();
311
- await this.p2pClient.broadcastCheckpointProposal(proposal);
312
-
313
- this.setStateFn(SequencerState.COLLECTING_ATTESTATIONS, this.slot);
314
- const attestations = await this.waitForAttestations(proposal);
315
- const blockAttestedAt = this.dateProvider.now();
316
-
317
- this.metrics.recordCheckpointAttestationDelay(blockAttestedAt - blockProposedAt);
803
+ // Advance our own optimistic proposed-checkpoint tip locally before gossiping. Gossipsub
804
+ // doesn't echo our own messages back, so this is how the proposer makes its own proposed
805
+ // checkpoint visible for pipelining the next slot. Built from local checkpoint data — never
806
+ // from the broadcast proposal archive, which may be deliberately corrupted under test flags.
807
+ // Fail closed: if this throws, the outer catch aborts the slot before gossiping.
808
+ await this.syncProposedCheckpointToArchiver(checkpoint, blocksInCheckpoint.length, feeAssetPriceModifier);
318
809
 
319
- // Proposer must sign over the attestations before pushing them to L1
320
- const signer = this.proposer ?? this.publisher.getSenderAddress();
321
- let attestationsSignature: Signature;
322
- try {
323
- attestationsSignature = await this.validatorClient.signAttestationsAndSigners(
324
- attestations,
325
- signer,
326
- this.slot,
327
- this.checkpointNumber,
328
- );
329
- } catch (err) {
330
- // We shouldn't really get here since we yield to another HA node
331
- // as soon as we see these errors when creating block or checkpoint proposals.
332
- if (this.handleHASigningError(err, 'Attestations signature')) {
333
- return undefined;
334
- }
335
- throw err;
336
- }
337
-
338
- // Enqueue publishing the checkpoint to L1
339
- this.setStateFn(SequencerState.PUBLISHING_CHECKPOINT, this.slot);
340
- const aztecSlotDuration = this.l1Constants.slotDuration;
341
- const slotStartBuildTimestamp = this.getSlotStartBuildTimestamp();
342
- const txTimeoutAt = new Date((slotStartBuildTimestamp + aztecSlotDuration) * 1000);
343
-
344
- // If we have been configured to potentially skip publishing checkpoint then roll the dice here
345
- if (
346
- this.config.skipPublishingCheckpointsPercent !== undefined &&
347
- this.config.skipPublishingCheckpointsPercent > 0
348
- ) {
349
- const result = Math.max(0, randomInt(100));
350
- if (result < this.config.skipPublishingCheckpointsPercent) {
351
- this.log.warn(
352
- `Skipping publishing proposal for checkpoint ${checkpoint.number}. Configured percentage: ${this.config.skipPublishingCheckpointsPercent}, generated value: ${result}`,
353
- );
354
- return checkpoint;
810
+ const blockProposedAt = this.dateProvider.now();
811
+ if (this.config.skipBroadcastCheckpointProposal) {
812
+ // Test-only: suppress the CheckpointProposal so peers never see a proposed checkpoint for
813
+ // this slot, but still broadcast the held last block standalone so peers' archivers ingest
814
+ // it as a proposed-but-uncheckpointed tip — the exact orphan-block state that
815
+ // pruneOrphanProposedBlocks / checkSync must handle.
816
+ if (blockPendingBroadcast && !this.config.skipBroadcastProposals) {
817
+ await this.p2pClient.broadcastProposal(blockPendingBroadcast);
355
818
  }
819
+ } else if (!this.config.skipBroadcastProposals) {
820
+ await this.p2pClient.broadcastCheckpointProposal(proposal);
821
+ this.checkpointMetrics.noteCheckpointBroadcast(this.dateProvider.now());
356
822
  }
357
823
 
358
- await this.publisher.enqueueProposeCheckpoint(checkpoint, attestations, attestationsSignature, {
359
- txTimeoutAt,
360
- forcePendingCheckpointNumber: this.invalidateCheckpoint?.forcePendingCheckpointNumber,
361
- });
362
-
363
- return checkpoint;
824
+ // Return immediately after broadcast — attestation collection happens in the background
825
+ return { checkpoint, proposal, blockProposedAt };
364
826
  } catch (err) {
365
827
  if (err && (err instanceof DutyAlreadySignedError || err instanceof SlashingProtectionError)) {
366
828
  // swallow this error. It's already been logged by a function deeper in the stack
367
829
  return undefined;
368
830
  }
369
831
 
370
- this.log.error(`Error building checkpoint at slot ${this.slot}`, err);
832
+ this.log.error(`Error building checkpoint at slot ${this.targetSlot}`, err);
371
833
  return undefined;
372
834
  }
373
835
  }
@@ -383,29 +845,35 @@ export class CheckpointProposalJob implements Traceable {
383
845
  blockProposalOptions: BlockProposalOptions,
384
846
  ): Promise<{
385
847
  blocksInCheckpoint: L2Block[];
386
- blockPendingBroadcast: { block: L2Block; txs: Tx[] } | undefined;
848
+ blockPendingBroadcast: BlockProposal | undefined;
387
849
  }> {
388
850
  const blocksInCheckpoint: L2Block[] = [];
389
851
  const txHashesAlreadyIncluded = new Set<string>();
390
852
  const initialBlockNumber = BlockNumber(this.syncedToBlockNumber + 1);
391
853
 
392
- // Remaining blob fields available for blocks (checkpoint end marker already subtracted)
393
- let remainingBlobFields = BLOBS_PER_CHECKPOINT * FIELDS_PER_BLOB - NUM_CHECKPOINT_END_MARKER_FIELDS;
394
-
395
854
  // Last block in the checkpoint will usually be flagged as pending broadcast, so we send it along with the checkpoint proposal
396
- let blockPendingBroadcast: { block: L2Block; txs: Tx[] } | undefined = undefined;
855
+ let blockPendingBroadcast: BlockProposal | undefined = undefined;
397
856
 
398
857
  while (true) {
399
858
  const blocksBuilt = blocksInCheckpoint.length;
400
859
  const indexWithinCheckpoint = IndexWithinCheckpoint(blocksBuilt);
401
860
  const blockNumber = BlockNumber(initialBlockNumber + blocksBuilt);
402
861
 
862
+ if (blocksBuilt >= this.config.maxBlocksPerCheckpoint) {
863
+ this.log.debug(`Reached max blocks per checkpoint`, {
864
+ slot: this.targetSlot,
865
+ blocksBuilt,
866
+ maxBlocksPerCheckpoint: this.config.maxBlocksPerCheckpoint,
867
+ });
868
+ break;
869
+ }
870
+
403
871
  const secondsIntoSlot = this.getSecondsIntoSlot();
404
872
  const timingInfo = this.timetable.canStartNextBlock(secondsIntoSlot);
405
873
 
406
874
  if (!timingInfo.canStart) {
407
875
  this.log.debug(`Not enough time left in slot to start another block`, {
408
- slot: this.slot,
876
+ slot: this.targetSlot,
409
877
  blocksBuilt,
410
878
  secondsIntoSlot,
411
879
  });
@@ -424,25 +892,25 @@ export class CheckpointProposalJob implements Traceable {
424
892
  blockNumber,
425
893
  indexWithinCheckpoint,
426
894
  txHashesAlreadyIncluded,
427
- remainingBlobFields,
428
895
  });
429
896
 
430
- // TODO(palla/mbps): Review these conditions. We may want to keep trying in some scenarios.
431
- if (!buildResult && timingInfo.isLastBlock) {
432
- // If no block was produced due to not enough txs and this was the last subslot, exit
433
- break;
434
- } else if (!buildResult && timingInfo.deadline !== undefined) {
435
- // But if there is still time for more blocks, wait until the next subslot and try again
897
+ // If we failed to build the block due to insufficient txs, we try again if there is still time left in the slot
898
+ if ('failure' in buildResult) {
899
+ // If this was the last subslot, or we're running with a single block per slot, we're done
900
+ if (timingInfo.isLastBlock || timingInfo.deadline === undefined) {
901
+ break;
902
+ }
903
+ // Otherwise, if there is still time for more blocks, we wait until the next subslot and try again
436
904
  await this.waitUntilNextSubslot(timingInfo.deadline);
437
905
  continue;
438
- } else if (!buildResult) {
439
- // Exit if there is no possibility of building more blocks
440
- break;
441
- } else if ('error' in buildResult) {
442
- // If there was an error building the block, just exit the loop and give up the rest of the slot
906
+ }
907
+
908
+ // If there was an error building the block, we just exit the loop and give up the rest of the slot.
909
+ // We don't want to risk building more blocks if something went wrong.
910
+ if ('error' in buildResult) {
443
911
  if (!(buildResult.error instanceof SequencerInterruptedError)) {
444
- this.log.warn(`Halting block building for slot ${this.slot}`, {
445
- slot: this.slot,
912
+ this.log.warn(`Halting block building for slot ${this.targetSlot}`, {
913
+ slot: this.targetSlot,
446
914
  blocksBuilt,
447
915
  error: buildResult.error,
448
916
  });
@@ -450,45 +918,43 @@ export class CheckpointProposalJob implements Traceable {
450
918
  break;
451
919
  }
452
920
 
453
- const { block, usedTxs, remainingBlobFields: newRemainingBlobFields } = buildResult;
454
- blocksInCheckpoint.push(block);
921
+ const { block, usedTxs } = buildResult;
922
+ this.checkpointMetrics.noteCheckpointBlockBuilt(this.dateProvider.now(), {
923
+ isFirstBlock: blocksBuilt === 0,
924
+ isLastBlock: timingInfo.isLastBlock,
925
+ });
455
926
 
456
- // Update remaining blob fields for the next block
457
- remainingBlobFields = newRemainingBlobFields;
927
+ blocksInCheckpoint.push(block);
928
+ usedTxs.forEach(tx => txHashesAlreadyIncluded.add(tx.txHash.toString()));
458
929
 
459
- // Sync the proposed block to the archiver to make it available
460
- // Note that the checkpoint builder uses its own fork so it should not need to wait for this syncing
461
- // Eventually we should refactor the checkpoint builder to not need a separate long-lived fork
462
- // Fire and forget - don't block the critical path, but log errors
463
- this.syncProposedBlockToArchiver(block).catch(err => {
464
- this.log.error(`Failed to sync proposed block ${block.number} to archiver`, { blockNumber: block.number, err });
930
+ // Sign the block proposal. This will throw if HA signing fails.
931
+ const proposal = await this.createBlockProposal(block, inHash, usedTxs, {
932
+ ...blockProposalOptions,
933
+ broadcastInvalidBlockProposal:
934
+ blockProposalOptions.broadcastInvalidBlockProposal ||
935
+ block.indexWithinCheckpoint === this.config.invalidBlockProposalIndexWithinCheckpoint,
465
936
  });
466
937
 
467
- usedTxs.forEach(tx => txHashesAlreadyIncluded.add(tx.txHash.toString()));
938
+ // Sync the proposed block to the archiver to make it available, only after we've managed to sign the proposal,
939
+ // so we avoid polluting our archive with a block that would fail.
940
+ // We wait for the sync to succeed, as this helps catch consistency errors, even if it means we lose some time for block-building.
941
+ // If this throws, we abort the entire checkpoint.
942
+ await this.syncProposedBlockToArchiver(block);
468
943
 
469
- // If this is the last block, exit the loop now so we start collecting attestations
944
+ // If this is the last block, do not broadcast it, since it will be included in the checkpoint proposal.
470
945
  if (timingInfo.isLastBlock) {
471
- this.log.verbose(`Completed final block ${blockNumber} for slot ${this.slot}`, {
472
- slot: this.slot,
946
+ this.log.verbose(`Completed final block ${blockNumber} for slot ${this.targetSlot}`, {
947
+ slot: this.targetSlot,
473
948
  blockNumber,
474
949
  blocksBuilt,
475
950
  });
476
- blockPendingBroadcast = { block, txs: usedTxs };
951
+
952
+ blockPendingBroadcast = proposal;
477
953
  break;
478
954
  }
479
955
 
480
- // For non-last blocks, broadcast the block proposal (unless we're in fisherman mode)
481
- // If the block is the last one, we'll broadcast it along with the checkpoint at the end of the loop
482
- if (!this.config.fishermanMode) {
483
- const proposal = await this.validatorClient.createBlockProposal(
484
- block.header,
485
- block.indexWithinCheckpoint,
486
- inHash,
487
- block.archive.root,
488
- usedTxs,
489
- this.proposer,
490
- blockProposalOptions,
491
- );
956
+ // Once we have a signed proposal and the archiver agreed with our proposed block, then we broadcast it.
957
+ if (proposal && !this.config.skipBroadcastProposals) {
492
958
  await this.p2pClient.broadcastProposal(proposal);
493
959
  }
494
960
 
@@ -496,19 +962,44 @@ export class CheckpointProposalJob implements Traceable {
496
962
  await this.waitUntilNextSubslot(timingInfo.deadline);
497
963
  }
498
964
 
499
- this.log.verbose(`Block building loop completed for slot ${this.slot}`, {
500
- slot: this.slot,
965
+ this.log.verbose(`Block building loop completed for slot ${this.targetSlot}`, {
966
+ slot: this.targetSlot,
501
967
  blocksBuilt: blocksInCheckpoint.length,
502
968
  });
503
969
 
504
970
  return { blocksInCheckpoint, blockPendingBroadcast };
505
971
  }
506
972
 
973
+ /** Creates a block proposal for a given block via the validator client (unless in fisherman mode) */
974
+ private createBlockProposal(
975
+ block: L2Block,
976
+ inHash: Fr,
977
+ usedTxs: Tx[],
978
+ blockProposalOptions: BlockProposalOptions,
979
+ ): Promise<BlockProposal | undefined> {
980
+ if (this.config.fishermanMode) {
981
+ this.log.info(`Skipping block proposal for block ${block.number} in fisherman mode`);
982
+ return Promise.resolve(undefined);
983
+ }
984
+ return this.validatorClient.createBlockProposal(
985
+ block.header,
986
+ this.checkpointNumber,
987
+ block.indexWithinCheckpoint,
988
+ inHash,
989
+ block.archive.root,
990
+ usedTxs,
991
+ this.proposer,
992
+ blockProposalOptions,
993
+ );
994
+ }
995
+
507
996
  /** Sleeps until it is time to produce the next block in the slot */
508
997
  @trackSpan('CheckpointProposalJob.waitUntilNextSubslot')
509
998
  private async waitUntilNextSubslot(nextSubslotStart: number) {
510
- this.setStateFn(SequencerState.WAITING_UNTIL_NEXT_BLOCK, this.slot);
511
- this.log.verbose(`Waiting until time for the next block at ${nextSubslotStart}s into slot`, { slot: this.slot });
999
+ this.setStateFn(SequencerState.WAITING_UNTIL_NEXT_BLOCK, this.targetSlot, this.slotNow);
1000
+ this.log.verbose(`Waiting until time for the next block at ${nextSubslotStart}s into slot`, {
1001
+ slot: this.targetSlot,
1002
+ });
512
1003
  await this.waitUntilTimeInSlot(nextSubslotStart);
513
1004
  }
514
1005
 
@@ -523,36 +1014,46 @@ export class CheckpointProposalJob implements Traceable {
523
1014
  indexWithinCheckpoint: IndexWithinCheckpoint;
524
1015
  buildDeadline: Date | undefined;
525
1016
  txHashesAlreadyIncluded: Set<string>;
526
- remainingBlobFields: number;
527
1017
  },
528
- ): Promise<{ block: L2Block; usedTxs: Tx[]; remainingBlobFields: number } | { error: Error } | undefined> {
529
- const {
530
- blockTimestamp,
531
- forceCreate,
532
- blockNumber,
533
- indexWithinCheckpoint,
534
- buildDeadline,
535
- txHashesAlreadyIncluded,
536
- remainingBlobFields,
537
- } = opts;
1018
+ ): Promise<
1019
+ { block: L2Block; usedTxs: Tx[] } | { failure: 'insufficient-txs' | 'insufficient-valid-txs' } | { error: Error }
1020
+ > {
1021
+ const { blockTimestamp, forceCreate, blockNumber, indexWithinCheckpoint, buildDeadline, txHashesAlreadyIncluded } =
1022
+ opts;
538
1023
 
539
1024
  this.log.verbose(
540
- `Preparing block ${blockNumber} index ${indexWithinCheckpoint} at checkpoint ${this.checkpointNumber} for slot ${this.slot}`,
1025
+ `Preparing block ${blockNumber} index ${indexWithinCheckpoint} at checkpoint ${this.checkpointNumber} for slot ${this.targetSlot}`,
541
1026
  { ...checkpointBuilder.getConstantData(), ...opts },
542
1027
  );
543
1028
 
544
1029
  try {
545
1030
  // Wait until we have enough txs to build the block
546
- const minTxs = this.config.minTxsPerBlock;
547
- const { availableTxs, canStartBuilding } = await this.waitForMinTxs(opts);
1031
+ const { availableTxs, canStartBuilding, minTxs } = await this.waitForMinTxs(opts);
548
1032
  if (!canStartBuilding) {
1033
+ this.logCheckpointEvent('block-build-failed', `Block build failed for slot ${this.targetSlot}`, {
1034
+ reason: 'insufficient_txs',
1035
+ blockNumber,
1036
+ slot: this.targetSlot,
1037
+ checkpointNumber: this.checkpointNumber,
1038
+ indexWithinCheckpoint,
1039
+ availableTxs,
1040
+ minTxs,
1041
+ });
549
1042
  this.log.warn(
550
- `Not enough txs to build block ${blockNumber} at index ${indexWithinCheckpoint} in slot ${this.slot} (got ${availableTxs} txs but needs ${minTxs})`,
551
- { blockNumber, slot: this.slot, indexWithinCheckpoint },
1043
+ `Not enough txs to build block ${blockNumber} at index ${indexWithinCheckpoint} in slot ${this.targetSlot} (got ${availableTxs} txs but needs ${minTxs})`,
1044
+ {
1045
+ reason: 'insufficient_txs',
1046
+ blockNumber,
1047
+ slot: this.targetSlot,
1048
+ checkpointNumber: this.checkpointNumber,
1049
+ indexWithinCheckpoint,
1050
+ availableTxs,
1051
+ minTxs,
1052
+ },
552
1053
  );
553
- this.eventEmitter.emit('block-tx-count-check-failed', { minTxs, availableTxs, slot: this.slot });
1054
+ this.eventEmitter.emit('block-tx-count-check-failed', { minTxs, availableTxs, slot: this.targetSlot });
554
1055
  this.metrics.recordBlockProposalFailed('insufficient_txs');
555
- return undefined;
1056
+ return { failure: 'insufficient-txs' };
556
1057
  }
557
1058
 
558
1059
  // Create iterator to pending txs. We filter out txs already included in previous blocks in the checkpoint
@@ -563,24 +1064,31 @@ export class CheckpointProposalJob implements Traceable {
563
1064
  );
564
1065
 
565
1066
  this.log.debug(
566
- `Building block ${blockNumber} at index ${indexWithinCheckpoint} for slot ${this.slot} with ${availableTxs} available txs`,
567
- { slot: this.slot, blockNumber, indexWithinCheckpoint },
1067
+ `Building block ${blockNumber} at index ${indexWithinCheckpoint} for slot ${this.targetSlot} with ${availableTxs} available txs`,
1068
+ { slot: this.targetSlot, blockNumber, indexWithinCheckpoint },
568
1069
  );
569
- this.setStateFn(SequencerState.CREATING_BLOCK, this.slot);
570
-
571
- // Calculate blob fields limit for txs (remaining capacity - this block's end overhead)
572
- const blockEndOverhead = getNumBlockEndBlobFields(indexWithinCheckpoint === 0);
573
- const maxBlobFieldsForTxs = remainingBlobFields - blockEndOverhead;
1070
+ this.setStateFn(SequencerState.CREATING_BLOCK, this.targetSlot, this.slotNow);
574
1071
 
575
- const blockBuilderOptions: PublicProcessorLimits = {
1072
+ // Per-block limits are operator overrides (from SEQ_MAX_L2_BLOCK_GAS etc.) further capped
1073
+ // by remaining checkpoint-level budgets inside CheckpointBuilder before each block is built.
1074
+ // minValidTxs is passed into the builder so it can reject the block *before* updating state.
1075
+ const minValidTxs = forceCreate ? 0 : (this.config.minValidTxsPerBlock ?? minTxs);
1076
+ const blockBuilderOptions: BlockBuilderOptions = {
576
1077
  maxTransactions: this.config.maxTxsPerBlock,
577
- maxBlockSize: this.config.maxBlockSizeInBytes,
578
- maxBlockGas: new Gas(this.config.maxDABlockGas, this.config.maxL2BlockGas),
579
- maxBlobFields: maxBlobFieldsForTxs,
1078
+ maxBlockGas:
1079
+ this.config.maxL2BlockGas !== undefined || this.config.maxDABlockGas !== undefined
1080
+ ? new Gas(this.config.maxDABlockGas ?? Infinity, this.config.maxL2BlockGas ?? Infinity)
1081
+ : undefined,
580
1082
  deadline: buildDeadline,
1083
+ isBuildingProposal: true,
1084
+ minValidTxs,
1085
+ maxBlocksPerCheckpoint: this.timetable.maxNumberOfBlocks,
1086
+ perBlockAllocationMultiplier: this.config.perBlockAllocationMultiplier,
581
1087
  };
582
1088
 
583
- // Actually build the block by executing txs
1089
+ // Actually build the block by executing txs. The builder throws InsufficientValidTxsError
1090
+ // if the number of successfully processed txs is below minValidTxs, ensuring state is not
1091
+ // updated for blocks that will be discarded.
584
1092
  const buildResult = await this.buildSingleBlockWithCheckpointBuilder(
585
1093
  checkpointBuilder,
586
1094
  pendingTxs,
@@ -592,22 +1100,38 @@ export class CheckpointProposalJob implements Traceable {
592
1100
  // If any txs failed during execution, drop them from the mempool so we don't pick them up again
593
1101
  await this.dropFailedTxsFromP2P(buildResult.failedTxs);
594
1102
 
595
- // Check if we have created a block with enough txs. If there were invalid txs in the pool, or if execution took
596
- // too long, then we may not get to minTxsPerBlock after executing public functions.
597
- const minValidTxs = this.config.minValidTxsPerBlock ?? minTxs;
598
- const numTxs = buildResult.status === 'no-valid-txs' ? 0 : buildResult.numTxs;
599
- if (buildResult.status === 'no-valid-txs' || (!forceCreate && numTxs < minValidTxs)) {
1103
+ if (buildResult.status === 'insufficient-valid-txs') {
1104
+ this.logCheckpointEvent('block-build-failed', `Block build failed for slot ${this.targetSlot}`, {
1105
+ reason: 'insufficient_valid_txs',
1106
+ slot: this.targetSlot,
1107
+ checkpointNumber: this.checkpointNumber,
1108
+ blockNumber,
1109
+ numTxs: buildResult.processedCount,
1110
+ indexWithinCheckpoint,
1111
+ minValidTxs,
1112
+ });
600
1113
  this.log.warn(
601
- `Block ${blockNumber} at index ${indexWithinCheckpoint} on slot ${this.slot} has too few valid txs to be proposed`,
602
- { slot: this.slot, blockNumber, numTxs, indexWithinCheckpoint, minValidTxs, buildResult: buildResult.status },
1114
+ `Block ${blockNumber} at index ${indexWithinCheckpoint} on slot ${this.targetSlot} has too few valid txs to be proposed`,
1115
+ {
1116
+ reason: 'insufficient_valid_txs',
1117
+ slot: this.targetSlot,
1118
+ checkpointNumber: this.checkpointNumber,
1119
+ blockNumber,
1120
+ numTxs: buildResult.processedCount,
1121
+ indexWithinCheckpoint,
1122
+ minValidTxs,
1123
+ },
603
1124
  );
604
- this.eventEmitter.emit('block-build-failed', { reason: `Insufficient valid txs`, slot: this.slot });
1125
+ this.eventEmitter.emit('block-build-failed', {
1126
+ reason: `Insufficient valid txs`,
1127
+ slot: this.targetSlot,
1128
+ });
605
1129
  this.metrics.recordBlockProposalFailed('insufficient_valid_txs');
606
- return undefined;
1130
+ return { failure: 'insufficient-valid-txs' };
607
1131
  }
608
1132
 
609
1133
  // Block creation succeeded, emit stats and metrics
610
- const { publicGas, block, publicProcessorDuration, usedTxs, usedTxBlobFields, blockBuildDuration } = buildResult;
1134
+ const { block, publicProcessorDuration, usedTxs, blockBuildDuration, numTxs } = buildResult;
611
1135
 
612
1136
  const blockStats = {
613
1137
  eventName: 'l2-block-built',
@@ -618,33 +1142,56 @@ export class CheckpointProposalJob implements Traceable {
618
1142
 
619
1143
  const blockHash = await block.hash();
620
1144
  const txHashes = block.body.txEffects.map(tx => tx.txHash);
621
- const manaPerSec = publicGas.l2Gas / (blockBuildDuration / 1000);
1145
+ const manaPerSec = block.header.totalManaUsed.toNumberUnsafe() / (blockBuildDuration / 1000);
622
1146
 
623
1147
  this.log.info(
624
- `Built block ${block.number} at checkpoint ${this.checkpointNumber} for slot ${this.slot} with ${numTxs} txs`,
1148
+ `Built block ${block.number} at checkpoint ${this.checkpointNumber} for slot ${this.targetSlot} with ${numTxs} txs`,
625
1149
  { blockHash, txHashes, manaPerSec, ...blockStats },
626
1150
  );
627
1151
 
628
- this.eventEmitter.emit('block-proposed', { blockNumber: block.number, slot: this.slot });
629
- this.metrics.recordBuiltBlock(blockBuildDuration, publicGas.l2Gas);
1152
+ // `slot` is the target/submission slot (may be one ahead when pipelining),
1153
+ // `buildSlot` is the wall-clock slot during which the block was actually built.
1154
+ this.eventEmitter.emit('block-proposed', {
1155
+ blockNumber: block.number,
1156
+ blockHash,
1157
+ checkpointNumber: this.checkpointNumber,
1158
+ indexWithinCheckpoint: block.indexWithinCheckpoint,
1159
+ slot: this.targetSlot,
1160
+ buildSlot: this.slotNow,
1161
+ });
1162
+ this.metrics.recordBuiltBlock(blockBuildDuration, block.header.totalManaUsed.toNumberUnsafe(), this.targetSlot);
630
1163
 
631
- return { block, usedTxs, remainingBlobFields: maxBlobFieldsForTxs - usedTxBlobFields };
1164
+ return { block, usedTxs };
632
1165
  } catch (err: any) {
633
- this.eventEmitter.emit('block-build-failed', { reason: err.message, slot: this.slot });
634
- this.log.error(`Error building block`, err, { blockNumber, slot: this.slot });
1166
+ this.eventEmitter.emit('block-build-failed', {
1167
+ reason: err.message,
1168
+ slot: this.targetSlot,
1169
+ });
1170
+ this.logCheckpointEvent('block-build-failed', `Block build failed for slot ${this.targetSlot}`, {
1171
+ reason: err instanceof Error ? err.message : String(err),
1172
+ slot: this.targetSlot,
1173
+ checkpointNumber: this.checkpointNumber,
1174
+ blockNumber,
1175
+ });
1176
+ this.log.error(`Error building block`, err, {
1177
+ reason: err instanceof Error ? err.message : String(err),
1178
+ slot: this.targetSlot,
1179
+ checkpointNumber: this.checkpointNumber,
1180
+ blockNumber,
1181
+ });
635
1182
  this.metrics.recordBlockProposalFailed(err.name || 'unknown_error');
636
1183
  this.metrics.recordFailedBlock();
637
1184
  return { error: err };
638
1185
  }
639
1186
  }
640
1187
 
641
- /** Uses the checkpoint builder to build a block, catching specific txs */
1188
+ /** Uses the checkpoint builder to build a block, catching InsufficientValidTxsError. */
642
1189
  private async buildSingleBlockWithCheckpointBuilder(
643
1190
  checkpointBuilder: CheckpointBuilder,
644
1191
  pendingTxs: AsyncIterable<Tx>,
645
1192
  blockNumber: BlockNumber,
646
1193
  blockTimestamp: bigint,
647
- blockBuilderOptions: PublicProcessorLimits,
1194
+ blockBuilderOptions: BlockBuilderOptions,
648
1195
  ) {
649
1196
  try {
650
1197
  const workTimer = new Timer();
@@ -652,8 +1199,12 @@ export class CheckpointProposalJob implements Traceable {
652
1199
  const blockBuildDuration = workTimer.ms();
653
1200
  return { ...result, blockBuildDuration, status: 'success' as const };
654
1201
  } catch (err: unknown) {
655
- if (isErrorClass(err, NoValidTxsError)) {
656
- return { failedTxs: err.failedTxs, status: 'no-valid-txs' as const };
1202
+ if (isErrorClass(err, InsufficientValidTxsError)) {
1203
+ return {
1204
+ failedTxs: err.failedTxs,
1205
+ processedCount: err.processedCount,
1206
+ status: 'insufficient-valid-txs' as const,
1207
+ };
657
1208
  }
658
1209
  throw err;
659
1210
  }
@@ -666,7 +1217,7 @@ export class CheckpointProposalJob implements Traceable {
666
1217
  blockNumber: BlockNumber;
667
1218
  indexWithinCheckpoint: IndexWithinCheckpoint;
668
1219
  buildDeadline: Date | undefined;
669
- }): Promise<{ canStartBuilding: boolean; availableTxs: number }> {
1220
+ }): Promise<{ canStartBuilding: boolean; availableTxs: number; minTxs: number }> {
670
1221
  const { indexWithinCheckpoint, blockNumber, buildDeadline, forceCreate } = opts;
671
1222
 
672
1223
  // We only allow a block with 0 txs in the first block of the checkpoint
@@ -683,20 +1234,50 @@ export class CheckpointProposalJob implements Traceable {
683
1234
  // If we're past deadline, or we have no deadline, give up
684
1235
  const now = this.dateProvider.nowAsDate();
685
1236
  if (startBuildingDeadline === undefined || now >= startBuildingDeadline) {
686
- return { canStartBuilding: false, availableTxs: availableTxs };
1237
+ return { canStartBuilding: false, availableTxs, minTxs };
687
1238
  }
688
1239
 
689
1240
  // Wait a bit before checking again
690
- this.setStateFn(SequencerState.WAITING_FOR_TXS, this.slot);
1241
+ this.setStateFn(SequencerState.WAITING_FOR_TXS, this.targetSlot, this.slotNow);
691
1242
  this.log.verbose(
692
- `Waiting for enough txs to build block ${blockNumber} at index ${indexWithinCheckpoint} in slot ${this.slot} (have ${availableTxs} but need ${minTxs})`,
693
- { blockNumber, slot: this.slot, indexWithinCheckpoint },
1243
+ `Waiting for enough txs to build block ${blockNumber} at index ${indexWithinCheckpoint} in slot ${this.targetSlot} (have ${availableTxs} but need ${minTxs})`,
1244
+ { blockNumber, slot: this.targetSlot, indexWithinCheckpoint },
694
1245
  );
695
1246
  await this.waitForTxsPollingInterval();
696
1247
  availableTxs = await this.p2pClient.getPendingTxCount();
697
1248
  }
698
1249
 
699
- return { canStartBuilding: true, availableTxs };
1250
+ return { canStartBuilding: true, availableTxs, minTxs };
1251
+ }
1252
+
1253
+ private async getSignedCommitteeAttestations(
1254
+ broadcast: CheckpointProposalBroadcast,
1255
+ ): Promise<{ attestations: CommitteeAttestationsAndSigners; attestationsSignature: Signature } | undefined> {
1256
+ const { proposal, blockProposedAt } = broadcast;
1257
+ this.setStateFn(SequencerState.COLLECTING_ATTESTATIONS, this.targetSlot, this.slotNow);
1258
+ const attestations = await this.waitForAttestations(proposal);
1259
+ if (!attestations) {
1260
+ return undefined;
1261
+ }
1262
+ this.checkpointMetrics.recordCheckpointAttestationDelay(this.dateProvider.now() - blockProposedAt);
1263
+
1264
+ // Proposer must sign over the attestations before pushing them to L1
1265
+ const signer = this.proposer ?? this.publisher.getSenderAddress();
1266
+ try {
1267
+ const attestationsSignature = await this.validatorClient.signAttestationsAndSigners(
1268
+ attestations,
1269
+ signer,
1270
+ this.targetSlot,
1271
+ this.checkpointNumber,
1272
+ );
1273
+ return { attestations, attestationsSignature };
1274
+ } catch (err) {
1275
+ if (this.handleHASigningError(err, 'Attestations signature')) {
1276
+ return;
1277
+ }
1278
+ this.log.error(`Error signing attestations for checkpoint proposal at slot ${proposal.slotNumber}`, err);
1279
+ return undefined;
1280
+ }
700
1281
  }
701
1282
 
702
1283
  /**
@@ -704,10 +1285,12 @@ export class CheckpointProposalJob implements Traceable {
704
1285
  * This is run after all blocks for the checkpoint have been built.
705
1286
  */
706
1287
  @trackSpan('CheckpointProposalJob.waitForAttestations')
707
- private async waitForAttestations(proposal: CheckpointProposal): Promise<CommitteeAttestationsAndSigners> {
1288
+ private async waitForAttestations(
1289
+ proposal: CheckpointProposal,
1290
+ ): Promise<CommitteeAttestationsAndSigners | undefined> {
708
1291
  if (this.config.fishermanMode) {
709
1292
  this.log.debug('Skipping attestation collection in fisherman mode');
710
- return CommitteeAttestationsAndSigners.empty();
1293
+ return CommitteeAttestationsAndSigners.empty(this.getSignatureContext());
711
1294
  }
712
1295
 
713
1296
  const slotNumber = proposal.slotNumber;
@@ -717,21 +1300,27 @@ export class CheckpointProposalJob implements Traceable {
717
1300
  throw new Error('No committee when collecting attestations');
718
1301
  } else if (committee.length === 0) {
719
1302
  this.log.verbose(`Attesting committee is empty`);
720
- return CommitteeAttestationsAndSigners.empty();
1303
+ return CommitteeAttestationsAndSigners.empty(this.getSignatureContext());
721
1304
  } else {
722
1305
  this.log.debug(`Attesting committee length is ${committee.length}`, { committee });
723
1306
  }
724
1307
 
725
- const numberOfRequiredAttestations = Math.floor((committee.length * 2) / 3) + 1;
1308
+ const numberOfRequiredAttestations = computeQuorum(committee.length);
726
1309
 
727
1310
  if (this.config.skipCollectingAttestations) {
728
1311
  this.log.warn('Skipping attestation collection as per config (attesting with own keys only)');
729
- const attestations = await this.validatorClient?.collectOwnAttestations(proposal);
730
- return new CommitteeAttestationsAndSigners(orderAttestations(attestations ?? [], committee));
1312
+ const attestations = await this.validatorClient?.collectOwnAttestations(proposal, this.checkpointNumber);
1313
+ this.logCheckpointAttestations('collected', committee, attestations ?? [], numberOfRequiredAttestations, {
1314
+ reason: 'collect_own_only',
1315
+ });
1316
+ return new CommitteeAttestationsAndSigners(
1317
+ orderAttestations(attestations ?? [], committee),
1318
+ this.getSignatureContext(),
1319
+ );
731
1320
  }
732
1321
 
733
1322
  const attestationTimeAllowed = this.config.enforceTimeTable
734
- ? this.timetable.getMaxAllowedTime(SequencerState.PUBLISHING_CHECKPOINT)!
1323
+ ? this.timetable.getCheckpointAttestationDeadline()
735
1324
  : this.l1Constants.slotDuration;
736
1325
  const attestationDeadline = new Date((this.getSlotStartBuildTimestamp() + attestationTimeAllowed) * 1000);
737
1326
 
@@ -744,6 +1333,7 @@ export class CheckpointProposalJob implements Traceable {
744
1333
  proposal,
745
1334
  numberOfRequiredAttestations,
746
1335
  attestationDeadline,
1336
+ this.checkpointNumber,
747
1337
  );
748
1338
 
749
1339
  collectedAttestationsCount = attestations.length;
@@ -762,6 +1352,9 @@ export class CheckpointProposalJob implements Traceable {
762
1352
 
763
1353
  // Rollup contract requires that the signatures are provided in the order of the committee
764
1354
  const sorted = orderAttestations(trimmed, committee);
1355
+ this.logCheckpointAttestations('collected', committee, attestations, numberOfRequiredAttestations, {
1356
+ submittedCount: trimmed.length,
1357
+ });
765
1358
 
766
1359
  // Manipulate the attestations if we've been configured to do so
767
1360
  if (
@@ -773,17 +1366,56 @@ export class CheckpointProposalJob implements Traceable {
773
1366
  return this.manipulateAttestations(proposal.slotNumber, epoch, seed, committee, sorted);
774
1367
  }
775
1368
 
776
- return new CommitteeAttestationsAndSigners(sorted);
1369
+ return new CommitteeAttestationsAndSigners(sorted, this.getSignatureContext());
777
1370
  } catch (err) {
778
1371
  if (err && err instanceof AttestationTimeoutError) {
779
1372
  collectedAttestationsCount = err.collectedCount;
1373
+ this.logCheckpointAttestations('failed', committee, undefined, numberOfRequiredAttestations, {
1374
+ collectedCount: collectedAttestationsCount,
1375
+ reason: 'timeout',
1376
+ });
1377
+ this.log.error(
1378
+ `Timeout while waiting for attestations for checkpoint proposal at slot ${proposal.slotNumber} (collected ${collectedAttestationsCount}/${numberOfRequiredAttestations})`,
1379
+ err,
1380
+ );
1381
+ } else {
1382
+ this.logCheckpointAttestations('failed', committee, undefined, numberOfRequiredAttestations, {
1383
+ collectedCount: collectedAttestationsCount,
1384
+ reason: err instanceof Error ? err.message : String(err),
1385
+ });
1386
+ this.log.error(`Error collecting attestations for checkpoint proposal at slot ${proposal.slotNumber}`, err);
780
1387
  }
781
- throw err;
1388
+ return undefined;
782
1389
  } finally {
783
1390
  this.metrics.recordCollectedAttestations(collectedAttestationsCount, collectAttestationsTimer.ms());
784
1391
  }
785
1392
  }
786
1393
 
1394
+ private logCheckpointAttestations(
1395
+ status: 'collected' | 'failed',
1396
+ committee: EthAddress[],
1397
+ attestations: CheckpointAttestation[] | undefined,
1398
+ requiredAttestations: number,
1399
+ opts: { collectedCount?: number; submittedCount?: number; reason?: string } = {},
1400
+ ) {
1401
+ const signedValidators =
1402
+ attestations
1403
+ ?.map(attestation => attestation.getSender()?.toString())
1404
+ .filter((address): address is `0x${string}` => address !== undefined) ?? [];
1405
+ const collectedCount = opts.collectedCount ?? new Set(signedValidators).size;
1406
+ const missingValidatorCount = status === 'failed' ? Math.max(0, requiredAttestations - collectedCount) : undefined;
1407
+ this.logCheckpointEvent(`attestations-${status}`, `Checkpoint attestations ${status} for slot ${this.targetSlot}`, {
1408
+ slot: this.targetSlot,
1409
+ checkpointNumber: this.checkpointNumber,
1410
+ committeeSize: committee.length,
1411
+ requiredAttestations,
1412
+ collectedAttestations: collectedCount,
1413
+ ...(opts.submittedCount !== undefined && { submittedAttestations: opts.submittedCount }),
1414
+ ...(missingValidatorCount !== undefined && { missingValidatorCount }),
1415
+ ...(opts.reason !== undefined && { reason: opts.reason }),
1416
+ });
1417
+ }
1418
+
787
1419
  /** Breaks the attestations before publishing based on attack configs */
788
1420
  private manipulateAttestations(
789
1421
  slotNumber: SlotNumber,
@@ -827,7 +1459,7 @@ export class CheckpointProposalJob implements Traceable {
827
1459
  unfreeze(attestations[targetIndex]).signature = generateRecoverableSignature();
828
1460
  }
829
1461
  }
830
- return new CommitteeAttestationsAndSigners(attestations);
1462
+ return new CommitteeAttestationsAndSigners(attestations, this.getSignatureContext());
831
1463
  }
832
1464
 
833
1465
  if (this.config.shuffleAttestationOrdering) {
@@ -849,11 +1481,11 @@ export class CheckpointProposalJob implements Traceable {
849
1481
  [shuffled[i], shuffled[j]] = [shuffled[j], shuffled[i]];
850
1482
  }
851
1483
 
852
- const signers = new CommitteeAttestationsAndSigners(attestations).getSigners();
853
- return new MaliciousCommitteeAttestationsAndSigners(shuffled, signers);
1484
+ const signers = new CommitteeAttestationsAndSigners(attestations, this.getSignatureContext()).getSigners();
1485
+ return new MaliciousCommitteeAttestationsAndSigners(shuffled, signers, this.getSignatureContext());
854
1486
  }
855
1487
 
856
- return new CommitteeAttestationsAndSigners(attestations);
1488
+ return new CommitteeAttestationsAndSigners(attestations, this.getSignatureContext());
857
1489
  }
858
1490
 
859
1491
  private async dropFailedTxsFromP2P(failedTxs: FailedTx[]) {
@@ -870,9 +1502,13 @@ export class CheckpointProposalJob implements Traceable {
870
1502
  * Adds the proposed block to the archiver so it's available via P2P.
871
1503
  * Gossip doesn't echo messages back to the sender, so the proposer's archiver/world-state
872
1504
  * would never receive its own block without this explicit sync.
1505
+ *
1506
+ * In fisherman mode we skip this push: the fisherman builds blocks locally for validation
1507
+ * and fee analysis only, and pushing them to the archiver causes spurious reorg cascades
1508
+ * whenever the real proposer's block arrives from L1.
873
1509
  */
874
1510
  private async syncProposedBlockToArchiver(block: L2Block): Promise<void> {
875
- if (this.config.skipPushProposedBlocksToArchiver !== false) {
1511
+ if (this.config.skipPushProposedBlocksToArchiver || this.config.fishermanMode) {
876
1512
  this.log.warn(`Skipping push of proposed block ${block.number} to archiver`, {
877
1513
  blockNumber: block.number,
878
1514
  slot: block.header.globalVariables.slotNumber,
@@ -886,23 +1522,58 @@ export class CheckpointProposalJob implements Traceable {
886
1522
  await this.blockSink.addBlock(block);
887
1523
  }
888
1524
 
1525
+ /**
1526
+ * Adds the proposed checkpoint to the archiver so the proposer's optimistic proposed-checkpoint
1527
+ * tip advances locally. Gossip doesn't echo our own messages back, so without this the proposer
1528
+ * would never see its own proposed checkpoint and couldn't pipeline the next slot.
1529
+ *
1530
+ * Skipped whenever proposed blocks aren't pushed (`skipPushProposedBlocksToArchiver`, fisherman
1531
+ * mode): the archiver derives the checkpoint archive from its stored blocks, so without them the
1532
+ * push would fail. All blocks were already added (and awaited) during block building, so this
1533
+ * needs no retry — they are guaranteed present by the time we get here.
1534
+ */
1535
+ private async syncProposedCheckpointToArchiver(
1536
+ checkpoint: Checkpoint,
1537
+ blockCount: number,
1538
+ feeAssetPriceModifier: bigint,
1539
+ ): Promise<void> {
1540
+ if (this.config.skipPushProposedBlocksToArchiver || this.config.fishermanMode) {
1541
+ return;
1542
+ }
1543
+ const startBlock = BlockNumber(this.syncedToBlockNumber + 1);
1544
+ this.log.debug(`Syncing proposed checkpoint ${this.checkpointNumber} to archiver`, {
1545
+ checkpointNumber: this.checkpointNumber,
1546
+ slot: this.targetSlot,
1547
+ startBlock,
1548
+ blockCount,
1549
+ });
1550
+ await this.blockSink.addProposedCheckpoint({
1551
+ header: checkpoint.header,
1552
+ checkpointNumber: this.checkpointNumber,
1553
+ startBlock,
1554
+ blockCount,
1555
+ totalManaUsed: checkpoint.header.totalManaUsed.toBigInt(),
1556
+ feeAssetPriceModifier,
1557
+ });
1558
+ }
1559
+
889
1560
  /** Runs fee analysis and logs checkpoint outcome as fisherman */
890
1561
  private async handleCheckpointEndAsFisherman(checkpoint: Checkpoint | undefined) {
891
1562
  // Perform L1 fee analysis before clearing requests
892
1563
  // The callback is invoked asynchronously after the next block is mined
893
- const feeAnalysis = await this.publisher.analyzeL1Fees(this.slot, analysis =>
1564
+ const feeAnalysis = await this.publisher.analyzeL1Fees(this.targetSlot, analysis =>
894
1565
  this.metrics.recordFishermanFeeAnalysis(analysis),
895
1566
  );
896
1567
 
897
1568
  if (checkpoint) {
898
- this.log.info(`Validation checkpoint building SUCCEEDED for slot ${this.slot}`, {
1569
+ this.log.info(`Validation checkpoint building SUCCEEDED for slot ${this.targetSlot}`, {
899
1570
  ...checkpoint.toCheckpointInfo(),
900
1571
  ...checkpoint.getStats(),
901
1572
  feeAnalysisId: feeAnalysis?.id,
902
1573
  });
903
1574
  } else {
904
- this.log.warn(`Validation block building FAILED for slot ${this.slot}`, {
905
- slot: this.slot,
1575
+ this.log.warn(`Validation block building FAILED for slot ${this.targetSlot}`, {
1576
+ slot: this.targetSlot,
906
1577
  feeAnalysisId: feeAnalysis?.id,
907
1578
  });
908
1579
  this.metrics.recordCheckpointProposalFailed('block_build_failed');
@@ -916,15 +1587,15 @@ export class CheckpointProposalJob implements Traceable {
916
1587
  */
917
1588
  private handleHASigningError(err: any, errorContext: string): boolean {
918
1589
  if (err instanceof DutyAlreadySignedError) {
919
- this.log.info(`${errorContext} for slot ${this.slot} already signed by another HA node, yielding`, {
920
- slot: this.slot,
1590
+ this.log.info(`${errorContext} for slot ${this.targetSlot} already signed by another HA node, yielding`, {
1591
+ slot: this.targetSlot,
921
1592
  signedByNode: err.signedByNode,
922
1593
  });
923
1594
  return true;
924
1595
  }
925
1596
  if (err instanceof SlashingProtectionError) {
926
- this.log.info(`${errorContext} for slot ${this.slot} blocked by slashing protection, yielding`, {
927
- slot: this.slot,
1597
+ this.log.info(`${errorContext} for slot ${this.targetSlot} blocked by slashing protection, yielding`, {
1598
+ slot: this.targetSlot,
928
1599
  existingMessageHash: err.existingMessageHash,
929
1600
  attemptedMessageHash: err.attemptedMessageHash,
930
1601
  });
@@ -938,16 +1609,17 @@ export class CheckpointProposalJob implements Traceable {
938
1609
  protected async waitUntilTimeInSlot(targetSecondsIntoSlot: number): Promise<void> {
939
1610
  const slotStartTimestamp = this.getSlotStartBuildTimestamp();
940
1611
  const targetTimestamp = slotStartTimestamp + targetSecondsIntoSlot;
941
- await sleepUntil(new Date(targetTimestamp * 1000), this.dateProvider.nowAsDate());
1612
+ const delayMs = targetTimestamp * 1000 - this.dateProvider.nowAsDate().getTime();
1613
+ await this.awaitInterruptibleSleep(delayMs);
942
1614
  }
943
1615
 
944
- /** Waits the polling interval for transactions. Extracted for test overriding. */
1616
+ /** Pause between mempool polls in waitForMinTxs. Extracted for test overriding. */
945
1617
  protected async waitForTxsPollingInterval(): Promise<void> {
946
- await sleep(TXS_POLLING_MS);
1618
+ await this.awaitInterruptibleSleep(TXS_POLLING_MS);
947
1619
  }
948
1620
 
949
1621
  private getSlotStartBuildTimestamp(): number {
950
- return getSlotStartBuildTimestamp(this.slot, this.l1Constants);
1622
+ return getSlotStartBuildTimestamp(this.slotNow, this.l1Constants);
951
1623
  }
952
1624
 
953
1625
  private getSecondsIntoSlot(): number {