@aztec/validator-client 0.0.1-commit.fcb71a6 → 0.0.1-commit.ff7989d6c
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 +285 -0
- package/dest/block_proposal_handler.d.ts +21 -11
- package/dest/block_proposal_handler.d.ts.map +1 -1
- package/dest/block_proposal_handler.js +327 -85
- package/dest/checkpoint_builder.d.ts +66 -0
- package/dest/checkpoint_builder.d.ts.map +1 -0
- package/dest/checkpoint_builder.js +175 -0
- package/dest/config.d.ts +1 -1
- package/dest/config.d.ts.map +1 -1
- package/dest/config.js +16 -8
- package/dest/duties/validation_service.d.ts +41 -12
- package/dest/duties/validation_service.d.ts.map +1 -1
- package/dest/duties/validation_service.js +109 -26
- package/dest/factory.d.ts +13 -10
- package/dest/factory.d.ts.map +1 -1
- package/dest/factory.js +2 -2
- package/dest/index.d.ts +3 -1
- package/dest/index.d.ts.map +1 -1
- package/dest/index.js +2 -0
- package/dest/key_store/ha_key_store.d.ts +99 -0
- package/dest/key_store/ha_key_store.d.ts.map +1 -0
- package/dest/key_store/ha_key_store.js +208 -0
- package/dest/key_store/index.d.ts +2 -1
- package/dest/key_store/index.d.ts.map +1 -1
- package/dest/key_store/index.js +1 -0
- package/dest/key_store/interface.d.ts +36 -6
- package/dest/key_store/interface.d.ts.map +1 -1
- package/dest/key_store/local_key_store.d.ts +10 -5
- package/dest/key_store/local_key_store.d.ts.map +1 -1
- package/dest/key_store/local_key_store.js +8 -4
- package/dest/key_store/node_keystore_adapter.d.ts +18 -5
- package/dest/key_store/node_keystore_adapter.d.ts.map +1 -1
- package/dest/key_store/node_keystore_adapter.js +18 -4
- package/dest/key_store/web3signer_key_store.d.ts +10 -5
- package/dest/key_store/web3signer_key_store.d.ts.map +1 -1
- package/dest/key_store/web3signer_key_store.js +8 -4
- package/dest/metrics.d.ts +4 -3
- package/dest/metrics.d.ts.map +1 -1
- package/dest/metrics.js +34 -30
- package/dest/tx_validator/index.d.ts +3 -0
- package/dest/tx_validator/index.d.ts.map +1 -0
- package/dest/tx_validator/index.js +2 -0
- package/dest/tx_validator/nullifier_cache.d.ts +14 -0
- package/dest/tx_validator/nullifier_cache.d.ts.map +1 -0
- package/dest/tx_validator/nullifier_cache.js +24 -0
- package/dest/tx_validator/tx_validator_factory.d.ts +19 -0
- package/dest/tx_validator/tx_validator_factory.d.ts.map +1 -0
- package/dest/tx_validator/tx_validator_factory.js +54 -0
- package/dest/validator.d.ts +73 -24
- package/dest/validator.d.ts.map +1 -1
- package/dest/validator.js +452 -91
- package/package.json +21 -13
- package/src/block_proposal_handler.ts +246 -57
- package/src/checkpoint_builder.ts +321 -0
- package/src/config.ts +15 -7
- package/src/duties/validation_service.ts +160 -31
- package/src/factory.ts +17 -11
- package/src/index.ts +2 -0
- package/src/key_store/ha_key_store.ts +269 -0
- package/src/key_store/index.ts +1 -0
- package/src/key_store/interface.ts +44 -5
- package/src/key_store/local_key_store.ts +13 -4
- package/src/key_store/node_keystore_adapter.ts +27 -4
- package/src/key_store/web3signer_key_store.ts +17 -4
- package/src/metrics.ts +45 -33
- package/src/tx_validator/index.ts +2 -0
- package/src/tx_validator/nullifier_cache.ts +30 -0
- package/src/tx_validator/tx_validator_factory.ts +154 -0
- package/src/validator.ts +615 -120
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@aztec/validator-client",
|
|
3
|
-
"version": "0.0.1-commit.
|
|
3
|
+
"version": "0.0.1-commit.ff7989d6c",
|
|
4
4
|
"main": "dest/index.js",
|
|
5
5
|
"type": "module",
|
|
6
6
|
"exports": {
|
|
@@ -64,27 +64,35 @@
|
|
|
64
64
|
]
|
|
65
65
|
},
|
|
66
66
|
"dependencies": {
|
|
67
|
-
"@aztec/blob-client": "0.0.1-commit.
|
|
68
|
-
"@aztec/blob-lib": "0.0.1-commit.
|
|
69
|
-
"@aztec/constants": "0.0.1-commit.
|
|
70
|
-
"@aztec/epoch-cache": "0.0.1-commit.
|
|
71
|
-
"@aztec/ethereum": "0.0.1-commit.
|
|
72
|
-
"@aztec/foundation": "0.0.1-commit.
|
|
73
|
-
"@aztec/node-keystore": "0.0.1-commit.
|
|
74
|
-
"@aztec/
|
|
75
|
-
"@aztec/
|
|
76
|
-
"@aztec/
|
|
77
|
-
"@aztec/
|
|
67
|
+
"@aztec/blob-client": "0.0.1-commit.ff7989d6c",
|
|
68
|
+
"@aztec/blob-lib": "0.0.1-commit.ff7989d6c",
|
|
69
|
+
"@aztec/constants": "0.0.1-commit.ff7989d6c",
|
|
70
|
+
"@aztec/epoch-cache": "0.0.1-commit.ff7989d6c",
|
|
71
|
+
"@aztec/ethereum": "0.0.1-commit.ff7989d6c",
|
|
72
|
+
"@aztec/foundation": "0.0.1-commit.ff7989d6c",
|
|
73
|
+
"@aztec/node-keystore": "0.0.1-commit.ff7989d6c",
|
|
74
|
+
"@aztec/noir-protocol-circuits-types": "0.0.1-commit.ff7989d6c",
|
|
75
|
+
"@aztec/p2p": "0.0.1-commit.ff7989d6c",
|
|
76
|
+
"@aztec/protocol-contracts": "0.0.1-commit.ff7989d6c",
|
|
77
|
+
"@aztec/prover-client": "0.0.1-commit.ff7989d6c",
|
|
78
|
+
"@aztec/simulator": "0.0.1-commit.ff7989d6c",
|
|
79
|
+
"@aztec/slasher": "0.0.1-commit.ff7989d6c",
|
|
80
|
+
"@aztec/stdlib": "0.0.1-commit.ff7989d6c",
|
|
81
|
+
"@aztec/telemetry-client": "0.0.1-commit.ff7989d6c",
|
|
82
|
+
"@aztec/validator-ha-signer": "0.0.1-commit.ff7989d6c",
|
|
78
83
|
"koa": "^2.16.1",
|
|
79
84
|
"koa-router": "^13.1.1",
|
|
80
85
|
"tslib": "^2.4.0",
|
|
81
86
|
"viem": "npm:@aztec/viem@2.38.2"
|
|
82
87
|
},
|
|
83
88
|
"devDependencies": {
|
|
89
|
+
"@aztec/archiver": "0.0.1-commit.ff7989d6c",
|
|
90
|
+
"@aztec/world-state": "0.0.1-commit.ff7989d6c",
|
|
91
|
+
"@electric-sql/pglite": "^0.3.14",
|
|
84
92
|
"@jest/globals": "^30.0.0",
|
|
85
93
|
"@types/jest": "^30.0.0",
|
|
86
94
|
"@types/node": "^22.15.17",
|
|
87
|
-
"@typescript/native-preview": "7.0.0-dev.
|
|
95
|
+
"@typescript/native-preview": "7.0.0-dev.20260113.1",
|
|
88
96
|
"jest": "^30.0.0",
|
|
89
97
|
"jest-mock-extended": "^4.0.0",
|
|
90
98
|
"ts-node": "^10.9.1",
|
|
@@ -1,4 +1,5 @@
|
|
|
1
1
|
import { INITIAL_L2_BLOCK_NUM } from '@aztec/constants';
|
|
2
|
+
import type { EpochCache } from '@aztec/epoch-cache';
|
|
2
3
|
import { BlockNumber, CheckpointNumber, SlotNumber } from '@aztec/foundation/branded-types';
|
|
3
4
|
import { Fr } from '@aztec/foundation/curves/bn254';
|
|
4
5
|
import { TimeoutError } from '@aztec/foundation/error';
|
|
@@ -6,14 +7,13 @@ import { createLogger } from '@aztec/foundation/log';
|
|
|
6
7
|
import { retryUntil } from '@aztec/foundation/retry';
|
|
7
8
|
import { DateProvider, Timer } from '@aztec/foundation/timer';
|
|
8
9
|
import type { P2P, PeerId } from '@aztec/p2p';
|
|
9
|
-
import { TxProvider } from '@aztec/p2p';
|
|
10
10
|
import { BlockProposalValidator } from '@aztec/p2p/msg_validators';
|
|
11
|
-
import type { L2Block, L2BlockSource } from '@aztec/stdlib/block';
|
|
12
|
-
import { getTimestampForSlot } from '@aztec/stdlib/epoch-helpers';
|
|
13
|
-
import type {
|
|
11
|
+
import type { BlockData, L2Block, L2BlockSink, L2BlockSource } from '@aztec/stdlib/block';
|
|
12
|
+
import { getEpochAtSlot, getTimestampForSlot } from '@aztec/stdlib/epoch-helpers';
|
|
13
|
+
import type { ITxProvider, ValidatorClientFullConfig, WorldStateSynchronizer } from '@aztec/stdlib/interfaces/server';
|
|
14
14
|
import { type L1ToL2MessageSource, computeInHashFromL1ToL2Messages } from '@aztec/stdlib/messaging';
|
|
15
|
-
import {
|
|
16
|
-
import {
|
|
15
|
+
import type { BlockProposal } from '@aztec/stdlib/p2p';
|
|
16
|
+
import type { CheckpointGlobalVariables, FailedTx, Tx } from '@aztec/stdlib/tx';
|
|
17
17
|
import {
|
|
18
18
|
ReExFailedTxsError,
|
|
19
19
|
ReExStateMismatchError,
|
|
@@ -22,6 +22,7 @@ import {
|
|
|
22
22
|
} from '@aztec/stdlib/validators';
|
|
23
23
|
import { type TelemetryClient, type Tracer, getTelemetryClient } from '@aztec/telemetry-client';
|
|
24
24
|
|
|
25
|
+
import type { FullNodeCheckpointsBuilder } from './checkpoint_builder.js';
|
|
25
26
|
import type { ValidatorMetrics } from './metrics.js';
|
|
26
27
|
|
|
27
28
|
export type BlockProposalValidationFailureReason =
|
|
@@ -29,6 +30,7 @@ export type BlockProposalValidationFailureReason =
|
|
|
29
30
|
| 'parent_block_not_found'
|
|
30
31
|
| 'parent_block_wrong_slot'
|
|
31
32
|
| 'in_hash_mismatch'
|
|
33
|
+
| 'global_variables_mismatch'
|
|
32
34
|
| 'block_number_already_exists'
|
|
33
35
|
| 'txs_not_available'
|
|
34
36
|
| 'state_mismatch'
|
|
@@ -58,15 +60,21 @@ export type BlockProposalValidationFailureResult = {
|
|
|
58
60
|
|
|
59
61
|
export type BlockProposalValidationResult = BlockProposalValidationSuccessResult | BlockProposalValidationFailureResult;
|
|
60
62
|
|
|
63
|
+
type CheckpointComputationResult =
|
|
64
|
+
| { checkpointNumber: CheckpointNumber; reason?: undefined }
|
|
65
|
+
| { checkpointNumber?: undefined; reason: 'invalid_proposal' | 'global_variables_mismatch' };
|
|
66
|
+
|
|
61
67
|
export class BlockProposalHandler {
|
|
62
68
|
public readonly tracer: Tracer;
|
|
63
69
|
|
|
64
70
|
constructor(
|
|
65
|
-
private
|
|
66
|
-
private
|
|
71
|
+
private checkpointsBuilder: FullNodeCheckpointsBuilder,
|
|
72
|
+
private worldState: WorldStateSynchronizer,
|
|
73
|
+
private blockSource: L2BlockSource & L2BlockSink,
|
|
67
74
|
private l1ToL2MessageSource: L1ToL2MessageSource,
|
|
68
|
-
private txProvider:
|
|
75
|
+
private txProvider: ITxProvider,
|
|
69
76
|
private blockProposalValidator: BlockProposalValidator,
|
|
77
|
+
private epochCache: EpochCache,
|
|
70
78
|
private config: ValidatorClientFullConfig,
|
|
71
79
|
private metrics?: ValidatorMetrics,
|
|
72
80
|
private dateProvider: DateProvider = new DateProvider(),
|
|
@@ -80,7 +88,9 @@ export class BlockProposalHandler {
|
|
|
80
88
|
}
|
|
81
89
|
|
|
82
90
|
registerForReexecution(p2pClient: P2P): BlockProposalHandler {
|
|
83
|
-
|
|
91
|
+
// Non-validator handler that re-executes for monitoring but does not attest.
|
|
92
|
+
// Returns boolean indicating whether the proposal was valid.
|
|
93
|
+
const handler = async (proposal: BlockProposal, proposalSender: PeerId): Promise<boolean> => {
|
|
84
94
|
try {
|
|
85
95
|
const result = await this.handleBlockProposal(proposal, proposalSender, true);
|
|
86
96
|
if (result.isValid) {
|
|
@@ -90,16 +100,18 @@ export class BlockProposalHandler {
|
|
|
90
100
|
totalManaUsed: result.reexecutionResult?.totalManaUsed,
|
|
91
101
|
numTxs: result.reexecutionResult?.block?.body?.txEffects?.length ?? 0,
|
|
92
102
|
});
|
|
103
|
+
return true;
|
|
93
104
|
} else {
|
|
94
105
|
this.log.warn(`Non-validator reexecution failed for slot ${proposal.slotNumber}`, {
|
|
95
106
|
blockNumber: result.blockNumber,
|
|
96
107
|
reason: result.reason,
|
|
97
108
|
});
|
|
109
|
+
return false;
|
|
98
110
|
}
|
|
99
111
|
} catch (error) {
|
|
100
112
|
this.log.error('Error processing block proposal in non-validator handler', error);
|
|
113
|
+
return false;
|
|
101
114
|
}
|
|
102
|
-
return undefined; // Non-validator nodes don't return attestations
|
|
103
115
|
};
|
|
104
116
|
|
|
105
117
|
p2pClient.registerBlockProposalHandler(handler);
|
|
@@ -113,7 +125,7 @@ export class BlockProposalHandler {
|
|
|
113
125
|
): Promise<BlockProposalValidationResult> {
|
|
114
126
|
const slotNumber = proposal.slotNumber;
|
|
115
127
|
const proposer = proposal.getSender();
|
|
116
|
-
const config = this.
|
|
128
|
+
const config = this.checkpointsBuilder.getConfig();
|
|
117
129
|
|
|
118
130
|
// Reject proposals with invalid signatures
|
|
119
131
|
if (!proposer) {
|
|
@@ -129,23 +141,23 @@ export class BlockProposalHandler {
|
|
|
129
141
|
|
|
130
142
|
// Check that the proposal is from the current proposer, or the next proposer
|
|
131
143
|
// This should have been handled by the p2p layer, but we double check here out of caution
|
|
132
|
-
const
|
|
133
|
-
if (
|
|
144
|
+
const validationResult = await this.blockProposalValidator.validate(proposal);
|
|
145
|
+
if (validationResult.result !== 'accept') {
|
|
134
146
|
this.log.warn(`Proposal is not valid, skipping processing`, proposalInfo);
|
|
135
147
|
return { isValid: false, reason: 'invalid_proposal' };
|
|
136
148
|
}
|
|
137
149
|
|
|
138
150
|
// Check that the parent proposal is a block we know, otherwise reexecution would fail
|
|
139
|
-
const
|
|
140
|
-
if (
|
|
151
|
+
const parentBlock = await this.getParentBlock(proposal);
|
|
152
|
+
if (parentBlock === undefined) {
|
|
141
153
|
this.log.warn(`Parent block for proposal not found, skipping processing`, proposalInfo);
|
|
142
154
|
return { isValid: false, reason: 'parent_block_not_found' };
|
|
143
155
|
}
|
|
144
156
|
|
|
145
|
-
// Check that the parent block's slot is
|
|
146
|
-
if (
|
|
147
|
-
this.log.warn(`Parent block slot is greater than
|
|
148
|
-
parentBlockSlot:
|
|
157
|
+
// Check that the parent block's slot is not greater than the proposal's slot.
|
|
158
|
+
if (parentBlock !== 'genesis' && parentBlock.header.getSlot() > slotNumber) {
|
|
159
|
+
this.log.warn(`Parent block slot is greater than proposal slot, skipping processing`, {
|
|
160
|
+
parentBlockSlot: parentBlock.header.getSlot().toString(),
|
|
149
161
|
proposalSlot: slotNumber.toString(),
|
|
150
162
|
...proposalInfo,
|
|
151
163
|
});
|
|
@@ -154,9 +166,9 @@ export class BlockProposalHandler {
|
|
|
154
166
|
|
|
155
167
|
// Compute the block number based on the parent block
|
|
156
168
|
const blockNumber =
|
|
157
|
-
|
|
169
|
+
parentBlock === 'genesis'
|
|
158
170
|
? BlockNumber(INITIAL_L2_BLOCK_NUM)
|
|
159
|
-
: BlockNumber(
|
|
171
|
+
: BlockNumber(parentBlock.header.getBlockNumber() + 1);
|
|
160
172
|
|
|
161
173
|
// Check that this block number does not exist already
|
|
162
174
|
const existingBlock = await this.blockSource.getBlockHeader(blockNumber);
|
|
@@ -172,12 +184,17 @@ export class BlockProposalHandler {
|
|
|
172
184
|
deadline: this.getReexecutionDeadline(slotNumber, config),
|
|
173
185
|
});
|
|
174
186
|
|
|
187
|
+
// Compute the checkpoint number for this block and validate checkpoint consistency
|
|
188
|
+
const checkpointResult = this.computeCheckpointNumber(proposal, parentBlock, proposalInfo);
|
|
189
|
+
if (checkpointResult.reason) {
|
|
190
|
+
return { isValid: false, blockNumber, reason: checkpointResult.reason };
|
|
191
|
+
}
|
|
192
|
+
const checkpointNumber = checkpointResult.checkpointNumber;
|
|
193
|
+
|
|
175
194
|
// Check that I have the same set of l1ToL2Messages as the proposal
|
|
176
|
-
const l1ToL2Messages = await this.l1ToL2MessageSource.getL1ToL2Messages(
|
|
177
|
-
CheckpointNumber.fromBlockNumber(blockNumber),
|
|
178
|
-
);
|
|
195
|
+
const l1ToL2Messages = await this.l1ToL2MessageSource.getL1ToL2Messages(checkpointNumber);
|
|
179
196
|
const computedInHash = computeInHashFromL1ToL2Messages(l1ToL2Messages);
|
|
180
|
-
const proposalInHash = proposal.
|
|
197
|
+
const proposalInHash = proposal.inHash;
|
|
181
198
|
if (!computedInHash.equals(proposalInHash)) {
|
|
182
199
|
this.log.warn(`L1 to L2 messages in hash mismatch, skipping processing`, {
|
|
183
200
|
proposalInHash: proposalInHash.toString(),
|
|
@@ -196,9 +213,22 @@ export class BlockProposalHandler {
|
|
|
196
213
|
// Try re-executing the transactions in the proposal if needed
|
|
197
214
|
let reexecutionResult;
|
|
198
215
|
if (shouldReexecute) {
|
|
216
|
+
// Collect the out hashes of all the checkpoints before this one in the same epoch
|
|
217
|
+
const epoch = getEpochAtSlot(slotNumber, this.epochCache.getL1Constants());
|
|
218
|
+
const previousCheckpointOutHashes = (await this.blockSource.getCheckpointsDataForEpoch(epoch))
|
|
219
|
+
.filter(c => c.checkpointNumber < checkpointNumber)
|
|
220
|
+
.map(c => c.checkpointOutHash);
|
|
221
|
+
|
|
199
222
|
try {
|
|
200
223
|
this.log.verbose(`Re-executing transactions in the proposal`, proposalInfo);
|
|
201
|
-
reexecutionResult = await this.reexecuteTransactions(
|
|
224
|
+
reexecutionResult = await this.reexecuteTransactions(
|
|
225
|
+
proposal,
|
|
226
|
+
blockNumber,
|
|
227
|
+
checkpointNumber,
|
|
228
|
+
txs,
|
|
229
|
+
l1ToL2Messages,
|
|
230
|
+
previousCheckpointOutHashes,
|
|
231
|
+
);
|
|
202
232
|
} catch (error) {
|
|
203
233
|
this.log.error(`Error reexecuting txs while processing block proposal`, error, proposalInfo);
|
|
204
234
|
const reason = this.getReexecuteFailureReason(error);
|
|
@@ -206,14 +236,23 @@ export class BlockProposalHandler {
|
|
|
206
236
|
}
|
|
207
237
|
}
|
|
208
238
|
|
|
209
|
-
this
|
|
239
|
+
// If we succeeded, push this block into the archiver (unless disabled)
|
|
240
|
+
if (reexecutionResult?.block && this.config.skipPushProposedBlocksToArchiver === false) {
|
|
241
|
+
await this.blockSource.addBlock(reexecutionResult?.block);
|
|
242
|
+
}
|
|
243
|
+
|
|
244
|
+
this.log.info(
|
|
245
|
+
`Successfully processed block ${blockNumber} proposal at index ${proposal.indexWithinCheckpoint} on slot ${slotNumber}`,
|
|
246
|
+
proposalInfo,
|
|
247
|
+
);
|
|
248
|
+
|
|
210
249
|
return { isValid: true, blockNumber, reexecutionResult };
|
|
211
250
|
}
|
|
212
251
|
|
|
213
|
-
private async getParentBlock(proposal: BlockProposal): Promise<'genesis' |
|
|
214
|
-
const parentArchive = proposal.
|
|
252
|
+
private async getParentBlock(proposal: BlockProposal): Promise<'genesis' | BlockData | undefined> {
|
|
253
|
+
const parentArchive = proposal.blockHeader.lastArchive.root;
|
|
215
254
|
const slot = proposal.slotNumber;
|
|
216
|
-
const config = this.
|
|
255
|
+
const config = this.checkpointsBuilder.getConfig();
|
|
217
256
|
const { genesisArchiveRoot } = await this.blockSource.getGenesisValues();
|
|
218
257
|
|
|
219
258
|
if (parentArchive.equals(genesisArchiveRoot)) {
|
|
@@ -226,12 +265,11 @@ export class BlockProposalHandler {
|
|
|
226
265
|
|
|
227
266
|
try {
|
|
228
267
|
return (
|
|
229
|
-
(await this.blockSource.
|
|
268
|
+
(await this.blockSource.getBlockDataByArchive(parentArchive)) ??
|
|
230
269
|
(timeoutDurationMs <= 0
|
|
231
270
|
? undefined
|
|
232
271
|
: await retryUntil(
|
|
233
|
-
() =>
|
|
234
|
-
this.blockSource.syncImmediate().then(() => this.blockSource.getBlockHeaderByArchive(parentArchive)),
|
|
272
|
+
() => this.blockSource.syncImmediate().then(() => this.blockSource.getBlockDataByArchive(parentArchive)),
|
|
235
273
|
'force archiver sync',
|
|
236
274
|
timeoutDurationMs / 1000,
|
|
237
275
|
0.5,
|
|
@@ -247,10 +285,132 @@ export class BlockProposalHandler {
|
|
|
247
285
|
}
|
|
248
286
|
}
|
|
249
287
|
|
|
288
|
+
private computeCheckpointNumber(
|
|
289
|
+
proposal: BlockProposal,
|
|
290
|
+
parentBlock: 'genesis' | BlockData,
|
|
291
|
+
proposalInfo: object,
|
|
292
|
+
): CheckpointComputationResult {
|
|
293
|
+
if (parentBlock === 'genesis') {
|
|
294
|
+
// First block is in checkpoint 1
|
|
295
|
+
if (proposal.indexWithinCheckpoint !== 0) {
|
|
296
|
+
this.log.warn(`First block proposal has non-zero indexWithinCheckpoint`, proposalInfo);
|
|
297
|
+
return { reason: 'invalid_proposal' };
|
|
298
|
+
}
|
|
299
|
+
return { checkpointNumber: CheckpointNumber.INITIAL };
|
|
300
|
+
}
|
|
301
|
+
|
|
302
|
+
if (proposal.indexWithinCheckpoint === 0) {
|
|
303
|
+
// If this is the first block in a new checkpoint, increment the checkpoint number
|
|
304
|
+
if (!(proposal.blockHeader.getSlot() > parentBlock.header.getSlot())) {
|
|
305
|
+
this.log.warn(`Slot should be greater than parent block slot for first block in checkpoint`, proposalInfo);
|
|
306
|
+
return { reason: 'invalid_proposal' };
|
|
307
|
+
}
|
|
308
|
+
return { checkpointNumber: CheckpointNumber(parentBlock.checkpointNumber + 1) };
|
|
309
|
+
}
|
|
310
|
+
|
|
311
|
+
// Otherwise it should follow the previous block in the same checkpoint
|
|
312
|
+
if (proposal.indexWithinCheckpoint !== parentBlock.indexWithinCheckpoint + 1) {
|
|
313
|
+
this.log.warn(`Non-sequential indexWithinCheckpoint`, proposalInfo);
|
|
314
|
+
return { reason: 'invalid_proposal' };
|
|
315
|
+
}
|
|
316
|
+
if (proposal.blockHeader.getSlot() !== parentBlock.header.getSlot()) {
|
|
317
|
+
this.log.warn(`Slot should be equal to parent block slot for non-first block in checkpoint`, proposalInfo);
|
|
318
|
+
return { reason: 'invalid_proposal' };
|
|
319
|
+
}
|
|
320
|
+
|
|
321
|
+
// For non-first blocks in a checkpoint, validate global variables match parent (except blockNumber)
|
|
322
|
+
const validationResult = this.validateNonFirstBlockInCheckpoint(proposal, parentBlock, proposalInfo);
|
|
323
|
+
if (validationResult) {
|
|
324
|
+
return validationResult;
|
|
325
|
+
}
|
|
326
|
+
|
|
327
|
+
return { checkpointNumber: parentBlock.checkpointNumber };
|
|
328
|
+
}
|
|
329
|
+
|
|
330
|
+
/**
|
|
331
|
+
* Validates that a non-first block in a checkpoint has consistent global variables with its parent.
|
|
332
|
+
* For blocks with indexWithinCheckpoint > 0, all global variables except blockNumber must match the parent.
|
|
333
|
+
* @returns A failure result if validation fails, undefined if validation passes
|
|
334
|
+
*/
|
|
335
|
+
private validateNonFirstBlockInCheckpoint(
|
|
336
|
+
proposal: BlockProposal,
|
|
337
|
+
parentBlock: BlockData,
|
|
338
|
+
proposalInfo: object,
|
|
339
|
+
): CheckpointComputationResult | undefined {
|
|
340
|
+
const proposalGlobals = proposal.blockHeader.globalVariables;
|
|
341
|
+
const parentGlobals = parentBlock.header.globalVariables;
|
|
342
|
+
|
|
343
|
+
// All global variables except blockNumber should match the parent
|
|
344
|
+
// blockNumber naturally increments between blocks
|
|
345
|
+
if (!proposalGlobals.chainId.equals(parentGlobals.chainId)) {
|
|
346
|
+
this.log.warn(`Non-first block in checkpoint has mismatched chainId`, {
|
|
347
|
+
...proposalInfo,
|
|
348
|
+
proposalChainId: proposalGlobals.chainId.toString(),
|
|
349
|
+
parentChainId: parentGlobals.chainId.toString(),
|
|
350
|
+
});
|
|
351
|
+
return { reason: 'global_variables_mismatch' };
|
|
352
|
+
}
|
|
353
|
+
|
|
354
|
+
if (!proposalGlobals.version.equals(parentGlobals.version)) {
|
|
355
|
+
this.log.warn(`Non-first block in checkpoint has mismatched version`, {
|
|
356
|
+
...proposalInfo,
|
|
357
|
+
proposalVersion: proposalGlobals.version.toString(),
|
|
358
|
+
parentVersion: parentGlobals.version.toString(),
|
|
359
|
+
});
|
|
360
|
+
return { reason: 'global_variables_mismatch' };
|
|
361
|
+
}
|
|
362
|
+
|
|
363
|
+
if (proposalGlobals.slotNumber !== parentGlobals.slotNumber) {
|
|
364
|
+
this.log.warn(`Non-first block in checkpoint has mismatched slotNumber`, {
|
|
365
|
+
...proposalInfo,
|
|
366
|
+
proposalSlotNumber: proposalGlobals.slotNumber,
|
|
367
|
+
parentSlotNumber: parentGlobals.slotNumber,
|
|
368
|
+
});
|
|
369
|
+
return { reason: 'global_variables_mismatch' };
|
|
370
|
+
}
|
|
371
|
+
|
|
372
|
+
if (proposalGlobals.timestamp !== parentGlobals.timestamp) {
|
|
373
|
+
this.log.warn(`Non-first block in checkpoint has mismatched timestamp`, {
|
|
374
|
+
...proposalInfo,
|
|
375
|
+
proposalTimestamp: proposalGlobals.timestamp.toString(),
|
|
376
|
+
parentTimestamp: parentGlobals.timestamp.toString(),
|
|
377
|
+
});
|
|
378
|
+
return { reason: 'global_variables_mismatch' };
|
|
379
|
+
}
|
|
380
|
+
|
|
381
|
+
if (!proposalGlobals.coinbase.equals(parentGlobals.coinbase)) {
|
|
382
|
+
this.log.warn(`Non-first block in checkpoint has mismatched coinbase`, {
|
|
383
|
+
...proposalInfo,
|
|
384
|
+
proposalCoinbase: proposalGlobals.coinbase.toString(),
|
|
385
|
+
parentCoinbase: parentGlobals.coinbase.toString(),
|
|
386
|
+
});
|
|
387
|
+
return { reason: 'global_variables_mismatch' };
|
|
388
|
+
}
|
|
389
|
+
|
|
390
|
+
if (!proposalGlobals.feeRecipient.equals(parentGlobals.feeRecipient)) {
|
|
391
|
+
this.log.warn(`Non-first block in checkpoint has mismatched feeRecipient`, {
|
|
392
|
+
...proposalInfo,
|
|
393
|
+
proposalFeeRecipient: proposalGlobals.feeRecipient.toString(),
|
|
394
|
+
parentFeeRecipient: parentGlobals.feeRecipient.toString(),
|
|
395
|
+
});
|
|
396
|
+
return { reason: 'global_variables_mismatch' };
|
|
397
|
+
}
|
|
398
|
+
|
|
399
|
+
if (!proposalGlobals.gasFees.equals(parentGlobals.gasFees)) {
|
|
400
|
+
this.log.warn(`Non-first block in checkpoint has mismatched gasFees`, {
|
|
401
|
+
...proposalInfo,
|
|
402
|
+
proposalGasFees: proposalGlobals.gasFees.toInspect(),
|
|
403
|
+
parentGasFees: parentGlobals.gasFees.toInspect(),
|
|
404
|
+
});
|
|
405
|
+
return { reason: 'global_variables_mismatch' };
|
|
406
|
+
}
|
|
407
|
+
|
|
408
|
+
return undefined;
|
|
409
|
+
}
|
|
410
|
+
|
|
250
411
|
private getReexecutionDeadline(slot: SlotNumber, config: { l1GenesisTime: bigint; slotDuration: number }): Date {
|
|
251
412
|
const nextSlotTimestampSeconds = Number(getTimestampForSlot(SlotNumber(slot + 1), config));
|
|
252
|
-
|
|
253
|
-
return new Date(nextSlotTimestampSeconds * 1000 - msNeededForPropagationAndPublishing);
|
|
413
|
+
return new Date(nextSlotTimestampSeconds * 1000);
|
|
254
414
|
}
|
|
255
415
|
|
|
256
416
|
private getReexecuteFailureReason(err: any) {
|
|
@@ -268,11 +428,12 @@ export class BlockProposalHandler {
|
|
|
268
428
|
async reexecuteTransactions(
|
|
269
429
|
proposal: BlockProposal,
|
|
270
430
|
blockNumber: BlockNumber,
|
|
431
|
+
checkpointNumber: CheckpointNumber,
|
|
271
432
|
txs: Tx[],
|
|
272
433
|
l1ToL2Messages: Fr[],
|
|
434
|
+
previousCheckpointOutHashes: Fr[],
|
|
273
435
|
): Promise<ReexecuteTransactionsResult> {
|
|
274
|
-
const {
|
|
275
|
-
const { txHashes } = proposal;
|
|
436
|
+
const { blockHeader, txHashes } = proposal;
|
|
276
437
|
|
|
277
438
|
// If we do not have all of the transactions, then we should fail
|
|
278
439
|
if (txs.length !== txHashes.length) {
|
|
@@ -281,28 +442,52 @@ export class BlockProposalHandler {
|
|
|
281
442
|
throw new TransactionsNotAvailableError(missingTxHashes);
|
|
282
443
|
}
|
|
283
444
|
|
|
284
|
-
// Use the sequencer's block building logic to re-execute the transactions
|
|
285
445
|
const timer = new Timer();
|
|
286
|
-
const
|
|
287
|
-
|
|
288
|
-
|
|
289
|
-
|
|
290
|
-
|
|
291
|
-
|
|
292
|
-
|
|
293
|
-
|
|
294
|
-
|
|
295
|
-
|
|
446
|
+
const slot = proposal.slotNumber;
|
|
447
|
+
const config = this.checkpointsBuilder.getConfig();
|
|
448
|
+
|
|
449
|
+
// Get prior blocks in this checkpoint (same slot before current block)
|
|
450
|
+
const allBlocksInSlot = await this.blockSource.getBlocksForSlot(slot);
|
|
451
|
+
const priorBlocks = allBlocksInSlot.filter(b => b.number < blockNumber && b.header.getSlot() === slot);
|
|
452
|
+
|
|
453
|
+
// Fork before the block to be built
|
|
454
|
+
const parentBlockNumber = BlockNumber(blockNumber - 1);
|
|
455
|
+
await this.worldState.syncImmediate(parentBlockNumber);
|
|
456
|
+
await using fork = await this.worldState.fork(parentBlockNumber);
|
|
457
|
+
|
|
458
|
+
// Build checkpoint constants from proposal (excludes blockNumber which is per-block)
|
|
459
|
+
const constants: CheckpointGlobalVariables = {
|
|
296
460
|
chainId: new Fr(config.l1ChainId),
|
|
297
461
|
version: new Fr(config.rollupVersion),
|
|
298
|
-
|
|
462
|
+
slotNumber: slot,
|
|
463
|
+
timestamp: blockHeader.globalVariables.timestamp,
|
|
464
|
+
coinbase: blockHeader.globalVariables.coinbase,
|
|
465
|
+
feeRecipient: blockHeader.globalVariables.feeRecipient,
|
|
466
|
+
gasFees: blockHeader.globalVariables.gasFees,
|
|
467
|
+
};
|
|
468
|
+
|
|
469
|
+
// Create checkpoint builder with prior blocks
|
|
470
|
+
const checkpointBuilder = await this.checkpointsBuilder.openCheckpoint(
|
|
471
|
+
checkpointNumber,
|
|
472
|
+
constants,
|
|
473
|
+
0n, // only takes effect in the following checkpoint.
|
|
474
|
+
l1ToL2Messages,
|
|
475
|
+
previousCheckpointOutHashes,
|
|
476
|
+
fork,
|
|
477
|
+
priorBlocks,
|
|
478
|
+
this.log.getBindings(),
|
|
479
|
+
);
|
|
299
480
|
|
|
300
|
-
|
|
301
|
-
|
|
481
|
+
// Build the new block
|
|
482
|
+
const deadline = this.getReexecutionDeadline(slot, config);
|
|
483
|
+
const result = await checkpointBuilder.buildBlock(txs, blockNumber, blockHeader.globalVariables.timestamp, {
|
|
484
|
+
deadline,
|
|
485
|
+
expectedEndState: blockHeader.state,
|
|
302
486
|
});
|
|
303
487
|
|
|
488
|
+
const { block, failedTxs } = result;
|
|
304
489
|
const numFailedTxs = failedTxs.length;
|
|
305
|
-
|
|
490
|
+
|
|
306
491
|
this.log.verbose(`Transaction re-execution complete for slot ${slot}`, {
|
|
307
492
|
numFailedTxs,
|
|
308
493
|
numProposalTxs: txHashes.length,
|
|
@@ -321,11 +506,15 @@ export class BlockProposalHandler {
|
|
|
321
506
|
}
|
|
322
507
|
|
|
323
508
|
// Throw a ReExStateMismatchError error if state updates do not match
|
|
324
|
-
|
|
325
|
-
|
|
509
|
+
// Compare the full block structure (archive and header) from the built block with the proposal
|
|
510
|
+
const archiveMatches = proposal.archive.equals(block.archive.root);
|
|
511
|
+
const headerMatches = proposal.blockHeader.equals(block.header);
|
|
512
|
+
if (!archiveMatches || !headerMatches) {
|
|
326
513
|
this.log.warn(`Re-execution state mismatch for slot ${slot}`, {
|
|
327
|
-
|
|
328
|
-
|
|
514
|
+
expectedArchive: block.archive.root.toString(),
|
|
515
|
+
actualArchive: proposal.archive.toString(),
|
|
516
|
+
expectedHeader: block.header.toInspect(),
|
|
517
|
+
actualHeader: proposal.blockHeader.toInspect(),
|
|
329
518
|
});
|
|
330
519
|
this.metrics?.recordFailedReexecution(proposal);
|
|
331
520
|
throw new ReExStateMismatchError(proposal.archive, block.archive.root);
|