@aztec/sequencer-client 0.0.1-commit.c2eed6949 → 0.0.1-commit.c52d6e7

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 (122) 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 +12 -4
  6. package/dest/config.d.ts.map +1 -1
  7. package/dest/config.js +61 -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 +101 -69
  42. package/dest/publisher/sequencer-publisher.d.ts.map +1 -1
  43. package/dest/publisher/sequencer-publisher.js +635 -538
  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 +696 -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 +733 -213
  59. package/dest/sequencer/checkpoint_proposal_job_metrics.d.ts +34 -0
  60. package/dest/sequencer/checkpoint_proposal_job_metrics.d.ts.map +1 -0
  61. package/dest/sequencer/checkpoint_proposal_job_metrics.js +72 -0
  62. package/dest/sequencer/checkpoint_voter.d.ts +1 -2
  63. package/dest/sequencer/checkpoint_voter.d.ts.map +1 -1
  64. package/dest/sequencer/checkpoint_voter.js +2 -5
  65. package/dest/sequencer/events.d.ts +61 -5
  66. package/dest/sequencer/events.d.ts.map +1 -1
  67. package/dest/sequencer/metrics.d.ts +9 -10
  68. package/dest/sequencer/metrics.d.ts.map +1 -1
  69. package/dest/sequencer/metrics.js +34 -20
  70. package/dest/sequencer/missing_committee.d.ts +72 -0
  71. package/dest/sequencer/missing_committee.d.ts.map +1 -0
  72. package/dest/sequencer/missing_committee.js +139 -0
  73. package/dest/sequencer/requests_tracker.d.ts +22 -0
  74. package/dest/sequencer/requests_tracker.d.ts.map +1 -0
  75. package/dest/sequencer/requests_tracker.js +33 -0
  76. package/dest/sequencer/sequencer.d.ts +158 -35
  77. package/dest/sequencer/sequencer.d.ts.map +1 -1
  78. package/dest/sequencer/sequencer.js +556 -162
  79. package/dest/sequencer/types.d.ts +2 -2
  80. package/dest/sequencer/types.d.ts.map +1 -1
  81. package/dest/test/index.d.ts +3 -3
  82. package/dest/test/index.d.ts.map +1 -1
  83. package/dest/test/utils.d.ts +15 -1
  84. package/dest/test/utils.d.ts.map +1 -1
  85. package/dest/test/utils.js +25 -7
  86. package/package.json +29 -28
  87. package/src/client/sequencer-client.ts +37 -27
  88. package/src/config.ts +71 -26
  89. package/src/global_variable_builder/README.md +44 -0
  90. package/src/global_variable_builder/fee_predictor.ts +182 -0
  91. package/src/global_variable_builder/fee_provider.ts +97 -0
  92. package/src/global_variable_builder/global_builder.ts +12 -80
  93. package/src/global_variable_builder/index.ts +2 -0
  94. package/src/publisher/config.ts +30 -7
  95. package/src/publisher/l1_to_l2_messaging.ts +85 -0
  96. package/src/publisher/l1_tx_failed_store/failed_tx_store.ts +117 -8
  97. package/src/publisher/l1_tx_failed_store/file_store_failed_tx_store.ts +4 -3
  98. package/src/publisher/l1_tx_failed_store/index.ts +1 -1
  99. package/src/publisher/sequencer-bundle-simulator.ts +254 -0
  100. package/src/publisher/sequencer-publisher-factory.ts +0 -3
  101. package/src/publisher/sequencer-publisher.ts +735 -612
  102. package/src/publisher/write_json.ts +78 -0
  103. package/src/sequencer/README.md +162 -450
  104. package/src/sequencer/automine/README.md +60 -0
  105. package/src/sequencer/automine/automine_factory.ts +152 -0
  106. package/src/sequencer/automine/automine_sequencer.ts +800 -0
  107. package/src/sequencer/automine/index.ts +6 -0
  108. package/src/sequencer/checkpoint_proposal_job.ts +849 -242
  109. package/src/sequencer/checkpoint_proposal_job_metrics.ts +128 -0
  110. package/src/sequencer/checkpoint_voter.ts +1 -12
  111. package/src/sequencer/events.ts +66 -5
  112. package/src/sequencer/metrics.ts +43 -24
  113. package/src/sequencer/missing_committee.ts +192 -0
  114. package/src/sequencer/requests_tracker.ts +43 -0
  115. package/src/sequencer/sequencer.ts +639 -180
  116. package/src/sequencer/types.ts +1 -1
  117. package/src/test/index.ts +2 -2
  118. package/src/test/utils.ts +61 -10
  119. package/dest/sequencer/timetable.d.ts +0 -88
  120. package/dest/sequencer/timetable.d.ts.map +0 -1
  121. package/dest/sequencer/timetable.js +0 -222
  122. 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,31 +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';
386
+ import { InterruptibleSleep } from '@aztec/foundation/sleep';
389
387
  import { bufferToHex } from '@aztec/foundation/string';
390
388
  import { Timer } from '@aztec/foundation/timer';
391
- import { EmpireBaseAbi, ErrorsAbi, RollupAbi } from '@aztec/l1-artifacts';
389
+ import { EmpireBaseAbi, ErrorsAbi, RollupAbi, SlashingProposerAbi } from '@aztec/l1-artifacts';
392
390
  import { encodeSlashConsensusVotes } from '@aztec/slasher';
393
391
  import { CommitteeAttestationsAndSigners } from '@aztec/stdlib/block';
394
- import { getNextL1SlotTimestamp } from '@aztec/stdlib/epoch-helpers';
392
+ import { getLastL1SlotTimestampForL2Slot, getNextL1SlotTimestamp, getTimestampForSlot } from '@aztec/stdlib/epoch-helpers';
395
393
  import { getTelemetryClient, trackSpan } from '@aztec/telemetry-client';
396
- import { encodeFunctionData, keccak256, multicall3Abi, toHex } from 'viem';
394
+ import { encodeFunctionData, keccak256, toHex } from 'viem';
397
395
  import { createL1TxFailedStore } from './l1_tx_failed_store/index.js';
396
+ import { SequencerBundleSimulator } from './sequencer-bundle-simulator.js';
398
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
+ }
399
408
  export const Actions = [
400
409
  'invalidate-by-invalid-attestation',
401
410
  'invalidate-by-insufficient-attestations',
411
+ 'prune',
402
412
  'propose',
403
413
  'governance-signal',
404
- 'empire-slashing-signal',
405
- 'create-empire-payload',
406
- 'execute-empire-payload',
407
414
  'vote-offenses',
408
415
  'execute-slash'
409
416
  ];
410
417
  // Sorting for actions such that invalidations go before proposals, and proposals go before votes
411
418
  export const compareActions = (a, b)=>Actions.indexOf(a) - Actions.indexOf(b);
412
- _dec = trackSpan('SequencerPublisher.sendRequests'), _dec1 = trackSpan('SequencerPublisher.validateBlockHeader'), _dec2 = trackSpan('SequencerPublisher.validateCheckpointForSubmission');
419
+ _dec = trackSpan('SequencerPublisher.sendRequests'), _dec1 = trackSpan('SequencerPublisher.validateBlockHeader');
413
420
  export class SequencerPublisher {
414
421
  config;
415
422
  static{
@@ -423,59 +430,59 @@ export class SequencerPublisher {
423
430
  _dec1,
424
431
  2,
425
432
  "validateBlockHeader"
426
- ],
427
- [
428
- _dec2,
429
- 2,
430
- "validateCheckpointForSubmission"
431
433
  ]
432
434
  ], []));
433
435
  }
434
436
  interrupted;
435
437
  metrics;
438
+ bundleSimulator;
436
439
  epochCache;
437
440
  failedTxStore;
438
- governanceLog;
439
- 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;
440
446
  lastActions;
441
- isPayloadEmptyCache;
442
- payloadProposedCache;
443
447
  log;
444
448
  ethereumSlotDuration;
445
449
  aztecSlotDuration;
446
- dateProvider;
450
+ previousL1BlockWaitTimeoutMs;
451
+ previousL1BlockWaitPollIntervalMs;
452
+ /** Date provider for wall-clock time. */ dateProvider;
447
453
  blobClient;
448
- /** Address to use for simulations in fisherman mode (actual proposer's address) */ proposerAddressForSimulation;
449
454
  /** Optional callback to obtain a replacement publisher when the current one fails to send. */ getNextPublisher;
450
455
  /** L1 fee analyzer for fisherman mode */ l1FeeAnalyzer;
451
456
  /** Fee asset price oracle for computing price modifiers from Uniswap V4 */ feeAssetPriceOracle;
452
- // A CALL to a cold address is 2700 gas
453
- static MULTICALL_OVERHEAD_GAS_GUESS = 5000n;
454
- // Gas report for VotingWithSigTest shows a max gas of 100k, but we've seen it cost 700k+ in testnet
455
- static VOTE_GAS_GUESS = 800_000n;
457
+ /** Interruptible sleep used by sendRequestsAt to wait until a target timestamp. */ interruptibleSleep;
456
458
  l1TxUtils;
457
459
  rollupContract;
458
460
  govProposerContract;
459
461
  slashingProposerContract;
460
- slashFactoryContract;
461
462
  tracer;
462
463
  requests;
463
464
  constructor(config, deps){
464
465
  this.config = config;
465
466
  this.interrupted = (_initProto(this), false);
466
- this.governanceLog = createLogger('sequencer:publisher:governance');
467
- this.slashingLog = createLogger('sequencer:publisher:slashing');
467
+ this.revertDecoderAbi = mergeAbis([
468
+ RollupAbi,
469
+ SlashingProposerAbi,
470
+ EmpireBaseAbi,
471
+ ErrorsAbi
472
+ ]);
468
473
  this.lastActions = {};
469
- this.isPayloadEmptyCache = new Map();
470
- this.payloadProposedCache = new Set();
474
+ this.interruptibleSleep = new InterruptibleSleep();
471
475
  this.requests = [];
472
476
  this.log = deps.log ?? createLogger('sequencer:publisher');
473
477
  this.ethereumSlotDuration = BigInt(config.ethereumSlotDuration);
474
478
  this.aztecSlotDuration = BigInt(config.aztecSlotDuration);
479
+ this.previousL1BlockWaitTimeoutMs = config.sequencerPublisherPreviousL1BlockWaitTimeoutMs;
480
+ this.previousL1BlockWaitPollIntervalMs = config.sequencerPublisherPreviousL1BlockWaitPollIntervalMs;
475
481
  this.dateProvider = deps.dateProvider;
476
482
  this.epochCache = deps.epochCache;
477
483
  this.lastActions = deps.lastActions;
478
484
  this.blobClient = deps.blobClient;
485
+ this.dateProvider = deps.dateProvider;
479
486
  const telemetry = deps.telemetry ?? getTelemetryClient();
480
487
  this.metrics = deps.metrics ?? new SequencerPublisherMetrics(telemetry, 'SequencerPublisher');
481
488
  this.tracer = telemetry.getTracer('SequencerPublisher');
@@ -489,40 +496,116 @@ export class SequencerPublisher {
489
496
  const newSlashingProposer = await this.rollupContract.getSlashingProposer();
490
497
  this.slashingProposerContract = newSlashingProposer;
491
498
  });
492
- this.slashFactoryContract = deps.slashFactoryContract;
493
499
  // Initialize L1 fee analyzer for fisherman mode
494
500
  if (config.fishermanMode) {
495
- 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'));
496
502
  }
497
503
  // Initialize fee asset price oracle
498
- 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'));
499
505
  // Initialize failed L1 tx store (optional, for test networks)
500
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
+ });
501
513
  }
502
514
  /**
503
- * Backs up a failed L1 transaction to the configured store for debugging.
504
- * Does nothing if no store is configured.
505
- */ backupFailedTx(failedTx) {
506
- if (!this.failedTxStore) {
507
- return;
508
- }
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) {
509
520
  const tx = {
510
521
  ...failedTx,
511
522
  timestamp: Date.now()
512
523
  };
513
524
  // Fire and forget - don't block on backup
514
- void this.failedTxStore.then((store)=>store?.saveFailedTx(tx)).catch((err)=>{
515
- this.log.warn(`Failed to backup failed L1 tx to store`, err);
516
- });
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()}`));
517
596
  }
518
597
  getRollupContract() {
519
598
  return this.rollupContract;
520
599
  }
521
600
  /**
522
601
  * Gets the fee asset price modifier from the oracle.
523
- * Returns 0n if the oracle query fails.
524
- */ getFeeAssetPriceModifier() {
525
- 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);
526
609
  }
527
610
  getSenderAddress() {
528
611
  return this.l1TxUtils.getSenderAddress();
@@ -532,12 +615,6 @@ export class SequencerPublisher {
532
615
  */ getL1FeeAnalyzer() {
533
616
  return this.l1FeeAnalyzer;
534
617
  }
535
- /**
536
- * Sets the proposer address to use for simulations in fisherman mode.
537
- * @param proposerAddress - The actual proposer's address to use for balance lookups in simulations
538
- */ setProposerAddressForSimulation(proposerAddress) {
539
- this.proposerAddressForSimulation = proposerAddress;
540
- }
541
618
  addRequest(request) {
542
619
  this.requests.push(request);
543
620
  }
@@ -595,11 +672,15 @@ export class SequencerPublisher {
595
672
  }
596
673
  /**
597
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.
598
679
  * @returns one of:
599
680
  * - A receipt and stats if the tx succeeded
600
681
  * - a receipt and errorMsg if it failed on L1
601
682
  * - undefined if no valid requests are found OR the tx failed to send.
602
- */ async sendRequests() {
683
+ */ async sendRequests(targetSlot) {
603
684
  const requestsToProcess = [
604
685
  ...this.requests
605
686
  ];
@@ -607,10 +688,9 @@ export class SequencerPublisher {
607
688
  if (this.interrupted || requestsToProcess.length === 0) {
608
689
  return undefined;
609
690
  }
610
- const currentL2Slot = this.getCurrentL2Slot();
691
+ const currentL2Slot = targetSlot ?? this.getCurrentL2Slot();
611
692
  this.log.debug(`Sending requests on L2 slot ${currentL2Slot}`);
612
693
  const validRequests = requestsToProcess.filter((request)=>request.lastValidL2Slot >= currentL2Slot);
613
- const validActions = validRequests.map((x)=>x.action);
614
694
  const expiredActions = requestsToProcess.filter((request)=>request.lastValidL2Slot < currentL2Slot).map((x)=>x.action);
615
695
  if (validRequests.length !== requestsToProcess.length) {
616
696
  this.log.warn(`Some requests were expired for slot ${currentL2Slot}`, {
@@ -628,75 +708,98 @@ export class SequencerPublisher {
628
708
  this.log.debug(`No valid requests to send`);
629
709
  return undefined;
630
710
  }
631
- // @note - we can only have one blob config per bundle
632
- // find requests with gas and blob configs
633
- // See https://github.com/AztecProtocol/aztec-packages/issues/11513
711
+ // Collect earliest txTimeoutAt across all requests.
634
712
  const gasConfigs = validRequests.filter((request)=>request.gasConfig).map((request)=>request.gasConfig);
635
- const blobConfigs = validRequests.filter((request)=>request.blobConfig).map((request)=>request.blobConfig);
636
- if (blobConfigs.length > 1) {
637
- throw new Error('Multiple blob configs found');
638
- }
639
- const blobConfig = blobConfigs[0];
640
- // Merge gasConfigs. Yields the sum of gasLimits, and the earliest txTimeoutAt, or undefined if no gasConfig sets them.
641
- const gasLimits = gasConfigs.map((g)=>g?.gasLimit).filter((g)=>g !== undefined);
642
- let gasLimit = gasLimits.length > 0 ? sumBigint(gasLimits) : undefined; // sum
643
- // Cap at L1 block gas limit so the node accepts the tx ("gas limit too high" otherwise).
644
- const maxGas = MAX_L1_TX_LIMIT;
645
- if (gasLimit !== undefined && gasLimit > maxGas) {
646
- this.log.debug('Capping bundled tx gas limit to L1 max', {
647
- requested: gasLimit,
648
- capped: maxGas
649
- });
650
- gasLimit = maxGas;
651
- }
652
713
  const txTimeoutAts = gasConfigs.map((g)=>g?.txTimeoutAt).filter((g)=>g !== undefined);
653
- const txTimeoutAt = txTimeoutAts.length > 0 ? new Date(Math.min(...txTimeoutAts.map((g)=>g.getTime()))) : undefined; // earliest
654
- const txConfig = {
655
- gasLimit,
656
- txTimeoutAt
657
- };
714
+ const txTimeoutAt = txTimeoutAts.length > 0 ? new Date(Math.min(...txTimeoutAts.map((g)=>g.getTime()))) : undefined;
658
715
  // Sort the requests so that proposals always go first
659
716
  // This ensures the committee gets precomputed correctly
660
717
  validRequests.sort((a, b)=>compareActions(a.action, b.action));
661
718
  try {
662
- // Capture context for failed tx backup before sending
663
- const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
664
- const multicallData = encodeFunctionData({
665
- abi: multicall3Abi,
666
- functionName: 'aggregate3',
667
- args: [
668
- validRequests.map((r)=>({
669
- target: r.request.to,
670
- callData: r.request.data,
671
- allowFailure: true
672
- }))
673
- ]
674
- });
675
- const blobDataHex = blobConfig?.blobs?.map((b)=>toHex(b));
676
- const txContext = {
677
- multicallData,
678
- blobData: blobDataHex,
679
- 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
680
738
  };
681
739
  this.log.debug('Forwarding transactions', {
682
- validRequests: validRequests.map((request)=>request.action),
740
+ requests: requests.map((request)=>request.action),
683
741
  txConfig
684
742
  });
685
- const result = await this.forwardWithPublisherRotation(validRequests, txConfig, blobConfig);
743
+ const result = await this.forwardWithPublisherRotation(requests, txConfig, blobConfig, currentL2Slot);
686
744
  if (result === undefined) {
687
745
  return undefined;
688
746
  }
689
- 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
+ ];
690
752
  return {
691
753
  result,
692
754
  expiredActions,
693
- sentActions: validActions,
755
+ sentActions: requests.map((x)=>x.action),
694
756
  successfulActions,
695
- failedActions
757
+ failedActions: allFailedActions
696
758
  };
697
759
  } catch (err) {
698
760
  const viemError = formatViemError(err);
699
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
+ }
700
803
  return undefined;
701
804
  } finally{
702
805
  try {
@@ -706,26 +809,96 @@ export class SequencerPublisher {
706
809
  }
707
810
  }
708
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
+ }
709
862
  /**
710
863
  * Forwards transactions via Multicall3, rotating to the next available publisher if a send
711
864
  * failure occurs (i.e. the tx never reached the chain).
712
865
  * On-chain reverts and simulation errors are returned as-is without rotation.
713
- */ 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;
714
871
  const triedAddresses = [];
715
872
  let currentPublisher = this.l1TxUtils;
716
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
+ }
717
880
  triedAddresses.push(currentPublisher.getSenderAddress());
718
881
  try {
719
- 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
+ });
720
885
  this.l1TxUtils = currentPublisher;
721
886
  return result;
722
887
  } catch (err) {
723
888
  if (err instanceof TimeoutError) {
724
889
  throw err;
725
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
+ }
726
898
  const viemError = formatViemError(err);
727
899
  if (!this.getNextPublisher) {
728
900
  this.log.error('Failed to publish bundled transactions', viemError);
901
+ this.backupSendFailure(validRequests, viemError, currentPublisher, targetSlot);
729
902
  return undefined;
730
903
  }
731
904
  this.log.warn(`Publisher ${currentPublisher.getSenderAddress()} failed to send, rotating to next publisher`, viemError);
@@ -733,122 +906,205 @@ export class SequencerPublisher {
733
906
  ...triedAddresses
734
907
  ]);
735
908
  if (!nextPublisher) {
736
- 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);
737
913
  return undefined;
738
914
  }
739
915
  currentPublisher = nextPublisher;
740
916
  }
741
917
  }
742
918
  }
743
- callbackBundledTransactions(requests, result, txContext) {
744
- const actionsListStr = requests.map((r)=>r.action).join(', ');
745
- if (result instanceof FormattedViemError) {
746
- this.log.error(`Failed to publish bundled transactions (${actionsListStr})`, result);
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
+ });
958
+ }
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)=>{
747
962
  this.backupFailedTx({
748
- id: keccak256(txContext.multicallData),
963
+ id: this.failureRecordId(requests.map((r)=>r.action), targetSlot),
749
964
  failureType: 'send-error',
750
965
  request: {
751
966
  to: MULTI_CALL_3_ADDRESS,
752
- data: txContext.multicallData
967
+ data: '0x'
753
968
  },
754
- blobData: txContext.blobData,
755
- l1BlockNumber: txContext.l1BlockNumber.toString(),
969
+ l1BlockNumber,
756
970
  error: {
757
- message: result.message,
758
- name: result.name
971
+ message: error.message,
972
+ name: 'name' in error ? error.name : undefined
759
973
  },
760
974
  context: {
761
975
  actions: requests.map((r)=>r.action),
762
- requests: requests.map((r)=>({
976
+ requests: requests.filter((r)=>r.request.to !== null).map((r)=>({
763
977
  action: r.action,
764
978
  to: r.request.to,
765
979
  data: r.request.data
766
980
  })),
767
- sender: this.getSenderAddress().toString()
768
- }
769
- });
770
- return {
771
- failedActions: requests.map((r)=>r.action)
772
- };
773
- } else {
774
- this.log.verbose(`Published bundled transactions (${actionsListStr})`, {
775
- result,
776
- requests: requests.map((r)=>({
777
- ...r,
778
- // Avoid logging large blob data
779
- blobConfig: r.blobConfig ? {
780
- ...r.blobConfig,
781
- blobs: r.blobConfig.blobs.map((b)=>({
782
- size: trimmedBytesLength(b)
783
- }))
784
- } : undefined
785
- }))
981
+ sender: publisher.getSenderAddress().toString(),
982
+ slot: targetSlot !== undefined ? Number(targetSlot) : undefined
983
+ },
984
+ timing: this.computeTimingInfo(targetSlot)
985
+ }, {
986
+ captureFeeSummary: true,
987
+ targetSlot
786
988
  });
787
- const successfulActions = [];
788
- const failedActions = [];
789
- for (const request of requests){
790
- if (request.checkSuccess(request.request, result)) {
791
- successfulActions.push(request.action);
792
- } else {
793
- 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;
794
1039
  }
795
- }
796
- // Single backup for the whole reverted tx
797
- if (failedActions.length > 0 && result?.receipt?.status === 'reverted') {
798
- this.backupFailedTx({
799
- id: result.receipt.transactionHash,
800
- failureType: 'revert',
801
- request: {
802
- to: MULTI_CALL_3_ADDRESS,
803
- data: txContext.multicallData
804
- },
805
- blobData: txContext.blobData,
806
- l1BlockNumber: result.receipt.blockNumber.toString(),
807
- receipt: {
808
- transactionHash: result.receipt.transactionHash,
809
- blockNumber: result.receipt.blockNumber.toString(),
810
- gasUsed: (result.receipt.gasUsed ?? 0n).toString(),
811
- status: 'reverted'
812
- },
813
- error: {
814
- message: result.errorMsg ?? 'Transaction reverted'
815
- },
816
- context: {
817
- actions: failedActions,
818
- requests: requests.filter((r)=>failedActions.includes(r.action)).map((r)=>({
819
- action: r.action,
820
- to: r.request.to,
821
- data: r.request.data
822
- })),
823
- sender: this.getSenderAddress().toString()
824
- }
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
825
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);
826
1056
  }
827
- return {
828
- successfulActions,
829
- failedActions
830
- };
831
1057
  }
832
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
+ }
833
1088
  /**
834
1089
  * @notice Will call `canProposeAt` to make sure that it is possible to propose
835
1090
  * @param tipArchive - The archive to check
836
1091
  * @returns The slot and block number if it is possible to propose, undefined otherwise
837
- */ canProposeAt(tipArchive, msgSender, opts = {}) {
1092
+ */ async canProposeAt(tipArchive, msgSender, simulationOverridesPlan) {
838
1093
  // TODO: #14291 - should loop through multiple keys to check if any of them can propose
839
- 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 = [
840
1099
  'SlotAlreadyInChain',
841
1100
  'InvalidProposer',
842
1101
  'InvalidArchive'
843
1102
  ];
844
- const pipelined = opts.pipelined ?? this.epochCache.isProposerPipeliningEnabled();
845
- const slotOffset = pipelined ? this.aztecSlotDuration : 0n;
1103
+ const slotOffset = this.aztecSlotDuration;
846
1104
  const nextL1SlotTs = this.getNextL1SlotTimestamp() + slotOffset;
847
- return this.rollupContract.canProposeAt(tipArchive.toBuffer(), msgSender.toString(), nextL1SlotTs, {
848
- forcePendingCheckpointNumber: opts.forcePendingCheckpointNumber
849
- }).catch((err)=>{
850
- if (err instanceof FormattedViemError && ignoredErrors.find((e)=>err.message.includes(e))) {
851
- 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))}`, {
852
1108
  error: err.message
853
1109
  });
854
1110
  } else {
@@ -862,22 +1118,23 @@ export class SequencerPublisher {
862
1118
  * @dev This is a convenience function that can be used by the sequencer to validate a "partial" header.
863
1119
  * It will throw if the block header is invalid.
864
1120
  * @param header - The block header to validate
865
- */ async validateBlockHeader(header, opts) {
1121
+ */ async validateBlockHeader(header, simulationOverridesPlan) {
866
1122
  const flags = {
867
1123
  ignoreDA: true,
868
1124
  ignoreSignatures: true
869
1125
  };
870
1126
  const args = [
871
1127
  header.toViem(),
872
- CommitteeAttestationsAndSigners.empty().getPackedAttestations(),
1128
+ CommitteeAttestationsAndSigners.packAttestations([]),
873
1129
  [],
874
1130
  Signature.empty().toViemSignature(),
875
1131
  `0x${'0'.repeat(64)}`,
876
1132
  header.blobsHash.toString(),
877
1133
  flags
878
1134
  ];
879
- const ts = this.getNextL1SlotTimestamp();
880
- 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);
881
1138
  let balance = 0n;
882
1139
  if (this.config.fishermanMode) {
883
1140
  // In fisherman mode, we can't know where the proposer is publishing from
@@ -899,7 +1156,7 @@ export class SequencerPublisher {
899
1156
  }),
900
1157
  from: MULTI_CALL_3_ADDRESS
901
1158
  }, {
902
- time: ts + 1n
1159
+ time: ts
903
1160
  }, stateOverrides);
904
1161
  this.log.debug(`Simulated validateHeader`);
905
1162
  }
@@ -945,6 +1202,7 @@ export class SequencerPublisher {
945
1202
  gasUsed,
946
1203
  checkpointNumber,
947
1204
  forcePendingCheckpointNumber: CheckpointNumber(checkpointNumber - 1),
1205
+ lastArchive: validationResult.checkpoint.lastArchive,
948
1206
  reason
949
1207
  };
950
1208
  } catch (err) {
@@ -957,8 +1215,8 @@ export class SequencerPublisher {
957
1215
  request,
958
1216
  error: viemError.message
959
1217
  });
960
- const latestPendingCheckpointNumber = await this.rollupContract.getCheckpointNumber();
961
- if (latestPendingCheckpointNumber < checkpointNumber) {
1218
+ const latestProposedCheckpointNumber = await this.rollupContract.getCheckpointNumber();
1219
+ if (latestProposedCheckpointNumber < checkpointNumber) {
962
1220
  this.log.verbose(`Checkpoint ${checkpointNumber} has already been invalidated`, {
963
1221
  ...logData
964
1222
  });
@@ -978,9 +1236,9 @@ export class SequencerPublisher {
978
1236
  request: {
979
1237
  to: request.to,
980
1238
  data: request.data,
981
- value: request.value?.toString()
1239
+ value: request.value
982
1240
  },
983
- l1BlockNumber: l1BlockNumber.toString(),
1241
+ l1BlockNumber,
984
1242
  error: {
985
1243
  message: viemError.message,
986
1244
  name: viemError.name
@@ -991,7 +1249,11 @@ export class SequencerPublisher {
991
1249
  ],
992
1250
  checkpointNumber,
993
1251
  sender: this.getSenderAddress().toString()
994
- }
1252
+ },
1253
+ timing: this.computeTimingInfo(this.getCurrentL2Slot())
1254
+ }, {
1255
+ captureFeeSummary: true,
1256
+ targetSlot: this.getCurrentL2Slot()
995
1257
  });
996
1258
  throw new Error(`Failed to simulate invalidate checkpoint ${checkpointNumber}`, {
997
1259
  cause: viemError
@@ -1008,7 +1270,11 @@ export class SequencerPublisher {
1008
1270
  reason
1009
1271
  };
1010
1272
  this.log.debug(`Building invalidate checkpoint ${checkpoint.checkpointNumber} request`, logData);
1011
- 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;
1012
1278
  if (reason === 'invalid-attestation') {
1013
1279
  return this.rollupContract.buildInvalidateBadAttestationRequest(checkpoint.checkpointNumber, attestationsAndSigners, committee, validationResult.invalidIndex);
1014
1280
  } else if (reason === 'insufficient-attestations') {
@@ -1018,30 +1284,7 @@ export class SequencerPublisher {
1018
1284
  throw new Error(`Unknown reason for invalidation`);
1019
1285
  }
1020
1286
  }
1021
- /** Simulates `propose` to make sure that the checkpoint is valid for submission */ async validateCheckpointForSubmission(checkpoint, attestationsAndSigners, attestationsAndSignersSignature, options) {
1022
- // Anchor the simulation timestamp to the checkpoint's own slot start time
1023
- // rather than the current L1 block timestamp, which may overshoot into the next slot if the build ran late.
1024
- const ts = checkpoint.header.timestamp;
1025
- const blobFields = checkpoint.toBlobFields();
1026
- const blobs = await getBlobsPerL1Block(blobFields);
1027
- const blobInput = getPrefixedEthBlobCommitments(blobs);
1028
- const args = [
1029
- {
1030
- header: checkpoint.header.toViem(),
1031
- archive: toHex(checkpoint.archive.root.toBuffer()),
1032
- oracleInput: {
1033
- feeAssetPriceModifier: checkpoint.feeAssetPriceModifier
1034
- }
1035
- },
1036
- attestationsAndSigners.getPackedAttestations(),
1037
- attestationsAndSigners.getSigners().map((signer)=>signer.toString()),
1038
- attestationsAndSignersSignature.toViemSignature(),
1039
- blobInput
1040
- ];
1041
- await this.simulateProposeTx(args, ts, options);
1042
- return ts;
1043
- }
1044
- async enqueueCastSignalHelper(slotNumber, timestamp, signalType, payload, base, signerAddress, signer) {
1287
+ async enqueueCastSignalHelper(slotNumber, signalType, payload, base, signerAddress, signer) {
1045
1288
  if (this.lastActions[signalType] && this.lastActions[signalType] === slotNumber) {
1046
1289
  this.log.debug(`Skipping duplicate vote cast signal ${signalType} for slot ${slotNumber}`);
1047
1290
  return false;
@@ -1053,6 +1296,17 @@ export class SequencerPublisher {
1053
1296
  this.log.warn(`Cannot enqueue vote cast signal ${signalType} for address zero at slot ${slotNumber}`);
1054
1297
  return false;
1055
1298
  }
1299
+ const canonicalRollup = await base.getRollupAddress();
1300
+ if (!canonicalRollup.equals(EthAddress.fromString(this.rollupContract.address))) {
1301
+ this.log.warn(`Rollup ${this.rollupContract.address} is not canonical, skipping governance signal`, {
1302
+ slotNumber,
1303
+ signalType,
1304
+ canonicalRollup,
1305
+ targetRollup: this.rollupContract.address,
1306
+ payload: payload.toString()
1307
+ });
1308
+ return false;
1309
+ }
1056
1310
  const round = await base.computeRound(slotNumber);
1057
1311
  const roundInfo = await base.getRoundInfo(this.rollupContract.address, round);
1058
1312
  if (roundInfo.quorumReached) {
@@ -1061,30 +1315,40 @@ export class SequencerPublisher {
1061
1315
  if (roundInfo.lastSignalSlot >= slotNumber) {
1062
1316
  return false;
1063
1317
  }
1064
- if (await this.isPayloadEmpty(payload)) {
1318
+ if (await base.isPayloadEmpty(payload)) {
1065
1319
  this.log.warn(`Skipping vote cast for payload with empty code`);
1066
1320
  return false;
1067
1321
  }
1068
- // Check if payload was already submitted to governance
1069
- const cacheKey = payload.toString();
1070
- if (!this.payloadProposedCache.has(cacheKey)) {
1071
- try {
1072
- const l1StartBlock = await this.rollupContract.getL1StartBlock();
1073
- const proposed = await retry(()=>base.hasPayloadBeenProposed(payload.toString(), l1StartBlock), 'Check if payload was proposed', makeBackoff([
1074
- 0,
1075
- 1,
1076
- 2
1077
- ]), this.log, true);
1078
- if (proposed) {
1079
- this.payloadProposedCache.add(cacheKey);
1080
- }
1081
- } catch (err) {
1082
- this.log.warn(`Failed to check if payload ${payload} was proposed after retries, skipping signal`, err);
1083
- return false;
1084
- }
1322
+ // Classify the payload against the Governance proposal history so we stop signalling once its
1323
+ // proposal is live or was already executed, while still re-signalling one whose proposal was
1324
+ // merely rejected/dropped/expired.
1325
+ let status = 'none';
1326
+ try {
1327
+ status = await base.getPayloadProposalStatus(payload.toString());
1328
+ } catch (err) {
1329
+ // We deliberately swallow the error and proceed to signal. Failing closed (skipping the
1330
+ // signal) on transient RPC errors would let a flaky L1 endpoint silence governance
1331
+ // participation entirely; failing open at worst produces a duplicate signal that the
1332
+ // contract will simply count alongside others in the round.
1333
+ this.log.error(`Failed to check governance proposal status for payload ${payload} (signalling anyway)`, err, {
1334
+ slotNumber,
1335
+ signalType
1336
+ });
1337
+ }
1338
+ if (status === 'live') {
1339
+ this.log.info(`Payload ${payload} has a live governance proposal, stopping signals`, {
1340
+ slotNumber,
1341
+ signalType,
1342
+ payload: payload.toString()
1343
+ });
1344
+ return false;
1085
1345
  }
1086
- if (this.payloadProposedCache.has(cacheKey)) {
1087
- this.log.info(`Payload ${payload} was already proposed to governance, stopping signals`);
1346
+ if (status === 'executed' && !this.config.governanceProposerForcePayloadVote) {
1347
+ this.log.info(`Payload ${payload} was executed by governance within lookback, stopping signals ` + `(set GOVERNANCE_PROPOSER_FORCE_PAYLOAD_VOTE to re-signal)`, {
1348
+ slotNumber,
1349
+ signalType,
1350
+ payload: payload.toString()
1351
+ });
1088
1352
  return false;
1089
1353
  }
1090
1354
  const cachedLastVote = this.lastActions[signalType];
@@ -1097,53 +1361,17 @@ export class SequencerPublisher {
1097
1361
  signer: this.l1TxUtils.client.account?.address,
1098
1362
  lastValidL2Slot: slotNumber
1099
1363
  });
1100
- const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
1101
- try {
1102
- await this.l1TxUtils.simulate(request, {
1103
- time: timestamp
1104
- }, [], mergeAbis([
1105
- request.abi ?? [],
1106
- ErrorsAbi
1107
- ]));
1108
- this.log.debug(`Simulation for ${action} at slot ${slotNumber} succeeded`, {
1109
- request
1110
- });
1111
- } catch (err) {
1112
- const viemError = formatViemError(err);
1113
- this.log.error(`Failed simulation for ${action} at slot ${slotNumber} (enqueuing the action anyway)`, viemError);
1114
- this.backupFailedTx({
1115
- id: keccak256(request.data),
1116
- failureType: 'simulation',
1117
- request: {
1118
- to: request.to,
1119
- data: request.data,
1120
- value: request.value?.toString()
1121
- },
1122
- l1BlockNumber: l1BlockNumber.toString(),
1123
- error: {
1124
- message: viemError.message,
1125
- name: viemError.name
1126
- },
1127
- context: {
1128
- actions: [
1129
- action
1130
- ],
1131
- slot: slotNumber,
1132
- sender: this.getSenderAddress().toString()
1133
- }
1134
- });
1135
- // Yes, we enqueue the request anyway, in case there was a bug with the simulation itself
1136
- }
1137
1364
  // TODO(palla/slash): All votes (governance and slashing) should txTimeoutAt at the end of the slot.
1138
1365
  this.addRequest({
1139
- gasConfig: {
1140
- gasLimit: SequencerPublisher.VOTE_GAS_GUESS
1141
- },
1142
1366
  action,
1143
1367
  request,
1144
1368
  lastValidL2Slot: slotNumber,
1145
1369
  checkSuccess: (_request, result)=>{
1146
- const success = result && result.receipt && result.receipt.status === 'success' && tryExtractEvent(result.receipt.logs, base.address.toString(), EmpireBaseAbi, 'SignalCast');
1370
+ const success = result && extractEventSuccess(result.receipt, {
1371
+ address: base.address.toString(),
1372
+ abi: EmpireBaseAbi,
1373
+ eventName: 'SignalCast'
1374
+ });
1147
1375
  const logData = {
1148
1376
  ...result,
1149
1377
  slotNumber,
@@ -1162,68 +1390,70 @@ export class SequencerPublisher {
1162
1390
  });
1163
1391
  return true;
1164
1392
  }
1165
- async isPayloadEmpty(payload) {
1166
- const key = payload.toString();
1167
- const cached = this.isPayloadEmptyCache.get(key);
1168
- if (cached) {
1169
- return cached;
1170
- }
1171
- const isEmpty = !await this.l1TxUtils.getCode(payload);
1172
- this.isPayloadEmptyCache.set(key, isEmpty);
1173
- return isEmpty;
1174
- }
1175
1393
  /**
1176
1394
  * Enqueues a governance castSignal transaction to cast a signal for a given slot number.
1177
1395
  * @param slotNumber - The slot number to cast a signal for.
1178
- * @param timestamp - The timestamp of the slot to cast a signal for.
1179
1396
  * @returns True if the signal was successfully enqueued, false otherwise.
1180
- */ enqueueGovernanceCastSignal(governancePayload, slotNumber, timestamp, signerAddress, signer) {
1181
- return this.enqueueCastSignalHelper(slotNumber, timestamp, 'governance-signal', governancePayload, this.govProposerContract, signerAddress, signer);
1397
+ */ enqueueGovernanceCastSignal(governancePayload, slotNumber, signerAddress, signer) {
1398
+ return this.enqueueCastSignalHelper(slotNumber, 'governance-signal', governancePayload, this.govProposerContract, signerAddress, signer);
1182
1399
  }
1183
- /** Enqueues all slashing actions as returned by the slasher client. */ async enqueueSlashingActions(actions, slotNumber, timestamp, signerAddress, signer) {
1400
+ /**
1401
+ * Enqueues a `prune()` transaction if the rollup is prunable at the given slot's L1 timestamp.
1402
+ * `prune()` is permissionless and idempotent — if the chain is no longer prunable by send time the
1403
+ * bundle simulation usually drops the entry; on a node without `eth_simulateV1` the bundle is sent
1404
+ * as-is and the prune reverts `Rollup__NothingToPrune` inside `aggregate3(allowFailure: true)`
1405
+ * (a failed action, never a whole-tx revert). Used by the failed-sync fallback so a stuck pending
1406
+ * chain (e.g. bad data blocking sync) can be wound back to recover.
1407
+ * @returns true if a prune request was enqueued, false otherwise.
1408
+ */ async enqueuePruneIfPrunable(slotNumber) {
1409
+ if (this.lastActions['prune'] === slotNumber) {
1410
+ this.log.debug(`Skipping duplicate prune for slot ${slotNumber}`, {
1411
+ slotNumber
1412
+ });
1413
+ return false;
1414
+ }
1415
+ // Use the SAME timestamp the bundle simulator overrides block.timestamp with at send time
1416
+ // (sequencer-bundle-simulator.ts) so this upfront check and the send-time sim agree. Slot-start
1417
+ // and last-L1-slot both fall within the same L2 slot (and epoch, which is what `canPruneAtTime`
1418
+ // derives), so they agree today; matching the simulator keeps it robust if the contract ever uses
1419
+ // the timestamp more granularly.
1420
+ const ts = getLastL1SlotTimestampForL2Slot(slotNumber, this.epochCache.getL1Constants());
1421
+ const canPrune = await this.rollupContract.canPruneAtTime(ts).catch((err)=>{
1422
+ this.log.error(`Failed to check canPruneAtTime for slot ${slotNumber}`, err, {
1423
+ slotNumber
1424
+ });
1425
+ return false;
1426
+ });
1427
+ if (!canPrune) {
1428
+ this.log.debug(`Rollup not prunable at slot ${slotNumber}`, {
1429
+ slotNumber
1430
+ });
1431
+ return false;
1432
+ }
1433
+ const request = {
1434
+ to: this.rollupContract.address,
1435
+ data: encodeFunctionData({
1436
+ abi: RollupAbi,
1437
+ functionName: 'prune',
1438
+ args: []
1439
+ })
1440
+ };
1441
+ this.log.info(`Enqueuing rollup prune for slot ${slotNumber}`, {
1442
+ slotNumber
1443
+ });
1444
+ return this.enqueueRequest('prune', request, {
1445
+ address: this.rollupContract.address,
1446
+ abi: RollupAbi,
1447
+ eventName: 'PrunedPending'
1448
+ }, slotNumber);
1449
+ }
1450
+ /** Enqueues all slashing actions as returned by the slasher client. */ async enqueueSlashingActions(actions, slotNumber, signerAddress, signer) {
1184
1451
  if (actions.length === 0) {
1185
1452
  this.log.debug(`No slashing actions to enqueue for slot ${slotNumber}`);
1186
1453
  return false;
1187
1454
  }
1188
1455
  for (const action of actions){
1189
1456
  switch(action.type){
1190
- case 'vote-empire-payload':
1191
- {
1192
- if (this.slashingProposerContract?.type !== 'empire') {
1193
- this.log.error('Cannot vote for empire payload on non-empire slashing contract');
1194
- break;
1195
- }
1196
- this.log.debug(`Enqueuing slashing vote for payload ${action.payload} at slot ${slotNumber}`, {
1197
- signerAddress
1198
- });
1199
- await this.enqueueCastSignalHelper(slotNumber, timestamp, 'empire-slashing-signal', action.payload, this.slashingProposerContract, signerAddress, signer);
1200
- break;
1201
- }
1202
- case 'create-empire-payload':
1203
- {
1204
- this.log.debug(`Enqueuing slashing create payload at slot ${slotNumber}`, {
1205
- slotNumber,
1206
- signerAddress
1207
- });
1208
- const request = this.slashFactoryContract.buildCreatePayloadRequest(action.data);
1209
- await this.simulateAndEnqueueRequest('create-empire-payload', request, (receipt)=>!!this.slashFactoryContract.tryExtractSlashPayloadCreatedEvent(receipt.logs), slotNumber, timestamp);
1210
- break;
1211
- }
1212
- case 'execute-empire-payload':
1213
- {
1214
- this.log.debug(`Enqueuing slashing execute payload at slot ${slotNumber}`, {
1215
- slotNumber,
1216
- signerAddress
1217
- });
1218
- if (this.slashingProposerContract?.type !== 'empire') {
1219
- this.log.error('Cannot execute slashing payload on non-empire slashing contract');
1220
- return false;
1221
- }
1222
- const empireSlashingProposer = this.slashingProposerContract;
1223
- const request = empireSlashingProposer.buildExecuteRoundRequest(action.round);
1224
- await this.simulateAndEnqueueRequest('execute-empire-payload', request, (receipt)=>!!empireSlashingProposer.tryExtractPayloadSubmittedEvent(receipt.logs), slotNumber, timestamp);
1225
- break;
1226
- }
1227
1457
  case 'vote-offenses':
1228
1458
  {
1229
1459
  this.log.debug(`Enqueuing slashing vote for ${action.votes.length} votes at slot ${slotNumber}`, {
@@ -1232,14 +1462,17 @@ export class SequencerPublisher {
1232
1462
  votesCount: action.votes.length,
1233
1463
  signerAddress
1234
1464
  });
1235
- if (this.slashingProposerContract?.type !== 'tally') {
1236
- this.log.error('Cannot vote for slashing offenses on non-tally slashing contract');
1465
+ if (!this.slashingProposerContract) {
1466
+ this.log.error('No slashing proposer contract available');
1237
1467
  return false;
1238
1468
  }
1239
- const tallySlashingProposer = this.slashingProposerContract;
1240
1469
  const votes = bufferToHex(encodeSlashConsensusVotes(action.votes));
1241
- const request = await tallySlashingProposer.buildVoteRequestFromSigner(votes, slotNumber, signer);
1242
- await this.simulateAndEnqueueRequest('vote-offenses', request, (receipt)=>!!tallySlashingProposer.tryExtractVoteCastEvent(receipt.logs), slotNumber, timestamp);
1470
+ const request = await this.slashingProposerContract.buildVoteRequestFromSigner(votes, slotNumber, signer);
1471
+ this.enqueueRequest('vote-offenses', request, {
1472
+ address: this.slashingProposerContract.address.toString(),
1473
+ abi: SlashingProposerAbi,
1474
+ eventName: 'VoteCast'
1475
+ }, slotNumber);
1243
1476
  break;
1244
1477
  }
1245
1478
  case 'execute-slash':
@@ -1249,13 +1482,16 @@ export class SequencerPublisher {
1249
1482
  round: action.round,
1250
1483
  signerAddress
1251
1484
  });
1252
- if (this.slashingProposerContract?.type !== 'tally') {
1253
- this.log.error('Cannot execute slashing offenses on non-tally slashing contract');
1485
+ if (!this.slashingProposerContract) {
1486
+ this.log.error('No slashing proposer contract available');
1254
1487
  return false;
1255
1488
  }
1256
- const tallySlashingProposer = this.slashingProposerContract;
1257
- const request = tallySlashingProposer.buildExecuteRoundRequest(action.round, action.committees);
1258
- await this.simulateAndEnqueueRequest('execute-slash', request, (receipt)=>!!tallySlashingProposer.tryExtractRoundExecutedEvent(receipt.logs), slotNumber, timestamp);
1489
+ const executeRequest = this.slashingProposerContract.buildExecuteRoundRequest(action.round, action.committees);
1490
+ this.enqueueRequest('execute-slash', executeRequest, {
1491
+ address: this.slashingProposerContract.address.toString(),
1492
+ abi: SlashingProposerAbi,
1493
+ eventName: 'RoundExecuted'
1494
+ }, slotNumber);
1259
1495
  break;
1260
1496
  }
1261
1497
  default:
@@ -1267,7 +1503,7 @@ export class SequencerPublisher {
1267
1503
  }
1268
1504
  return true;
1269
1505
  }
1270
- /** Simulates and enqueues a proposal for a checkpoint on L1 */ async enqueueProposeCheckpoint(checkpoint, attestationsAndSigners, attestationsAndSignersSignature, opts = {}) {
1506
+ /** Enqueues a proposal for a checkpoint on L1 */ async enqueueProposeCheckpoint(checkpoint, attestationsAndSigners, attestationsAndSignersSignature, opts = {}) {
1271
1507
  const checkpointHeader = checkpoint.header;
1272
1508
  const blobFields = checkpoint.toBlobFields();
1273
1509
  const blobs = await getBlobsPerL1Block(blobFields);
@@ -1279,52 +1515,38 @@ export class SequencerPublisher {
1279
1515
  attestationsAndSignersSignature,
1280
1516
  feeAssetPriceModifier: checkpoint.feeAssetPriceModifier
1281
1517
  };
1282
- let ts;
1283
- try {
1284
- // @note This will make sure that we are passing the checks for our header ASSUMING that the data is also made available
1285
- // This means that we can avoid the simulation issues in later checks.
1286
- // By simulation issue, I mean the fact that the block.timestamp is equal to the last block, not the next, which
1287
- // make time consistency checks break.
1288
- // TODO(palla): Check whether we're validating twice, once here and once within addProposeTx, since we call simulateProposeTx in both places.
1289
- ts = await this.validateCheckpointForSubmission(checkpoint, attestationsAndSigners, attestationsAndSignersSignature, opts);
1290
- } catch (err) {
1291
- this.log.error(`Checkpoint validation failed. ${err instanceof Error ? err.message : 'No error message'}`, err, {
1292
- ...checkpoint.getStats(),
1293
- slotNumber: checkpoint.header.slotNumber,
1294
- forcePendingCheckpointNumber: opts.forcePendingCheckpointNumber
1295
- });
1296
- throw err;
1297
- }
1298
1518
  this.log.verbose(`Enqueuing checkpoint propose transaction`, {
1299
1519
  ...checkpoint.toCheckpointInfo(),
1300
- ...opts
1520
+ txTimeoutAt: opts.txTimeoutAt
1521
+ });
1522
+ await this.addProposeTx(checkpoint, proposeTxArgs, {
1523
+ txTimeoutAt: opts.txTimeoutAt
1301
1524
  });
1302
- await this.addProposeTx(checkpoint, proposeTxArgs, opts, ts);
1303
1525
  }
1304
1526
  enqueueInvalidateCheckpoint(request, opts = {}) {
1305
1527
  if (!request) {
1306
1528
  return;
1307
1529
  }
1308
- // We issued the simulation against the rollup contract, so we need to account for the overhead of the multicall3
1309
- const gasLimit = this.l1TxUtils.bumpGasLimit(BigInt(Math.ceil(Number(request.gasUsed) * 64 / 63)));
1310
1530
  const { gasUsed, checkpointNumber } = request;
1311
1531
  const logData = {
1312
1532
  gasUsed,
1313
1533
  checkpointNumber,
1314
- gasLimit,
1315
1534
  opts
1316
1535
  };
1317
1536
  this.log.verbose(`Enqueuing invalidate checkpoint request`, logData);
1318
1537
  this.addRequest({
1319
1538
  action: `invalidate-by-${request.reason}`,
1320
1539
  request: request.request,
1321
- gasConfig: {
1322
- gasLimit,
1540
+ gasConfig: opts.txTimeoutAt ? {
1323
1541
  txTimeoutAt: opts.txTimeoutAt
1324
- },
1542
+ } : undefined,
1325
1543
  lastValidL2Slot: SlotNumber(this.getCurrentL2Slot() + 2),
1326
1544
  checkSuccess: (_req, result)=>{
1327
- const success = result && result.receipt && result.receipt.status === 'success' && tryExtractEvent(result.receipt.logs, this.rollupContract.address, RollupAbi, 'CheckpointInvalidated');
1545
+ const success = result && extractEventSuccess(result.receipt, {
1546
+ address: this.rollupContract.address,
1547
+ abi: RollupAbi,
1548
+ eventName: 'CheckpointInvalidated'
1549
+ });
1328
1550
  if (!success) {
1329
1551
  this.log.warn(`Invalidate checkpoint ${request.checkpointNumber} failed`, {
1330
1552
  ...result,
@@ -1340,89 +1562,36 @@ export class SequencerPublisher {
1340
1562
  }
1341
1563
  });
1342
1564
  }
1343
- async simulateAndEnqueueRequest(action, request, checkSuccess, slotNumber, timestamp) {
1344
- const logData = {
1345
- slotNumber,
1346
- timestamp,
1347
- gasLimit: undefined
1348
- };
1565
+ /**
1566
+ * Dedup-checked enqueue helper for actions that are simulated at bundle-send time rather
1567
+ * than at enqueue time. Validates the (action, slot) dedup key, sets `lastActions`, and
1568
+ * enqueues without a gasLimit so the bundle simulate sets the only gasLimit that matters.
1569
+ */ enqueueRequest(action, request, eventOpts, slotNumber) {
1349
1570
  if (this.lastActions[action] && this.lastActions[action] === slotNumber) {
1350
1571
  this.log.debug(`Skipping duplicate action ${action} for slot ${slotNumber}`);
1351
1572
  return false;
1352
1573
  }
1353
1574
  const cachedLastActionSlot = this.lastActions[action];
1354
1575
  this.lastActions[action] = slotNumber;
1355
- this.log.debug(`Simulating ${action} for slot ${slotNumber}`, logData);
1356
- const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
1357
- let gasUsed;
1358
- const simulateAbi = mergeAbis([
1359
- request.abi ?? [],
1360
- ErrorsAbi
1361
- ]);
1362
- try {
1363
- ({ gasUsed } = await this.l1TxUtils.simulate(request, {
1364
- time: timestamp
1365
- }, [], simulateAbi)); // TODO(palla/slash): Check the timestamp logic
1366
- this.log.verbose(`Simulation for ${action} succeeded`, {
1367
- ...logData,
1368
- request,
1369
- gasUsed
1370
- });
1371
- } catch (err) {
1372
- const viemError = formatViemError(err, simulateAbi);
1373
- this.log.error(`Simulation for ${action} at ${slotNumber} failed`, viemError, logData);
1374
- this.backupFailedTx({
1375
- id: keccak256(request.data),
1376
- failureType: 'simulation',
1377
- request: {
1378
- to: request.to,
1379
- data: request.data,
1380
- value: request.value?.toString()
1381
- },
1382
- l1BlockNumber: l1BlockNumber.toString(),
1383
- error: {
1384
- message: viemError.message,
1385
- name: viemError.name
1386
- },
1387
- context: {
1388
- actions: [
1389
- action
1390
- ],
1391
- slot: slotNumber,
1392
- sender: this.getSenderAddress().toString()
1393
- }
1394
- });
1395
- return false;
1396
- }
1397
- // We issued the simulation against the rollup contract, so we need to account for the overhead of the multicall3
1398
- const gasLimit = this.l1TxUtils.bumpGasLimit(BigInt(Math.ceil(Number(gasUsed) * 64 / 63)));
1399
- logData.gasLimit = gasLimit;
1400
- // Store the ABI used for simulation on the request so Multicall3.forward can decode errors
1401
- // when the tx is sent and a revert is diagnosed via simulation.
1402
- const requestWithAbi = {
1403
- ...request,
1404
- abi: simulateAbi
1405
- };
1406
- this.log.debug(`Enqueuing ${action}`, logData);
1576
+ this.log.debug(`Enqueuing ${action}`, {
1577
+ slotNumber
1578
+ });
1407
1579
  this.addRequest({
1408
1580
  action,
1409
- request: requestWithAbi,
1410
- gasConfig: {
1411
- gasLimit
1412
- },
1581
+ request,
1413
1582
  lastValidL2Slot: slotNumber,
1414
- checkSuccess: (_req, result)=>{
1415
- const success = result && result.receipt && result.receipt.status === 'success' && checkSuccess(result.receipt);
1583
+ checkSuccess: (_request, result)=>{
1584
+ const success = result && extractEventSuccess(result.receipt, eventOpts);
1416
1585
  if (!success) {
1417
1586
  this.log.warn(`Action ${action} at ${slotNumber} failed`, {
1418
1587
  ...result,
1419
- ...logData
1588
+ slotNumber
1420
1589
  });
1421
1590
  this.lastActions[action] = cachedLastActionSlot;
1422
1591
  } else {
1423
1592
  this.log.info(`Action ${action} at ${slotNumber} succeeded`, {
1424
1593
  ...result,
1425
- ...logData
1594
+ slotNumber
1426
1595
  });
1427
1596
  }
1428
1597
  return !!success;
@@ -1437,13 +1606,14 @@ export class SequencerPublisher {
1437
1606
  * A call to `restart` is required before you can continue publishing.
1438
1607
  */ interrupt() {
1439
1608
  this.interrupted = true;
1609
+ this.interruptibleSleep.interrupt();
1440
1610
  this.l1TxUtils.interrupt();
1441
1611
  }
1442
1612
  /** Restarts the publisher after calling `interrupt`. */ restart() {
1443
1613
  this.interrupted = false;
1444
1614
  this.l1TxUtils.restart();
1445
1615
  }
1446
- async prepareProposeTx(encodedData, timestamp, options) {
1616
+ async prepareProposeTx(encodedData) {
1447
1617
  const kzg = Blob.getViemKzgInstance();
1448
1618
  const blobInput = getPrefixedEthBlobCommitments(encodedData.blobs);
1449
1619
  this.log.debug('Validating blob input', {
@@ -1457,7 +1627,11 @@ export class SequencerPublisher {
1457
1627
  blobEvaluationGas = BigInt(encodedData.blobs.length) * 21_000n;
1458
1628
  this.log.debug(`Using fixed blob evaluation gas estimate in fisherman mode: ${blobEvaluationGas}`);
1459
1629
  } else {
1460
- // Normal mode - use estimateGas with blob inputs
1630
+ // We call validateBlobs via estimateGas with real blob+kzg sidecars as a consistency check
1631
+ // that our locally-built blob commitments match the blob data. The bundle simulate at send
1632
+ // time uses eth_simulateV1, which cannot carry blob inputs, so the rollup's on-chain blob
1633
+ // check is forced off there — making this the only pre-flight detector of a commitment/data
1634
+ // mismatch. The returned gas estimate is stashed on the request for the bundle path to read.
1461
1635
  blobEvaluationGas = await this.l1TxUtils.estimateGas(this.getSenderAddress().toString(), {
1462
1636
  to: this.rollupContract.address,
1463
1637
  data: encodeFunctionData({
@@ -1491,7 +1665,7 @@ export class SequencerPublisher {
1491
1665
  data: validateBlobsData
1492
1666
  },
1493
1667
  blobData: encodedData.blobs.map((b)=>toHex(b.data)),
1494
- l1BlockNumber: l1BlockNumber.toString(),
1668
+ l1BlockNumber,
1495
1669
  error: {
1496
1670
  message: viemError.message,
1497
1671
  name: viemError.name
@@ -1501,7 +1675,11 @@ export class SequencerPublisher {
1501
1675
  'validate-blobs'
1502
1676
  ],
1503
1677
  sender: this.getSenderAddress().toString()
1504
- }
1678
+ },
1679
+ timing: this.computeTimingInfo(this.getCurrentL2Slot())
1680
+ }, {
1681
+ captureFeeSummary: true,
1682
+ targetSlot: this.getCurrentL2Slot()
1505
1683
  });
1506
1684
  throw new Error('Failed to validate blobs');
1507
1685
  });
@@ -1520,109 +1698,23 @@ export class SequencerPublisher {
1520
1698
  encodedData.attestationsAndSignersSignature.toViemSignature(),
1521
1699
  blobInput
1522
1700
  ];
1523
- const { rollupData, simulationResult } = await this.simulateProposeTx(args, timestamp, options);
1524
- return {
1525
- args,
1526
- blobEvaluationGas,
1527
- rollupData,
1528
- simulationResult
1529
- };
1530
- }
1531
- /**
1532
- * Simulates the propose tx with eth_simulateV1
1533
- * @param args - The propose tx args
1534
- * @param timestamp - The timestamp to simulate proposal at
1535
- * @returns The simulation result
1536
- */ async simulateProposeTx(args, timestamp, options) {
1537
1701
  const rollupData = encodeFunctionData({
1538
1702
  abi: RollupAbi,
1539
1703
  functionName: 'propose',
1540
1704
  args
1541
1705
  });
1542
- // override the pending checkpoint number if requested
1543
- const forcePendingCheckpointNumberStateDiff = (options.forcePendingCheckpointNumber !== undefined ? await this.rollupContract.makePendingCheckpointNumberOverride(options.forcePendingCheckpointNumber) : []).flatMap((override)=>override.stateDiff ?? []);
1544
- const stateOverrides = [
1545
- {
1546
- address: this.rollupContract.address,
1547
- // @note we override checkBlob to false since blobs are not part simulate()
1548
- stateDiff: [
1549
- {
1550
- slot: toPaddedHex(RollupContract.checkBlobStorageSlot, true),
1551
- value: toPaddedHex(0n, true)
1552
- },
1553
- ...forcePendingCheckpointNumberStateDiff
1554
- ]
1555
- }
1556
- ];
1557
- // In fisherman mode, simulate as the proposer but with sufficient balance
1558
- if (this.proposerAddressForSimulation) {
1559
- stateOverrides.push({
1560
- address: this.proposerAddressForSimulation.toString(),
1561
- balance: 10n * WEI_CONST * WEI_CONST
1562
- });
1563
- }
1564
- const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
1565
- const simulationResult = await this.l1TxUtils.simulate({
1566
- to: this.rollupContract.address,
1567
- data: rollupData,
1568
- gas: MAX_L1_TX_LIMIT,
1569
- ...this.proposerAddressForSimulation && {
1570
- from: this.proposerAddressForSimulation.toString()
1571
- }
1572
- }, {
1573
- // @note we add 1n to the timestamp because geth implementation doesn't like simulation timestamp to be equal to the current block timestamp
1574
- time: timestamp + 1n,
1575
- // @note reth should have a 30m gas limit per block but throws errors that this tx is beyond limit so we increase here
1576
- gasLimit: MAX_L1_TX_LIMIT * 2n
1577
- }, stateOverrides, RollupAbi, {
1578
- // @note fallback gas estimate to use if the node doesn't support simulation API
1579
- fallbackGasEstimate: MAX_L1_TX_LIMIT
1580
- }).catch((err)=>{
1581
- // In fisherman mode, we expect ValidatorSelection__MissingProposerSignature since fisherman doesn't have proposer signature
1582
- const viemError = formatViemError(err);
1583
- if (this.config.fishermanMode && viemError.message?.includes('ValidatorSelection__MissingProposerSignature')) {
1584
- this.log.debug(`Ignoring expected ValidatorSelection__MissingProposerSignature error in fisherman mode`);
1585
- // Return a minimal simulation result with the fallback gas estimate
1586
- return {
1587
- gasUsed: MAX_L1_TX_LIMIT,
1588
- logs: []
1589
- };
1590
- }
1591
- this.log.error(`Failed to simulate propose tx`, viemError);
1592
- this.backupFailedTx({
1593
- id: keccak256(rollupData),
1594
- failureType: 'simulation',
1595
- request: {
1596
- to: this.rollupContract.address,
1597
- data: rollupData
1598
- },
1599
- l1BlockNumber: l1BlockNumber.toString(),
1600
- error: {
1601
- message: viemError.message,
1602
- name: viemError.name
1603
- },
1604
- context: {
1605
- actions: [
1606
- 'propose'
1607
- ],
1608
- slot: Number(args[0].header.slotNumber),
1609
- sender: this.getSenderAddress().toString()
1610
- }
1611
- });
1612
- throw err;
1613
- });
1614
1706
  return {
1615
- rollupData,
1616
- simulationResult
1707
+ args,
1708
+ blobEvaluationGas,
1709
+ rollupData
1617
1710
  };
1618
1711
  }
1619
- async addProposeTx(checkpoint, encodedData, opts = {}, timestamp) {
1712
+ async addProposeTx(checkpoint, encodedData, opts = {}) {
1620
1713
  const slot = checkpoint.header.slotNumber;
1621
1714
  const timer = new Timer();
1622
1715
  const kzg = Blob.getViemKzgInstance();
1623
- const { rollupData, simulationResult, blobEvaluationGas } = await this.prepareProposeTx(encodedData, timestamp, opts);
1716
+ const { rollupData, blobEvaluationGas } = await this.prepareProposeTx(encodedData);
1624
1717
  const startBlock = await this.l1TxUtils.getBlockNumber();
1625
- const gasLimit = this.l1TxUtils.bumpGasLimit(BigInt(Math.ceil(Number(simulationResult.gasUsed) * 64 / 63)) + blobEvaluationGas + SequencerPublisher.MULTICALL_OVERHEAD_GAS_GUESS);
1626
1718
  // Send the blobs to the blob client preemptively. This helps in tests where the sequencer mistakingly thinks that the propose
1627
1719
  // tx fails but it does get mined. We make sure that the blobs are sent to the blob client regardless of the tx outcome.
1628
1720
  void Promise.resolve().then(()=>this.blobClient.sendBlobsToFilestore(encodedData.blobs).catch((_err)=>{
@@ -1636,9 +1728,10 @@ export class SequencerPublisher {
1636
1728
  },
1637
1729
  lastValidL2Slot: checkpoint.header.slotNumber,
1638
1730
  gasConfig: {
1639
- ...opts,
1640
- gasLimit
1731
+ txTimeoutAt: opts.txTimeoutAt,
1732
+ gasLimit: undefined
1641
1733
  },
1734
+ blobEvaluationGas,
1642
1735
  blobConfig: {
1643
1736
  blobs: encodedData.blobs.map((b)=>b.data),
1644
1737
  kzg
@@ -1648,7 +1741,11 @@ export class SequencerPublisher {
1648
1741
  return false;
1649
1742
  }
1650
1743
  const { receipt, stats, errorMsg } = result;
1651
- const success = receipt && receipt.status === 'success' && tryExtractEvent(receipt.logs, this.rollupContract.address, RollupAbi, 'CheckpointProposed');
1744
+ const success = extractEventSuccess(receipt, {
1745
+ address: this.rollupContract.address,
1746
+ abi: RollupAbi,
1747
+ eventName: 'CheckpointProposed'
1748
+ });
1652
1749
  if (success) {
1653
1750
  const endBlock = receipt.blockNumber;
1654
1751
  const inclusionBlocks = Number(endBlock - startBlock);
@@ -1685,7 +1782,7 @@ export class SequencerPublisher {
1685
1782
  }
1686
1783
  });
1687
1784
  }
1688
- /** Returns the timestamp to use when simulating L1 proposal calls */ getNextL1SlotTimestamp() {
1785
+ /** Returns the timestamp of the next L1 slot boundary after now. */ getNextL1SlotTimestamp() {
1689
1786
  const l1Constants = this.epochCache.getL1Constants();
1690
1787
  return getNextL1SlotTimestamp(this.dateProvider.nowInSeconds(), l1Constants);
1691
1788
  }