@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
@@ -370,14 +370,12 @@ function applyDecs2203RFactory() {
370
370
  function _apply_decs_2203_r(targetClass, memberDecs, classDecs, parentClass) {
371
371
  return (_apply_decs_2203_r = applyDecs2203RFactory())(targetClass, memberDecs, classDecs, parentClass);
372
372
  }
373
- var _dec, _dec1, _dec2, _initProto;
373
+ var _dec, _dec1, _initProto;
374
374
  import { Blob, getBlobsPerL1Block, getPrefixedEthBlobCommitments } from '@aztec/blob-lib';
375
- import { FeeAssetPriceOracle, MULTI_CALL_3_ADDRESS, Multicall3, RollupContract } from '@aztec/ethereum/contracts';
376
- import { L1FeeAnalyzer } from '@aztec/ethereum/l1-fee-analysis';
377
- import { MAX_L1_TX_LIMIT, WEI_CONST } from '@aztec/ethereum/l1-tx-utils';
378
- import { FormattedViemError, formatViemError, mergeAbis, tryExtractEvent } from '@aztec/ethereum/utils';
379
- import { sumBigint } from '@aztec/foundation/bigint';
380
- import { toHex as toPaddedHex } from '@aztec/foundation/bigint-buffer';
375
+ import { FeeAssetPriceOracle, MULTI_CALL_3_ADDRESS, Multicall3, MulticallForwarderRevertedError, buildSimulationOverridesStateOverride } from '@aztec/ethereum/contracts';
376
+ import { L1FeeAnalyzer, captureWindowBlockFees } from '@aztec/ethereum/l1-fee-analysis';
377
+ import { L1TxTimeoutError, MAX_L1_TX_LIMIT, WEI_CONST } from '@aztec/ethereum/l1-tx-utils';
378
+ import { FormattedViemError, formatViemError, mergeAbis, tryDecodeRevertReason, tryExtractEvent } from '@aztec/ethereum/utils';
381
379
  import { CheckpointNumber, SlotNumber } from '@aztec/foundation/branded-types';
382
380
  import { trimmedBytesLength } from '@aztec/foundation/buffer';
383
381
  import { pick } from '@aztec/foundation/collection';
@@ -385,32 +383,40 @@ import { TimeoutError } from '@aztec/foundation/error';
385
383
  import { EthAddress } from '@aztec/foundation/eth-address';
386
384
  import { Signature } from '@aztec/foundation/eth-signature';
387
385
  import { createLogger } from '@aztec/foundation/log';
388
- import { makeBackoff, retry } from '@aztec/foundation/retry';
389
386
  import { InterruptibleSleep } from '@aztec/foundation/sleep';
390
387
  import { bufferToHex } from '@aztec/foundation/string';
391
388
  import { Timer } from '@aztec/foundation/timer';
392
- import { EmpireBaseAbi, ErrorsAbi, RollupAbi } from '@aztec/l1-artifacts';
389
+ import { EmpireBaseAbi, ErrorsAbi, RollupAbi, SlashingProposerAbi } from '@aztec/l1-artifacts';
393
390
  import { encodeSlashConsensusVotes } from '@aztec/slasher';
394
391
  import { CommitteeAttestationsAndSigners } from '@aztec/stdlib/block';
395
- import { getLastL1SlotTimestampForL2Slot, getNextL1SlotTimestamp } from '@aztec/stdlib/epoch-helpers';
392
+ import { getLastL1SlotTimestampForL2Slot, getNextL1SlotTimestamp, getTimestampForSlot } from '@aztec/stdlib/epoch-helpers';
396
393
  import { getTelemetryClient, trackSpan } from '@aztec/telemetry-client';
397
- import { encodeFunctionData, keccak256, multicall3Abi, toHex } from 'viem';
394
+ import { encodeFunctionData, keccak256, toHex } from 'viem';
398
395
  import { createL1TxFailedStore } from './l1_tx_failed_store/index.js';
396
+ import { SequencerBundleSimulator } from './sequencer-bundle-simulator.js';
399
397
  import { SequencerPublisherMetrics } from './sequencer-publisher-metrics.js';
398
+ /**
399
+ * Returns true if the receipt indicates a successful send AND the expected event was emitted
400
+ * by the target contract. Both pieces are required: an aggregate3 entry that reverted will
401
+ * have receipt.status === 'success' but no event log.
402
+ */ function extractEventSuccess(receipt, opts) {
403
+ if (!receipt || receipt.status !== 'success') {
404
+ return false;
405
+ }
406
+ return !!tryExtractEvent(receipt.logs, opts.address.toString(), opts.abi, opts.eventName);
407
+ }
400
408
  export const Actions = [
401
409
  'invalidate-by-invalid-attestation',
402
410
  'invalidate-by-insufficient-attestations',
411
+ 'prune',
403
412
  'propose',
404
413
  'governance-signal',
405
- 'empire-slashing-signal',
406
- 'create-empire-payload',
407
- 'execute-empire-payload',
408
414
  'vote-offenses',
409
415
  'execute-slash'
410
416
  ];
411
417
  // Sorting for actions such that invalidations go before proposals, and proposals go before votes
412
418
  export const compareActions = (a, b)=>Actions.indexOf(a) - Actions.indexOf(b);
413
- _dec = trackSpan('SequencerPublisher.sendRequests'), _dec1 = trackSpan('SequencerPublisher.validateBlockHeader'), _dec2 = trackSpan('SequencerPublisher.validateCheckpointForSubmission');
419
+ _dec = trackSpan('SequencerPublisher.sendRequests'), _dec1 = trackSpan('SequencerPublisher.validateBlockHeader');
414
420
  export class SequencerPublisher {
415
421
  config;
416
422
  static{
@@ -424,57 +430,54 @@ export class SequencerPublisher {
424
430
  _dec1,
425
431
  2,
426
432
  "validateBlockHeader"
427
- ],
428
- [
429
- _dec2,
430
- 2,
431
- "validateCheckpointForSubmission"
432
433
  ]
433
434
  ], []));
434
435
  }
435
436
  interrupted;
436
437
  metrics;
438
+ bundleSimulator;
437
439
  epochCache;
438
440
  failedTxStore;
439
- governanceLog;
440
- slashingLog;
441
+ /**
442
+ * ABI used to decode raw revert payloads from dropped bundle entries when the original
443
+ * request did not carry an abi (e.g. the propose request). Merges every contract the
444
+ * publisher can route to so any of their custom errors decode against it.
445
+ */ revertDecoderAbi;
441
446
  lastActions;
442
- isPayloadEmptyCache;
443
- payloadProposedCache;
444
447
  log;
445
448
  ethereumSlotDuration;
446
449
  aztecSlotDuration;
450
+ previousL1BlockWaitTimeoutMs;
451
+ previousL1BlockWaitPollIntervalMs;
447
452
  /** Date provider for wall-clock time. */ dateProvider;
448
453
  blobClient;
449
- /** Address to use for simulations in fisherman mode (actual proposer's address) */ proposerAddressForSimulation;
450
454
  /** Optional callback to obtain a replacement publisher when the current one fails to send. */ getNextPublisher;
451
455
  /** L1 fee analyzer for fisherman mode */ l1FeeAnalyzer;
452
456
  /** Fee asset price oracle for computing price modifiers from Uniswap V4 */ feeAssetPriceOracle;
453
457
  /** Interruptible sleep used by sendRequestsAt to wait until a target timestamp. */ interruptibleSleep;
454
- // A CALL to a cold address is 2700 gas
455
- static MULTICALL_OVERHEAD_GAS_GUESS = 5000n;
456
- // Gas report for VotingWithSigTest shows a max gas of 100k, but we've seen it cost 700k+ in testnet
457
- static VOTE_GAS_GUESS = 800_000n;
458
458
  l1TxUtils;
459
459
  rollupContract;
460
460
  govProposerContract;
461
461
  slashingProposerContract;
462
- slashFactoryContract;
463
462
  tracer;
464
463
  requests;
465
464
  constructor(config, deps){
466
465
  this.config = config;
467
466
  this.interrupted = (_initProto(this), false);
468
- this.governanceLog = createLogger('sequencer:publisher:governance');
469
- this.slashingLog = createLogger('sequencer:publisher:slashing');
467
+ this.revertDecoderAbi = mergeAbis([
468
+ RollupAbi,
469
+ SlashingProposerAbi,
470
+ EmpireBaseAbi,
471
+ ErrorsAbi
472
+ ]);
470
473
  this.lastActions = {};
471
- this.isPayloadEmptyCache = new Map();
472
- this.payloadProposedCache = new Set();
473
474
  this.interruptibleSleep = new InterruptibleSleep();
474
475
  this.requests = [];
475
476
  this.log = deps.log ?? createLogger('sequencer:publisher');
476
477
  this.ethereumSlotDuration = BigInt(config.ethereumSlotDuration);
477
478
  this.aztecSlotDuration = BigInt(config.aztecSlotDuration);
479
+ this.previousL1BlockWaitTimeoutMs = config.sequencerPublisherPreviousL1BlockWaitTimeoutMs;
480
+ this.previousL1BlockWaitPollIntervalMs = config.sequencerPublisherPreviousL1BlockWaitPollIntervalMs;
478
481
  this.dateProvider = deps.dateProvider;
479
482
  this.epochCache = deps.epochCache;
480
483
  this.lastActions = deps.lastActions;
@@ -493,40 +496,116 @@ export class SequencerPublisher {
493
496
  const newSlashingProposer = await this.rollupContract.getSlashingProposer();
494
497
  this.slashingProposerContract = newSlashingProposer;
495
498
  });
496
- this.slashFactoryContract = deps.slashFactoryContract;
497
499
  // Initialize L1 fee analyzer for fisherman mode
498
500
  if (config.fishermanMode) {
499
- this.l1FeeAnalyzer = new L1FeeAnalyzer(this.l1TxUtils.client, deps.dateProvider, createLogger('sequencer:publisher:fee-analyzer'));
501
+ this.l1FeeAnalyzer = new L1FeeAnalyzer(this.l1TxUtils.client, deps.dateProvider, this.log.createChild('fee-analyzer'));
500
502
  }
501
503
  // Initialize fee asset price oracle
502
- this.feeAssetPriceOracle = new FeeAssetPriceOracle(this.l1TxUtils.client, this.rollupContract, createLogger('sequencer:publisher:price-oracle'));
504
+ this.feeAssetPriceOracle = new FeeAssetPriceOracle(this.l1TxUtils.client, this.rollupContract, this.log.createChild('price-oracle'));
503
505
  // Initialize failed L1 tx store (optional, for test networks)
504
506
  this.failedTxStore = createL1TxFailedStore(config.l1TxFailedStore, this.log);
507
+ this.bundleSimulator = new SequencerBundleSimulator({
508
+ getL1TxUtils: ()=>this.l1TxUtils,
509
+ rollupContract: this.rollupContract,
510
+ epochCache: this.epochCache,
511
+ log: this.log.createChild('bundle-simulator')
512
+ });
505
513
  }
506
514
  /**
507
- * Backs up a failed L1 transaction to the configured store for debugging.
508
- * Does nothing if no store is configured.
509
- */ backupFailedTx(failedTx) {
510
- if (!this.failedTxStore) {
511
- return;
512
- }
515
+ * Logs the gas-pricing data of a failed L1 transaction at warn so underpricing is diagnosable
516
+ * from logs even with no failed-tx store configured — and backs the record up to the store when
517
+ * one is. When captureFeeSummary is true, also records the fee data of the already-mined L1
518
+ * blocks in the target slot's inclusion window.
519
+ */ backupFailedTx(failedTx, opts) {
513
520
  const tx = {
514
521
  ...failedTx,
515
522
  timestamp: Date.now()
516
523
  };
517
524
  // Fire and forget - don't block on backup
518
- void this.failedTxStore.then((store)=>store?.saveFailedTx(tx)).catch((err)=>{
519
- this.log.warn(`Failed to backup failed L1 tx to store`, err);
520
- });
525
+ void (async ()=>{
526
+ try {
527
+ // Prefer a pre-captured summary (shared across a batch of failures in the same slot) so we
528
+ // don't re-read the fee window per record. A capture error must not lose the record itself.
529
+ const feeSummary = opts?.sharedFeeSummary ?? (opts?.captureFeeSummary ? await this.captureFeeEnvironment(opts.targetSlot).catch(()=>undefined) : undefined);
530
+ if (feeSummary) {
531
+ tx.gasInfo = {
532
+ ...tx.gasInfo,
533
+ ...feeSummary
534
+ };
535
+ }
536
+ if (tx.gasInfo) {
537
+ this.log.warn(`Gas pricing data for failed L1 tx (${tx.failureType})`, {
538
+ failureType: tx.failureType,
539
+ actions: tx.context.actions,
540
+ slot: tx.context.slot,
541
+ ...tx.gasInfo,
542
+ ...tx.timing
543
+ });
544
+ }
545
+ const store = await this.failedTxStore;
546
+ if (store) {
547
+ await store.saveFailedTx(tx);
548
+ }
549
+ } catch (err) {
550
+ this.log.warn(`Failed to backup failed L1 tx to store`, err);
551
+ }
552
+ })();
553
+ }
554
+ /**
555
+ * Captures per-block fee data for the L1 blocks in the target slot's inclusion window (the blocks the
556
+ * tx could have landed in) for underpricing diagnostics. Reads only already-mined blocks, so it never
557
+ * waits on the chain. Safe to call off the critical path: the underlying capture never throws, and this
558
+ * returns undefined when there is no target slot or the window is not yet mined (e.g. an early send
559
+ * failure), in which case the record simply carries no window data.
560
+ */ async captureFeeEnvironment(targetL2Slot) {
561
+ if (targetL2Slot === undefined) {
562
+ return undefined;
563
+ }
564
+ const l1Constants = this.epochCache.getL1Constants();
565
+ // The inclusion window is [start of slot N, start of slot N+1): all L1 blocks that can include a tx
566
+ // for this L2 slot. getTimestampForSlot returns seconds, matching block.timestamp.
567
+ const windowStartS = getTimestampForSlot(targetL2Slot, l1Constants);
568
+ const windowEndS = getTimestampForSlot(SlotNumber(Number(targetL2Slot) + 1), l1Constants);
569
+ const windowBlocks = await captureWindowBlockFees(this.l1TxUtils.client, windowStartS, windowEndS);
570
+ if (windowBlocks.length === 0) {
571
+ return undefined;
572
+ }
573
+ return {
574
+ windowBlocks
575
+ };
576
+ }
577
+ /** Computes timing info relative to the L2 slot deadline. */ computeTimingInfo(targetL2Slot) {
578
+ if (targetL2Slot === undefined) {
579
+ return undefined;
580
+ }
581
+ const l1Constants = this.epochCache.getL1Constants();
582
+ const slotDeadlineS = getTimestampForSlot(SlotNumber(Number(targetL2Slot) + 1), l1Constants);
583
+ const slotDeadlineMs = Number(slotDeadlineS) * 1000;
584
+ return {
585
+ targetL2Slot: Number(targetL2Slot),
586
+ slotDeadlineTimestampS: slotDeadlineS,
587
+ msUntilSlotDeadline: slotDeadlineMs - this.dateProvider.now()
588
+ };
589
+ }
590
+ /**
591
+ * Builds an id for a synthetic failure record (send-error/timeout) that has no on-chain tx hash.
592
+ * Includes the failure time so each attempt — including retries of the same slot — is stored as its
593
+ * own record rather than overwriting the previous one.
594
+ */ failureRecordId(actions, targetSlot) {
595
+ return keccak256(toHex(`${actions.join(',')}:${targetSlot ?? ''}:${Date.now()}`));
521
596
  }
522
597
  getRollupContract() {
523
598
  return this.rollupContract;
524
599
  }
525
600
  /**
526
601
  * Gets the fee asset price modifier from the oracle.
527
- * Returns 0n if the oracle query fails.
528
- */ getFeeAssetPriceModifier() {
529
- return this.feeAssetPriceOracle.computePriceModifier();
602
+ *
603
+ * @param predictedParentEthPerFeeAssetE12 - Optional predicted parent eth-per-fee-asset (E12).
604
+ * Pipelined proposers should pass the value from the predicted parent fee header so the
605
+ * modifier matches the parent L1 will use when applying it.
606
+ * @returns The fee asset price modifier in basis points, or 0n if the oracle query fails.
607
+ */ getFeeAssetPriceModifier(predictedParentEthPerFeeAssetE12) {
608
+ return this.feeAssetPriceOracle.computePriceModifier(predictedParentEthPerFeeAssetE12);
530
609
  }
531
610
  getSenderAddress() {
532
611
  return this.l1TxUtils.getSenderAddress();
@@ -536,12 +615,6 @@ export class SequencerPublisher {
536
615
  */ getL1FeeAnalyzer() {
537
616
  return this.l1FeeAnalyzer;
538
617
  }
539
- /**
540
- * Sets the proposer address to use for simulations in fisherman mode.
541
- * @param proposerAddress - The actual proposer's address to use for balance lookups in simulations
542
- */ setProposerAddressForSimulation(proposerAddress) {
543
- this.proposerAddressForSimulation = proposerAddress;
544
- }
545
618
  addRequest(request) {
546
619
  this.requests.push(request);
547
620
  }
@@ -599,11 +672,15 @@ export class SequencerPublisher {
599
672
  }
600
673
  /**
601
674
  * Sends all requests that are still valid.
675
+ * @param targetSlot - The target L2 slot for this send. When provided (the production path, via
676
+ * sendRequestsAt), it is threaded into bundleSimulate so the block.timestamp override matches
677
+ * the slot the propose is built for. When omitted, falls back to getCurrentL2Slot() for the
678
+ * AutomineSequencer, which publishes synchronously within the current slot.
602
679
  * @returns one of:
603
680
  * - A receipt and stats if the tx succeeded
604
681
  * - a receipt and errorMsg if it failed on L1
605
682
  * - undefined if no valid requests are found OR the tx failed to send.
606
- */ async sendRequests() {
683
+ */ async sendRequests(targetSlot) {
607
684
  const requestsToProcess = [
608
685
  ...this.requests
609
686
  ];
@@ -611,10 +688,9 @@ export class SequencerPublisher {
611
688
  if (this.interrupted || requestsToProcess.length === 0) {
612
689
  return undefined;
613
690
  }
614
- const currentL2Slot = this.getCurrentL2Slot();
691
+ const currentL2Slot = targetSlot ?? this.getCurrentL2Slot();
615
692
  this.log.debug(`Sending requests on L2 slot ${currentL2Slot}`);
616
693
  const validRequests = requestsToProcess.filter((request)=>request.lastValidL2Slot >= currentL2Slot);
617
- const validActions = validRequests.map((x)=>x.action);
618
694
  const expiredActions = requestsToProcess.filter((request)=>request.lastValidL2Slot < currentL2Slot).map((x)=>x.action);
619
695
  if (validRequests.length !== requestsToProcess.length) {
620
696
  this.log.warn(`Some requests were expired for slot ${currentL2Slot}`, {
@@ -632,75 +708,98 @@ export class SequencerPublisher {
632
708
  this.log.debug(`No valid requests to send`);
633
709
  return undefined;
634
710
  }
635
- // @note - we can only have one blob config per bundle
636
- // find requests with gas and blob configs
637
- // See https://github.com/AztecProtocol/aztec-packages/issues/11513
711
+ // Collect earliest txTimeoutAt across all requests.
638
712
  const gasConfigs = validRequests.filter((request)=>request.gasConfig).map((request)=>request.gasConfig);
639
- const blobConfigs = validRequests.filter((request)=>request.blobConfig).map((request)=>request.blobConfig);
640
- if (blobConfigs.length > 1) {
641
- throw new Error('Multiple blob configs found');
642
- }
643
- const blobConfig = blobConfigs[0];
644
- // Merge gasConfigs. Yields the sum of gasLimits, and the earliest txTimeoutAt, or undefined if no gasConfig sets them.
645
- const gasLimits = gasConfigs.map((g)=>g?.gasLimit).filter((g)=>g !== undefined);
646
- let gasLimit = gasLimits.length > 0 ? sumBigint(gasLimits) : undefined; // sum
647
- // Cap at L1 block gas limit so the node accepts the tx ("gas limit too high" otherwise).
648
- const maxGas = MAX_L1_TX_LIMIT;
649
- if (gasLimit !== undefined && gasLimit > maxGas) {
650
- this.log.debug('Capping bundled tx gas limit to L1 max', {
651
- requested: gasLimit,
652
- capped: maxGas
653
- });
654
- gasLimit = maxGas;
655
- }
656
713
  const txTimeoutAts = gasConfigs.map((g)=>g?.txTimeoutAt).filter((g)=>g !== undefined);
657
- const txTimeoutAt = txTimeoutAts.length > 0 ? new Date(Math.min(...txTimeoutAts.map((g)=>g.getTime()))) : undefined; // earliest
658
- const txConfig = {
659
- gasLimit,
660
- txTimeoutAt
661
- };
714
+ const txTimeoutAt = txTimeoutAts.length > 0 ? new Date(Math.min(...txTimeoutAts.map((g)=>g.getTime()))) : undefined;
662
715
  // Sort the requests so that proposals always go first
663
716
  // This ensures the committee gets precomputed correctly
664
717
  validRequests.sort((a, b)=>compareActions(a.action, b.action));
665
718
  try {
666
- // Capture context for failed tx backup before sending
667
- const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
668
- const multicallData = encodeFunctionData({
669
- abi: multicall3Abi,
670
- functionName: 'aggregate3',
671
- args: [
672
- validRequests.map((r)=>({
673
- target: r.request.to,
674
- callData: r.request.data,
675
- allowFailure: true
676
- }))
677
- ]
678
- });
679
- const blobDataHex = blobConfig?.blobs?.map((b)=>toHex(b));
680
- const txContext = {
681
- multicallData,
682
- blobData: blobDataHex,
683
- l1BlockNumber
719
+ // Bundle-level eth_simulateV1: filters out entries that revert and derives the gasLimit.
720
+ const bundleResult = await this.bundleSimulator.simulate(validRequests, currentL2Slot);
721
+ if (bundleResult.kind === 'aborted') {
722
+ this.logDroppedInSim(bundleResult.droppedRequests);
723
+ void this.backupDroppedInSim(bundleResult.droppedRequests, currentL2Slot).catch((err)=>this.log.error(`Failed to backup requests dropped in simulation`, err));
724
+ return undefined;
725
+ }
726
+ const { requests, droppedRequests, gasLimit } = bundleResult.kind === 'fallback' ? {
727
+ requests: bundleResult.requests,
728
+ droppedRequests: bundleResult.droppedRequests,
729
+ gasLimit: MAX_L1_TX_LIMIT
730
+ } : bundleResult;
731
+ this.logDroppedInSim(droppedRequests);
732
+ // Compute blobConfig from survivors (not original validRequests) so that if the propose
733
+ // entry was dropped by bundleSimulate we don't attach a blob-typed config to a non-blob tx.
734
+ const [blobConfig] = requests.filter((r)=>r.blobConfig).map((r)=>r.blobConfig);
735
+ const txConfig = {
736
+ gasLimit,
737
+ txTimeoutAt
684
738
  };
685
739
  this.log.debug('Forwarding transactions', {
686
- validRequests: validRequests.map((request)=>request.action),
740
+ requests: requests.map((request)=>request.action),
687
741
  txConfig
688
742
  });
689
- const result = await this.forwardWithPublisherRotation(validRequests, txConfig, blobConfig);
743
+ const result = await this.forwardWithPublisherRotation(requests, txConfig, blobConfig, currentL2Slot);
690
744
  if (result === undefined) {
691
745
  return undefined;
692
746
  }
693
- const { successfulActions = [], failedActions = [] } = this.callbackBundledTransactions(validRequests, result, txContext);
747
+ const { successfulActions = [], failedActions = [] } = this.callbackBundledTransactions(requests, result);
748
+ const allFailedActions = [
749
+ ...failedActions,
750
+ ...droppedRequests.map((d)=>d.request.action)
751
+ ];
694
752
  return {
695
753
  result,
696
754
  expiredActions,
697
- sentActions: validActions,
755
+ sentActions: requests.map((x)=>x.action),
698
756
  successfulActions,
699
- failedActions
757
+ failedActions: allFailedActions
700
758
  };
701
759
  } catch (err) {
702
760
  const viemError = formatViemError(err);
703
761
  this.log.error(`Failed to publish bundled transactions`, viemError);
762
+ if (err instanceof TimeoutError) {
763
+ const timeoutState = err instanceof L1TxTimeoutError ? err.txState : undefined;
764
+ void (async ()=>{
765
+ // The RPC is likely degraded right after a timeout, so back up without the block number
766
+ // rather than leaking an unhandled rejection.
767
+ const l1BlockNumber = await this.l1TxUtils.getBlockNumber().catch(()=>0n);
768
+ this.backupFailedTx({
769
+ id: this.failureRecordId(validRequests.map((r)=>r.action), currentL2Slot),
770
+ failureType: 'timeout',
771
+ request: {
772
+ to: MULTI_CALL_3_ADDRESS,
773
+ data: '0x'
774
+ },
775
+ l1BlockNumber,
776
+ error: {
777
+ message: viemError.message,
778
+ name: 'TimeoutError'
779
+ },
780
+ context: {
781
+ actions: validRequests.map((r)=>r.action),
782
+ requests: validRequests.filter((r)=>r.request.to !== null).map((r)=>({
783
+ action: r.action,
784
+ to: r.request.to,
785
+ data: r.request.data
786
+ })),
787
+ sender: this.getSenderAddress().toString(),
788
+ slot: Number(currentL2Slot)
789
+ },
790
+ timing: this.computeTimingInfo(currentL2Slot),
791
+ gasInfo: timeoutState ? {
792
+ sentGasPriceLadder: timeoutState.gasPriceHistory,
793
+ attempts: timeoutState.attempts,
794
+ gasLimit: timeoutState.gasLimit,
795
+ nonce: timeoutState.nonce
796
+ } : undefined
797
+ }, {
798
+ captureFeeSummary: true,
799
+ targetSlot: currentL2Slot
800
+ });
801
+ })();
802
+ }
704
803
  return undefined;
705
804
  } finally{
706
805
  try {
@@ -710,26 +809,96 @@ export class SequencerPublisher {
710
809
  }
711
810
  }
712
811
  }
812
+ /** Logs entries dropped by bundle simulation as warnings on the publisher's logger. */ logDroppedInSim(dropped) {
813
+ for (const drop of dropped){
814
+ const revertReasonDecoded = drop.revertReason ?? tryDecodeRevertReason(drop.returnData, this.revertDecoderAbi);
815
+ this.log.warn('Bundle entry dropped: action reverted in sim', {
816
+ action: drop.request.action,
817
+ revertReason: revertReasonDecoded ?? drop.returnData,
818
+ revertReasonDecoded,
819
+ returnData: drop.returnData
820
+ });
821
+ }
822
+ }
823
+ /** Backs up entries dropped by bundle simulation, one record per dropped action. */ async backupDroppedInSim(dropped, targetSlot) {
824
+ if (dropped.length === 0) {
825
+ return;
826
+ }
827
+ // Invoked as `void backupDroppedInSim(...)` on the publish path, so it must not throw.
828
+ try {
829
+ const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
830
+ // Every dropped entry failed in the same slot against the same L1 fee conditions, so capture
831
+ // the fee environment once and share it rather than re-reading the window per entry.
832
+ const sharedFeeSummary = await this.captureFeeEnvironment(targetSlot).catch(()=>undefined);
833
+ const timing = this.computeTimingInfo(targetSlot);
834
+ for (const { request: req } of dropped){
835
+ this.backupFailedTx({
836
+ id: keccak256(req.request.data),
837
+ failureType: 'simulation',
838
+ request: {
839
+ to: req.request.to,
840
+ data: req.request.data
841
+ },
842
+ l1BlockNumber,
843
+ error: {
844
+ message: 'Bundle entry dropped: action reverted in sim'
845
+ },
846
+ context: {
847
+ actions: [
848
+ req.action
849
+ ],
850
+ sender: this.getSenderAddress().toString(),
851
+ slot: targetSlot !== undefined ? Number(targetSlot) : undefined
852
+ },
853
+ timing
854
+ }, {
855
+ sharedFeeSummary
856
+ });
857
+ }
858
+ } catch (err) {
859
+ this.log.warn(`Failed to back up dropped-in-sim entries`, err);
860
+ }
861
+ }
713
862
  /**
714
863
  * Forwards transactions via Multicall3, rotating to the next available publisher if a send
715
864
  * failure occurs (i.e. the tx never reached the chain).
716
865
  * On-chain reverts and simulation errors are returned as-is without rotation.
717
- */ async forwardWithPublisherRotation(validRequests, txConfig, blobConfig) {
866
+ */ async forwardWithPublisherRotation(validRequests, txConfig, blobConfig, targetSlot) {
867
+ if (!txConfig?.gasLimit) {
868
+ throw new Error('gasLimit is required for bundled transactions');
869
+ }
870
+ const txConfigWithGasLimit = txConfig;
718
871
  const triedAddresses = [];
719
872
  let currentPublisher = this.l1TxUtils;
720
873
  while(true){
874
+ if (txConfig.txTimeoutAt && new Date() > txConfig.txTimeoutAt) {
875
+ this.log.warn(`Tx timeout (${txConfig.txTimeoutAt.toISOString()}) elapsed; stopping publisher rotation`, {
876
+ triedAddresses: triedAddresses.map((a)=>a.toString())
877
+ });
878
+ return undefined;
879
+ }
721
880
  triedAddresses.push(currentPublisher.getSenderAddress());
722
881
  try {
723
- const result = await Multicall3.forward(validRequests.map((r)=>r.request), currentPublisher, txConfig, blobConfig, this.rollupContract.address, this.log);
882
+ const result = await Multicall3.forward(validRequests.map((r)=>r.request), currentPublisher, txConfigWithGasLimit, blobConfig, {
883
+ gasLimitRequired: true
884
+ });
724
885
  this.l1TxUtils = currentPublisher;
725
886
  return result;
726
887
  } catch (err) {
727
888
  if (err instanceof TimeoutError) {
728
889
  throw err;
729
890
  }
891
+ if (err instanceof MulticallForwarderRevertedError) {
892
+ this.log.error('Forwarder transaction reverted on-chain; not rotating publisher', err, {
893
+ transactionHash: err.receipt.transactionHash
894
+ });
895
+ this.backupRevertFailure(validRequests, err, currentPublisher, targetSlot);
896
+ return undefined;
897
+ }
730
898
  const viemError = formatViemError(err);
731
899
  if (!this.getNextPublisher) {
732
900
  this.log.error('Failed to publish bundled transactions', viemError);
901
+ this.backupSendFailure(validRequests, viemError, currentPublisher, targetSlot);
733
902
  return undefined;
734
903
  }
735
904
  this.log.warn(`Publisher ${currentPublisher.getSenderAddress()} failed to send, rotating to next publisher`, viemError);
@@ -737,140 +906,205 @@ export class SequencerPublisher {
737
906
  ...triedAddresses
738
907
  ]);
739
908
  if (!nextPublisher) {
740
- this.log.error('All available publishers exhausted, failed to publish bundled transactions');
909
+ this.log.error(`All available publishers exhausted (tried ${triedAddresses.length}), failed to publish bundled transactions`, viemError, {
910
+ triedAddresses: triedAddresses.map((a)=>a.toString())
911
+ });
912
+ this.backupSendFailure(validRequests, viemError, currentPublisher, targetSlot);
741
913
  return undefined;
742
914
  }
743
915
  currentPublisher = nextPublisher;
744
916
  }
745
917
  }
746
918
  }
747
- /*
748
- * Schedules sending all enqueued requests at (or after) the given timestamp.
749
- * Uses InterruptibleSleep so it can be cancelled via interrupt().
750
- * Returns the promise for the L1 response (caller should NOT await this in the work loop).
751
- */ async sendRequestsAt(submitAfter) {
752
- const ms = submitAfter.getTime() - this.dateProvider.now();
753
- if (ms > 0) {
754
- this.log.debug(`Sleeping ${ms}ms before sending requests`, {
755
- submitAfter
756
- });
757
- await this.interruptibleSleep.sleep(ms);
758
- }
759
- if (this.interrupted) {
760
- return undefined;
761
- }
762
- return this.sendRequests();
919
+ /** Backs up an on-chain revert failure to the failed tx store. */ backupRevertFailure(requests, err, publisher, targetSlot) {
920
+ this.backupFailedTx({
921
+ id: err.receipt.transactionHash,
922
+ failureType: 'revert',
923
+ request: {
924
+ to: MULTI_CALL_3_ADDRESS,
925
+ data: '0x'
926
+ },
927
+ l1BlockNumber: err.receipt.blockNumber,
928
+ receipt: {
929
+ transactionHash: err.receipt.transactionHash,
930
+ blockNumber: err.receipt.blockNumber,
931
+ gasUsed: err.receipt.gasUsed,
932
+ status: 'reverted'
933
+ },
934
+ error: {
935
+ message: err.message,
936
+ name: err.name
937
+ },
938
+ context: {
939
+ actions: requests.map((r)=>r.action),
940
+ requests: requests.filter((r)=>r.request.to !== null).map((r)=>({
941
+ action: r.action,
942
+ to: r.request.to,
943
+ data: r.request.data
944
+ })),
945
+ sender: publisher.getSenderAddress().toString(),
946
+ slot: targetSlot !== undefined ? Number(targetSlot) : undefined
947
+ },
948
+ gasInfo: err.txState ? {
949
+ sentGasPrice: err.txState.gasPrice,
950
+ gasLimit: err.txState.gasLimit,
951
+ nonce: err.txState.nonce
952
+ } : undefined,
953
+ timing: this.computeTimingInfo(targetSlot)
954
+ }, {
955
+ captureFeeSummary: true,
956
+ targetSlot
957
+ });
763
958
  }
764
- callbackBundledTransactions(requests, result, txContext) {
765
- const actionsListStr = requests.map((r)=>r.action).join(', ');
766
- if (result instanceof FormattedViemError) {
767
- this.log.error(`Failed to publish bundled transactions (${actionsListStr})`, result);
959
+ /** Backs up a send failure (tx never reached chain) to the failed tx store. */ backupSendFailure(requests, error, publisher, targetSlot) {
960
+ // If we can't get the block number, still back up without it.
961
+ void this.l1TxUtils.getBlockNumber().catch(()=>0n).then((l1BlockNumber)=>{
768
962
  this.backupFailedTx({
769
- id: keccak256(txContext.multicallData),
963
+ id: this.failureRecordId(requests.map((r)=>r.action), targetSlot),
770
964
  failureType: 'send-error',
771
965
  request: {
772
966
  to: MULTI_CALL_3_ADDRESS,
773
- data: txContext.multicallData
967
+ data: '0x'
774
968
  },
775
- blobData: txContext.blobData,
776
- l1BlockNumber: txContext.l1BlockNumber.toString(),
969
+ l1BlockNumber,
777
970
  error: {
778
- message: result.message,
779
- name: result.name
971
+ message: error.message,
972
+ name: 'name' in error ? error.name : undefined
780
973
  },
781
974
  context: {
782
975
  actions: requests.map((r)=>r.action),
783
- requests: requests.map((r)=>({
976
+ requests: requests.filter((r)=>r.request.to !== null).map((r)=>({
784
977
  action: r.action,
785
978
  to: r.request.to,
786
979
  data: r.request.data
787
980
  })),
788
- sender: this.getSenderAddress().toString()
789
- }
790
- });
791
- return {
792
- failedActions: requests.map((r)=>r.action)
793
- };
794
- } else {
795
- this.log.verbose(`Published bundled transactions (${actionsListStr})`, {
796
- result,
797
- requests: requests.map((r)=>({
798
- ...r,
799
- // Avoid logging large blob data
800
- blobConfig: r.blobConfig ? {
801
- ...r.blobConfig,
802
- blobs: r.blobConfig.blobs.map((b)=>({
803
- size: trimmedBytesLength(b)
804
- }))
805
- } : undefined
806
- }))
981
+ sender: publisher.getSenderAddress().toString(),
982
+ slot: targetSlot !== undefined ? Number(targetSlot) : undefined
983
+ },
984
+ timing: this.computeTimingInfo(targetSlot)
985
+ }, {
986
+ captureFeeSummary: true,
987
+ targetSlot
807
988
  });
808
- const successfulActions = [];
809
- const failedActions = [];
810
- for (const request of requests){
811
- if (request.checkSuccess(request.request, result)) {
812
- successfulActions.push(request.action);
813
- } else {
814
- failedActions.push(request.action);
989
+ });
990
+ }
991
+ /*
992
+ * Schedules sending all enqueued requests at (or after) the start of the given L2 slot.
993
+ */ async sendRequestsAt(targetSlot) {
994
+ await this.waitForTargetSlot(targetSlot);
995
+ if (this.interrupted) {
996
+ return undefined;
997
+ }
998
+ return this.sendRequests(targetSlot);
999
+ }
1000
+ /**
1001
+ * Sleeps until one L1 slot before the L2 slot boundary, and then waits for that L1 block
1002
+ * to be mined, so we don't risk being included in it. If that block never gets mined after
1003
+ * a timeout, we assume it got skipped on L1, so we send the tx anyway.
1004
+ */ async waitForTargetSlot(targetSlot) {
1005
+ const l1Constants = this.epochCache.getL1Constants();
1006
+ const nowInSeconds = this.dateProvider.nowInSeconds();
1007
+ const startOfTargetSlotTs = getTimestampForSlot(targetSlot, l1Constants);
1008
+ const previousL1BlockTs = startOfTargetSlotTs - this.ethereumSlotDuration;
1009
+ const waitDeadlineTs = previousL1BlockTs + BigInt(this.previousL1BlockWaitTimeoutMs / 1000);
1010
+ const logCtx = {
1011
+ targetSlot,
1012
+ startOfTargetSlotTs,
1013
+ nowInSeconds,
1014
+ previousL1BlockTs,
1015
+ waitDeadlineTs
1016
+ };
1017
+ // Check if we are already past time
1018
+ if (nowInSeconds >= startOfTargetSlotTs) {
1019
+ this.log.verbose(`Target slot ${targetSlot} already started, sending requests immediately`, logCtx);
1020
+ return;
1021
+ }
1022
+ // Otherwise we wait
1023
+ this.log.debug(`Waiting for slot ${targetSlot} before sending requests`, logCtx);
1024
+ // Wait until previous L1 block timestamp first
1025
+ const sleepMs = (Number(previousL1BlockTs) - nowInSeconds) * 1000;
1026
+ if (sleepMs > 0 && !this.interrupted) {
1027
+ this.log.trace(`Sleeping ${sleepMs}ms before waiting for previous L1 block`, logCtx);
1028
+ await this.interruptibleSleep.sleep(sleepMs);
1029
+ }
1030
+ // Then loop until we see the previous L1 block, so we know that we cannot be included in it.
1031
+ // We time out after a while, once we are sure that that block is skipped in L1.
1032
+ while(!this.interrupted){
1033
+ try {
1034
+ const nowInSeconds = this.dateProvider.nowInSeconds();
1035
+ logCtx.nowInSeconds = nowInSeconds;
1036
+ if (nowInSeconds >= waitDeadlineTs) {
1037
+ this.log.warn(`Timed out waiting for previous L1 block before sending requests, proceeding`, logCtx);
1038
+ return;
815
1039
  }
816
- }
817
- // Single backup for the whole reverted tx
818
- if (failedActions.length > 0 && result?.receipt?.status === 'reverted') {
819
- this.backupFailedTx({
820
- id: result.receipt.transactionHash,
821
- failureType: 'revert',
822
- request: {
823
- to: MULTI_CALL_3_ADDRESS,
824
- data: txContext.multicallData
825
- },
826
- blobData: txContext.blobData,
827
- l1BlockNumber: result.receipt.blockNumber.toString(),
828
- receipt: {
829
- transactionHash: result.receipt.transactionHash,
830
- blockNumber: result.receipt.blockNumber.toString(),
831
- gasUsed: (result.receipt.gasUsed ?? 0n).toString(),
832
- status: 'reverted'
833
- },
834
- error: {
835
- message: result.errorMsg ?? 'Transaction reverted'
836
- },
837
- context: {
838
- actions: failedActions,
839
- requests: requests.filter((r)=>failedActions.includes(r.action)).map((r)=>({
840
- action: r.action,
841
- to: r.request.to,
842
- data: r.request.data
843
- })),
844
- sender: this.getSenderAddress().toString()
845
- }
1040
+ const latestBlockTs = await this.l1TxUtils.getBlock().then((b)=>b.timestamp);
1041
+ if (latestBlockTs >= previousL1BlockTs) {
1042
+ this.log.debug(`Previous L1 block mined, proceeding to send requests`, {
1043
+ ...logCtx,
1044
+ latestBlockTs
1045
+ });
1046
+ return;
1047
+ }
1048
+ this.log.trace(`Previous L1 block not mined yet, continuing to wait`, {
1049
+ ...logCtx,
1050
+ latestBlockTs
846
1051
  });
1052
+ } catch (err) {
1053
+ this.log.error(`Error while waiting for previous L1 block before sending requests; retrying`, err, logCtx);
1054
+ } finally{
1055
+ await this.interruptibleSleep.sleep(this.previousL1BlockWaitPollIntervalMs);
847
1056
  }
848
- return {
849
- successfulActions,
850
- failedActions
851
- };
852
1057
  }
853
1058
  }
1059
+ callbackBundledTransactions(requests, result) {
1060
+ const actionsListStr = requests.map((r)=>r.action).join(', ');
1061
+ this.log.verbose(`Published bundled transactions (${actionsListStr})`, {
1062
+ result,
1063
+ requests: requests.map((r)=>({
1064
+ ...r,
1065
+ // Avoid logging large blob data
1066
+ blobConfig: r.blobConfig ? {
1067
+ ...r.blobConfig,
1068
+ blobs: r.blobConfig.blobs.map((b)=>({
1069
+ size: trimmedBytesLength(b)
1070
+ }))
1071
+ } : undefined
1072
+ }))
1073
+ });
1074
+ const successfulActions = [];
1075
+ const failedActions = [];
1076
+ for (const request of requests){
1077
+ if (request.checkSuccess(request.request, result)) {
1078
+ successfulActions.push(request.action);
1079
+ } else {
1080
+ failedActions.push(request.action);
1081
+ }
1082
+ }
1083
+ return {
1084
+ successfulActions,
1085
+ failedActions
1086
+ };
1087
+ }
854
1088
  /**
855
1089
  * @notice Will call `canProposeAt` to make sure that it is possible to propose
856
1090
  * @param tipArchive - The archive to check
857
1091
  * @returns The slot and block number if it is possible to propose, undefined otherwise
858
- */ canProposeAt(tipArchive, msgSender, opts = {}) {
1092
+ */ async canProposeAt(tipArchive, msgSender, simulationOverridesPlan) {
859
1093
  // TODO: #14291 - should loop through multiple keys to check if any of them can propose
860
- const ignoredErrors = [
1094
+ // These errors are expected when we cannot actually propose right now — usually because our
1095
+ // local view of the chain is ahead of L1 (proposed parent hasn't landed yet, or someone
1096
+ // else has just landed the slot, or the archive override doesn't match). We log a warn and
1097
+ // skip the proposal; we do NOT treat these as bugs.
1098
+ const expectedErrors = [
861
1099
  'SlotAlreadyInChain',
862
1100
  'InvalidProposer',
863
1101
  'InvalidArchive'
864
1102
  ];
865
- const pipelined = opts.pipelined ?? this.epochCache.isProposerPipeliningEnabled();
866
- const slotOffset = pipelined ? this.aztecSlotDuration : 0n;
1103
+ const slotOffset = this.aztecSlotDuration;
867
1104
  const nextL1SlotTs = this.getNextL1SlotTimestamp() + slotOffset;
868
- return this.rollupContract.canProposeAt(tipArchive.toBuffer(), msgSender.toString(), nextL1SlotTs, {
869
- forcePendingCheckpointNumber: opts.forcePendingCheckpointNumber,
870
- forceArchive: opts.forceArchive
871
- }).catch((err)=>{
872
- if (err instanceof FormattedViemError && ignoredErrors.find((e)=>err.message.includes(e))) {
873
- this.log.warn(`Failed canProposeAtTime check with ${ignoredErrors.find((e)=>err.message.includes(e))}`, {
1105
+ return this.rollupContract.canProposeAt(tipArchive.toBuffer(), msgSender.toString(), nextL1SlotTs, await buildSimulationOverridesStateOverride(this.rollupContract, simulationOverridesPlan)).catch((err)=>{
1106
+ if (err instanceof FormattedViemError && expectedErrors.find((e)=>err.message.includes(e))) {
1107
+ this.log.warn(`Failed canProposeAtTime check with ${expectedErrors.find((e)=>err.message.includes(e))}`, {
874
1108
  error: err.message
875
1109
  });
876
1110
  } else {
@@ -884,22 +1118,23 @@ export class SequencerPublisher {
884
1118
  * @dev This is a convenience function that can be used by the sequencer to validate a "partial" header.
885
1119
  * It will throw if the block header is invalid.
886
1120
  * @param header - The block header to validate
887
- */ async validateBlockHeader(header, opts) {
1121
+ */ async validateBlockHeader(header, simulationOverridesPlan) {
888
1122
  const flags = {
889
1123
  ignoreDA: true,
890
1124
  ignoreSignatures: true
891
1125
  };
892
1126
  const args = [
893
1127
  header.toViem(),
894
- CommitteeAttestationsAndSigners.empty().getPackedAttestations(),
1128
+ CommitteeAttestationsAndSigners.packAttestations([]),
895
1129
  [],
896
1130
  Signature.empty().toViemSignature(),
897
1131
  `0x${'0'.repeat(64)}`,
898
1132
  header.blobsHash.toString(),
899
1133
  flags
900
1134
  ];
901
- const ts = this.getSimulationTimestamp(header.slotNumber);
902
- const stateOverrides = await this.rollupContract.makePendingCheckpointNumberOverride(opts?.forcePendingCheckpointNumber);
1135
+ const l1Constants = this.epochCache.getL1Constants();
1136
+ const ts = getLastL1SlotTimestampForL2Slot(header.slotNumber, l1Constants);
1137
+ const stateOverrides = await buildSimulationOverridesStateOverride(this.rollupContract, simulationOverridesPlan);
903
1138
  let balance = 0n;
904
1139
  if (this.config.fishermanMode) {
905
1140
  // In fisherman mode, we can't know where the proposer is publishing from
@@ -1001,9 +1236,9 @@ export class SequencerPublisher {
1001
1236
  request: {
1002
1237
  to: request.to,
1003
1238
  data: request.data,
1004
- value: request.value?.toString()
1239
+ value: request.value
1005
1240
  },
1006
- l1BlockNumber: l1BlockNumber.toString(),
1241
+ l1BlockNumber,
1007
1242
  error: {
1008
1243
  message: viemError.message,
1009
1244
  name: viemError.name
@@ -1014,7 +1249,11 @@ export class SequencerPublisher {
1014
1249
  ],
1015
1250
  checkpointNumber,
1016
1251
  sender: this.getSenderAddress().toString()
1017
- }
1252
+ },
1253
+ timing: this.computeTimingInfo(this.getCurrentL2Slot())
1254
+ }, {
1255
+ captureFeeSummary: true,
1256
+ targetSlot: this.getCurrentL2Slot()
1018
1257
  });
1019
1258
  throw new Error(`Failed to simulate invalidate checkpoint ${checkpointNumber}`, {
1020
1259
  cause: viemError
@@ -1031,7 +1270,11 @@ export class SequencerPublisher {
1031
1270
  reason
1032
1271
  };
1033
1272
  this.log.debug(`Building invalidate checkpoint ${checkpoint.checkpointNumber} request`, logData);
1034
- const attestationsAndSigners = new CommitteeAttestationsAndSigners(validationResult.attestations).getPackedAttestations();
1273
+ // Use the exact packed tuple posted to L1 verbatim. A repack via `packAttestations` is not a
1274
+ // byte-faithful inverse of `fromPacked` (a canonicalized yParity byte or an all-zero signature slot
1275
+ // round-trips differently), so it would diverge from the stored `attestationsHash` and revert the
1276
+ // invalidation.
1277
+ const attestationsAndSigners = validationResult.verbatimAttestations;
1035
1278
  if (reason === 'invalid-attestation') {
1036
1279
  return this.rollupContract.buildInvalidateBadAttestationRequest(checkpoint.checkpointNumber, attestationsAndSigners, committee, validationResult.invalidIndex);
1037
1280
  } else if (reason === 'insufficient-attestations') {
@@ -1041,25 +1284,6 @@ export class SequencerPublisher {
1041
1284
  throw new Error(`Unknown reason for invalidation`);
1042
1285
  }
1043
1286
  }
1044
- /** Simulates `propose` to make sure that the checkpoint is valid for submission */ async validateCheckpointForSubmission(checkpoint, attestationsAndSigners, attestationsAndSignersSignature, options) {
1045
- const blobFields = checkpoint.toBlobFields();
1046
- const blobs = await getBlobsPerL1Block(blobFields);
1047
- const blobInput = getPrefixedEthBlobCommitments(blobs);
1048
- const args = [
1049
- {
1050
- header: checkpoint.header.toViem(),
1051
- archive: toHex(checkpoint.archive.root.toBuffer()),
1052
- oracleInput: {
1053
- feeAssetPriceModifier: checkpoint.feeAssetPriceModifier
1054
- }
1055
- },
1056
- attestationsAndSigners.getPackedAttestations(),
1057
- attestationsAndSigners.getSigners().map((signer)=>signer.toString()),
1058
- attestationsAndSignersSignature.toViemSignature(),
1059
- blobInput
1060
- ];
1061
- await this.simulateProposeTx(args, options);
1062
- }
1063
1287
  async enqueueCastSignalHelper(slotNumber, signalType, payload, base, signerAddress, signer) {
1064
1288
  if (this.lastActions[signalType] && this.lastActions[signalType] === slotNumber) {
1065
1289
  this.log.debug(`Skipping duplicate vote cast signal ${signalType} for slot ${slotNumber}`);
@@ -1080,30 +1304,26 @@ export class SequencerPublisher {
1080
1304
  if (roundInfo.lastSignalSlot >= slotNumber) {
1081
1305
  return false;
1082
1306
  }
1083
- if (await this.isPayloadEmpty(payload)) {
1307
+ if (await base.isPayloadEmpty(payload)) {
1084
1308
  this.log.warn(`Skipping vote cast for payload with empty code`);
1085
1309
  return false;
1086
1310
  }
1087
- // Check if payload was already submitted to governance
1088
- const cacheKey = payload.toString();
1089
- if (!this.payloadProposedCache.has(cacheKey)) {
1090
- try {
1091
- const l1StartBlock = await this.rollupContract.getL1StartBlock();
1092
- const proposed = await retry(()=>base.hasPayloadBeenProposed(payload.toString(), l1StartBlock), 'Check if payload was proposed', makeBackoff([
1093
- 0,
1094
- 1,
1095
- 2
1096
- ]), this.log, true);
1097
- if (proposed) {
1098
- this.payloadProposedCache.add(cacheKey);
1099
- }
1100
- } catch (err) {
1101
- this.log.warn(`Failed to check if payload ${payload} was proposed after retries, skipping signal`, err);
1102
- return false;
1103
- }
1311
+ // Skip signaling if there is already a live (non-terminal) Governance proposal for this
1312
+ // payload. This is intentionally not cached: a previously-live proposal may transition to
1313
+ // a terminal state (Dropped/Rejected/Expired/Executed), at which point we may want to re-signal
1314
+ // the same payload in a future round.
1315
+ let proposed = false;
1316
+ try {
1317
+ proposed = await base.hasActiveProposalWithPayload(payload.toString());
1318
+ } catch (err) {
1319
+ // We deliberately swallow the error and proceed to signal. Failing closed (skipping the
1320
+ // signal) on transient RPC errors would let a flaky L1 endpoint silence governance
1321
+ // participation entirely; failing open at worst produces a duplicate signal that the
1322
+ // contract will simply count alongside others in the round.
1323
+ this.log.error(`Failed to check if payload ${payload} was already proposed (signalling anyway)`, err);
1104
1324
  }
1105
- if (this.payloadProposedCache.has(cacheKey)) {
1106
- this.log.info(`Payload ${payload} was already proposed to governance, stopping signals`);
1325
+ if (proposed) {
1326
+ this.log.info(`Payload ${payload} has a live governance proposal, stopping signals`);
1107
1327
  return false;
1108
1328
  }
1109
1329
  const cachedLastVote = this.lastActions[signalType];
@@ -1116,57 +1336,17 @@ export class SequencerPublisher {
1116
1336
  signer: this.l1TxUtils.client.account?.address,
1117
1337
  lastValidL2Slot: slotNumber
1118
1338
  });
1119
- const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
1120
- const timestamp = this.getSimulationTimestamp(slotNumber);
1121
- try {
1122
- await this.l1TxUtils.simulate(request, {
1123
- time: timestamp
1124
- }, [], mergeAbis([
1125
- request.abi ?? [],
1126
- ErrorsAbi
1127
- ]));
1128
- this.log.debug(`Simulation for ${action} at slot ${slotNumber} succeeded`, {
1129
- request
1130
- });
1131
- } catch (err) {
1132
- const viemError = formatViemError(err);
1133
- this.log.error(`Failed simulation for ${action} at slot ${slotNumber} (enqueuing the action anyway)`, viemError, {
1134
- simulationTimestamp: timestamp,
1135
- l1BlockNumber
1136
- });
1137
- this.backupFailedTx({
1138
- id: keccak256(request.data),
1139
- failureType: 'simulation',
1140
- request: {
1141
- to: request.to,
1142
- data: request.data,
1143
- value: request.value?.toString()
1144
- },
1145
- l1BlockNumber: l1BlockNumber.toString(),
1146
- error: {
1147
- message: viemError.message,
1148
- name: viemError.name
1149
- },
1150
- context: {
1151
- actions: [
1152
- action
1153
- ],
1154
- slot: slotNumber,
1155
- sender: this.getSenderAddress().toString()
1156
- }
1157
- });
1158
- // Yes, we enqueue the request anyway, in case there was a bug with the simulation itself
1159
- }
1160
1339
  // TODO(palla/slash): All votes (governance and slashing) should txTimeoutAt at the end of the slot.
1161
1340
  this.addRequest({
1162
- gasConfig: {
1163
- gasLimit: SequencerPublisher.VOTE_GAS_GUESS
1164
- },
1165
1341
  action,
1166
1342
  request,
1167
1343
  lastValidL2Slot: slotNumber,
1168
1344
  checkSuccess: (_request, result)=>{
1169
- const success = result && result.receipt && result.receipt.status === 'success' && tryExtractEvent(result.receipt.logs, base.address.toString(), EmpireBaseAbi, 'SignalCast');
1345
+ const success = result && extractEventSuccess(result.receipt, {
1346
+ address: base.address.toString(),
1347
+ abi: EmpireBaseAbi,
1348
+ eventName: 'SignalCast'
1349
+ });
1170
1350
  const logData = {
1171
1351
  ...result,
1172
1352
  slotNumber,
@@ -1185,16 +1365,6 @@ export class SequencerPublisher {
1185
1365
  });
1186
1366
  return true;
1187
1367
  }
1188
- async isPayloadEmpty(payload) {
1189
- const key = payload.toString();
1190
- const cached = this.isPayloadEmptyCache.get(key);
1191
- if (cached) {
1192
- return cached;
1193
- }
1194
- const isEmpty = !await this.l1TxUtils.getCode(payload);
1195
- this.isPayloadEmptyCache.set(key, isEmpty);
1196
- return isEmpty;
1197
- }
1198
1368
  /**
1199
1369
  * Enqueues a governance castSignal transaction to cast a signal for a given slot number.
1200
1370
  * @param slotNumber - The slot number to cast a signal for.
@@ -1202,6 +1372,56 @@ export class SequencerPublisher {
1202
1372
  */ enqueueGovernanceCastSignal(governancePayload, slotNumber, signerAddress, signer) {
1203
1373
  return this.enqueueCastSignalHelper(slotNumber, 'governance-signal', governancePayload, this.govProposerContract, signerAddress, signer);
1204
1374
  }
1375
+ /**
1376
+ * Enqueues a `prune()` transaction if the rollup is prunable at the given slot's L1 timestamp.
1377
+ * `prune()` is permissionless and idempotent — if the chain is no longer prunable by send time the
1378
+ * bundle simulation usually drops the entry; on a node without `eth_simulateV1` the bundle is sent
1379
+ * as-is and the prune reverts `Rollup__NothingToPrune` inside `aggregate3(allowFailure: true)`
1380
+ * (a failed action, never a whole-tx revert). Used by the failed-sync fallback so a stuck pending
1381
+ * chain (e.g. bad data blocking sync) can be wound back to recover.
1382
+ * @returns true if a prune request was enqueued, false otherwise.
1383
+ */ async enqueuePruneIfPrunable(slotNumber) {
1384
+ if (this.lastActions['prune'] === slotNumber) {
1385
+ this.log.debug(`Skipping duplicate prune for slot ${slotNumber}`, {
1386
+ slotNumber
1387
+ });
1388
+ return false;
1389
+ }
1390
+ // Use the SAME timestamp the bundle simulator overrides block.timestamp with at send time
1391
+ // (sequencer-bundle-simulator.ts) so this upfront check and the send-time sim agree. Slot-start
1392
+ // and last-L1-slot both fall within the same L2 slot (and epoch, which is what `canPruneAtTime`
1393
+ // derives), so they agree today; matching the simulator keeps it robust if the contract ever uses
1394
+ // the timestamp more granularly.
1395
+ const ts = getLastL1SlotTimestampForL2Slot(slotNumber, this.epochCache.getL1Constants());
1396
+ const canPrune = await this.rollupContract.canPruneAtTime(ts).catch((err)=>{
1397
+ this.log.error(`Failed to check canPruneAtTime for slot ${slotNumber}`, err, {
1398
+ slotNumber
1399
+ });
1400
+ return false;
1401
+ });
1402
+ if (!canPrune) {
1403
+ this.log.debug(`Rollup not prunable at slot ${slotNumber}`, {
1404
+ slotNumber
1405
+ });
1406
+ return false;
1407
+ }
1408
+ const request = {
1409
+ to: this.rollupContract.address,
1410
+ data: encodeFunctionData({
1411
+ abi: RollupAbi,
1412
+ functionName: 'prune',
1413
+ args: []
1414
+ })
1415
+ };
1416
+ this.log.info(`Enqueuing rollup prune for slot ${slotNumber}`, {
1417
+ slotNumber
1418
+ });
1419
+ return this.enqueueRequest('prune', request, {
1420
+ address: this.rollupContract.address,
1421
+ abi: RollupAbi,
1422
+ eventName: 'PrunedPending'
1423
+ }, slotNumber);
1424
+ }
1205
1425
  /** Enqueues all slashing actions as returned by the slasher client. */ async enqueueSlashingActions(actions, slotNumber, signerAddress, signer) {
1206
1426
  if (actions.length === 0) {
1207
1427
  this.log.debug(`No slashing actions to enqueue for slot ${slotNumber}`);
@@ -1209,43 +1429,6 @@ export class SequencerPublisher {
1209
1429
  }
1210
1430
  for (const action of actions){
1211
1431
  switch(action.type){
1212
- case 'vote-empire-payload':
1213
- {
1214
- if (this.slashingProposerContract?.type !== 'empire') {
1215
- this.log.error('Cannot vote for empire payload on non-empire slashing contract');
1216
- break;
1217
- }
1218
- this.log.debug(`Enqueuing slashing vote for payload ${action.payload} at slot ${slotNumber}`, {
1219
- signerAddress
1220
- });
1221
- await this.enqueueCastSignalHelper(slotNumber, 'empire-slashing-signal', action.payload, this.slashingProposerContract, signerAddress, signer);
1222
- break;
1223
- }
1224
- case 'create-empire-payload':
1225
- {
1226
- this.log.debug(`Enqueuing slashing create payload at slot ${slotNumber}`, {
1227
- slotNumber,
1228
- signerAddress
1229
- });
1230
- const request = this.slashFactoryContract.buildCreatePayloadRequest(action.data);
1231
- await this.simulateAndEnqueueRequest('create-empire-payload', request, (receipt)=>!!this.slashFactoryContract.tryExtractSlashPayloadCreatedEvent(receipt.logs), slotNumber);
1232
- break;
1233
- }
1234
- case 'execute-empire-payload':
1235
- {
1236
- this.log.debug(`Enqueuing slashing execute payload at slot ${slotNumber}`, {
1237
- slotNumber,
1238
- signerAddress
1239
- });
1240
- if (this.slashingProposerContract?.type !== 'empire') {
1241
- this.log.error('Cannot execute slashing payload on non-empire slashing contract');
1242
- return false;
1243
- }
1244
- const empireSlashingProposer = this.slashingProposerContract;
1245
- const request = empireSlashingProposer.buildExecuteRoundRequest(action.round);
1246
- await this.simulateAndEnqueueRequest('execute-empire-payload', request, (receipt)=>!!empireSlashingProposer.tryExtractPayloadSubmittedEvent(receipt.logs), slotNumber);
1247
- break;
1248
- }
1249
1432
  case 'vote-offenses':
1250
1433
  {
1251
1434
  this.log.debug(`Enqueuing slashing vote for ${action.votes.length} votes at slot ${slotNumber}`, {
@@ -1254,14 +1437,17 @@ export class SequencerPublisher {
1254
1437
  votesCount: action.votes.length,
1255
1438
  signerAddress
1256
1439
  });
1257
- if (this.slashingProposerContract?.type !== 'tally') {
1258
- this.log.error('Cannot vote for slashing offenses on non-tally slashing contract');
1440
+ if (!this.slashingProposerContract) {
1441
+ this.log.error('No slashing proposer contract available');
1259
1442
  return false;
1260
1443
  }
1261
- const tallySlashingProposer = this.slashingProposerContract;
1262
1444
  const votes = bufferToHex(encodeSlashConsensusVotes(action.votes));
1263
- const request = await tallySlashingProposer.buildVoteRequestFromSigner(votes, slotNumber, signer);
1264
- await this.simulateAndEnqueueRequest('vote-offenses', request, (receipt)=>!!tallySlashingProposer.tryExtractVoteCastEvent(receipt.logs), slotNumber);
1445
+ const request = await this.slashingProposerContract.buildVoteRequestFromSigner(votes, slotNumber, signer);
1446
+ this.enqueueRequest('vote-offenses', request, {
1447
+ address: this.slashingProposerContract.address.toString(),
1448
+ abi: SlashingProposerAbi,
1449
+ eventName: 'VoteCast'
1450
+ }, slotNumber);
1265
1451
  break;
1266
1452
  }
1267
1453
  case 'execute-slash':
@@ -1271,13 +1457,16 @@ export class SequencerPublisher {
1271
1457
  round: action.round,
1272
1458
  signerAddress
1273
1459
  });
1274
- if (this.slashingProposerContract?.type !== 'tally') {
1275
- this.log.error('Cannot execute slashing offenses on non-tally slashing contract');
1460
+ if (!this.slashingProposerContract) {
1461
+ this.log.error('No slashing proposer contract available');
1276
1462
  return false;
1277
1463
  }
1278
- const tallySlashingProposer = this.slashingProposerContract;
1279
- const request = tallySlashingProposer.buildExecuteRoundRequest(action.round, action.committees);
1280
- await this.simulateAndEnqueueRequest('execute-slash', request, (receipt)=>!!tallySlashingProposer.tryExtractRoundExecutedEvent(receipt.logs), slotNumber);
1464
+ const executeRequest = this.slashingProposerContract.buildExecuteRoundRequest(action.round, action.committees);
1465
+ this.enqueueRequest('execute-slash', executeRequest, {
1466
+ address: this.slashingProposerContract.address.toString(),
1467
+ abi: SlashingProposerAbi,
1468
+ eventName: 'RoundExecuted'
1469
+ }, slotNumber);
1281
1470
  break;
1282
1471
  }
1283
1472
  default:
@@ -1289,7 +1478,7 @@ export class SequencerPublisher {
1289
1478
  }
1290
1479
  return true;
1291
1480
  }
1292
- /** Simulates and enqueues a proposal for a checkpoint on L1 */ async enqueueProposeCheckpoint(checkpoint, attestationsAndSigners, attestationsAndSignersSignature, opts = {}) {
1481
+ /** Enqueues a proposal for a checkpoint on L1 */ async enqueueProposeCheckpoint(checkpoint, attestationsAndSigners, attestationsAndSignersSignature, opts = {}) {
1293
1482
  const checkpointHeader = checkpoint.header;
1294
1483
  const blobFields = checkpoint.toBlobFields();
1295
1484
  const blobs = await getBlobsPerL1Block(blobFields);
@@ -1301,51 +1490,38 @@ export class SequencerPublisher {
1301
1490
  attestationsAndSignersSignature,
1302
1491
  feeAssetPriceModifier: checkpoint.feeAssetPriceModifier
1303
1492
  };
1304
- try {
1305
- // @note This will make sure that we are passing the checks for our header ASSUMING that the data is also made available
1306
- // This means that we can avoid the simulation issues in later checks.
1307
- // By simulation issue, I mean the fact that the block.timestamp is equal to the last block, not the next, which
1308
- // make time consistency checks break.
1309
- // TODO(palla): Check whether we're validating twice, once here and once within addProposeTx, since we call simulateProposeTx in both places.
1310
- await this.validateCheckpointForSubmission(checkpoint, attestationsAndSigners, attestationsAndSignersSignature, opts);
1311
- } catch (err) {
1312
- this.log.error(`Checkpoint validation failed. ${err instanceof Error ? err.message : 'No error message'}`, err, {
1313
- ...checkpoint.getStats(),
1314
- slotNumber: checkpoint.header.slotNumber,
1315
- forcePendingCheckpointNumber: opts.forcePendingCheckpointNumber
1316
- });
1317
- throw err;
1318
- }
1319
1493
  this.log.verbose(`Enqueuing checkpoint propose transaction`, {
1320
1494
  ...checkpoint.toCheckpointInfo(),
1321
- ...opts
1495
+ txTimeoutAt: opts.txTimeoutAt
1496
+ });
1497
+ await this.addProposeTx(checkpoint, proposeTxArgs, {
1498
+ txTimeoutAt: opts.txTimeoutAt
1322
1499
  });
1323
- await this.addProposeTx(checkpoint, proposeTxArgs, opts);
1324
1500
  }
1325
1501
  enqueueInvalidateCheckpoint(request, opts = {}) {
1326
1502
  if (!request) {
1327
1503
  return;
1328
1504
  }
1329
- // We issued the simulation against the rollup contract, so we need to account for the overhead of the multicall3
1330
- const gasLimit = this.l1TxUtils.bumpGasLimit(BigInt(Math.ceil(Number(request.gasUsed) * 64 / 63)));
1331
1505
  const { gasUsed, checkpointNumber } = request;
1332
1506
  const logData = {
1333
1507
  gasUsed,
1334
1508
  checkpointNumber,
1335
- gasLimit,
1336
1509
  opts
1337
1510
  };
1338
1511
  this.log.verbose(`Enqueuing invalidate checkpoint request`, logData);
1339
1512
  this.addRequest({
1340
1513
  action: `invalidate-by-${request.reason}`,
1341
1514
  request: request.request,
1342
- gasConfig: {
1343
- gasLimit,
1515
+ gasConfig: opts.txTimeoutAt ? {
1344
1516
  txTimeoutAt: opts.txTimeoutAt
1345
- },
1517
+ } : undefined,
1346
1518
  lastValidL2Slot: SlotNumber(this.getCurrentL2Slot() + 2),
1347
1519
  checkSuccess: (_req, result)=>{
1348
- const success = result && result.receipt && result.receipt.status === 'success' && tryExtractEvent(result.receipt.logs, this.rollupContract.address, RollupAbi, 'CheckpointInvalidated');
1520
+ const success = result && extractEventSuccess(result.receipt, {
1521
+ address: this.rollupContract.address,
1522
+ abi: RollupAbi,
1523
+ eventName: 'CheckpointInvalidated'
1524
+ });
1349
1525
  if (!success) {
1350
1526
  this.log.warn(`Invalidate checkpoint ${request.checkpointNumber} failed`, {
1351
1527
  ...result,
@@ -1361,90 +1537,36 @@ export class SequencerPublisher {
1361
1537
  }
1362
1538
  });
1363
1539
  }
1364
- async simulateAndEnqueueRequest(action, request, checkSuccess, slotNumber) {
1365
- const timestamp = this.getSimulationTimestamp(slotNumber);
1366
- const logData = {
1367
- slotNumber,
1368
- timestamp,
1369
- gasLimit: undefined
1370
- };
1540
+ /**
1541
+ * Dedup-checked enqueue helper for actions that are simulated at bundle-send time rather
1542
+ * than at enqueue time. Validates the (action, slot) dedup key, sets `lastActions`, and
1543
+ * enqueues without a gasLimit so the bundle simulate sets the only gasLimit that matters.
1544
+ */ enqueueRequest(action, request, eventOpts, slotNumber) {
1371
1545
  if (this.lastActions[action] && this.lastActions[action] === slotNumber) {
1372
1546
  this.log.debug(`Skipping duplicate action ${action} for slot ${slotNumber}`);
1373
1547
  return false;
1374
1548
  }
1375
1549
  const cachedLastActionSlot = this.lastActions[action];
1376
1550
  this.lastActions[action] = slotNumber;
1377
- this.log.debug(`Simulating ${action} for slot ${slotNumber}`, logData);
1378
- const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
1379
- let gasUsed;
1380
- const simulateAbi = mergeAbis([
1381
- request.abi ?? [],
1382
- ErrorsAbi
1383
- ]);
1384
- try {
1385
- ({ gasUsed } = await this.l1TxUtils.simulate(request, {
1386
- time: timestamp
1387
- }, [], simulateAbi));
1388
- this.log.verbose(`Simulation for ${action} succeeded`, {
1389
- ...logData,
1390
- request,
1391
- gasUsed
1392
- });
1393
- } catch (err) {
1394
- const viemError = formatViemError(err, simulateAbi);
1395
- this.log.error(`Simulation for ${action} at ${slotNumber} failed`, viemError, logData);
1396
- this.backupFailedTx({
1397
- id: keccak256(request.data),
1398
- failureType: 'simulation',
1399
- request: {
1400
- to: request.to,
1401
- data: request.data,
1402
- value: request.value?.toString()
1403
- },
1404
- l1BlockNumber: l1BlockNumber.toString(),
1405
- error: {
1406
- message: viemError.message,
1407
- name: viemError.name
1408
- },
1409
- context: {
1410
- actions: [
1411
- action
1412
- ],
1413
- slot: slotNumber,
1414
- sender: this.getSenderAddress().toString()
1415
- }
1416
- });
1417
- return false;
1418
- }
1419
- // We issued the simulation against the rollup contract, so we need to account for the overhead of the multicall3
1420
- const gasLimit = this.l1TxUtils.bumpGasLimit(BigInt(Math.ceil(Number(gasUsed) * 64 / 63)));
1421
- logData.gasLimit = gasLimit;
1422
- // Store the ABI used for simulation on the request so Multicall3.forward can decode errors
1423
- // when the tx is sent and a revert is diagnosed via simulation.
1424
- const requestWithAbi = {
1425
- ...request,
1426
- abi: simulateAbi
1427
- };
1428
- this.log.debug(`Enqueuing ${action}`, logData);
1551
+ this.log.debug(`Enqueuing ${action}`, {
1552
+ slotNumber
1553
+ });
1429
1554
  this.addRequest({
1430
1555
  action,
1431
- request: requestWithAbi,
1432
- gasConfig: {
1433
- gasLimit
1434
- },
1556
+ request,
1435
1557
  lastValidL2Slot: slotNumber,
1436
- checkSuccess: (_req, result)=>{
1437
- const success = result && result.receipt && result.receipt.status === 'success' && checkSuccess(result.receipt);
1558
+ checkSuccess: (_request, result)=>{
1559
+ const success = result && extractEventSuccess(result.receipt, eventOpts);
1438
1560
  if (!success) {
1439
1561
  this.log.warn(`Action ${action} at ${slotNumber} failed`, {
1440
1562
  ...result,
1441
- ...logData
1563
+ slotNumber
1442
1564
  });
1443
1565
  this.lastActions[action] = cachedLastActionSlot;
1444
1566
  } else {
1445
1567
  this.log.info(`Action ${action} at ${slotNumber} succeeded`, {
1446
1568
  ...result,
1447
- ...logData
1569
+ slotNumber
1448
1570
  });
1449
1571
  }
1450
1572
  return !!success;
@@ -1466,7 +1588,7 @@ export class SequencerPublisher {
1466
1588
  this.interrupted = false;
1467
1589
  this.l1TxUtils.restart();
1468
1590
  }
1469
- async prepareProposeTx(encodedData, options) {
1591
+ async prepareProposeTx(encodedData) {
1470
1592
  const kzg = Blob.getViemKzgInstance();
1471
1593
  const blobInput = getPrefixedEthBlobCommitments(encodedData.blobs);
1472
1594
  this.log.debug('Validating blob input', {
@@ -1480,7 +1602,11 @@ export class SequencerPublisher {
1480
1602
  blobEvaluationGas = BigInt(encodedData.blobs.length) * 21_000n;
1481
1603
  this.log.debug(`Using fixed blob evaluation gas estimate in fisherman mode: ${blobEvaluationGas}`);
1482
1604
  } else {
1483
- // Normal mode - use estimateGas with blob inputs
1605
+ // We call validateBlobs via estimateGas with real blob+kzg sidecars as a consistency check
1606
+ // that our locally-built blob commitments match the blob data. The bundle simulate at send
1607
+ // time uses eth_simulateV1, which cannot carry blob inputs, so the rollup's on-chain blob
1608
+ // check is forced off there — making this the only pre-flight detector of a commitment/data
1609
+ // mismatch. The returned gas estimate is stashed on the request for the bundle path to read.
1484
1610
  blobEvaluationGas = await this.l1TxUtils.estimateGas(this.getSenderAddress().toString(), {
1485
1611
  to: this.rollupContract.address,
1486
1612
  data: encodeFunctionData({
@@ -1514,7 +1640,7 @@ export class SequencerPublisher {
1514
1640
  data: validateBlobsData
1515
1641
  },
1516
1642
  blobData: encodedData.blobs.map((b)=>toHex(b.data)),
1517
- l1BlockNumber: l1BlockNumber.toString(),
1643
+ l1BlockNumber,
1518
1644
  error: {
1519
1645
  message: viemError.message,
1520
1646
  name: viemError.name
@@ -1524,7 +1650,11 @@ export class SequencerPublisher {
1524
1650
  'validate-blobs'
1525
1651
  ],
1526
1652
  sender: this.getSenderAddress().toString()
1527
- }
1653
+ },
1654
+ timing: this.computeTimingInfo(this.getCurrentL2Slot())
1655
+ }, {
1656
+ captureFeeSummary: true,
1657
+ targetSlot: this.getCurrentL2Slot()
1528
1658
  });
1529
1659
  throw new Error('Failed to validate blobs');
1530
1660
  });
@@ -1543,113 +1673,23 @@ export class SequencerPublisher {
1543
1673
  encodedData.attestationsAndSignersSignature.toViemSignature(),
1544
1674
  blobInput
1545
1675
  ];
1546
- const { rollupData, simulationResult } = await this.simulateProposeTx(args, options);
1547
- return {
1548
- args,
1549
- blobEvaluationGas,
1550
- rollupData,
1551
- simulationResult
1552
- };
1553
- }
1554
- /**
1555
- * Simulates the propose tx with eth_simulateV1
1556
- * @param args - The propose tx args
1557
- * @returns The simulation result
1558
- */ async simulateProposeTx(args, options) {
1559
1676
  const rollupData = encodeFunctionData({
1560
1677
  abi: RollupAbi,
1561
1678
  functionName: 'propose',
1562
1679
  args
1563
1680
  });
1564
- // override the proposed checkpoint number if requested
1565
- const forcePendingCheckpointNumberStateDiff = (options.forcePendingCheckpointNumber !== undefined ? await this.rollupContract.makePendingCheckpointNumberOverride(options.forcePendingCheckpointNumber) : []).flatMap((override)=>override.stateDiff ?? []);
1566
- // override the fee header for a specific checkpoint number if requested (used when pipelining)
1567
- const forceProposedFeeHeaderStateDiff = (options.forceProposedFeeHeader !== undefined ? await this.rollupContract.makeFeeHeaderOverride(options.forceProposedFeeHeader.checkpointNumber, options.forceProposedFeeHeader.feeHeader) : []).flatMap((override)=>override.stateDiff ?? []);
1568
- const stateOverrides = [
1569
- {
1570
- address: this.rollupContract.address,
1571
- // @note we override checkBlob to false since blobs are not part simulate()
1572
- stateDiff: [
1573
- {
1574
- slot: toPaddedHex(RollupContract.checkBlobStorageSlot, true),
1575
- value: toPaddedHex(0n, true)
1576
- },
1577
- ...forcePendingCheckpointNumberStateDiff,
1578
- ...forceProposedFeeHeaderStateDiff
1579
- ]
1580
- }
1581
- ];
1582
- // In fisherman mode, simulate as the proposer but with sufficient balance
1583
- if (this.proposerAddressForSimulation) {
1584
- stateOverrides.push({
1585
- address: this.proposerAddressForSimulation.toString(),
1586
- balance: 10n * WEI_CONST * WEI_CONST
1587
- });
1588
- }
1589
- const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
1590
- const simTs = this.getSimulationTimestamp(SlotNumber.fromBigInt(args[0].header.slotNumber));
1591
- const simulationResult = await this.l1TxUtils.simulate({
1592
- to: this.rollupContract.address,
1593
- data: rollupData,
1594
- gas: MAX_L1_TX_LIMIT,
1595
- ...this.proposerAddressForSimulation && {
1596
- from: this.proposerAddressForSimulation.toString()
1597
- }
1598
- }, {
1599
- time: simTs,
1600
- // @note reth should have a 30m gas limit per block but throws errors that this tx is beyond limit so we increase here
1601
- gasLimit: MAX_L1_TX_LIMIT * 2n
1602
- }, stateOverrides, RollupAbi, {
1603
- // @note fallback gas estimate to use if the node doesn't support simulation API
1604
- fallbackGasEstimate: MAX_L1_TX_LIMIT
1605
- }).catch((err)=>{
1606
- // In fisherman mode, we expect ValidatorSelection__MissingProposerSignature since fisherman doesn't have proposer signature
1607
- const viemError = formatViemError(err);
1608
- if (this.config.fishermanMode && viemError.message?.includes('ValidatorSelection__MissingProposerSignature')) {
1609
- this.log.debug(`Ignoring expected ValidatorSelection__MissingProposerSignature error in fisherman mode`);
1610
- // Return a minimal simulation result with the fallback gas estimate
1611
- return {
1612
- gasUsed: MAX_L1_TX_LIMIT,
1613
- logs: []
1614
- };
1615
- }
1616
- this.log.error(`Failed to simulate propose tx`, viemError, {
1617
- simulationTimestamp: simTs
1618
- });
1619
- this.backupFailedTx({
1620
- id: keccak256(rollupData),
1621
- failureType: 'simulation',
1622
- request: {
1623
- to: this.rollupContract.address,
1624
- data: rollupData
1625
- },
1626
- l1BlockNumber: l1BlockNumber.toString(),
1627
- error: {
1628
- message: viemError.message,
1629
- name: viemError.name
1630
- },
1631
- context: {
1632
- actions: [
1633
- 'propose'
1634
- ],
1635
- slot: Number(args[0].header.slotNumber),
1636
- sender: this.getSenderAddress().toString()
1637
- }
1638
- });
1639
- throw err;
1640
- });
1641
1681
  return {
1642
- rollupData,
1643
- simulationResult
1682
+ args,
1683
+ blobEvaluationGas,
1684
+ rollupData
1644
1685
  };
1645
1686
  }
1646
1687
  async addProposeTx(checkpoint, encodedData, opts = {}) {
1647
1688
  const slot = checkpoint.header.slotNumber;
1648
1689
  const timer = new Timer();
1649
1690
  const kzg = Blob.getViemKzgInstance();
1650
- const { rollupData, simulationResult, blobEvaluationGas } = await this.prepareProposeTx(encodedData, opts);
1691
+ const { rollupData, blobEvaluationGas } = await this.prepareProposeTx(encodedData);
1651
1692
  const startBlock = await this.l1TxUtils.getBlockNumber();
1652
- const gasLimit = this.l1TxUtils.bumpGasLimit(BigInt(Math.ceil(Number(simulationResult.gasUsed) * 64 / 63)) + blobEvaluationGas + SequencerPublisher.MULTICALL_OVERHEAD_GAS_GUESS);
1653
1693
  // Send the blobs to the blob client preemptively. This helps in tests where the sequencer mistakingly thinks that the propose
1654
1694
  // tx fails but it does get mined. We make sure that the blobs are sent to the blob client regardless of the tx outcome.
1655
1695
  void Promise.resolve().then(()=>this.blobClient.sendBlobsToFilestore(encodedData.blobs).catch((_err)=>{
@@ -1663,9 +1703,10 @@ export class SequencerPublisher {
1663
1703
  },
1664
1704
  lastValidL2Slot: checkpoint.header.slotNumber,
1665
1705
  gasConfig: {
1666
- ...opts,
1667
- gasLimit
1706
+ txTimeoutAt: opts.txTimeoutAt,
1707
+ gasLimit: undefined
1668
1708
  },
1709
+ blobEvaluationGas,
1669
1710
  blobConfig: {
1670
1711
  blobs: encodedData.blobs.map((b)=>b.data),
1671
1712
  kzg
@@ -1675,7 +1716,11 @@ export class SequencerPublisher {
1675
1716
  return false;
1676
1717
  }
1677
1718
  const { receipt, stats, errorMsg } = result;
1678
- const success = receipt && receipt.status === 'success' && tryExtractEvent(receipt.logs, this.rollupContract.address, RollupAbi, 'CheckpointProposed');
1719
+ const success = extractEventSuccess(receipt, {
1720
+ address: this.rollupContract.address,
1721
+ abi: RollupAbi,
1722
+ eventName: 'CheckpointProposed'
1723
+ });
1679
1724
  if (success) {
1680
1725
  const endBlock = receipt.blockNumber;
1681
1726
  const inclusionBlocks = Number(endBlock - startBlock);
@@ -1712,11 +1757,6 @@ export class SequencerPublisher {
1712
1757
  }
1713
1758
  });
1714
1759
  }
1715
- /** Returns the timestamp of the last L1 slot within a given L2 slot. Used as the simulation timestamp
1716
- * for eth_simulateV1 calls, since it's guaranteed to be greater than any L1 block produced during the slot. */ getSimulationTimestamp(slot) {
1717
- const l1Constants = this.epochCache.getL1Constants();
1718
- return getLastL1SlotTimestampForL2Slot(slot, l1Constants);
1719
- }
1720
1760
  /** Returns the timestamp of the next L1 slot boundary after now. */ getNextL1SlotTimestamp() {
1721
1761
  const l1Constants = this.epochCache.getL1Constants();
1722
1762
  return getNextL1SlotTimestamp(this.dateProvider.nowInSeconds(), l1Constants);