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

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 +6 -25
  15. package/dest/global_variable_builder/global_builder.d.ts.map +1 -1
  16. package/dest/global_variable_builder/global_builder.js +7 -65
  17. package/dest/global_variable_builder/index.d.ts +3 -1
  18. package/dest/global_variable_builder/index.d.ts.map +1 -1
  19. package/dest/global_variable_builder/index.js +2 -0
  20. package/dest/publisher/config.d.ts +7 -3
  21. package/dest/publisher/config.d.ts.map +1 -1
  22. package/dest/publisher/config.js +13 -3
  23. package/dest/publisher/l1_to_l2_messaging.d.ts +21 -0
  24. package/dest/publisher/l1_to_l2_messaging.d.ts.map +1 -0
  25. package/dest/publisher/l1_to_l2_messaging.js +70 -0
  26. package/dest/publisher/l1_tx_failed_store/failed_tx_store.d.ts +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 +97 -68
  42. package/dest/publisher/sequencer-publisher.d.ts.map +1 -1
  43. package/dest/publisher/sequencer-publisher.js +603 -536
  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 +71 -21
  57. package/dest/sequencer/checkpoint_proposal_job.d.ts.map +1 -1
  58. package/dest/sequencer/checkpoint_proposal_job.js +724 -212
  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 +155 -33
  71. package/dest/sequencer/sequencer.d.ts.map +1 -1
  72. package/dest/sequencer/sequencer.js +543 -158
  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 +12 -80
  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 +702 -598
  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 +844 -241
  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 +625 -175
  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
@@ -3,66 +3,102 @@ import { Blob, getBlobsPerL1Block, getPrefixedEthBlobCommitments } from '@aztec/
3
3
  import type { EpochCache } from '@aztec/epoch-cache';
4
4
  import type { L1ContractsConfig } from '@aztec/ethereum/config';
5
5
  import {
6
- type EmpireSlashingProposerContract,
7
6
  FeeAssetPriceOracle,
8
7
  type GovernanceProposerContract,
9
- type IEmpireBase,
10
8
  MULTI_CALL_3_ADDRESS,
11
9
  Multicall3,
12
- RollupContract,
13
- type TallySlashingProposerContract,
14
- type ViemCommitteeAttestations,
15
- type ViemHeader,
10
+ MulticallForwarderRevertedError,
11
+ type RollupContract,
12
+ type SimulationOverridesPlan,
13
+ type SlashingProposerContract,
14
+ buildSimulationOverridesStateOverride,
16
15
  } from '@aztec/ethereum/contracts';
17
- import { type L1FeeAnalysisResult, L1FeeAnalyzer } from '@aztec/ethereum/l1-fee-analysis';
16
+ import { type L1FeeAnalysisResult, L1FeeAnalyzer, captureWindowBlockFees } from '@aztec/ethereum/l1-fee-analysis';
18
17
  import {
19
18
  type L1BlobInputs,
20
19
  type L1TxConfig,
21
20
  type L1TxRequest,
21
+ L1TxTimeoutError,
22
22
  type L1TxUtils,
23
23
  MAX_L1_TX_LIMIT,
24
24
  type TransactionStats,
25
25
  WEI_CONST,
26
26
  } from '@aztec/ethereum/l1-tx-utils';
27
- import { FormattedViemError, formatViemError, mergeAbis, tryExtractEvent } from '@aztec/ethereum/utils';
28
- import { sumBigint } from '@aztec/foundation/bigint';
29
- import { toHex as toPaddedHex } from '@aztec/foundation/bigint-buffer';
27
+ import {
28
+ FormattedViemError,
29
+ formatViemError,
30
+ mergeAbis,
31
+ tryDecodeRevertReason,
32
+ tryExtractEvent,
33
+ } from '@aztec/ethereum/utils';
30
34
  import { CheckpointNumber, SlotNumber } from '@aztec/foundation/branded-types';
31
35
  import { trimmedBytesLength } from '@aztec/foundation/buffer';
32
36
  import { pick } from '@aztec/foundation/collection';
33
37
  import type { Fr } from '@aztec/foundation/curves/bn254';
34
38
  import { TimeoutError } from '@aztec/foundation/error';
35
39
  import { EthAddress } from '@aztec/foundation/eth-address';
36
- import { Signature, type ViemSignature } from '@aztec/foundation/eth-signature';
40
+ import { Signature } from '@aztec/foundation/eth-signature';
37
41
  import { type Logger, createLogger } from '@aztec/foundation/log';
38
- import { makeBackoff, retry } from '@aztec/foundation/retry';
42
+ import { InterruptibleSleep } from '@aztec/foundation/sleep';
39
43
  import { bufferToHex } from '@aztec/foundation/string';
40
- import { DateProvider, Timer } from '@aztec/foundation/timer';
41
- import { EmpireBaseAbi, ErrorsAbi, RollupAbi } from '@aztec/l1-artifacts';
44
+ import { type DateProvider, Timer } from '@aztec/foundation/timer';
45
+ import { EmpireBaseAbi, ErrorsAbi, RollupAbi, SlashingProposerAbi } from '@aztec/l1-artifacts';
42
46
  import { type ProposerSlashAction, encodeSlashConsensusVotes } from '@aztec/slasher';
43
47
  import { CommitteeAttestationsAndSigners, type ValidateCheckpointResult } from '@aztec/stdlib/block';
44
48
  import type { Checkpoint } from '@aztec/stdlib/checkpoint';
45
- import { getLastL1SlotTimestampForL2Slot, getNextL1SlotTimestamp } from '@aztec/stdlib/epoch-helpers';
46
- import { SlashFactoryContract } from '@aztec/stdlib/l1-contracts';
49
+ import {
50
+ getLastL1SlotTimestampForL2Slot,
51
+ getNextL1SlotTimestamp,
52
+ getTimestampForSlot,
53
+ } from '@aztec/stdlib/epoch-helpers';
47
54
  import type { CheckpointHeader } from '@aztec/stdlib/rollup';
48
55
  import type { L1PublishCheckpointStats } from '@aztec/stdlib/stats';
49
56
  import { type TelemetryClient, type Tracer, getTelemetryClient, trackSpan } from '@aztec/telemetry-client';
50
57
 
51
58
  import {
59
+ type Abi,
52
60
  type Hex,
53
- type StateOverride,
54
61
  type TransactionReceipt,
55
62
  type TypedDataDefinition,
56
63
  encodeFunctionData,
57
64
  keccak256,
58
- multicall3Abi,
59
65
  toHex,
60
66
  } from 'viem';
61
67
 
62
68
  import type { SequencerPublisherConfig } from './config.js';
63
69
  import { type FailedL1Tx, type L1TxFailedStore, createL1TxFailedStore } from './l1_tx_failed_store/index.js';
70
+ import { type DroppedRequest, SequencerBundleSimulator } from './sequencer-bundle-simulator.js';
64
71
  import { SequencerPublisherMetrics } from './sequencer-publisher-metrics.js';
65
72
 
73
+ /**
74
+ * Returns true if the receipt indicates a successful send AND the expected event was emitted
75
+ * by the target contract. Both pieces are required: an aggregate3 entry that reverted will
76
+ * have receipt.status === 'success' but no event log.
77
+ */
78
+ function extractEventSuccess(
79
+ receipt: TransactionReceipt | undefined,
80
+ opts: { address: string; abi: Abi; eventName: string },
81
+ ): boolean {
82
+ if (!receipt || receipt.status !== 'success') {
83
+ return false;
84
+ }
85
+ return !!tryExtractEvent(receipt.logs, opts.address.toString() as Hex, opts.abi, opts.eventName);
86
+ }
87
+
88
+ /** Result of a sendRequests call, returned by both sendRequests() and sendRequestsAt(). */
89
+ export type SendRequestsResult = {
90
+ /** The L1 transaction receipt from the bundled multicall. */
91
+ result: { receipt: TransactionReceipt };
92
+ /** Actions that expired (past their deadline) before the request was sent. */
93
+ expiredActions: Action[];
94
+ /** Actions that were included in the sent L1 transaction. */
95
+ sentActions: Action[];
96
+ /** Actions whose L1 simulation succeeded (subset of sentActions). */
97
+ successfulActions: Action[];
98
+ /** Actions whose L1 simulation failed (subset of sentActions). */
99
+ failedActions: Action[];
100
+ };
101
+
66
102
  /** Arguments to the process method of the rollup contract */
67
103
  type L1ProcessArgs = {
68
104
  /** The L2 block header. */
@@ -82,18 +118,16 @@ type L1ProcessArgs = {
82
118
  export const Actions = [
83
119
  'invalidate-by-invalid-attestation',
84
120
  'invalidate-by-insufficient-attestations',
121
+ 'prune',
85
122
  'propose',
86
123
  'governance-signal',
87
- 'empire-slashing-signal',
88
- 'create-empire-payload',
89
- 'execute-empire-payload',
90
124
  'vote-offenses',
91
125
  'execute-slash',
92
126
  ] as const;
93
127
 
94
128
  export type Action = (typeof Actions)[number];
95
129
 
96
- type GovernanceSignalAction = Extract<Action, 'governance-signal' | 'empire-slashing-signal'>;
130
+ type GovernanceSignalAction = Extract<Action, 'governance-signal'>;
97
131
 
98
132
  // Sorting for actions such that invalidations go before proposals, and proposals go before votes
99
133
  export const compareActions = (a: Action, b: Action) => Actions.indexOf(a) - Actions.indexOf(b);
@@ -104,14 +138,22 @@ export type InvalidateCheckpointRequest = {
104
138
  gasUsed: bigint;
105
139
  checkpointNumber: CheckpointNumber;
106
140
  forcePendingCheckpointNumber: CheckpointNumber;
141
+ /** Archive at the rollback target checkpoint (checkpoint N-1). */
142
+ lastArchive: Fr;
143
+ };
144
+
145
+ type EnqueueProposeCheckpointOpts = {
146
+ txTimeoutAt?: Date;
107
147
  };
108
148
 
109
- interface RequestWithExpiry {
149
+ export interface RequestWithExpiry {
110
150
  action: Action;
111
151
  request: L1TxRequest;
112
152
  lastValidL2Slot: SlotNumber;
113
153
  gasConfig?: Pick<L1TxConfig, 'txTimeoutAt' | 'gasLimit'>;
114
154
  blobConfig?: L1BlobInputs;
155
+ /** Gas consumed by validateBlobs; stashed for the bundle simulate at send time. */
156
+ blobEvaluationGas?: bigint;
115
157
  checkSuccess: (
116
158
  request: L1TxRequest,
117
159
  result?: { receipt: TransactionReceipt; stats?: TransactionStats; errorMsg?: string },
@@ -121,26 +163,29 @@ interface RequestWithExpiry {
121
163
  export class SequencerPublisher {
122
164
  private interrupted = false;
123
165
  private metrics: SequencerPublisherMetrics;
166
+ private bundleSimulator: SequencerBundleSimulator;
124
167
  public epochCache: EpochCache;
125
168
  private failedTxStore?: Promise<L1TxFailedStore | undefined>;
126
169
 
127
- protected governanceLog = createLogger('sequencer:publisher:governance');
128
- protected slashingLog = createLogger('sequencer:publisher:slashing');
170
+ /**
171
+ * ABI used to decode raw revert payloads from dropped bundle entries when the original
172
+ * request did not carry an abi (e.g. the propose request). Merges every contract the
173
+ * publisher can route to so any of their custom errors decode against it.
174
+ */
175
+ private readonly revertDecoderAbi: Abi = mergeAbis([RollupAbi, SlashingProposerAbi, EmpireBaseAbi, ErrorsAbi]);
129
176
 
130
177
  protected lastActions: Partial<Record<Action, SlotNumber>> = {};
131
178
 
132
- private isPayloadEmptyCache: Map<string, boolean> = new Map<string, boolean>();
133
- private payloadProposedCache: Set<string> = new Set<string>();
134
-
135
179
  protected log: Logger;
136
180
  protected ethereumSlotDuration: bigint;
137
181
  protected aztecSlotDuration: bigint;
138
- private dateProvider: DateProvider;
182
+ private readonly previousL1BlockWaitTimeoutMs: number;
183
+ private readonly previousL1BlockWaitPollIntervalMs: number;
139
184
 
140
- private blobClient: BlobClientInterface;
185
+ /** Date provider for wall-clock time. */
186
+ private readonly dateProvider: DateProvider;
141
187
 
142
- /** Address to use for simulations in fisherman mode (actual proposer's address) */
143
- private proposerAddressForSimulation?: EthAddress;
188
+ private blobClient: BlobClientInterface;
144
189
 
145
190
  /** Optional callback to obtain a replacement publisher when the current one fails to send. */
146
191
  private getNextPublisher?: (excludeAddresses: EthAddress[]) => Promise<L1TxUtils | undefined>;
@@ -151,33 +196,34 @@ export class SequencerPublisher {
151
196
  /** Fee asset price oracle for computing price modifiers from Uniswap V4 */
152
197
  private feeAssetPriceOracle: FeeAssetPriceOracle;
153
198
 
154
- // A CALL to a cold address is 2700 gas
155
- public static MULTICALL_OVERHEAD_GAS_GUESS = 5000n;
156
-
157
- // Gas report for VotingWithSigTest shows a max gas of 100k, but we've seen it cost 700k+ in testnet
158
- public static VOTE_GAS_GUESS: bigint = 800_000n;
199
+ /** Interruptible sleep used by sendRequestsAt to wait until a target timestamp. */
200
+ private readonly interruptibleSleep = new InterruptibleSleep();
159
201
 
160
202
  public l1TxUtils: L1TxUtils;
161
203
  public rollupContract: RollupContract;
162
204
  public govProposerContract: GovernanceProposerContract;
163
- public slashingProposerContract: EmpireSlashingProposerContract | TallySlashingProposerContract | undefined;
164
- public slashFactoryContract: SlashFactoryContract;
205
+ public slashingProposerContract: SlashingProposerContract | undefined;
165
206
 
166
207
  public readonly tracer: Tracer;
167
208
 
168
209
  protected requests: RequestWithExpiry[] = [];
169
210
 
170
211
  constructor(
171
- private config: Pick<SequencerPublisherConfig, 'fishermanMode' | 'l1TxFailedStore'> &
212
+ private config: Pick<
213
+ SequencerPublisherConfig,
214
+ | 'fishermanMode'
215
+ | 'l1TxFailedStore'
216
+ | 'sequencerPublisherPreviousL1BlockWaitTimeoutMs'
217
+ | 'sequencerPublisherPreviousL1BlockWaitPollIntervalMs'
218
+ > &
172
219
  Pick<L1ContractsConfig, 'ethereumSlotDuration' | 'aztecSlotDuration'> & { l1ChainId: number },
173
220
  deps: {
174
221
  telemetry?: TelemetryClient;
175
222
  blobClient: BlobClientInterface;
176
223
  l1TxUtils: L1TxUtils;
177
224
  rollupContract: RollupContract;
178
- slashingProposerContract: EmpireSlashingProposerContract | TallySlashingProposerContract | undefined;
225
+ slashingProposerContract: SlashingProposerContract | undefined;
179
226
  governanceProposerContract: GovernanceProposerContract;
180
- slashFactoryContract: SlashFactoryContract;
181
227
  epochCache: EpochCache;
182
228
  dateProvider: DateProvider;
183
229
  metrics: SequencerPublisherMetrics;
@@ -189,11 +235,14 @@ export class SequencerPublisher {
189
235
  this.log = deps.log ?? createLogger('sequencer:publisher');
190
236
  this.ethereumSlotDuration = BigInt(config.ethereumSlotDuration);
191
237
  this.aztecSlotDuration = BigInt(config.aztecSlotDuration);
238
+ this.previousL1BlockWaitTimeoutMs = config.sequencerPublisherPreviousL1BlockWaitTimeoutMs;
239
+ this.previousL1BlockWaitPollIntervalMs = config.sequencerPublisherPreviousL1BlockWaitPollIntervalMs;
192
240
  this.dateProvider = deps.dateProvider;
193
241
  this.epochCache = deps.epochCache;
194
242
  this.lastActions = deps.lastActions;
195
243
 
196
244
  this.blobClient = deps.blobClient;
245
+ this.dateProvider = deps.dateProvider;
197
246
 
198
247
  const telemetry = deps.telemetry ?? getTelemetryClient();
199
248
  this.metrics = deps.metrics ?? new SequencerPublisherMetrics(telemetry, 'SequencerPublisher');
@@ -211,14 +260,12 @@ export class SequencerPublisher {
211
260
  const newSlashingProposer = await this.rollupContract.getSlashingProposer();
212
261
  this.slashingProposerContract = newSlashingProposer;
213
262
  });
214
- this.slashFactoryContract = deps.slashFactoryContract;
215
-
216
263
  // Initialize L1 fee analyzer for fisherman mode
217
264
  if (config.fishermanMode) {
218
265
  this.l1FeeAnalyzer = new L1FeeAnalyzer(
219
266
  this.l1TxUtils.client,
220
267
  deps.dateProvider,
221
- createLogger('sequencer:publisher:fee-analyzer'),
268
+ this.log.createChild('fee-analyzer'),
222
269
  );
223
270
  }
224
271
 
@@ -226,33 +273,112 @@ export class SequencerPublisher {
226
273
  this.feeAssetPriceOracle = new FeeAssetPriceOracle(
227
274
  this.l1TxUtils.client,
228
275
  this.rollupContract,
229
- createLogger('sequencer:publisher:price-oracle'),
276
+ this.log.createChild('price-oracle'),
230
277
  );
231
278
 
232
279
  // Initialize failed L1 tx store (optional, for test networks)
233
280
  this.failedTxStore = createL1TxFailedStore(config.l1TxFailedStore, this.log);
281
+
282
+ this.bundleSimulator = new SequencerBundleSimulator({
283
+ getL1TxUtils: () => this.l1TxUtils,
284
+ rollupContract: this.rollupContract,
285
+ epochCache: this.epochCache,
286
+ log: this.log.createChild('bundle-simulator'),
287
+ });
234
288
  }
235
289
 
236
290
  /**
237
- * Backs up a failed L1 transaction to the configured store for debugging.
238
- * Does nothing if no store is configured.
291
+ * Logs the gas-pricing data of a failed L1 transaction at warn so underpricing is diagnosable
292
+ * from logs even with no failed-tx store configured — and backs the record up to the store when
293
+ * one is. When captureFeeSummary is true, also records the fee data of the already-mined L1
294
+ * blocks in the target slot's inclusion window.
239
295
  */
240
- private backupFailedTx(failedTx: Omit<FailedL1Tx, 'timestamp'>): void {
241
- if (!this.failedTxStore) {
242
- return;
243
- }
244
-
296
+ private backupFailedTx(
297
+ failedTx: Omit<FailedL1Tx, 'timestamp'>,
298
+ opts?: { captureFeeSummary?: boolean; targetSlot?: SlotNumber; sharedFeeSummary?: FailedL1Tx['gasInfo'] },
299
+ ): void {
245
300
  const tx: FailedL1Tx = {
246
301
  ...failedTx,
247
302
  timestamp: Date.now(),
248
303
  };
249
304
 
250
305
  // Fire and forget - don't block on backup
251
- void this.failedTxStore
252
- .then(store => store?.saveFailedTx(tx))
253
- .catch(err => {
306
+ void (async () => {
307
+ try {
308
+ // Prefer a pre-captured summary (shared across a batch of failures in the same slot) so we
309
+ // don't re-read the fee window per record. A capture error must not lose the record itself.
310
+ const feeSummary =
311
+ opts?.sharedFeeSummary ??
312
+ (opts?.captureFeeSummary
313
+ ? await this.captureFeeEnvironment(opts.targetSlot).catch(() => undefined)
314
+ : undefined);
315
+ if (feeSummary) {
316
+ tx.gasInfo = { ...tx.gasInfo, ...feeSummary };
317
+ }
318
+ if (tx.gasInfo) {
319
+ this.log.warn(`Gas pricing data for failed L1 tx (${tx.failureType})`, {
320
+ failureType: tx.failureType,
321
+ actions: tx.context.actions,
322
+ slot: tx.context.slot,
323
+ ...tx.gasInfo,
324
+ ...tx.timing,
325
+ });
326
+ }
327
+ const store = await this.failedTxStore;
328
+ if (store) {
329
+ await store.saveFailedTx(tx);
330
+ }
331
+ } catch (err) {
254
332
  this.log.warn(`Failed to backup failed L1 tx to store`, err);
255
- });
333
+ }
334
+ })();
335
+ }
336
+
337
+ /**
338
+ * Captures per-block fee data for the L1 blocks in the target slot's inclusion window (the blocks the
339
+ * tx could have landed in) for underpricing diagnostics. Reads only already-mined blocks, so it never
340
+ * waits on the chain. Safe to call off the critical path: the underlying capture never throws, and this
341
+ * returns undefined when there is no target slot or the window is not yet mined (e.g. an early send
342
+ * failure), in which case the record simply carries no window data.
343
+ */
344
+ private async captureFeeEnvironment(targetL2Slot: SlotNumber | undefined): Promise<FailedL1Tx['gasInfo']> {
345
+ if (targetL2Slot === undefined) {
346
+ return undefined;
347
+ }
348
+ const l1Constants = this.epochCache.getL1Constants();
349
+ // The inclusion window is [start of slot N, start of slot N+1): all L1 blocks that can include a tx
350
+ // for this L2 slot. getTimestampForSlot returns seconds, matching block.timestamp.
351
+ const windowStartS = getTimestampForSlot(targetL2Slot, l1Constants);
352
+ const windowEndS = getTimestampForSlot(SlotNumber(Number(targetL2Slot) + 1), l1Constants);
353
+ const windowBlocks = await captureWindowBlockFees(this.l1TxUtils.client, windowStartS, windowEndS);
354
+ if (windowBlocks.length === 0) {
355
+ return undefined;
356
+ }
357
+ return { windowBlocks };
358
+ }
359
+
360
+ /** Computes timing info relative to the L2 slot deadline. */
361
+ private computeTimingInfo(targetL2Slot: SlotNumber | undefined): FailedL1Tx['timing'] {
362
+ if (targetL2Slot === undefined) {
363
+ return undefined;
364
+ }
365
+ const l1Constants = this.epochCache.getL1Constants();
366
+ const slotDeadlineS = getTimestampForSlot(SlotNumber(Number(targetL2Slot) + 1), l1Constants);
367
+ const slotDeadlineMs = Number(slotDeadlineS) * 1000;
368
+ return {
369
+ targetL2Slot: Number(targetL2Slot),
370
+ slotDeadlineTimestampS: slotDeadlineS,
371
+ msUntilSlotDeadline: slotDeadlineMs - this.dateProvider.now(),
372
+ };
373
+ }
374
+
375
+ /**
376
+ * Builds an id for a synthetic failure record (send-error/timeout) that has no on-chain tx hash.
377
+ * Includes the failure time so each attempt — including retries of the same slot — is stored as its
378
+ * own record rather than overwriting the previous one.
379
+ */
380
+ private failureRecordId(actions: string[], targetSlot: SlotNumber | undefined): Hex {
381
+ return keccak256(toHex(`${actions.join(',')}:${targetSlot ?? ''}:${Date.now()}`));
256
382
  }
257
383
 
258
384
  public getRollupContract(): RollupContract {
@@ -261,10 +387,14 @@ export class SequencerPublisher {
261
387
 
262
388
  /**
263
389
  * Gets the fee asset price modifier from the oracle.
264
- * Returns 0n if the oracle query fails.
390
+ *
391
+ * @param predictedParentEthPerFeeAssetE12 - Optional predicted parent eth-per-fee-asset (E12).
392
+ * Pipelined proposers should pass the value from the predicted parent fee header so the
393
+ * modifier matches the parent L1 will use when applying it.
394
+ * @returns The fee asset price modifier in basis points, or 0n if the oracle query fails.
265
395
  */
266
- public getFeeAssetPriceModifier(): Promise<bigint> {
267
- return this.feeAssetPriceOracle.computePriceModifier();
396
+ public getFeeAssetPriceModifier(predictedParentEthPerFeeAssetE12?: bigint): Promise<bigint> {
397
+ return this.feeAssetPriceOracle.computePriceModifier(predictedParentEthPerFeeAssetE12);
268
398
  }
269
399
 
270
400
  public getSenderAddress() {
@@ -278,14 +408,6 @@ export class SequencerPublisher {
278
408
  return this.l1FeeAnalyzer;
279
409
  }
280
410
 
281
- /**
282
- * Sets the proposer address to use for simulations in fisherman mode.
283
- * @param proposerAddress - The actual proposer's address to use for balance lookups in simulations
284
- */
285
- public setProposerAddressForSimulation(proposerAddress: EthAddress | undefined) {
286
- this.proposerAddressForSimulation = proposerAddress;
287
- }
288
-
289
411
  public addRequest(request: RequestWithExpiry) {
290
412
  this.requests.push(request);
291
413
  }
@@ -363,22 +485,26 @@ export class SequencerPublisher {
363
485
 
364
486
  /**
365
487
  * Sends all requests that are still valid.
488
+ * @param targetSlot - The target L2 slot for this send. When provided (the production path, via
489
+ * sendRequestsAt), it is threaded into bundleSimulate so the block.timestamp override matches
490
+ * the slot the propose is built for. When omitted, falls back to getCurrentL2Slot() for the
491
+ * AutomineSequencer, which publishes synchronously within the current slot.
366
492
  * @returns one of:
367
493
  * - A receipt and stats if the tx succeeded
368
494
  * - a receipt and errorMsg if it failed on L1
369
495
  * - undefined if no valid requests are found OR the tx failed to send.
370
496
  */
371
497
  @trackSpan('SequencerPublisher.sendRequests')
372
- public async sendRequests() {
498
+ public async sendRequests(targetSlot?: SlotNumber): Promise<SendRequestsResult | undefined> {
373
499
  const requestsToProcess = [...this.requests];
374
500
  this.requests = [];
501
+
375
502
  if (this.interrupted || requestsToProcess.length === 0) {
376
503
  return undefined;
377
504
  }
378
- const currentL2Slot = this.getCurrentL2Slot();
505
+ const currentL2Slot = targetSlot ?? this.getCurrentL2Slot();
379
506
  this.log.debug(`Sending requests on L2 slot ${currentL2Slot}`);
380
507
  const validRequests = requestsToProcess.filter(request => request.lastValidL2Slot >= currentL2Slot);
381
- const validActions = validRequests.map(x => x.action);
382
508
  const expiredActions = requestsToProcess
383
509
  .filter(request => request.lastValidL2Slot < currentL2Slot)
384
510
  .map(x => x.action);
@@ -401,73 +527,101 @@ export class SequencerPublisher {
401
527
  return undefined;
402
528
  }
403
529
 
404
- // @note - we can only have one blob config per bundle
405
- // find requests with gas and blob configs
406
- // See https://github.com/AztecProtocol/aztec-packages/issues/11513
530
+ // Collect earliest txTimeoutAt across all requests.
407
531
  const gasConfigs = validRequests.filter(request => request.gasConfig).map(request => request.gasConfig);
408
- const blobConfigs = validRequests.filter(request => request.blobConfig).map(request => request.blobConfig);
409
-
410
- if (blobConfigs.length > 1) {
411
- throw new Error('Multiple blob configs found');
412
- }
413
-
414
- const blobConfig = blobConfigs[0];
415
-
416
- // Merge gasConfigs. Yields the sum of gasLimits, and the earliest txTimeoutAt, or undefined if no gasConfig sets them.
417
- const gasLimits = gasConfigs.map(g => g?.gasLimit).filter((g): g is bigint => g !== undefined);
418
- let gasLimit = gasLimits.length > 0 ? sumBigint(gasLimits) : undefined; // sum
419
- // Cap at L1 block gas limit so the node accepts the tx ("gas limit too high" otherwise).
420
- const maxGas = MAX_L1_TX_LIMIT;
421
- if (gasLimit !== undefined && gasLimit > maxGas) {
422
- this.log.debug('Capping bundled tx gas limit to L1 max', {
423
- requested: gasLimit,
424
- capped: maxGas,
425
- });
426
- gasLimit = maxGas;
427
- }
428
532
  const txTimeoutAts = gasConfigs.map(g => g?.txTimeoutAt).filter((g): g is Date => g !== undefined);
429
- const txTimeoutAt = txTimeoutAts.length > 0 ? new Date(Math.min(...txTimeoutAts.map(g => g.getTime()))) : undefined; // earliest
430
- const txConfig: RequestWithExpiry['gasConfig'] = { gasLimit, txTimeoutAt };
533
+ const txTimeoutAt = txTimeoutAts.length > 0 ? new Date(Math.min(...txTimeoutAts.map(g => g.getTime()))) : undefined;
431
534
 
432
535
  // Sort the requests so that proposals always go first
433
536
  // This ensures the committee gets precomputed correctly
434
537
  validRequests.sort((a, b) => compareActions(a.action, b.action));
435
538
 
436
539
  try {
437
- // Capture context for failed tx backup before sending
438
- const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
439
- const multicallData = encodeFunctionData({
440
- abi: multicall3Abi,
441
- functionName: 'aggregate3',
442
- args: [
443
- validRequests.map(r => ({
444
- target: r.request.to!,
445
- callData: r.request.data!,
446
- allowFailure: true,
447
- })),
448
- ],
449
- });
450
- const blobDataHex = blobConfig?.blobs?.map(b => toHex(b)) as Hex[] | undefined;
540
+ // Bundle-level eth_simulateV1: filters out entries that revert and derives the gasLimit.
541
+ const bundleResult = await this.bundleSimulator.simulate(validRequests, currentL2Slot);
451
542
 
452
- const txContext = { multicallData, blobData: blobDataHex, l1BlockNumber };
543
+ if (bundleResult.kind === 'aborted') {
544
+ this.logDroppedInSim(bundleResult.droppedRequests);
545
+ void this.backupDroppedInSim(bundleResult.droppedRequests, currentL2Slot).catch(err =>
546
+ this.log.error(`Failed to backup requests dropped in simulation`, err),
547
+ );
548
+ return undefined;
549
+ }
550
+
551
+ const { requests, droppedRequests, gasLimit } =
552
+ bundleResult.kind === 'fallback'
553
+ ? {
554
+ requests: bundleResult.requests,
555
+ droppedRequests: bundleResult.droppedRequests,
556
+ gasLimit: MAX_L1_TX_LIMIT,
557
+ }
558
+ : bundleResult;
559
+
560
+ this.logDroppedInSim(droppedRequests);
561
+
562
+ // Compute blobConfig from survivors (not original validRequests) so that if the propose
563
+ // entry was dropped by bundleSimulate we don't attach a blob-typed config to a non-blob tx.
564
+ const [blobConfig] = requests.filter(r => r.blobConfig).map(r => r.blobConfig);
565
+ const txConfig: RequestWithExpiry['gasConfig'] = { gasLimit, txTimeoutAt };
453
566
 
454
567
  this.log.debug('Forwarding transactions', {
455
- validRequests: validRequests.map(request => request.action),
568
+ requests: requests.map(request => request.action),
456
569
  txConfig,
457
570
  });
458
- const result = await this.forwardWithPublisherRotation(validRequests, txConfig, blobConfig);
571
+ const result = await this.forwardWithPublisherRotation(requests, txConfig, blobConfig, currentL2Slot);
459
572
  if (result === undefined) {
460
573
  return undefined;
461
574
  }
462
- const { successfulActions = [], failedActions = [] } = this.callbackBundledTransactions(
463
- validRequests,
575
+ const { successfulActions = [], failedActions = [] } = this.callbackBundledTransactions(requests, result);
576
+ const allFailedActions = [...failedActions, ...droppedRequests.map(d => d.request.action)];
577
+ return {
464
578
  result,
465
- txContext,
466
- );
467
- return { result, expiredActions, sentActions: validActions, successfulActions, failedActions };
579
+ expiredActions,
580
+ sentActions: requests.map(x => x.action),
581
+ successfulActions,
582
+ failedActions: allFailedActions,
583
+ };
468
584
  } catch (err) {
469
585
  const viemError = formatViemError(err);
470
586
  this.log.error(`Failed to publish bundled transactions`, viemError);
587
+ if (err instanceof TimeoutError) {
588
+ const timeoutState = err instanceof L1TxTimeoutError ? err.txState : undefined;
589
+ void (async () => {
590
+ // The RPC is likely degraded right after a timeout, so back up without the block number
591
+ // rather than leaking an unhandled rejection.
592
+ const l1BlockNumber = await this.l1TxUtils.getBlockNumber().catch(() => 0n);
593
+ this.backupFailedTx(
594
+ {
595
+ id: this.failureRecordId(
596
+ validRequests.map(r => r.action),
597
+ currentL2Slot,
598
+ ),
599
+ failureType: 'timeout',
600
+ request: { to: MULTI_CALL_3_ADDRESS as Hex, data: '0x' as Hex },
601
+ l1BlockNumber,
602
+ error: { message: viemError.message, name: 'TimeoutError' },
603
+ context: {
604
+ actions: validRequests.map(r => r.action),
605
+ requests: validRequests
606
+ .filter(r => r.request.to !== null)
607
+ .map(r => ({ action: r.action, to: r.request.to! as Hex, data: r.request.data! })),
608
+ sender: this.getSenderAddress().toString(),
609
+ slot: Number(currentL2Slot),
610
+ },
611
+ timing: this.computeTimingInfo(currentL2Slot),
612
+ gasInfo: timeoutState
613
+ ? {
614
+ sentGasPriceLadder: timeoutState.gasPriceHistory,
615
+ attempts: timeoutState.attempts,
616
+ gasLimit: timeoutState.gasLimit,
617
+ nonce: timeoutState.nonce,
618
+ }
619
+ : undefined,
620
+ },
621
+ { captureFeeSummary: true, targetSlot: currentL2Slot },
622
+ );
623
+ })();
624
+ }
471
625
  return undefined;
472
626
  } finally {
473
627
  try {
@@ -481,6 +635,54 @@ export class SequencerPublisher {
481
635
  }
482
636
  }
483
637
 
638
+ /** Logs entries dropped by bundle simulation as warnings on the publisher's logger. */
639
+ private logDroppedInSim(dropped: DroppedRequest[]): void {
640
+ for (const drop of dropped) {
641
+ const revertReasonDecoded = drop.revertReason ?? tryDecodeRevertReason(drop.returnData, this.revertDecoderAbi);
642
+ this.log.warn('Bundle entry dropped: action reverted in sim', {
643
+ action: drop.request.action,
644
+ revertReason: revertReasonDecoded ?? drop.returnData,
645
+ revertReasonDecoded,
646
+ returnData: drop.returnData,
647
+ });
648
+ }
649
+ }
650
+
651
+ /** Backs up entries dropped by bundle simulation, one record per dropped action. */
652
+ private async backupDroppedInSim(dropped: DroppedRequest[], targetSlot?: SlotNumber): Promise<void> {
653
+ if (dropped.length === 0) {
654
+ return;
655
+ }
656
+ // Invoked as `void backupDroppedInSim(...)` on the publish path, so it must not throw.
657
+ try {
658
+ const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
659
+ // Every dropped entry failed in the same slot against the same L1 fee conditions, so capture
660
+ // the fee environment once and share it rather than re-reading the window per entry.
661
+ const sharedFeeSummary = await this.captureFeeEnvironment(targetSlot).catch(() => undefined);
662
+ const timing = this.computeTimingInfo(targetSlot);
663
+ for (const { request: req } of dropped) {
664
+ this.backupFailedTx(
665
+ {
666
+ id: keccak256(req.request.data!),
667
+ failureType: 'simulation',
668
+ request: { to: req.request.to! as Hex, data: req.request.data! },
669
+ l1BlockNumber,
670
+ error: { message: 'Bundle entry dropped: action reverted in sim' },
671
+ context: {
672
+ actions: [req.action],
673
+ sender: this.getSenderAddress().toString(),
674
+ slot: targetSlot !== undefined ? Number(targetSlot) : undefined,
675
+ },
676
+ timing,
677
+ },
678
+ { sharedFeeSummary },
679
+ );
680
+ }
681
+ } catch (err) {
682
+ this.log.warn(`Failed to back up dropped-in-sim entries`, err);
683
+ }
684
+ }
685
+
484
686
  /**
485
687
  * Forwards transactions via Multicall3, rotating to the next available publisher if a send
486
688
  * failure occurs (i.e. the tx never reached the chain).
@@ -490,20 +692,32 @@ export class SequencerPublisher {
490
692
  validRequests: RequestWithExpiry[],
491
693
  txConfig: RequestWithExpiry['gasConfig'],
492
694
  blobConfig: L1BlobInputs | undefined,
695
+ targetSlot?: SlotNumber,
493
696
  ) {
697
+ if (!txConfig?.gasLimit) {
698
+ throw new Error('gasLimit is required for bundled transactions');
699
+ }
700
+ const txConfigWithGasLimit = txConfig as L1TxConfig & { gasLimit: bigint };
701
+
494
702
  const triedAddresses: EthAddress[] = [];
495
703
  let currentPublisher = this.l1TxUtils;
496
704
 
497
705
  while (true) {
706
+ if (txConfig.txTimeoutAt && new Date() > txConfig.txTimeoutAt) {
707
+ this.log.warn(`Tx timeout (${txConfig.txTimeoutAt.toISOString()}) elapsed; stopping publisher rotation`, {
708
+ triedAddresses: triedAddresses.map(a => a.toString()),
709
+ });
710
+ return undefined;
711
+ }
498
712
  triedAddresses.push(currentPublisher.getSenderAddress());
713
+
499
714
  try {
500
715
  const result = await Multicall3.forward(
501
716
  validRequests.map(r => r.request),
502
717
  currentPublisher,
503
- txConfig,
718
+ txConfigWithGasLimit,
504
719
  blobConfig,
505
- this.rollupContract.address,
506
- this.log,
720
+ { gasLimitRequired: true },
507
721
  );
508
722
  this.l1TxUtils = currentPublisher;
509
723
  return result;
@@ -511,9 +725,17 @@ export class SequencerPublisher {
511
725
  if (err instanceof TimeoutError) {
512
726
  throw err;
513
727
  }
728
+ if (err instanceof MulticallForwarderRevertedError) {
729
+ this.log.error('Forwarder transaction reverted on-chain; not rotating publisher', err, {
730
+ transactionHash: err.receipt.transactionHash,
731
+ });
732
+ this.backupRevertFailure(validRequests, err, currentPublisher, targetSlot);
733
+ return undefined;
734
+ }
514
735
  const viemError = formatViemError(err);
515
736
  if (!this.getNextPublisher) {
516
737
  this.log.error('Failed to publish bundled transactions', viemError);
738
+ this.backupSendFailure(validRequests, viemError, currentPublisher, targetSlot);
517
739
  return undefined;
518
740
  }
519
741
  this.log.warn(
@@ -522,7 +744,12 @@ export class SequencerPublisher {
522
744
  );
523
745
  const nextPublisher = await this.getNextPublisher([...triedAddresses]);
524
746
  if (!nextPublisher) {
525
- this.log.error('All available publishers exhausted, failed to publish bundled transactions');
747
+ this.log.error(
748
+ `All available publishers exhausted (tried ${triedAddresses.length}), failed to publish bundled transactions`,
749
+ viemError,
750
+ { triedAddresses: triedAddresses.map(a => a.toString()) },
751
+ );
752
+ this.backupSendFailure(validRequests, viemError, currentPublisher, targetSlot);
526
753
  return undefined;
527
754
  }
528
755
  currentPublisher = nextPublisher;
@@ -530,74 +757,179 @@ export class SequencerPublisher {
530
757
  }
531
758
  }
532
759
 
533
- private callbackBundledTransactions(
760
+ /** Backs up an on-chain revert failure to the failed tx store. */
761
+ private backupRevertFailure(
534
762
  requests: RequestWithExpiry[],
535
- result: { receipt: TransactionReceipt; errorMsg?: string } | FormattedViemError | undefined,
536
- txContext: { multicallData: Hex; blobData?: Hex[]; l1BlockNumber: bigint },
537
- ) {
538
- const actionsListStr = requests.map(r => r.action).join(', ');
539
- if (result instanceof FormattedViemError) {
540
- this.log.error(`Failed to publish bundled transactions (${actionsListStr})`, result);
541
- this.backupFailedTx({
542
- id: keccak256(txContext.multicallData),
543
- failureType: 'send-error',
544
- request: { to: MULTI_CALL_3_ADDRESS, data: txContext.multicallData },
545
- blobData: txContext.blobData,
546
- l1BlockNumber: txContext.l1BlockNumber.toString(),
547
- error: { message: result.message, name: result.name },
763
+ err: MulticallForwarderRevertedError,
764
+ publisher: L1TxUtils,
765
+ targetSlot?: SlotNumber,
766
+ ): void {
767
+ this.backupFailedTx(
768
+ {
769
+ id: err.receipt.transactionHash,
770
+ failureType: 'revert',
771
+ request: { to: MULTI_CALL_3_ADDRESS as Hex, data: '0x' as Hex },
772
+ l1BlockNumber: err.receipt.blockNumber,
773
+ receipt: {
774
+ transactionHash: err.receipt.transactionHash,
775
+ blockNumber: err.receipt.blockNumber,
776
+ gasUsed: err.receipt.gasUsed,
777
+ status: 'reverted',
778
+ },
779
+ error: { message: err.message, name: err.name },
548
780
  context: {
549
781
  actions: requests.map(r => r.action),
550
- requests: requests.map(r => ({ action: r.action, to: r.request.to! as Hex, data: r.request.data! })),
551
- sender: this.getSenderAddress().toString(),
782
+ requests: requests
783
+ .filter(r => r.request.to !== null)
784
+ .map(r => ({ action: r.action, to: r.request.to! as Hex, data: r.request.data! })),
785
+ sender: publisher.getSenderAddress().toString(),
786
+ slot: targetSlot !== undefined ? Number(targetSlot) : undefined,
552
787
  },
788
+ gasInfo: err.txState
789
+ ? {
790
+ sentGasPrice: err.txState.gasPrice,
791
+ gasLimit: err.txState.gasLimit,
792
+ nonce: err.txState.nonce,
793
+ }
794
+ : undefined,
795
+ timing: this.computeTimingInfo(targetSlot),
796
+ },
797
+ { captureFeeSummary: true, targetSlot },
798
+ );
799
+ }
800
+
801
+ /** Backs up a send failure (tx never reached chain) to the failed tx store. */
802
+ private backupSendFailure(
803
+ requests: RequestWithExpiry[],
804
+ error: FormattedViemError | Error,
805
+ publisher: L1TxUtils,
806
+ targetSlot?: SlotNumber,
807
+ ): void {
808
+ // If we can't get the block number, still back up without it.
809
+ void this.l1TxUtils
810
+ .getBlockNumber()
811
+ .catch(() => 0n)
812
+ .then(l1BlockNumber => {
813
+ this.backupFailedTx(
814
+ {
815
+ id: this.failureRecordId(
816
+ requests.map(r => r.action),
817
+ targetSlot,
818
+ ),
819
+ failureType: 'send-error',
820
+ request: { to: MULTI_CALL_3_ADDRESS as Hex, data: '0x' as Hex },
821
+ l1BlockNumber,
822
+ error: {
823
+ message: error.message,
824
+ name: 'name' in error ? error.name : undefined,
825
+ },
826
+ context: {
827
+ actions: requests.map(r => r.action),
828
+ requests: requests
829
+ .filter(r => r.request.to !== null)
830
+ .map(r => ({ action: r.action, to: r.request.to! as Hex, data: r.request.data! })),
831
+ sender: publisher.getSenderAddress().toString(),
832
+ slot: targetSlot !== undefined ? Number(targetSlot) : undefined,
833
+ },
834
+ timing: this.computeTimingInfo(targetSlot),
835
+ },
836
+ { captureFeeSummary: true, targetSlot },
837
+ );
553
838
  });
554
- return { failedActions: requests.map(r => r.action) };
555
- } else {
556
- this.log.verbose(`Published bundled transactions (${actionsListStr})`, {
557
- result,
558
- requests: requests.map(r => ({
559
- ...r,
560
- // Avoid logging large blob data
561
- blobConfig: r.blobConfig
562
- ? { ...r.blobConfig, blobs: r.blobConfig.blobs.map(b => ({ size: trimmedBytesLength(b) })) }
563
- : undefined,
564
- })),
565
- });
566
- const successfulActions: Action[] = [];
567
- const failedActions: Action[] = [];
568
- for (const request of requests) {
569
- if (request.checkSuccess(request.request, result)) {
570
- successfulActions.push(request.action);
571
- } else {
572
- failedActions.push(request.action);
839
+ }
840
+
841
+ /*
842
+ * Schedules sending all enqueued requests at (or after) the start of the given L2 slot.
843
+ */
844
+ public async sendRequestsAt(targetSlot: SlotNumber): Promise<SendRequestsResult | undefined> {
845
+ await this.waitForTargetSlot(targetSlot);
846
+ if (this.interrupted) {
847
+ return undefined;
848
+ }
849
+
850
+ return this.sendRequests(targetSlot);
851
+ }
852
+
853
+ /**
854
+ * Sleeps until one L1 slot before the L2 slot boundary, and then waits for that L1 block
855
+ * to be mined, so we don't risk being included in it. If that block never gets mined after
856
+ * a timeout, we assume it got skipped on L1, so we send the tx anyway.
857
+ */
858
+ private async waitForTargetSlot(targetSlot: SlotNumber): Promise<void> {
859
+ const l1Constants = this.epochCache.getL1Constants();
860
+ const nowInSeconds = this.dateProvider.nowInSeconds();
861
+ const startOfTargetSlotTs = getTimestampForSlot(targetSlot, l1Constants);
862
+ const previousL1BlockTs = startOfTargetSlotTs - this.ethereumSlotDuration;
863
+ const waitDeadlineTs = previousL1BlockTs + BigInt(this.previousL1BlockWaitTimeoutMs / 1000);
864
+ const logCtx = { targetSlot, startOfTargetSlotTs, nowInSeconds, previousL1BlockTs, waitDeadlineTs };
865
+
866
+ // Check if we are already past time
867
+ if (nowInSeconds >= startOfTargetSlotTs) {
868
+ this.log.verbose(`Target slot ${targetSlot} already started, sending requests immediately`, logCtx);
869
+ return;
870
+ }
871
+
872
+ // Otherwise we wait
873
+ this.log.debug(`Waiting for slot ${targetSlot} before sending requests`, logCtx);
874
+
875
+ // Wait until previous L1 block timestamp first
876
+ const sleepMs = (Number(previousL1BlockTs) - nowInSeconds) * 1000;
877
+ if (sleepMs > 0 && !this.interrupted) {
878
+ this.log.trace(`Sleeping ${sleepMs}ms before waiting for previous L1 block`, logCtx);
879
+ await this.interruptibleSleep.sleep(sleepMs);
880
+ }
881
+
882
+ // Then loop until we see the previous L1 block, so we know that we cannot be included in it.
883
+ // We time out after a while, once we are sure that that block is skipped in L1.
884
+ while (!this.interrupted) {
885
+ try {
886
+ const nowInSeconds = this.dateProvider.nowInSeconds();
887
+ logCtx.nowInSeconds = nowInSeconds;
888
+
889
+ if (nowInSeconds >= waitDeadlineTs) {
890
+ this.log.warn(`Timed out waiting for previous L1 block before sending requests, proceeding`, logCtx);
891
+ return;
573
892
  }
893
+
894
+ const latestBlockTs = await this.l1TxUtils.getBlock().then(b => b.timestamp);
895
+ if (latestBlockTs >= previousL1BlockTs) {
896
+ this.log.debug(`Previous L1 block mined, proceeding to send requests`, { ...logCtx, latestBlockTs });
897
+ return;
898
+ }
899
+ this.log.trace(`Previous L1 block not mined yet, continuing to wait`, { ...logCtx, latestBlockTs });
900
+ } catch (err) {
901
+ this.log.error(`Error while waiting for previous L1 block before sending requests; retrying`, err, logCtx);
902
+ } finally {
903
+ await this.interruptibleSleep.sleep(this.previousL1BlockWaitPollIntervalMs);
574
904
  }
575
- // Single backup for the whole reverted tx
576
- if (failedActions.length > 0 && result?.receipt?.status === 'reverted') {
577
- this.backupFailedTx({
578
- id: result.receipt.transactionHash,
579
- failureType: 'revert',
580
- request: { to: MULTI_CALL_3_ADDRESS, data: txContext.multicallData },
581
- blobData: txContext.blobData,
582
- l1BlockNumber: result.receipt.blockNumber.toString(),
583
- receipt: {
584
- transactionHash: result.receipt.transactionHash,
585
- blockNumber: result.receipt.blockNumber.toString(),
586
- gasUsed: (result.receipt.gasUsed ?? 0n).toString(),
587
- status: 'reverted',
588
- },
589
- error: { message: result.errorMsg ?? 'Transaction reverted' },
590
- context: {
591
- actions: failedActions,
592
- requests: requests
593
- .filter(r => failedActions.includes(r.action))
594
- .map(r => ({ action: r.action, to: r.request.to! as Hex, data: r.request.data! })),
595
- sender: this.getSenderAddress().toString(),
596
- },
597
- });
905
+ }
906
+ }
907
+
908
+ private callbackBundledTransactions(
909
+ requests: RequestWithExpiry[],
910
+ result: { receipt: TransactionReceipt; multicallData: Hex },
911
+ ) {
912
+ const actionsListStr = requests.map(r => r.action).join(', ');
913
+ this.log.verbose(`Published bundled transactions (${actionsListStr})`, {
914
+ result,
915
+ requests: requests.map(r => ({
916
+ ...r,
917
+ // Avoid logging large blob data
918
+ blobConfig: r.blobConfig
919
+ ? { ...r.blobConfig, blobs: r.blobConfig.blobs.map(b => ({ size: trimmedBytesLength(b) })) }
920
+ : undefined,
921
+ })),
922
+ });
923
+ const successfulActions: Action[] = [];
924
+ const failedActions: Action[] = [];
925
+ for (const request of requests) {
926
+ if (request.checkSuccess(request.request, result)) {
927
+ successfulActions.push(request.action);
928
+ } else {
929
+ failedActions.push(request.action);
598
930
  }
599
- return { successfulActions, failedActions };
600
931
  }
932
+ return { successfulActions, failedActions };
601
933
  }
602
934
 
603
935
  /**
@@ -605,25 +937,27 @@ export class SequencerPublisher {
605
937
  * @param tipArchive - The archive to check
606
938
  * @returns The slot and block number if it is possible to propose, undefined otherwise
607
939
  */
608
- public canProposeAt(
609
- tipArchive: Fr,
610
- msgSender: EthAddress,
611
- opts: { forcePendingCheckpointNumber?: CheckpointNumber; pipelined?: boolean } = {},
612
- ) {
940
+ public async canProposeAt(tipArchive: Fr, msgSender: EthAddress, simulationOverridesPlan?: SimulationOverridesPlan) {
613
941
  // TODO: #14291 - should loop through multiple keys to check if any of them can propose
614
- const ignoredErrors = ['SlotAlreadyInChain', 'InvalidProposer', 'InvalidArchive'];
942
+ // These errors are expected when we cannot actually propose right now — usually because our
943
+ // local view of the chain is ahead of L1 (proposed parent hasn't landed yet, or someone
944
+ // else has just landed the slot, or the archive override doesn't match). We log a warn and
945
+ // skip the proposal; we do NOT treat these as bugs.
946
+ const expectedErrors = ['SlotAlreadyInChain', 'InvalidProposer', 'InvalidArchive'];
615
947
 
616
- const pipelined = opts.pipelined ?? this.epochCache.isProposerPipeliningEnabled();
617
- const slotOffset = pipelined ? this.aztecSlotDuration : 0n;
948
+ const slotOffset = this.aztecSlotDuration;
618
949
  const nextL1SlotTs = this.getNextL1SlotTimestamp() + slotOffset;
619
950
 
620
951
  return this.rollupContract
621
- .canProposeAt(tipArchive.toBuffer(), msgSender.toString(), nextL1SlotTs, {
622
- forcePendingCheckpointNumber: opts.forcePendingCheckpointNumber,
623
- })
952
+ .canProposeAt(
953
+ tipArchive.toBuffer(),
954
+ msgSender.toString(),
955
+ nextL1SlotTs,
956
+ await buildSimulationOverridesStateOverride(this.rollupContract, simulationOverridesPlan),
957
+ )
624
958
  .catch(err => {
625
- if (err instanceof FormattedViemError && ignoredErrors.find(e => err.message.includes(e))) {
626
- this.log.warn(`Failed canProposeAtTime check with ${ignoredErrors.find(e => err.message.includes(e))}`, {
959
+ if (err instanceof FormattedViemError && expectedErrors.find(e => err.message.includes(e))) {
960
+ this.log.warn(`Failed canProposeAtTime check with ${expectedErrors.find(e => err.message.includes(e))}`, {
627
961
  error: err.message,
628
962
  });
629
963
  } else {
@@ -642,13 +976,13 @@ export class SequencerPublisher {
642
976
  @trackSpan('SequencerPublisher.validateBlockHeader')
643
977
  public async validateBlockHeader(
644
978
  header: CheckpointHeader,
645
- opts?: { forcePendingCheckpointNumber: CheckpointNumber | undefined },
979
+ simulationOverridesPlan?: SimulationOverridesPlan,
646
980
  ): Promise<void> {
647
981
  const flags = { ignoreDA: true, ignoreSignatures: true };
648
982
 
649
983
  const args = [
650
984
  header.toViem(),
651
- CommitteeAttestationsAndSigners.empty().getPackedAttestations(),
985
+ CommitteeAttestationsAndSigners.packAttestations([]),
652
986
  [], // no signers
653
987
  Signature.empty().toViemSignature(),
654
988
  `0x${'0'.repeat(64)}`, // 32 empty bytes
@@ -656,10 +990,9 @@ export class SequencerPublisher {
656
990
  flags,
657
991
  ] as const;
658
992
 
659
- const ts = this.getSimulationTimestamp(header.slotNumber);
660
- const stateOverrides = await this.rollupContract.makePendingCheckpointNumberOverride(
661
- opts?.forcePendingCheckpointNumber,
662
- );
993
+ const l1Constants = this.epochCache.getL1Constants();
994
+ const ts = getLastL1SlotTimestampForL2Slot(header.slotNumber, l1Constants);
995
+ const stateOverrides = await buildSimulationOverridesStateOverride(this.rollupContract, simulationOverridesPlan);
663
996
  let balance = 0n;
664
997
  if (this.config.fishermanMode) {
665
998
  // In fisherman mode, we can't know where the proposer is publishing from
@@ -732,6 +1065,7 @@ export class SequencerPublisher {
732
1065
  gasUsed,
733
1066
  checkpointNumber,
734
1067
  forcePendingCheckpointNumber: CheckpointNumber(checkpointNumber - 1),
1068
+ lastArchive: validationResult.checkpoint.lastArchive,
735
1069
  reason,
736
1070
  };
737
1071
  } catch (err) {
@@ -744,8 +1078,8 @@ export class SequencerPublisher {
744
1078
  `Simulation for invalidate checkpoint ${checkpointNumber} failed due to checkpoint not being in pending chain`,
745
1079
  { ...logData, request, error: viemError.message },
746
1080
  );
747
- const latestPendingCheckpointNumber = await this.rollupContract.getCheckpointNumber();
748
- if (latestPendingCheckpointNumber < checkpointNumber) {
1081
+ const latestProposedCheckpointNumber = await this.rollupContract.getCheckpointNumber();
1082
+ if (latestProposedCheckpointNumber < checkpointNumber) {
749
1083
  this.log.verbose(`Checkpoint ${checkpointNumber} has already been invalidated`, { ...logData });
750
1084
  return undefined;
751
1085
  } else {
@@ -765,18 +1099,22 @@ export class SequencerPublisher {
765
1099
 
766
1100
  // Otherwise, throw. We cannot build the next checkpoint if we cannot invalidate the previous one.
767
1101
  this.log.error(`Simulation for invalidate checkpoint ${checkpointNumber} failed`, viemError, logData);
768
- this.backupFailedTx({
769
- id: keccak256(request.data!),
770
- failureType: 'simulation',
771
- request: { to: request.to!, data: request.data!, value: request.value?.toString() },
772
- l1BlockNumber: l1BlockNumber.toString(),
773
- error: { message: viemError.message, name: viemError.name },
774
- context: {
775
- actions: [`invalidate-${reason}`],
776
- checkpointNumber,
777
- sender: this.getSenderAddress().toString(),
1102
+ this.backupFailedTx(
1103
+ {
1104
+ id: keccak256(request.data!),
1105
+ failureType: 'simulation',
1106
+ request: { to: request.to!, data: request.data!, value: request.value },
1107
+ l1BlockNumber,
1108
+ error: { message: viemError.message, name: viemError.name },
1109
+ context: {
1110
+ actions: [`invalidate-${reason}`],
1111
+ checkpointNumber,
1112
+ sender: this.getSenderAddress().toString(),
1113
+ },
1114
+ timing: this.computeTimingInfo(this.getCurrentL2Slot()),
778
1115
  },
779
- });
1116
+ { captureFeeSummary: true, targetSlot: this.getCurrentL2Slot() },
1117
+ );
780
1118
  throw new Error(`Failed to simulate invalidate checkpoint ${checkpointNumber}`, { cause: viemError });
781
1119
  }
782
1120
  }
@@ -790,9 +1128,11 @@ export class SequencerPublisher {
790
1128
  const logData = { ...checkpoint, reason };
791
1129
  this.log.debug(`Building invalidate checkpoint ${checkpoint.checkpointNumber} request`, logData);
792
1130
 
793
- const attestationsAndSigners = new CommitteeAttestationsAndSigners(
794
- validationResult.attestations,
795
- ).getPackedAttestations();
1131
+ // Use the exact packed tuple posted to L1 verbatim. A repack via `packAttestations` is not a
1132
+ // byte-faithful inverse of `fromPacked` (a canonicalized yParity byte or an all-zero signature slot
1133
+ // round-trips differently), so it would diverge from the stored `attestationsHash` and revert the
1134
+ // invalidation.
1135
+ const attestationsAndSigners = validationResult.verbatimAttestations;
796
1136
 
797
1137
  if (reason === 'invalid-attestation') {
798
1138
  return this.rollupContract.buildInvalidateBadAttestationRequest(
@@ -813,40 +1153,11 @@ export class SequencerPublisher {
813
1153
  }
814
1154
  }
815
1155
 
816
- /** Simulates `propose` to make sure that the checkpoint is valid for submission */
817
- @trackSpan('SequencerPublisher.validateCheckpointForSubmission')
818
- public async validateCheckpointForSubmission(
819
- checkpoint: Checkpoint,
820
- attestationsAndSigners: CommitteeAttestationsAndSigners,
821
- attestationsAndSignersSignature: Signature,
822
- options: { forcePendingCheckpointNumber?: CheckpointNumber },
823
- ): Promise<void> {
824
- const blobFields = checkpoint.toBlobFields();
825
- const blobs = await getBlobsPerL1Block(blobFields);
826
- const blobInput = getPrefixedEthBlobCommitments(blobs);
827
-
828
- const args = [
829
- {
830
- header: checkpoint.header.toViem(),
831
- archive: toHex(checkpoint.archive.root.toBuffer()),
832
- oracleInput: {
833
- feeAssetPriceModifier: checkpoint.feeAssetPriceModifier,
834
- },
835
- },
836
- attestationsAndSigners.getPackedAttestations(),
837
- attestationsAndSigners.getSigners().map(signer => signer.toString()),
838
- attestationsAndSignersSignature.toViemSignature(),
839
- blobInput,
840
- ] as const;
841
-
842
- await this.simulateProposeTx(args, options);
843
- }
844
-
845
1156
  private async enqueueCastSignalHelper(
846
1157
  slotNumber: SlotNumber,
847
1158
  signalType: GovernanceSignalAction,
848
1159
  payload: EthAddress,
849
- base: IEmpireBase,
1160
+ base: GovernanceProposerContract,
850
1161
  signerAddress: EthAddress,
851
1162
  signer: (msg: TypedDataDefinition) => Promise<`0x${string}`>,
852
1163
  ): Promise<boolean> {
@@ -872,34 +1183,28 @@ export class SequencerPublisher {
872
1183
  return false;
873
1184
  }
874
1185
 
875
- if (await this.isPayloadEmpty(payload)) {
1186
+ if (await base.isPayloadEmpty(payload)) {
876
1187
  this.log.warn(`Skipping vote cast for payload with empty code`);
877
1188
  return false;
878
1189
  }
879
1190
 
880
- // Check if payload was already submitted to governance
881
- const cacheKey = payload.toString();
882
- if (!this.payloadProposedCache.has(cacheKey)) {
883
- try {
884
- const l1StartBlock = await this.rollupContract.getL1StartBlock();
885
- const proposed = await retry(
886
- () => base.hasPayloadBeenProposed(payload.toString(), l1StartBlock),
887
- 'Check if payload was proposed',
888
- makeBackoff([0, 1, 2]),
889
- this.log,
890
- true,
891
- );
892
- if (proposed) {
893
- this.payloadProposedCache.add(cacheKey);
894
- }
895
- } catch (err) {
896
- this.log.warn(`Failed to check if payload ${payload} was proposed after retries, skipping signal`, err);
897
- return false;
898
- }
1191
+ // Skip signaling if there is already a live (non-terminal) Governance proposal for this
1192
+ // payload. This is intentionally not cached: a previously-live proposal may transition to
1193
+ // a terminal state (Dropped/Rejected/Expired/Executed), at which point we may want to re-signal
1194
+ // the same payload in a future round.
1195
+ let proposed = false;
1196
+ try {
1197
+ proposed = await base.hasActiveProposalWithPayload(payload.toString());
1198
+ } catch (err) {
1199
+ // We deliberately swallow the error and proceed to signal. Failing closed (skipping the
1200
+ // signal) on transient RPC errors would let a flaky L1 endpoint silence governance
1201
+ // participation entirely; failing open at worst produces a duplicate signal that the
1202
+ // contract will simply count alongside others in the round.
1203
+ this.log.error(`Failed to check if payload ${payload} was already proposed (signalling anyway)`, err);
899
1204
  }
900
1205
 
901
- if (this.payloadProposedCache.has(cacheKey)) {
902
- this.log.info(`Payload ${payload} was already proposed to governance, stopping signals`);
1206
+ if (proposed) {
1207
+ this.log.info(`Payload ${payload} has a live governance proposal, stopping signals`);
903
1208
  return false;
904
1209
  }
905
1210
 
@@ -921,45 +1226,19 @@ export class SequencerPublisher {
921
1226
  lastValidL2Slot: slotNumber,
922
1227
  });
923
1228
 
924
- const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
925
- const timestamp = this.getSimulationTimestamp(slotNumber);
926
-
927
- try {
928
- await this.l1TxUtils.simulate(request, { time: timestamp }, [], mergeAbis([request.abi ?? [], ErrorsAbi]));
929
- this.log.debug(`Simulation for ${action} at slot ${slotNumber} succeeded`, { request });
930
- } catch (err) {
931
- const viemError = formatViemError(err);
932
- this.log.error(`Failed simulation for ${action} at slot ${slotNumber} (enqueuing the action anyway)`, viemError, {
933
- simulationTimestamp: timestamp,
934
- l1BlockNumber,
935
- });
936
- this.backupFailedTx({
937
- id: keccak256(request.data!),
938
- failureType: 'simulation',
939
- request: { to: request.to!, data: request.data!, value: request.value?.toString() },
940
- l1BlockNumber: l1BlockNumber.toString(),
941
- error: { message: viemError.message, name: viemError.name },
942
- context: {
943
- actions: [action],
944
- slot: slotNumber,
945
- sender: this.getSenderAddress().toString(),
946
- },
947
- });
948
- // Yes, we enqueue the request anyway, in case there was a bug with the simulation itself
949
- }
950
-
951
1229
  // TODO(palla/slash): All votes (governance and slashing) should txTimeoutAt at the end of the slot.
952
1230
  this.addRequest({
953
- gasConfig: { gasLimit: SequencerPublisher.VOTE_GAS_GUESS },
954
1231
  action,
955
1232
  request,
956
1233
  lastValidL2Slot: slotNumber,
957
1234
  checkSuccess: (_request, result) => {
958
1235
  const success =
959
1236
  result &&
960
- result.receipt &&
961
- result.receipt.status === 'success' &&
962
- tryExtractEvent(result.receipt.logs, base.address.toString(), EmpireBaseAbi, 'SignalCast');
1237
+ extractEventSuccess(result.receipt, {
1238
+ address: base.address.toString(),
1239
+ abi: EmpireBaseAbi,
1240
+ eventName: 'SignalCast',
1241
+ });
963
1242
 
964
1243
  const logData = { ...result, slotNumber, round, payload: payload.toString() };
965
1244
  if (!success) {
@@ -981,17 +1260,6 @@ export class SequencerPublisher {
981
1260
  return true;
982
1261
  }
983
1262
 
984
- private async isPayloadEmpty(payload: EthAddress): Promise<boolean> {
985
- const key = payload.toString();
986
- const cached = this.isPayloadEmptyCache.get(key);
987
- if (cached) {
988
- return cached;
989
- }
990
- const isEmpty = !(await this.l1TxUtils.getCode(payload));
991
- this.isPayloadEmptyCache.set(key, isEmpty);
992
- return isEmpty;
993
- }
994
-
995
1263
  /**
996
1264
  * Enqueues a governance castSignal transaction to cast a signal for a given slot number.
997
1265
  * @param slotNumber - The slot number to cast a signal for.
@@ -1013,6 +1281,47 @@ export class SequencerPublisher {
1013
1281
  );
1014
1282
  }
1015
1283
 
1284
+ /**
1285
+ * Enqueues a `prune()` transaction if the rollup is prunable at the given slot's L1 timestamp.
1286
+ * `prune()` is permissionless and idempotent — if the chain is no longer prunable by send time the
1287
+ * bundle simulation usually drops the entry; on a node without `eth_simulateV1` the bundle is sent
1288
+ * as-is and the prune reverts `Rollup__NothingToPrune` inside `aggregate3(allowFailure: true)`
1289
+ * (a failed action, never a whole-tx revert). Used by the failed-sync fallback so a stuck pending
1290
+ * chain (e.g. bad data blocking sync) can be wound back to recover.
1291
+ * @returns true if a prune request was enqueued, false otherwise.
1292
+ */
1293
+ public async enqueuePruneIfPrunable(slotNumber: SlotNumber): Promise<boolean> {
1294
+ if (this.lastActions['prune'] === slotNumber) {
1295
+ this.log.debug(`Skipping duplicate prune for slot ${slotNumber}`, { slotNumber });
1296
+ return false;
1297
+ }
1298
+ // Use the SAME timestamp the bundle simulator overrides block.timestamp with at send time
1299
+ // (sequencer-bundle-simulator.ts) so this upfront check and the send-time sim agree. Slot-start
1300
+ // and last-L1-slot both fall within the same L2 slot (and epoch, which is what `canPruneAtTime`
1301
+ // derives), so they agree today; matching the simulator keeps it robust if the contract ever uses
1302
+ // the timestamp more granularly.
1303
+ const ts = getLastL1SlotTimestampForL2Slot(slotNumber, this.epochCache.getL1Constants());
1304
+ const canPrune = await this.rollupContract.canPruneAtTime(ts).catch(err => {
1305
+ this.log.error(`Failed to check canPruneAtTime for slot ${slotNumber}`, err, { slotNumber });
1306
+ return false;
1307
+ });
1308
+ if (!canPrune) {
1309
+ this.log.debug(`Rollup not prunable at slot ${slotNumber}`, { slotNumber });
1310
+ return false;
1311
+ }
1312
+ const request: L1TxRequest = {
1313
+ to: this.rollupContract.address,
1314
+ data: encodeFunctionData({ abi: RollupAbi, functionName: 'prune', args: [] }),
1315
+ };
1316
+ this.log.info(`Enqueuing rollup prune for slot ${slotNumber}`, { slotNumber });
1317
+ return this.enqueueRequest(
1318
+ 'prune',
1319
+ request,
1320
+ { address: this.rollupContract.address, abi: RollupAbi, eventName: 'PrunedPending' },
1321
+ slotNumber,
1322
+ );
1323
+ }
1324
+
1016
1325
  /** Enqueues all slashing actions as returned by the slasher client. */
1017
1326
  public async enqueueSlashingActions(
1018
1327
  actions: ProposerSlashAction[],
@@ -1027,55 +1336,6 @@ export class SequencerPublisher {
1027
1336
 
1028
1337
  for (const action of actions) {
1029
1338
  switch (action.type) {
1030
- case 'vote-empire-payload': {
1031
- if (this.slashingProposerContract?.type !== 'empire') {
1032
- this.log.error('Cannot vote for empire payload on non-empire slashing contract');
1033
- break;
1034
- }
1035
- this.log.debug(`Enqueuing slashing vote for payload ${action.payload} at slot ${slotNumber}`, {
1036
- signerAddress,
1037
- });
1038
- await this.enqueueCastSignalHelper(
1039
- slotNumber,
1040
- 'empire-slashing-signal',
1041
- action.payload,
1042
- this.slashingProposerContract,
1043
- signerAddress,
1044
- signer,
1045
- );
1046
- break;
1047
- }
1048
-
1049
- case 'create-empire-payload': {
1050
- this.log.debug(`Enqueuing slashing create payload at slot ${slotNumber}`, { slotNumber, signerAddress });
1051
- const request = this.slashFactoryContract.buildCreatePayloadRequest(action.data);
1052
- await this.simulateAndEnqueueRequest(
1053
- 'create-empire-payload',
1054
- request,
1055
- (receipt: TransactionReceipt) =>
1056
- !!this.slashFactoryContract.tryExtractSlashPayloadCreatedEvent(receipt.logs),
1057
- slotNumber,
1058
- );
1059
- break;
1060
- }
1061
-
1062
- case 'execute-empire-payload': {
1063
- this.log.debug(`Enqueuing slashing execute payload at slot ${slotNumber}`, { slotNumber, signerAddress });
1064
- if (this.slashingProposerContract?.type !== 'empire') {
1065
- this.log.error('Cannot execute slashing payload on non-empire slashing contract');
1066
- return false;
1067
- }
1068
- const empireSlashingProposer = this.slashingProposerContract as EmpireSlashingProposerContract;
1069
- const request = empireSlashingProposer.buildExecuteRoundRequest(action.round);
1070
- await this.simulateAndEnqueueRequest(
1071
- 'execute-empire-payload',
1072
- request,
1073
- (receipt: TransactionReceipt) => !!empireSlashingProposer.tryExtractPayloadSubmittedEvent(receipt.logs),
1074
- slotNumber,
1075
- );
1076
- break;
1077
- }
1078
-
1079
1339
  case 'vote-offenses': {
1080
1340
  this.log.debug(`Enqueuing slashing vote for ${action.votes.length} votes at slot ${slotNumber}`, {
1081
1341
  slotNumber,
@@ -1083,17 +1343,20 @@ export class SequencerPublisher {
1083
1343
  votesCount: action.votes.length,
1084
1344
  signerAddress,
1085
1345
  });
1086
- if (this.slashingProposerContract?.type !== 'tally') {
1087
- this.log.error('Cannot vote for slashing offenses on non-tally slashing contract');
1346
+ if (!this.slashingProposerContract) {
1347
+ this.log.error('No slashing proposer contract available');
1088
1348
  return false;
1089
1349
  }
1090
- const tallySlashingProposer = this.slashingProposerContract as TallySlashingProposerContract;
1091
1350
  const votes = bufferToHex(encodeSlashConsensusVotes(action.votes));
1092
- const request = await tallySlashingProposer.buildVoteRequestFromSigner(votes, slotNumber, signer);
1093
- await this.simulateAndEnqueueRequest(
1351
+ const request = await this.slashingProposerContract.buildVoteRequestFromSigner(votes, slotNumber, signer);
1352
+ this.enqueueRequest(
1094
1353
  'vote-offenses',
1095
1354
  request,
1096
- (receipt: TransactionReceipt) => !!tallySlashingProposer.tryExtractVoteCastEvent(receipt.logs),
1355
+ {
1356
+ address: this.slashingProposerContract.address.toString(),
1357
+ abi: SlashingProposerAbi,
1358
+ eventName: 'VoteCast',
1359
+ },
1097
1360
  slotNumber,
1098
1361
  );
1099
1362
  break;
@@ -1105,16 +1368,22 @@ export class SequencerPublisher {
1105
1368
  round: action.round,
1106
1369
  signerAddress,
1107
1370
  });
1108
- if (this.slashingProposerContract?.type !== 'tally') {
1109
- this.log.error('Cannot execute slashing offenses on non-tally slashing contract');
1371
+ if (!this.slashingProposerContract) {
1372
+ this.log.error('No slashing proposer contract available');
1110
1373
  return false;
1111
1374
  }
1112
- const tallySlashingProposer = this.slashingProposerContract as TallySlashingProposerContract;
1113
- const request = tallySlashingProposer.buildExecuteRoundRequest(action.round, action.committees);
1114
- await this.simulateAndEnqueueRequest(
1375
+ const executeRequest = this.slashingProposerContract.buildExecuteRoundRequest(
1376
+ action.round,
1377
+ action.committees,
1378
+ );
1379
+ this.enqueueRequest(
1115
1380
  'execute-slash',
1116
- request,
1117
- (receipt: TransactionReceipt) => !!tallySlashingProposer.tryExtractRoundExecutedEvent(receipt.logs),
1381
+ executeRequest,
1382
+ {
1383
+ address: this.slashingProposerContract.address.toString(),
1384
+ abi: SlashingProposerAbi,
1385
+ eventName: 'RoundExecuted',
1386
+ },
1118
1387
  slotNumber,
1119
1388
  );
1120
1389
  break;
@@ -1130,12 +1399,12 @@ export class SequencerPublisher {
1130
1399
  return true;
1131
1400
  }
1132
1401
 
1133
- /** Simulates and enqueues a proposal for a checkpoint on L1 */
1402
+ /** Enqueues a proposal for a checkpoint on L1 */
1134
1403
  public async enqueueProposeCheckpoint(
1135
1404
  checkpoint: Checkpoint,
1136
1405
  attestationsAndSigners: CommitteeAttestationsAndSigners,
1137
1406
  attestationsAndSignersSignature: Signature,
1138
- opts: { txTimeoutAt?: Date; forcePendingCheckpointNumber?: CheckpointNumber } = {},
1407
+ opts: EnqueueProposeCheckpointOpts = {},
1139
1408
  ): Promise<void> {
1140
1409
  const checkpointHeader = checkpoint.header;
1141
1410
 
@@ -1151,29 +1420,11 @@ export class SequencerPublisher {
1151
1420
  feeAssetPriceModifier: checkpoint.feeAssetPriceModifier,
1152
1421
  };
1153
1422
 
1154
- try {
1155
- // @note This will make sure that we are passing the checks for our header ASSUMING that the data is also made available
1156
- // This means that we can avoid the simulation issues in later checks.
1157
- // By simulation issue, I mean the fact that the block.timestamp is equal to the last block, not the next, which
1158
- // make time consistency checks break.
1159
- // TODO(palla): Check whether we're validating twice, once here and once within addProposeTx, since we call simulateProposeTx in both places.
1160
- await this.validateCheckpointForSubmission(
1161
- checkpoint,
1162
- attestationsAndSigners,
1163
- attestationsAndSignersSignature,
1164
- opts,
1165
- );
1166
- } catch (err: any) {
1167
- this.log.error(`Checkpoint validation failed. ${err instanceof Error ? err.message : 'No error message'}`, err, {
1168
- ...checkpoint.getStats(),
1169
- slotNumber: checkpoint.header.slotNumber,
1170
- forcePendingCheckpointNumber: opts.forcePendingCheckpointNumber,
1171
- });
1172
- throw err;
1173
- }
1174
-
1175
- this.log.verbose(`Enqueuing checkpoint propose transaction`, { ...checkpoint.toCheckpointInfo(), ...opts });
1176
- await this.addProposeTx(checkpoint, proposeTxArgs, opts);
1423
+ this.log.verbose(`Enqueuing checkpoint propose transaction`, {
1424
+ ...checkpoint.toCheckpointInfo(),
1425
+ txTimeoutAt: opts.txTimeoutAt,
1426
+ });
1427
+ await this.addProposeTx(checkpoint, proposeTxArgs, { txTimeoutAt: opts.txTimeoutAt });
1177
1428
  }
1178
1429
 
1179
1430
  public enqueueInvalidateCheckpoint(
@@ -1184,23 +1435,22 @@ export class SequencerPublisher {
1184
1435
  return;
1185
1436
  }
1186
1437
 
1187
- // We issued the simulation against the rollup contract, so we need to account for the overhead of the multicall3
1188
- const gasLimit = this.l1TxUtils.bumpGasLimit(BigInt(Math.ceil((Number(request.gasUsed) * 64) / 63)));
1189
-
1190
1438
  const { gasUsed, checkpointNumber } = request;
1191
- const logData = { gasUsed, checkpointNumber, gasLimit, opts };
1439
+ const logData = { gasUsed, checkpointNumber, opts };
1192
1440
  this.log.verbose(`Enqueuing invalidate checkpoint request`, logData);
1193
1441
  this.addRequest({
1194
1442
  action: `invalidate-by-${request.reason}`,
1195
1443
  request: request.request,
1196
- gasConfig: { gasLimit, txTimeoutAt: opts.txTimeoutAt },
1444
+ gasConfig: opts.txTimeoutAt ? { txTimeoutAt: opts.txTimeoutAt } : undefined,
1197
1445
  lastValidL2Slot: SlotNumber(this.getCurrentL2Slot() + 2),
1198
1446
  checkSuccess: (_req, result) => {
1199
1447
  const success =
1200
1448
  result &&
1201
- result.receipt &&
1202
- result.receipt.status === 'success' &&
1203
- tryExtractEvent(result.receipt.logs, this.rollupContract.address, RollupAbi, 'CheckpointInvalidated');
1449
+ extractEventSuccess(result.receipt, {
1450
+ address: this.rollupContract.address,
1451
+ abi: RollupAbi,
1452
+ eventName: 'CheckpointInvalidated',
1453
+ });
1204
1454
  if (!success) {
1205
1455
  this.log.warn(`Invalidate checkpoint ${request.checkpointNumber} failed`, { ...result, ...logData });
1206
1456
  } else {
@@ -1211,73 +1461,36 @@ export class SequencerPublisher {
1211
1461
  });
1212
1462
  }
1213
1463
 
1214
- private async simulateAndEnqueueRequest(
1464
+ /**
1465
+ * Dedup-checked enqueue helper for actions that are simulated at bundle-send time rather
1466
+ * than at enqueue time. Validates the (action, slot) dedup key, sets `lastActions`, and
1467
+ * enqueues without a gasLimit so the bundle simulate sets the only gasLimit that matters.
1468
+ */
1469
+ private enqueueRequest(
1215
1470
  action: Action,
1216
1471
  request: L1TxRequest,
1217
- checkSuccess: (receipt: TransactionReceipt) => boolean | undefined,
1472
+ eventOpts: { address: string; abi: Abi; eventName: string },
1218
1473
  slotNumber: SlotNumber,
1219
- ) {
1220
- const timestamp = this.getSimulationTimestamp(slotNumber);
1221
- const logData = { slotNumber, timestamp, gasLimit: undefined as bigint | undefined };
1474
+ ): boolean {
1222
1475
  if (this.lastActions[action] && this.lastActions[action] === slotNumber) {
1223
1476
  this.log.debug(`Skipping duplicate action ${action} for slot ${slotNumber}`);
1224
1477
  return false;
1225
1478
  }
1226
-
1227
1479
  const cachedLastActionSlot = this.lastActions[action];
1228
1480
  this.lastActions[action] = slotNumber;
1229
1481
 
1230
- this.log.debug(`Simulating ${action} for slot ${slotNumber}`, logData);
1231
-
1232
- const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
1233
-
1234
- let gasUsed: bigint;
1235
- const simulateAbi = mergeAbis([request.abi ?? [], ErrorsAbi]);
1236
-
1237
- try {
1238
- ({ gasUsed } = await this.l1TxUtils.simulate(request, { time: timestamp }, [], simulateAbi));
1239
- this.log.verbose(`Simulation for ${action} succeeded`, { ...logData, request, gasUsed });
1240
- } catch (err) {
1241
- const viemError = formatViemError(err, simulateAbi);
1242
- this.log.error(`Simulation for ${action} at ${slotNumber} failed`, viemError, logData);
1243
-
1244
- this.backupFailedTx({
1245
- id: keccak256(request.data!),
1246
- failureType: 'simulation',
1247
- request: { to: request.to!, data: request.data!, value: request.value?.toString() },
1248
- l1BlockNumber: l1BlockNumber.toString(),
1249
- error: { message: viemError.message, name: viemError.name },
1250
- context: {
1251
- actions: [action],
1252
- slot: slotNumber,
1253
- sender: this.getSenderAddress().toString(),
1254
- },
1255
- });
1256
-
1257
- return false;
1258
- }
1259
-
1260
- // We issued the simulation against the rollup contract, so we need to account for the overhead of the multicall3
1261
- const gasLimit = this.l1TxUtils.bumpGasLimit(BigInt(Math.ceil((Number(gasUsed) * 64) / 63)));
1262
- logData.gasLimit = gasLimit;
1263
-
1264
- // Store the ABI used for simulation on the request so Multicall3.forward can decode errors
1265
- // when the tx is sent and a revert is diagnosed via simulation.
1266
- const requestWithAbi = { ...request, abi: simulateAbi };
1267
-
1268
- this.log.debug(`Enqueuing ${action}`, logData);
1482
+ this.log.debug(`Enqueuing ${action}`, { slotNumber });
1269
1483
  this.addRequest({
1270
1484
  action,
1271
- request: requestWithAbi,
1272
- gasConfig: { gasLimit },
1485
+ request,
1273
1486
  lastValidL2Slot: slotNumber,
1274
- checkSuccess: (_req, result) => {
1275
- const success = result && result.receipt && result.receipt.status === 'success' && checkSuccess(result.receipt);
1487
+ checkSuccess: (_request, result) => {
1488
+ const success = result && extractEventSuccess(result.receipt, eventOpts);
1276
1489
  if (!success) {
1277
- this.log.warn(`Action ${action} at ${slotNumber} failed`, { ...result, ...logData });
1490
+ this.log.warn(`Action ${action} at ${slotNumber} failed`, { ...result, slotNumber });
1278
1491
  this.lastActions[action] = cachedLastActionSlot;
1279
1492
  } else {
1280
- this.log.info(`Action ${action} at ${slotNumber} succeeded`, { ...result, ...logData });
1493
+ this.log.info(`Action ${action} at ${slotNumber} succeeded`, { ...result, slotNumber });
1281
1494
  }
1282
1495
  return !!success;
1283
1496
  },
@@ -1293,6 +1506,7 @@ export class SequencerPublisher {
1293
1506
  */
1294
1507
  public interrupt() {
1295
1508
  this.interrupted = true;
1509
+ this.interruptibleSleep.interrupt();
1296
1510
  this.l1TxUtils.interrupt();
1297
1511
  }
1298
1512
 
@@ -1302,10 +1516,7 @@ export class SequencerPublisher {
1302
1516
  this.l1TxUtils.restart();
1303
1517
  }
1304
1518
 
1305
- private async prepareProposeTx(
1306
- encodedData: L1ProcessArgs,
1307
- options: { forcePendingCheckpointNumber?: CheckpointNumber },
1308
- ) {
1519
+ private async prepareProposeTx(encodedData: L1ProcessArgs) {
1309
1520
  const kzg = Blob.getViemKzgInstance();
1310
1521
  const blobInput = getPrefixedEthBlobCommitments(encodedData.blobs);
1311
1522
  this.log.debug('Validating blob input', { blobInput });
@@ -1318,7 +1529,11 @@ export class SequencerPublisher {
1318
1529
  blobEvaluationGas = BigInt(encodedData.blobs.length) * 21_000n;
1319
1530
  this.log.debug(`Using fixed blob evaluation gas estimate in fisherman mode: ${blobEvaluationGas}`);
1320
1531
  } else {
1321
- // Normal mode - use estimateGas with blob inputs
1532
+ // We call validateBlobs via estimateGas with real blob+kzg sidecars as a consistency check
1533
+ // that our locally-built blob commitments match the blob data. The bundle simulate at send
1534
+ // time uses eth_simulateV1, which cannot carry blob inputs, so the rollup's on-chain blob
1535
+ // check is forced off there — making this the only pre-flight detector of a commitment/data
1536
+ // mismatch. The returned gas estimate is stashed on the request for the bundle path to read.
1322
1537
  blobEvaluationGas = await this.l1TxUtils
1323
1538
  .estimateGas(
1324
1539
  this.getSenderAddress().toString(),
@@ -1345,18 +1560,22 @@ export class SequencerPublisher {
1345
1560
  args: [blobInput],
1346
1561
  });
1347
1562
  const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
1348
- this.backupFailedTx({
1349
- id: keccak256(validateBlobsData),
1350
- failureType: 'simulation',
1351
- request: { to: this.rollupContract.address as Hex, data: validateBlobsData },
1352
- blobData: encodedData.blobs.map(b => toHex(b.data)) as Hex[],
1353
- l1BlockNumber: l1BlockNumber.toString(),
1354
- error: { message: viemError.message, name: viemError.name },
1355
- context: {
1356
- actions: ['validate-blobs'],
1357
- sender: this.getSenderAddress().toString(),
1563
+ this.backupFailedTx(
1564
+ {
1565
+ id: keccak256(validateBlobsData),
1566
+ failureType: 'simulation',
1567
+ request: { to: this.rollupContract.address as Hex, data: validateBlobsData },
1568
+ blobData: encodedData.blobs.map(b => toHex(b.data)) as Hex[],
1569
+ l1BlockNumber,
1570
+ error: { message: viemError.message, name: viemError.name },
1571
+ context: {
1572
+ actions: ['validate-blobs'],
1573
+ sender: this.getSenderAddress().toString(),
1574
+ },
1575
+ timing: this.computeTimingInfo(this.getCurrentL2Slot()),
1358
1576
  },
1359
- });
1577
+ { captureFeeSummary: true, targetSlot: this.getCurrentL2Slot() },
1578
+ );
1360
1579
  throw new Error('Failed to validate blobs');
1361
1580
  });
1362
1581
  }
@@ -1376,131 +1595,21 @@ export class SequencerPublisher {
1376
1595
  blobInput,
1377
1596
  ] as const;
1378
1597
 
1379
- const { rollupData, simulationResult } = await this.simulateProposeTx(args, options);
1598
+ const rollupData = encodeFunctionData({ abi: RollupAbi, functionName: 'propose', args });
1380
1599
 
1381
- return { args, blobEvaluationGas, rollupData, simulationResult };
1382
- }
1383
-
1384
- /**
1385
- * Simulates the propose tx with eth_simulateV1
1386
- * @param args - The propose tx args
1387
- * @returns The simulation result
1388
- */
1389
- private async simulateProposeTx(
1390
- args: readonly [
1391
- {
1392
- readonly header: ViemHeader;
1393
- readonly archive: `0x${string}`;
1394
- readonly oracleInput: {
1395
- readonly feeAssetPriceModifier: bigint;
1396
- };
1397
- },
1398
- ViemCommitteeAttestations,
1399
- `0x${string}`[], // Signers
1400
- ViemSignature,
1401
- `0x${string}`,
1402
- ],
1403
- options: { forcePendingCheckpointNumber?: CheckpointNumber },
1404
- ) {
1405
- const rollupData = encodeFunctionData({
1406
- abi: RollupAbi,
1407
- functionName: 'propose',
1408
- args,
1409
- });
1410
-
1411
- // override the pending checkpoint number if requested
1412
- const forcePendingCheckpointNumberStateDiff = (
1413
- options.forcePendingCheckpointNumber !== undefined
1414
- ? await this.rollupContract.makePendingCheckpointNumberOverride(options.forcePendingCheckpointNumber)
1415
- : []
1416
- ).flatMap(override => override.stateDiff ?? []);
1417
-
1418
- const stateOverrides: StateOverride = [
1419
- {
1420
- address: this.rollupContract.address,
1421
- // @note we override checkBlob to false since blobs are not part simulate()
1422
- stateDiff: [
1423
- { slot: toPaddedHex(RollupContract.checkBlobStorageSlot, true), value: toPaddedHex(0n, true) },
1424
- ...forcePendingCheckpointNumberStateDiff,
1425
- ],
1426
- },
1427
- ];
1428
- // In fisherman mode, simulate as the proposer but with sufficient balance
1429
- if (this.proposerAddressForSimulation) {
1430
- stateOverrides.push({
1431
- address: this.proposerAddressForSimulation.toString(),
1432
- balance: 10n * WEI_CONST * WEI_CONST, // 10 ETH
1433
- });
1434
- }
1435
-
1436
- const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
1437
- const simTs = this.getSimulationTimestamp(SlotNumber.fromBigInt(args[0].header.slotNumber));
1438
-
1439
- const simulationResult = await this.l1TxUtils
1440
- .simulate(
1441
- {
1442
- to: this.rollupContract.address,
1443
- data: rollupData,
1444
- gas: MAX_L1_TX_LIMIT,
1445
- ...(this.proposerAddressForSimulation && { from: this.proposerAddressForSimulation.toString() }),
1446
- },
1447
- {
1448
- time: simTs,
1449
- // @note reth should have a 30m gas limit per block but throws errors that this tx is beyond limit so we increase here
1450
- gasLimit: MAX_L1_TX_LIMIT * 2n,
1451
- },
1452
- stateOverrides,
1453
- RollupAbi,
1454
- {
1455
- // @note fallback gas estimate to use if the node doesn't support simulation API
1456
- fallbackGasEstimate: MAX_L1_TX_LIMIT,
1457
- },
1458
- )
1459
- .catch(err => {
1460
- // In fisherman mode, we expect ValidatorSelection__MissingProposerSignature since fisherman doesn't have proposer signature
1461
- const viemError = formatViemError(err);
1462
- if (this.config.fishermanMode && viemError.message?.includes('ValidatorSelection__MissingProposerSignature')) {
1463
- this.log.debug(`Ignoring expected ValidatorSelection__MissingProposerSignature error in fisherman mode`);
1464
- // Return a minimal simulation result with the fallback gas estimate
1465
- return {
1466
- gasUsed: MAX_L1_TX_LIMIT,
1467
- logs: [],
1468
- };
1469
- }
1470
- this.log.error(`Failed to simulate propose tx`, viemError, { simulationTimestamp: simTs });
1471
- this.backupFailedTx({
1472
- id: keccak256(rollupData),
1473
- failureType: 'simulation',
1474
- request: { to: this.rollupContract.address, data: rollupData },
1475
- l1BlockNumber: l1BlockNumber.toString(),
1476
- error: { message: viemError.message, name: viemError.name },
1477
- context: {
1478
- actions: ['propose'],
1479
- slot: Number(args[0].header.slotNumber),
1480
- sender: this.getSenderAddress().toString(),
1481
- },
1482
- });
1483
- throw err;
1484
- });
1485
-
1486
- return { rollupData, simulationResult };
1600
+ return { args, blobEvaluationGas, rollupData };
1487
1601
  }
1488
1602
 
1489
1603
  private async addProposeTx(
1490
1604
  checkpoint: Checkpoint,
1491
1605
  encodedData: L1ProcessArgs,
1492
- opts: { txTimeoutAt?: Date; forcePendingCheckpointNumber?: CheckpointNumber } = {},
1606
+ opts: EnqueueProposeCheckpointOpts = {},
1493
1607
  ): Promise<void> {
1494
1608
  const slot = checkpoint.header.slotNumber;
1495
1609
  const timer = new Timer();
1496
1610
  const kzg = Blob.getViemKzgInstance();
1497
- const { rollupData, simulationResult, blobEvaluationGas } = await this.prepareProposeTx(encodedData, opts);
1611
+ const { rollupData, blobEvaluationGas } = await this.prepareProposeTx(encodedData);
1498
1612
  const startBlock = await this.l1TxUtils.getBlockNumber();
1499
- const gasLimit = this.l1TxUtils.bumpGasLimit(
1500
- BigInt(Math.ceil((Number(simulationResult.gasUsed) * 64) / 63)) +
1501
- blobEvaluationGas +
1502
- SequencerPublisher.MULTICALL_OVERHEAD_GAS_GUESS, // We issue the simulation against the rollup contract, so we need to account for the overhead of the multicall3
1503
- );
1504
1613
 
1505
1614
  // Send the blobs to the blob client preemptively. This helps in tests where the sequencer mistakingly thinks that the propose
1506
1615
  // tx fails but it does get mined. We make sure that the blobs are sent to the blob client regardless of the tx outcome.
@@ -1517,7 +1626,8 @@ export class SequencerPublisher {
1517
1626
  data: rollupData,
1518
1627
  },
1519
1628
  lastValidL2Slot: checkpoint.header.slotNumber,
1520
- gasConfig: { ...opts, gasLimit },
1629
+ gasConfig: { txTimeoutAt: opts.txTimeoutAt, gasLimit: undefined },
1630
+ blobEvaluationGas,
1521
1631
  blobConfig: {
1522
1632
  blobs: encodedData.blobs.map(b => b.data),
1523
1633
  kzg,
@@ -1527,10 +1637,11 @@ export class SequencerPublisher {
1527
1637
  return false;
1528
1638
  }
1529
1639
  const { receipt, stats, errorMsg } = result;
1530
- const success =
1531
- receipt &&
1532
- receipt.status === 'success' &&
1533
- tryExtractEvent(receipt.logs, this.rollupContract.address, RollupAbi, 'CheckpointProposed');
1640
+ const success = extractEventSuccess(receipt, {
1641
+ address: this.rollupContract.address,
1642
+ abi: RollupAbi,
1643
+ eventName: 'CheckpointProposed',
1644
+ });
1534
1645
 
1535
1646
  if (success) {
1536
1647
  const endBlock = receipt.blockNumber;
@@ -1571,13 +1682,6 @@ export class SequencerPublisher {
1571
1682
  });
1572
1683
  }
1573
1684
 
1574
- /** Returns the timestamp of the last L1 slot within a given L2 slot. Used as the simulation timestamp
1575
- * for eth_simulateV1 calls, since it's guaranteed to be greater than any L1 block produced during the slot. */
1576
- private getSimulationTimestamp(slot: SlotNumber): bigint {
1577
- const l1Constants = this.epochCache.getL1Constants();
1578
- return getLastL1SlotTimestampForL2Slot(slot, l1Constants);
1579
- }
1580
-
1581
1685
  /** Returns the timestamp of the next L1 slot boundary after now. */
1582
1686
  private getNextL1SlotTimestamp(): bigint {
1583
1687
  const l1Constants = this.epochCache.getL1Constants();