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