@aztec/sequencer-client 5.0.0-private.20260319 → 5.0.0-rc.1

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 (105) hide show
  1. package/README.md +281 -21
  2. package/dest/client/sequencer-client.d.ts +8 -3
  3. package/dest/client/sequencer-client.d.ts.map +1 -1
  4. package/dest/client/sequencer-client.js +17 -32
  5. package/dest/config.d.ts +10 -4
  6. package/dest/config.d.ts.map +1 -1
  7. package/dest/config.js +48 -24
  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 +128 -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 +67 -0
  14. package/dest/global_variable_builder/global_builder.d.ts +14 -14
  15. package/dest/global_variable_builder/global_builder.d.ts.map +1 -1
  16. package/dest/global_variable_builder/global_builder.js +16 -51
  17. package/dest/global_variable_builder/index.d.ts +4 -2
  18. package/dest/global_variable_builder/index.d.ts.map +1 -1
  19. package/dest/global_variable_builder/index.js +2 -0
  20. package/dest/publisher/config.d.ts +15 -3
  21. package/dest/publisher/config.d.ts.map +1 -1
  22. package/dest/publisher/config.js +19 -4
  23. package/dest/publisher/l1_tx_failed_store/failed_tx_store.d.ts +3 -4
  24. package/dest/publisher/l1_tx_failed_store/failed_tx_store.d.ts.map +1 -1
  25. package/dest/publisher/sequencer-bundle-simulator.d.ts +96 -0
  26. package/dest/publisher/sequencer-bundle-simulator.d.ts.map +1 -0
  27. package/dest/publisher/sequencer-bundle-simulator.js +198 -0
  28. package/dest/publisher/sequencer-publisher-factory.d.ts +3 -5
  29. package/dest/publisher/sequencer-publisher-factory.d.ts.map +1 -1
  30. package/dest/publisher/sequencer-publisher-factory.js +2 -3
  31. package/dest/publisher/sequencer-publisher.d.ts +70 -64
  32. package/dest/publisher/sequencer-publisher.d.ts.map +1 -1
  33. package/dest/publisher/sequencer-publisher.js +312 -533
  34. package/dest/sequencer/automine/automine_factory.d.ts +56 -0
  35. package/dest/sequencer/automine/automine_factory.d.ts.map +1 -0
  36. package/dest/sequencer/automine/automine_factory.js +85 -0
  37. package/dest/sequencer/automine/automine_sequencer.d.ts +184 -0
  38. package/dest/sequencer/automine/automine_sequencer.d.ts.map +1 -0
  39. package/dest/sequencer/automine/automine_sequencer.js +677 -0
  40. package/dest/sequencer/automine/index.d.ts +3 -0
  41. package/dest/sequencer/automine/index.d.ts.map +1 -0
  42. package/dest/sequencer/automine/index.js +2 -0
  43. package/dest/sequencer/chain_state_overrides.d.ts +61 -0
  44. package/dest/sequencer/chain_state_overrides.d.ts.map +1 -0
  45. package/dest/sequencer/chain_state_overrides.js +98 -0
  46. package/dest/sequencer/checkpoint_proposal_job.d.ts +73 -21
  47. package/dest/sequencer/checkpoint_proposal_job.d.ts.map +1 -1
  48. package/dest/sequencer/checkpoint_proposal_job.js +709 -199
  49. package/dest/sequencer/checkpoint_proposal_job_metrics.d.ts +34 -0
  50. package/dest/sequencer/checkpoint_proposal_job_metrics.d.ts.map +1 -0
  51. package/dest/sequencer/checkpoint_proposal_job_metrics.js +72 -0
  52. package/dest/sequencer/checkpoint_voter.d.ts +1 -2
  53. package/dest/sequencer/checkpoint_voter.d.ts.map +1 -1
  54. package/dest/sequencer/checkpoint_voter.js +2 -5
  55. package/dest/sequencer/errors.d.ts +1 -8
  56. package/dest/sequencer/errors.d.ts.map +1 -1
  57. package/dest/sequencer/errors.js +0 -9
  58. package/dest/sequencer/events.d.ts +61 -4
  59. package/dest/sequencer/events.d.ts.map +1 -1
  60. package/dest/sequencer/metrics.d.ts +9 -10
  61. package/dest/sequencer/metrics.d.ts.map +1 -1
  62. package/dest/sequencer/metrics.js +34 -20
  63. package/dest/sequencer/sequencer.d.ts +112 -27
  64. package/dest/sequencer/sequencer.d.ts.map +1 -1
  65. package/dest/sequencer/sequencer.js +467 -147
  66. package/dest/sequencer/types.d.ts +2 -2
  67. package/dest/sequencer/types.d.ts.map +1 -1
  68. package/dest/test/index.d.ts +3 -3
  69. package/dest/test/index.d.ts.map +1 -1
  70. package/dest/test/utils.d.ts +15 -1
  71. package/dest/test/utils.d.ts.map +1 -1
  72. package/dest/test/utils.js +23 -6
  73. package/package.json +28 -27
  74. package/src/client/sequencer-client.ts +29 -41
  75. package/src/config.ts +57 -24
  76. package/src/global_variable_builder/README.md +44 -0
  77. package/src/global_variable_builder/fee_predictor.ts +172 -0
  78. package/src/global_variable_builder/fee_provider.ts +80 -0
  79. package/src/global_variable_builder/global_builder.ts +25 -63
  80. package/src/global_variable_builder/index.ts +3 -1
  81. package/src/publisher/config.ts +40 -6
  82. package/src/publisher/l1_tx_failed_store/failed_tx_store.ts +3 -1
  83. package/src/publisher/sequencer-bundle-simulator.ts +254 -0
  84. package/src/publisher/sequencer-publisher-factory.ts +3 -6
  85. package/src/publisher/sequencer-publisher.ts +368 -577
  86. package/src/sequencer/automine/README.md +60 -0
  87. package/src/sequencer/automine/automine_factory.ts +152 -0
  88. package/src/sequencer/automine/automine_sequencer.ts +783 -0
  89. package/src/sequencer/automine/index.ts +6 -0
  90. package/src/sequencer/chain_state_overrides.ts +169 -0
  91. package/src/sequencer/checkpoint_proposal_job.ts +818 -224
  92. package/src/sequencer/checkpoint_proposal_job_metrics.ts +128 -0
  93. package/src/sequencer/checkpoint_voter.ts +1 -12
  94. package/src/sequencer/errors.ts +0 -15
  95. package/src/sequencer/events.ts +65 -4
  96. package/src/sequencer/metrics.ts +43 -24
  97. package/src/sequencer/sequencer.ts +542 -164
  98. package/src/sequencer/types.ts +1 -1
  99. package/src/test/index.ts +2 -2
  100. package/src/test/utils.ts +59 -10
  101. package/dest/sequencer/timetable.d.ts +0 -88
  102. package/dest/sequencer/timetable.d.ts.map +0 -1
  103. package/dest/sequencer/timetable.js +0 -222
  104. package/src/sequencer/README.md +0 -531
  105. 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';
375
+ import { FeeAssetPriceOracle, MULTI_CALL_3_ADDRESS, Multicall3, MulticallForwarderRevertedError, buildSimulationOverridesStateOverride } from '@aztec/ethereum/contracts';
376
376
  import { L1FeeAnalyzer } from '@aztec/ethereum/l1-fee-analysis';
377
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';
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,30 +383,39 @@ 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';
392
+ import { getLastL1SlotTimestampForL2Slot, getNextL1SlotTimestamp, getTimestampForSlot } from '@aztec/stdlib/epoch-helpers';
394
393
  import { getTelemetryClient, trackSpan } from '@aztec/telemetry-client';
395
- import { encodeFunctionData, keccak256, multicall3Abi, toHex } from 'viem';
394
+ import { encodeFunctionData, keccak256, toHex } from 'viem';
396
395
  import { createL1TxFailedStore } from './l1_tx_failed_store/index.js';
396
+ import { SequencerBundleSimulator } from './sequencer-bundle-simulator.js';
397
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
+ }
398
408
  export const Actions = [
399
409
  'invalidate-by-invalid-attestation',
400
410
  'invalidate-by-insufficient-attestations',
401
411
  'propose',
402
412
  'governance-signal',
403
- 'empire-slashing-signal',
404
- 'create-empire-payload',
405
- 'execute-empire-payload',
406
413
  'vote-offenses',
407
414
  'execute-slash'
408
415
  ];
409
416
  // Sorting for actions such that invalidations go before proposals, and proposals go before votes
410
417
  export const compareActions = (a, b)=>Actions.indexOf(a) - Actions.indexOf(b);
411
- _dec = trackSpan('SequencerPublisher.sendRequests'), _dec1 = trackSpan('SequencerPublisher.validateBlockHeader'), _dec2 = trackSpan('SequencerPublisher.validateCheckpointForSubmission');
418
+ _dec = trackSpan('SequencerPublisher.sendRequests'), _dec1 = trackSpan('SequencerPublisher.validateBlockHeader');
412
419
  export class SequencerPublisher {
413
420
  config;
414
421
  static{
@@ -422,57 +429,55 @@ export class SequencerPublisher {
422
429
  _dec1,
423
430
  2,
424
431
  "validateBlockHeader"
425
- ],
426
- [
427
- _dec2,
428
- 2,
429
- "validateCheckpointForSubmission"
430
432
  ]
431
433
  ], []));
432
434
  }
433
435
  interrupted;
434
436
  metrics;
437
+ bundleSimulator;
435
438
  epochCache;
436
439
  failedTxStore;
437
- governanceLog;
438
- slashingLog;
440
+ /**
441
+ * ABI used to decode raw revert payloads from dropped bundle entries when the original
442
+ * request did not carry an abi (e.g. the propose request). Merges every contract the
443
+ * publisher can route to so any of their custom errors decode against it.
444
+ */ revertDecoderAbi;
439
445
  lastActions;
440
- isPayloadEmptyCache;
441
- payloadProposedCache;
442
446
  log;
443
447
  ethereumSlotDuration;
444
448
  aztecSlotDuration;
449
+ /** Date provider for wall-clock time. */ dateProvider;
445
450
  blobClient;
446
- /** Address to use for simulations in fisherman mode (actual proposer's address) */ proposerAddressForSimulation;
447
451
  /** Optional callback to obtain a replacement publisher when the current one fails to send. */ getNextPublisher;
448
452
  /** L1 fee analyzer for fisherman mode */ l1FeeAnalyzer;
449
453
  /** Fee asset price oracle for computing price modifiers from Uniswap V4 */ feeAssetPriceOracle;
450
- // A CALL to a cold address is 2700 gas
451
- static MULTICALL_OVERHEAD_GAS_GUESS = 5000n;
452
- // Gas report for VotingWithSigTest shows a max gas of 100k, but we've seen it cost 700k+ in testnet
453
- static VOTE_GAS_GUESS = 800_000n;
454
+ /** Interruptible sleep used by sendRequestsAt to wait until a target timestamp. */ interruptibleSleep;
454
455
  l1TxUtils;
455
456
  rollupContract;
456
457
  govProposerContract;
457
458
  slashingProposerContract;
458
- slashFactoryContract;
459
459
  tracer;
460
460
  requests;
461
461
  constructor(config, deps){
462
462
  this.config = config;
463
463
  this.interrupted = (_initProto(this), false);
464
- this.governanceLog = createLogger('sequencer:publisher:governance');
465
- this.slashingLog = createLogger('sequencer:publisher:slashing');
464
+ this.revertDecoderAbi = mergeAbis([
465
+ RollupAbi,
466
+ SlashingProposerAbi,
467
+ EmpireBaseAbi,
468
+ ErrorsAbi
469
+ ]);
466
470
  this.lastActions = {};
467
- this.isPayloadEmptyCache = new Map();
468
- this.payloadProposedCache = new Set();
471
+ this.interruptibleSleep = new InterruptibleSleep();
469
472
  this.requests = [];
470
473
  this.log = deps.log ?? createLogger('sequencer:publisher');
471
474
  this.ethereumSlotDuration = BigInt(config.ethereumSlotDuration);
472
475
  this.aztecSlotDuration = BigInt(config.aztecSlotDuration);
476
+ this.dateProvider = deps.dateProvider;
473
477
  this.epochCache = deps.epochCache;
474
478
  this.lastActions = deps.lastActions;
475
479
  this.blobClient = deps.blobClient;
480
+ this.dateProvider = deps.dateProvider;
476
481
  const telemetry = deps.telemetry ?? getTelemetryClient();
477
482
  this.metrics = deps.metrics ?? new SequencerPublisherMetrics(telemetry, 'SequencerPublisher');
478
483
  this.tracer = telemetry.getTracer('SequencerPublisher');
@@ -486,15 +491,20 @@ export class SequencerPublisher {
486
491
  const newSlashingProposer = await this.rollupContract.getSlashingProposer();
487
492
  this.slashingProposerContract = newSlashingProposer;
488
493
  });
489
- this.slashFactoryContract = deps.slashFactoryContract;
490
494
  // Initialize L1 fee analyzer for fisherman mode
491
495
  if (config.fishermanMode) {
492
- this.l1FeeAnalyzer = new L1FeeAnalyzer(this.l1TxUtils.client, deps.dateProvider, createLogger('sequencer:publisher:fee-analyzer'));
496
+ this.l1FeeAnalyzer = new L1FeeAnalyzer(this.l1TxUtils.client, deps.dateProvider, this.log.createChild('fee-analyzer'));
493
497
  }
494
498
  // Initialize fee asset price oracle
495
- this.feeAssetPriceOracle = new FeeAssetPriceOracle(this.l1TxUtils.client, this.rollupContract, createLogger('sequencer:publisher:price-oracle'));
499
+ this.feeAssetPriceOracle = new FeeAssetPriceOracle(this.l1TxUtils.client, this.rollupContract, this.log.createChild('price-oracle'));
496
500
  // Initialize failed L1 tx store (optional, for test networks)
497
501
  this.failedTxStore = createL1TxFailedStore(config.l1TxFailedStore, this.log);
502
+ this.bundleSimulator = new SequencerBundleSimulator({
503
+ getL1TxUtils: ()=>this.l1TxUtils,
504
+ rollupContract: this.rollupContract,
505
+ epochCache: this.epochCache,
506
+ log: this.log.createChild('bundle-simulator')
507
+ });
498
508
  }
499
509
  /**
500
510
  * Backs up a failed L1 transaction to the configured store for debugging.
@@ -517,9 +527,13 @@ export class SequencerPublisher {
517
527
  }
518
528
  /**
519
529
  * Gets the fee asset price modifier from the oracle.
520
- * Returns 0n if the oracle query fails.
521
- */ getFeeAssetPriceModifier() {
522
- return this.feeAssetPriceOracle.computePriceModifier();
530
+ *
531
+ * @param predictedParentEthPerFeeAssetE12 - Optional predicted parent eth-per-fee-asset (E12).
532
+ * Pipelined proposers should pass the value from the predicted parent fee header so the
533
+ * modifier matches the parent L1 will use when applying it.
534
+ * @returns The fee asset price modifier in basis points, or 0n if the oracle query fails.
535
+ */ getFeeAssetPriceModifier(predictedParentEthPerFeeAssetE12) {
536
+ return this.feeAssetPriceOracle.computePriceModifier(predictedParentEthPerFeeAssetE12);
523
537
  }
524
538
  getSenderAddress() {
525
539
  return this.l1TxUtils.getSenderAddress();
@@ -529,12 +543,6 @@ export class SequencerPublisher {
529
543
  */ getL1FeeAnalyzer() {
530
544
  return this.l1FeeAnalyzer;
531
545
  }
532
- /**
533
- * Sets the proposer address to use for simulations in fisherman mode.
534
- * @param proposerAddress - The actual proposer's address to use for balance lookups in simulations
535
- */ setProposerAddressForSimulation(proposerAddress) {
536
- this.proposerAddressForSimulation = proposerAddress;
537
- }
538
546
  addRequest(request) {
539
547
  this.requests.push(request);
540
548
  }
@@ -592,11 +600,15 @@ export class SequencerPublisher {
592
600
  }
593
601
  /**
594
602
  * Sends all requests that are still valid.
603
+ * @param targetSlot - The target L2 slot for this send. When provided (the production path, via
604
+ * sendRequestsAt), it is threaded into bundleSimulate so the block.timestamp override matches
605
+ * the slot the propose is built for. When omitted, falls back to getCurrentL2Slot() for the
606
+ * AutomineSequencer, which publishes synchronously within the current slot.
595
607
  * @returns one of:
596
608
  * - A receipt and stats if the tx succeeded
597
609
  * - a receipt and errorMsg if it failed on L1
598
610
  * - undefined if no valid requests are found OR the tx failed to send.
599
- */ async sendRequests() {
611
+ */ async sendRequests(targetSlot) {
600
612
  const requestsToProcess = [
601
613
  ...this.requests
602
614
  ];
@@ -604,10 +616,9 @@ export class SequencerPublisher {
604
616
  if (this.interrupted || requestsToProcess.length === 0) {
605
617
  return undefined;
606
618
  }
607
- const currentL2Slot = this.getCurrentL2Slot();
619
+ const currentL2Slot = targetSlot ?? this.getCurrentL2Slot();
608
620
  this.log.debug(`Sending requests on L2 slot ${currentL2Slot}`);
609
621
  const validRequests = requestsToProcess.filter((request)=>request.lastValidL2Slot >= currentL2Slot);
610
- const validActions = validRequests.map((x)=>x.action);
611
622
  const expiredActions = requestsToProcess.filter((request)=>request.lastValidL2Slot < currentL2Slot).map((x)=>x.action);
612
623
  if (validRequests.length !== requestsToProcess.length) {
613
624
  this.log.warn(`Some requests were expired for slot ${currentL2Slot}`, {
@@ -625,71 +636,53 @@ export class SequencerPublisher {
625
636
  this.log.debug(`No valid requests to send`);
626
637
  return undefined;
627
638
  }
628
- // @note - we can only have one blob config per bundle
629
- // find requests with gas and blob configs
630
- // See https://github.com/AztecProtocol/aztec-packages/issues/11513
639
+ // Collect earliest txTimeoutAt across all requests.
631
640
  const gasConfigs = validRequests.filter((request)=>request.gasConfig).map((request)=>request.gasConfig);
632
- const blobConfigs = validRequests.filter((request)=>request.blobConfig).map((request)=>request.blobConfig);
633
- if (blobConfigs.length > 1) {
634
- throw new Error('Multiple blob configs found');
635
- }
636
- const blobConfig = blobConfigs[0];
637
- // Merge gasConfigs. Yields the sum of gasLimits, and the earliest txTimeoutAt, or undefined if no gasConfig sets them.
638
- const gasLimits = gasConfigs.map((g)=>g?.gasLimit).filter((g)=>g !== undefined);
639
- let gasLimit = gasLimits.length > 0 ? sumBigint(gasLimits) : undefined; // sum
640
- // Cap at L1 block gas limit so the node accepts the tx ("gas limit too high" otherwise).
641
- const maxGas = MAX_L1_TX_LIMIT;
642
- if (gasLimit !== undefined && gasLimit > maxGas) {
643
- this.log.debug('Capping bundled tx gas limit to L1 max', {
644
- requested: gasLimit,
645
- capped: maxGas
646
- });
647
- gasLimit = maxGas;
648
- }
649
641
  const txTimeoutAts = gasConfigs.map((g)=>g?.txTimeoutAt).filter((g)=>g !== undefined);
650
- const txTimeoutAt = txTimeoutAts.length > 0 ? new Date(Math.min(...txTimeoutAts.map((g)=>g.getTime()))) : undefined; // earliest
651
- const txConfig = {
652
- gasLimit,
653
- txTimeoutAt
654
- };
642
+ const txTimeoutAt = txTimeoutAts.length > 0 ? new Date(Math.min(...txTimeoutAts.map((g)=>g.getTime()))) : undefined;
655
643
  // Sort the requests so that proposals always go first
656
644
  // This ensures the committee gets precomputed correctly
657
645
  validRequests.sort((a, b)=>compareActions(a.action, b.action));
658
646
  try {
659
- // Capture context for failed tx backup before sending
660
- const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
661
- const multicallData = encodeFunctionData({
662
- abi: multicall3Abi,
663
- functionName: 'aggregate3',
664
- args: [
665
- validRequests.map((r)=>({
666
- target: r.request.to,
667
- callData: r.request.data,
668
- allowFailure: true
669
- }))
670
- ]
671
- });
672
- const blobDataHex = blobConfig?.blobs?.map((b)=>toHex(b));
673
- const txContext = {
674
- multicallData,
675
- blobData: blobDataHex,
676
- l1BlockNumber
647
+ // Bundle-level eth_simulateV1: filters out entries that revert and derives the gasLimit.
648
+ const bundleResult = await this.bundleSimulator.simulate(validRequests, currentL2Slot);
649
+ if (bundleResult.kind === 'aborted') {
650
+ this.logDroppedInSim(bundleResult.droppedRequests);
651
+ void this.backupDroppedInSim(bundleResult.droppedRequests);
652
+ return undefined;
653
+ }
654
+ const { requests, droppedRequests, gasLimit } = bundleResult.kind === 'fallback' ? {
655
+ requests: bundleResult.requests,
656
+ droppedRequests: bundleResult.droppedRequests,
657
+ gasLimit: MAX_L1_TX_LIMIT
658
+ } : bundleResult;
659
+ this.logDroppedInSim(droppedRequests);
660
+ // Compute blobConfig from survivors (not original validRequests) so that if the propose
661
+ // entry was dropped by bundleSimulate we don't attach a blob-typed config to a non-blob tx.
662
+ const [blobConfig] = requests.filter((r)=>r.blobConfig).map((r)=>r.blobConfig);
663
+ const txConfig = {
664
+ gasLimit,
665
+ txTimeoutAt
677
666
  };
678
667
  this.log.debug('Forwarding transactions', {
679
- validRequests: validRequests.map((request)=>request.action),
668
+ requests: requests.map((request)=>request.action),
680
669
  txConfig
681
670
  });
682
- const result = await this.forwardWithPublisherRotation(validRequests, txConfig, blobConfig);
671
+ const result = await this.forwardWithPublisherRotation(requests, txConfig, blobConfig);
683
672
  if (result === undefined) {
684
673
  return undefined;
685
674
  }
686
- const { successfulActions = [], failedActions = [] } = this.callbackBundledTransactions(validRequests, result, txContext);
675
+ const { successfulActions = [], failedActions = [] } = this.callbackBundledTransactions(requests, result);
676
+ const allFailedActions = [
677
+ ...failedActions,
678
+ ...droppedRequests.map((d)=>d.request.action)
679
+ ];
687
680
  return {
688
681
  result,
689
682
  expiredActions,
690
- sentActions: validActions,
683
+ sentActions: requests.map((x)=>x.action),
691
684
  successfulActions,
692
- failedActions
685
+ failedActions: allFailedActions
693
686
  };
694
687
  } catch (err) {
695
688
  const viemError = formatViemError(err);
@@ -703,23 +696,78 @@ export class SequencerPublisher {
703
696
  }
704
697
  }
705
698
  }
699
+ /** Logs entries dropped by bundle simulation as warnings on the publisher's logger. */ logDroppedInSim(dropped) {
700
+ for (const drop of dropped){
701
+ const revertReasonDecoded = drop.revertReason ?? tryDecodeRevertReason(drop.returnData, this.revertDecoderAbi);
702
+ this.log.warn('Bundle entry dropped: action reverted in sim', {
703
+ action: drop.request.action,
704
+ revertReason: revertReasonDecoded ?? drop.returnData,
705
+ revertReasonDecoded,
706
+ returnData: drop.returnData
707
+ });
708
+ }
709
+ }
710
+ /** Backs up entries dropped by bundle simulation, one record per dropped action. */ async backupDroppedInSim(dropped) {
711
+ if (dropped.length === 0) {
712
+ return;
713
+ }
714
+ const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
715
+ for (const { request: req } of dropped){
716
+ this.backupFailedTx({
717
+ id: keccak256(req.request.data),
718
+ failureType: 'simulation',
719
+ request: {
720
+ to: req.request.to,
721
+ data: req.request.data
722
+ },
723
+ l1BlockNumber: l1BlockNumber.toString(),
724
+ error: {
725
+ message: 'Bundle entry dropped: action reverted in sim'
726
+ },
727
+ context: {
728
+ actions: [
729
+ req.action
730
+ ],
731
+ sender: this.getSenderAddress().toString()
732
+ }
733
+ });
734
+ }
735
+ }
706
736
  /**
707
737
  * Forwards transactions via Multicall3, rotating to the next available publisher if a send
708
738
  * failure occurs (i.e. the tx never reached the chain).
709
739
  * On-chain reverts and simulation errors are returned as-is without rotation.
710
740
  */ async forwardWithPublisherRotation(validRequests, txConfig, blobConfig) {
741
+ if (!txConfig?.gasLimit) {
742
+ throw new Error('gasLimit is required for bundled transactions');
743
+ }
744
+ const txConfigWithGasLimit = txConfig;
711
745
  const triedAddresses = [];
712
746
  let currentPublisher = this.l1TxUtils;
713
747
  while(true){
748
+ if (txConfig.txTimeoutAt && new Date() > txConfig.txTimeoutAt) {
749
+ this.log.warn(`Tx timeout (${txConfig.txTimeoutAt.toISOString()}) elapsed; stopping publisher rotation`, {
750
+ triedAddresses: triedAddresses.map((a)=>a.toString())
751
+ });
752
+ return undefined;
753
+ }
714
754
  triedAddresses.push(currentPublisher.getSenderAddress());
715
755
  try {
716
- const result = await Multicall3.forward(validRequests.map((r)=>r.request), currentPublisher, txConfig, blobConfig, this.rollupContract.address, this.log);
756
+ const result = await Multicall3.forward(validRequests.map((r)=>r.request), currentPublisher, txConfigWithGasLimit, blobConfig, {
757
+ gasLimitRequired: true
758
+ });
717
759
  this.l1TxUtils = currentPublisher;
718
760
  return result;
719
761
  } catch (err) {
720
762
  if (err instanceof TimeoutError) {
721
763
  throw err;
722
764
  }
765
+ if (err instanceof MulticallForwarderRevertedError) {
766
+ this.log.error('Forwarder transaction reverted on-chain; not rotating publisher', err, {
767
+ transactionHash: err.receipt.transactionHash
768
+ });
769
+ return undefined;
770
+ }
723
771
  const viemError = formatViemError(err);
724
772
  if (!this.getNextPublisher) {
725
773
  this.log.error('Failed to publish bundled transactions', viemError);
@@ -730,121 +778,94 @@ export class SequencerPublisher {
730
778
  ...triedAddresses
731
779
  ]);
732
780
  if (!nextPublisher) {
733
- this.log.error('All available publishers exhausted, failed to publish bundled transactions');
781
+ this.log.error(`All available publishers exhausted (tried ${triedAddresses.length}), failed to publish bundled transactions`, viemError, {
782
+ triedAddresses: triedAddresses.map((a)=>a.toString())
783
+ });
734
784
  return undefined;
735
785
  }
736
786
  currentPublisher = nextPublisher;
737
787
  }
738
788
  }
739
789
  }
740
- callbackBundledTransactions(requests, result, txContext) {
741
- const actionsListStr = requests.map((r)=>r.action).join(', ');
742
- if (result instanceof FormattedViemError) {
743
- this.log.error(`Failed to publish bundled transactions (${actionsListStr})`, result);
744
- this.backupFailedTx({
745
- id: keccak256(txContext.multicallData),
746
- failureType: 'send-error',
747
- request: {
748
- to: MULTI_CALL_3_ADDRESS,
749
- data: txContext.multicallData
750
- },
751
- blobData: txContext.blobData,
752
- l1BlockNumber: txContext.l1BlockNumber.toString(),
753
- error: {
754
- message: result.message,
755
- name: result.name
756
- },
757
- context: {
758
- actions: requests.map((r)=>r.action),
759
- requests: requests.map((r)=>({
760
- action: r.action,
761
- to: r.request.to,
762
- data: r.request.data
763
- })),
764
- sender: this.getSenderAddress().toString()
765
- }
766
- });
767
- return {
768
- failedActions: requests.map((r)=>r.action)
769
- };
770
- } else {
771
- this.log.verbose(`Published bundled transactions (${actionsListStr})`, {
772
- result,
773
- requests: requests.map((r)=>({
774
- ...r,
775
- // Avoid logging large blob data
776
- blobConfig: r.blobConfig ? {
777
- ...r.blobConfig,
778
- blobs: r.blobConfig.blobs.map((b)=>({
779
- size: trimmedBytesLength(b)
780
- }))
781
- } : undefined
782
- }))
790
+ /*
791
+ * Schedules sending all enqueued requests at (or after) the start of the given L2 slot.
792
+ * Sleeps until one L1 slot before the L2 slot boundary so the tx has a chance of being
793
+ * picked up by the first L1 block of the L2 slot.
794
+ * NB: there is a known correctness risk — being included in the L1 block right before the
795
+ * L2 slot starts would revert propose with HeaderLib__InvalidSlotNumber.
796
+ * Uses InterruptibleSleep so it can be cancelled via interrupt().
797
+ */ async sendRequestsAt(targetSlot) {
798
+ const l1Constants = this.epochCache.getL1Constants();
799
+ // Start of the target L2 slot, in ms (getTimestampForSlot returns seconds).
800
+ const startOfTargetSlotMs = Number(getTimestampForSlot(targetSlot, l1Constants)) * 1000;
801
+ // Aim to be in the mempool one L1 slot before the L2 slot starts, so we have a chance of
802
+ // being picked up by the first L1 block of the L2 slot.
803
+ const submitAfterMs = startOfTargetSlotMs - Number(this.ethereumSlotDuration) * 1000;
804
+ if (this.interrupted) {
805
+ return undefined;
806
+ }
807
+ const sleepMs = submitAfterMs - this.dateProvider.now();
808
+ if (sleepMs > 0) {
809
+ this.log.debug(`Sleeping ${sleepMs}ms before sending requests`, {
810
+ targetSlot,
811
+ submitAfterMs
783
812
  });
784
- const successfulActions = [];
785
- const failedActions = [];
786
- for (const request of requests){
787
- if (request.checkSuccess(request.request, result)) {
788
- successfulActions.push(request.action);
789
- } else {
790
- failedActions.push(request.action);
791
- }
792
- }
793
- // Single backup for the whole reverted tx
794
- if (failedActions.length > 0 && result?.receipt?.status === 'reverted') {
795
- this.backupFailedTx({
796
- id: result.receipt.transactionHash,
797
- failureType: 'revert',
798
- request: {
799
- to: MULTI_CALL_3_ADDRESS,
800
- data: txContext.multicallData
801
- },
802
- blobData: txContext.blobData,
803
- l1BlockNumber: result.receipt.blockNumber.toString(),
804
- receipt: {
805
- transactionHash: result.receipt.transactionHash,
806
- blockNumber: result.receipt.blockNumber.toString(),
807
- gasUsed: (result.receipt.gasUsed ?? 0n).toString(),
808
- status: 'reverted'
809
- },
810
- error: {
811
- message: result.errorMsg ?? 'Transaction reverted'
812
- },
813
- context: {
814
- actions: failedActions,
815
- requests: requests.filter((r)=>failedActions.includes(r.action)).map((r)=>({
816
- action: r.action,
817
- to: r.request.to,
818
- data: r.request.data
819
- })),
820
- sender: this.getSenderAddress().toString()
821
- }
822
- });
813
+ await this.interruptibleSleep.sleep(sleepMs);
814
+ }
815
+ if (this.interrupted) {
816
+ return undefined;
817
+ }
818
+ return this.sendRequests(targetSlot);
819
+ }
820
+ callbackBundledTransactions(requests, result) {
821
+ const actionsListStr = requests.map((r)=>r.action).join(', ');
822
+ this.log.verbose(`Published bundled transactions (${actionsListStr})`, {
823
+ result,
824
+ requests: requests.map((r)=>({
825
+ ...r,
826
+ // Avoid logging large blob data
827
+ blobConfig: r.blobConfig ? {
828
+ ...r.blobConfig,
829
+ blobs: r.blobConfig.blobs.map((b)=>({
830
+ size: trimmedBytesLength(b)
831
+ }))
832
+ } : undefined
833
+ }))
834
+ });
835
+ const successfulActions = [];
836
+ const failedActions = [];
837
+ for (const request of requests){
838
+ if (request.checkSuccess(request.request, result)) {
839
+ successfulActions.push(request.action);
840
+ } else {
841
+ failedActions.push(request.action);
823
842
  }
824
- return {
825
- successfulActions,
826
- failedActions
827
- };
828
843
  }
844
+ return {
845
+ successfulActions,
846
+ failedActions
847
+ };
829
848
  }
830
849
  /**
831
850
  * @notice Will call `canProposeAt` to make sure that it is possible to propose
832
851
  * @param tipArchive - The archive to check
833
852
  * @returns The slot and block number if it is possible to propose, undefined otherwise
834
- */ canProposeAt(tipArchive, msgSender, opts = {}) {
853
+ */ async canProposeAt(tipArchive, msgSender, simulationOverridesPlan) {
835
854
  // TODO: #14291 - should loop through multiple keys to check if any of them can propose
836
- const ignoredErrors = [
855
+ // These errors are expected when we cannot actually propose right now — usually because our
856
+ // local view of the chain is ahead of L1 (proposed parent hasn't landed yet, or someone
857
+ // else has just landed the slot, or the archive override doesn't match). We log a warn and
858
+ // skip the proposal; we do NOT treat these as bugs.
859
+ const expectedErrors = [
837
860
  'SlotAlreadyInChain',
838
861
  'InvalidProposer',
839
862
  'InvalidArchive'
840
863
  ];
841
- const pipelined = opts.pipelined ?? this.epochCache.isProposerPipeliningEnabled();
842
- const slotOffset = pipelined ? this.aztecSlotDuration : 0n;
843
- return this.rollupContract.canProposeAt(tipArchive.toBuffer(), msgSender.toString(), this.ethereumSlotDuration, slotOffset, {
844
- forcePendingCheckpointNumber: opts.forcePendingCheckpointNumber
845
- }).catch((err)=>{
846
- if (err instanceof FormattedViemError && ignoredErrors.find((e)=>err.message.includes(e))) {
847
- this.log.warn(`Failed canProposeAtTime check with ${ignoredErrors.find((e)=>err.message.includes(e))}`, {
864
+ const slotOffset = this.aztecSlotDuration;
865
+ const nextL1SlotTs = this.getNextL1SlotTimestamp() + slotOffset;
866
+ return this.rollupContract.canProposeAt(tipArchive.toBuffer(), msgSender.toString(), nextL1SlotTs, await buildSimulationOverridesStateOverride(this.rollupContract, simulationOverridesPlan)).catch((err)=>{
867
+ if (err instanceof FormattedViemError && expectedErrors.find((e)=>err.message.includes(e))) {
868
+ this.log.warn(`Failed canProposeAtTime check with ${expectedErrors.find((e)=>err.message.includes(e))}`, {
848
869
  error: err.message
849
870
  });
850
871
  } else {
@@ -858,22 +879,23 @@ export class SequencerPublisher {
858
879
  * @dev This is a convenience function that can be used by the sequencer to validate a "partial" header.
859
880
  * It will throw if the block header is invalid.
860
881
  * @param header - The block header to validate
861
- */ async validateBlockHeader(header, opts) {
882
+ */ async validateBlockHeader(header, simulationOverridesPlan) {
862
883
  const flags = {
863
884
  ignoreDA: true,
864
885
  ignoreSignatures: true
865
886
  };
866
887
  const args = [
867
888
  header.toViem(),
868
- CommitteeAttestationsAndSigners.empty().getPackedAttestations(),
889
+ CommitteeAttestationsAndSigners.packAttestations([]),
869
890
  [],
870
891
  Signature.empty().toViemSignature(),
871
892
  `0x${'0'.repeat(64)}`,
872
893
  header.blobsHash.toString(),
873
894
  flags
874
895
  ];
875
- const ts = BigInt((await this.l1TxUtils.getBlock()).timestamp + this.ethereumSlotDuration);
876
- const stateOverrides = await this.rollupContract.makePendingCheckpointNumberOverride(opts?.forcePendingCheckpointNumber);
896
+ const l1Constants = this.epochCache.getL1Constants();
897
+ const ts = getLastL1SlotTimestampForL2Slot(header.slotNumber, l1Constants);
898
+ const stateOverrides = await buildSimulationOverridesStateOverride(this.rollupContract, simulationOverridesPlan);
877
899
  let balance = 0n;
878
900
  if (this.config.fishermanMode) {
879
901
  // In fisherman mode, we can't know where the proposer is publishing from
@@ -895,7 +917,7 @@ export class SequencerPublisher {
895
917
  }),
896
918
  from: MULTI_CALL_3_ADDRESS
897
919
  }, {
898
- time: ts + 1n
920
+ time: ts
899
921
  }, stateOverrides);
900
922
  this.log.debug(`Simulated validateHeader`);
901
923
  }
@@ -941,6 +963,7 @@ export class SequencerPublisher {
941
963
  gasUsed,
942
964
  checkpointNumber,
943
965
  forcePendingCheckpointNumber: CheckpointNumber(checkpointNumber - 1),
966
+ lastArchive: validationResult.checkpoint.lastArchive,
944
967
  reason
945
968
  };
946
969
  } catch (err) {
@@ -953,8 +976,8 @@ export class SequencerPublisher {
953
976
  request,
954
977
  error: viemError.message
955
978
  });
956
- const latestPendingCheckpointNumber = await this.rollupContract.getCheckpointNumber();
957
- if (latestPendingCheckpointNumber < checkpointNumber) {
979
+ const latestProposedCheckpointNumber = await this.rollupContract.getCheckpointNumber();
980
+ if (latestProposedCheckpointNumber < checkpointNumber) {
958
981
  this.log.verbose(`Checkpoint ${checkpointNumber} has already been invalidated`, {
959
982
  ...logData
960
983
  });
@@ -1004,7 +1027,7 @@ export class SequencerPublisher {
1004
1027
  reason
1005
1028
  };
1006
1029
  this.log.debug(`Building invalidate checkpoint ${checkpoint.checkpointNumber} request`, logData);
1007
- const attestationsAndSigners = new CommitteeAttestationsAndSigners(validationResult.attestations).getPackedAttestations();
1030
+ const attestationsAndSigners = CommitteeAttestationsAndSigners.packAttestations(validationResult.attestations);
1008
1031
  if (reason === 'invalid-attestation') {
1009
1032
  return this.rollupContract.buildInvalidateBadAttestationRequest(checkpoint.checkpointNumber, attestationsAndSigners, committee, validationResult.invalidIndex);
1010
1033
  } else if (reason === 'insufficient-attestations') {
@@ -1014,30 +1037,7 @@ export class SequencerPublisher {
1014
1037
  throw new Error(`Unknown reason for invalidation`);
1015
1038
  }
1016
1039
  }
1017
- /** Simulates `propose` to make sure that the checkpoint is valid for submission */ async validateCheckpointForSubmission(checkpoint, attestationsAndSigners, attestationsAndSignersSignature, options) {
1018
- // Anchor the simulation timestamp to the checkpoint's own slot start time
1019
- // rather than the current L1 block timestamp, which may overshoot into the next slot if the build ran late.
1020
- const ts = checkpoint.header.timestamp;
1021
- const blobFields = checkpoint.toBlobFields();
1022
- const blobs = await getBlobsPerL1Block(blobFields);
1023
- const blobInput = getPrefixedEthBlobCommitments(blobs);
1024
- const args = [
1025
- {
1026
- header: checkpoint.header.toViem(),
1027
- archive: toHex(checkpoint.archive.root.toBuffer()),
1028
- oracleInput: {
1029
- feeAssetPriceModifier: checkpoint.feeAssetPriceModifier
1030
- }
1031
- },
1032
- attestationsAndSigners.getPackedAttestations(),
1033
- attestationsAndSigners.getSigners().map((signer)=>signer.toString()),
1034
- attestationsAndSignersSignature.toViemSignature(),
1035
- blobInput
1036
- ];
1037
- await this.simulateProposeTx(args, ts, options);
1038
- return ts;
1039
- }
1040
- async enqueueCastSignalHelper(slotNumber, timestamp, signalType, payload, base, signerAddress, signer) {
1040
+ async enqueueCastSignalHelper(slotNumber, signalType, payload, base, signerAddress, signer) {
1041
1041
  if (this.lastActions[signalType] && this.lastActions[signalType] === slotNumber) {
1042
1042
  this.log.debug(`Skipping duplicate vote cast signal ${signalType} for slot ${slotNumber}`);
1043
1043
  return false;
@@ -1057,30 +1057,26 @@ export class SequencerPublisher {
1057
1057
  if (roundInfo.lastSignalSlot >= slotNumber) {
1058
1058
  return false;
1059
1059
  }
1060
- if (await this.isPayloadEmpty(payload)) {
1060
+ if (await base.isPayloadEmpty(payload)) {
1061
1061
  this.log.warn(`Skipping vote cast for payload with empty code`);
1062
1062
  return false;
1063
1063
  }
1064
- // Check if payload was already submitted to governance
1065
- const cacheKey = payload.toString();
1066
- if (!this.payloadProposedCache.has(cacheKey)) {
1067
- try {
1068
- const l1StartBlock = await this.rollupContract.getL1StartBlock();
1069
- const proposed = await retry(()=>base.hasPayloadBeenProposed(payload.toString(), l1StartBlock), 'Check if payload was proposed', makeBackoff([
1070
- 0,
1071
- 1,
1072
- 2
1073
- ]), this.log, true);
1074
- if (proposed) {
1075
- this.payloadProposedCache.add(cacheKey);
1076
- }
1077
- } catch (err) {
1078
- this.log.warn(`Failed to check if payload ${payload} was proposed after retries, skipping signal`, err);
1079
- return false;
1080
- }
1064
+ // Skip signaling if there is already a live (non-terminal) Governance proposal for this
1065
+ // payload. This is intentionally not cached: a previously-live proposal may transition to
1066
+ // a terminal state (Dropped/Rejected/Expired/Executed), at which point we may want to re-signal
1067
+ // the same payload in a future round.
1068
+ let proposed = false;
1069
+ try {
1070
+ proposed = await base.hasActiveProposalWithPayload(payload.toString());
1071
+ } catch (err) {
1072
+ // We deliberately swallow the error and proceed to signal. Failing closed (skipping the
1073
+ // signal) on transient RPC errors would let a flaky L1 endpoint silence governance
1074
+ // participation entirely; failing open at worst produces a duplicate signal that the
1075
+ // contract will simply count alongside others in the round.
1076
+ this.log.error(`Failed to check if payload ${payload} was already proposed (signalling anyway)`, err);
1081
1077
  }
1082
- if (this.payloadProposedCache.has(cacheKey)) {
1083
- this.log.info(`Payload ${payload} was already proposed to governance, stopping signals`);
1078
+ if (proposed) {
1079
+ this.log.info(`Payload ${payload} has a live governance proposal, stopping signals`);
1084
1080
  return false;
1085
1081
  }
1086
1082
  const cachedLastVote = this.lastActions[signalType];
@@ -1093,53 +1089,17 @@ export class SequencerPublisher {
1093
1089
  signer: this.l1TxUtils.client.account?.address,
1094
1090
  lastValidL2Slot: slotNumber
1095
1091
  });
1096
- const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
1097
- try {
1098
- await this.l1TxUtils.simulate(request, {
1099
- time: timestamp
1100
- }, [], mergeAbis([
1101
- request.abi ?? [],
1102
- ErrorsAbi
1103
- ]));
1104
- this.log.debug(`Simulation for ${action} at slot ${slotNumber} succeeded`, {
1105
- request
1106
- });
1107
- } catch (err) {
1108
- const viemError = formatViemError(err);
1109
- this.log.error(`Failed simulation for ${action} at slot ${slotNumber} (enqueuing the action anyway)`, viemError);
1110
- this.backupFailedTx({
1111
- id: keccak256(request.data),
1112
- failureType: 'simulation',
1113
- request: {
1114
- to: request.to,
1115
- data: request.data,
1116
- value: request.value?.toString()
1117
- },
1118
- l1BlockNumber: l1BlockNumber.toString(),
1119
- error: {
1120
- message: viemError.message,
1121
- name: viemError.name
1122
- },
1123
- context: {
1124
- actions: [
1125
- action
1126
- ],
1127
- slot: slotNumber,
1128
- sender: this.getSenderAddress().toString()
1129
- }
1130
- });
1131
- // Yes, we enqueue the request anyway, in case there was a bug with the simulation itself
1132
- }
1133
1092
  // TODO(palla/slash): All votes (governance and slashing) should txTimeoutAt at the end of the slot.
1134
1093
  this.addRequest({
1135
- gasConfig: {
1136
- gasLimit: SequencerPublisher.VOTE_GAS_GUESS
1137
- },
1138
1094
  action,
1139
1095
  request,
1140
1096
  lastValidL2Slot: slotNumber,
1141
1097
  checkSuccess: (_request, result)=>{
1142
- const success = result && result.receipt && result.receipt.status === 'success' && tryExtractEvent(result.receipt.logs, base.address.toString(), EmpireBaseAbi, 'SignalCast');
1098
+ const success = result && extractEventSuccess(result.receipt, {
1099
+ address: base.address.toString(),
1100
+ abi: EmpireBaseAbi,
1101
+ eventName: 'SignalCast'
1102
+ });
1143
1103
  const logData = {
1144
1104
  ...result,
1145
1105
  slotNumber,
@@ -1158,68 +1118,20 @@ export class SequencerPublisher {
1158
1118
  });
1159
1119
  return true;
1160
1120
  }
1161
- async isPayloadEmpty(payload) {
1162
- const key = payload.toString();
1163
- const cached = this.isPayloadEmptyCache.get(key);
1164
- if (cached) {
1165
- return cached;
1166
- }
1167
- const isEmpty = !await this.l1TxUtils.getCode(payload);
1168
- this.isPayloadEmptyCache.set(key, isEmpty);
1169
- return isEmpty;
1170
- }
1171
1121
  /**
1172
1122
  * Enqueues a governance castSignal transaction to cast a signal for a given slot number.
1173
1123
  * @param slotNumber - The slot number to cast a signal for.
1174
- * @param timestamp - The timestamp of the slot to cast a signal for.
1175
1124
  * @returns True if the signal was successfully enqueued, false otherwise.
1176
- */ enqueueGovernanceCastSignal(governancePayload, slotNumber, timestamp, signerAddress, signer) {
1177
- return this.enqueueCastSignalHelper(slotNumber, timestamp, 'governance-signal', governancePayload, this.govProposerContract, signerAddress, signer);
1125
+ */ enqueueGovernanceCastSignal(governancePayload, slotNumber, signerAddress, signer) {
1126
+ return this.enqueueCastSignalHelper(slotNumber, 'governance-signal', governancePayload, this.govProposerContract, signerAddress, signer);
1178
1127
  }
1179
- /** Enqueues all slashing actions as returned by the slasher client. */ async enqueueSlashingActions(actions, slotNumber, timestamp, signerAddress, signer) {
1128
+ /** Enqueues all slashing actions as returned by the slasher client. */ async enqueueSlashingActions(actions, slotNumber, signerAddress, signer) {
1180
1129
  if (actions.length === 0) {
1181
1130
  this.log.debug(`No slashing actions to enqueue for slot ${slotNumber}`);
1182
1131
  return false;
1183
1132
  }
1184
1133
  for (const action of actions){
1185
1134
  switch(action.type){
1186
- case 'vote-empire-payload':
1187
- {
1188
- if (this.slashingProposerContract?.type !== 'empire') {
1189
- this.log.error('Cannot vote for empire payload on non-empire slashing contract');
1190
- break;
1191
- }
1192
- this.log.debug(`Enqueuing slashing vote for payload ${action.payload} at slot ${slotNumber}`, {
1193
- signerAddress
1194
- });
1195
- await this.enqueueCastSignalHelper(slotNumber, timestamp, 'empire-slashing-signal', action.payload, this.slashingProposerContract, signerAddress, signer);
1196
- break;
1197
- }
1198
- case 'create-empire-payload':
1199
- {
1200
- this.log.debug(`Enqueuing slashing create payload at slot ${slotNumber}`, {
1201
- slotNumber,
1202
- signerAddress
1203
- });
1204
- const request = this.slashFactoryContract.buildCreatePayloadRequest(action.data);
1205
- await this.simulateAndEnqueueRequest('create-empire-payload', request, (receipt)=>!!this.slashFactoryContract.tryExtractSlashPayloadCreatedEvent(receipt.logs), slotNumber, timestamp);
1206
- break;
1207
- }
1208
- case 'execute-empire-payload':
1209
- {
1210
- this.log.debug(`Enqueuing slashing execute payload at slot ${slotNumber}`, {
1211
- slotNumber,
1212
- signerAddress
1213
- });
1214
- if (this.slashingProposerContract?.type !== 'empire') {
1215
- this.log.error('Cannot execute slashing payload on non-empire slashing contract');
1216
- return false;
1217
- }
1218
- const empireSlashingProposer = this.slashingProposerContract;
1219
- const request = empireSlashingProposer.buildExecuteRoundRequest(action.round);
1220
- await this.simulateAndEnqueueRequest('execute-empire-payload', request, (receipt)=>!!empireSlashingProposer.tryExtractPayloadSubmittedEvent(receipt.logs), slotNumber, timestamp);
1221
- break;
1222
- }
1223
1135
  case 'vote-offenses':
1224
1136
  {
1225
1137
  this.log.debug(`Enqueuing slashing vote for ${action.votes.length} votes at slot ${slotNumber}`, {
@@ -1228,14 +1140,17 @@ export class SequencerPublisher {
1228
1140
  votesCount: action.votes.length,
1229
1141
  signerAddress
1230
1142
  });
1231
- if (this.slashingProposerContract?.type !== 'tally') {
1232
- this.log.error('Cannot vote for slashing offenses on non-tally slashing contract');
1143
+ if (!this.slashingProposerContract) {
1144
+ this.log.error('No slashing proposer contract available');
1233
1145
  return false;
1234
1146
  }
1235
- const tallySlashingProposer = this.slashingProposerContract;
1236
1147
  const votes = bufferToHex(encodeSlashConsensusVotes(action.votes));
1237
- const request = await tallySlashingProposer.buildVoteRequestFromSigner(votes, slotNumber, signer);
1238
- await this.simulateAndEnqueueRequest('vote-offenses', request, (receipt)=>!!tallySlashingProposer.tryExtractVoteCastEvent(receipt.logs), slotNumber, timestamp);
1148
+ const request = await this.slashingProposerContract.buildVoteRequestFromSigner(votes, slotNumber, signer);
1149
+ this.enqueueRequest('vote-offenses', request, {
1150
+ address: this.slashingProposerContract.address.toString(),
1151
+ abi: SlashingProposerAbi,
1152
+ eventName: 'VoteCast'
1153
+ }, slotNumber);
1239
1154
  break;
1240
1155
  }
1241
1156
  case 'execute-slash':
@@ -1245,13 +1160,16 @@ export class SequencerPublisher {
1245
1160
  round: action.round,
1246
1161
  signerAddress
1247
1162
  });
1248
- if (this.slashingProposerContract?.type !== 'tally') {
1249
- this.log.error('Cannot execute slashing offenses on non-tally slashing contract');
1163
+ if (!this.slashingProposerContract) {
1164
+ this.log.error('No slashing proposer contract available');
1250
1165
  return false;
1251
1166
  }
1252
- const tallySlashingProposer = this.slashingProposerContract;
1253
- const request = tallySlashingProposer.buildExecuteRoundRequest(action.round, action.committees);
1254
- await this.simulateAndEnqueueRequest('execute-slash', request, (receipt)=>!!tallySlashingProposer.tryExtractRoundExecutedEvent(receipt.logs), slotNumber, timestamp);
1167
+ const executeRequest = this.slashingProposerContract.buildExecuteRoundRequest(action.round, action.committees);
1168
+ this.enqueueRequest('execute-slash', executeRequest, {
1169
+ address: this.slashingProposerContract.address.toString(),
1170
+ abi: SlashingProposerAbi,
1171
+ eventName: 'RoundExecuted'
1172
+ }, slotNumber);
1255
1173
  break;
1256
1174
  }
1257
1175
  default:
@@ -1263,7 +1181,7 @@ export class SequencerPublisher {
1263
1181
  }
1264
1182
  return true;
1265
1183
  }
1266
- /** Simulates and enqueues a proposal for a checkpoint on L1 */ async enqueueProposeCheckpoint(checkpoint, attestationsAndSigners, attestationsAndSignersSignature, opts = {}) {
1184
+ /** Enqueues a proposal for a checkpoint on L1 */ async enqueueProposeCheckpoint(checkpoint, attestationsAndSigners, attestationsAndSignersSignature, opts = {}) {
1267
1185
  const checkpointHeader = checkpoint.header;
1268
1186
  const blobFields = checkpoint.toBlobFields();
1269
1187
  const blobs = await getBlobsPerL1Block(blobFields);
@@ -1275,52 +1193,38 @@ export class SequencerPublisher {
1275
1193
  attestationsAndSignersSignature,
1276
1194
  feeAssetPriceModifier: checkpoint.feeAssetPriceModifier
1277
1195
  };
1278
- let ts;
1279
- try {
1280
- // @note This will make sure that we are passing the checks for our header ASSUMING that the data is also made available
1281
- // This means that we can avoid the simulation issues in later checks.
1282
- // By simulation issue, I mean the fact that the block.timestamp is equal to the last block, not the next, which
1283
- // make time consistency checks break.
1284
- // TODO(palla): Check whether we're validating twice, once here and once within addProposeTx, since we call simulateProposeTx in both places.
1285
- ts = await this.validateCheckpointForSubmission(checkpoint, attestationsAndSigners, attestationsAndSignersSignature, opts);
1286
- } catch (err) {
1287
- this.log.error(`Checkpoint validation failed. ${err instanceof Error ? err.message : 'No error message'}`, err, {
1288
- ...checkpoint.getStats(),
1289
- slotNumber: checkpoint.header.slotNumber,
1290
- forcePendingCheckpointNumber: opts.forcePendingCheckpointNumber
1291
- });
1292
- throw err;
1293
- }
1294
1196
  this.log.verbose(`Enqueuing checkpoint propose transaction`, {
1295
1197
  ...checkpoint.toCheckpointInfo(),
1296
- ...opts
1198
+ txTimeoutAt: opts.txTimeoutAt
1199
+ });
1200
+ await this.addProposeTx(checkpoint, proposeTxArgs, {
1201
+ txTimeoutAt: opts.txTimeoutAt
1297
1202
  });
1298
- await this.addProposeTx(checkpoint, proposeTxArgs, opts, ts);
1299
1203
  }
1300
1204
  enqueueInvalidateCheckpoint(request, opts = {}) {
1301
1205
  if (!request) {
1302
1206
  return;
1303
1207
  }
1304
- // We issued the simulation against the rollup contract, so we need to account for the overhead of the multicall3
1305
- const gasLimit = this.l1TxUtils.bumpGasLimit(BigInt(Math.ceil(Number(request.gasUsed) * 64 / 63)));
1306
1208
  const { gasUsed, checkpointNumber } = request;
1307
1209
  const logData = {
1308
1210
  gasUsed,
1309
1211
  checkpointNumber,
1310
- gasLimit,
1311
1212
  opts
1312
1213
  };
1313
1214
  this.log.verbose(`Enqueuing invalidate checkpoint request`, logData);
1314
1215
  this.addRequest({
1315
1216
  action: `invalidate-by-${request.reason}`,
1316
1217
  request: request.request,
1317
- gasConfig: {
1318
- gasLimit,
1218
+ gasConfig: opts.txTimeoutAt ? {
1319
1219
  txTimeoutAt: opts.txTimeoutAt
1320
- },
1220
+ } : undefined,
1321
1221
  lastValidL2Slot: SlotNumber(this.getCurrentL2Slot() + 2),
1322
1222
  checkSuccess: (_req, result)=>{
1323
- const success = result && result.receipt && result.receipt.status === 'success' && tryExtractEvent(result.receipt.logs, this.rollupContract.address, RollupAbi, 'CheckpointInvalidated');
1223
+ const success = result && extractEventSuccess(result.receipt, {
1224
+ address: this.rollupContract.address,
1225
+ abi: RollupAbi,
1226
+ eventName: 'CheckpointInvalidated'
1227
+ });
1324
1228
  if (!success) {
1325
1229
  this.log.warn(`Invalidate checkpoint ${request.checkpointNumber} failed`, {
1326
1230
  ...result,
@@ -1336,89 +1240,36 @@ export class SequencerPublisher {
1336
1240
  }
1337
1241
  });
1338
1242
  }
1339
- async simulateAndEnqueueRequest(action, request, checkSuccess, slotNumber, timestamp) {
1340
- const logData = {
1341
- slotNumber,
1342
- timestamp,
1343
- gasLimit: undefined
1344
- };
1243
+ /**
1244
+ * Dedup-checked enqueue helper for actions that are simulated at bundle-send time rather
1245
+ * than at enqueue time. Validates the (action, slot) dedup key, sets `lastActions`, and
1246
+ * enqueues without a gasLimit so the bundle simulate sets the only gasLimit that matters.
1247
+ */ enqueueRequest(action, request, eventOpts, slotNumber) {
1345
1248
  if (this.lastActions[action] && this.lastActions[action] === slotNumber) {
1346
1249
  this.log.debug(`Skipping duplicate action ${action} for slot ${slotNumber}`);
1347
1250
  return false;
1348
1251
  }
1349
1252
  const cachedLastActionSlot = this.lastActions[action];
1350
1253
  this.lastActions[action] = slotNumber;
1351
- this.log.debug(`Simulating ${action} for slot ${slotNumber}`, logData);
1352
- const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
1353
- let gasUsed;
1354
- const simulateAbi = mergeAbis([
1355
- request.abi ?? [],
1356
- ErrorsAbi
1357
- ]);
1358
- try {
1359
- ({ gasUsed } = await this.l1TxUtils.simulate(request, {
1360
- time: timestamp
1361
- }, [], simulateAbi)); // TODO(palla/slash): Check the timestamp logic
1362
- this.log.verbose(`Simulation for ${action} succeeded`, {
1363
- ...logData,
1364
- request,
1365
- gasUsed
1366
- });
1367
- } catch (err) {
1368
- const viemError = formatViemError(err, simulateAbi);
1369
- this.log.error(`Simulation for ${action} at ${slotNumber} failed`, viemError, logData);
1370
- this.backupFailedTx({
1371
- id: keccak256(request.data),
1372
- failureType: 'simulation',
1373
- request: {
1374
- to: request.to,
1375
- data: request.data,
1376
- value: request.value?.toString()
1377
- },
1378
- l1BlockNumber: l1BlockNumber.toString(),
1379
- error: {
1380
- message: viemError.message,
1381
- name: viemError.name
1382
- },
1383
- context: {
1384
- actions: [
1385
- action
1386
- ],
1387
- slot: slotNumber,
1388
- sender: this.getSenderAddress().toString()
1389
- }
1390
- });
1391
- return false;
1392
- }
1393
- // We issued the simulation against the rollup contract, so we need to account for the overhead of the multicall3
1394
- const gasLimit = this.l1TxUtils.bumpGasLimit(BigInt(Math.ceil(Number(gasUsed) * 64 / 63)));
1395
- logData.gasLimit = gasLimit;
1396
- // Store the ABI used for simulation on the request so Multicall3.forward can decode errors
1397
- // when the tx is sent and a revert is diagnosed via simulation.
1398
- const requestWithAbi = {
1399
- ...request,
1400
- abi: simulateAbi
1401
- };
1402
- this.log.debug(`Enqueuing ${action}`, logData);
1254
+ this.log.debug(`Enqueuing ${action}`, {
1255
+ slotNumber
1256
+ });
1403
1257
  this.addRequest({
1404
1258
  action,
1405
- request: requestWithAbi,
1406
- gasConfig: {
1407
- gasLimit
1408
- },
1259
+ request,
1409
1260
  lastValidL2Slot: slotNumber,
1410
- checkSuccess: (_req, result)=>{
1411
- const success = result && result.receipt && result.receipt.status === 'success' && checkSuccess(result.receipt);
1261
+ checkSuccess: (_request, result)=>{
1262
+ const success = result && extractEventSuccess(result.receipt, eventOpts);
1412
1263
  if (!success) {
1413
1264
  this.log.warn(`Action ${action} at ${slotNumber} failed`, {
1414
1265
  ...result,
1415
- ...logData
1266
+ slotNumber
1416
1267
  });
1417
1268
  this.lastActions[action] = cachedLastActionSlot;
1418
1269
  } else {
1419
1270
  this.log.info(`Action ${action} at ${slotNumber} succeeded`, {
1420
1271
  ...result,
1421
- ...logData
1272
+ slotNumber
1422
1273
  });
1423
1274
  }
1424
1275
  return !!success;
@@ -1433,13 +1284,14 @@ export class SequencerPublisher {
1433
1284
  * A call to `restart` is required before you can continue publishing.
1434
1285
  */ interrupt() {
1435
1286
  this.interrupted = true;
1287
+ this.interruptibleSleep.interrupt();
1436
1288
  this.l1TxUtils.interrupt();
1437
1289
  }
1438
1290
  /** Restarts the publisher after calling `interrupt`. */ restart() {
1439
1291
  this.interrupted = false;
1440
1292
  this.l1TxUtils.restart();
1441
1293
  }
1442
- async prepareProposeTx(encodedData, timestamp, options) {
1294
+ async prepareProposeTx(encodedData) {
1443
1295
  const kzg = Blob.getViemKzgInstance();
1444
1296
  const blobInput = getPrefixedEthBlobCommitments(encodedData.blobs);
1445
1297
  this.log.debug('Validating blob input', {
@@ -1453,7 +1305,11 @@ export class SequencerPublisher {
1453
1305
  blobEvaluationGas = BigInt(encodedData.blobs.length) * 21_000n;
1454
1306
  this.log.debug(`Using fixed blob evaluation gas estimate in fisherman mode: ${blobEvaluationGas}`);
1455
1307
  } else {
1456
- // Normal mode - use estimateGas with blob inputs
1308
+ // We call validateBlobs via estimateGas with real blob+kzg sidecars as a consistency check
1309
+ // that our locally-built blob commitments match the blob data. The bundle simulate at send
1310
+ // time uses eth_simulateV1, which cannot carry blob inputs, so the rollup's on-chain blob
1311
+ // check is forced off there — making this the only pre-flight detector of a commitment/data
1312
+ // mismatch. The returned gas estimate is stashed on the request for the bundle path to read.
1457
1313
  blobEvaluationGas = await this.l1TxUtils.estimateGas(this.getSenderAddress().toString(), {
1458
1314
  to: this.rollupContract.address,
1459
1315
  data: encodeFunctionData({
@@ -1516,109 +1372,23 @@ export class SequencerPublisher {
1516
1372
  encodedData.attestationsAndSignersSignature.toViemSignature(),
1517
1373
  blobInput
1518
1374
  ];
1519
- const { rollupData, simulationResult } = await this.simulateProposeTx(args, timestamp, options);
1520
- return {
1521
- args,
1522
- blobEvaluationGas,
1523
- rollupData,
1524
- simulationResult
1525
- };
1526
- }
1527
- /**
1528
- * Simulates the propose tx with eth_simulateV1
1529
- * @param args - The propose tx args
1530
- * @param timestamp - The timestamp to simulate proposal at
1531
- * @returns The simulation result
1532
- */ async simulateProposeTx(args, timestamp, options) {
1533
1375
  const rollupData = encodeFunctionData({
1534
1376
  abi: RollupAbi,
1535
1377
  functionName: 'propose',
1536
1378
  args
1537
1379
  });
1538
- // override the pending checkpoint number if requested
1539
- const forcePendingCheckpointNumberStateDiff = (options.forcePendingCheckpointNumber !== undefined ? await this.rollupContract.makePendingCheckpointNumberOverride(options.forcePendingCheckpointNumber) : []).flatMap((override)=>override.stateDiff ?? []);
1540
- const stateOverrides = [
1541
- {
1542
- address: this.rollupContract.address,
1543
- // @note we override checkBlob to false since blobs are not part simulate()
1544
- stateDiff: [
1545
- {
1546
- slot: toPaddedHex(RollupContract.checkBlobStorageSlot, true),
1547
- value: toPaddedHex(0n, true)
1548
- },
1549
- ...forcePendingCheckpointNumberStateDiff
1550
- ]
1551
- }
1552
- ];
1553
- // In fisherman mode, simulate as the proposer but with sufficient balance
1554
- if (this.proposerAddressForSimulation) {
1555
- stateOverrides.push({
1556
- address: this.proposerAddressForSimulation.toString(),
1557
- balance: 10n * WEI_CONST * WEI_CONST
1558
- });
1559
- }
1560
- const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
1561
- const simulationResult = await this.l1TxUtils.simulate({
1562
- to: this.rollupContract.address,
1563
- data: rollupData,
1564
- gas: MAX_L1_TX_LIMIT,
1565
- ...this.proposerAddressForSimulation && {
1566
- from: this.proposerAddressForSimulation.toString()
1567
- }
1568
- }, {
1569
- // @note we add 1n to the timestamp because geth implementation doesn't like simulation timestamp to be equal to the current block timestamp
1570
- time: timestamp + 1n,
1571
- // @note reth should have a 30m gas limit per block but throws errors that this tx is beyond limit so we increase here
1572
- gasLimit: MAX_L1_TX_LIMIT * 2n
1573
- }, stateOverrides, RollupAbi, {
1574
- // @note fallback gas estimate to use if the node doesn't support simulation API
1575
- fallbackGasEstimate: MAX_L1_TX_LIMIT
1576
- }).catch((err)=>{
1577
- // In fisherman mode, we expect ValidatorSelection__MissingProposerSignature since fisherman doesn't have proposer signature
1578
- const viemError = formatViemError(err);
1579
- if (this.config.fishermanMode && viemError.message?.includes('ValidatorSelection__MissingProposerSignature')) {
1580
- this.log.debug(`Ignoring expected ValidatorSelection__MissingProposerSignature error in fisherman mode`);
1581
- // Return a minimal simulation result with the fallback gas estimate
1582
- return {
1583
- gasUsed: MAX_L1_TX_LIMIT,
1584
- logs: []
1585
- };
1586
- }
1587
- this.log.error(`Failed to simulate propose tx`, viemError);
1588
- this.backupFailedTx({
1589
- id: keccak256(rollupData),
1590
- failureType: 'simulation',
1591
- request: {
1592
- to: this.rollupContract.address,
1593
- data: rollupData
1594
- },
1595
- l1BlockNumber: l1BlockNumber.toString(),
1596
- error: {
1597
- message: viemError.message,
1598
- name: viemError.name
1599
- },
1600
- context: {
1601
- actions: [
1602
- 'propose'
1603
- ],
1604
- slot: Number(args[0].header.slotNumber),
1605
- sender: this.getSenderAddress().toString()
1606
- }
1607
- });
1608
- throw err;
1609
- });
1610
1380
  return {
1611
- rollupData,
1612
- simulationResult
1381
+ args,
1382
+ blobEvaluationGas,
1383
+ rollupData
1613
1384
  };
1614
1385
  }
1615
- async addProposeTx(checkpoint, encodedData, opts = {}, timestamp) {
1386
+ async addProposeTx(checkpoint, encodedData, opts = {}) {
1616
1387
  const slot = checkpoint.header.slotNumber;
1617
1388
  const timer = new Timer();
1618
1389
  const kzg = Blob.getViemKzgInstance();
1619
- const { rollupData, simulationResult, blobEvaluationGas } = await this.prepareProposeTx(encodedData, timestamp, opts);
1390
+ const { rollupData, blobEvaluationGas } = await this.prepareProposeTx(encodedData);
1620
1391
  const startBlock = await this.l1TxUtils.getBlockNumber();
1621
- const gasLimit = this.l1TxUtils.bumpGasLimit(BigInt(Math.ceil(Number(simulationResult.gasUsed) * 64 / 63)) + blobEvaluationGas + SequencerPublisher.MULTICALL_OVERHEAD_GAS_GUESS);
1622
1392
  // Send the blobs to the blob client preemptively. This helps in tests where the sequencer mistakingly thinks that the propose
1623
1393
  // tx fails but it does get mined. We make sure that the blobs are sent to the blob client regardless of the tx outcome.
1624
1394
  void Promise.resolve().then(()=>this.blobClient.sendBlobsToFilestore(encodedData.blobs).catch((_err)=>{
@@ -1632,9 +1402,10 @@ export class SequencerPublisher {
1632
1402
  },
1633
1403
  lastValidL2Slot: checkpoint.header.slotNumber,
1634
1404
  gasConfig: {
1635
- ...opts,
1636
- gasLimit
1405
+ txTimeoutAt: opts.txTimeoutAt,
1406
+ gasLimit: undefined
1637
1407
  },
1408
+ blobEvaluationGas,
1638
1409
  blobConfig: {
1639
1410
  blobs: encodedData.blobs.map((b)=>b.data),
1640
1411
  kzg
@@ -1644,7 +1415,11 @@ export class SequencerPublisher {
1644
1415
  return false;
1645
1416
  }
1646
1417
  const { receipt, stats, errorMsg } = result;
1647
- const success = receipt && receipt.status === 'success' && tryExtractEvent(receipt.logs, this.rollupContract.address, RollupAbi, 'CheckpointProposed');
1418
+ const success = extractEventSuccess(receipt, {
1419
+ address: this.rollupContract.address,
1420
+ abi: RollupAbi,
1421
+ eventName: 'CheckpointProposed'
1422
+ });
1648
1423
  if (success) {
1649
1424
  const endBlock = receipt.blockNumber;
1650
1425
  const inclusionBlocks = Number(endBlock - startBlock);
@@ -1681,4 +1456,8 @@ export class SequencerPublisher {
1681
1456
  }
1682
1457
  });
1683
1458
  }
1459
+ /** Returns the timestamp of the next L1 slot boundary after now. */ getNextL1SlotTimestamp() {
1460
+ const l1Constants = this.epochCache.getL1Constants();
1461
+ return getNextL1SlotTimestamp(this.dateProvider.nowInSeconds(), l1Constants);
1462
+ }
1684
1463
  }