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