@aztec/sequencer-client 0.0.1-commit.d3ec352c → 0.0.1-commit.d58ff9d0
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 +41 -18
- package/dest/client/sequencer-client.d.ts.map +1 -1
- package/dest/client/sequencer-client.js +88 -43
- package/dest/config.d.ts +41 -8
- package/dest/config.d.ts.map +1 -1
- package/dest/config.js +146 -55
- 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 +20 -26
- package/dest/global_variable_builder/global_builder.d.ts.map +1 -1
- package/dest/global_variable_builder/global_builder.js +28 -75
- 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/index.d.ts +2 -3
- package/dest/index.d.ts.map +1 -1
- package/dest/index.js +1 -2
- package/dest/publisher/config.d.ts +57 -20
- package/dest/publisher/config.d.ts.map +1 -1
- package/dest/publisher/config.js +134 -39
- 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 +15 -8
- package/dest/publisher/sequencer-publisher-factory.d.ts.map +1 -1
- package/dest/publisher/sequencer-publisher-factory.js +28 -4
- package/dest/publisher/sequencer-publisher-metrics.d.ts +3 -3
- package/dest/publisher/sequencer-publisher-metrics.d.ts.map +1 -1
- package/dest/publisher/sequencer-publisher-metrics.js +23 -86
- package/dest/publisher/sequencer-publisher.d.ts +152 -90
- package/dest/publisher/sequencer-publisher.d.ts.map +1 -1
- package/dest/publisher/sequencer-publisher.js +1257 -421
- 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 +158 -0
- package/dest/sequencer/checkpoint_proposal_job.d.ts.map +1 -0
- package/dest/sequencer/checkpoint_proposal_job.js +1801 -0
- 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 +34 -0
- package/dest/sequencer/checkpoint_voter.d.ts.map +1 -0
- package/dest/sequencer/checkpoint_voter.js +106 -0
- package/dest/sequencer/config.d.ts +3 -2
- package/dest/sequencer/config.d.ts.map +1 -1
- package/dest/sequencer/events.d.ts +103 -0
- package/dest/sequencer/events.d.ts.map +1 -0
- package/dest/sequencer/events.js +1 -0
- package/dest/sequencer/index.d.ts +4 -2
- package/dest/sequencer/index.d.ts.map +1 -1
- package/dest/sequencer/index.js +3 -1
- package/dest/sequencer/metrics.d.ts +42 -7
- package/dest/sequencer/metrics.d.ts.map +1 -1
- package/dest/sequencer/metrics.js +241 -72
- 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 +251 -138
- package/dest/sequencer/sequencer.d.ts.map +1 -1
- package/dest/sequencer/sequencer.js +1190 -704
- package/dest/sequencer/types.d.ts +3 -0
- package/dest/sequencer/types.d.ts.map +1 -0
- package/dest/sequencer/types.js +1 -0
- package/dest/sequencer/utils.d.ts +14 -8
- package/dest/sequencer/utils.d.ts.map +1 -1
- package/dest/sequencer/utils.js +7 -4
- package/dest/test/index.d.ts +8 -9
- package/dest/test/index.d.ts.map +1 -1
- package/dest/test/mock_checkpoint_builder.d.ts +95 -0
- package/dest/test/mock_checkpoint_builder.d.ts.map +1 -0
- package/dest/test/mock_checkpoint_builder.js +231 -0
- package/dest/test/utils.d.ts +67 -0
- package/dest/test/utils.d.ts.map +1 -0
- package/dest/test/utils.js +122 -0
- package/package.json +32 -29
- package/src/client/sequencer-client.ts +118 -66
- package/src/config.ts +173 -64
- 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 +45 -110
- package/src/global_variable_builder/index.ts +3 -1
- package/src/index.ts +1 -7
- package/src/publisher/config.ts +182 -46
- 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 +43 -13
- package/src/publisher/sequencer-publisher-metrics.ts +19 -71
- package/src/publisher/sequencer-publisher.ts +1042 -542
- package/src/publisher/write_json.ts +78 -0
- package/src/sequencer/README.md +243 -0
- 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 +1652 -0
- package/src/sequencer/checkpoint_proposal_job_metrics.ts +128 -0
- package/src/sequencer/checkpoint_voter.ts +119 -0
- package/src/sequencer/config.ts +2 -1
- package/src/sequencer/events.ts +88 -0
- package/src/sequencer/index.ts +3 -1
- package/src/sequencer/metrics.ts +302 -83
- package/src/sequencer/requests_tracker.ts +43 -0
- package/src/sequencer/sequencer.ts +1011 -898
- package/src/sequencer/types.ts +6 -0
- package/src/sequencer/utils.ts +18 -9
- package/src/test/index.ts +7 -8
- package/src/test/mock_checkpoint_builder.ts +323 -0
- package/src/test/utils.ts +218 -0
- package/dest/sequencer/block_builder.d.ts +0 -28
- package/dest/sequencer/block_builder.d.ts.map +0 -1
- package/dest/sequencer/block_builder.js +0 -134
- package/dest/sequencer/timetable.d.ts +0 -50
- package/dest/sequencer/timetable.d.ts.map +0 -1
- package/dest/sequencer/timetable.js +0 -137
- package/dest/tx_validator/nullifier_cache.d.ts +0 -14
- package/dest/tx_validator/nullifier_cache.d.ts.map +0 -1
- package/dest/tx_validator/nullifier_cache.js +0 -24
- package/dest/tx_validator/tx_validator_factory.d.ts +0 -18
- package/dest/tx_validator/tx_validator_factory.d.ts.map +0 -1
- package/dest/tx_validator/tx_validator_factory.js +0 -53
- package/src/sequencer/block_builder.ts +0 -222
- package/src/sequencer/timetable.ts +0 -188
- package/src/tx_validator/nullifier_cache.ts +0 -30
- package/src/tx_validator/tx_validator_factory.ts +0 -133
|
@@ -1,50 +1,104 @@
|
|
|
1
|
-
import {
|
|
1
|
+
import type { BlobClientInterface } from '@aztec/blob-client/client';
|
|
2
2
|
import { Blob, getBlobsPerL1Block, getPrefixedEthBlobCommitments } from '@aztec/blob-lib';
|
|
3
|
-
import { type BlobSinkClientInterface, createBlobSinkClient } from '@aztec/blob-sink/client';
|
|
4
3
|
import type { EpochCache } from '@aztec/epoch-cache';
|
|
4
|
+
import type { L1ContractsConfig } from '@aztec/ethereum/config';
|
|
5
5
|
import {
|
|
6
|
-
|
|
7
|
-
FormattedViemError,
|
|
6
|
+
FeeAssetPriceOracle,
|
|
8
7
|
type GovernanceProposerContract,
|
|
9
|
-
|
|
8
|
+
MULTI_CALL_3_ADDRESS,
|
|
9
|
+
Multicall3,
|
|
10
|
+
MulticallForwarderRevertedError,
|
|
11
|
+
type RollupContract,
|
|
12
|
+
type SimulationOverridesPlan,
|
|
13
|
+
type SlashingProposerContract,
|
|
14
|
+
buildSimulationOverridesStateOverride,
|
|
15
|
+
} from '@aztec/ethereum/contracts';
|
|
16
|
+
import { type L1FeeAnalysisResult, L1FeeAnalyzer, captureWindowBlockFees } from '@aztec/ethereum/l1-fee-analysis';
|
|
17
|
+
import {
|
|
10
18
|
type L1BlobInputs,
|
|
11
|
-
type L1ContractsConfig,
|
|
12
19
|
type L1TxConfig,
|
|
13
20
|
type L1TxRequest,
|
|
14
|
-
|
|
15
|
-
|
|
16
|
-
|
|
17
|
-
type TallySlashingProposerContract,
|
|
21
|
+
L1TxTimeoutError,
|
|
22
|
+
type L1TxUtils,
|
|
23
|
+
MAX_L1_TX_LIMIT,
|
|
18
24
|
type TransactionStats,
|
|
19
|
-
type ViemCommitteeAttestations,
|
|
20
|
-
type ViemHeader,
|
|
21
25
|
WEI_CONST,
|
|
26
|
+
} from '@aztec/ethereum/l1-tx-utils';
|
|
27
|
+
import {
|
|
28
|
+
FormattedViemError,
|
|
22
29
|
formatViemError,
|
|
30
|
+
mergeAbis,
|
|
31
|
+
tryDecodeRevertReason,
|
|
23
32
|
tryExtractEvent,
|
|
24
|
-
} from '@aztec/ethereum';
|
|
25
|
-
import
|
|
26
|
-
import {
|
|
27
|
-
import {
|
|
28
|
-
import {
|
|
33
|
+
} from '@aztec/ethereum/utils';
|
|
34
|
+
import { CheckpointNumber, SlotNumber } from '@aztec/foundation/branded-types';
|
|
35
|
+
import { trimmedBytesLength } from '@aztec/foundation/buffer';
|
|
36
|
+
import { pick } from '@aztec/foundation/collection';
|
|
37
|
+
import type { Fr } from '@aztec/foundation/curves/bn254';
|
|
38
|
+
import { TimeoutError } from '@aztec/foundation/error';
|
|
29
39
|
import { EthAddress } from '@aztec/foundation/eth-address';
|
|
30
|
-
import { Signature
|
|
31
|
-
import type { Fr } from '@aztec/foundation/fields';
|
|
40
|
+
import { Signature } from '@aztec/foundation/eth-signature';
|
|
32
41
|
import { type Logger, createLogger } from '@aztec/foundation/log';
|
|
42
|
+
import { InterruptibleSleep } from '@aztec/foundation/sleep';
|
|
33
43
|
import { bufferToHex } from '@aztec/foundation/string';
|
|
34
|
-
import { DateProvider, Timer } from '@aztec/foundation/timer';
|
|
35
|
-
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';
|
|
36
46
|
import { type ProposerSlashAction, encodeSlashConsensusVotes } from '@aztec/slasher';
|
|
37
|
-
import {
|
|
38
|
-
import {
|
|
47
|
+
import { CommitteeAttestationsAndSigners, type ValidateCheckpointResult } from '@aztec/stdlib/block';
|
|
48
|
+
import type { Checkpoint } from '@aztec/stdlib/checkpoint';
|
|
49
|
+
import {
|
|
50
|
+
getLastL1SlotTimestampForL2Slot,
|
|
51
|
+
getNextL1SlotTimestamp,
|
|
52
|
+
getTimestampForSlot,
|
|
53
|
+
} from '@aztec/stdlib/epoch-helpers';
|
|
39
54
|
import type { CheckpointHeader } from '@aztec/stdlib/rollup';
|
|
40
|
-
import type {
|
|
41
|
-
import { type TelemetryClient, getTelemetryClient } from '@aztec/telemetry-client';
|
|
55
|
+
import type { L1PublishCheckpointStats } from '@aztec/stdlib/stats';
|
|
56
|
+
import { type TelemetryClient, type Tracer, getTelemetryClient, trackSpan } from '@aztec/telemetry-client';
|
|
42
57
|
|
|
43
|
-
import {
|
|
44
|
-
|
|
45
|
-
|
|
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';
|
|
46
71
|
import { SequencerPublisherMetrics } from './sequencer-publisher-metrics.js';
|
|
47
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
|
+
|
|
48
102
|
/** Arguments to the process method of the rollup contract */
|
|
49
103
|
type L1ProcessArgs = {
|
|
50
104
|
/** The L2 block header. */
|
|
@@ -57,41 +111,49 @@ type L1ProcessArgs = {
|
|
|
57
111
|
attestationsAndSigners: CommitteeAttestationsAndSigners;
|
|
58
112
|
/** Attestations and signers signature */
|
|
59
113
|
attestationsAndSignersSignature: Signature;
|
|
114
|
+
/** The fee asset price modifier in basis points (from oracle) */
|
|
115
|
+
feeAssetPriceModifier: bigint;
|
|
60
116
|
};
|
|
61
117
|
|
|
62
118
|
export const Actions = [
|
|
63
119
|
'invalidate-by-invalid-attestation',
|
|
64
120
|
'invalidate-by-insufficient-attestations',
|
|
121
|
+
'prune',
|
|
65
122
|
'propose',
|
|
66
123
|
'governance-signal',
|
|
67
|
-
'empire-slashing-signal',
|
|
68
|
-
'create-empire-payload',
|
|
69
|
-
'execute-empire-payload',
|
|
70
124
|
'vote-offenses',
|
|
71
125
|
'execute-slash',
|
|
72
126
|
] as const;
|
|
73
127
|
|
|
74
128
|
export type Action = (typeof Actions)[number];
|
|
75
129
|
|
|
76
|
-
type GovernanceSignalAction = Extract<Action, 'governance-signal'
|
|
130
|
+
type GovernanceSignalAction = Extract<Action, 'governance-signal'>;
|
|
77
131
|
|
|
78
132
|
// Sorting for actions such that invalidations go before proposals, and proposals go before votes
|
|
79
133
|
export const compareActions = (a: Action, b: Action) => Actions.indexOf(a) - Actions.indexOf(b);
|
|
80
134
|
|
|
81
|
-
export type
|
|
135
|
+
export type InvalidateCheckpointRequest = {
|
|
82
136
|
request: L1TxRequest;
|
|
83
137
|
reason: 'invalid-attestation' | 'insufficient-attestations';
|
|
84
138
|
gasUsed: bigint;
|
|
85
|
-
|
|
86
|
-
|
|
139
|
+
checkpointNumber: CheckpointNumber;
|
|
140
|
+
forcePendingCheckpointNumber: CheckpointNumber;
|
|
141
|
+
/** Archive at the rollback target checkpoint (checkpoint N-1). */
|
|
142
|
+
lastArchive: Fr;
|
|
143
|
+
};
|
|
144
|
+
|
|
145
|
+
type EnqueueProposeCheckpointOpts = {
|
|
146
|
+
txTimeoutAt?: Date;
|
|
87
147
|
};
|
|
88
148
|
|
|
89
|
-
interface RequestWithExpiry {
|
|
149
|
+
export interface RequestWithExpiry {
|
|
90
150
|
action: Action;
|
|
91
151
|
request: L1TxRequest;
|
|
92
152
|
lastValidL2Slot: SlotNumber;
|
|
93
153
|
gasConfig?: Pick<L1TxConfig, 'txTimeoutAt' | 'gasLimit'>;
|
|
94
154
|
blobConfig?: L1BlobInputs;
|
|
155
|
+
/** Gas consumed by validateBlobs; stashed for the bundle simulate at send time. */
|
|
156
|
+
blobEvaluationGas?: bigint;
|
|
95
157
|
checkSuccess: (
|
|
96
158
|
request: L1TxRequest,
|
|
97
159
|
result?: { receipt: TransactionReceipt; stats?: TransactionStats; errorMsg?: string },
|
|
@@ -101,67 +163,92 @@ interface RequestWithExpiry {
|
|
|
101
163
|
export class SequencerPublisher {
|
|
102
164
|
private interrupted = false;
|
|
103
165
|
private metrics: SequencerPublisherMetrics;
|
|
166
|
+
private bundleSimulator: SequencerBundleSimulator;
|
|
104
167
|
public epochCache: EpochCache;
|
|
168
|
+
private failedTxStore?: Promise<L1TxFailedStore | undefined>;
|
|
105
169
|
|
|
106
|
-
|
|
107
|
-
|
|
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]);
|
|
108
176
|
|
|
109
177
|
protected lastActions: Partial<Record<Action, SlotNumber>> = {};
|
|
110
178
|
|
|
111
179
|
protected log: Logger;
|
|
112
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;
|
|
113
187
|
|
|
114
|
-
private
|
|
188
|
+
private blobClient: BlobClientInterface;
|
|
115
189
|
|
|
116
|
-
/**
|
|
117
|
-
private
|
|
118
|
-
// @note - with blobs, the below estimate seems too large.
|
|
119
|
-
// Total used for full block from int_l1_pub e2e test: 1m (of which 86k is 1x blob)
|
|
120
|
-
// Total used for emptier block from above test: 429k (of which 84k is 1x blob)
|
|
121
|
-
public static PROPOSE_GAS_GUESS: bigint = 12_000_000n;
|
|
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
|
-
|
|
124
|
-
|
|
193
|
+
/** L1 fee analyzer for fisherman mode */
|
|
194
|
+
private l1FeeAnalyzer?: L1FeeAnalyzer;
|
|
125
195
|
|
|
126
|
-
|
|
127
|
-
|
|
196
|
+
/** Fee asset price oracle for computing price modifiers from Uniswap V4 */
|
|
197
|
+
private feeAssetPriceOracle: FeeAssetPriceOracle;
|
|
128
198
|
|
|
129
|
-
|
|
199
|
+
/** Interruptible sleep used by sendRequestsAt to wait until a target timestamp. */
|
|
200
|
+
private readonly interruptibleSleep = new InterruptibleSleep();
|
|
201
|
+
|
|
202
|
+
public l1TxUtils: L1TxUtils;
|
|
130
203
|
public rollupContract: RollupContract;
|
|
131
204
|
public govProposerContract: GovernanceProposerContract;
|
|
132
|
-
public slashingProposerContract:
|
|
133
|
-
|
|
205
|
+
public slashingProposerContract: SlashingProposerContract | undefined;
|
|
206
|
+
|
|
207
|
+
public readonly tracer: Tracer;
|
|
134
208
|
|
|
135
209
|
protected requests: RequestWithExpiry[] = [];
|
|
136
210
|
|
|
137
211
|
constructor(
|
|
138
|
-
private config:
|
|
212
|
+
private config: Pick<
|
|
213
|
+
SequencerPublisherConfig,
|
|
214
|
+
| 'fishermanMode'
|
|
215
|
+
| 'l1TxFailedStore'
|
|
216
|
+
| 'sequencerPublisherPreviousL1BlockWaitTimeoutMs'
|
|
217
|
+
| 'sequencerPublisherPreviousL1BlockWaitPollIntervalMs'
|
|
218
|
+
> &
|
|
219
|
+
Pick<L1ContractsConfig, 'ethereumSlotDuration' | 'aztecSlotDuration'> & { l1ChainId: number },
|
|
139
220
|
deps: {
|
|
140
221
|
telemetry?: TelemetryClient;
|
|
141
|
-
|
|
142
|
-
l1TxUtils:
|
|
222
|
+
blobClient: BlobClientInterface;
|
|
223
|
+
l1TxUtils: L1TxUtils;
|
|
143
224
|
rollupContract: RollupContract;
|
|
144
|
-
slashingProposerContract:
|
|
225
|
+
slashingProposerContract: SlashingProposerContract | undefined;
|
|
145
226
|
governanceProposerContract: GovernanceProposerContract;
|
|
146
|
-
slashFactoryContract: SlashFactoryContract;
|
|
147
227
|
epochCache: EpochCache;
|
|
148
228
|
dateProvider: DateProvider;
|
|
149
229
|
metrics: SequencerPublisherMetrics;
|
|
150
230
|
lastActions: Partial<Record<Action, SlotNumber>>;
|
|
151
231
|
log?: Logger;
|
|
232
|
+
getNextPublisher?: (excludeAddresses: EthAddress[]) => Promise<L1TxUtils | undefined>;
|
|
152
233
|
},
|
|
153
234
|
) {
|
|
154
235
|
this.log = deps.log ?? createLogger('sequencer:publisher');
|
|
155
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;
|
|
156
241
|
this.epochCache = deps.epochCache;
|
|
157
242
|
this.lastActions = deps.lastActions;
|
|
158
243
|
|
|
159
|
-
this.
|
|
160
|
-
|
|
244
|
+
this.blobClient = deps.blobClient;
|
|
245
|
+
this.dateProvider = deps.dateProvider;
|
|
161
246
|
|
|
162
247
|
const telemetry = deps.telemetry ?? getTelemetryClient();
|
|
163
248
|
this.metrics = deps.metrics ?? new SequencerPublisherMetrics(telemetry, 'SequencerPublisher');
|
|
249
|
+
this.tracer = telemetry.getTracer('SequencerPublisher');
|
|
164
250
|
this.l1TxUtils = deps.l1TxUtils;
|
|
251
|
+
this.getNextPublisher = deps.getNextPublisher;
|
|
165
252
|
|
|
166
253
|
this.rollupContract = deps.rollupContract;
|
|
167
254
|
|
|
@@ -173,23 +260,152 @@ export class SequencerPublisher {
|
|
|
173
260
|
const newSlashingProposer = await this.rollupContract.getSlashingProposer();
|
|
174
261
|
this.slashingProposerContract = newSlashingProposer;
|
|
175
262
|
});
|
|
176
|
-
|
|
263
|
+
// Initialize L1 fee analyzer for fisherman mode
|
|
264
|
+
if (config.fishermanMode) {
|
|
265
|
+
this.l1FeeAnalyzer = new L1FeeAnalyzer(
|
|
266
|
+
this.l1TxUtils.client,
|
|
267
|
+
deps.dateProvider,
|
|
268
|
+
this.log.createChild('fee-analyzer'),
|
|
269
|
+
);
|
|
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()}`));
|
|
177
382
|
}
|
|
178
383
|
|
|
179
384
|
public getRollupContract(): RollupContract {
|
|
180
385
|
return this.rollupContract;
|
|
181
386
|
}
|
|
182
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
|
+
|
|
183
400
|
public getSenderAddress() {
|
|
184
401
|
return this.l1TxUtils.getSenderAddress();
|
|
185
402
|
}
|
|
186
403
|
|
|
187
404
|
/**
|
|
188
|
-
*
|
|
189
|
-
* @param proposerAddress - The actual proposer's address to use for balance lookups in simulations
|
|
405
|
+
* Gets the L1 fee analyzer instance (only available in fisherman mode)
|
|
190
406
|
*/
|
|
191
|
-
public
|
|
192
|
-
this.
|
|
407
|
+
public getL1FeeAnalyzer(): L1FeeAnalyzer | undefined {
|
|
408
|
+
return this.l1FeeAnalyzer;
|
|
193
409
|
}
|
|
194
410
|
|
|
195
411
|
public addRequest(request: RequestWithExpiry) {
|
|
@@ -197,7 +413,7 @@ export class SequencerPublisher {
|
|
|
197
413
|
}
|
|
198
414
|
|
|
199
415
|
public getCurrentL2Slot(): SlotNumber {
|
|
200
|
-
return this.epochCache.
|
|
416
|
+
return this.epochCache.getSlotNow();
|
|
201
417
|
}
|
|
202
418
|
|
|
203
419
|
/**
|
|
@@ -211,23 +427,84 @@ export class SequencerPublisher {
|
|
|
211
427
|
}
|
|
212
428
|
}
|
|
213
429
|
|
|
430
|
+
/**
|
|
431
|
+
* Analyzes L1 fees for the pending requests without sending them.
|
|
432
|
+
* This is used in fisherman mode to validate fee calculations.
|
|
433
|
+
* @param l2SlotNumber - The L2 slot number for this analysis
|
|
434
|
+
* @param onComplete - Optional callback to invoke when analysis completes (after block is mined)
|
|
435
|
+
* @returns The analysis result (incomplete until block mines), or undefined if no requests
|
|
436
|
+
*/
|
|
437
|
+
public async analyzeL1Fees(
|
|
438
|
+
l2SlotNumber: SlotNumber,
|
|
439
|
+
onComplete?: (analysis: L1FeeAnalysisResult) => void,
|
|
440
|
+
): Promise<L1FeeAnalysisResult | undefined> {
|
|
441
|
+
if (!this.l1FeeAnalyzer) {
|
|
442
|
+
this.log.warn('L1 fee analyzer not available (not in fisherman mode)');
|
|
443
|
+
return undefined;
|
|
444
|
+
}
|
|
445
|
+
|
|
446
|
+
const requestsToAnalyze = [...this.requests];
|
|
447
|
+
if (requestsToAnalyze.length === 0) {
|
|
448
|
+
this.log.debug('No requests to analyze for L1 fees');
|
|
449
|
+
return undefined;
|
|
450
|
+
}
|
|
451
|
+
|
|
452
|
+
// Extract blob config from requests (if any)
|
|
453
|
+
const blobConfigs = requestsToAnalyze.filter(request => request.blobConfig).map(request => request.blobConfig);
|
|
454
|
+
const blobConfig = blobConfigs[0];
|
|
455
|
+
|
|
456
|
+
// Get gas configs
|
|
457
|
+
const gasConfigs = requestsToAnalyze.filter(request => request.gasConfig).map(request => request.gasConfig);
|
|
458
|
+
const gasLimits = gasConfigs.map(g => g?.gasLimit).filter((g): g is bigint => g !== undefined);
|
|
459
|
+
const gasLimit = gasLimits.length > 0 ? gasLimits.reduce((sum, g) => sum + g, 0n) : 0n;
|
|
460
|
+
|
|
461
|
+
// Get the transaction requests
|
|
462
|
+
const l1Requests = requestsToAnalyze.map(r => r.request);
|
|
463
|
+
|
|
464
|
+
// Start the analysis
|
|
465
|
+
const analysisId = await this.l1FeeAnalyzer.startAnalysis(
|
|
466
|
+
l2SlotNumber,
|
|
467
|
+
gasLimit > 0n ? gasLimit : MAX_L1_TX_LIMIT,
|
|
468
|
+
l1Requests,
|
|
469
|
+
blobConfig,
|
|
470
|
+
onComplete,
|
|
471
|
+
);
|
|
472
|
+
|
|
473
|
+
this.log.info('Started L1 fee analysis', {
|
|
474
|
+
analysisId,
|
|
475
|
+
l2SlotNumber: l2SlotNumber.toString(),
|
|
476
|
+
requestCount: requestsToAnalyze.length,
|
|
477
|
+
hasBlobConfig: !!blobConfig,
|
|
478
|
+
gasLimit: gasLimit.toString(),
|
|
479
|
+
actions: requestsToAnalyze.map(r => r.action),
|
|
480
|
+
});
|
|
481
|
+
|
|
482
|
+
// Return the analysis result (will be incomplete until block mines)
|
|
483
|
+
return this.l1FeeAnalyzer.getAnalysis(analysisId);
|
|
484
|
+
}
|
|
485
|
+
|
|
214
486
|
/**
|
|
215
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.
|
|
216
492
|
* @returns one of:
|
|
217
493
|
* - A receipt and stats if the tx succeeded
|
|
218
494
|
* - a receipt and errorMsg if it failed on L1
|
|
219
495
|
* - undefined if no valid requests are found OR the tx failed to send.
|
|
220
496
|
*/
|
|
221
|
-
|
|
497
|
+
@trackSpan('SequencerPublisher.sendRequests')
|
|
498
|
+
public async sendRequests(targetSlot?: SlotNumber): Promise<SendRequestsResult | undefined> {
|
|
222
499
|
const requestsToProcess = [...this.requests];
|
|
223
500
|
this.requests = [];
|
|
224
|
-
|
|
501
|
+
|
|
502
|
+
if (this.interrupted || requestsToProcess.length === 0) {
|
|
225
503
|
return undefined;
|
|
226
504
|
}
|
|
227
|
-
const currentL2Slot = this.getCurrentL2Slot();
|
|
505
|
+
const currentL2Slot = targetSlot ?? this.getCurrentL2Slot();
|
|
228
506
|
this.log.debug(`Sending requests on L2 slot ${currentL2Slot}`);
|
|
229
507
|
const validRequests = requestsToProcess.filter(request => request.lastValidL2Slot >= currentL2Slot);
|
|
230
|
-
const validActions = validRequests.map(x => x.action);
|
|
231
508
|
const expiredActions = requestsToProcess
|
|
232
509
|
.filter(request => request.lastValidL2Slot < currentL2Slot)
|
|
233
510
|
.map(x => x.action);
|
|
@@ -250,47 +527,101 @@ export class SequencerPublisher {
|
|
|
250
527
|
return undefined;
|
|
251
528
|
}
|
|
252
529
|
|
|
253
|
-
//
|
|
254
|
-
|
|
255
|
-
// See https://github.com/AztecProtocol/aztec-packages/issues/11513
|
|
256
|
-
const gasConfigs = requestsToProcess.filter(request => request.gasConfig).map(request => request.gasConfig);
|
|
257
|
-
const blobConfigs = requestsToProcess.filter(request => request.blobConfig).map(request => request.blobConfig);
|
|
258
|
-
|
|
259
|
-
if (blobConfigs.length > 1) {
|
|
260
|
-
throw new Error('Multiple blob configs found');
|
|
261
|
-
}
|
|
262
|
-
|
|
263
|
-
const blobConfig = blobConfigs[0];
|
|
264
|
-
|
|
265
|
-
// Merge gasConfigs. Yields the sum of gasLimits, and the earliest txTimeoutAt, or undefined if no gasConfig sets them.
|
|
266
|
-
const gasLimits = gasConfigs.map(g => g?.gasLimit).filter((g): g is bigint => g !== undefined);
|
|
267
|
-
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);
|
|
268
532
|
const txTimeoutAts = gasConfigs.map(g => g?.txTimeoutAt).filter((g): g is Date => g !== undefined);
|
|
269
|
-
const txTimeoutAt = txTimeoutAts.length > 0 ? new Date(Math.min(...txTimeoutAts.map(g => g.getTime()))) : undefined;
|
|
270
|
-
const txConfig: RequestWithExpiry['gasConfig'] = { gasLimit, txTimeoutAt };
|
|
533
|
+
const txTimeoutAt = txTimeoutAts.length > 0 ? new Date(Math.min(...txTimeoutAts.map(g => g.getTime()))) : undefined;
|
|
271
534
|
|
|
272
535
|
// Sort the requests so that proposals always go first
|
|
273
536
|
// This ensures the committee gets precomputed correctly
|
|
274
537
|
validRequests.sort((a, b) => compareActions(a.action, b.action));
|
|
275
538
|
|
|
276
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
|
+
|
|
277
567
|
this.log.debug('Forwarding transactions', {
|
|
278
|
-
|
|
568
|
+
requests: requests.map(request => request.action),
|
|
279
569
|
txConfig,
|
|
280
570
|
});
|
|
281
|
-
const result = await
|
|
282
|
-
|
|
283
|
-
|
|
284
|
-
|
|
285
|
-
|
|
286
|
-
|
|
287
|
-
|
|
288
|
-
|
|
289
|
-
|
|
290
|
-
|
|
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
|
+
};
|
|
291
584
|
} catch (err) {
|
|
292
585
|
const viemError = formatViemError(err);
|
|
293
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
|
+
}
|
|
294
625
|
return undefined;
|
|
295
626
|
} finally {
|
|
296
627
|
try {
|
|
@@ -304,52 +635,329 @@ export class SequencerPublisher {
|
|
|
304
635
|
}
|
|
305
636
|
}
|
|
306
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
|
+
|
|
307
908
|
private callbackBundledTransactions(
|
|
308
909
|
requests: RequestWithExpiry[],
|
|
309
|
-
result
|
|
910
|
+
result: { receipt: TransactionReceipt; multicallData: Hex },
|
|
310
911
|
) {
|
|
311
912
|
const actionsListStr = requests.map(r => r.action).join(', ');
|
|
312
|
-
|
|
313
|
-
|
|
314
|
-
|
|
315
|
-
|
|
316
|
-
|
|
317
|
-
|
|
318
|
-
|
|
319
|
-
|
|
320
|
-
|
|
321
|
-
|
|
322
|
-
|
|
323
|
-
|
|
324
|
-
|
|
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);
|
|
325
930
|
}
|
|
326
|
-
return { successfulActions, failedActions };
|
|
327
931
|
}
|
|
932
|
+
return { successfulActions, failedActions };
|
|
328
933
|
}
|
|
329
934
|
|
|
330
935
|
/**
|
|
331
|
-
* @notice Will call `
|
|
936
|
+
* @notice Will call `canProposeAt` to make sure that it is possible to propose
|
|
332
937
|
* @param tipArchive - The archive to check
|
|
333
938
|
* @returns The slot and block number if it is possible to propose, undefined otherwise
|
|
334
939
|
*/
|
|
335
|
-
public
|
|
336
|
-
tipArchive: Fr,
|
|
337
|
-
msgSender: EthAddress,
|
|
338
|
-
opts: { forcePendingBlockNumber?: BlockNumber } = {},
|
|
339
|
-
) {
|
|
940
|
+
public async canProposeAt(tipArchive: Fr, msgSender: EthAddress, simulationOverridesPlan?: SimulationOverridesPlan) {
|
|
340
941
|
// TODO: #14291 - should loop through multiple keys to check if any of them can propose
|
|
341
|
-
|
|
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;
|
|
342
950
|
|
|
343
951
|
return this.rollupContract
|
|
344
|
-
.
|
|
345
|
-
|
|
346
|
-
|
|
347
|
-
|
|
348
|
-
|
|
349
|
-
|
|
952
|
+
.canProposeAt(
|
|
953
|
+
tipArchive.toBuffer(),
|
|
954
|
+
msgSender.toString(),
|
|
955
|
+
nextL1SlotTs,
|
|
956
|
+
await buildSimulationOverridesStateOverride(this.rollupContract, simulationOverridesPlan),
|
|
957
|
+
)
|
|
350
958
|
.catch(err => {
|
|
351
|
-
if (err instanceof FormattedViemError &&
|
|
352
|
-
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))}`, {
|
|
353
961
|
error: err.message,
|
|
354
962
|
});
|
|
355
963
|
} else {
|
|
@@ -358,36 +966,33 @@ export class SequencerPublisher {
|
|
|
358
966
|
return undefined;
|
|
359
967
|
});
|
|
360
968
|
}
|
|
969
|
+
|
|
361
970
|
/**
|
|
362
971
|
* @notice Will simulate `validateHeader` to make sure that the block header is valid
|
|
363
972
|
* @dev This is a convenience function that can be used by the sequencer to validate a "partial" header.
|
|
364
973
|
* It will throw if the block header is invalid.
|
|
365
974
|
* @param header - The block header to validate
|
|
366
975
|
*/
|
|
976
|
+
@trackSpan('SequencerPublisher.validateBlockHeader')
|
|
367
977
|
public async validateBlockHeader(
|
|
368
978
|
header: CheckpointHeader,
|
|
369
|
-
|
|
370
|
-
) {
|
|
979
|
+
simulationOverridesPlan?: SimulationOverridesPlan,
|
|
980
|
+
): Promise<void> {
|
|
371
981
|
const flags = { ignoreDA: true, ignoreSignatures: true };
|
|
372
982
|
|
|
373
983
|
const args = [
|
|
374
984
|
header.toViem(),
|
|
375
|
-
CommitteeAttestationsAndSigners.
|
|
985
|
+
CommitteeAttestationsAndSigners.packAttestations([]),
|
|
376
986
|
[], // no signers
|
|
377
987
|
Signature.empty().toViemSignature(),
|
|
378
988
|
`0x${'0'.repeat(64)}`, // 32 empty bytes
|
|
379
|
-
header.
|
|
989
|
+
header.blobsHash.toString(),
|
|
380
990
|
flags,
|
|
381
991
|
] as const;
|
|
382
992
|
|
|
383
|
-
const
|
|
384
|
-
const
|
|
385
|
-
|
|
386
|
-
? CheckpointNumber.fromBlockNumber(opts.forcePendingBlockNumber)
|
|
387
|
-
: undefined;
|
|
388
|
-
const stateOverrides = await this.rollupContract.makePendingCheckpointNumberOverride(
|
|
389
|
-
optsForcePendingCheckpointNumber,
|
|
390
|
-
);
|
|
993
|
+
const l1Constants = this.epochCache.getL1Constants();
|
|
994
|
+
const ts = getLastL1SlotTimestampForL2Slot(header.slotNumber, l1Constants);
|
|
995
|
+
const stateOverrides = await buildSimulationOverridesStateOverride(this.rollupContract, simulationOverridesPlan);
|
|
391
996
|
let balance = 0n;
|
|
392
997
|
if (this.config.fishermanMode) {
|
|
393
998
|
// In fisherman mode, we can't know where the proposer is publishing from
|
|
@@ -407,99 +1012,138 @@ export class SequencerPublisher {
|
|
|
407
1012
|
data: encodeFunctionData({ abi: RollupAbi, functionName: 'validateHeaderWithAttestations', args }),
|
|
408
1013
|
from: MULTI_CALL_3_ADDRESS,
|
|
409
1014
|
},
|
|
410
|
-
{ time: ts
|
|
1015
|
+
{ time: ts },
|
|
411
1016
|
stateOverrides,
|
|
412
1017
|
);
|
|
413
1018
|
this.log.debug(`Simulated validateHeader`);
|
|
414
1019
|
}
|
|
415
1020
|
|
|
416
1021
|
/**
|
|
417
|
-
* Simulate making a call to invalidate a
|
|
418
|
-
* @param
|
|
1022
|
+
* Simulate making a call to invalidate a checkpoint with invalid attestations. Returns undefined if no need to invalidate.
|
|
1023
|
+
* @param validationResult - The validation result indicating which checkpoint to invalidate (as returned by the archiver)
|
|
419
1024
|
*/
|
|
420
|
-
public async
|
|
421
|
-
validationResult:
|
|
422
|
-
): Promise<
|
|
1025
|
+
public async simulateInvalidateCheckpoint(
|
|
1026
|
+
validationResult: ValidateCheckpointResult,
|
|
1027
|
+
): Promise<InvalidateCheckpointRequest | undefined> {
|
|
423
1028
|
if (validationResult.valid) {
|
|
424
1029
|
return undefined;
|
|
425
1030
|
}
|
|
426
1031
|
|
|
427
|
-
const { reason,
|
|
428
|
-
const
|
|
429
|
-
const logData = { ...
|
|
1032
|
+
const { reason, checkpoint } = validationResult;
|
|
1033
|
+
const checkpointNumber = checkpoint.checkpointNumber;
|
|
1034
|
+
const logData = { ...checkpoint, reason };
|
|
430
1035
|
|
|
431
|
-
const
|
|
432
|
-
if (
|
|
1036
|
+
const currentCheckpointNumber = await this.rollupContract.getCheckpointNumber();
|
|
1037
|
+
if (currentCheckpointNumber < checkpointNumber) {
|
|
433
1038
|
this.log.verbose(
|
|
434
|
-
`Skipping
|
|
435
|
-
{
|
|
1039
|
+
`Skipping checkpoint ${checkpointNumber} invalidation since it has already been removed from the pending chain`,
|
|
1040
|
+
{ currentCheckpointNumber, ...logData },
|
|
436
1041
|
);
|
|
437
1042
|
return undefined;
|
|
438
1043
|
}
|
|
439
1044
|
|
|
440
|
-
const request = this.
|
|
441
|
-
this.log.debug(`Simulating invalidate
|
|
1045
|
+
const request = this.buildInvalidateCheckpointRequest(validationResult);
|
|
1046
|
+
this.log.debug(`Simulating invalidate checkpoint ${checkpointNumber}`, { ...logData, request });
|
|
1047
|
+
|
|
1048
|
+
const l1BlockNumber = await this.l1TxUtils.getBlockNumber();
|
|
442
1049
|
|
|
443
1050
|
try {
|
|
444
|
-
const { gasUsed } = await this.l1TxUtils.simulate(
|
|
445
|
-
|
|
1051
|
+
const { gasUsed } = await this.l1TxUtils.simulate(
|
|
1052
|
+
request,
|
|
1053
|
+
undefined,
|
|
1054
|
+
undefined,
|
|
1055
|
+
mergeAbis([request.abi ?? [], ErrorsAbi]),
|
|
1056
|
+
);
|
|
1057
|
+
this.log.verbose(`Simulation for invalidate checkpoint ${checkpointNumber} succeeded`, {
|
|
1058
|
+
...logData,
|
|
1059
|
+
request,
|
|
1060
|
+
gasUsed,
|
|
1061
|
+
});
|
|
446
1062
|
|
|
447
|
-
return {
|
|
1063
|
+
return {
|
|
1064
|
+
request,
|
|
1065
|
+
gasUsed,
|
|
1066
|
+
checkpointNumber,
|
|
1067
|
+
forcePendingCheckpointNumber: CheckpointNumber(checkpointNumber - 1),
|
|
1068
|
+
lastArchive: validationResult.checkpoint.lastArchive,
|
|
1069
|
+
reason,
|
|
1070
|
+
};
|
|
448
1071
|
} catch (err) {
|
|
449
1072
|
const viemError = formatViemError(err);
|
|
450
1073
|
|
|
451
|
-
// If the error is due to the
|
|
452
|
-
// we can safely ignore it and return undefined so we go ahead with
|
|
453
|
-
if (viemError.message?.includes('
|
|
1074
|
+
// If the error is due to the checkpoint not being in the pending chain, and it was indeed removed by someone else,
|
|
1075
|
+
// we can safely ignore it and return undefined so we go ahead with checkpoint building.
|
|
1076
|
+
if (viemError.message?.includes('Rollup__CheckpointNotInPendingChain')) {
|
|
454
1077
|
this.log.verbose(
|
|
455
|
-
`Simulation for invalidate
|
|
1078
|
+
`Simulation for invalidate checkpoint ${checkpointNumber} failed due to checkpoint not being in pending chain`,
|
|
456
1079
|
{ ...logData, request, error: viemError.message },
|
|
457
1080
|
);
|
|
458
|
-
const
|
|
459
|
-
if (
|
|
460
|
-
this.log.verbose(`
|
|
1081
|
+
const latestProposedCheckpointNumber = await this.rollupContract.getCheckpointNumber();
|
|
1082
|
+
if (latestProposedCheckpointNumber < checkpointNumber) {
|
|
1083
|
+
this.log.verbose(`Checkpoint ${checkpointNumber} has already been invalidated`, { ...logData });
|
|
461
1084
|
return undefined;
|
|
462
1085
|
} else {
|
|
463
1086
|
this.log.error(
|
|
464
|
-
`Simulation for invalidate ${
|
|
1087
|
+
`Simulation for invalidate checkpoint ${checkpointNumber} failed and it is still in pending chain`,
|
|
465
1088
|
viemError,
|
|
466
1089
|
logData,
|
|
467
1090
|
);
|
|
468
|
-
throw new Error(
|
|
469
|
-
|
|
470
|
-
|
|
1091
|
+
throw new Error(
|
|
1092
|
+
`Failed to simulate invalidate checkpoint ${checkpointNumber} while it is still in pending chain`,
|
|
1093
|
+
{
|
|
1094
|
+
cause: viemError,
|
|
1095
|
+
},
|
|
1096
|
+
);
|
|
471
1097
|
}
|
|
472
1098
|
}
|
|
473
1099
|
|
|
474
|
-
// Otherwise, throw. We cannot build the next
|
|
475
|
-
this.log.error(`Simulation for invalidate
|
|
476
|
-
|
|
1100
|
+
// Otherwise, throw. We cannot build the next checkpoint if we cannot invalidate the previous one.
|
|
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
|
+
);
|
|
1118
|
+
throw new Error(`Failed to simulate invalidate checkpoint ${checkpointNumber}`, { cause: viemError });
|
|
477
1119
|
}
|
|
478
1120
|
}
|
|
479
1121
|
|
|
480
|
-
private
|
|
1122
|
+
private buildInvalidateCheckpointRequest(validationResult: ValidateCheckpointResult) {
|
|
481
1123
|
if (validationResult.valid) {
|
|
482
|
-
throw new Error('Cannot invalidate a valid
|
|
1124
|
+
throw new Error('Cannot invalidate a valid checkpoint');
|
|
483
1125
|
}
|
|
484
1126
|
|
|
485
|
-
const {
|
|
486
|
-
const logData = { ...
|
|
487
|
-
this.log.debug(`
|
|
1127
|
+
const { checkpoint, committee, reason } = validationResult;
|
|
1128
|
+
const logData = { ...checkpoint, reason };
|
|
1129
|
+
this.log.debug(`Building invalidate checkpoint ${checkpoint.checkpointNumber} request`, logData);
|
|
488
1130
|
|
|
489
|
-
|
|
490
|
-
|
|
491
|
-
)
|
|
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;
|
|
492
1136
|
|
|
493
1137
|
if (reason === 'invalid-attestation') {
|
|
494
1138
|
return this.rollupContract.buildInvalidateBadAttestationRequest(
|
|
495
|
-
|
|
1139
|
+
checkpoint.checkpointNumber,
|
|
496
1140
|
attestationsAndSigners,
|
|
497
1141
|
committee,
|
|
498
1142
|
validationResult.invalidIndex,
|
|
499
1143
|
);
|
|
500
1144
|
} else if (reason === 'insufficient-attestations') {
|
|
501
1145
|
return this.rollupContract.buildInvalidateInsufficientAttestationsRequest(
|
|
502
|
-
|
|
1146
|
+
checkpoint.checkpointNumber,
|
|
503
1147
|
attestationsAndSigners,
|
|
504
1148
|
committee,
|
|
505
1149
|
);
|
|
@@ -509,65 +1153,11 @@ export class SequencerPublisher {
|
|
|
509
1153
|
}
|
|
510
1154
|
}
|
|
511
1155
|
|
|
512
|
-
/**
|
|
513
|
-
* @notice Will simulate `propose` to make sure that the block is valid for submission
|
|
514
|
-
*
|
|
515
|
-
* @dev Throws if unable to propose
|
|
516
|
-
*
|
|
517
|
-
* @param block - The block to propose
|
|
518
|
-
* @param attestationData - The block's attestation data
|
|
519
|
-
*
|
|
520
|
-
*/
|
|
521
|
-
public async validateBlockForSubmission(
|
|
522
|
-
block: L2Block,
|
|
523
|
-
attestationsAndSigners: CommitteeAttestationsAndSigners,
|
|
524
|
-
attestationsAndSignersSignature: Signature,
|
|
525
|
-
options: { forcePendingBlockNumber?: BlockNumber },
|
|
526
|
-
): Promise<bigint> {
|
|
527
|
-
const ts = BigInt((await this.l1TxUtils.getBlock()).timestamp + this.ethereumSlotDuration);
|
|
528
|
-
|
|
529
|
-
// If we have no attestations, we still need to provide the empty attestations
|
|
530
|
-
// so that the committee is recalculated correctly
|
|
531
|
-
const ignoreSignatures = attestationsAndSigners.attestations.length === 0;
|
|
532
|
-
if (ignoreSignatures) {
|
|
533
|
-
const { committee } = await this.epochCache.getCommittee(block.header.globalVariables.slotNumber);
|
|
534
|
-
if (!committee) {
|
|
535
|
-
this.log.warn(`No committee found for slot ${block.header.globalVariables.slotNumber}`);
|
|
536
|
-
throw new Error(`No committee found for slot ${block.header.globalVariables.slotNumber}`);
|
|
537
|
-
}
|
|
538
|
-
attestationsAndSigners.attestations = committee.map(committeeMember =>
|
|
539
|
-
CommitteeAttestation.fromAddress(committeeMember),
|
|
540
|
-
);
|
|
541
|
-
}
|
|
542
|
-
|
|
543
|
-
const blobFields = block.getCheckpointBlobFields();
|
|
544
|
-
const blobs = getBlobsPerL1Block(blobFields);
|
|
545
|
-
const blobInput = getPrefixedEthBlobCommitments(blobs);
|
|
546
|
-
|
|
547
|
-
const args = [
|
|
548
|
-
{
|
|
549
|
-
header: block.getCheckpointHeader().toViem(),
|
|
550
|
-
archive: toHex(block.archive.root.toBuffer()),
|
|
551
|
-
oracleInput: {
|
|
552
|
-
feeAssetPriceModifier: 0n,
|
|
553
|
-
},
|
|
554
|
-
},
|
|
555
|
-
attestationsAndSigners.getPackedAttestations(),
|
|
556
|
-
attestationsAndSigners.getSigners().map(signer => signer.toString()),
|
|
557
|
-
attestationsAndSignersSignature.toViemSignature(),
|
|
558
|
-
blobInput,
|
|
559
|
-
] as const;
|
|
560
|
-
|
|
561
|
-
await this.simulateProposeTx(args, ts, options);
|
|
562
|
-
return ts;
|
|
563
|
-
}
|
|
564
|
-
|
|
565
1156
|
private async enqueueCastSignalHelper(
|
|
566
1157
|
slotNumber: SlotNumber,
|
|
567
|
-
timestamp: bigint,
|
|
568
1158
|
signalType: GovernanceSignalAction,
|
|
569
1159
|
payload: EthAddress,
|
|
570
|
-
base:
|
|
1160
|
+
base: GovernanceProposerContract,
|
|
571
1161
|
signerAddress: EthAddress,
|
|
572
1162
|
signer: (msg: TypedDataDefinition) => Promise<`0x${string}`>,
|
|
573
1163
|
): Promise<boolean> {
|
|
@@ -585,10 +1175,39 @@ export class SequencerPublisher {
|
|
|
585
1175
|
const round = await base.computeRound(slotNumber);
|
|
586
1176
|
const roundInfo = await base.getRoundInfo(this.rollupContract.address, round);
|
|
587
1177
|
|
|
1178
|
+
if (roundInfo.quorumReached) {
|
|
1179
|
+
return false;
|
|
1180
|
+
}
|
|
1181
|
+
|
|
588
1182
|
if (roundInfo.lastSignalSlot >= slotNumber) {
|
|
589
1183
|
return false;
|
|
590
1184
|
}
|
|
591
1185
|
|
|
1186
|
+
if (await base.isPayloadEmpty(payload)) {
|
|
1187
|
+
this.log.warn(`Skipping vote cast for payload with empty code`);
|
|
1188
|
+
return false;
|
|
1189
|
+
}
|
|
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
|
+
|
|
592
1211
|
const cachedLastVote = this.lastActions[signalType];
|
|
593
1212
|
this.lastActions[signalType] = slotNumber;
|
|
594
1213
|
const action = signalType;
|
|
@@ -607,38 +1226,31 @@ export class SequencerPublisher {
|
|
|
607
1226
|
lastValidL2Slot: slotNumber,
|
|
608
1227
|
});
|
|
609
1228
|
|
|
610
|
-
try {
|
|
611
|
-
await this.l1TxUtils.simulate(request, { time: timestamp }, [], ErrorsAbi);
|
|
612
|
-
this.log.debug(`Simulation for ${action} at slot ${slotNumber} succeeded`, { request });
|
|
613
|
-
} catch (err) {
|
|
614
|
-
this.log.error(`Failed simulation for ${action} at slot ${slotNumber} (enqueuing the action anyway)`, err);
|
|
615
|
-
// Yes, we enqueue the request anyway, in case there was a bug with the simulation itself
|
|
616
|
-
}
|
|
617
|
-
|
|
618
1229
|
// TODO(palla/slash): All votes (governance and slashing) should txTimeoutAt at the end of the slot.
|
|
619
1230
|
this.addRequest({
|
|
620
|
-
gasConfig: { gasLimit: SequencerPublisher.VOTE_GAS_GUESS },
|
|
621
1231
|
action,
|
|
622
1232
|
request,
|
|
623
1233
|
lastValidL2Slot: slotNumber,
|
|
624
1234
|
checkSuccess: (_request, result) => {
|
|
625
1235
|
const success =
|
|
626
1236
|
result &&
|
|
627
|
-
result.receipt
|
|
628
|
-
|
|
629
|
-
|
|
1237
|
+
extractEventSuccess(result.receipt, {
|
|
1238
|
+
address: base.address.toString(),
|
|
1239
|
+
abi: EmpireBaseAbi,
|
|
1240
|
+
eventName: 'SignalCast',
|
|
1241
|
+
});
|
|
630
1242
|
|
|
631
1243
|
const logData = { ...result, slotNumber, round, payload: payload.toString() };
|
|
632
1244
|
if (!success) {
|
|
633
1245
|
this.log.error(
|
|
634
|
-
`Signaling in
|
|
1246
|
+
`Signaling in ${action} for ${payload} at slot ${slotNumber} in round ${round} failed`,
|
|
635
1247
|
logData,
|
|
636
1248
|
);
|
|
637
1249
|
this.lastActions[signalType] = cachedLastVote;
|
|
638
1250
|
return false;
|
|
639
1251
|
} else {
|
|
640
1252
|
this.log.info(
|
|
641
|
-
`Signaling in
|
|
1253
|
+
`Signaling in ${action} for ${payload} at slot ${slotNumber} in round ${round} succeeded`,
|
|
642
1254
|
logData,
|
|
643
1255
|
);
|
|
644
1256
|
return true;
|
|
@@ -651,19 +1263,16 @@ export class SequencerPublisher {
|
|
|
651
1263
|
/**
|
|
652
1264
|
* Enqueues a governance castSignal transaction to cast a signal for a given slot number.
|
|
653
1265
|
* @param slotNumber - The slot number to cast a signal for.
|
|
654
|
-
* @param timestamp - The timestamp of the slot to cast a signal for.
|
|
655
1266
|
* @returns True if the signal was successfully enqueued, false otherwise.
|
|
656
1267
|
*/
|
|
657
1268
|
public enqueueGovernanceCastSignal(
|
|
658
1269
|
governancePayload: EthAddress,
|
|
659
1270
|
slotNumber: SlotNumber,
|
|
660
|
-
timestamp: bigint,
|
|
661
1271
|
signerAddress: EthAddress,
|
|
662
1272
|
signer: (msg: TypedDataDefinition) => Promise<`0x${string}`>,
|
|
663
1273
|
): Promise<boolean> {
|
|
664
1274
|
return this.enqueueCastSignalHelper(
|
|
665
1275
|
slotNumber,
|
|
666
|
-
timestamp,
|
|
667
1276
|
'governance-signal',
|
|
668
1277
|
governancePayload,
|
|
669
1278
|
this.govProposerContract,
|
|
@@ -672,11 +1281,51 @@ export class SequencerPublisher {
|
|
|
672
1281
|
);
|
|
673
1282
|
}
|
|
674
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
|
+
|
|
675
1325
|
/** Enqueues all slashing actions as returned by the slasher client. */
|
|
676
1326
|
public async enqueueSlashingActions(
|
|
677
1327
|
actions: ProposerSlashAction[],
|
|
678
1328
|
slotNumber: SlotNumber,
|
|
679
|
-
timestamp: bigint,
|
|
680
1329
|
signerAddress: EthAddress,
|
|
681
1330
|
signer: (msg: TypedDataDefinition) => Promise<`0x${string}`>,
|
|
682
1331
|
): Promise<boolean> {
|
|
@@ -687,58 +1336,6 @@ export class SequencerPublisher {
|
|
|
687
1336
|
|
|
688
1337
|
for (const action of actions) {
|
|
689
1338
|
switch (action.type) {
|
|
690
|
-
case 'vote-empire-payload': {
|
|
691
|
-
if (this.slashingProposerContract?.type !== 'empire') {
|
|
692
|
-
this.log.error('Cannot vote for empire payload on non-empire slashing contract');
|
|
693
|
-
break;
|
|
694
|
-
}
|
|
695
|
-
this.log.debug(`Enqueuing slashing vote for payload ${action.payload} at slot ${slotNumber}`, {
|
|
696
|
-
signerAddress,
|
|
697
|
-
});
|
|
698
|
-
await this.enqueueCastSignalHelper(
|
|
699
|
-
slotNumber,
|
|
700
|
-
timestamp,
|
|
701
|
-
'empire-slashing-signal',
|
|
702
|
-
action.payload,
|
|
703
|
-
this.slashingProposerContract,
|
|
704
|
-
signerAddress,
|
|
705
|
-
signer,
|
|
706
|
-
);
|
|
707
|
-
break;
|
|
708
|
-
}
|
|
709
|
-
|
|
710
|
-
case 'create-empire-payload': {
|
|
711
|
-
this.log.debug(`Enqueuing slashing create payload at slot ${slotNumber}`, { slotNumber, signerAddress });
|
|
712
|
-
const request = this.slashFactoryContract.buildCreatePayloadRequest(action.data);
|
|
713
|
-
await this.simulateAndEnqueueRequest(
|
|
714
|
-
'create-empire-payload',
|
|
715
|
-
request,
|
|
716
|
-
(receipt: TransactionReceipt) =>
|
|
717
|
-
!!this.slashFactoryContract.tryExtractSlashPayloadCreatedEvent(receipt.logs),
|
|
718
|
-
slotNumber,
|
|
719
|
-
timestamp,
|
|
720
|
-
);
|
|
721
|
-
break;
|
|
722
|
-
}
|
|
723
|
-
|
|
724
|
-
case 'execute-empire-payload': {
|
|
725
|
-
this.log.debug(`Enqueuing slashing execute payload at slot ${slotNumber}`, { slotNumber, signerAddress });
|
|
726
|
-
if (this.slashingProposerContract?.type !== 'empire') {
|
|
727
|
-
this.log.error('Cannot execute slashing payload on non-empire slashing contract');
|
|
728
|
-
return false;
|
|
729
|
-
}
|
|
730
|
-
const empireSlashingProposer = this.slashingProposerContract as EmpireSlashingProposerContract;
|
|
731
|
-
const request = empireSlashingProposer.buildExecuteRoundRequest(action.round);
|
|
732
|
-
await this.simulateAndEnqueueRequest(
|
|
733
|
-
'execute-empire-payload',
|
|
734
|
-
request,
|
|
735
|
-
(receipt: TransactionReceipt) => !!empireSlashingProposer.tryExtractPayloadSubmittedEvent(receipt.logs),
|
|
736
|
-
slotNumber,
|
|
737
|
-
timestamp,
|
|
738
|
-
);
|
|
739
|
-
break;
|
|
740
|
-
}
|
|
741
|
-
|
|
742
1339
|
case 'vote-offenses': {
|
|
743
1340
|
this.log.debug(`Enqueuing slashing vote for ${action.votes.length} votes at slot ${slotNumber}`, {
|
|
744
1341
|
slotNumber,
|
|
@@ -746,19 +1343,21 @@ export class SequencerPublisher {
|
|
|
746
1343
|
votesCount: action.votes.length,
|
|
747
1344
|
signerAddress,
|
|
748
1345
|
});
|
|
749
|
-
if (this.slashingProposerContract
|
|
750
|
-
this.log.error('
|
|
1346
|
+
if (!this.slashingProposerContract) {
|
|
1347
|
+
this.log.error('No slashing proposer contract available');
|
|
751
1348
|
return false;
|
|
752
1349
|
}
|
|
753
|
-
const tallySlashingProposer = this.slashingProposerContract as TallySlashingProposerContract;
|
|
754
1350
|
const votes = bufferToHex(encodeSlashConsensusVotes(action.votes));
|
|
755
|
-
const request = await
|
|
756
|
-
|
|
1351
|
+
const request = await this.slashingProposerContract.buildVoteRequestFromSigner(votes, slotNumber, signer);
|
|
1352
|
+
this.enqueueRequest(
|
|
757
1353
|
'vote-offenses',
|
|
758
1354
|
request,
|
|
759
|
-
|
|
1355
|
+
{
|
|
1356
|
+
address: this.slashingProposerContract.address.toString(),
|
|
1357
|
+
abi: SlashingProposerAbi,
|
|
1358
|
+
eventName: 'VoteCast',
|
|
1359
|
+
},
|
|
760
1360
|
slotNumber,
|
|
761
|
-
timestamp,
|
|
762
1361
|
);
|
|
763
1362
|
break;
|
|
764
1363
|
}
|
|
@@ -769,18 +1368,23 @@ export class SequencerPublisher {
|
|
|
769
1368
|
round: action.round,
|
|
770
1369
|
signerAddress,
|
|
771
1370
|
});
|
|
772
|
-
if (this.slashingProposerContract
|
|
773
|
-
this.log.error('
|
|
1371
|
+
if (!this.slashingProposerContract) {
|
|
1372
|
+
this.log.error('No slashing proposer contract available');
|
|
774
1373
|
return false;
|
|
775
1374
|
}
|
|
776
|
-
const
|
|
777
|
-
|
|
778
|
-
|
|
1375
|
+
const executeRequest = this.slashingProposerContract.buildExecuteRoundRequest(
|
|
1376
|
+
action.round,
|
|
1377
|
+
action.committees,
|
|
1378
|
+
);
|
|
1379
|
+
this.enqueueRequest(
|
|
779
1380
|
'execute-slash',
|
|
780
|
-
|
|
781
|
-
|
|
1381
|
+
executeRequest,
|
|
1382
|
+
{
|
|
1383
|
+
address: this.slashingProposerContract.address.toString(),
|
|
1384
|
+
abi: SlashingProposerAbi,
|
|
1385
|
+
eventName: 'RoundExecuted',
|
|
1386
|
+
},
|
|
782
1387
|
slotNumber,
|
|
783
|
-
timestamp,
|
|
784
1388
|
);
|
|
785
1389
|
break;
|
|
786
1390
|
}
|
|
@@ -795,132 +1399,98 @@ export class SequencerPublisher {
|
|
|
795
1399
|
return true;
|
|
796
1400
|
}
|
|
797
1401
|
|
|
798
|
-
/**
|
|
799
|
-
|
|
800
|
-
|
|
801
|
-
* @param block - L2 block to propose.
|
|
802
|
-
* @returns True if the tx has been enqueued, throws otherwise. See #9315
|
|
803
|
-
*/
|
|
804
|
-
public async enqueueProposeL2Block(
|
|
805
|
-
block: L2Block,
|
|
1402
|
+
/** Enqueues a proposal for a checkpoint on L1 */
|
|
1403
|
+
public async enqueueProposeCheckpoint(
|
|
1404
|
+
checkpoint: Checkpoint,
|
|
806
1405
|
attestationsAndSigners: CommitteeAttestationsAndSigners,
|
|
807
1406
|
attestationsAndSignersSignature: Signature,
|
|
808
|
-
opts:
|
|
809
|
-
): Promise<
|
|
810
|
-
const checkpointHeader =
|
|
1407
|
+
opts: EnqueueProposeCheckpointOpts = {},
|
|
1408
|
+
): Promise<void> {
|
|
1409
|
+
const checkpointHeader = checkpoint.header;
|
|
811
1410
|
|
|
812
|
-
const blobFields =
|
|
813
|
-
const blobs = getBlobsPerL1Block(blobFields);
|
|
1411
|
+
const blobFields = checkpoint.toBlobFields();
|
|
1412
|
+
const blobs = await getBlobsPerL1Block(blobFields);
|
|
814
1413
|
|
|
815
|
-
const proposeTxArgs = {
|
|
1414
|
+
const proposeTxArgs: L1ProcessArgs = {
|
|
816
1415
|
header: checkpointHeader,
|
|
817
|
-
archive:
|
|
818
|
-
body: block.body.toBuffer(),
|
|
1416
|
+
archive: checkpoint.archive.root.toBuffer(),
|
|
819
1417
|
blobs,
|
|
820
1418
|
attestationsAndSigners,
|
|
821
1419
|
attestationsAndSignersSignature,
|
|
1420
|
+
feeAssetPriceModifier: checkpoint.feeAssetPriceModifier,
|
|
822
1421
|
};
|
|
823
1422
|
|
|
824
|
-
|
|
825
|
-
|
|
826
|
-
|
|
827
|
-
|
|
828
|
-
|
|
829
|
-
// By simulation issue, I mean the fact that the block.timestamp is equal to the last block, not the next, which
|
|
830
|
-
// make time consistency checks break.
|
|
831
|
-
// TODO(palla): Check whether we're validating twice, once here and once within addProposeTx, since we call simulateProposeTx in both places.
|
|
832
|
-
ts = await this.validateBlockForSubmission(block, attestationsAndSigners, attestationsAndSignersSignature, opts);
|
|
833
|
-
} catch (err: any) {
|
|
834
|
-
this.log.error(`Block validation failed. ${err instanceof Error ? err.message : 'No error message'}`, err, {
|
|
835
|
-
...block.getStats(),
|
|
836
|
-
slotNumber: block.header.globalVariables.slotNumber,
|
|
837
|
-
forcePendingBlockNumber: opts.forcePendingBlockNumber,
|
|
838
|
-
});
|
|
839
|
-
throw err;
|
|
840
|
-
}
|
|
841
|
-
|
|
842
|
-
this.log.verbose(`Enqueuing block propose transaction`, { ...block.toBlockInfo(), ...opts });
|
|
843
|
-
await this.addProposeTx(block, proposeTxArgs, opts, ts);
|
|
844
|
-
return true;
|
|
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 });
|
|
845
1428
|
}
|
|
846
1429
|
|
|
847
|
-
public
|
|
1430
|
+
public enqueueInvalidateCheckpoint(
|
|
1431
|
+
request: InvalidateCheckpointRequest | undefined,
|
|
1432
|
+
opts: { txTimeoutAt?: Date } = {},
|
|
1433
|
+
) {
|
|
848
1434
|
if (!request) {
|
|
849
1435
|
return;
|
|
850
1436
|
}
|
|
851
1437
|
|
|
852
|
-
|
|
853
|
-
const
|
|
854
|
-
|
|
855
|
-
const { gasUsed, blockNumber } = request;
|
|
856
|
-
const logData = { gasUsed, blockNumber, gasLimit, opts };
|
|
857
|
-
this.log.verbose(`Enqueuing invalidate block request`, logData);
|
|
1438
|
+
const { gasUsed, checkpointNumber } = request;
|
|
1439
|
+
const logData = { gasUsed, checkpointNumber, opts };
|
|
1440
|
+
this.log.verbose(`Enqueuing invalidate checkpoint request`, logData);
|
|
858
1441
|
this.addRequest({
|
|
859
1442
|
action: `invalidate-by-${request.reason}`,
|
|
860
1443
|
request: request.request,
|
|
861
|
-
gasConfig: {
|
|
1444
|
+
gasConfig: opts.txTimeoutAt ? { txTimeoutAt: opts.txTimeoutAt } : undefined,
|
|
862
1445
|
lastValidL2Slot: SlotNumber(this.getCurrentL2Slot() + 2),
|
|
863
1446
|
checkSuccess: (_req, result) => {
|
|
864
1447
|
const success =
|
|
865
1448
|
result &&
|
|
866
|
-
result.receipt
|
|
867
|
-
|
|
868
|
-
|
|
1449
|
+
extractEventSuccess(result.receipt, {
|
|
1450
|
+
address: this.rollupContract.address,
|
|
1451
|
+
abi: RollupAbi,
|
|
1452
|
+
eventName: 'CheckpointInvalidated',
|
|
1453
|
+
});
|
|
869
1454
|
if (!success) {
|
|
870
|
-
this.log.warn(`Invalidate
|
|
1455
|
+
this.log.warn(`Invalidate checkpoint ${request.checkpointNumber} failed`, { ...result, ...logData });
|
|
871
1456
|
} else {
|
|
872
|
-
this.log.info(`Invalidate
|
|
1457
|
+
this.log.info(`Invalidate checkpoint ${request.checkpointNumber} succeeded`, { ...result, ...logData });
|
|
873
1458
|
}
|
|
874
1459
|
return !!success;
|
|
875
1460
|
},
|
|
876
1461
|
});
|
|
877
1462
|
}
|
|
878
1463
|
|
|
879
|
-
|
|
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(
|
|
880
1470
|
action: Action,
|
|
881
1471
|
request: L1TxRequest,
|
|
882
|
-
|
|
1472
|
+
eventOpts: { address: string; abi: Abi; eventName: string },
|
|
883
1473
|
slotNumber: SlotNumber,
|
|
884
|
-
|
|
885
|
-
) {
|
|
886
|
-
const logData = { slotNumber, timestamp, gasLimit: undefined as bigint | undefined };
|
|
1474
|
+
): boolean {
|
|
887
1475
|
if (this.lastActions[action] && this.lastActions[action] === slotNumber) {
|
|
888
1476
|
this.log.debug(`Skipping duplicate action ${action} for slot ${slotNumber}`);
|
|
889
1477
|
return false;
|
|
890
1478
|
}
|
|
891
|
-
|
|
892
1479
|
const cachedLastActionSlot = this.lastActions[action];
|
|
893
1480
|
this.lastActions[action] = slotNumber;
|
|
894
1481
|
|
|
895
|
-
this.log.debug(`
|
|
896
|
-
|
|
897
|
-
let gasUsed: bigint;
|
|
898
|
-
try {
|
|
899
|
-
({ gasUsed } = await this.l1TxUtils.simulate(request, { time: timestamp }, [], ErrorsAbi)); // TODO(palla/slash): Check the timestamp logic
|
|
900
|
-
this.log.verbose(`Simulation for ${action} succeeded`, { ...logData, request, gasUsed });
|
|
901
|
-
} catch (err) {
|
|
902
|
-
const viemError = formatViemError(err);
|
|
903
|
-
this.log.error(`Simulation for ${action} at ${slotNumber} failed`, viemError, logData);
|
|
904
|
-
return false;
|
|
905
|
-
}
|
|
906
|
-
|
|
907
|
-
// We issued the simulation against the rollup contract, so we need to account for the overhead of the multicall3
|
|
908
|
-
const gasLimit = this.l1TxUtils.bumpGasLimit(BigInt(Math.ceil((Number(gasUsed) * 64) / 63)));
|
|
909
|
-
logData.gasLimit = gasLimit;
|
|
910
|
-
|
|
911
|
-
this.log.debug(`Enqueuing ${action}`, logData);
|
|
1482
|
+
this.log.debug(`Enqueuing ${action}`, { slotNumber });
|
|
912
1483
|
this.addRequest({
|
|
913
1484
|
action,
|
|
914
1485
|
request,
|
|
915
|
-
gasConfig: { gasLimit },
|
|
916
1486
|
lastValidL2Slot: slotNumber,
|
|
917
|
-
checkSuccess: (
|
|
918
|
-
const success = result && result.receipt
|
|
1487
|
+
checkSuccess: (_request, result) => {
|
|
1488
|
+
const success = result && extractEventSuccess(result.receipt, eventOpts);
|
|
919
1489
|
if (!success) {
|
|
920
|
-
this.log.warn(`Action ${action} at ${slotNumber} failed`, { ...result,
|
|
1490
|
+
this.log.warn(`Action ${action} at ${slotNumber} failed`, { ...result, slotNumber });
|
|
921
1491
|
this.lastActions[action] = cachedLastActionSlot;
|
|
922
1492
|
} else {
|
|
923
|
-
this.log.info(`Action ${action} at ${slotNumber} succeeded`, { ...result,
|
|
1493
|
+
this.log.info(`Action ${action} at ${slotNumber} succeeded`, { ...result, slotNumber });
|
|
924
1494
|
}
|
|
925
1495
|
return !!success;
|
|
926
1496
|
},
|
|
@@ -936,6 +1506,7 @@ export class SequencerPublisher {
|
|
|
936
1506
|
*/
|
|
937
1507
|
public interrupt() {
|
|
938
1508
|
this.interrupted = true;
|
|
1509
|
+
this.interruptibleSleep.interrupt();
|
|
939
1510
|
this.l1TxUtils.interrupt();
|
|
940
1511
|
}
|
|
941
1512
|
|
|
@@ -945,11 +1516,7 @@ export class SequencerPublisher {
|
|
|
945
1516
|
this.l1TxUtils.restart();
|
|
946
1517
|
}
|
|
947
1518
|
|
|
948
|
-
private async prepareProposeTx(
|
|
949
|
-
encodedData: L1ProcessArgs,
|
|
950
|
-
timestamp: bigint,
|
|
951
|
-
options: { forcePendingBlockNumber?: BlockNumber },
|
|
952
|
-
) {
|
|
1519
|
+
private async prepareProposeTx(encodedData: L1ProcessArgs) {
|
|
953
1520
|
const kzg = Blob.getViemKzgInstance();
|
|
954
1521
|
const blobInput = getPrefixedEthBlobCommitments(encodedData.blobs);
|
|
955
1522
|
this.log.debug('Validating blob input', { blobInput });
|
|
@@ -962,7 +1529,11 @@ export class SequencerPublisher {
|
|
|
962
1529
|
blobEvaluationGas = BigInt(encodedData.blobs.length) * 21_000n;
|
|
963
1530
|
this.log.debug(`Using fixed blob evaluation gas estimate in fisherman mode: ${blobEvaluationGas}`);
|
|
964
1531
|
} else {
|
|
965
|
-
//
|
|
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.
|
|
966
1537
|
blobEvaluationGas = await this.l1TxUtils
|
|
967
1538
|
.estimateGas(
|
|
968
1539
|
this.getSenderAddress().toString(),
|
|
@@ -980,9 +1551,31 @@ export class SequencerPublisher {
|
|
|
980
1551
|
kzg,
|
|
981
1552
|
},
|
|
982
1553
|
)
|
|
983
|
-
.catch(err => {
|
|
984
|
-
const
|
|
985
|
-
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
|
+
);
|
|
986
1579
|
throw new Error('Failed to validate blobs');
|
|
987
1580
|
});
|
|
988
1581
|
}
|
|
@@ -993,8 +1586,7 @@ export class SequencerPublisher {
|
|
|
993
1586
|
header: encodedData.header.toViem(),
|
|
994
1587
|
archive: toHex(encodedData.archive),
|
|
995
1588
|
oracleInput: {
|
|
996
|
-
|
|
997
|
-
feeAssetPriceModifier: 0n,
|
|
1589
|
+
feeAssetPriceModifier: encodedData.feeAssetPriceModifier,
|
|
998
1590
|
},
|
|
999
1591
|
},
|
|
1000
1592
|
encodedData.attestationsAndSigners.getPackedAttestations(),
|
|
@@ -1003,133 +1595,29 @@ export class SequencerPublisher {
|
|
|
1003
1595
|
blobInput,
|
|
1004
1596
|
] as const;
|
|
1005
1597
|
|
|
1006
|
-
const
|
|
1598
|
+
const rollupData = encodeFunctionData({ abi: RollupAbi, functionName: 'propose', args });
|
|
1007
1599
|
|
|
1008
|
-
return { args, blobEvaluationGas, rollupData
|
|
1009
|
-
}
|
|
1010
|
-
|
|
1011
|
-
/**
|
|
1012
|
-
* Simulates the propose tx with eth_simulateV1
|
|
1013
|
-
* @param args - The propose tx args
|
|
1014
|
-
* @param timestamp - The timestamp to simulate proposal at
|
|
1015
|
-
* @returns The simulation result
|
|
1016
|
-
*/
|
|
1017
|
-
private async simulateProposeTx(
|
|
1018
|
-
args: readonly [
|
|
1019
|
-
{
|
|
1020
|
-
readonly header: ViemHeader;
|
|
1021
|
-
readonly archive: `0x${string}`;
|
|
1022
|
-
readonly oracleInput: {
|
|
1023
|
-
readonly feeAssetPriceModifier: 0n;
|
|
1024
|
-
};
|
|
1025
|
-
},
|
|
1026
|
-
ViemCommitteeAttestations,
|
|
1027
|
-
`0x${string}`[], // Signers
|
|
1028
|
-
ViemSignature,
|
|
1029
|
-
`0x${string}`,
|
|
1030
|
-
],
|
|
1031
|
-
timestamp: bigint,
|
|
1032
|
-
options: { forcePendingBlockNumber?: BlockNumber },
|
|
1033
|
-
) {
|
|
1034
|
-
const rollupData = encodeFunctionData({
|
|
1035
|
-
abi: RollupAbi,
|
|
1036
|
-
functionName: 'propose',
|
|
1037
|
-
args,
|
|
1038
|
-
});
|
|
1039
|
-
|
|
1040
|
-
// override the pending checkpoint number if requested
|
|
1041
|
-
const optsForcePendingCheckpointNumber =
|
|
1042
|
-
options.forcePendingBlockNumber !== undefined
|
|
1043
|
-
? CheckpointNumber.fromBlockNumber(options.forcePendingBlockNumber)
|
|
1044
|
-
: undefined;
|
|
1045
|
-
const forcePendingCheckpointNumberStateDiff = (
|
|
1046
|
-
optsForcePendingCheckpointNumber !== undefined
|
|
1047
|
-
? await this.rollupContract.makePendingCheckpointNumberOverride(optsForcePendingCheckpointNumber)
|
|
1048
|
-
: []
|
|
1049
|
-
).flatMap(override => override.stateDiff ?? []);
|
|
1050
|
-
|
|
1051
|
-
const stateOverrides: StateOverride = [
|
|
1052
|
-
{
|
|
1053
|
-
address: this.rollupContract.address,
|
|
1054
|
-
// @note we override checkBlob to false since blobs are not part simulate()
|
|
1055
|
-
stateDiff: [
|
|
1056
|
-
{ slot: toPaddedHex(RollupContract.checkBlobStorageSlot, true), value: toPaddedHex(0n, true) },
|
|
1057
|
-
...forcePendingCheckpointNumberStateDiff,
|
|
1058
|
-
],
|
|
1059
|
-
},
|
|
1060
|
-
];
|
|
1061
|
-
// In fisherman mode, simulate as the proposer but with sufficient balance
|
|
1062
|
-
if (this.proposerAddressForSimulation) {
|
|
1063
|
-
stateOverrides.push({
|
|
1064
|
-
address: this.proposerAddressForSimulation.toString(),
|
|
1065
|
-
balance: 10n * WEI_CONST * WEI_CONST, // 10 ETH
|
|
1066
|
-
});
|
|
1067
|
-
}
|
|
1068
|
-
|
|
1069
|
-
const simulationResult = await this.l1TxUtils
|
|
1070
|
-
.simulate(
|
|
1071
|
-
{
|
|
1072
|
-
to: this.rollupContract.address,
|
|
1073
|
-
data: rollupData,
|
|
1074
|
-
gas: SequencerPublisher.PROPOSE_GAS_GUESS,
|
|
1075
|
-
...(this.proposerAddressForSimulation && { from: this.proposerAddressForSimulation.toString() }),
|
|
1076
|
-
},
|
|
1077
|
-
{
|
|
1078
|
-
// @note we add 1n to the timestamp because geth implementation doesn't like simulation timestamp to be equal to the current block timestamp
|
|
1079
|
-
time: timestamp + 1n,
|
|
1080
|
-
// @note reth should have a 30m gas limit per block but throws errors that this tx is beyond limit so we increase here
|
|
1081
|
-
gasLimit: SequencerPublisher.PROPOSE_GAS_GUESS * 2n,
|
|
1082
|
-
},
|
|
1083
|
-
stateOverrides,
|
|
1084
|
-
RollupAbi,
|
|
1085
|
-
{
|
|
1086
|
-
// @note fallback gas estimate to use if the node doesn't support simulation API
|
|
1087
|
-
fallbackGasEstimate: SequencerPublisher.PROPOSE_GAS_GUESS,
|
|
1088
|
-
},
|
|
1089
|
-
)
|
|
1090
|
-
.catch(err => {
|
|
1091
|
-
// In fisherman mode, we expect ValidatorSelection__MissingProposerSignature since fisherman doesn't have proposer signature
|
|
1092
|
-
const viemError = formatViemError(err);
|
|
1093
|
-
if (this.config.fishermanMode && viemError.message?.includes('ValidatorSelection__MissingProposerSignature')) {
|
|
1094
|
-
this.log.debug(`Ignoring expected ValidatorSelection__MissingProposerSignature error in fisherman mode`);
|
|
1095
|
-
// Return a minimal simulation result with the fallback gas estimate
|
|
1096
|
-
return {
|
|
1097
|
-
gasUsed: SequencerPublisher.PROPOSE_GAS_GUESS,
|
|
1098
|
-
logs: [],
|
|
1099
|
-
};
|
|
1100
|
-
}
|
|
1101
|
-
this.log.error(`Failed to simulate propose tx`, viemError);
|
|
1102
|
-
throw err;
|
|
1103
|
-
});
|
|
1104
|
-
|
|
1105
|
-
return { rollupData, simulationResult };
|
|
1600
|
+
return { args, blobEvaluationGas, rollupData };
|
|
1106
1601
|
}
|
|
1107
1602
|
|
|
1108
1603
|
private async addProposeTx(
|
|
1109
|
-
|
|
1604
|
+
checkpoint: Checkpoint,
|
|
1110
1605
|
encodedData: L1ProcessArgs,
|
|
1111
|
-
opts:
|
|
1112
|
-
timestamp: bigint,
|
|
1606
|
+
opts: EnqueueProposeCheckpointOpts = {},
|
|
1113
1607
|
): Promise<void> {
|
|
1608
|
+
const slot = checkpoint.header.slotNumber;
|
|
1114
1609
|
const timer = new Timer();
|
|
1115
1610
|
const kzg = Blob.getViemKzgInstance();
|
|
1116
|
-
const { rollupData,
|
|
1117
|
-
encodedData,
|
|
1118
|
-
timestamp,
|
|
1119
|
-
opts,
|
|
1120
|
-
);
|
|
1611
|
+
const { rollupData, blobEvaluationGas } = await this.prepareProposeTx(encodedData);
|
|
1121
1612
|
const startBlock = await this.l1TxUtils.getBlockNumber();
|
|
1122
|
-
const gasLimit = this.l1TxUtils.bumpGasLimit(
|
|
1123
|
-
BigInt(Math.ceil((Number(simulationResult.gasUsed) * 64) / 63)) +
|
|
1124
|
-
blobEvaluationGas +
|
|
1125
|
-
SequencerPublisher.MULTICALL_OVERHEAD_GAS_GUESS, // We issue the simulation against the rollup contract, so we need to account for the overhead of the multicall3
|
|
1126
|
-
);
|
|
1127
1613
|
|
|
1128
|
-
// Send the blobs to the blob
|
|
1129
|
-
// tx fails but it does get mined. We make sure that the blobs are sent to the blob
|
|
1130
|
-
void
|
|
1131
|
-
this.
|
|
1132
|
-
|
|
1614
|
+
// Send the blobs to the blob client preemptively. This helps in tests where the sequencer mistakingly thinks that the propose
|
|
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.
|
|
1616
|
+
void Promise.resolve().then(() =>
|
|
1617
|
+
this.blobClient.sendBlobsToFilestore(encodedData.blobs).catch(_err => {
|
|
1618
|
+
this.log.error('Failed to send blobs to blob client');
|
|
1619
|
+
}),
|
|
1620
|
+
);
|
|
1133
1621
|
|
|
1134
1622
|
return this.addRequest({
|
|
1135
1623
|
action: 'propose',
|
|
@@ -1137,8 +1625,9 @@ export class SequencerPublisher {
|
|
|
1137
1625
|
to: this.rollupContract.address,
|
|
1138
1626
|
data: rollupData,
|
|
1139
1627
|
},
|
|
1140
|
-
lastValidL2Slot:
|
|
1141
|
-
gasConfig: {
|
|
1628
|
+
lastValidL2Slot: checkpoint.header.slotNumber,
|
|
1629
|
+
gasConfig: { txTimeoutAt: opts.txTimeoutAt, gasLimit: undefined },
|
|
1630
|
+
blobEvaluationGas,
|
|
1142
1631
|
blobConfig: {
|
|
1143
1632
|
blobs: encodedData.blobs.map(b => b.data),
|
|
1144
1633
|
kzg,
|
|
@@ -1148,15 +1637,17 @@ export class SequencerPublisher {
|
|
|
1148
1637
|
return false;
|
|
1149
1638
|
}
|
|
1150
1639
|
const { receipt, stats, errorMsg } = result;
|
|
1151
|
-
const success =
|
|
1152
|
-
|
|
1153
|
-
|
|
1154
|
-
|
|
1640
|
+
const success = extractEventSuccess(receipt, {
|
|
1641
|
+
address: this.rollupContract.address,
|
|
1642
|
+
abi: RollupAbi,
|
|
1643
|
+
eventName: 'CheckpointProposed',
|
|
1644
|
+
});
|
|
1645
|
+
|
|
1155
1646
|
if (success) {
|
|
1156
1647
|
const endBlock = receipt.blockNumber;
|
|
1157
1648
|
const inclusionBlocks = Number(endBlock - startBlock);
|
|
1158
1649
|
const { calldataGas, calldataSize, sender } = stats!;
|
|
1159
|
-
const publishStats:
|
|
1650
|
+
const publishStats: L1PublishCheckpointStats = {
|
|
1160
1651
|
gasPrice: receipt.effectiveGasPrice,
|
|
1161
1652
|
gasUsed: receipt.gasUsed,
|
|
1162
1653
|
blobGasUsed: receipt.blobGasUsed ?? 0n,
|
|
@@ -1165,26 +1656,35 @@ export class SequencerPublisher {
|
|
|
1165
1656
|
calldataGas,
|
|
1166
1657
|
calldataSize,
|
|
1167
1658
|
sender,
|
|
1168
|
-
...
|
|
1659
|
+
...checkpoint.getStats(),
|
|
1169
1660
|
eventName: 'rollup-published-to-l1',
|
|
1170
1661
|
blobCount: encodedData.blobs.length,
|
|
1171
1662
|
inclusionBlocks,
|
|
1172
1663
|
};
|
|
1173
|
-
this.log.info(`Published
|
|
1664
|
+
this.log.info(`Published checkpoint ${checkpoint.number} at slot ${slot} to rollup contract`, {
|
|
1665
|
+
...stats,
|
|
1666
|
+
...checkpoint.getStats(),
|
|
1667
|
+
...pick(receipt, 'transactionHash', 'blockHash'),
|
|
1668
|
+
});
|
|
1174
1669
|
this.metrics.recordProcessBlockTx(timer.ms(), publishStats);
|
|
1175
1670
|
|
|
1176
1671
|
return true;
|
|
1177
1672
|
} else {
|
|
1178
1673
|
this.metrics.recordFailedTx('process');
|
|
1179
|
-
this.log.error(
|
|
1180
|
-
|
|
1181
|
-
|
|
1182
|
-
|
|
1183
|
-
|
|
1184
|
-
});
|
|
1674
|
+
this.log.error(
|
|
1675
|
+
`Publishing checkpoint at slot ${slot} failed with ${errorMsg ?? 'no error message'}`,
|
|
1676
|
+
undefined,
|
|
1677
|
+
{ ...checkpoint.getStats(), ...receipt },
|
|
1678
|
+
);
|
|
1185
1679
|
return false;
|
|
1186
1680
|
}
|
|
1187
1681
|
},
|
|
1188
1682
|
});
|
|
1189
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
|
+
}
|
|
1190
1690
|
}
|