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

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