@aztec/sequencer-client 0.0.1-commit.4d3c002 → 0.0.1-commit.4d9804df

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (114) hide show
  1. package/README.md +281 -21
  2. package/dest/client/sequencer-client.d.ts +20 -5
  3. package/dest/client/sequencer-client.d.ts.map +1 -1
  4. package/dest/client/sequencer-client.js +26 -22
  5. package/dest/config.d.ts +11 -4
  6. package/dest/config.d.ts.map +1 -1
  7. package/dest/config.js +55 -26
  8. package/dest/global_variable_builder/fee_predictor.d.ts +37 -0
  9. package/dest/global_variable_builder/fee_predictor.d.ts.map +1 -0
  10. package/dest/global_variable_builder/fee_predictor.js +138 -0
  11. package/dest/global_variable_builder/fee_provider.d.ts +21 -0
  12. package/dest/global_variable_builder/fee_provider.d.ts.map +1 -0
  13. package/dest/global_variable_builder/fee_provider.js +80 -0
  14. package/dest/global_variable_builder/global_builder.d.ts +7 -26
  15. package/dest/global_variable_builder/global_builder.d.ts.map +1 -1
  16. package/dest/global_variable_builder/global_builder.js +6 -67
  17. package/dest/global_variable_builder/index.d.ts +3 -1
  18. package/dest/global_variable_builder/index.d.ts.map +1 -1
  19. package/dest/global_variable_builder/index.js +2 -0
  20. package/dest/publisher/config.d.ts +7 -3
  21. package/dest/publisher/config.d.ts.map +1 -1
  22. package/dest/publisher/config.js +13 -3
  23. package/dest/publisher/l1_to_l2_messaging.d.ts +21 -0
  24. package/dest/publisher/l1_to_l2_messaging.d.ts.map +1 -0
  25. package/dest/publisher/l1_to_l2_messaging.js +70 -0
  26. package/dest/publisher/l1_tx_failed_store/failed_tx_store.d.ts +50 -11
  27. package/dest/publisher/l1_tx_failed_store/failed_tx_store.d.ts.map +1 -1
  28. package/dest/publisher/l1_tx_failed_store/failed_tx_store.js +68 -1
  29. package/dest/publisher/l1_tx_failed_store/file_store_failed_tx_store.d.ts +2 -2
  30. package/dest/publisher/l1_tx_failed_store/file_store_failed_tx_store.d.ts.map +1 -1
  31. package/dest/publisher/l1_tx_failed_store/file_store_failed_tx_store.js +4 -2
  32. package/dest/publisher/l1_tx_failed_store/index.d.ts +2 -2
  33. package/dest/publisher/l1_tx_failed_store/index.d.ts.map +1 -1
  34. package/dest/publisher/l1_tx_failed_store/index.js +1 -0
  35. package/dest/publisher/sequencer-bundle-simulator.d.ts +96 -0
  36. package/dest/publisher/sequencer-bundle-simulator.d.ts.map +1 -0
  37. package/dest/publisher/sequencer-bundle-simulator.js +198 -0
  38. package/dest/publisher/sequencer-publisher-factory.d.ts +1 -3
  39. package/dest/publisher/sequencer-publisher-factory.d.ts.map +1 -1
  40. package/dest/publisher/sequencer-publisher-factory.js +0 -1
  41. package/dest/publisher/sequencer-publisher.d.ts +77 -68
  42. package/dest/publisher/sequencer-publisher.d.ts.map +1 -1
  43. package/dest/publisher/sequencer-publisher.js +593 -553
  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 +67 -36
  57. package/dest/sequencer/checkpoint_proposal_job.d.ts.map +1 -1
  58. package/dest/sequencer/checkpoint_proposal_job.js +686 -253
  59. package/dest/sequencer/checkpoint_proposal_job_metrics.d.ts +34 -0
  60. package/dest/sequencer/checkpoint_proposal_job_metrics.d.ts.map +1 -0
  61. package/dest/sequencer/checkpoint_proposal_job_metrics.js +72 -0
  62. package/dest/sequencer/events.d.ts +61 -5
  63. package/dest/sequencer/events.d.ts.map +1 -1
  64. package/dest/sequencer/metrics.d.ts +9 -10
  65. package/dest/sequencer/metrics.d.ts.map +1 -1
  66. package/dest/sequencer/metrics.js +34 -20
  67. package/dest/sequencer/requests_tracker.d.ts +22 -0
  68. package/dest/sequencer/requests_tracker.d.ts.map +1 -0
  69. package/dest/sequencer/requests_tracker.js +33 -0
  70. package/dest/sequencer/sequencer.d.ts +145 -33
  71. package/dest/sequencer/sequencer.d.ts.map +1 -1
  72. package/dest/sequencer/sequencer.js +520 -180
  73. package/dest/sequencer/types.d.ts +2 -2
  74. package/dest/sequencer/types.d.ts.map +1 -1
  75. package/dest/test/index.d.ts +3 -3
  76. package/dest/test/index.d.ts.map +1 -1
  77. package/dest/test/utils.d.ts +15 -1
  78. package/dest/test/utils.d.ts.map +1 -1
  79. package/dest/test/utils.js +25 -7
  80. package/package.json +30 -28
  81. package/src/client/sequencer-client.ts +37 -27
  82. package/src/config.ts +64 -26
  83. package/src/global_variable_builder/README.md +44 -0
  84. package/src/global_variable_builder/fee_predictor.ts +182 -0
  85. package/src/global_variable_builder/fee_provider.ts +97 -0
  86. package/src/global_variable_builder/global_builder.ts +11 -91
  87. package/src/global_variable_builder/index.ts +2 -0
  88. package/src/publisher/config.ts +30 -7
  89. package/src/publisher/l1_to_l2_messaging.ts +85 -0
  90. package/src/publisher/l1_tx_failed_store/failed_tx_store.ts +117 -8
  91. package/src/publisher/l1_tx_failed_store/file_store_failed_tx_store.ts +4 -3
  92. package/src/publisher/l1_tx_failed_store/index.ts +1 -1
  93. package/src/publisher/sequencer-bundle-simulator.ts +254 -0
  94. package/src/publisher/sequencer-publisher-factory.ts +0 -3
  95. package/src/publisher/sequencer-publisher.ts +671 -641
  96. package/src/publisher/write_json.ts +78 -0
  97. package/src/sequencer/README.md +162 -450
  98. package/src/sequencer/automine/README.md +60 -0
  99. package/src/sequencer/automine/automine_factory.ts +152 -0
  100. package/src/sequencer/automine/automine_sequencer.ts +796 -0
  101. package/src/sequencer/automine/index.ts +6 -0
  102. package/src/sequencer/checkpoint_proposal_job.ts +808 -313
  103. package/src/sequencer/checkpoint_proposal_job_metrics.ts +128 -0
  104. package/src/sequencer/events.ts +66 -5
  105. package/src/sequencer/metrics.ts +43 -24
  106. package/src/sequencer/requests_tracker.ts +43 -0
  107. package/src/sequencer/sequencer.ts +586 -206
  108. package/src/sequencer/types.ts +1 -1
  109. package/src/test/index.ts +2 -2
  110. package/src/test/utils.ts +61 -10
  111. package/dest/sequencer/timetable.d.ts +0 -88
  112. package/dest/sequencer/timetable.d.ts.map +0 -1
  113. package/dest/sequencer/timetable.js +0 -222
  114. package/src/sequencer/timetable.ts +0 -283
@@ -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, ProposedCheckpointData } 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;
@@ -73,13 +109,25 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
73
109
  private lastEpochForStrategyComparison: EpochNumber | undefined;
74
110
 
75
111
  /** The last checkpoint proposal job, tracked so we can await its pending L1 submission during shutdown. */
76
- private lastCheckpointProposalJob: CheckpointProposalJob | undefined;
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();
77
124
 
78
- /** The maximum number of seconds that the sequencer can be into a slot to transition to a particular state. */
79
- protected timetable!: SequencerTimetable;
125
+ /** Proposer schedule and block sub-slot timetable for the sequencer, rebuilt on every config update. */
126
+ protected timetable!: ProposerTimetable;
80
127
 
81
128
  /** Config for the sequencer */
82
129
  protected config: ResolvedSequencerConfig = DefaultSequencerConfig;
130
+ private readonly signatureContext: CoordinationSignatureContext;
83
131
 
84
132
  constructor(
85
133
  protected publisherFactory: SequencerPublisherFactory,
@@ -88,45 +136,148 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
88
136
  protected p2pClient: P2P,
89
137
  protected worldState: WorldStateSynchronizer,
90
138
  protected slasherClient: SlasherClientInterface | undefined,
91
- protected l2BlockSource: L2BlockSource & L2BlockSink,
139
+ protected l2BlockSource: L2BlockSource & L2BlockSink & ProposedCheckpointSink,
92
140
  protected l1ToL2MessageSource: L1ToL2MessageSource,
93
141
  protected checkpointsBuilder: FullNodeCheckpointsBuilder,
94
142
  protected l1Constants: SequencerRollupConstants,
95
143
  protected dateProvider: DateProvider,
96
144
  protected epochCache: EpochCache,
97
145
  protected rollupContract: RollupContract,
98
- config: SequencerConfig,
146
+ config: SequencerConfig & Pick<ChainConfig, 'l1ChainId' | 'rollupAddress'>,
99
147
  protected telemetry: TelemetryClient = getTelemetryClient(),
100
148
  protected log = createLogger('sequencer'),
101
149
  ) {
102
150
  super();
151
+ this.stateLog = log.createChild('state');
152
+ this.stateEnteredAtMs = this.dateProvider.now();
103
153
 
104
154
  // Add [FISHERMAN] prefix to logger if in fisherman mode
105
155
  if (config.fishermanMode) {
106
156
  this.log = log.createChild('[FISHERMAN]');
107
157
  }
108
158
 
159
+ this.signatureContext = {
160
+ chainId: config.l1ChainId,
161
+ rollupAddress: config.rollupAddress,
162
+ };
109
163
  this.metrics = new SequencerMetrics(telemetry, this.rollupContract, 'Sequencer');
164
+ this.checkpointProposalJobMetrics = new CheckpointProposalJobMetrics(telemetry);
110
165
  this.updateConfig(config);
111
166
  }
112
167
 
113
- /** 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
+ */
114
176
  public updateConfig(config: Partial<SequencerConfig>) {
115
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;
116
190
  this.log.info(`Updated sequencer config`, omit(filteredConfig, 'txPublicSetupAllowListExtend'));
117
- this.config = merge(this.config, filteredConfig);
118
- this.timetable = new SequencerTimetable(
119
- {
120
- ethereumSlotDuration: this.l1Constants.ethereumSlotDuration,
121
- aztecSlotDuration: this.aztecSlotDuration,
122
- l1PublishingTime: this.l1PublishingTime,
123
- p2pPropagationTime: this.config.attestationPropagationTime,
124
- blockDurationMs: this.config.blockDurationMs,
125
- enforce: this.config.enforceTimeTable,
126
- },
127
- this.metrics,
128
- this.log,
129
- );
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
+ }
130
281
  }
131
282
 
132
283
  /** Initializes the sequencer (precomputes tables). Takes about 3s. */
@@ -134,8 +285,22 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
134
285
  getKzg();
135
286
  }
136
287
 
137
- /** 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
+ */
138
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
+ }
139
304
  this.runningPromise = new RunningPromise(
140
305
  this.safeWork.bind(this),
141
306
  this.log,
@@ -146,35 +311,68 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
146
311
  this.log.info('Started sequencer');
147
312
  }
148
313
 
149
- /** 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
+ */
150
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
+ }
151
331
  this.log.info(`Stopping sequencer`);
152
332
  this.setState(SequencerState.STOPPING, undefined, { force: true });
333
+ this.lastCheckpointProposalJob?.interrupt();
153
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.
154
339
  await this.runningPromise?.stop();
155
- await this.lastCheckpointProposalJob?.awaitPendingSubmission();
340
+ this.pendingRequests.interruptRequests();
341
+ await this.pendingRequests.awaitRequests();
156
342
  this.setState(SequencerState.STOPPED, undefined, { force: true });
157
343
  this.log.info('Stopped sequencer');
158
344
  }
159
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
+
160
369
  /** Main sequencer loop with a try/catch */
161
370
  protected async safeWork() {
162
371
  try {
163
372
  await this.work();
164
373
  } catch (err) {
165
374
  this.emit('checkpoint-error', { error: err as Error });
166
- if (err instanceof SequencerTooSlowError) {
167
- // Log as warn only if we had to abort halfway through the block proposal
168
- const logLvl = [SequencerState.INITIALIZING_CHECKPOINT, SequencerState.PROPOSER_CHECK].includes(
169
- err.proposedState,
170
- )
171
- ? ('debug' as const)
172
- : ('warn' as const);
173
- this.log[logLvl](err.message, { now: this.dateProvider.nowInSeconds() });
174
- } else {
175
- // Re-throw other errors
176
- throw err;
177
- }
375
+ throw err;
178
376
  } finally {
179
377
  this.setState(SequencerState.IDLE, undefined);
180
378
  }
@@ -196,8 +394,7 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
196
394
  @trackSpan('Sequencer.work')
197
395
  protected async work() {
198
396
  this.setState(SequencerState.SYNCHRONIZING, undefined);
199
- const { slot, ts, nowSeconds, epoch } = this.epochCache.getEpochAndSlotInNextL1Slot();
200
- const { slot: targetSlot, epoch: targetEpoch } = this.epochCache.getTargetEpochAndSlotInNextL1Slot();
397
+ const { slot, targetSlot, epoch, targetEpoch, ts, nowSeconds } = this.getSlotContextInNextL1Slot();
201
398
 
202
399
  // Check if we are synced and it's our slot, grab a publisher, check previous block invalidation, etc
203
400
  const checkpointProposalJob = await this.prepareCheckpointProposal(
@@ -235,6 +432,13 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
235
432
  return checkpoint;
236
433
  }
237
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
+
238
442
  /**
239
443
  * Prepares the checkpoint proposal by performing all necessary checks and setup.
240
444
  * This is the initial step in the main loop.
@@ -249,14 +453,8 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
249
453
  ts: bigint,
250
454
  nowSeconds: bigint,
251
455
  ): Promise<CheckpointProposalJob | undefined> {
252
- // Check we have not already processed this target slot (cheapest check)
253
- // We only check this if enforce timetable is set, since we want to keep processing the same slot if we are not
254
- // running against actual time (eg when we use sandbox-style automining)
255
- if (
256
- this.lastSlotForCheckpointProposalJob &&
257
- this.lastSlotForCheckpointProposalJob >= targetSlot &&
258
- this.config.enforceTimeTable
259
- ) {
456
+ // Check we have not already processed this target slot (cheapest check).
457
+ if (this.lastSlotForCheckpointProposalJob && this.lastSlotForCheckpointProposalJob >= targetSlot) {
260
458
  this.log.trace(`Target slot ${targetSlot} has already been processed`);
261
459
  return undefined;
262
460
  }
@@ -269,23 +467,29 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
269
467
  return undefined;
270
468
  }
271
469
 
272
- // Check all components are synced to latest as seen by the archiver (queries all subsystems)
273
- const syncedTo = await this.checkSync({ ts, slot });
274
- if (!syncedTo) {
275
- await this.tryVoteWhenSyncFails({ slot, targetSlot, 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
+ });
276
476
  return undefined;
277
477
  }
278
478
 
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
+
279
485
  // If escape hatch is open for the target epoch, do not start checkpoint proposal work and do not attempt invalidations.
280
486
  // Still perform governance/slashing voting (as proposer) once per slot.
281
487
  // When pipelining, we check the target epoch (slot+1's epoch) since that's the epoch we're building for.
282
488
  const isEscapeHatchOpen = await this.epochCache.isEscapeHatchOpen(targetEpoch);
283
489
 
284
490
  if (isEscapeHatchOpen) {
285
- this.setState(SequencerState.PROPOSER_CHECK, slot);
286
- const [canPropose, proposer] = await this.checkCanPropose(targetSlot);
287
491
  if (canPropose) {
288
- await this.tryVoteWhenEscapeHatchOpen({ slot, proposer });
492
+ await this.tryVoteWhenEscapeHatchOpen({ slot, targetSlot, proposer });
289
493
  } else {
290
494
  this.log.trace(`Escape hatch open but we are not proposer, skipping vote-only actions`, {
291
495
  slot,
@@ -296,19 +500,53 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
296
500
  return undefined;
297
501
  }
298
502
 
299
- // Next checkpoint follows from the last synced one
300
- const checkpointNumber = CheckpointNumber(syncedTo.checkpointNumber + 1);
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
+ }
301
518
 
302
- // Guard: don't exceed 1-deep pipeline. Without a proposed checkpoint, we can only build
303
- // confirmed + 1. With a proposed checkpoint, we can build confirmed + 2.
304
- const confirmedCkpt = syncedTo.checkpointedCheckpointNumber;
305
- if (checkpointNumber > confirmedCkpt + 2) {
306
- this.log.warn(
307
- `Skipping slot ${targetSlot}: checkpoint ${checkpointNumber} exceeds max pipeline depth (confirmed=${confirmedCkpt})`,
308
- );
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;
309
537
  return undefined;
310
538
  }
311
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
+
547
+ // Next checkpoint follows from the last synced one
548
+ const checkpointNumber = CheckpointNumber(syncedTo.checkpointNumber + 1);
549
+
312
550
  const logCtx = {
313
551
  nowSeconds,
314
552
  syncedToL2Slot: syncedTo.syncedL2Slot,
@@ -319,13 +557,16 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
319
557
  isPendingChainValid: pick(syncedTo.pendingChainValidationStatus, 'valid', 'reason', 'invalidIndex'),
320
558
  };
321
559
 
322
- // Check that we are a proposer for the target slot.
323
- this.setState(SequencerState.PROPOSER_CHECK, slot);
324
- const [canPropose, proposer] = await this.checkCanPropose(targetSlot);
560
+ // We are the synced proposer within the build window; enter the proposer-check state and build.
561
+ this.setState(SequencerState.PROPOSER_CHECK, targetSlot);
325
562
 
326
- // If we are not a proposer check if we should invalidate an invalid checkpoint, and bail
327
- if (!canPropose) {
328
- 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
+ );
329
570
  return undefined;
330
571
  }
331
572
 
@@ -347,53 +588,96 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
347
588
  const { attestorAddress, publisher } = await this.publisherFactory.create(proposerForPublisher);
348
589
  this.log.verbose(`Created publisher at address ${publisher.getSenderAddress()} for attestor ${attestorAddress}`);
349
590
 
350
- // In fisherman mode, set the actual proposer's address for simulations
351
- if (this.config.fishermanMode && proposer) {
352
- publisher.setProposerAddressForSimulation(proposer);
353
- this.log.debug(`Set proposer address ${proposer} for simulation in fisherman mode`);
354
- }
355
-
356
- // Prepare invalidation request if the pending chain is invalid (returns undefined if no need)
357
- let invalidateCheckpoint = await publisher.simulateInvalidateCheckpoint(syncedTo.pendingChainValidationStatus);
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);
358
598
 
359
599
  // Determine the correct archive and L1 state overrides for the canProposeAt check.
360
600
  // The L1 contract reads archives[proposedCheckpointNumber] and compares it with the provided archive.
361
601
  // When invalidating or pipelining, the local archive may differ from L1's, so we adjust accordingly.
362
602
  let archiveForCheck = syncedTo.archive;
363
- const l1Overrides: {
364
- forcePendingCheckpointNumber?: CheckpointNumber;
365
- forceArchive?: { checkpointNumber: CheckpointNumber; archive: Fr };
366
- } = {};
367
-
368
- if (this.epochCache.isProposerPipeliningEnabled() && syncedTo.hasProposedCheckpoint) {
369
- // Parent checkpoint hasn't landed on L1 yet. Override both the proposed checkpoint number
370
- // and the archive at that checkpoint so L1 simulation sees the correct chain tip.
371
- const parentCheckpointNumber = CheckpointNumber(checkpointNumber - 1);
372
- l1Overrides.forcePendingCheckpointNumber = parentCheckpointNumber;
373
- l1Overrides.forceArchive = { checkpointNumber: parentCheckpointNumber, archive: syncedTo.archive };
374
- this.metrics.recordPipelineDepth(1);
375
603
 
604
+ if (syncedTo.hasProposedCheckpoint) {
605
+ this.metrics.recordPipelineDepth(syncedTo.checkpointNumber - syncedTo.checkpointedCheckpointNumber);
376
606
  this.log.verbose(
377
- `Building on top of proposed checkpoint (pending=${syncedTo.proposedCheckpointData?.checkpointNumber})`,
607
+ `Building on top of proposed checkpoint (pending=${syncedTo.proposedCheckpointData?.checkpointNumber}) for target slot ${targetSlot}`,
608
+ { targetSlot, parentCheckpointNumber: CheckpointNumber(checkpointNumber - 1) },
378
609
  );
379
- // Clear the invalidation - the proposed checkpoint should handle it.
380
- invalidateCheckpoint = undefined;
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;
381
616
  } else if (invalidateCheckpoint) {
382
617
  // After invalidation, L1 will roll back to checkpoint N-1. The archive at N-1 already
383
618
  // exists on L1, so we just pass the matching archive (the lastArchive of the invalid checkpoint).
384
619
  archiveForCheck = invalidateCheckpoint.lastArchive;
385
- l1Overrides.forcePendingCheckpointNumber = invalidateCheckpoint.forcePendingCheckpointNumber;
386
620
  this.metrics.recordPipelineDepth(0);
387
621
  } else {
388
622
  this.metrics.recordPipelineDepth(0);
389
623
  }
390
624
 
391
- const canProposeCheck = await publisher.canProposeAt(archiveForCheck, proposer ?? EthAddress.ZERO, l1Overrides);
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
+ }
650
+
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,
661
+ proposer ?? EthAddress.ZERO,
662
+ simulationOverridesPlan,
663
+ );
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
+ };
392
676
 
393
677
  if (canProposeCheck === undefined) {
394
678
  this.log.warn(
395
679
  `Cannot propose checkpoint ${checkpointNumber} at slot ${slot} due to failed rollup contract check`,
396
- logCtx,
680
+ { ...logCtx, ...proposeContext },
397
681
  );
398
682
  this.emit('proposer-rollup-check-failed', { reason: 'Rollup contract check failed', slot });
399
683
  this.metrics.recordCheckpointPrecheckFailed('rollup_contract_check_failed');
@@ -403,7 +687,7 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
403
687
  if (canProposeCheck.slot !== targetSlot) {
404
688
  this.log.warn(
405
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}.`,
406
- { ...logCtx, rollup: canProposeCheck, expectedSlot: targetSlot },
690
+ { ...logCtx, ...proposeContext, rollup: canProposeCheck, expectedSlot: targetSlot },
407
691
  );
408
692
  this.emit('proposer-rollup-check-failed', { reason: 'Slot mismatch', slot });
409
693
  this.metrics.recordCheckpointPrecheckFailed('slot_mismatch');
@@ -413,7 +697,7 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
413
697
  if (canProposeCheck.checkpointNumber !== checkpointNumber) {
414
698
  this.log.warn(
415
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}.`,
416
- { ...logCtx, rollup: canProposeCheck, expectedSlot: slot },
700
+ { ...logCtx, ...proposeContext, rollup: canProposeCheck, expectedSlot: slot },
417
701
  );
418
702
  this.emit('proposer-rollup-check-failed', { reason: 'Block mismatch', slot });
419
703
  this.metrics.recordCheckpointPrecheckFailed('block_number_mismatch');
@@ -427,18 +711,18 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
427
711
  `Preparing checkpoint proposal ${checkpointNumber} for target slot ${targetSlot} during wall-clock slot ${slot}`,
428
712
  {
429
713
  ...logCtx,
714
+ ...proposeContext,
430
715
  proposer,
431
- pipeliningEnabled: this.epochCache.isProposerPipeliningEnabled(),
432
716
  },
433
717
  );
434
718
 
435
719
  // Create and return the checkpoint proposal job
436
720
  return this.createCheckpointProposalJob(
437
- slot,
438
721
  targetSlot,
439
722
  targetEpoch,
440
723
  checkpointNumber,
441
724
  syncedTo.blockNumber,
725
+ syncedTo.checkpointedCheckpointNumber,
442
726
  proposer,
443
727
  publisher,
444
728
  attestorAddress,
@@ -448,11 +732,11 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
448
732
  }
449
733
 
450
734
  protected createCheckpointProposalJob(
451
- slot: SlotNumber,
452
735
  targetSlot: SlotNumber,
453
736
  targetEpoch: EpochNumber,
454
737
  checkpointNumber: CheckpointNumber,
455
738
  syncedToBlockNumber: BlockNumber,
739
+ checkpointedCheckpointNumber: CheckpointNumber,
456
740
  proposer: EthAddress | undefined,
457
741
  publisher: SequencerPublisher,
458
742
  attestorAddress: EthAddress,
@@ -460,11 +744,11 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
460
744
  proposedCheckpointData?: ProposedCheckpointData,
461
745
  ): CheckpointProposalJob {
462
746
  return new CheckpointProposalJob(
463
- slot,
464
747
  targetSlot,
465
748
  targetEpoch,
466
749
  checkpointNumber,
467
750
  syncedToBlockNumber,
751
+ checkpointedCheckpointNumber,
468
752
  proposer,
469
753
  publisher,
470
754
  attestorAddress,
@@ -478,13 +762,16 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
478
762
  this.checkpointsBuilder,
479
763
  this.l2BlockSource,
480
764
  this.l1Constants,
765
+ this.signatureContext,
481
766
  this.config,
482
767
  this.timetable,
483
768
  this.slasherClient,
484
769
  this.epochCache,
485
770
  this.dateProvider,
486
771
  this.metrics,
772
+ this.checkpointProposalJobMetrics.createRecorder(),
487
773
  this,
774
+ this.pendingRequests,
488
775
  this.setState.bind(this),
489
776
  this.tracer,
490
777
  this.log.getBindings(),
@@ -500,9 +787,11 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
500
787
  }
501
788
 
502
789
  /**
503
- * 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.
504
792
  * @param proposedState - The new state to transition to.
505
- * @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.
506
795
  * @param force - Whether to force the transition even if the sequencer is stopped.
507
796
  */
508
797
  protected setState(
@@ -518,25 +807,39 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
518
807
  this.log.warn(`Cannot set sequencer from ${this.state} to ${proposedState} as it is stopped.`);
519
808
  return;
520
809
  }
521
- let secondsIntoSlot = undefined;
522
- if (slotNumber !== undefined) {
523
- secondsIntoSlot = this.getSecondsIntoSlot(slotNumber);
524
- this.timetable.assertTimeLeft(proposedState, secondsIntoSlot);
525
- }
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;
526
819
 
527
820
  const boringStates = [SequencerState.IDLE, SequencerState.SYNCHRONIZING];
528
821
  const logLevel =
529
- boringStates.includes(proposedState) && boringStates.includes(this.state)
530
- ? ('trace' as const)
531
- : ('debug' as const);
532
- 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
+ });
533
831
 
534
832
  this.emit('state-changed', {
535
- oldState: this.state,
833
+ oldState,
536
834
  newState: proposedState,
537
- secondsIntoSlot,
538
- slot: slotNumber,
835
+ secondsIntoBuildFrame,
836
+ targetSlot: slotNumber,
539
837
  });
838
+ if (stateChanged) {
839
+ this.metrics.recordStateDuration(stateDurationMs, oldState);
840
+ this.stateEnteredAtMs = transitionAtMs;
841
+ this.stateSlotNumber = slotNumber;
842
+ }
540
843
  this.state = proposedState;
541
844
  }
542
845
 
@@ -563,63 +866,85 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
563
866
  number: syncSummary.latestBlockNumber,
564
867
  hash: syncSummary.latestBlockHash,
565
868
  })),
566
- this.l2BlockSource
567
- .getL2Tips()
568
- .then(t => ({ proposed: t.proposed, checkpointed: t.checkpointed, proposedCheckpoint: t.proposedCheckpoint })),
869
+ this.l2BlockSource.getL2Tips().then(t => ({ proposed: t.proposed, checkpointed: t.checkpointed })),
569
870
  this.p2pClient.getStatus().then(p2p => p2p.syncedToL2Block),
570
- this.l1ToL2MessageSource.getL2Tips().then(t => t.proposed),
871
+ this.l1ToL2MessageSource.getL2Tips().then(t => ({ proposed: t.proposed, checkpointed: t.checkpointed })),
571
872
  this.l2BlockSource.getPendingChainValidationStatus(),
572
- this.l2BlockSource.getProposedCheckpointOnly(),
873
+ this.l2BlockSource.getProposedCheckpointData(),
573
874
  ] as const);
574
875
 
575
- const [worldState, l2Tips, p2p, l1ToL2MessageSource, pendingChainValidationStatus, proposedCheckpointData] =
876
+ const [worldState, l2Tips, p2p, l1ToL2MessageSourceTips, pendingChainValidationStatus, proposedCheckpointData] =
576
877
  syncedBlocks;
577
878
 
578
- // 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,
579
- // as the world state can compute the new genesis block hash, but other components use the hardcoded constant.
580
- // TODO(palla/mbps): Fix the above. All components should be able to handle dynamic genesis block hashes.
581
879
  const result =
582
- (l2Tips.proposed.number === 0 &&
583
- worldState.number === 0 &&
584
- p2p.number === 0 &&
585
- l1ToL2MessageSource.number === 0) ||
586
- (worldState.hash === l2Tips.proposed.hash &&
587
- p2p.hash === l2Tips.proposed.hash &&
588
- l1ToL2MessageSource.hash === l2Tips.proposed.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;
589
885
 
590
886
  if (!result) {
591
887
  this.log.debug(`Sequencer sync check failed`, {
592
888
  worldState,
593
889
  l2BlockSource: l2Tips.proposed,
594
890
  p2p,
595
- l1ToL2MessageSource,
891
+ l1ToL2MessageSourceTips,
596
892
  });
597
893
  return undefined;
598
894
  }
599
895
 
600
- // Special case for genesis state
601
896
  const blockNumber = worldState.number;
602
- if (blockNumber < INITIAL_L2_BLOCK_NUM) {
603
- const archive = new Fr((await this.worldState.getCommitted().getTreeInfo(MerkleTreeId.ARCHIVE)).root);
604
- return {
605
- checkpointNumber: CheckpointNumber.ZERO,
606
- checkpointedCheckpointNumber: CheckpointNumber.ZERO,
607
- blockNumber: BlockNumber.ZERO,
608
- archive,
609
- hasProposedCheckpoint: false,
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,
610
902
  syncedL2Slot,
611
- pendingChainValidationStatus,
612
- };
903
+ ...args,
904
+ });
905
+ return undefined;
613
906
  }
614
907
 
615
- const blockData = await this.l2BlockSource.getBlockData(blockNumber);
616
- if (!blockData) {
617
- // this shouldn't really happen because a moment ago we checked that all components were in sync
618
- this.log.error(`Failed to get L2 block data ${blockNumber} from the archiver with all components in sync`);
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,
929
+ };
930
+
931
+ this.log.debug(`Waiting for proposed checkpoint to catch up with reexecuted block`, logCtx);
619
932
  return undefined;
620
933
  }
621
934
 
622
- const hasProposedCheckpoint = l2Tips.proposedCheckpoint.checkpoint.number > l2Tips.checkpointed.checkpoint.number;
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
+ });
946
+ return undefined;
947
+ }
623
948
 
624
949
  return {
625
950
  blockData,
@@ -684,36 +1009,29 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
684
1009
  }
685
1010
 
686
1011
  /**
687
- * Tries to vote on slashing actions and governance when the sync check fails but we're past the max time for initializing a proposal.
688
- * 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.
689
1015
  */
690
- @trackSpan('Seqeuencer.tryVoteWhenSyncFails', ({ slot }) => ({ [Attributes.SLOT_NUMBER]: slot }))
691
- protected async tryVoteWhenSyncFails(args: { slot: SlotNumber; targetSlot: SlotNumber; ts: bigint }): Promise<void> {
1016
+ @trackSpan('Sequencer.tryVoteAndPruneWhenCannotBuild', ({ slot }) => ({ [Attributes.SLOT_NUMBER]: slot }))
1017
+ protected async tryVoteAndPruneWhenCannotBuild(args: { slot: SlotNumber; targetSlot: SlotNumber }): Promise<void> {
692
1018
  const { slot, targetSlot } = args;
693
1019
 
694
1020
  // Prevent duplicate attempts in the same slot
695
- if (this.lastSlotForFallbackVote === slot) {
696
- 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)`);
697
1023
  return;
698
1024
  }
699
1025
 
700
- // Check if we're past the max time for initializing a proposal
701
- const secondsIntoSlot = this.getSecondsIntoSlot(slot);
702
- const maxAllowedTime = this.timetable.getMaxAllowedTime(SequencerState.INITIALIZING_CHECKPOINT);
703
-
704
- // If we haven't exceeded the time limit for initializing a proposal, don't proceed with voting
705
- // We use INITIALIZING_PROPOSAL time limit because if we're past that, we can't build a block anyway
706
- if (maxAllowedTime === undefined || secondsIntoSlot <= maxAllowedTime) {
707
- this.log.trace(`Not attempting to vote since there is still time for block building`, {
708
- secondsIntoSlot,
709
- maxAllowedTime,
710
- });
711
- return;
712
- }
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);
713
1031
 
714
- this.log.trace(`Sync for slot ${slot} failed, checking for voting opportunities`, {
715
- secondsIntoSlot,
716
- maxAllowedTime,
1032
+ this.log.trace(`Cannot build for slot ${slot}, checking for voting opportunities`, {
1033
+ nowSeconds,
1034
+ startDeadline,
717
1035
  });
718
1036
 
719
1037
  // Check if we're a proposer or proposal is open
@@ -724,7 +1042,7 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
724
1042
  }
725
1043
 
726
1044
  // Mark this slot as attempted
727
- this.lastSlotForFallbackVote = slot;
1045
+ this.lastSlotForFallbackAction = slot;
728
1046
 
729
1047
  // Get a publisher for voting
730
1048
  const { attestorAddress, publisher } = await this.publisherFactory.create(proposer);
@@ -749,13 +1067,41 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
749
1067
  const votesPromises = voter.enqueueVotes();
750
1068
  const votes = await Promise.all(votesPromises);
751
1069
 
752
- if (votes.every(p => !p)) {
753
- 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)`);
754
1077
  return;
755
1078
  }
756
1079
 
757
- this.log.info(`Voting in slot ${slot} despite sync failure`, { slot });
758
- 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
+ }
759
1105
  }
760
1106
 
761
1107
  /**
@@ -765,25 +1111,34 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
765
1111
  @trackSpan('Sequencer.tryVoteWhenEscapeHatchOpen', ({ slot }) => ({ [Attributes.SLOT_NUMBER]: slot }))
766
1112
  protected async tryVoteWhenEscapeHatchOpen(args: {
767
1113
  slot: SlotNumber;
1114
+ targetSlot: SlotNumber;
768
1115
  proposer: EthAddress | undefined;
769
1116
  }): Promise<void> {
770
- const { slot, proposer } = args;
1117
+ const { slot, targetSlot, proposer } = args;
771
1118
 
772
1119
  // Prevent duplicate attempts in the same slot
773
- if (this.lastSlotForFallbackVote === slot) {
1120
+ if (this.lastSlotForFallbackAction === slot) {
774
1121
  this.log.trace(`Already attempted to vote in slot ${slot} (escape hatch open, skipping)`);
775
1122
  return;
776
1123
  }
777
1124
 
778
1125
  // Mark this slot as attempted
779
- this.lastSlotForFallbackVote = slot;
1126
+ this.lastSlotForFallbackAction = slot;
780
1127
 
781
1128
  const { attestorAddress, publisher } = await this.publisherFactory.create(proposer);
782
1129
 
783
- 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
+ });
784
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`.
785
1140
  const voter = new CheckpointVoter(
786
- slot,
1141
+ targetSlot,
787
1142
  publisher,
788
1143
  attestorAddress,
789
1144
  this.validatorClient,
@@ -802,8 +1157,17 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
802
1157
  return;
803
1158
  }
804
1159
 
805
- this.log.info(`Voting in slot ${slot} (escape hatch open)`, { slot });
806
- 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());
807
1171
  }
808
1172
 
809
1173
  /**
@@ -811,17 +1175,34 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
811
1175
  * has been there without being invalidated and whether the sequencer is in the committee or not. We always
812
1176
  * have the proposer try to invalidate, but if they fail, the sequencers in the committee are expected to try,
813
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.
814
1180
  */
815
1181
  protected async considerInvalidatingCheckpoint(
816
- syncedTo: SequencerSyncCheckResult,
1182
+ pendingChainValidationStatus: ValidateCheckpointResult,
817
1183
  currentSlot: SlotNumber,
818
1184
  ): Promise<void> {
819
- const { pendingChainValidationStatus, syncedL2Slot } = syncedTo;
820
1185
  if (pendingChainValidationStatus.valid) {
821
1186
  return;
822
1187
  }
823
1188
 
824
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
+
825
1206
  const invalidCheckpointTimestamp = pendingChainValidationStatus.checkpoint.timestamp;
826
1207
  const timeSinceChainInvalid = this.dateProvider.nowInSeconds() - Number(invalidCheckpointTimestamp);
827
1208
  const ourValidatorAddresses = this.validatorClient.getValidatorAddresses();
@@ -831,7 +1212,6 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
831
1212
 
832
1213
  const logData = {
833
1214
  invalidL1Timestamp: invalidCheckpointTimestamp,
834
- syncedL2Slot,
835
1215
  invalidCheckpoint: pendingChainValidationStatus.checkpoint,
836
1216
  secondsBeforeInvalidatingBlockAsCommitteeMember,
837
1217
  secondsBeforeInvalidatingBlockAsNonCommitteeMember,
@@ -884,6 +1264,10 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
884
1264
  return;
885
1265
  }
886
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
+
887
1271
  this.log.info(
888
1272
  invalidateAsCommitteeMember
889
1273
  ? `Invalidating checkpoint ${invalidCheckpointNumber} as committee member`
@@ -930,13 +1314,13 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
930
1314
  });
931
1315
  }
932
1316
 
933
- private getSlotStartBuildTimestamp(slotNumber: SlotNumber): number {
934
- return getSlotStartBuildTimestamp(slotNumber, this.l1Constants);
935
- }
936
-
937
- private getSecondsIntoSlot(slotNumber: SlotNumber): number {
938
- const slotStartTimestamp = this.getSlotStartBuildTimestamp(slotNumber);
939
- 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));
940
1324
  }
941
1325
 
942
1326
  public get aztecSlotDuration() {
@@ -967,10 +1351,6 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
967
1351
  public getConfig() {
968
1352
  return this.config;
969
1353
  }
970
-
971
- private get l1PublishingTime(): number {
972
- return this.config.l1PublishingTime ?? this.l1Constants.ethereumSlotDuration;
973
- }
974
1354
  }
975
1355
 
976
1356
  type SequencerSyncCheckResult = {