@aztec/sequencer-client 0.0.1-commit.343b43af6 → 0.0.1-commit.350e0a4d

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 (121) hide show
  1. package/README.md +281 -21
  2. package/dest/client/sequencer-client.d.ts +23 -16
  3. package/dest/client/sequencer-client.d.ts.map +1 -1
  4. package/dest/client/sequencer-client.js +49 -94
  5. package/dest/config.d.ts +10 -2
  6. package/dest/config.d.ts.map +1 -1
  7. package/dest/config.js +61 -25
  8. package/dest/global_variable_builder/fee_predictor.d.ts +37 -0
  9. package/dest/global_variable_builder/fee_predictor.d.ts.map +1 -0
  10. package/dest/global_variable_builder/fee_predictor.js +138 -0
  11. package/dest/global_variable_builder/fee_provider.d.ts +21 -0
  12. package/dest/global_variable_builder/fee_provider.d.ts.map +1 -0
  13. package/dest/global_variable_builder/fee_provider.js +80 -0
  14. package/dest/global_variable_builder/global_builder.d.ts +13 -26
  15. package/dest/global_variable_builder/global_builder.d.ts.map +1 -1
  16. package/dest/global_variable_builder/global_builder.js +9 -66
  17. package/dest/global_variable_builder/index.d.ts +4 -2
  18. package/dest/global_variable_builder/index.d.ts.map +1 -1
  19. package/dest/global_variable_builder/index.js +2 -0
  20. package/dest/publisher/config.d.ts +19 -3
  21. package/dest/publisher/config.d.ts.map +1 -1
  22. package/dest/publisher/config.js +30 -5
  23. package/dest/publisher/l1_to_l2_messaging.d.ts +21 -0
  24. package/dest/publisher/l1_to_l2_messaging.d.ts.map +1 -0
  25. package/dest/publisher/l1_to_l2_messaging.js +70 -0
  26. package/dest/publisher/l1_tx_failed_store/failed_tx_store.d.ts +50 -11
  27. package/dest/publisher/l1_tx_failed_store/failed_tx_store.d.ts.map +1 -1
  28. package/dest/publisher/l1_tx_failed_store/failed_tx_store.js +68 -1
  29. package/dest/publisher/l1_tx_failed_store/file_store_failed_tx_store.d.ts +2 -2
  30. package/dest/publisher/l1_tx_failed_store/file_store_failed_tx_store.d.ts.map +1 -1
  31. package/dest/publisher/l1_tx_failed_store/file_store_failed_tx_store.js +4 -2
  32. package/dest/publisher/l1_tx_failed_store/index.d.ts +2 -2
  33. package/dest/publisher/l1_tx_failed_store/index.d.ts.map +1 -1
  34. package/dest/publisher/l1_tx_failed_store/index.js +1 -0
  35. package/dest/publisher/sequencer-bundle-simulator.d.ts +96 -0
  36. package/dest/publisher/sequencer-bundle-simulator.d.ts.map +1 -0
  37. package/dest/publisher/sequencer-bundle-simulator.js +198 -0
  38. package/dest/publisher/sequencer-publisher-factory.d.ts +3 -5
  39. package/dest/publisher/sequencer-publisher-factory.d.ts.map +1 -1
  40. package/dest/publisher/sequencer-publisher-factory.js +2 -3
  41. package/dest/publisher/sequencer-publisher.d.ts +103 -67
  42. package/dest/publisher/sequencer-publisher.d.ts.map +1 -1
  43. package/dest/publisher/sequencer-publisher.js +621 -525
  44. package/dest/publisher/write_json.d.ts +11 -0
  45. package/dest/publisher/write_json.d.ts.map +1 -0
  46. package/dest/publisher/write_json.js +57 -0
  47. package/dest/sequencer/automine/automine_factory.d.ts +56 -0
  48. package/dest/sequencer/automine/automine_factory.d.ts.map +1 -0
  49. package/dest/sequencer/automine/automine_factory.js +85 -0
  50. package/dest/sequencer/automine/automine_sequencer.d.ts +189 -0
  51. package/dest/sequencer/automine/automine_sequencer.d.ts.map +1 -0
  52. package/dest/sequencer/automine/automine_sequencer.js +690 -0
  53. package/dest/sequencer/automine/index.d.ts +3 -0
  54. package/dest/sequencer/automine/index.d.ts.map +1 -0
  55. package/dest/sequencer/automine/index.js +2 -0
  56. package/dest/sequencer/checkpoint_proposal_job.d.ts +75 -17
  57. package/dest/sequencer/checkpoint_proposal_job.d.ts.map +1 -1
  58. package/dest/sequencer/checkpoint_proposal_job.js +795 -239
  59. package/dest/sequencer/checkpoint_proposal_job_metrics.d.ts +34 -0
  60. package/dest/sequencer/checkpoint_proposal_job_metrics.d.ts.map +1 -0
  61. package/dest/sequencer/checkpoint_proposal_job_metrics.js +72 -0
  62. package/dest/sequencer/checkpoint_voter.d.ts +1 -2
  63. package/dest/sequencer/checkpoint_voter.d.ts.map +1 -1
  64. package/dest/sequencer/checkpoint_voter.js +2 -5
  65. package/dest/sequencer/events.d.ts +62 -5
  66. package/dest/sequencer/events.d.ts.map +1 -1
  67. package/dest/sequencer/metrics.d.ts +13 -10
  68. package/dest/sequencer/metrics.d.ts.map +1 -1
  69. package/dest/sequencer/metrics.js +45 -20
  70. package/dest/sequencer/requests_tracker.d.ts +22 -0
  71. package/dest/sequencer/requests_tracker.d.ts.map +1 -0
  72. package/dest/sequencer/requests_tracker.js +33 -0
  73. package/dest/sequencer/sequencer.d.ts +159 -35
  74. package/dest/sequencer/sequencer.d.ts.map +1 -1
  75. package/dest/sequencer/sequencer.js +585 -188
  76. package/dest/sequencer/types.d.ts +2 -2
  77. package/dest/sequencer/types.d.ts.map +1 -1
  78. package/dest/test/index.d.ts +3 -3
  79. package/dest/test/index.d.ts.map +1 -1
  80. package/dest/test/mock_checkpoint_builder.d.ts +4 -4
  81. package/dest/test/mock_checkpoint_builder.d.ts.map +1 -1
  82. package/dest/test/utils.d.ts +15 -1
  83. package/dest/test/utils.d.ts.map +1 -1
  84. package/dest/test/utils.js +25 -7
  85. package/package.json +30 -29
  86. package/src/client/sequencer-client.ts +70 -124
  87. package/src/config.ts +71 -22
  88. package/src/global_variable_builder/README.md +44 -0
  89. package/src/global_variable_builder/fee_predictor.ts +182 -0
  90. package/src/global_variable_builder/fee_provider.ts +97 -0
  91. package/src/global_variable_builder/global_builder.ts +19 -88
  92. package/src/global_variable_builder/index.ts +3 -1
  93. package/src/publisher/config.ts +62 -7
  94. package/src/publisher/l1_to_l2_messaging.ts +85 -0
  95. package/src/publisher/l1_tx_failed_store/failed_tx_store.ts +117 -8
  96. package/src/publisher/l1_tx_failed_store/file_store_failed_tx_store.ts +4 -3
  97. package/src/publisher/l1_tx_failed_store/index.ts +1 -1
  98. package/src/publisher/sequencer-bundle-simulator.ts +254 -0
  99. package/src/publisher/sequencer-publisher-factory.ts +3 -6
  100. package/src/publisher/sequencer-publisher.ts +720 -600
  101. package/src/publisher/write_json.ts +78 -0
  102. package/src/sequencer/README.md +162 -450
  103. package/src/sequencer/automine/README.md +60 -0
  104. package/src/sequencer/automine/automine_factory.ts +152 -0
  105. package/src/sequencer/automine/automine_sequencer.ts +796 -0
  106. package/src/sequencer/automine/index.ts +6 -0
  107. package/src/sequencer/checkpoint_proposal_job.ts +958 -265
  108. package/src/sequencer/checkpoint_proposal_job_metrics.ts +128 -0
  109. package/src/sequencer/checkpoint_voter.ts +1 -12
  110. package/src/sequencer/events.ts +66 -5
  111. package/src/sequencer/metrics.ts +57 -24
  112. package/src/sequencer/requests_tracker.ts +43 -0
  113. package/src/sequencer/sequencer.ts +690 -209
  114. package/src/sequencer/types.ts +1 -1
  115. package/src/test/index.ts +2 -2
  116. package/src/test/mock_checkpoint_builder.ts +3 -3
  117. package/src/test/utils.ts +61 -10
  118. package/dest/sequencer/timetable.d.ts +0 -88
  119. package/dest/sequencer/timetable.d.ts.map +0 -1
  120. package/dest/sequencer/timetable.js +0 -222
  121. package/src/sequencer/timetable.ts +0 -283
@@ -1,20 +1,35 @@
1
1
  import { getKzg } from '@aztec/blob-lib';
2
- import { INITIAL_L2_BLOCK_NUM } from '@aztec/constants';
3
- import type { EpochCache } from '@aztec/epoch-cache';
2
+ import { type EpochCache, PROPOSER_PIPELINING_SLOT_OFFSET } from '@aztec/epoch-cache';
4
3
  import { NoCommitteeError, type RollupContract } from '@aztec/ethereum/contracts';
5
4
  import { BlockNumber, CheckpointNumber, EpochNumber, SlotNumber } from '@aztec/foundation/branded-types';
6
5
  import { merge, omit, pick } from '@aztec/foundation/collection';
7
6
  import { Fr } from '@aztec/foundation/curves/bn254';
8
7
  import { EthAddress } from '@aztec/foundation/eth-address';
9
- import { createLogger } from '@aztec/foundation/log';
8
+ import { type Logger, createLogger } from '@aztec/foundation/log';
10
9
  import { RunningPromise } from '@aztec/foundation/running-promise';
11
10
  import type { DateProvider } from '@aztec/foundation/timer';
12
11
  import type { TypedEventEmitter } from '@aztec/foundation/types';
13
12
  import type { P2P } from '@aztec/p2p';
14
13
  import type { SlasherClientInterface } from '@aztec/slasher';
15
- import type { BlockData, L2BlockSink, L2BlockSource, ValidateCheckpointResult } from '@aztec/stdlib/block';
16
- import type { Checkpoint } from '@aztec/stdlib/checkpoint';
17
- import { getSlotStartBuildTimestamp } from '@aztec/stdlib/epoch-helpers';
14
+ import type {
15
+ BlockData,
16
+ L2BlockSink,
17
+ L2BlockSource,
18
+ ProposedCheckpointSink,
19
+ ValidateCheckpointResult,
20
+ } from '@aztec/stdlib/block';
21
+ import {
22
+ type Checkpoint,
23
+ type ProposedCheckpointData,
24
+ buildCheckpointSimulationOverridesPlan,
25
+ } from '@aztec/stdlib/checkpoint';
26
+ import type { ChainConfig } from '@aztec/stdlib/config';
27
+ import { getEpochAtSlot } from '@aztec/stdlib/epoch-helpers';
28
+ import {
29
+ MIN_PER_BLOCK_ALLOCATION_MULTIPLIER,
30
+ MIN_PER_BLOCK_DA_ALLOCATION_MULTIPLIER,
31
+ computeNetworkTxGasLimits,
32
+ } from '@aztec/stdlib/gas';
18
33
  import {
19
34
  type ResolvedSequencerConfig,
20
35
  type SequencerConfig,
@@ -22,8 +37,9 @@ import {
22
37
  type WorldStateSynchronizer,
23
38
  } from '@aztec/stdlib/interfaces/server';
24
39
  import type { L1ToL2MessageSource } from '@aztec/stdlib/messaging';
40
+ import type { CoordinationSignatureContext } from '@aztec/stdlib/p2p';
25
41
  import { pickFromSchema } from '@aztec/stdlib/schemas';
26
- import { MerkleTreeId } from '@aztec/stdlib/trees';
42
+ import { ProposerTimetable, buildProposerTimetable } from '@aztec/stdlib/timetable';
27
43
  import { Attributes, type TelemetryClient, type Tracer, getTelemetryClient, trackSpan } from '@aztec/telemetry-client';
28
44
  import { FullNodeCheckpointsBuilder, NodeKeystoreAdapter, type ValidatorClient } from '@aztec/validator-client';
29
45
 
@@ -34,16 +50,27 @@ import type { GlobalVariableBuilder } from '../global_variable_builder/global_bu
34
50
  import type { SequencerPublisherFactory } from '../publisher/sequencer-publisher-factory.js';
35
51
  import type { InvalidateCheckpointRequest, SequencerPublisher } from '../publisher/sequencer-publisher.js';
36
52
  import { CheckpointProposalJob } from './checkpoint_proposal_job.js';
53
+ import { CheckpointProposalJobMetrics } from './checkpoint_proposal_job_metrics.js';
37
54
  import { CheckpointVoter } from './checkpoint_voter.js';
38
- import { SequencerInterruptedError, SequencerTooSlowError } from './errors.js';
55
+ import { SequencerInterruptedError } from './errors.js';
39
56
  import type { SequencerEvents } from './events.js';
40
57
  import { SequencerMetrics } from './metrics.js';
41
- import { SequencerTimetable } from './timetable.js';
58
+ import { RequestsTracker } from './requests_tracker.js';
42
59
  import type { SequencerRollupConstants } from './types.js';
43
60
  import { SequencerState } from './utils.js';
44
61
 
45
62
  export { SequencerState };
46
63
 
64
+ /** Slot snapshot used to prepare a checkpoint proposal. */
65
+ type SequencerSlotContext = {
66
+ slot: SlotNumber;
67
+ targetSlot: SlotNumber;
68
+ epoch: EpochNumber;
69
+ targetEpoch: EpochNumber;
70
+ ts: bigint;
71
+ nowSeconds: bigint;
72
+ };
73
+
47
74
  /**
48
75
  * Sequencer client
49
76
  * - Checks whether it is elected as proposer for the next slot
@@ -53,18 +80,27 @@ export { SequencerState };
53
80
  * - Votes for proposals and slashes on L1
54
81
  */
55
82
  export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<SequencerEvents>) {
56
- private runningPromise?: RunningPromise;
83
+ protected runningPromise?: RunningPromise;
57
84
  private state = SequencerState.STOPPED;
85
+ private stateSlotNumber: SlotNumber | undefined;
86
+ /** Wall-clock time (ms, via the date provider) at which the current state was entered. */
87
+ private stateEnteredAtMs: number;
58
88
  private metrics: SequencerMetrics;
89
+ private checkpointProposalJobMetrics: CheckpointProposalJobMetrics;
90
+ private readonly stateLog: Logger;
91
+
92
+ /** The last slot for which we attempted to perform our fallback duties (votes and/or prune) with degraded block production */
93
+ private lastSlotForFallbackAction: SlotNumber | undefined;
59
94
 
60
- /** The last slot for which we attempted to perform our voting duties with degraded block production */
61
- private lastSlotForFallbackVote: SlotNumber | undefined;
95
+ /** The (checkpoint, slot) of the last invalidation request we successfully simulated, to prevent
96
+ * re-simulating and re-submitting the same invalidation across the many ticks within a single slot. */
97
+ private lastInvalidationAttempt: { slot: SlotNumber; checkpointNumber: CheckpointNumber } | undefined;
62
98
 
63
99
  /** The last slot for which we logged "no committee" warning, to avoid spam */
64
100
  private lastSlotForNoCommitteeWarning: SlotNumber | undefined;
65
101
 
66
102
  /** The last slot for which we triggered a checkpoint proposal job, to prevent duplicate attempts. */
67
- private lastSlotForCheckpointProposalJob: SlotNumber | undefined;
103
+ protected lastSlotForCheckpointProposalJob: SlotNumber | undefined;
68
104
 
69
105
  /** Last successful checkpoint proposed */
70
106
  private lastCheckpointProposed: Checkpoint | undefined;
@@ -72,11 +108,26 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
72
108
  /** The last epoch for which we logged strategy comparison in fisherman mode. */
73
109
  private lastEpochForStrategyComparison: EpochNumber | undefined;
74
110
 
75
- /** The maximum number of seconds that the sequencer can be into a slot to transition to a particular state. */
76
- protected timetable!: SequencerTimetable;
111
+ /** The last checkpoint proposal job, tracked so we can await its pending L1 submission during shutdown. */
112
+ protected lastCheckpointProposalJob: CheckpointProposalJob | undefined;
113
+
114
+ /**
115
+ * In-flight fire-and-forget requests that {@link stop} interrupts and drains, and {@link pause} awaits
116
+ * untouched: the checkpoint proposal jobs' backgrounded L1 submissions (each job is handed this shared
117
+ * tracker) plus the sequencer's own fallback submissions (votes/prune when we cannot build, or
118
+ * escape-hatch votes). Each fallback send is gated by its wrapper publisher's interruptible target-slot
119
+ * sleep; the tracked interrupt wakes that sleep so the send short-circuits without publishing. A wrapper
120
+ * is created for a single send and is never restarted, so once interrupted its sleeper can never publish
121
+ * a stale-slot tx, even after the pooled publishers are restarted by a later {@link start}.
122
+ */
123
+ protected readonly pendingRequests = new RequestsTracker();
124
+
125
+ /** Proposer schedule and block sub-slot timetable for the sequencer, rebuilt on every config update. */
126
+ protected timetable!: ProposerTimetable;
77
127
 
78
128
  /** Config for the sequencer */
79
129
  protected config: ResolvedSequencerConfig = DefaultSequencerConfig;
130
+ private readonly signatureContext: CoordinationSignatureContext;
80
131
 
81
132
  constructor(
82
133
  protected publisherFactory: SequencerPublisherFactory,
@@ -85,45 +136,148 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
85
136
  protected p2pClient: P2P,
86
137
  protected worldState: WorldStateSynchronizer,
87
138
  protected slasherClient: SlasherClientInterface | undefined,
88
- protected l2BlockSource: L2BlockSource & L2BlockSink,
139
+ protected l2BlockSource: L2BlockSource & L2BlockSink & ProposedCheckpointSink,
89
140
  protected l1ToL2MessageSource: L1ToL2MessageSource,
90
141
  protected checkpointsBuilder: FullNodeCheckpointsBuilder,
91
142
  protected l1Constants: SequencerRollupConstants,
92
143
  protected dateProvider: DateProvider,
93
144
  protected epochCache: EpochCache,
94
145
  protected rollupContract: RollupContract,
95
- config: SequencerConfig,
146
+ config: SequencerConfig & Pick<ChainConfig, 'l1ChainId' | 'rollupAddress'>,
96
147
  protected telemetry: TelemetryClient = getTelemetryClient(),
97
148
  protected log = createLogger('sequencer'),
98
149
  ) {
99
150
  super();
151
+ this.stateLog = log.createChild('state');
152
+ this.stateEnteredAtMs = this.dateProvider.now();
100
153
 
101
154
  // Add [FISHERMAN] prefix to logger if in fisherman mode
102
155
  if (config.fishermanMode) {
103
156
  this.log = log.createChild('[FISHERMAN]');
104
157
  }
105
158
 
159
+ this.signatureContext = {
160
+ chainId: config.l1ChainId,
161
+ rollupAddress: config.rollupAddress,
162
+ };
106
163
  this.metrics = new SequencerMetrics(telemetry, this.rollupContract, 'Sequencer');
164
+ this.checkpointProposalJobMetrics = new CheckpointProposalJobMetrics(telemetry);
107
165
  this.updateConfig(config);
108
166
  }
109
167
 
110
- /** Updates sequencer config by the defined values and updates the timetable */
168
+ /**
169
+ * Updates sequencer config by the defined values and rebuilds the timetable.
170
+ *
171
+ * The merged config is validated against a candidate before being committed: {@link buildTimetable} may
172
+ * reject the candidate (invalid timing geometry, or per-block allocation multipliers below the network
173
+ * minimums). On rejection we leave `this.config` and `this.timetable` untouched and rethrow, so a bad update
174
+ * never leaves the sequencer running with a rejected config and a stale timetable.
175
+ */
111
176
  public updateConfig(config: Partial<SequencerConfig>) {
112
177
  const filteredConfig = pickFromSchema(config, SequencerConfigSchema);
178
+ const candidate = merge(this.config, filteredConfig);
179
+ let timetable: ProposerTimetable;
180
+ try {
181
+ timetable = this.buildTimetable(candidate);
182
+ } catch (err) {
183
+ this.log.warn(`Rejecting sequencer config update: ${(err as Error).message}`, {
184
+ rejectedConfig: omit(filteredConfig, 'txPublicSetupAllowListExtend'),
185
+ });
186
+ throw err;
187
+ }
188
+ this.config = candidate;
189
+ this.timetable = timetable;
113
190
  this.log.info(`Updated sequencer config`, omit(filteredConfig, 'txPublicSetupAllowListExtend'));
114
- this.config = merge(this.config, filteredConfig);
115
- this.timetable = new SequencerTimetable(
116
- {
117
- ethereumSlotDuration: this.l1Constants.ethereumSlotDuration,
118
- aztecSlotDuration: this.aztecSlotDuration,
119
- l1PublishingTime: this.l1PublishingTime,
120
- p2pPropagationTime: this.config.attestationPropagationTime,
121
- blockDurationMs: this.config.blockDurationMs,
122
- enforce: this.config.enforceTimeTable,
123
- },
124
- this.metrics,
125
- this.log,
126
- );
191
+ }
192
+
193
+ /**
194
+ * Builds the proposer timetable from the given config and L1 constants via the shared
195
+ * {@link buildProposerTimetable} helper, so the sequencer derives the same blocks-per-checkpoint as the p2p
196
+ * layer and `getNodeInfo`. The fast local/e2e profile and budget clamping happen inside
197
+ * {@link ProposerTimetable}.
198
+ *
199
+ * Throws if the timing geometry is invalid or the per-block allocation multipliers are below the network
200
+ * minimums; callers must treat a throw as a rejected config and not commit it.
201
+ */
202
+ private buildTimetable(config: ResolvedSequencerConfig): ProposerTimetable {
203
+ const timetable = buildProposerTimetable(config, this.l1Constants);
204
+
205
+ const maxNumberOfBlocks = timetable.getMaxBlocksPerCheckpoint();
206
+ this.log.info(`Sequencer timetable initialized with ${maxNumberOfBlocks} blocks per slot`, {
207
+ aztecSlotDuration: timetable.aztecSlotDuration,
208
+ ethereumSlotDuration: timetable.ethereumSlotDuration,
209
+ blockDuration: timetable.blockDuration,
210
+ minBlockDuration: timetable.minBlockDuration,
211
+ p2pPropagationTime: timetable.p2pPropagationTime,
212
+ checkpointProposalPrepareTime: timetable.checkpointProposalPrepareTime,
213
+ maxNumberOfBlocks,
214
+ });
215
+
216
+ this.assertConfigMeetsNetworkTxLimits(config, maxNumberOfBlocks);
217
+
218
+ return timetable;
219
+ }
220
+
221
+ /**
222
+ * Checks this node's configured per-block allocation against the network admission limit. A node
223
+ * advertises and admits txs up to the limit derived from the network-minimum multipliers (see
224
+ * {@link computeNetworkTxGasLimits}).
225
+ *
226
+ * Fails startup (and runtime config updates) only when the configured per-block allocation *multipliers*
227
+ * (`perBlockAllocationMultiplier` / `perBlockDAAllocationMultiplier`) are below the network minimums: such
228
+ * a node would accept txs over RPC/gossip that its builder can never pack into a block regardless of block
229
+ * size. Operators may configure a higher (more generous) multiplier, but not a lower one.
230
+ *
231
+ * When the multipliers meet the floor but an absolute per-block gas cap (`maxDABlockGas` / `maxL2BlockGas`)
232
+ * shrinks the builder's effective grant below the network limit, this is legitimate operator
233
+ * restrictiveness — the node simply builds smaller blocks and such txs stay in the pool for other
234
+ * proposers — so we only log a warning rather than failing startup. Restrictive tx-count caps
235
+ * (`maxTxsPerBlock` / `maxTxsPerCheckpoint`) can likewise make the builder skip admitted txs; they are
236
+ * intentionally not modeled here for the same reason.
237
+ */
238
+ private assertConfigMeetsNetworkTxLimits(config: ResolvedSequencerConfig, maxBlocksPerCheckpoint: number) {
239
+ // Mirror CheckpointBuilder.capLimitsByCheckpointBudgets: DA falls back to the general multiplier.
240
+ const l2Multiplier = config.perBlockAllocationMultiplier;
241
+ const daMultiplier = config.perBlockDAAllocationMultiplier ?? l2Multiplier;
242
+
243
+ // The allocation is monotonic in the multiplier, so a multiplier at or above the network minimum
244
+ // guarantees the builder grants at least the network admission limit. Checking the multipliers directly
245
+ // is sufficient (and strictly more conservative than modeling the resulting gas grant).
246
+ if (daMultiplier < MIN_PER_BLOCK_DA_ALLOCATION_MULTIPLIER) {
247
+ throw new Error(
248
+ `perBlockDAAllocationMultiplier (${daMultiplier}) is below the network minimum ` +
249
+ `${MIN_PER_BLOCK_DA_ALLOCATION_MULTIPLIER}; the node would admit txs its own builder can never include.`,
250
+ );
251
+ }
252
+ if (l2Multiplier < MIN_PER_BLOCK_ALLOCATION_MULTIPLIER) {
253
+ throw new Error(
254
+ `perBlockAllocationMultiplier (${l2Multiplier}) is below the network minimum ` +
255
+ `${MIN_PER_BLOCK_ALLOCATION_MULTIPLIER}; the node would admit txs its own builder can never include.`,
256
+ );
257
+ }
258
+
259
+ // Absolute per-block gas caps below the network admission limit are legitimate operator restrictiveness:
260
+ // the node simply builds smaller blocks and such txs stay in the pool for other proposers. Warn only.
261
+ const networkLimit = computeNetworkTxGasLimits({
262
+ maxBlocksPerCheckpoint,
263
+ manaCheckpointBudget: this.l1Constants.rollupManaLimit,
264
+ });
265
+ if (config.maxDABlockGas !== undefined && config.maxDABlockGas < networkLimit.daGas) {
266
+ this.log.warn(
267
+ `Sequencer maxDABlockGas (${config.maxDABlockGas}) is below the network DA admission limit ` +
268
+ `(${networkLimit.daGas}): txs declaring more DA gas are admitted over RPC/gossip but will be skipped ` +
269
+ `by this proposer's own blocks and left in the pool for other proposers.`,
270
+ { maxDABlockGas: config.maxDABlockGas, networkDaGas: networkLimit.daGas, maxBlocksPerCheckpoint },
271
+ );
272
+ }
273
+ if (config.maxL2BlockGas !== undefined && config.maxL2BlockGas < networkLimit.l2Gas) {
274
+ this.log.warn(
275
+ `Sequencer maxL2BlockGas (${config.maxL2BlockGas}) is below the network L2 admission limit ` +
276
+ `(${networkLimit.l2Gas}): txs declaring more L2 gas are admitted over RPC/gossip but will be skipped ` +
277
+ `by this proposer's own blocks and left in the pool for other proposers.`,
278
+ { maxL2BlockGas: config.maxL2BlockGas, networkL2Gas: networkLimit.l2Gas, maxBlocksPerCheckpoint },
279
+ );
280
+ }
127
281
  }
128
282
 
129
283
  /** Initializes the sequencer (precomputes tables). Takes about 3s. */
@@ -131,8 +285,22 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
131
285
  getKzg();
132
286
  }
133
287
 
134
- /** Starts the sequencer and moves to IDLE state. */
288
+ /**
289
+ * Starts (or restarts) the sequencer and moves it to the IDLE state. Idempotent: a start while already
290
+ * running is a no-op, so it never orphans the previous poll loop. A start while STOPPING throws, since the
291
+ * in-flight stop would mark the fresh loop's state STOPPED and orphan it — silently doing nothing would
292
+ * leave the caller believing the sequencer is running when it is on its way to STOPPED. Safe to call after
293
+ * a previous {@link stop} has resolved; the caller must also restart the publishers (see
294
+ * {@link SequencerClient.start}) so L1 publishing is re-enabled.
295
+ */
135
296
  public start() {
297
+ if (this.state === SequencerState.STOPPING) {
298
+ throw new Error('Cannot start sequencer while it is stopping');
299
+ }
300
+ if (this.runningPromise?.isRunning()) {
301
+ this.log.warn('Attempted to start sequencer that is already running');
302
+ return;
303
+ }
136
304
  this.runningPromise = new RunningPromise(
137
305
  this.safeWork.bind(this),
138
306
  this.log,
@@ -143,34 +311,68 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
143
311
  this.log.info('Started sequencer');
144
312
  }
145
313
 
146
- /** Stops the sequencer from building blocks and moves to STOPPED state. */
314
+ /** Triggers an immediate run of the sequencer, bypassing the polling interval. */
315
+ public trigger() {
316
+ return this.runningPromise?.trigger();
317
+ }
318
+
319
+ /**
320
+ * Stops the sequencer from building blocks and moves it to STOPPED state, interrupting the in-flight
321
+ * work() iteration and its pending L1 submissions for a fast shutdown, then draining before returning.
322
+ * Idempotent: a second call while already stopped or stopping is a no-op. Lifecycle calls are expected
323
+ * to be serialized by the caller. For a restartable pause that lets in-flight work finish untouched
324
+ * (e.g. around a test clock warp), use {@link pause} instead.
325
+ */
147
326
  public async stop(): Promise<void> {
327
+ if (this.state === SequencerState.STOPPED || this.state === SequencerState.STOPPING) {
328
+ this.log.debug(`Sequencer already ${this.state.toLowerCase()}, ignoring stop`);
329
+ return;
330
+ }
148
331
  this.log.info(`Stopping sequencer`);
149
332
  this.setState(SequencerState.STOPPING, undefined, { force: true });
150
- this.publisherFactory.interruptAll();
333
+ this.lastCheckpointProposalJob?.interrupt();
334
+ await this.publisherFactory.stopAll();
335
+ // Stop the poll loop and await the in-flight work() iteration. work() registers its fire-and-forget
336
+ // requests synchronously before returning, so once the loop has stopped, pendingRequests holds every
337
+ // request that will ever exist. Interrupting any earlier could miss a request registered by the last
338
+ // in-flight iteration.
151
339
  await this.runningPromise?.stop();
340
+ this.pendingRequests.interruptRequests();
341
+ await this.pendingRequests.awaitRequests();
152
342
  this.setState(SequencerState.STOPPED, undefined, { force: true });
153
343
  this.log.info('Stopped sequencer');
154
344
  }
155
345
 
346
+ /**
347
+ * Gracefully pauses block production so the sequencer can later be resumed with {@link start}: halts the
348
+ * poll loop and waits for the in-flight work() iteration and every pending L1 submission / fallback send
349
+ * to finish, without interrupting them. No interrupt lands mid-build, so no spurious checkpoint-error is
350
+ * emitted and no enqueued checkpoint is dropped. Deliberately does not stop inner services (validator/HA
351
+ * signer, publishers), so the slashing-protection store stays open and the sequencer stays restartable.
352
+ * Used by tests that pause sequencers around an L1 clock warp. Idempotent.
353
+ */
354
+ public async pause(): Promise<void> {
355
+ if (!this.runningPromise?.isRunning()) {
356
+ this.log.debug('Sequencer not running, ignoring pause');
357
+ return;
358
+ }
359
+ this.log.info('Pausing sequencer');
360
+ // Halt the poll loop and let the in-flight iteration finish naturally — no interrupt, and no STOPPING
361
+ // state, since entering STOPPING would make the iteration's own setState calls throw and fail it.
362
+ // work() registers its fire-and-forget requests synchronously before returning, so once the loop has
363
+ // stopped pendingRequests holds every request that will ever exist; awaiting it lets them all finish.
364
+ await this.runningPromise.stop();
365
+ await this.pendingRequests.awaitRequests();
366
+ this.log.info('Paused sequencer');
367
+ }
368
+
156
369
  /** Main sequencer loop with a try/catch */
157
370
  protected async safeWork() {
158
371
  try {
159
372
  await this.work();
160
373
  } catch (err) {
161
374
  this.emit('checkpoint-error', { error: err as Error });
162
- if (err instanceof SequencerTooSlowError) {
163
- // Log as warn only if we had to abort halfway through the block proposal
164
- const logLvl = [SequencerState.INITIALIZING_CHECKPOINT, SequencerState.PROPOSER_CHECK].includes(
165
- err.proposedState,
166
- )
167
- ? ('debug' as const)
168
- : ('warn' as const);
169
- this.log[logLvl](err.message, { now: this.dateProvider.nowInSeconds() });
170
- } else {
171
- // Re-throw other errors
172
- throw err;
173
- }
375
+ throw err;
174
376
  } finally {
175
377
  this.setState(SequencerState.IDLE, undefined);
176
378
  }
@@ -192,14 +394,24 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
192
394
  @trackSpan('Sequencer.work')
193
395
  protected async work() {
194
396
  this.setState(SequencerState.SYNCHRONIZING, undefined);
195
- const { slot, ts, now, epoch } = this.epochCache.getEpochAndSlotInNextL1Slot();
397
+ const { slot, targetSlot, epoch, targetEpoch, ts, nowSeconds } = this.getSlotContextInNextL1Slot();
196
398
 
197
399
  // Check if we are synced and it's our slot, grab a publisher, check previous block invalidation, etc
198
- const checkpointProposalJob = await this.prepareCheckpointProposal(epoch, slot, ts, now);
400
+ const checkpointProposalJob = await this.prepareCheckpointProposal(
401
+ slot,
402
+ targetSlot,
403
+ epoch,
404
+ targetEpoch,
405
+ ts,
406
+ nowSeconds,
407
+ );
199
408
  if (!checkpointProposalJob) {
200
409
  return;
201
410
  }
202
411
 
412
+ // Track the job so we can await its pending L1 submission during shutdown
413
+ this.lastCheckpointProposalJob = checkpointProposalJob;
414
+
203
415
  // Execute the checkpoint proposal job
204
416
  const checkpoint = await checkpointProposalJob.execute();
205
417
 
@@ -208,64 +420,76 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
208
420
  this.lastCheckpointProposed = checkpoint;
209
421
  }
210
422
 
211
- // Log fee strategy comparison if on fisherman
423
+ // Log fee strategy comparison if on fisherman (uses target epoch since we mirror the proposer's perspective)
212
424
  if (
213
425
  this.config.fishermanMode &&
214
- (this.lastEpochForStrategyComparison === undefined || epoch > this.lastEpochForStrategyComparison)
426
+ (this.lastEpochForStrategyComparison === undefined || targetEpoch > this.lastEpochForStrategyComparison)
215
427
  ) {
216
- this.logStrategyComparison(epoch, checkpointProposalJob.getPublisher());
217
- this.lastEpochForStrategyComparison = epoch;
428
+ this.logStrategyComparison(targetEpoch, checkpointProposalJob.getPublisher());
429
+ this.lastEpochForStrategyComparison = targetEpoch;
218
430
  }
219
431
 
220
432
  return checkpoint;
221
433
  }
222
434
 
435
+ /** Returns slot and target slot from a single clock snapshot. */
436
+ protected getSlotContextInNextL1Slot(): SequencerSlotContext {
437
+ const { slot, ts, nowSeconds, epoch } = this.epochCache.getEpochAndSlotInNextL1Slot();
438
+ const targetSlot = SlotNumber(slot + PROPOSER_PIPELINING_SLOT_OFFSET);
439
+ return { slot, targetSlot, epoch, targetEpoch: getEpochAtSlot(targetSlot, this.l1Constants), ts, nowSeconds };
440
+ }
441
+
223
442
  /**
224
443
  * Prepares the checkpoint proposal by performing all necessary checks and setup.
225
444
  * This is the initial step in the main loop.
226
445
  * @returns CheckpointProposalJob if successful, undefined if we are not yet synced or are not the proposer.
227
446
  */
228
447
  @trackSpan('Sequencer.prepareCheckpointProposal')
229
- private async prepareCheckpointProposal(
230
- epoch: EpochNumber,
448
+ protected async prepareCheckpointProposal(
231
449
  slot: SlotNumber,
450
+ targetSlot: SlotNumber,
451
+ epoch: EpochNumber,
452
+ targetEpoch: EpochNumber,
232
453
  ts: bigint,
233
- now: bigint,
454
+ nowSeconds: bigint,
234
455
  ): Promise<CheckpointProposalJob | undefined> {
235
- // Check we have not already processed this slot (cheapest check)
236
- // We only check this if enforce timetable is set, since we want to keep processing the same slot if we are not
237
- // running against actual time (eg when we use sandbox-style automining)
238
- if (
239
- this.lastSlotForCheckpointProposalJob &&
240
- this.lastSlotForCheckpointProposalJob >= slot &&
241
- this.config.enforceTimeTable
242
- ) {
243
- this.log.trace(`Slot ${slot} has already been processed`);
456
+ // Check we have not already processed this target slot (cheapest check).
457
+ if (this.lastSlotForCheckpointProposalJob && this.lastSlotForCheckpointProposalJob >= targetSlot) {
458
+ this.log.trace(`Target slot ${targetSlot} has already been processed`);
244
459
  return undefined;
245
460
  }
246
461
 
247
- // But if we have already proposed for this slot, the we definitely have to skip it, automining or not
248
- if (this.lastCheckpointProposed && this.lastCheckpointProposed.header.slotNumber >= slot) {
249
- this.log.trace(`Slot ${slot} has already been published as checkpoint ${this.lastCheckpointProposed.number}`);
462
+ // But if we have already proposed for this slot, then we definitely have to skip it, automining or not
463
+ if (this.lastCheckpointProposed && this.lastCheckpointProposed.header.slotNumber >= targetSlot) {
464
+ this.log.trace(
465
+ `Slot ${targetSlot} has already been published as checkpoint ${this.lastCheckpointProposed.number}`,
466
+ );
250
467
  return undefined;
251
468
  }
252
469
 
253
- // Check all components are synced to latest as seen by the archiver (queries all subsystems)
254
- const syncedTo = await this.checkSync({ ts, slot });
255
- if (!syncedTo) {
256
- await this.tryVoteWhenSyncFails({ slot, ts });
470
+ // Test-only: skip proposing for explicitly paused slots. Attestation paths run in the validator
471
+ // client and are not gated by this hook, so paused proposers still attest to others' proposals.
472
+ if (this.config.pauseProposingForSlots?.some(s => s === targetSlot)) {
473
+ this.log.warn(`Skipping proposal for paused slot ${targetSlot} (test-only pauseProposingForSlots hook)`, {
474
+ targetSlot,
475
+ });
257
476
  return undefined;
258
477
  }
259
478
 
260
- // If escape hatch is open for this epoch, do not start checkpoint proposal work and do not attempt invalidations.
479
+ // Cheap proposer check first: most nodes are not the proposer for most slots, so gate the
480
+ // expensive multi-subsystem checkSync (and the rest of the build path) behind it. Computed once
481
+ // here and reused for the escape-hatch voting path below. No setState/timing gate on this path:
482
+ // the build-start deadline gate runs only on the proposer build path after a successful checkSync.
483
+ const [canPropose, proposer] = await this.checkCanPropose(targetSlot);
484
+
485
+ // If escape hatch is open for the target epoch, do not start checkpoint proposal work and do not attempt invalidations.
261
486
  // Still perform governance/slashing voting (as proposer) once per slot.
262
- const isEscapeHatchOpen = await this.epochCache.isEscapeHatchOpen(epoch);
487
+ // When pipelining, we check the target epoch (slot+1's epoch) since that's the epoch we're building for.
488
+ const isEscapeHatchOpen = await this.epochCache.isEscapeHatchOpen(targetEpoch);
263
489
 
264
490
  if (isEscapeHatchOpen) {
265
- this.setState(SequencerState.PROPOSER_CHECK, slot);
266
- const [canPropose, proposer] = await this.checkCanPropose(slot);
267
491
  if (canPropose) {
268
- await this.tryVoteWhenEscapeHatchOpen({ slot, proposer });
492
+ await this.tryVoteWhenEscapeHatchOpen({ slot, targetSlot, proposer });
269
493
  } else {
270
494
  this.log.trace(`Escape hatch open but we are not proposer, skipping vote-only actions`, {
271
495
  slot,
@@ -276,32 +500,80 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
276
500
  return undefined;
277
501
  }
278
502
 
503
+ // If we are not the proposer, check whether we should invalidate an invalid pending chain (a
504
+ // liveness backstop) and bail before any sync work or build-timing gate. This reads only the
505
+ // archiver's pending-chain validation status, which is authoritative on its own, instead of
506
+ // running the full sync check. Wrapped in try/catch because this leaner path skips the broader
507
+ // proposed-checkpoint/tip coherence screen that checkSync applied, so transient archiver
508
+ // incoherence surfaces as a quiet skip rather than a work-loop error.
509
+ if (!canPropose) {
510
+ try {
511
+ const pendingChainValidationStatus = await this.l2BlockSource.getPendingChainValidationStatus();
512
+ await this.considerInvalidatingCheckpoint(pendingChainValidationStatus, slot);
513
+ } catch (err) {
514
+ this.log.warn(`Failed to consider invalidating checkpoint`, { err, slot, targetSlot });
515
+ }
516
+ return undefined;
517
+ }
518
+
519
+ // Explicit build-loop entry gate: if we are past the latest useful block-building start for the
520
+ // target slot, abandon building for this slot. The proposer prioritizes the ideal L1-publish path
521
+ // and does not plan around the late consensus-handoff path. This is the proposer build path's
522
+ // timing gate; it runs only after we know we are the synced proposer, so non-proposer invalidation
523
+ // and escape-hatch voting (which returned above) are never gated by build timing. Vote-only paths
524
+ // still run when block building is abandoned.
525
+ const startDeadline = this.timetable.getBuildStartDeadline(targetSlot);
526
+ const nowForStartGate = this.dateProvider.now() / 1000;
527
+ if (nowForStartGate > startDeadline) {
528
+ this.log.debug(`Past start deadline for slot ${targetSlot}, abandoning block building`, {
529
+ targetSlot,
530
+ nowForStartGate,
531
+ startDeadline,
532
+ });
533
+ // Mark the slot as attempted so a deadline abort is not retried within the same slot. Vote-only actions
534
+ // still need to run because sync can succeed even when it is too late to start building a checkpoint.
535
+ await this.tryVoteAndPruneWhenCannotBuild({ slot, targetSlot });
536
+ this.lastSlotForCheckpointProposalJob = targetSlot;
537
+ return undefined;
538
+ }
539
+
540
+ // We are the proposer, the escape hatch is closed, and we have time before the build start deadline.
541
+ // Now run the full sync check before building.
542
+ const syncedTo = await this.checkSync({ ts, slot });
543
+ if (!syncedTo) {
544
+ return undefined;
545
+ }
546
+
279
547
  // Next checkpoint follows from the last synced one
280
548
  const checkpointNumber = CheckpointNumber(syncedTo.checkpointNumber + 1);
281
549
 
282
550
  const logCtx = {
283
- now,
551
+ nowSeconds,
284
552
  syncedToL2Slot: syncedTo.syncedL2Slot,
285
553
  slot,
554
+ targetSlot,
286
555
  slotTs: ts,
287
556
  checkpointNumber,
288
557
  isPendingChainValid: pick(syncedTo.pendingChainValidationStatus, 'valid', 'reason', 'invalidIndex'),
289
558
  };
290
559
 
291
- // Check that we are a proposer for the next slot
292
- this.setState(SequencerState.PROPOSER_CHECK, slot);
293
- const [canPropose, proposer] = await this.checkCanPropose(slot);
560
+ // We are the synced proposer within the build window; enter the proposer-check state and build.
561
+ this.setState(SequencerState.PROPOSER_CHECK, targetSlot);
294
562
 
295
- // If we are not a proposer check if we should invalidate an invalid checkpoint, and bail
296
- if (!canPropose) {
297
- await this.considerInvalidatingCheckpoint(syncedTo, slot);
563
+ // Guard: don't exceed 1-deep pipeline. Without a proposed checkpoint, we can only build
564
+ // confirmed + 1. With a proposed checkpoint, we can build confirmed + 2.
565
+ const confirmedCkpt = syncedTo.checkpointedCheckpointNumber;
566
+ if (checkpointNumber > confirmedCkpt + 2) {
567
+ this.log.verbose(
568
+ `Skipping slot ${targetSlot}: checkpoint ${checkpointNumber} exceeds max pipeline depth (confirmed=${confirmedCkpt})`,
569
+ );
298
570
  return undefined;
299
571
  }
300
572
 
301
- // Check that the slot is not taken by a block already (should never happen, since only us can propose for this slot)
302
- if (syncedTo.blockData && syncedTo.blockData.header.getSlot() >= slot) {
573
+ // Check that the target slot is not taken by a block already (should never happen, since only us can propose for this slot)
574
+ if (syncedTo.blockData && syncedTo.blockData.header.getSlot() >= targetSlot) {
303
575
  this.log.warn(
304
- `Cannot propose block at next L2 slot ${slot} since that slot was taken by block ${syncedTo.blockNumber}`,
576
+ `Cannot propose block at target slot ${targetSlot} since that slot was taken by block ${syncedTo.blockNumber}`,
305
577
  { ...logCtx, block: syncedTo.blockData.header.toInspect() },
306
578
  );
307
579
  this.metrics.recordCheckpointPrecheckFailed('slot_already_taken');
@@ -316,37 +588,106 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
316
588
  const { attestorAddress, publisher } = await this.publisherFactory.create(proposerForPublisher);
317
589
  this.log.verbose(`Created publisher at address ${publisher.getSenderAddress()} for attestor ${attestorAddress}`);
318
590
 
319
- // In fisherman mode, set the actual proposer's address for simulations
320
- if (this.config.fishermanMode && proposer) {
321
- publisher.setProposerAddressForSimulation(proposer);
322
- this.log.debug(`Set proposer address ${proposer} for simulation in fisherman mode`);
591
+ // Prepare invalidation request if the pending chain is invalid (returns undefined if no need).
592
+ // Only simulate invalidation when there's no proposed parent, since we assume the proposed parent
593
+ // will invalidate the currently invalid checkpoint on L1.
594
+ const invalidateCheckpoint =
595
+ syncedTo.hasProposedCheckpoint || syncedTo.pendingChainValidationStatus.valid
596
+ ? undefined
597
+ : await publisher.simulateInvalidateCheckpoint(syncedTo.pendingChainValidationStatus);
598
+
599
+ // Determine the correct archive and L1 state overrides for the canProposeAt check.
600
+ // The L1 contract reads archives[proposedCheckpointNumber] and compares it with the provided archive.
601
+ // When invalidating or pipelining, the local archive may differ from L1's, so we adjust accordingly.
602
+ let archiveForCheck = syncedTo.archive;
603
+
604
+ if (syncedTo.hasProposedCheckpoint) {
605
+ this.metrics.recordPipelineDepth(syncedTo.checkpointNumber - syncedTo.checkpointedCheckpointNumber);
606
+ this.log.verbose(
607
+ `Building on top of proposed checkpoint (pending=${syncedTo.proposedCheckpointData?.checkpointNumber}) for target slot ${targetSlot}`,
608
+ { targetSlot, parentCheckpointNumber: CheckpointNumber(checkpointNumber - 1) },
609
+ );
610
+ // Match what L1 will see at archives[pending] once the proposed parent lands: the parent's
611
+ // own archive root from the gossiped proposal. `syncedTo.archive` is the world-state-local
612
+ // view and can transiently diverge from the proposed parent (e.g. before the proposed
613
+ // parent's blocks have been applied locally); diverging here would cause the canProposeAt
614
+ // override to set archives[pending] to one value while we present another for comparison.
615
+ archiveForCheck = syncedTo.proposedCheckpointData!.archive.root;
616
+ } else if (invalidateCheckpoint) {
617
+ // After invalidation, L1 will roll back to checkpoint N-1. The archive at N-1 already
618
+ // exists on L1, so we just pass the matching archive (the lastArchive of the invalid checkpoint).
619
+ archiveForCheck = invalidateCheckpoint.lastArchive;
620
+ this.metrics.recordPipelineDepth(0);
621
+ } else {
622
+ this.metrics.recordPipelineDepth(0);
323
623
  }
324
624
 
325
- // Prepare invalidation request if the pending chain is invalid (returns undefined if no need)
326
- const invalidateCheckpoint = await publisher.simulateInvalidateCheckpoint(syncedTo.pendingChainValidationStatus);
625
+ // Build the simulation plan: pending/proven override from pipelining or invalidation (or the
626
+ // current snapshot when neither applies, to short-circuit any pending prune in simulation),
627
+ // plus the parent checkpoint cell and fee header when pipelining.
628
+ const simulationOverridesPlan = await buildCheckpointSimulationOverridesPlan({
629
+ checkpointNumber,
630
+ proposedCheckpointData: syncedTo.hasProposedCheckpoint ? syncedTo.proposedCheckpointData : undefined,
631
+ invalidateToPendingCheckpointNumber: invalidateCheckpoint?.forcePendingCheckpointNumber,
632
+ checkpointedCheckpointNumber: syncedTo.checkpointedCheckpointNumber,
633
+ rollup: this.rollupContract,
634
+ signatureContext: this.signatureContext,
635
+ log: this.log,
636
+ });
637
+
638
+ // The plan always pins both pending/proven (to short-circuit `canPruneAtTime` in simulation),
639
+ // so `provenOverride` always reflects the assumed proven checkpoint we are pinning the
640
+ // simulation to. We additionally warn when the pin is load-bearing — i.e. when a prune would
641
+ // actually fire at the target slot without it — so observers can spot "we are building
642
+ // optimistically across a pruning boundary" in the logs.
643
+ const provenOverride = simulationOverridesPlan?.chainTipsOverride?.proven;
644
+ if (provenOverride !== undefined && (await this.l2BlockSource.isPruneDueAtSlot(targetSlot))) {
645
+ this.log.warn(
646
+ `Assuming proof for epoch ending at checkpoint ${provenOverride} lands by target slot ${targetSlot}`,
647
+ { checkpointNumber, slot, targetSlot, provenOverride },
648
+ );
649
+ }
327
650
 
328
- // Check with the rollup contract if we can indeed propose at the next L2 slot. This check should not fail
329
- // if all the previous checks are good, but we do it just in case.
330
- const canProposeCheck = await publisher.canProposeAtNextEthBlock(
331
- syncedTo.archive,
651
+ this.emit('preparing-checkpoint', {
652
+ targetSlot,
653
+ checkpointNumber,
654
+ hadProposedParent: syncedTo.hasProposedCheckpoint,
655
+ provenOverride,
656
+ simulatedPending: simulationOverridesPlan?.chainTipsOverride?.pending,
657
+ });
658
+
659
+ const canProposeCheck = await publisher.canProposeAt(
660
+ archiveForCheck,
332
661
  proposer ?? EthAddress.ZERO,
333
- invalidateCheckpoint,
662
+ simulationOverridesPlan,
334
663
  );
335
664
 
665
+ const proposeContext = {
666
+ hasProposedCheckpoint: syncedTo.hasProposedCheckpoint,
667
+ proposedCheckpointNumber: syncedTo.proposedCheckpointData?.checkpointNumber,
668
+ checkpointedCheckpointNumber: syncedTo.checkpointedCheckpointNumber,
669
+ isInvalidating: !!invalidateCheckpoint,
670
+ invalidatingCheckpointNumber: invalidateCheckpoint?.checkpointNumber,
671
+ archiveForCheck: archiveForCheck.toString(),
672
+ overridePendingCheckpointNumber: simulationOverridesPlan?.chainTipsOverride?.pending,
673
+ overrideArchive: simulationOverridesPlan?.pendingCheckpointState?.archive,
674
+ overrideFeeHeader: simulationOverridesPlan?.pendingCheckpointState?.feeHeader,
675
+ };
676
+
336
677
  if (canProposeCheck === undefined) {
337
678
  this.log.warn(
338
679
  `Cannot propose checkpoint ${checkpointNumber} at slot ${slot} due to failed rollup contract check`,
339
- logCtx,
680
+ { ...logCtx, ...proposeContext },
340
681
  );
341
682
  this.emit('proposer-rollup-check-failed', { reason: 'Rollup contract check failed', slot });
342
683
  this.metrics.recordCheckpointPrecheckFailed('rollup_contract_check_failed');
343
684
  return undefined;
344
685
  }
345
686
 
346
- if (canProposeCheck.slot !== slot) {
687
+ if (canProposeCheck.slot !== targetSlot) {
347
688
  this.log.warn(
348
- `Cannot propose block due to slot mismatch with rollup contract (this can be caused by a clock out of sync). Expected slot ${slot} but got ${canProposeCheck.slot}.`,
349
- { ...logCtx, rollup: canProposeCheck, expectedSlot: slot },
689
+ `Cannot propose block due to slot mismatch with rollup contract (this can be caused by a clock out of sync). Expected slot ${targetSlot} but got ${canProposeCheck.slot}.`,
690
+ { ...logCtx, ...proposeContext, rollup: canProposeCheck, expectedSlot: targetSlot },
350
691
  );
351
692
  this.emit('proposer-rollup-check-failed', { reason: 'Slot mismatch', slot });
352
693
  this.metrics.recordCheckpointPrecheckFailed('slot_mismatch');
@@ -356,45 +697,58 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
356
697
  if (canProposeCheck.checkpointNumber !== checkpointNumber) {
357
698
  this.log.warn(
358
699
  `Cannot propose due to block mismatch with rollup contract (this can be caused by a pending archiver sync). Expected checkpoint ${checkpointNumber} but got ${canProposeCheck.checkpointNumber}.`,
359
- { ...logCtx, rollup: canProposeCheck, expectedSlot: slot },
700
+ { ...logCtx, ...proposeContext, rollup: canProposeCheck, expectedSlot: slot },
360
701
  );
361
702
  this.emit('proposer-rollup-check-failed', { reason: 'Block mismatch', slot });
362
703
  this.metrics.recordCheckpointPrecheckFailed('block_number_mismatch');
363
704
  return undefined;
364
705
  }
365
706
 
366
- this.lastSlotForCheckpointProposalJob = slot;
367
- await this.p2pClient.prepareForSlot(slot);
368
- this.log.info(`Preparing checkpoint proposal ${checkpointNumber} at slot ${slot}`, { ...logCtx, proposer });
707
+ this.lastSlotForCheckpointProposalJob = targetSlot;
708
+
709
+ await this.p2pClient.prepareForSlot(targetSlot);
710
+ this.log.info(
711
+ `Preparing checkpoint proposal ${checkpointNumber} for target slot ${targetSlot} during wall-clock slot ${slot}`,
712
+ {
713
+ ...logCtx,
714
+ ...proposeContext,
715
+ proposer,
716
+ },
717
+ );
369
718
 
370
719
  // Create and return the checkpoint proposal job
371
720
  return this.createCheckpointProposalJob(
372
- epoch,
373
- slot,
721
+ targetSlot,
722
+ targetEpoch,
374
723
  checkpointNumber,
375
724
  syncedTo.blockNumber,
725
+ syncedTo.checkpointedCheckpointNumber,
376
726
  proposer,
377
727
  publisher,
378
728
  attestorAddress,
379
729
  invalidateCheckpoint,
730
+ syncedTo.proposedCheckpointData,
380
731
  );
381
732
  }
382
733
 
383
734
  protected createCheckpointProposalJob(
384
- epoch: EpochNumber,
385
- slot: SlotNumber,
735
+ targetSlot: SlotNumber,
736
+ targetEpoch: EpochNumber,
386
737
  checkpointNumber: CheckpointNumber,
387
738
  syncedToBlockNumber: BlockNumber,
739
+ checkpointedCheckpointNumber: CheckpointNumber,
388
740
  proposer: EthAddress | undefined,
389
741
  publisher: SequencerPublisher,
390
742
  attestorAddress: EthAddress,
391
743
  invalidateCheckpoint: InvalidateCheckpointRequest | undefined,
744
+ proposedCheckpointData?: ProposedCheckpointData,
392
745
  ): CheckpointProposalJob {
393
746
  return new CheckpointProposalJob(
394
- epoch,
395
- slot,
747
+ targetSlot,
748
+ targetEpoch,
396
749
  checkpointNumber,
397
750
  syncedToBlockNumber,
751
+ checkpointedCheckpointNumber,
398
752
  proposer,
399
753
  publisher,
400
754
  attestorAddress,
@@ -408,16 +762,20 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
408
762
  this.checkpointsBuilder,
409
763
  this.l2BlockSource,
410
764
  this.l1Constants,
765
+ this.signatureContext,
411
766
  this.config,
412
767
  this.timetable,
413
768
  this.slasherClient,
414
769
  this.epochCache,
415
770
  this.dateProvider,
416
771
  this.metrics,
772
+ this.checkpointProposalJobMetrics.createRecorder(),
417
773
  this,
774
+ this.pendingRequests,
418
775
  this.setState.bind(this),
419
776
  this.tracer,
420
777
  this.log.getBindings(),
778
+ proposedCheckpointData,
421
779
  );
422
780
  }
423
781
 
@@ -429,9 +787,11 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
429
787
  }
430
788
 
431
789
  /**
432
- * Internal helper for setting the sequencer state and checks if we have enough time left in the slot to transition to the new state.
790
+ * Internal helper for setting the sequencer state. Pure: sets the state, emits `state-changed`, and
791
+ * records metrics. Timing deadlines are queried explicitly at the relevant call sites, not gated here.
433
792
  * @param proposedState - The new state to transition to.
434
- * @param slotNumber - The current slot number.
793
+ * @param slotNumber - The target slot being proposed for, emitted as `targetSlot` on the event payload
794
+ * and used to anchor `secondsIntoBuildFrame`. Undefined for lifecycle states with no associated slot.
435
795
  * @param force - Whether to force the transition even if the sequencer is stopped.
436
796
  */
437
797
  protected setState(
@@ -447,25 +807,39 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
447
807
  this.log.warn(`Cannot set sequencer from ${this.state} to ${proposedState} as it is stopped.`);
448
808
  return;
449
809
  }
450
- let secondsIntoSlot = undefined;
451
- if (slotNumber !== undefined) {
452
- secondsIntoSlot = this.getSecondsIntoSlot(slotNumber);
453
- this.timetable.assertTimeLeft(proposedState, secondsIntoSlot);
454
- }
810
+ const secondsIntoBuildFrame = slotNumber !== undefined ? this.getSecondsIntoBuildFrame(slotNumber) : undefined;
811
+
812
+ const oldState = this.state;
813
+ const oldStateSlotNumber = this.stateSlotNumber;
814
+ const stateChanged = proposedState !== oldState;
815
+ // Wall-clock time spent in the previous state: the delta between consecutive state-changing setState
816
+ // calls, read from the date provider so it tracks simulated time under a test/manual clock.
817
+ const transitionAtMs = this.dateProvider.now();
818
+ const stateDurationMs = transitionAtMs - this.stateEnteredAtMs;
455
819
 
456
820
  const boringStates = [SequencerState.IDLE, SequencerState.SYNCHRONIZING];
457
821
  const logLevel =
458
- boringStates.includes(proposedState) && boringStates.includes(this.state)
459
- ? ('trace' as const)
460
- : ('debug' as const);
461
- this.log[logLevel](`Transitioning from ${this.state} to ${proposedState}`, { slotNumber, secondsIntoSlot });
822
+ boringStates.includes(proposedState) && boringStates.includes(oldState) ? ('trace' as const) : ('debug' as const);
823
+ this.stateLog[logLevel](`Transitioning from ${oldState} to ${proposedState}`, {
824
+ oldState,
825
+ newState: proposedState,
826
+ slotNumber,
827
+ stateSlotNumber: oldStateSlotNumber,
828
+ secondsIntoBuildFrame,
829
+ ...(stateChanged && { stateDurationMs: Math.ceil(stateDurationMs) }),
830
+ });
462
831
 
463
832
  this.emit('state-changed', {
464
- oldState: this.state,
833
+ oldState,
465
834
  newState: proposedState,
466
- secondsIntoSlot,
467
- slot: slotNumber,
835
+ secondsIntoBuildFrame,
836
+ targetSlot: slotNumber,
468
837
  });
838
+ if (stateChanged) {
839
+ this.metrics.recordStateDuration(stateDurationMs, oldState);
840
+ this.stateEnteredAtMs = transitionAtMs;
841
+ this.stateSlotNumber = slotNumber;
842
+ }
469
843
  this.state = proposedState;
470
844
  }
471
845
 
@@ -475,8 +849,8 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
475
849
  */
476
850
  protected async checkSync(args: { ts: bigint; slot: SlotNumber }): Promise<SequencerSyncCheckResult | undefined> {
477
851
  // Check that the archiver has fully synced the L2 slot before the one we want to propose in.
478
- // TODO(#14766): Archiver reports L1 timestamp based on L1 blocks seen, which means that a missed L1 block will
479
- // cause the archiver L1 timestamp to fall behind, and cause this sequencer to start processing one L1 slot later.
852
+ // The archiver reports sync progress via L1 block timestamps and synced checkpoint slots.
853
+ // See getSyncedL2SlotNumber for how missed L1 blocks are handled.
480
854
  const syncedL2Slot = await this.l2BlockSource.getSyncedL2SlotNumber();
481
855
  const { slot } = args;
482
856
  if (syncedL2Slot === undefined || syncedL2Slot + 1 < slot) {
@@ -492,45 +866,83 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
492
866
  number: syncSummary.latestBlockNumber,
493
867
  hash: syncSummary.latestBlockHash,
494
868
  })),
495
- this.l2BlockSource.getL2Tips().then(t => t.proposed),
869
+ this.l2BlockSource.getL2Tips().then(t => ({ proposed: t.proposed, checkpointed: t.checkpointed })),
496
870
  this.p2pClient.getStatus().then(p2p => p2p.syncedToL2Block),
497
- this.l1ToL2MessageSource.getL2Tips().then(t => t.proposed),
871
+ this.l1ToL2MessageSource.getL2Tips().then(t => ({ proposed: t.proposed, checkpointed: t.checkpointed })),
498
872
  this.l2BlockSource.getPendingChainValidationStatus(),
873
+ this.l2BlockSource.getProposedCheckpointData(),
499
874
  ] as const);
500
875
 
501
- const [worldState, l2BlockSource, p2p, l1ToL2MessageSource, pendingChainValidationStatus] = syncedBlocks;
876
+ const [worldState, l2Tips, p2p, l1ToL2MessageSourceTips, pendingChainValidationStatus, proposedCheckpointData] =
877
+ syncedBlocks;
502
878
 
503
- // Handle zero as a special case, since the block hash won't match across services if we're changing the prefilled data for the genesis block,
504
- // as the world state can compute the new genesis block hash, but other components use the hardcoded constant.
505
- // TODO(palla/mbps): Fix the above. All components should be able to handle dynamic genesis block hashes.
506
879
  const result =
507
- (l2BlockSource.number === 0 && worldState.number === 0 && p2p.number === 0 && l1ToL2MessageSource.number === 0) ||
508
- (worldState.hash === l2BlockSource.hash &&
509
- p2p.hash === l2BlockSource.hash &&
510
- l1ToL2MessageSource.hash === l2BlockSource.hash);
880
+ worldState.hash === l2Tips.proposed.hash &&
881
+ p2p.hash === l2Tips.proposed.hash &&
882
+ l1ToL2MessageSourceTips.proposed.hash === l2Tips.proposed.hash &&
883
+ l1ToL2MessageSourceTips.checkpointed.block.hash === l2Tips.checkpointed.block.hash &&
884
+ l1ToL2MessageSourceTips.checkpointed.checkpoint.hash === l2Tips.checkpointed.checkpoint.hash;
511
885
 
512
886
  if (!result) {
513
- this.log.debug(`Sequencer sync check failed`, { worldState, l2BlockSource, p2p, l1ToL2MessageSource });
887
+ this.log.debug(`Sequencer sync check failed`, {
888
+ worldState,
889
+ l2BlockSource: l2Tips.proposed,
890
+ p2p,
891
+ l1ToL2MessageSourceTips,
892
+ });
514
893
  return undefined;
515
894
  }
516
895
 
517
- // Special case for genesis state
518
896
  const blockNumber = worldState.number;
519
- if (blockNumber < INITIAL_L2_BLOCK_NUM) {
520
- const archive = new Fr((await this.worldState.getCommitted().getTreeInfo(MerkleTreeId.ARCHIVE)).root);
521
- return {
522
- checkpointNumber: CheckpointNumber.ZERO,
523
- blockNumber: BlockNumber.ZERO,
524
- archive,
897
+ const blockData = await this.l2BlockSource.getBlockData({ number: blockNumber });
898
+ if (!blockData) {
899
+ this.log.warn(`Sequencer sync check failed: failed to get L2 block data ${blockNumber} from the archiver`, {
900
+ blockNumber,
901
+ l2Tips,
525
902
  syncedL2Slot,
526
- pendingChainValidationStatus,
903
+ ...args,
904
+ });
905
+ return undefined;
906
+ }
907
+
908
+ // Refuse to build a checkpoint on top of a proposed block whose enclosing checkpoint was never
909
+ // proposed. Under pipelining we may have received and reexecuted such a block locally — advancing
910
+ // our world-state tip past the checkpointed tip — while the proposing node never published the
911
+ // matching proposed checkpoint (e.g. it crashed before assembling it). Building on this orphan block
912
+ // would fork the chain off a tip no other node can follow. The archiver prunes these orphan blocks
913
+ // once their build slot ends; this guard is the correctness barrier during the grace window before.
914
+ // `getProposedCheckpointData()` returns the latest proposed checkpoint payload, which is always
915
+ // the leading one (a proposed entry is only stored beyond the confirmed frontier and is deleted
916
+ // on confirmation). It carries no tip, so there is no tip-vs-payload split read to reconcile.
917
+ if (
918
+ blockData.checkpointNumber > l2Tips.checkpointed.checkpoint.number &&
919
+ proposedCheckpointData?.checkpointNumber !== blockData.checkpointNumber
920
+ ) {
921
+ const logCtx = {
922
+ blockCheckpointNumber: blockData.checkpointNumber,
923
+ checkpointedCheckpointNumber: l2Tips.checkpointed.checkpoint.number,
924
+ proposedCheckpointTipNumber: proposedCheckpointData?.checkpointNumber,
925
+ blockNumber: blockData.header.getBlockNumber(),
926
+ blockSlot: blockData.header.getSlot(),
927
+ syncedL2Slot,
928
+ ...args,
527
929
  };
930
+
931
+ this.log.debug(`Waiting for proposed checkpoint to catch up with reexecuted block`, logCtx);
932
+ return undefined;
528
933
  }
529
934
 
530
- const blockData = await this.l2BlockSource.getBlockData(blockNumber);
531
- if (!blockData) {
532
- // this shouldn't really happen because a moment ago we checked that all components were in sync
533
- this.log.error(`Failed to get L2 block data ${blockNumber} from the archiver with all components in sync`);
935
+ const hasProposedCheckpoint = proposedCheckpointData !== undefined;
936
+
937
+ // Check that the proposed checkpoint is indeed the parent of the checkpoint we'll be building
938
+ // The checkpoint number to build is derived as blockData.checkpointNumber + 1
939
+ if (proposedCheckpointData && proposedCheckpointData.checkpointNumber !== blockData.checkpointNumber) {
940
+ this.log.warn(`Sequencer sync check failed: proposed checkpoint number mismatch`, {
941
+ proposedCheckpointNumber: proposedCheckpointData.checkpointNumber,
942
+ blockCheckpointNumber: blockData.checkpointNumber,
943
+ syncedL2Slot,
944
+ ...args,
945
+ });
534
946
  return undefined;
535
947
  }
536
948
 
@@ -538,7 +950,10 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
538
950
  blockData,
539
951
  blockNumber: blockData.header.getBlockNumber(),
540
952
  checkpointNumber: blockData.checkpointNumber,
953
+ checkpointedCheckpointNumber: l2Tips.checkpointed.checkpoint.number,
541
954
  archive: blockData.archive.root,
955
+ hasProposedCheckpoint,
956
+ proposedCheckpointData,
542
957
  syncedL2Slot,
543
958
  pendingChainValidationStatus,
544
959
  };
@@ -548,20 +963,20 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
548
963
  * Checks if we are the proposer for the next slot.
549
964
  * @returns True if we can propose, and the proposer address (undefined if anyone can propose)
550
965
  */
551
- protected async checkCanPropose(slot: SlotNumber): Promise<[boolean, EthAddress | undefined]> {
966
+ protected async checkCanPropose(targetSlot: SlotNumber): Promise<[boolean, EthAddress | undefined]> {
552
967
  let proposer: EthAddress | undefined;
553
968
 
554
969
  try {
555
- proposer = await this.epochCache.getProposerAttesterAddressInSlot(slot);
970
+ proposer = await this.epochCache.getProposerAttesterAddressInSlot(targetSlot);
556
971
  } catch (e) {
557
972
  if (e instanceof NoCommitteeError) {
558
- if (this.lastSlotForNoCommitteeWarning !== slot) {
559
- this.lastSlotForNoCommitteeWarning = slot;
560
- this.log.warn(`Cannot propose at next L2 slot ${slot} since the committee does not exist on L1`);
973
+ if (this.lastSlotForNoCommitteeWarning !== targetSlot) {
974
+ this.lastSlotForNoCommitteeWarning = targetSlot;
975
+ this.log.warn(`Cannot propose at target slot ${targetSlot} since the committee does not exist on L1`);
561
976
  }
562
977
  return [false, undefined];
563
978
  }
564
- this.log.error(`Error getting proposer for slot ${slot}`, e);
979
+ this.log.error(`Error getting proposer for target slot ${targetSlot}`, e);
565
980
  return [false, undefined];
566
981
  }
567
982
 
@@ -578,55 +993,56 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
578
993
  const weAreProposer = validatorAddresses.some(addr => addr.equals(proposer));
579
994
 
580
995
  if (!weAreProposer) {
581
- this.log.debug(`Cannot propose at slot ${slot} since we are not a proposer`, { validatorAddresses, proposer });
996
+ this.log.debug(`Cannot propose at target slot ${targetSlot} since we are not a proposer`, {
997
+ targetSlot,
998
+ validatorAddresses,
999
+ proposer,
1000
+ });
582
1001
  return [false, proposer];
583
1002
  }
584
1003
 
1004
+ this.log.info(`We are the proposer for pipeline slot ${targetSlot}`, {
1005
+ targetSlot,
1006
+ proposer,
1007
+ });
585
1008
  return [true, proposer];
586
1009
  }
587
1010
 
588
1011
  /**
589
- * Tries to vote on slashing actions and governance when the sync check fails but we're past the max time for initializing a proposal.
590
- * This allows the sequencer to participate in governance/slashing votes even when it cannot build blocks.
1012
+ * Tries to vote on slashing actions and governance and to prune when we cannot build and are past the
1013
+ * block-building window. This allows the sequencer to participate in governance/slashing votes even when it
1014
+ * cannot build blocks, and to prune the pending chain so it can recover from bad data that is blocking sync.
591
1015
  */
592
- @trackSpan('Seqeuencer.tryVoteWhenSyncFails', ({ slot }) => ({ [Attributes.SLOT_NUMBER]: slot }))
593
- protected async tryVoteWhenSyncFails(args: { slot: SlotNumber; ts: bigint }): Promise<void> {
594
- const { slot } = args;
1016
+ @trackSpan('Sequencer.tryVoteAndPruneWhenCannotBuild', ({ slot }) => ({ [Attributes.SLOT_NUMBER]: slot }))
1017
+ protected async tryVoteAndPruneWhenCannotBuild(args: { slot: SlotNumber; targetSlot: SlotNumber }): Promise<void> {
1018
+ const { slot, targetSlot } = args;
595
1019
 
596
1020
  // Prevent duplicate attempts in the same slot
597
- if (this.lastSlotForFallbackVote === slot) {
598
- this.log.trace(`Already attempted to vote in slot ${slot} (skipping)`);
1021
+ if (this.lastSlotForFallbackAction === slot) {
1022
+ this.log.trace(`Already attempted fallback actions in slot ${slot} (skipping)`);
599
1023
  return;
600
1024
  }
601
1025
 
602
- // Check if we're past the max time for initializing a proposal
603
- const secondsIntoSlot = this.getSecondsIntoSlot(slot);
604
- const maxAllowedTime = this.timetable.getMaxAllowedTime(SequencerState.INITIALIZING_CHECKPOINT);
605
-
606
- // If we haven't exceeded the time limit for initializing a proposal, don't proceed with voting
607
- // We use INITIALIZING_PROPOSAL time limit because if we're past that, we can't build a block anyway
608
- if (maxAllowedTime === undefined || secondsIntoSlot <= maxAllowedTime) {
609
- this.log.trace(`Not attempting to vote since there is still time for block building`, {
610
- secondsIntoSlot,
611
- maxAllowedTime,
612
- });
613
- return;
614
- }
1026
+ // Vote-only actions do not give up the slot: if sync recovers, a later work-loop iteration can still build.
1027
+ // Under proposer pipelining the work loop reasons about the next L1 slot, so waiting for the target slot's
1028
+ // build-start deadline can miss the whole fallback window when the target advances with the clock.
1029
+ const nowSeconds = this.dateProvider.now() / 1000;
1030
+ const startDeadline = this.timetable.getBuildStartDeadline(targetSlot);
615
1031
 
616
- this.log.trace(`Sync for slot ${slot} failed, checking for voting opportunities`, {
617
- secondsIntoSlot,
618
- maxAllowedTime,
1032
+ this.log.trace(`Cannot build for slot ${slot}, checking for voting opportunities`, {
1033
+ nowSeconds,
1034
+ startDeadline,
619
1035
  });
620
1036
 
621
1037
  // Check if we're a proposer or proposal is open
622
- const [canPropose, proposer] = await this.checkCanPropose(slot);
1038
+ const [canPropose, proposer] = await this.checkCanPropose(targetSlot);
623
1039
  if (!canPropose) {
624
1040
  this.log.trace(`Cannot vote in slot ${slot} since we are not a proposer`, { slot, proposer });
625
1041
  return;
626
1042
  }
627
1043
 
628
1044
  // Mark this slot as attempted
629
- this.lastSlotForFallbackVote = slot;
1045
+ this.lastSlotForFallbackAction = slot;
630
1046
 
631
1047
  // Get a publisher for voting
632
1048
  const { attestorAddress, publisher } = await this.publisherFactory.create(proposer);
@@ -636,9 +1052,9 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
636
1052
  slot,
637
1053
  });
638
1054
 
639
- // Enqueue governance and slashing votes
1055
+ // Enqueue governance and slashing votes (voter uses the target slot for L1 submission)
640
1056
  const voter = new CheckpointVoter(
641
- slot,
1057
+ targetSlot,
642
1058
  publisher,
643
1059
  attestorAddress,
644
1060
  this.validatorClient,
@@ -651,13 +1067,41 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
651
1067
  const votesPromises = voter.enqueueVotes();
652
1068
  const votes = await Promise.all(votesPromises);
653
1069
 
654
- if (votes.every(p => !p)) {
655
- this.log.debug(`No votes to enqueue for slot ${slot}`);
1070
+ // Even if we cannot build, try to prune so a stuck pending chain (e.g. bad data blocking sync) can
1071
+ // recover. prune() is permissionless, so it rides the same fallback multicall as the votes.
1072
+ const pruneEnqueued = await this.tryEnqueuePruneIfPrunable(targetSlot, publisher);
1073
+
1074
+ // Bail if nothing to do
1075
+ if (votes.every(p => !p) && !pruneEnqueued) {
1076
+ this.log.debug(`Nothing to enqueue for slot ${slot} (no votes, not prunable)`);
656
1077
  return;
657
1078
  }
658
1079
 
659
- this.log.info(`Voting in slot ${slot} despite sync failure`, { slot });
660
- await publisher.sendRequests();
1080
+ const [governanceVoteEnqueued, slashingVoteEnqueued] = votes;
1081
+ this.log.info(`Submitting fallback requests in slot ${slot} despite sync failure`, {
1082
+ slot,
1083
+ pruneEnqueued,
1084
+ governanceVoteEnqueued: !!governanceVoteEnqueued,
1085
+ slashingVoteEnqueued: !!slashingVoteEnqueued,
1086
+ });
1087
+
1088
+ // Votes are EIP-712-signed for `targetSlot` (the pipelined slot in which the multicall is
1089
+ // expected to mine). Delay submission to the start of `targetSlot` so the tx mines in the
1090
+ // slot the votes were signed for. We fire-and-forget so we don't block the sequencer's
1091
+ // work loop while waiting for the target slot to start, but track it so stop() can drain it.
1092
+ const send = publisher.sendRequestsAt(targetSlot).catch(err => {
1093
+ this.log.error(`Failed to publish fallback requests despite sync failure for slot ${slot}`, err, { slot });
1094
+ });
1095
+ this.pendingRequests.trackRequest(send, () => publisher.interrupt());
1096
+ }
1097
+
1098
+ private async tryEnqueuePruneIfPrunable(targetSlot: SlotNumber, publisher: SequencerPublisher): Promise<boolean> {
1099
+ try {
1100
+ return await publisher.enqueuePruneIfPrunable(targetSlot);
1101
+ } catch (err) {
1102
+ this.log.error(`Failed to enqueue rollup prune for slot ${targetSlot}`, err, { targetSlot });
1103
+ return false;
1104
+ }
661
1105
  }
662
1106
 
663
1107
  /**
@@ -667,25 +1111,34 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
667
1111
  @trackSpan('Sequencer.tryVoteWhenEscapeHatchOpen', ({ slot }) => ({ [Attributes.SLOT_NUMBER]: slot }))
668
1112
  protected async tryVoteWhenEscapeHatchOpen(args: {
669
1113
  slot: SlotNumber;
1114
+ targetSlot: SlotNumber;
670
1115
  proposer: EthAddress | undefined;
671
1116
  }): Promise<void> {
672
- const { slot, proposer } = args;
1117
+ const { slot, targetSlot, proposer } = args;
673
1118
 
674
1119
  // Prevent duplicate attempts in the same slot
675
- if (this.lastSlotForFallbackVote === slot) {
1120
+ if (this.lastSlotForFallbackAction === slot) {
676
1121
  this.log.trace(`Already attempted to vote in slot ${slot} (escape hatch open, skipping)`);
677
1122
  return;
678
1123
  }
679
1124
 
680
1125
  // Mark this slot as attempted
681
- this.lastSlotForFallbackVote = slot;
1126
+ this.lastSlotForFallbackAction = slot;
682
1127
 
683
1128
  const { attestorAddress, publisher } = await this.publisherFactory.create(proposer);
684
1129
 
685
- this.log.debug(`Escape hatch open for slot ${slot}, attempting vote-only actions`, { slot, attestorAddress });
1130
+ this.log.debug(`Escape hatch open for slot ${slot}, attempting vote-only actions`, {
1131
+ slot,
1132
+ targetSlot,
1133
+ attestorAddress,
1134
+ });
686
1135
 
1136
+ // Under proposer pipelining, the multicall is expected to mine in `targetSlot` (slot + 1).
1137
+ // Governance and slashing votes are EIP-712-signed against the slot they will mine in, and the
1138
+ // L1 contract checks `msg.sender == getCurrentProposer()` using the mining slot. So we must
1139
+ // sign for `targetSlot` and delay submission to the start of `targetSlot`.
687
1140
  const voter = new CheckpointVoter(
688
- slot,
1141
+ targetSlot,
689
1142
  publisher,
690
1143
  attestorAddress,
691
1144
  this.validatorClient,
@@ -704,8 +1157,17 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
704
1157
  return;
705
1158
  }
706
1159
 
707
- this.log.info(`Voting in slot ${slot} (escape hatch open)`, { slot });
708
- await publisher.sendRequests();
1160
+ this.log.info(`Voting in slot ${slot} (escape hatch open)`, { slot, targetSlot });
1161
+ // Votes are EIP-712-signed for `targetSlot`. Delay submission to the start of `targetSlot` so
1162
+ // the multicall mines in the slot the votes were signed for; otherwise the L1 contract reads
1163
+ // `signaler = getCurrentProposer()` against the wrong slot and signature verification fails
1164
+ // silently inside Multicall3. Fire-and-forget so we don't block the sequencer's work loop while
1165
+ // waiting for the target slot to start, mirroring tryVoteAndPruneWhenCannotBuild, but tracked so
1166
+ // stop() can drain it.
1167
+ const send = publisher.sendRequestsAt(targetSlot).catch(err => {
1168
+ this.log.error(`Failed to publish escape-hatch votes for slot ${slot}`, err, { slot, targetSlot });
1169
+ });
1170
+ this.pendingRequests.trackRequest(send, () => publisher.interrupt());
709
1171
  }
710
1172
 
711
1173
  /**
@@ -713,17 +1175,34 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
713
1175
  * has been there without being invalidated and whether the sequencer is in the committee or not. We always
714
1176
  * have the proposer try to invalidate, but if they fail, the sequencers in the committee are expected to try,
715
1177
  * and if they fail, any sequencer will try as well.
1178
+ * @param pendingChainValidationStatus - The archiver's pending-chain validation status, authoritative on its own.
1179
+ * @param currentSlot - The wall-clock slot, used for committee lookup, the per-(checkpoint, slot) dedup guard, and logging.
716
1180
  */
717
1181
  protected async considerInvalidatingCheckpoint(
718
- syncedTo: SequencerSyncCheckResult,
1182
+ pendingChainValidationStatus: ValidateCheckpointResult,
719
1183
  currentSlot: SlotNumber,
720
1184
  ): Promise<void> {
721
- const { pendingChainValidationStatus, syncedL2Slot } = syncedTo;
722
1185
  if (pendingChainValidationStatus.valid) {
723
1186
  return;
724
1187
  }
725
1188
 
726
1189
  const invalidCheckpointNumber = pendingChainValidationStatus.checkpoint.checkpointNumber;
1190
+
1191
+ // Avoid re-running the committee lookup, simulation, and submission on every tick within a slot.
1192
+ // The guard is keyed by (checkpoint, slot) — so a different invalid checkpoint surfacing later in
1193
+ // the same slot is not suppressed — and is set only after a request is successfully simulated below,
1194
+ // so a transient simulation failure (or thresholds not yet met) still retries on the next tick.
1195
+ if (
1196
+ this.lastInvalidationAttempt?.slot === currentSlot &&
1197
+ this.lastInvalidationAttempt.checkpointNumber === invalidCheckpointNumber
1198
+ ) {
1199
+ this.log.trace(`Already attempted to invalidate checkpoint ${invalidCheckpointNumber} in slot ${currentSlot}`, {
1200
+ currentSlot,
1201
+ invalidCheckpointNumber,
1202
+ });
1203
+ return;
1204
+ }
1205
+
727
1206
  const invalidCheckpointTimestamp = pendingChainValidationStatus.checkpoint.timestamp;
728
1207
  const timeSinceChainInvalid = this.dateProvider.nowInSeconds() - Number(invalidCheckpointTimestamp);
729
1208
  const ourValidatorAddresses = this.validatorClient.getValidatorAddresses();
@@ -733,7 +1212,6 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
733
1212
 
734
1213
  const logData = {
735
1214
  invalidL1Timestamp: invalidCheckpointTimestamp,
736
- syncedL2Slot,
737
1215
  invalidCheckpoint: pendingChainValidationStatus.checkpoint,
738
1216
  secondsBeforeInvalidatingBlockAsCommitteeMember,
739
1217
  secondsBeforeInvalidatingBlockAsNonCommitteeMember,
@@ -786,6 +1264,10 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
786
1264
  return;
787
1265
  }
788
1266
 
1267
+ // We produced a valid invalidation request; record it so further ticks within this slot skip the
1268
+ // committee lookup, simulation, and submission above for this same invalid checkpoint.
1269
+ this.lastInvalidationAttempt = { slot: currentSlot, checkpointNumber: invalidCheckpointNumber };
1270
+
789
1271
  this.log.info(
790
1272
  invalidateAsCommitteeMember
791
1273
  ? `Invalidating checkpoint ${invalidCheckpointNumber} as committee member`
@@ -832,13 +1314,13 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
832
1314
  });
833
1315
  }
834
1316
 
835
- private getSlotStartBuildTimestamp(slotNumber: SlotNumber): number {
836
- return getSlotStartBuildTimestamp(slotNumber, this.l1Constants);
837
- }
838
-
839
- private getSecondsIntoSlot(slotNumber: SlotNumber): number {
840
- const slotStartTimestamp = this.getSlotStartBuildTimestamp(slotNumber);
841
- return Number((this.dateProvider.now() / 1000 - slotStartTimestamp).toFixed(3));
1317
+ /**
1318
+ * Wall-clock seconds elapsed since the build-frame start of the given target slot
1319
+ * (`now − getBuildFrameStart(targetSlot)`). May be negative if called before the build frame opens.
1320
+ */
1321
+ private getSecondsIntoBuildFrame(targetSlot: SlotNumber): number {
1322
+ const buildFrameStart = this.timetable.getBuildFrameStart(targetSlot);
1323
+ return Number((this.dateProvider.now() / 1000 - buildFrameStart).toFixed(3));
842
1324
  }
843
1325
 
844
1326
  public get aztecSlotDuration() {
@@ -869,17 +1351,16 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
869
1351
  public getConfig() {
870
1352
  return this.config;
871
1353
  }
872
-
873
- private get l1PublishingTime(): number {
874
- return this.config.l1PublishingTime ?? this.l1Constants.ethereumSlotDuration;
875
- }
876
1354
  }
877
1355
 
878
1356
  type SequencerSyncCheckResult = {
879
1357
  blockData?: BlockData;
880
1358
  checkpointNumber: CheckpointNumber;
1359
+ checkpointedCheckpointNumber: CheckpointNumber;
881
1360
  blockNumber: BlockNumber;
882
1361
  archive: Fr;
1362
+ hasProposedCheckpoint: boolean;
1363
+ proposedCheckpointData?: ProposedCheckpointData;
883
1364
  syncedL2Slot: SlotNumber;
884
1365
  pendingChainValidationStatus: ValidateCheckpointResult;
885
1366
  };