@aztec/sequencer-client 0.0.1-commit.b6e433891 → 0.0.1-commit.b8a057fa
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 +23 -5
- package/dest/client/sequencer-client.d.ts.map +1 -1
- package/dest/client/sequencer-client.js +33 -34
- package/dest/config.d.ts +11 -4
- package/dest/config.d.ts.map +1 -1
- package/dest/config.js +55 -26
- 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 +13 -26
- package/dest/global_variable_builder/global_builder.d.ts.map +1 -1
- package/dest/global_variable_builder/global_builder.js +9 -67
- package/dest/global_variable_builder/index.d.ts +4 -2
- 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 +19 -3
- package/dest/publisher/config.d.ts.map +1 -1
- package/dest/publisher/config.js +30 -5
- 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/failed_tx_store.d.ts +50 -11
- package/dest/publisher/l1_tx_failed_store/failed_tx_store.d.ts.map +1 -1
- package/dest/publisher/l1_tx_failed_store/failed_tx_store.js +68 -1
- package/dest/publisher/l1_tx_failed_store/file_store_failed_tx_store.d.ts +2 -2
- package/dest/publisher/l1_tx_failed_store/file_store_failed_tx_store.d.ts.map +1 -1
- package/dest/publisher/l1_tx_failed_store/file_store_failed_tx_store.js +4 -2
- package/dest/publisher/l1_tx_failed_store/index.d.ts +2 -2
- package/dest/publisher/l1_tx_failed_store/index.d.ts.map +1 -1
- package/dest/publisher/l1_tx_failed_store/index.js +1 -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 +2 -3
- package/dest/publisher/sequencer-publisher.d.ts +101 -67
- package/dest/publisher/sequencer-publisher.d.ts.map +1 -1
- package/dest/publisher/sequencer-publisher.js +615 -535
- 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 +71 -21
- package/dest/sequencer/checkpoint_proposal_job.d.ts.map +1 -1
- package/dest/sequencer/checkpoint_proposal_job.js +724 -212
- 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/checkpoint_voter.d.ts +1 -2
- package/dest/sequencer/checkpoint_voter.d.ts.map +1 -1
- package/dest/sequencer/checkpoint_voter.js +2 -5
- package/dest/sequencer/events.d.ts +61 -5
- package/dest/sequencer/events.d.ts.map +1 -1
- package/dest/sequencer/metrics.d.ts +9 -10
- package/dest/sequencer/metrics.d.ts.map +1 -1
- package/dest/sequencer/metrics.js +34 -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 -33
- package/dest/sequencer/sequencer.d.ts.map +1 -1
- package/dest/sequencer/sequencer.js +546 -161
- 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/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 -28
- package/src/client/sequencer-client.ts +47 -43
- package/src/config.ts +64 -26
- 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 +19 -89
- package/src/global_variable_builder/index.ts +3 -1
- package/src/publisher/config.ts +62 -7
- package/src/publisher/l1_to_l2_messaging.ts +85 -0
- package/src/publisher/l1_tx_failed_store/failed_tx_store.ts +117 -8
- package/src/publisher/l1_tx_failed_store/file_store_failed_tx_store.ts +4 -3
- package/src/publisher/l1_tx_failed_store/index.ts +1 -1
- package/src/publisher/sequencer-bundle-simulator.ts +254 -0
- package/src/publisher/sequencer-publisher-factory.ts +3 -6
- package/src/publisher/sequencer-publisher.ts +711 -609
- 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 +844 -241
- package/src/sequencer/checkpoint_proposal_job_metrics.ts +128 -0
- package/src/sequencer/checkpoint_voter.ts +1 -12
- package/src/sequencer/events.ts +66 -5
- package/src/sequencer/metrics.ts +43 -24
- package/src/sequencer/requests_tracker.ts +43 -0
- package/src/sequencer/sequencer.ts +628 -178
- package/src/sequencer/types.ts +1 -1
- package/src/test/index.ts +2 -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,65 +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';
|
|
34
38
|
import { TimeoutError } from '@aztec/foundation/error';
|
|
35
39
|
import { EthAddress } from '@aztec/foundation/eth-address';
|
|
36
|
-
import { Signature
|
|
40
|
+
import { Signature } from '@aztec/foundation/eth-signature';
|
|
37
41
|
import { type Logger, createLogger } from '@aztec/foundation/log';
|
|
38
|
-
import {
|
|
42
|
+
import { InterruptibleSleep } from '@aztec/foundation/sleep';
|
|
39
43
|
import { bufferToHex } from '@aztec/foundation/string';
|
|
40
|
-
import { DateProvider, Timer } from '@aztec/foundation/timer';
|
|
41
|
-
import { EmpireBaseAbi, ErrorsAbi, RollupAbi } from '@aztec/l1-artifacts';
|
|
44
|
+
import { type DateProvider, Timer } from '@aztec/foundation/timer';
|
|
45
|
+
import { EmpireBaseAbi, ErrorsAbi, RollupAbi, SlashingProposerAbi } from '@aztec/l1-artifacts';
|
|
42
46
|
import { type ProposerSlashAction, encodeSlashConsensusVotes } from '@aztec/slasher';
|
|
43
47
|
import { CommitteeAttestationsAndSigners, type ValidateCheckpointResult } from '@aztec/stdlib/block';
|
|
44
48
|
import type { Checkpoint } from '@aztec/stdlib/checkpoint';
|
|
45
|
-
import {
|
|
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
58
|
import {
|
|
59
|
+
type Abi,
|
|
51
60
|
type Hex,
|
|
52
|
-
type StateOverride,
|
|
53
61
|
type TransactionReceipt,
|
|
54
62
|
type TypedDataDefinition,
|
|
55
63
|
encodeFunctionData,
|
|
56
64
|
keccak256,
|
|
57
|
-
multicall3Abi,
|
|
58
65
|
toHex,
|
|
59
66
|
} from 'viem';
|
|
60
67
|
|
|
61
68
|
import type { SequencerPublisherConfig } from './config.js';
|
|
62
69
|
import { type FailedL1Tx, type L1TxFailedStore, createL1TxFailedStore } from './l1_tx_failed_store/index.js';
|
|
70
|
+
import { type DroppedRequest, SequencerBundleSimulator } from './sequencer-bundle-simulator.js';
|
|
63
71
|
import { SequencerPublisherMetrics } from './sequencer-publisher-metrics.js';
|
|
64
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
|
+
|
|
65
102
|
/** Arguments to the process method of the rollup contract */
|
|
66
103
|
type L1ProcessArgs = {
|
|
67
104
|
/** The L2 block header. */
|
|
@@ -81,18 +118,16 @@ type L1ProcessArgs = {
|
|
|
81
118
|
export const Actions = [
|
|
82
119
|
'invalidate-by-invalid-attestation',
|
|
83
120
|
'invalidate-by-insufficient-attestations',
|
|
121
|
+
'prune',
|
|
84
122
|
'propose',
|
|
85
123
|
'governance-signal',
|
|
86
|
-
'empire-slashing-signal',
|
|
87
|
-
'create-empire-payload',
|
|
88
|
-
'execute-empire-payload',
|
|
89
124
|
'vote-offenses',
|
|
90
125
|
'execute-slash',
|
|
91
126
|
] as const;
|
|
92
127
|
|
|
93
128
|
export type Action = (typeof Actions)[number];
|
|
94
129
|
|
|
95
|
-
type GovernanceSignalAction = Extract<Action, 'governance-signal'
|
|
130
|
+
type GovernanceSignalAction = Extract<Action, 'governance-signal'>;
|
|
96
131
|
|
|
97
132
|
// Sorting for actions such that invalidations go before proposals, and proposals go before votes
|
|
98
133
|
export const compareActions = (a: Action, b: Action) => Actions.indexOf(a) - Actions.indexOf(b);
|
|
@@ -103,14 +138,22 @@ export type InvalidateCheckpointRequest = {
|
|
|
103
138
|
gasUsed: bigint;
|
|
104
139
|
checkpointNumber: CheckpointNumber;
|
|
105
140
|
forcePendingCheckpointNumber: CheckpointNumber;
|
|
141
|
+
/** Archive at the rollback target checkpoint (checkpoint N-1). */
|
|
142
|
+
lastArchive: Fr;
|
|
143
|
+
};
|
|
144
|
+
|
|
145
|
+
type EnqueueProposeCheckpointOpts = {
|
|
146
|
+
txTimeoutAt?: Date;
|
|
106
147
|
};
|
|
107
148
|
|
|
108
|
-
interface RequestWithExpiry {
|
|
149
|
+
export interface RequestWithExpiry {
|
|
109
150
|
action: Action;
|
|
110
151
|
request: L1TxRequest;
|
|
111
152
|
lastValidL2Slot: SlotNumber;
|
|
112
153
|
gasConfig?: Pick<L1TxConfig, 'txTimeoutAt' | 'gasLimit'>;
|
|
113
154
|
blobConfig?: L1BlobInputs;
|
|
155
|
+
/** Gas consumed by validateBlobs; stashed for the bundle simulate at send time. */
|
|
156
|
+
blobEvaluationGas?: bigint;
|
|
114
157
|
checkSuccess: (
|
|
115
158
|
request: L1TxRequest,
|
|
116
159
|
result?: { receipt: TransactionReceipt; stats?: TransactionStats; errorMsg?: string },
|
|
@@ -120,25 +163,29 @@ interface RequestWithExpiry {
|
|
|
120
163
|
export class SequencerPublisher {
|
|
121
164
|
private interrupted = false;
|
|
122
165
|
private metrics: SequencerPublisherMetrics;
|
|
166
|
+
private bundleSimulator: SequencerBundleSimulator;
|
|
123
167
|
public epochCache: EpochCache;
|
|
124
168
|
private failedTxStore?: Promise<L1TxFailedStore | undefined>;
|
|
125
169
|
|
|
126
|
-
|
|
127
|
-
|
|
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]);
|
|
128
176
|
|
|
129
177
|
protected lastActions: Partial<Record<Action, SlotNumber>> = {};
|
|
130
178
|
|
|
131
|
-
private isPayloadEmptyCache: Map<string, boolean> = new Map<string, boolean>();
|
|
132
|
-
private payloadProposedCache: Set<string> = new Set<string>();
|
|
133
|
-
|
|
134
179
|
protected log: Logger;
|
|
135
180
|
protected ethereumSlotDuration: bigint;
|
|
136
181
|
protected aztecSlotDuration: bigint;
|
|
182
|
+
private readonly previousL1BlockWaitTimeoutMs: number;
|
|
183
|
+
private readonly previousL1BlockWaitPollIntervalMs: number;
|
|
137
184
|
|
|
138
|
-
|
|
185
|
+
/** Date provider for wall-clock time. */
|
|
186
|
+
private readonly dateProvider: DateProvider;
|
|
139
187
|
|
|
140
|
-
|
|
141
|
-
private proposerAddressForSimulation?: EthAddress;
|
|
188
|
+
private blobClient: BlobClientInterface;
|
|
142
189
|
|
|
143
190
|
/** Optional callback to obtain a replacement publisher when the current one fails to send. */
|
|
144
191
|
private getNextPublisher?: (excludeAddresses: EthAddress[]) => Promise<L1TxUtils | undefined>;
|
|
@@ -149,33 +196,34 @@ export class SequencerPublisher {
|
|
|
149
196
|
/** Fee asset price oracle for computing price modifiers from Uniswap V4 */
|
|
150
197
|
private feeAssetPriceOracle: FeeAssetPriceOracle;
|
|
151
198
|
|
|
152
|
-
|
|
153
|
-
|
|
154
|
-
|
|
155
|
-
// Gas report for VotingWithSigTest shows a max gas of 100k, but we've seen it cost 700k+ in testnet
|
|
156
|
-
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();
|
|
157
201
|
|
|
158
202
|
public l1TxUtils: L1TxUtils;
|
|
159
203
|
public rollupContract: RollupContract;
|
|
160
204
|
public govProposerContract: GovernanceProposerContract;
|
|
161
|
-
public slashingProposerContract:
|
|
162
|
-
public slashFactoryContract: SlashFactoryContract;
|
|
205
|
+
public slashingProposerContract: SlashingProposerContract | undefined;
|
|
163
206
|
|
|
164
207
|
public readonly tracer: Tracer;
|
|
165
208
|
|
|
166
209
|
protected requests: RequestWithExpiry[] = [];
|
|
167
210
|
|
|
168
211
|
constructor(
|
|
169
|
-
private config: Pick<
|
|
212
|
+
private config: Pick<
|
|
213
|
+
SequencerPublisherConfig,
|
|
214
|
+
| 'fishermanMode'
|
|
215
|
+
| 'l1TxFailedStore'
|
|
216
|
+
| 'sequencerPublisherPreviousL1BlockWaitTimeoutMs'
|
|
217
|
+
| 'sequencerPublisherPreviousL1BlockWaitPollIntervalMs'
|
|
218
|
+
> &
|
|
170
219
|
Pick<L1ContractsConfig, 'ethereumSlotDuration' | 'aztecSlotDuration'> & { l1ChainId: number },
|
|
171
220
|
deps: {
|
|
172
221
|
telemetry?: TelemetryClient;
|
|
173
222
|
blobClient: BlobClientInterface;
|
|
174
223
|
l1TxUtils: L1TxUtils;
|
|
175
224
|
rollupContract: RollupContract;
|
|
176
|
-
slashingProposerContract:
|
|
225
|
+
slashingProposerContract: SlashingProposerContract | undefined;
|
|
177
226
|
governanceProposerContract: GovernanceProposerContract;
|
|
178
|
-
slashFactoryContract: SlashFactoryContract;
|
|
179
227
|
epochCache: EpochCache;
|
|
180
228
|
dateProvider: DateProvider;
|
|
181
229
|
metrics: SequencerPublisherMetrics;
|
|
@@ -187,10 +235,14 @@ export class SequencerPublisher {
|
|
|
187
235
|
this.log = deps.log ?? createLogger('sequencer:publisher');
|
|
188
236
|
this.ethereumSlotDuration = BigInt(config.ethereumSlotDuration);
|
|
189
237
|
this.aztecSlotDuration = BigInt(config.aztecSlotDuration);
|
|
238
|
+
this.previousL1BlockWaitTimeoutMs = config.sequencerPublisherPreviousL1BlockWaitTimeoutMs;
|
|
239
|
+
this.previousL1BlockWaitPollIntervalMs = config.sequencerPublisherPreviousL1BlockWaitPollIntervalMs;
|
|
240
|
+
this.dateProvider = deps.dateProvider;
|
|
190
241
|
this.epochCache = deps.epochCache;
|
|
191
242
|
this.lastActions = deps.lastActions;
|
|
192
243
|
|
|
193
244
|
this.blobClient = deps.blobClient;
|
|
245
|
+
this.dateProvider = deps.dateProvider;
|
|
194
246
|
|
|
195
247
|
const telemetry = deps.telemetry ?? getTelemetryClient();
|
|
196
248
|
this.metrics = deps.metrics ?? new SequencerPublisherMetrics(telemetry, 'SequencerPublisher');
|
|
@@ -208,14 +260,12 @@ export class SequencerPublisher {
|
|
|
208
260
|
const newSlashingProposer = await this.rollupContract.getSlashingProposer();
|
|
209
261
|
this.slashingProposerContract = newSlashingProposer;
|
|
210
262
|
});
|
|
211
|
-
this.slashFactoryContract = deps.slashFactoryContract;
|
|
212
|
-
|
|
213
263
|
// Initialize L1 fee analyzer for fisherman mode
|
|
214
264
|
if (config.fishermanMode) {
|
|
215
265
|
this.l1FeeAnalyzer = new L1FeeAnalyzer(
|
|
216
266
|
this.l1TxUtils.client,
|
|
217
267
|
deps.dateProvider,
|
|
218
|
-
|
|
268
|
+
this.log.createChild('fee-analyzer'),
|
|
219
269
|
);
|
|
220
270
|
}
|
|
221
271
|
|
|
@@ -223,33 +273,112 @@ export class SequencerPublisher {
|
|
|
223
273
|
this.feeAssetPriceOracle = new FeeAssetPriceOracle(
|
|
224
274
|
this.l1TxUtils.client,
|
|
225
275
|
this.rollupContract,
|
|
226
|
-
|
|
276
|
+
this.log.createChild('price-oracle'),
|
|
227
277
|
);
|
|
228
278
|
|
|
229
279
|
// Initialize failed L1 tx store (optional, for test networks)
|
|
230
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
|
+
});
|
|
231
288
|
}
|
|
232
289
|
|
|
233
290
|
/**
|
|
234
|
-
*
|
|
235
|
-
*
|
|
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.
|
|
236
295
|
*/
|
|
237
|
-
private backupFailedTx(
|
|
238
|
-
|
|
239
|
-
|
|
240
|
-
|
|
241
|
-
|
|
296
|
+
private backupFailedTx(
|
|
297
|
+
failedTx: Omit<FailedL1Tx, 'timestamp'>,
|
|
298
|
+
opts?: { captureFeeSummary?: boolean; targetSlot?: SlotNumber; sharedFeeSummary?: FailedL1Tx['gasInfo'] },
|
|
299
|
+
): void {
|
|
242
300
|
const tx: FailedL1Tx = {
|
|
243
301
|
...failedTx,
|
|
244
302
|
timestamp: Date.now(),
|
|
245
303
|
};
|
|
246
304
|
|
|
247
305
|
// Fire and forget - don't block on backup
|
|
248
|
-
void
|
|
249
|
-
|
|
250
|
-
|
|
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) {
|
|
251
332
|
this.log.warn(`Failed to backup failed L1 tx to store`, err);
|
|
252
|
-
}
|
|
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()}`));
|
|
253
382
|
}
|
|
254
383
|
|
|
255
384
|
public getRollupContract(): RollupContract {
|
|
@@ -258,10 +387,14 @@ export class SequencerPublisher {
|
|
|
258
387
|
|
|
259
388
|
/**
|
|
260
389
|
* Gets the fee asset price modifier from the oracle.
|
|
261
|
-
*
|
|
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.
|
|
262
395
|
*/
|
|
263
|
-
public getFeeAssetPriceModifier(): Promise<bigint> {
|
|
264
|
-
return this.feeAssetPriceOracle.computePriceModifier();
|
|
396
|
+
public getFeeAssetPriceModifier(predictedParentEthPerFeeAssetE12?: bigint): Promise<bigint> {
|
|
397
|
+
return this.feeAssetPriceOracle.computePriceModifier(predictedParentEthPerFeeAssetE12);
|
|
265
398
|
}
|
|
266
399
|
|
|
267
400
|
public getSenderAddress() {
|
|
@@ -275,14 +408,6 @@ export class SequencerPublisher {
|
|
|
275
408
|
return this.l1FeeAnalyzer;
|
|
276
409
|
}
|
|
277
410
|
|
|
278
|
-
/**
|
|
279
|
-
* Sets the proposer address to use for simulations in fisherman mode.
|
|
280
|
-
* @param proposerAddress - The actual proposer's address to use for balance lookups in simulations
|
|
281
|
-
*/
|
|
282
|
-
public setProposerAddressForSimulation(proposerAddress: EthAddress | undefined) {
|
|
283
|
-
this.proposerAddressForSimulation = proposerAddress;
|
|
284
|
-
}
|
|
285
|
-
|
|
286
411
|
public addRequest(request: RequestWithExpiry) {
|
|
287
412
|
this.requests.push(request);
|
|
288
413
|
}
|
|
@@ -360,22 +485,26 @@ export class SequencerPublisher {
|
|
|
360
485
|
|
|
361
486
|
/**
|
|
362
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.
|
|
363
492
|
* @returns one of:
|
|
364
493
|
* - A receipt and stats if the tx succeeded
|
|
365
494
|
* - a receipt and errorMsg if it failed on L1
|
|
366
495
|
* - undefined if no valid requests are found OR the tx failed to send.
|
|
367
496
|
*/
|
|
368
497
|
@trackSpan('SequencerPublisher.sendRequests')
|
|
369
|
-
public async sendRequests() {
|
|
498
|
+
public async sendRequests(targetSlot?: SlotNumber): Promise<SendRequestsResult | undefined> {
|
|
370
499
|
const requestsToProcess = [...this.requests];
|
|
371
500
|
this.requests = [];
|
|
501
|
+
|
|
372
502
|
if (this.interrupted || requestsToProcess.length === 0) {
|
|
373
503
|
return undefined;
|
|
374
504
|
}
|
|
375
|
-
const currentL2Slot = this.getCurrentL2Slot();
|
|
505
|
+
const currentL2Slot = targetSlot ?? this.getCurrentL2Slot();
|
|
376
506
|
this.log.debug(`Sending requests on L2 slot ${currentL2Slot}`);
|
|
377
507
|
const validRequests = requestsToProcess.filter(request => request.lastValidL2Slot >= currentL2Slot);
|
|
378
|
-
const validActions = validRequests.map(x => x.action);
|
|
379
508
|
const expiredActions = requestsToProcess
|
|
380
509
|
.filter(request => request.lastValidL2Slot < currentL2Slot)
|
|
381
510
|
.map(x => x.action);
|
|
@@ -398,73 +527,101 @@ export class SequencerPublisher {
|
|
|
398
527
|
return undefined;
|
|
399
528
|
}
|
|
400
529
|
|
|
401
|
-
//
|
|
402
|
-
// find requests with gas and blob configs
|
|
403
|
-
// See https://github.com/AztecProtocol/aztec-packages/issues/11513
|
|
530
|
+
// Collect earliest txTimeoutAt across all requests.
|
|
404
531
|
const gasConfigs = validRequests.filter(request => request.gasConfig).map(request => request.gasConfig);
|
|
405
|
-
const blobConfigs = validRequests.filter(request => request.blobConfig).map(request => request.blobConfig);
|
|
406
|
-
|
|
407
|
-
if (blobConfigs.length > 1) {
|
|
408
|
-
throw new Error('Multiple blob configs found');
|
|
409
|
-
}
|
|
410
|
-
|
|
411
|
-
const blobConfig = blobConfigs[0];
|
|
412
|
-
|
|
413
|
-
// Merge gasConfigs. Yields the sum of gasLimits, and the earliest txTimeoutAt, or undefined if no gasConfig sets them.
|
|
414
|
-
const gasLimits = gasConfigs.map(g => g?.gasLimit).filter((g): g is bigint => g !== undefined);
|
|
415
|
-
let gasLimit = gasLimits.length > 0 ? sumBigint(gasLimits) : undefined; // sum
|
|
416
|
-
// Cap at L1 block gas limit so the node accepts the tx ("gas limit too high" otherwise).
|
|
417
|
-
const maxGas = MAX_L1_TX_LIMIT;
|
|
418
|
-
if (gasLimit !== undefined && gasLimit > maxGas) {
|
|
419
|
-
this.log.debug('Capping bundled tx gas limit to L1 max', {
|
|
420
|
-
requested: gasLimit,
|
|
421
|
-
capped: maxGas,
|
|
422
|
-
});
|
|
423
|
-
gasLimit = maxGas;
|
|
424
|
-
}
|
|
425
532
|
const txTimeoutAts = gasConfigs.map(g => g?.txTimeoutAt).filter((g): g is Date => g !== undefined);
|
|
426
|
-
const txTimeoutAt = txTimeoutAts.length > 0 ? new Date(Math.min(...txTimeoutAts.map(g => g.getTime()))) : undefined;
|
|
427
|
-
const txConfig: RequestWithExpiry['gasConfig'] = { gasLimit, txTimeoutAt };
|
|
533
|
+
const txTimeoutAt = txTimeoutAts.length > 0 ? new Date(Math.min(...txTimeoutAts.map(g => g.getTime()))) : undefined;
|
|
428
534
|
|
|
429
535
|
// Sort the requests so that proposals always go first
|
|
430
536
|
// This ensures the committee gets precomputed correctly
|
|
431
537
|
validRequests.sort((a, b) => compareActions(a.action, b.action));
|
|
432
538
|
|
|
433
539
|
try {
|
|
434
|
-
//
|
|
435
|
-
const
|
|
436
|
-
|
|
437
|
-
|
|
438
|
-
|
|
439
|
-
|
|
440
|
-
|
|
441
|
-
|
|
442
|
-
|
|
443
|
-
|
|
444
|
-
})),
|
|
445
|
-
],
|
|
446
|
-
});
|
|
447
|
-
const blobDataHex = blobConfig?.blobs?.map(b => toHex(b)) as Hex[] | undefined;
|
|
540
|
+
// Bundle-level eth_simulateV1: filters out entries that revert and derives the gasLimit.
|
|
541
|
+
const bundleResult = await this.bundleSimulator.simulate(validRequests, currentL2Slot);
|
|
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
|
+
}
|
|
448
550
|
|
|
449
|
-
const
|
|
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 };
|
|
450
566
|
|
|
451
567
|
this.log.debug('Forwarding transactions', {
|
|
452
|
-
|
|
568
|
+
requests: requests.map(request => request.action),
|
|
453
569
|
txConfig,
|
|
454
570
|
});
|
|
455
|
-
const result = await this.forwardWithPublisherRotation(
|
|
571
|
+
const result = await this.forwardWithPublisherRotation(requests, txConfig, blobConfig, currentL2Slot);
|
|
456
572
|
if (result === undefined) {
|
|
457
573
|
return undefined;
|
|
458
574
|
}
|
|
459
|
-
const { successfulActions = [], failedActions = [] } = this.callbackBundledTransactions(
|
|
460
|
-
|
|
575
|
+
const { successfulActions = [], failedActions = [] } = this.callbackBundledTransactions(requests, result);
|
|
576
|
+
const allFailedActions = [...failedActions, ...droppedRequests.map(d => d.request.action)];
|
|
577
|
+
return {
|
|
461
578
|
result,
|
|
462
|
-
|
|
463
|
-
|
|
464
|
-
|
|
579
|
+
expiredActions,
|
|
580
|
+
sentActions: requests.map(x => x.action),
|
|
581
|
+
successfulActions,
|
|
582
|
+
failedActions: allFailedActions,
|
|
583
|
+
};
|
|
465
584
|
} catch (err) {
|
|
466
585
|
const viemError = formatViemError(err);
|
|
467
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
|
+
}
|
|
468
625
|
return undefined;
|
|
469
626
|
} finally {
|
|
470
627
|
try {
|
|
@@ -478,6 +635,54 @@ export class SequencerPublisher {
|
|
|
478
635
|
}
|
|
479
636
|
}
|
|
480
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
|
+
|
|
481
686
|
/**
|
|
482
687
|
* Forwards transactions via Multicall3, rotating to the next available publisher if a send
|
|
483
688
|
* failure occurs (i.e. the tx never reached the chain).
|
|
@@ -487,20 +692,32 @@ export class SequencerPublisher {
|
|
|
487
692
|
validRequests: RequestWithExpiry[],
|
|
488
693
|
txConfig: RequestWithExpiry['gasConfig'],
|
|
489
694
|
blobConfig: L1BlobInputs | undefined,
|
|
695
|
+
targetSlot?: SlotNumber,
|
|
490
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
|
+
|
|
491
702
|
const triedAddresses: EthAddress[] = [];
|
|
492
703
|
let currentPublisher = this.l1TxUtils;
|
|
493
704
|
|
|
494
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
|
+
}
|
|
495
712
|
triedAddresses.push(currentPublisher.getSenderAddress());
|
|
713
|
+
|
|
496
714
|
try {
|
|
497
715
|
const result = await Multicall3.forward(
|
|
498
716
|
validRequests.map(r => r.request),
|
|
499
717
|
currentPublisher,
|
|
500
|
-
|
|
718
|
+
txConfigWithGasLimit,
|
|
501
719
|
blobConfig,
|
|
502
|
-
|
|
503
|
-
this.log,
|
|
720
|
+
{ gasLimitRequired: true },
|
|
504
721
|
);
|
|
505
722
|
this.l1TxUtils = currentPublisher;
|
|
506
723
|
return result;
|
|
@@ -508,9 +725,17 @@ export class SequencerPublisher {
|
|
|
508
725
|
if (err instanceof TimeoutError) {
|
|
509
726
|
throw err;
|
|
510
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
|
+
}
|
|
511
735
|
const viemError = formatViemError(err);
|
|
512
736
|
if (!this.getNextPublisher) {
|
|
513
737
|
this.log.error('Failed to publish bundled transactions', viemError);
|
|
738
|
+
this.backupSendFailure(validRequests, viemError, currentPublisher, targetSlot);
|
|
514
739
|
return undefined;
|
|
515
740
|
}
|
|
516
741
|
this.log.warn(
|
|
@@ -519,7 +744,12 @@ export class SequencerPublisher {
|
|
|
519
744
|
);
|
|
520
745
|
const nextPublisher = await this.getNextPublisher([...triedAddresses]);
|
|
521
746
|
if (!nextPublisher) {
|
|
522
|
-
this.log.error(
|
|
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);
|
|
523
753
|
return undefined;
|
|
524
754
|
}
|
|
525
755
|
currentPublisher = nextPublisher;
|
|
@@ -527,74 +757,179 @@ export class SequencerPublisher {
|
|
|
527
757
|
}
|
|
528
758
|
}
|
|
529
759
|
|
|
530
|
-
|
|
760
|
+
/** Backs up an on-chain revert failure to the failed tx store. */
|
|
761
|
+
private backupRevertFailure(
|
|
531
762
|
requests: RequestWithExpiry[],
|
|
532
|
-
|
|
533
|
-
|
|
534
|
-
|
|
535
|
-
|
|
536
|
-
|
|
537
|
-
|
|
538
|
-
|
|
539
|
-
|
|
540
|
-
|
|
541
|
-
|
|
542
|
-
|
|
543
|
-
|
|
544
|
-
|
|
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 },
|
|
545
780
|
context: {
|
|
546
781
|
actions: requests.map(r => r.action),
|
|
547
|
-
requests: requests
|
|
548
|
-
|
|
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,
|
|
549
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
|
+
);
|
|
550
838
|
});
|
|
551
|
-
|
|
552
|
-
|
|
553
|
-
|
|
554
|
-
|
|
555
|
-
|
|
556
|
-
|
|
557
|
-
|
|
558
|
-
|
|
559
|
-
|
|
560
|
-
|
|
561
|
-
|
|
562
|
-
|
|
563
|
-
|
|
564
|
-
|
|
565
|
-
|
|
566
|
-
|
|
567
|
-
|
|
568
|
-
|
|
569
|
-
|
|
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;
|
|
570
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);
|
|
571
904
|
}
|
|
572
|
-
|
|
573
|
-
|
|
574
|
-
|
|
575
|
-
|
|
576
|
-
|
|
577
|
-
|
|
578
|
-
|
|
579
|
-
|
|
580
|
-
|
|
581
|
-
|
|
582
|
-
|
|
583
|
-
|
|
584
|
-
|
|
585
|
-
|
|
586
|
-
|
|
587
|
-
|
|
588
|
-
|
|
589
|
-
|
|
590
|
-
|
|
591
|
-
|
|
592
|
-
|
|
593
|
-
|
|
594
|
-
|
|
905
|
+
}
|
|
906
|
+
}
|
|
907
|
+
|
|
908
|
+
private callbackBundledTransactions(
|
|
909
|
+
requests: RequestWithExpiry[],
|
|
910
|
+
result: { receipt: TransactionReceipt; multicallData: Hex },
|
|
911
|
+
) {
|
|
912
|
+
const actionsListStr = requests.map(r => r.action).join(', ');
|
|
913
|
+
this.log.verbose(`Published bundled transactions (${actionsListStr})`, {
|
|
914
|
+
result,
|
|
915
|
+
requests: requests.map(r => ({
|
|
916
|
+
...r,
|
|
917
|
+
// Avoid logging large blob data
|
|
918
|
+
blobConfig: r.blobConfig
|
|
919
|
+
? { ...r.blobConfig, blobs: r.blobConfig.blobs.map(b => ({ size: trimmedBytesLength(b) })) }
|
|
920
|
+
: undefined,
|
|
921
|
+
})),
|
|
922
|
+
});
|
|
923
|
+
const successfulActions: Action[] = [];
|
|
924
|
+
const failedActions: Action[] = [];
|
|
925
|
+
for (const request of requests) {
|
|
926
|
+
if (request.checkSuccess(request.request, result)) {
|
|
927
|
+
successfulActions.push(request.action);
|
|
928
|
+
} else {
|
|
929
|
+
failedActions.push(request.action);
|
|
595
930
|
}
|
|
596
|
-
return { successfulActions, failedActions };
|
|
597
931
|
}
|
|
932
|
+
return { successfulActions, failedActions };
|
|
598
933
|
}
|
|
599
934
|
|
|
600
935
|
/**
|
|
@@ -602,24 +937,27 @@ export class SequencerPublisher {
|
|
|
602
937
|
* @param tipArchive - The archive to check
|
|
603
938
|
* @returns The slot and block number if it is possible to propose, undefined otherwise
|
|
604
939
|
*/
|
|
605
|
-
public canProposeAt(
|
|
606
|
-
tipArchive: Fr,
|
|
607
|
-
msgSender: EthAddress,
|
|
608
|
-
opts: { forcePendingCheckpointNumber?: CheckpointNumber; pipelined?: boolean } = {},
|
|
609
|
-
) {
|
|
940
|
+
public async canProposeAt(tipArchive: Fr, msgSender: EthAddress, simulationOverridesPlan?: SimulationOverridesPlan) {
|
|
610
941
|
// TODO: #14291 - should loop through multiple keys to check if any of them can propose
|
|
611
|
-
|
|
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'];
|
|
612
947
|
|
|
613
|
-
const
|
|
614
|
-
const
|
|
948
|
+
const slotOffset = this.aztecSlotDuration;
|
|
949
|
+
const nextL1SlotTs = this.getNextL1SlotTimestamp() + slotOffset;
|
|
615
950
|
|
|
616
951
|
return this.rollupContract
|
|
617
|
-
.canProposeAt(
|
|
618
|
-
|
|
619
|
-
|
|
952
|
+
.canProposeAt(
|
|
953
|
+
tipArchive.toBuffer(),
|
|
954
|
+
msgSender.toString(),
|
|
955
|
+
nextL1SlotTs,
|
|
956
|
+
await buildSimulationOverridesStateOverride(this.rollupContract, simulationOverridesPlan),
|
|
957
|
+
)
|
|
620
958
|
.catch(err => {
|
|
621
|
-
if (err instanceof FormattedViemError &&
|
|
622
|
-
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))}`, {
|
|
623
961
|
error: err.message,
|
|
624
962
|
});
|
|
625
963
|
} else {
|
|
@@ -638,13 +976,13 @@ export class SequencerPublisher {
|
|
|
638
976
|
@trackSpan('SequencerPublisher.validateBlockHeader')
|
|
639
977
|
public async validateBlockHeader(
|
|
640
978
|
header: CheckpointHeader,
|
|
641
|
-
|
|
979
|
+
simulationOverridesPlan?: SimulationOverridesPlan,
|
|
642
980
|
): Promise<void> {
|
|
643
981
|
const flags = { ignoreDA: true, ignoreSignatures: true };
|
|
644
982
|
|
|
645
983
|
const args = [
|
|
646
984
|
header.toViem(),
|
|
647
|
-
CommitteeAttestationsAndSigners.
|
|
985
|
+
CommitteeAttestationsAndSigners.packAttestations([]),
|
|
648
986
|
[], // no signers
|
|
649
987
|
Signature.empty().toViemSignature(),
|
|
650
988
|
`0x${'0'.repeat(64)}`, // 32 empty bytes
|
|
@@ -652,10 +990,9 @@ export class SequencerPublisher {
|
|
|
652
990
|
flags,
|
|
653
991
|
] as const;
|
|
654
992
|
|
|
655
|
-
const
|
|
656
|
-
const
|
|
657
|
-
|
|
658
|
-
);
|
|
993
|
+
const l1Constants = this.epochCache.getL1Constants();
|
|
994
|
+
const ts = getLastL1SlotTimestampForL2Slot(header.slotNumber, l1Constants);
|
|
995
|
+
const stateOverrides = await buildSimulationOverridesStateOverride(this.rollupContract, simulationOverridesPlan);
|
|
659
996
|
let balance = 0n;
|
|
660
997
|
if (this.config.fishermanMode) {
|
|
661
998
|
// In fisherman mode, we can't know where the proposer is publishing from
|
|
@@ -675,7 +1012,7 @@ export class SequencerPublisher {
|
|
|
675
1012
|
data: encodeFunctionData({ abi: RollupAbi, functionName: 'validateHeaderWithAttestations', args }),
|
|
676
1013
|
from: MULTI_CALL_3_ADDRESS,
|
|
677
1014
|
},
|
|
678
|
-
{ time: ts
|
|
1015
|
+
{ time: ts },
|
|
679
1016
|
stateOverrides,
|
|
680
1017
|
);
|
|
681
1018
|
this.log.debug(`Simulated validateHeader`);
|
|
@@ -728,6 +1065,7 @@ export class SequencerPublisher {
|
|
|
728
1065
|
gasUsed,
|
|
729
1066
|
checkpointNumber,
|
|
730
1067
|
forcePendingCheckpointNumber: CheckpointNumber(checkpointNumber - 1),
|
|
1068
|
+
lastArchive: validationResult.checkpoint.lastArchive,
|
|
731
1069
|
reason,
|
|
732
1070
|
};
|
|
733
1071
|
} catch (err) {
|
|
@@ -740,8 +1078,8 @@ export class SequencerPublisher {
|
|
|
740
1078
|
`Simulation for invalidate checkpoint ${checkpointNumber} failed due to checkpoint not being in pending chain`,
|
|
741
1079
|
{ ...logData, request, error: viemError.message },
|
|
742
1080
|
);
|
|
743
|
-
const
|
|
744
|
-
if (
|
|
1081
|
+
const latestProposedCheckpointNumber = await this.rollupContract.getCheckpointNumber();
|
|
1082
|
+
if (latestProposedCheckpointNumber < checkpointNumber) {
|
|
745
1083
|
this.log.verbose(`Checkpoint ${checkpointNumber} has already been invalidated`, { ...logData });
|
|
746
1084
|
return undefined;
|
|
747
1085
|
} else {
|
|
@@ -761,18 +1099,22 @@ export class SequencerPublisher {
|
|
|
761
1099
|
|
|
762
1100
|
// Otherwise, throw. We cannot build the next checkpoint if we cannot invalidate the previous one.
|
|
763
1101
|
this.log.error(`Simulation for invalidate checkpoint ${checkpointNumber} failed`, viemError, logData);
|
|
764
|
-
this.backupFailedTx(
|
|
765
|
-
|
|
766
|
-
|
|
767
|
-
|
|
768
|
-
|
|
769
|
-
|
|
770
|
-
|
|
771
|
-
|
|
772
|
-
|
|
773
|
-
|
|
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()),
|
|
774
1115
|
},
|
|
775
|
-
|
|
1116
|
+
{ captureFeeSummary: true, targetSlot: this.getCurrentL2Slot() },
|
|
1117
|
+
);
|
|
776
1118
|
throw new Error(`Failed to simulate invalidate checkpoint ${checkpointNumber}`, { cause: viemError });
|
|
777
1119
|
}
|
|
778
1120
|
}
|
|
@@ -786,9 +1128,11 @@ export class SequencerPublisher {
|
|
|
786
1128
|
const logData = { ...checkpoint, reason };
|
|
787
1129
|
this.log.debug(`Building invalidate checkpoint ${checkpoint.checkpointNumber} request`, logData);
|
|
788
1130
|
|
|
789
|
-
|
|
790
|
-
|
|
791
|
-
)
|
|
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;
|
|
792
1136
|
|
|
793
1137
|
if (reason === 'invalid-attestation') {
|
|
794
1138
|
return this.rollupContract.buildInvalidateBadAttestationRequest(
|
|
@@ -809,45 +1153,11 @@ export class SequencerPublisher {
|
|
|
809
1153
|
}
|
|
810
1154
|
}
|
|
811
1155
|
|
|
812
|
-
/** Simulates `propose` to make sure that the checkpoint is valid for submission */
|
|
813
|
-
@trackSpan('SequencerPublisher.validateCheckpointForSubmission')
|
|
814
|
-
public async validateCheckpointForSubmission(
|
|
815
|
-
checkpoint: Checkpoint,
|
|
816
|
-
attestationsAndSigners: CommitteeAttestationsAndSigners,
|
|
817
|
-
attestationsAndSignersSignature: Signature,
|
|
818
|
-
options: { forcePendingCheckpointNumber?: CheckpointNumber },
|
|
819
|
-
): Promise<bigint> {
|
|
820
|
-
// Anchor the simulation timestamp to the checkpoint's own slot start time
|
|
821
|
-
// rather than the current L1 block timestamp, which may overshoot into the next slot if the build ran late.
|
|
822
|
-
const ts = checkpoint.header.timestamp;
|
|
823
|
-
const blobFields = checkpoint.toBlobFields();
|
|
824
|
-
const blobs = await getBlobsPerL1Block(blobFields);
|
|
825
|
-
const blobInput = getPrefixedEthBlobCommitments(blobs);
|
|
826
|
-
|
|
827
|
-
const args = [
|
|
828
|
-
{
|
|
829
|
-
header: checkpoint.header.toViem(),
|
|
830
|
-
archive: toHex(checkpoint.archive.root.toBuffer()),
|
|
831
|
-
oracleInput: {
|
|
832
|
-
feeAssetPriceModifier: checkpoint.feeAssetPriceModifier,
|
|
833
|
-
},
|
|
834
|
-
},
|
|
835
|
-
attestationsAndSigners.getPackedAttestations(),
|
|
836
|
-
attestationsAndSigners.getSigners().map(signer => signer.toString()),
|
|
837
|
-
attestationsAndSignersSignature.toViemSignature(),
|
|
838
|
-
blobInput,
|
|
839
|
-
] as const;
|
|
840
|
-
|
|
841
|
-
await this.simulateProposeTx(args, ts, options);
|
|
842
|
-
return ts;
|
|
843
|
-
}
|
|
844
|
-
|
|
845
1156
|
private async enqueueCastSignalHelper(
|
|
846
1157
|
slotNumber: SlotNumber,
|
|
847
|
-
timestamp: bigint,
|
|
848
1158
|
signalType: GovernanceSignalAction,
|
|
849
1159
|
payload: EthAddress,
|
|
850
|
-
base:
|
|
1160
|
+
base: GovernanceProposerContract,
|
|
851
1161
|
signerAddress: EthAddress,
|
|
852
1162
|
signer: (msg: TypedDataDefinition) => Promise<`0x${string}`>,
|
|
853
1163
|
): Promise<boolean> {
|
|
@@ -873,34 +1183,28 @@ export class SequencerPublisher {
|
|
|
873
1183
|
return false;
|
|
874
1184
|
}
|
|
875
1185
|
|
|
876
|
-
if (await
|
|
1186
|
+
if (await base.isPayloadEmpty(payload)) {
|
|
877
1187
|
this.log.warn(`Skipping vote cast for payload with empty code`);
|
|
878
1188
|
return false;
|
|
879
1189
|
}
|
|
880
1190
|
|
|
881
|
-
//
|
|
882
|
-
|
|
883
|
-
|
|
884
|
-
|
|
885
|
-
|
|
886
|
-
|
|
887
|
-
|
|
888
|
-
|
|
889
|
-
|
|
890
|
-
|
|
891
|
-
|
|
892
|
-
|
|
893
|
-
|
|
894
|
-
this.payloadProposedCache.add(cacheKey);
|
|
895
|
-
}
|
|
896
|
-
} catch (err) {
|
|
897
|
-
this.log.warn(`Failed to check if payload ${payload} was proposed after retries, skipping signal`, err);
|
|
898
|
-
return false;
|
|
899
|
-
}
|
|
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);
|
|
900
1204
|
}
|
|
901
1205
|
|
|
902
|
-
if (
|
|
903
|
-
this.log.info(`Payload ${payload}
|
|
1206
|
+
if (proposed) {
|
|
1207
|
+
this.log.info(`Payload ${payload} has a live governance proposal, stopping signals`);
|
|
904
1208
|
return false;
|
|
905
1209
|
}
|
|
906
1210
|
|
|
@@ -922,41 +1226,19 @@ export class SequencerPublisher {
|
|
|
922
1226
|
lastValidL2Slot: slotNumber,
|
|
923
1227
|
});
|
|
924
1228
|
|
|
925
|
-
const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
|
|
926
|
-
|
|
927
|
-
try {
|
|
928
|
-
await this.l1TxUtils.simulate(request, { time: timestamp }, [], mergeAbis([request.abi ?? [], ErrorsAbi]));
|
|
929
|
-
this.log.debug(`Simulation for ${action} at slot ${slotNumber} succeeded`, { request });
|
|
930
|
-
} catch (err) {
|
|
931
|
-
const viemError = formatViemError(err);
|
|
932
|
-
this.log.error(`Failed simulation for ${action} at slot ${slotNumber} (enqueuing the action anyway)`, viemError);
|
|
933
|
-
this.backupFailedTx({
|
|
934
|
-
id: keccak256(request.data!),
|
|
935
|
-
failureType: 'simulation',
|
|
936
|
-
request: { to: request.to!, data: request.data!, value: request.value?.toString() },
|
|
937
|
-
l1BlockNumber: l1BlockNumber.toString(),
|
|
938
|
-
error: { message: viemError.message, name: viemError.name },
|
|
939
|
-
context: {
|
|
940
|
-
actions: [action],
|
|
941
|
-
slot: slotNumber,
|
|
942
|
-
sender: this.getSenderAddress().toString(),
|
|
943
|
-
},
|
|
944
|
-
});
|
|
945
|
-
// Yes, we enqueue the request anyway, in case there was a bug with the simulation itself
|
|
946
|
-
}
|
|
947
|
-
|
|
948
1229
|
// TODO(palla/slash): All votes (governance and slashing) should txTimeoutAt at the end of the slot.
|
|
949
1230
|
this.addRequest({
|
|
950
|
-
gasConfig: { gasLimit: SequencerPublisher.VOTE_GAS_GUESS },
|
|
951
1231
|
action,
|
|
952
1232
|
request,
|
|
953
1233
|
lastValidL2Slot: slotNumber,
|
|
954
1234
|
checkSuccess: (_request, result) => {
|
|
955
1235
|
const success =
|
|
956
1236
|
result &&
|
|
957
|
-
result.receipt
|
|
958
|
-
|
|
959
|
-
|
|
1237
|
+
extractEventSuccess(result.receipt, {
|
|
1238
|
+
address: base.address.toString(),
|
|
1239
|
+
abi: EmpireBaseAbi,
|
|
1240
|
+
eventName: 'SignalCast',
|
|
1241
|
+
});
|
|
960
1242
|
|
|
961
1243
|
const logData = { ...result, slotNumber, round, payload: payload.toString() };
|
|
962
1244
|
if (!success) {
|
|
@@ -978,33 +1260,19 @@ export class SequencerPublisher {
|
|
|
978
1260
|
return true;
|
|
979
1261
|
}
|
|
980
1262
|
|
|
981
|
-
private async isPayloadEmpty(payload: EthAddress): Promise<boolean> {
|
|
982
|
-
const key = payload.toString();
|
|
983
|
-
const cached = this.isPayloadEmptyCache.get(key);
|
|
984
|
-
if (cached) {
|
|
985
|
-
return cached;
|
|
986
|
-
}
|
|
987
|
-
const isEmpty = !(await this.l1TxUtils.getCode(payload));
|
|
988
|
-
this.isPayloadEmptyCache.set(key, isEmpty);
|
|
989
|
-
return isEmpty;
|
|
990
|
-
}
|
|
991
|
-
|
|
992
1263
|
/**
|
|
993
1264
|
* Enqueues a governance castSignal transaction to cast a signal for a given slot number.
|
|
994
1265
|
* @param slotNumber - The slot number to cast a signal for.
|
|
995
|
-
* @param timestamp - The timestamp of the slot to cast a signal for.
|
|
996
1266
|
* @returns True if the signal was successfully enqueued, false otherwise.
|
|
997
1267
|
*/
|
|
998
1268
|
public enqueueGovernanceCastSignal(
|
|
999
1269
|
governancePayload: EthAddress,
|
|
1000
1270
|
slotNumber: SlotNumber,
|
|
1001
|
-
timestamp: bigint,
|
|
1002
1271
|
signerAddress: EthAddress,
|
|
1003
1272
|
signer: (msg: TypedDataDefinition) => Promise<`0x${string}`>,
|
|
1004
1273
|
): Promise<boolean> {
|
|
1005
1274
|
return this.enqueueCastSignalHelper(
|
|
1006
1275
|
slotNumber,
|
|
1007
|
-
timestamp,
|
|
1008
1276
|
'governance-signal',
|
|
1009
1277
|
governancePayload,
|
|
1010
1278
|
this.govProposerContract,
|
|
@@ -1013,11 +1281,51 @@ export class SequencerPublisher {
|
|
|
1013
1281
|
);
|
|
1014
1282
|
}
|
|
1015
1283
|
|
|
1284
|
+
/**
|
|
1285
|
+
* Enqueues a `prune()` transaction if the rollup is prunable at the given slot's L1 timestamp.
|
|
1286
|
+
* `prune()` is permissionless and idempotent — if the chain is no longer prunable by send time the
|
|
1287
|
+
* bundle simulation usually drops the entry; on a node without `eth_simulateV1` the bundle is sent
|
|
1288
|
+
* as-is and the prune reverts `Rollup__NothingToPrune` inside `aggregate3(allowFailure: true)`
|
|
1289
|
+
* (a failed action, never a whole-tx revert). Used by the failed-sync fallback so a stuck pending
|
|
1290
|
+
* chain (e.g. bad data blocking sync) can be wound back to recover.
|
|
1291
|
+
* @returns true if a prune request was enqueued, false otherwise.
|
|
1292
|
+
*/
|
|
1293
|
+
public async enqueuePruneIfPrunable(slotNumber: SlotNumber): Promise<boolean> {
|
|
1294
|
+
if (this.lastActions['prune'] === slotNumber) {
|
|
1295
|
+
this.log.debug(`Skipping duplicate prune for slot ${slotNumber}`, { slotNumber });
|
|
1296
|
+
return false;
|
|
1297
|
+
}
|
|
1298
|
+
// Use the SAME timestamp the bundle simulator overrides block.timestamp with at send time
|
|
1299
|
+
// (sequencer-bundle-simulator.ts) so this upfront check and the send-time sim agree. Slot-start
|
|
1300
|
+
// and last-L1-slot both fall within the same L2 slot (and epoch, which is what `canPruneAtTime`
|
|
1301
|
+
// derives), so they agree today; matching the simulator keeps it robust if the contract ever uses
|
|
1302
|
+
// the timestamp more granularly.
|
|
1303
|
+
const ts = getLastL1SlotTimestampForL2Slot(slotNumber, this.epochCache.getL1Constants());
|
|
1304
|
+
const canPrune = await this.rollupContract.canPruneAtTime(ts).catch(err => {
|
|
1305
|
+
this.log.error(`Failed to check canPruneAtTime for slot ${slotNumber}`, err, { slotNumber });
|
|
1306
|
+
return false;
|
|
1307
|
+
});
|
|
1308
|
+
if (!canPrune) {
|
|
1309
|
+
this.log.debug(`Rollup not prunable at slot ${slotNumber}`, { slotNumber });
|
|
1310
|
+
return false;
|
|
1311
|
+
}
|
|
1312
|
+
const request: L1TxRequest = {
|
|
1313
|
+
to: this.rollupContract.address,
|
|
1314
|
+
data: encodeFunctionData({ abi: RollupAbi, functionName: 'prune', args: [] }),
|
|
1315
|
+
};
|
|
1316
|
+
this.log.info(`Enqueuing rollup prune for slot ${slotNumber}`, { slotNumber });
|
|
1317
|
+
return this.enqueueRequest(
|
|
1318
|
+
'prune',
|
|
1319
|
+
request,
|
|
1320
|
+
{ address: this.rollupContract.address, abi: RollupAbi, eventName: 'PrunedPending' },
|
|
1321
|
+
slotNumber,
|
|
1322
|
+
);
|
|
1323
|
+
}
|
|
1324
|
+
|
|
1016
1325
|
/** Enqueues all slashing actions as returned by the slasher client. */
|
|
1017
1326
|
public async enqueueSlashingActions(
|
|
1018
1327
|
actions: ProposerSlashAction[],
|
|
1019
1328
|
slotNumber: SlotNumber,
|
|
1020
|
-
timestamp: bigint,
|
|
1021
1329
|
signerAddress: EthAddress,
|
|
1022
1330
|
signer: (msg: TypedDataDefinition) => Promise<`0x${string}`>,
|
|
1023
1331
|
): Promise<boolean> {
|
|
@@ -1028,58 +1336,6 @@ export class SequencerPublisher {
|
|
|
1028
1336
|
|
|
1029
1337
|
for (const action of actions) {
|
|
1030
1338
|
switch (action.type) {
|
|
1031
|
-
case 'vote-empire-payload': {
|
|
1032
|
-
if (this.slashingProposerContract?.type !== 'empire') {
|
|
1033
|
-
this.log.error('Cannot vote for empire payload on non-empire slashing contract');
|
|
1034
|
-
break;
|
|
1035
|
-
}
|
|
1036
|
-
this.log.debug(`Enqueuing slashing vote for payload ${action.payload} at slot ${slotNumber}`, {
|
|
1037
|
-
signerAddress,
|
|
1038
|
-
});
|
|
1039
|
-
await this.enqueueCastSignalHelper(
|
|
1040
|
-
slotNumber,
|
|
1041
|
-
timestamp,
|
|
1042
|
-
'empire-slashing-signal',
|
|
1043
|
-
action.payload,
|
|
1044
|
-
this.slashingProposerContract,
|
|
1045
|
-
signerAddress,
|
|
1046
|
-
signer,
|
|
1047
|
-
);
|
|
1048
|
-
break;
|
|
1049
|
-
}
|
|
1050
|
-
|
|
1051
|
-
case 'create-empire-payload': {
|
|
1052
|
-
this.log.debug(`Enqueuing slashing create payload at slot ${slotNumber}`, { slotNumber, signerAddress });
|
|
1053
|
-
const request = this.slashFactoryContract.buildCreatePayloadRequest(action.data);
|
|
1054
|
-
await this.simulateAndEnqueueRequest(
|
|
1055
|
-
'create-empire-payload',
|
|
1056
|
-
request,
|
|
1057
|
-
(receipt: TransactionReceipt) =>
|
|
1058
|
-
!!this.slashFactoryContract.tryExtractSlashPayloadCreatedEvent(receipt.logs),
|
|
1059
|
-
slotNumber,
|
|
1060
|
-
timestamp,
|
|
1061
|
-
);
|
|
1062
|
-
break;
|
|
1063
|
-
}
|
|
1064
|
-
|
|
1065
|
-
case 'execute-empire-payload': {
|
|
1066
|
-
this.log.debug(`Enqueuing slashing execute payload at slot ${slotNumber}`, { slotNumber, signerAddress });
|
|
1067
|
-
if (this.slashingProposerContract?.type !== 'empire') {
|
|
1068
|
-
this.log.error('Cannot execute slashing payload on non-empire slashing contract');
|
|
1069
|
-
return false;
|
|
1070
|
-
}
|
|
1071
|
-
const empireSlashingProposer = this.slashingProposerContract as EmpireSlashingProposerContract;
|
|
1072
|
-
const request = empireSlashingProposer.buildExecuteRoundRequest(action.round);
|
|
1073
|
-
await this.simulateAndEnqueueRequest(
|
|
1074
|
-
'execute-empire-payload',
|
|
1075
|
-
request,
|
|
1076
|
-
(receipt: TransactionReceipt) => !!empireSlashingProposer.tryExtractPayloadSubmittedEvent(receipt.logs),
|
|
1077
|
-
slotNumber,
|
|
1078
|
-
timestamp,
|
|
1079
|
-
);
|
|
1080
|
-
break;
|
|
1081
|
-
}
|
|
1082
|
-
|
|
1083
1339
|
case 'vote-offenses': {
|
|
1084
1340
|
this.log.debug(`Enqueuing slashing vote for ${action.votes.length} votes at slot ${slotNumber}`, {
|
|
1085
1341
|
slotNumber,
|
|
@@ -1087,19 +1343,21 @@ export class SequencerPublisher {
|
|
|
1087
1343
|
votesCount: action.votes.length,
|
|
1088
1344
|
signerAddress,
|
|
1089
1345
|
});
|
|
1090
|
-
if (this.slashingProposerContract
|
|
1091
|
-
this.log.error('
|
|
1346
|
+
if (!this.slashingProposerContract) {
|
|
1347
|
+
this.log.error('No slashing proposer contract available');
|
|
1092
1348
|
return false;
|
|
1093
1349
|
}
|
|
1094
|
-
const tallySlashingProposer = this.slashingProposerContract as TallySlashingProposerContract;
|
|
1095
1350
|
const votes = bufferToHex(encodeSlashConsensusVotes(action.votes));
|
|
1096
|
-
const request = await
|
|
1097
|
-
|
|
1351
|
+
const request = await this.slashingProposerContract.buildVoteRequestFromSigner(votes, slotNumber, signer);
|
|
1352
|
+
this.enqueueRequest(
|
|
1098
1353
|
'vote-offenses',
|
|
1099
1354
|
request,
|
|
1100
|
-
|
|
1355
|
+
{
|
|
1356
|
+
address: this.slashingProposerContract.address.toString(),
|
|
1357
|
+
abi: SlashingProposerAbi,
|
|
1358
|
+
eventName: 'VoteCast',
|
|
1359
|
+
},
|
|
1101
1360
|
slotNumber,
|
|
1102
|
-
timestamp,
|
|
1103
1361
|
);
|
|
1104
1362
|
break;
|
|
1105
1363
|
}
|
|
@@ -1110,18 +1368,23 @@ export class SequencerPublisher {
|
|
|
1110
1368
|
round: action.round,
|
|
1111
1369
|
signerAddress,
|
|
1112
1370
|
});
|
|
1113
|
-
if (this.slashingProposerContract
|
|
1114
|
-
this.log.error('
|
|
1371
|
+
if (!this.slashingProposerContract) {
|
|
1372
|
+
this.log.error('No slashing proposer contract available');
|
|
1115
1373
|
return false;
|
|
1116
1374
|
}
|
|
1117
|
-
const
|
|
1118
|
-
|
|
1119
|
-
|
|
1375
|
+
const executeRequest = this.slashingProposerContract.buildExecuteRoundRequest(
|
|
1376
|
+
action.round,
|
|
1377
|
+
action.committees,
|
|
1378
|
+
);
|
|
1379
|
+
this.enqueueRequest(
|
|
1120
1380
|
'execute-slash',
|
|
1121
|
-
|
|
1122
|
-
|
|
1381
|
+
executeRequest,
|
|
1382
|
+
{
|
|
1383
|
+
address: this.slashingProposerContract.address.toString(),
|
|
1384
|
+
abi: SlashingProposerAbi,
|
|
1385
|
+
eventName: 'RoundExecuted',
|
|
1386
|
+
},
|
|
1123
1387
|
slotNumber,
|
|
1124
|
-
timestamp,
|
|
1125
1388
|
);
|
|
1126
1389
|
break;
|
|
1127
1390
|
}
|
|
@@ -1136,12 +1399,12 @@ export class SequencerPublisher {
|
|
|
1136
1399
|
return true;
|
|
1137
1400
|
}
|
|
1138
1401
|
|
|
1139
|
-
/**
|
|
1402
|
+
/** Enqueues a proposal for a checkpoint on L1 */
|
|
1140
1403
|
public async enqueueProposeCheckpoint(
|
|
1141
1404
|
checkpoint: Checkpoint,
|
|
1142
1405
|
attestationsAndSigners: CommitteeAttestationsAndSigners,
|
|
1143
1406
|
attestationsAndSignersSignature: Signature,
|
|
1144
|
-
opts:
|
|
1407
|
+
opts: EnqueueProposeCheckpointOpts = {},
|
|
1145
1408
|
): Promise<void> {
|
|
1146
1409
|
const checkpointHeader = checkpoint.header;
|
|
1147
1410
|
|
|
@@ -1157,31 +1420,11 @@ export class SequencerPublisher {
|
|
|
1157
1420
|
feeAssetPriceModifier: checkpoint.feeAssetPriceModifier,
|
|
1158
1421
|
};
|
|
1159
1422
|
|
|
1160
|
-
|
|
1161
|
-
|
|
1162
|
-
|
|
1163
|
-
|
|
1164
|
-
|
|
1165
|
-
// By simulation issue, I mean the fact that the block.timestamp is equal to the last block, not the next, which
|
|
1166
|
-
// make time consistency checks break.
|
|
1167
|
-
// TODO(palla): Check whether we're validating twice, once here and once within addProposeTx, since we call simulateProposeTx in both places.
|
|
1168
|
-
ts = await this.validateCheckpointForSubmission(
|
|
1169
|
-
checkpoint,
|
|
1170
|
-
attestationsAndSigners,
|
|
1171
|
-
attestationsAndSignersSignature,
|
|
1172
|
-
opts,
|
|
1173
|
-
);
|
|
1174
|
-
} catch (err: any) {
|
|
1175
|
-
this.log.error(`Checkpoint validation failed. ${err instanceof Error ? err.message : 'No error message'}`, err, {
|
|
1176
|
-
...checkpoint.getStats(),
|
|
1177
|
-
slotNumber: checkpoint.header.slotNumber,
|
|
1178
|
-
forcePendingCheckpointNumber: opts.forcePendingCheckpointNumber,
|
|
1179
|
-
});
|
|
1180
|
-
throw err;
|
|
1181
|
-
}
|
|
1182
|
-
|
|
1183
|
-
this.log.verbose(`Enqueuing checkpoint propose transaction`, { ...checkpoint.toCheckpointInfo(), ...opts });
|
|
1184
|
-
await this.addProposeTx(checkpoint, proposeTxArgs, opts, ts);
|
|
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 });
|
|
1185
1428
|
}
|
|
1186
1429
|
|
|
1187
1430
|
public enqueueInvalidateCheckpoint(
|
|
@@ -1192,23 +1435,22 @@ export class SequencerPublisher {
|
|
|
1192
1435
|
return;
|
|
1193
1436
|
}
|
|
1194
1437
|
|
|
1195
|
-
// We issued the simulation against the rollup contract, so we need to account for the overhead of the multicall3
|
|
1196
|
-
const gasLimit = this.l1TxUtils.bumpGasLimit(BigInt(Math.ceil((Number(request.gasUsed) * 64) / 63)));
|
|
1197
|
-
|
|
1198
1438
|
const { gasUsed, checkpointNumber } = request;
|
|
1199
|
-
const logData = { gasUsed, checkpointNumber,
|
|
1439
|
+
const logData = { gasUsed, checkpointNumber, opts };
|
|
1200
1440
|
this.log.verbose(`Enqueuing invalidate checkpoint request`, logData);
|
|
1201
1441
|
this.addRequest({
|
|
1202
1442
|
action: `invalidate-by-${request.reason}`,
|
|
1203
1443
|
request: request.request,
|
|
1204
|
-
gasConfig: {
|
|
1444
|
+
gasConfig: opts.txTimeoutAt ? { txTimeoutAt: opts.txTimeoutAt } : undefined,
|
|
1205
1445
|
lastValidL2Slot: SlotNumber(this.getCurrentL2Slot() + 2),
|
|
1206
1446
|
checkSuccess: (_req, result) => {
|
|
1207
1447
|
const success =
|
|
1208
1448
|
result &&
|
|
1209
|
-
result.receipt
|
|
1210
|
-
|
|
1211
|
-
|
|
1449
|
+
extractEventSuccess(result.receipt, {
|
|
1450
|
+
address: this.rollupContract.address,
|
|
1451
|
+
abi: RollupAbi,
|
|
1452
|
+
eventName: 'CheckpointInvalidated',
|
|
1453
|
+
});
|
|
1212
1454
|
if (!success) {
|
|
1213
1455
|
this.log.warn(`Invalidate checkpoint ${request.checkpointNumber} failed`, { ...result, ...logData });
|
|
1214
1456
|
} else {
|
|
@@ -1219,72 +1461,36 @@ export class SequencerPublisher {
|
|
|
1219
1461
|
});
|
|
1220
1462
|
}
|
|
1221
1463
|
|
|
1222
|
-
|
|
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(
|
|
1223
1470
|
action: Action,
|
|
1224
1471
|
request: L1TxRequest,
|
|
1225
|
-
|
|
1472
|
+
eventOpts: { address: string; abi: Abi; eventName: string },
|
|
1226
1473
|
slotNumber: SlotNumber,
|
|
1227
|
-
|
|
1228
|
-
) {
|
|
1229
|
-
const logData = { slotNumber, timestamp, gasLimit: undefined as bigint | undefined };
|
|
1474
|
+
): boolean {
|
|
1230
1475
|
if (this.lastActions[action] && this.lastActions[action] === slotNumber) {
|
|
1231
1476
|
this.log.debug(`Skipping duplicate action ${action} for slot ${slotNumber}`);
|
|
1232
1477
|
return false;
|
|
1233
1478
|
}
|
|
1234
|
-
|
|
1235
1479
|
const cachedLastActionSlot = this.lastActions[action];
|
|
1236
1480
|
this.lastActions[action] = slotNumber;
|
|
1237
1481
|
|
|
1238
|
-
this.log.debug(`
|
|
1239
|
-
|
|
1240
|
-
const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
|
|
1241
|
-
|
|
1242
|
-
let gasUsed: bigint;
|
|
1243
|
-
const simulateAbi = mergeAbis([request.abi ?? [], ErrorsAbi]);
|
|
1244
|
-
try {
|
|
1245
|
-
({ gasUsed } = await this.l1TxUtils.simulate(request, { time: timestamp }, [], simulateAbi)); // TODO(palla/slash): Check the timestamp logic
|
|
1246
|
-
this.log.verbose(`Simulation for ${action} succeeded`, { ...logData, request, gasUsed });
|
|
1247
|
-
} catch (err) {
|
|
1248
|
-
const viemError = formatViemError(err, simulateAbi);
|
|
1249
|
-
this.log.error(`Simulation for ${action} at ${slotNumber} failed`, viemError, logData);
|
|
1250
|
-
|
|
1251
|
-
this.backupFailedTx({
|
|
1252
|
-
id: keccak256(request.data!),
|
|
1253
|
-
failureType: 'simulation',
|
|
1254
|
-
request: { to: request.to!, data: request.data!, value: request.value?.toString() },
|
|
1255
|
-
l1BlockNumber: l1BlockNumber.toString(),
|
|
1256
|
-
error: { message: viemError.message, name: viemError.name },
|
|
1257
|
-
context: {
|
|
1258
|
-
actions: [action],
|
|
1259
|
-
slot: slotNumber,
|
|
1260
|
-
sender: this.getSenderAddress().toString(),
|
|
1261
|
-
},
|
|
1262
|
-
});
|
|
1263
|
-
|
|
1264
|
-
return false;
|
|
1265
|
-
}
|
|
1266
|
-
|
|
1267
|
-
// We issued the simulation against the rollup contract, so we need to account for the overhead of the multicall3
|
|
1268
|
-
const gasLimit = this.l1TxUtils.bumpGasLimit(BigInt(Math.ceil((Number(gasUsed) * 64) / 63)));
|
|
1269
|
-
logData.gasLimit = gasLimit;
|
|
1270
|
-
|
|
1271
|
-
// Store the ABI used for simulation on the request so Multicall3.forward can decode errors
|
|
1272
|
-
// when the tx is sent and a revert is diagnosed via simulation.
|
|
1273
|
-
const requestWithAbi = { ...request, abi: simulateAbi };
|
|
1274
|
-
|
|
1275
|
-
this.log.debug(`Enqueuing ${action}`, logData);
|
|
1482
|
+
this.log.debug(`Enqueuing ${action}`, { slotNumber });
|
|
1276
1483
|
this.addRequest({
|
|
1277
1484
|
action,
|
|
1278
|
-
request
|
|
1279
|
-
gasConfig: { gasLimit },
|
|
1485
|
+
request,
|
|
1280
1486
|
lastValidL2Slot: slotNumber,
|
|
1281
|
-
checkSuccess: (
|
|
1282
|
-
const success = result && result.receipt
|
|
1487
|
+
checkSuccess: (_request, result) => {
|
|
1488
|
+
const success = result && extractEventSuccess(result.receipt, eventOpts);
|
|
1283
1489
|
if (!success) {
|
|
1284
|
-
this.log.warn(`Action ${action} at ${slotNumber} failed`, { ...result,
|
|
1490
|
+
this.log.warn(`Action ${action} at ${slotNumber} failed`, { ...result, slotNumber });
|
|
1285
1491
|
this.lastActions[action] = cachedLastActionSlot;
|
|
1286
1492
|
} else {
|
|
1287
|
-
this.log.info(`Action ${action} at ${slotNumber} succeeded`, { ...result,
|
|
1493
|
+
this.log.info(`Action ${action} at ${slotNumber} succeeded`, { ...result, slotNumber });
|
|
1288
1494
|
}
|
|
1289
1495
|
return !!success;
|
|
1290
1496
|
},
|
|
@@ -1300,6 +1506,7 @@ export class SequencerPublisher {
|
|
|
1300
1506
|
*/
|
|
1301
1507
|
public interrupt() {
|
|
1302
1508
|
this.interrupted = true;
|
|
1509
|
+
this.interruptibleSleep.interrupt();
|
|
1303
1510
|
this.l1TxUtils.interrupt();
|
|
1304
1511
|
}
|
|
1305
1512
|
|
|
@@ -1309,11 +1516,7 @@ export class SequencerPublisher {
|
|
|
1309
1516
|
this.l1TxUtils.restart();
|
|
1310
1517
|
}
|
|
1311
1518
|
|
|
1312
|
-
private async prepareProposeTx(
|
|
1313
|
-
encodedData: L1ProcessArgs,
|
|
1314
|
-
timestamp: bigint,
|
|
1315
|
-
options: { forcePendingCheckpointNumber?: CheckpointNumber },
|
|
1316
|
-
) {
|
|
1519
|
+
private async prepareProposeTx(encodedData: L1ProcessArgs) {
|
|
1317
1520
|
const kzg = Blob.getViemKzgInstance();
|
|
1318
1521
|
const blobInput = getPrefixedEthBlobCommitments(encodedData.blobs);
|
|
1319
1522
|
this.log.debug('Validating blob input', { blobInput });
|
|
@@ -1326,7 +1529,11 @@ export class SequencerPublisher {
|
|
|
1326
1529
|
blobEvaluationGas = BigInt(encodedData.blobs.length) * 21_000n;
|
|
1327
1530
|
this.log.debug(`Using fixed blob evaluation gas estimate in fisherman mode: ${blobEvaluationGas}`);
|
|
1328
1531
|
} else {
|
|
1329
|
-
//
|
|
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.
|
|
1330
1537
|
blobEvaluationGas = await this.l1TxUtils
|
|
1331
1538
|
.estimateGas(
|
|
1332
1539
|
this.getSenderAddress().toString(),
|
|
@@ -1353,18 +1560,22 @@ export class SequencerPublisher {
|
|
|
1353
1560
|
args: [blobInput],
|
|
1354
1561
|
});
|
|
1355
1562
|
const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
|
|
1356
|
-
this.backupFailedTx(
|
|
1357
|
-
|
|
1358
|
-
|
|
1359
|
-
|
|
1360
|
-
|
|
1361
|
-
|
|
1362
|
-
|
|
1363
|
-
|
|
1364
|
-
|
|
1365
|
-
|
|
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()),
|
|
1366
1576
|
},
|
|
1367
|
-
|
|
1577
|
+
{ captureFeeSummary: true, targetSlot: this.getCurrentL2Slot() },
|
|
1578
|
+
);
|
|
1368
1579
|
throw new Error('Failed to validate blobs');
|
|
1369
1580
|
});
|
|
1370
1581
|
}
|
|
@@ -1384,138 +1595,21 @@ export class SequencerPublisher {
|
|
|
1384
1595
|
blobInput,
|
|
1385
1596
|
] as const;
|
|
1386
1597
|
|
|
1387
|
-
const
|
|
1598
|
+
const rollupData = encodeFunctionData({ abi: RollupAbi, functionName: 'propose', args });
|
|
1388
1599
|
|
|
1389
|
-
return { args, blobEvaluationGas, rollupData
|
|
1390
|
-
}
|
|
1391
|
-
|
|
1392
|
-
/**
|
|
1393
|
-
* Simulates the propose tx with eth_simulateV1
|
|
1394
|
-
* @param args - The propose tx args
|
|
1395
|
-
* @param timestamp - The timestamp to simulate proposal at
|
|
1396
|
-
* @returns The simulation result
|
|
1397
|
-
*/
|
|
1398
|
-
private async simulateProposeTx(
|
|
1399
|
-
args: readonly [
|
|
1400
|
-
{
|
|
1401
|
-
readonly header: ViemHeader;
|
|
1402
|
-
readonly archive: `0x${string}`;
|
|
1403
|
-
readonly oracleInput: {
|
|
1404
|
-
readonly feeAssetPriceModifier: bigint;
|
|
1405
|
-
};
|
|
1406
|
-
},
|
|
1407
|
-
ViemCommitteeAttestations,
|
|
1408
|
-
`0x${string}`[], // Signers
|
|
1409
|
-
ViemSignature,
|
|
1410
|
-
`0x${string}`,
|
|
1411
|
-
],
|
|
1412
|
-
timestamp: bigint,
|
|
1413
|
-
options: { forcePendingCheckpointNumber?: CheckpointNumber },
|
|
1414
|
-
) {
|
|
1415
|
-
const rollupData = encodeFunctionData({
|
|
1416
|
-
abi: RollupAbi,
|
|
1417
|
-
functionName: 'propose',
|
|
1418
|
-
args,
|
|
1419
|
-
});
|
|
1420
|
-
|
|
1421
|
-
// override the pending checkpoint number if requested
|
|
1422
|
-
const forcePendingCheckpointNumberStateDiff = (
|
|
1423
|
-
options.forcePendingCheckpointNumber !== undefined
|
|
1424
|
-
? await this.rollupContract.makePendingCheckpointNumberOverride(options.forcePendingCheckpointNumber)
|
|
1425
|
-
: []
|
|
1426
|
-
).flatMap(override => override.stateDiff ?? []);
|
|
1427
|
-
|
|
1428
|
-
const stateOverrides: StateOverride = [
|
|
1429
|
-
{
|
|
1430
|
-
address: this.rollupContract.address,
|
|
1431
|
-
// @note we override checkBlob to false since blobs are not part simulate()
|
|
1432
|
-
stateDiff: [
|
|
1433
|
-
{ slot: toPaddedHex(RollupContract.checkBlobStorageSlot, true), value: toPaddedHex(0n, true) },
|
|
1434
|
-
...forcePendingCheckpointNumberStateDiff,
|
|
1435
|
-
],
|
|
1436
|
-
},
|
|
1437
|
-
];
|
|
1438
|
-
// In fisherman mode, simulate as the proposer but with sufficient balance
|
|
1439
|
-
if (this.proposerAddressForSimulation) {
|
|
1440
|
-
stateOverrides.push({
|
|
1441
|
-
address: this.proposerAddressForSimulation.toString(),
|
|
1442
|
-
balance: 10n * WEI_CONST * WEI_CONST, // 10 ETH
|
|
1443
|
-
});
|
|
1444
|
-
}
|
|
1445
|
-
|
|
1446
|
-
const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
|
|
1447
|
-
|
|
1448
|
-
const simulationResult = await this.l1TxUtils
|
|
1449
|
-
.simulate(
|
|
1450
|
-
{
|
|
1451
|
-
to: this.rollupContract.address,
|
|
1452
|
-
data: rollupData,
|
|
1453
|
-
gas: MAX_L1_TX_LIMIT,
|
|
1454
|
-
...(this.proposerAddressForSimulation && { from: this.proposerAddressForSimulation.toString() }),
|
|
1455
|
-
},
|
|
1456
|
-
{
|
|
1457
|
-
// @note we add 1n to the timestamp because geth implementation doesn't like simulation timestamp to be equal to the current block timestamp
|
|
1458
|
-
time: timestamp + 1n,
|
|
1459
|
-
// @note reth should have a 30m gas limit per block but throws errors that this tx is beyond limit so we increase here
|
|
1460
|
-
gasLimit: MAX_L1_TX_LIMIT * 2n,
|
|
1461
|
-
},
|
|
1462
|
-
stateOverrides,
|
|
1463
|
-
RollupAbi,
|
|
1464
|
-
{
|
|
1465
|
-
// @note fallback gas estimate to use if the node doesn't support simulation API
|
|
1466
|
-
fallbackGasEstimate: MAX_L1_TX_LIMIT,
|
|
1467
|
-
},
|
|
1468
|
-
)
|
|
1469
|
-
.catch(err => {
|
|
1470
|
-
// In fisherman mode, we expect ValidatorSelection__MissingProposerSignature since fisherman doesn't have proposer signature
|
|
1471
|
-
const viemError = formatViemError(err);
|
|
1472
|
-
if (this.config.fishermanMode && viemError.message?.includes('ValidatorSelection__MissingProposerSignature')) {
|
|
1473
|
-
this.log.debug(`Ignoring expected ValidatorSelection__MissingProposerSignature error in fisherman mode`);
|
|
1474
|
-
// Return a minimal simulation result with the fallback gas estimate
|
|
1475
|
-
return {
|
|
1476
|
-
gasUsed: MAX_L1_TX_LIMIT,
|
|
1477
|
-
logs: [],
|
|
1478
|
-
};
|
|
1479
|
-
}
|
|
1480
|
-
this.log.error(`Failed to simulate propose tx`, viemError);
|
|
1481
|
-
this.backupFailedTx({
|
|
1482
|
-
id: keccak256(rollupData),
|
|
1483
|
-
failureType: 'simulation',
|
|
1484
|
-
request: { to: this.rollupContract.address, data: rollupData },
|
|
1485
|
-
l1BlockNumber: l1BlockNumber.toString(),
|
|
1486
|
-
error: { message: viemError.message, name: viemError.name },
|
|
1487
|
-
context: {
|
|
1488
|
-
actions: ['propose'],
|
|
1489
|
-
slot: Number(args[0].header.slotNumber),
|
|
1490
|
-
sender: this.getSenderAddress().toString(),
|
|
1491
|
-
},
|
|
1492
|
-
});
|
|
1493
|
-
throw err;
|
|
1494
|
-
});
|
|
1495
|
-
|
|
1496
|
-
return { rollupData, simulationResult };
|
|
1600
|
+
return { args, blobEvaluationGas, rollupData };
|
|
1497
1601
|
}
|
|
1498
1602
|
|
|
1499
1603
|
private async addProposeTx(
|
|
1500
1604
|
checkpoint: Checkpoint,
|
|
1501
1605
|
encodedData: L1ProcessArgs,
|
|
1502
|
-
opts:
|
|
1503
|
-
timestamp: bigint,
|
|
1606
|
+
opts: EnqueueProposeCheckpointOpts = {},
|
|
1504
1607
|
): Promise<void> {
|
|
1505
1608
|
const slot = checkpoint.header.slotNumber;
|
|
1506
1609
|
const timer = new Timer();
|
|
1507
1610
|
const kzg = Blob.getViemKzgInstance();
|
|
1508
|
-
const { rollupData,
|
|
1509
|
-
encodedData,
|
|
1510
|
-
timestamp,
|
|
1511
|
-
opts,
|
|
1512
|
-
);
|
|
1611
|
+
const { rollupData, blobEvaluationGas } = await this.prepareProposeTx(encodedData);
|
|
1513
1612
|
const startBlock = await this.l1TxUtils.getBlockNumber();
|
|
1514
|
-
const gasLimit = this.l1TxUtils.bumpGasLimit(
|
|
1515
|
-
BigInt(Math.ceil((Number(simulationResult.gasUsed) * 64) / 63)) +
|
|
1516
|
-
blobEvaluationGas +
|
|
1517
|
-
SequencerPublisher.MULTICALL_OVERHEAD_GAS_GUESS, // We issue the simulation against the rollup contract, so we need to account for the overhead of the multicall3
|
|
1518
|
-
);
|
|
1519
1613
|
|
|
1520
1614
|
// Send the blobs to the blob client preemptively. This helps in tests where the sequencer mistakingly thinks that the propose
|
|
1521
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.
|
|
@@ -1532,7 +1626,8 @@ export class SequencerPublisher {
|
|
|
1532
1626
|
data: rollupData,
|
|
1533
1627
|
},
|
|
1534
1628
|
lastValidL2Slot: checkpoint.header.slotNumber,
|
|
1535
|
-
gasConfig: {
|
|
1629
|
+
gasConfig: { txTimeoutAt: opts.txTimeoutAt, gasLimit: undefined },
|
|
1630
|
+
blobEvaluationGas,
|
|
1536
1631
|
blobConfig: {
|
|
1537
1632
|
blobs: encodedData.blobs.map(b => b.data),
|
|
1538
1633
|
kzg,
|
|
@@ -1542,10 +1637,11 @@ export class SequencerPublisher {
|
|
|
1542
1637
|
return false;
|
|
1543
1638
|
}
|
|
1544
1639
|
const { receipt, stats, errorMsg } = result;
|
|
1545
|
-
const success =
|
|
1546
|
-
|
|
1547
|
-
|
|
1548
|
-
|
|
1640
|
+
const success = extractEventSuccess(receipt, {
|
|
1641
|
+
address: this.rollupContract.address,
|
|
1642
|
+
abi: RollupAbi,
|
|
1643
|
+
eventName: 'CheckpointProposed',
|
|
1644
|
+
});
|
|
1549
1645
|
|
|
1550
1646
|
if (success) {
|
|
1551
1647
|
const endBlock = receipt.blockNumber;
|
|
@@ -1585,4 +1681,10 @@ export class SequencerPublisher {
|
|
|
1585
1681
|
},
|
|
1586
1682
|
});
|
|
1587
1683
|
}
|
|
1684
|
+
|
|
1685
|
+
/** Returns the timestamp of the next L1 slot boundary after now. */
|
|
1686
|
+
private getNextL1SlotTimestamp(): bigint {
|
|
1687
|
+
const l1Constants = this.epochCache.getL1Constants();
|
|
1688
|
+
return getNextL1SlotTimestamp(this.dateProvider.nowInSeconds(), l1Constants);
|
|
1689
|
+
}
|
|
1588
1690
|
}
|