@aztec/slasher 0.0.1-commit.9b94fc1 → 0.0.1-commit.9badcec54

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.
Files changed (73) hide show
  1. package/README.md +51 -65
  2. package/dest/config.d.ts +1 -1
  3. package/dest/config.d.ts.map +1 -1
  4. package/dest/config.js +28 -28
  5. package/dest/factory/create_facade.d.ts +5 -4
  6. package/dest/factory/create_facade.d.ts.map +1 -1
  7. package/dest/factory/create_facade.js +26 -3
  8. package/dest/factory/create_implementation.d.ts +7 -8
  9. package/dest/factory/create_implementation.d.ts.map +1 -1
  10. package/dest/factory/create_implementation.js +8 -56
  11. package/dest/factory/get_settings.d.ts +4 -4
  12. package/dest/factory/get_settings.d.ts.map +1 -1
  13. package/dest/factory/get_settings.js +3 -3
  14. package/dest/factory/index.d.ts +2 -2
  15. package/dest/factory/index.d.ts.map +1 -1
  16. package/dest/factory/index.js +1 -1
  17. package/dest/generated/slasher-defaults.d.ts +19 -0
  18. package/dest/generated/slasher-defaults.d.ts.map +1 -0
  19. package/dest/generated/slasher-defaults.js +19 -0
  20. package/dest/index.d.ts +2 -3
  21. package/dest/index.d.ts.map +1 -1
  22. package/dest/index.js +1 -2
  23. package/dest/null_slasher_client.d.ts +3 -4
  24. package/dest/null_slasher_client.d.ts.map +1 -1
  25. package/dest/null_slasher_client.js +1 -4
  26. package/dest/slash_offenses_collector.d.ts +5 -8
  27. package/dest/slash_offenses_collector.d.ts.map +1 -1
  28. package/dest/slash_offenses_collector.js +9 -18
  29. package/dest/slasher_client.d.ts +112 -0
  30. package/dest/slasher_client.d.ts.map +1 -0
  31. package/dest/{tally_slasher_client.js → slasher_client.js} +30 -39
  32. package/dest/slasher_client_facade.d.ts +7 -9
  33. package/dest/slasher_client_facade.d.ts.map +1 -1
  34. package/dest/slasher_client_facade.js +6 -9
  35. package/dest/slasher_client_interface.d.ts +7 -21
  36. package/dest/slasher_client_interface.d.ts.map +1 -1
  37. package/dest/slasher_client_interface.js +1 -4
  38. package/dest/stores/offenses_store.d.ts +6 -12
  39. package/dest/stores/offenses_store.d.ts.map +1 -1
  40. package/dest/stores/offenses_store.js +8 -25
  41. package/dest/watchers/attestations_block_watcher.d.ts +7 -6
  42. package/dest/watchers/attestations_block_watcher.d.ts.map +1 -1
  43. package/dest/watchers/attestations_block_watcher.js +40 -34
  44. package/dest/watchers/epoch_prune_watcher.d.ts +9 -7
  45. package/dest/watchers/epoch_prune_watcher.d.ts.map +1 -1
  46. package/dest/watchers/epoch_prune_watcher.js +59 -15
  47. package/package.json +16 -14
  48. package/src/config.ts +31 -28
  49. package/src/factory/create_facade.ts +35 -6
  50. package/src/factory/create_implementation.ts +25 -106
  51. package/src/factory/get_settings.ts +8 -8
  52. package/src/factory/index.ts +1 -1
  53. package/src/generated/slasher-defaults.ts +21 -0
  54. package/src/index.ts +1 -2
  55. package/src/null_slasher_client.ts +2 -6
  56. package/src/slash_offenses_collector.ts +16 -20
  57. package/src/{tally_slasher_client.ts → slasher_client.ts} +37 -48
  58. package/src/slasher_client_facade.ts +7 -12
  59. package/src/slasher_client_interface.ts +6 -21
  60. package/src/stores/offenses_store.ts +11 -34
  61. package/src/watcher.ts +1 -1
  62. package/src/watchers/attestations_block_watcher.ts +57 -44
  63. package/src/watchers/epoch_prune_watcher.ts +87 -24
  64. package/dest/empire_slasher_client.d.ts +0 -190
  65. package/dest/empire_slasher_client.d.ts.map +0 -1
  66. package/dest/empire_slasher_client.js +0 -572
  67. package/dest/stores/payloads_store.d.ts +0 -29
  68. package/dest/stores/payloads_store.d.ts.map +0 -1
  69. package/dest/stores/payloads_store.js +0 -125
  70. package/dest/tally_slasher_client.d.ts +0 -125
  71. package/dest/tally_slasher_client.d.ts.map +0 -1
  72. package/src/empire_slasher_client.ts +0 -657
  73. package/src/stores/payloads_store.ts +0 -146
@@ -1,11 +1,10 @@
1
1
  import { EthAddress } from '@aztec/aztec.js/addresses';
2
2
  import type { EpochCache } from '@aztec/epoch-cache';
3
- import { RollupContract, SlasherContract, TallySlashingProposerContract } from '@aztec/ethereum/contracts';
3
+ import { RollupContract, SlasherContract, SlashingProposerContract } from '@aztec/ethereum/contracts';
4
4
  import { maxBigint } from '@aztec/foundation/bigint';
5
5
  import { SlotNumber } from '@aztec/foundation/branded-types';
6
6
  import { compactArray, partition, times } from '@aztec/foundation/collection';
7
7
  import { createLogger } from '@aztec/foundation/log';
8
- import { sleep } from '@aztec/foundation/sleep';
9
8
  import type { DateProvider } from '@aztec/foundation/timer';
10
9
  import type { Prettify } from '@aztec/foundation/types';
11
10
  import type { SlasherConfig } from '@aztec/stdlib/interfaces/server';
@@ -14,7 +13,6 @@ import {
14
13
  OffenseType,
15
14
  type ProposerSlashAction,
16
15
  type ProposerSlashActionProvider,
17
- type SlashPayloadRound,
18
16
  getEpochsForRound,
19
17
  getSlashConsensusVotesFromOffenses,
20
18
  } from '@aztec/stdlib/slashing';
@@ -31,8 +29,8 @@ import type { SlasherClientInterface } from './slasher_client_interface.js';
31
29
  import type { SlasherOffensesStore } from './stores/offenses_store.js';
32
30
  import type { Watcher } from './watcher.js';
33
31
 
34
- /** Settings used in the tally slasher client, loaded from the L1 contracts during initialization */
35
- export type TallySlasherSettings = Prettify<
32
+ /** Settings used in the slasher client, loaded from the L1 contracts during initialization */
33
+ export type SlasherSettings = Prettify<
36
34
  SlashRoundMonitorSettings &
37
35
  SlashOffensesCollectorSettings & {
38
36
  slashingLifetimeInRounds: number;
@@ -46,11 +44,14 @@ export type TallySlasherSettings = Prettify<
46
44
  }
47
45
  >;
48
46
 
49
- export type TallySlasherClientConfig = SlashOffensesCollectorConfig &
50
- Pick<SlasherConfig, 'slashValidatorsAlways' | 'slashValidatorsNever' | 'slashExecuteRoundsLookBack'>;
47
+ export type SlasherClientConfig = SlashOffensesCollectorConfig &
48
+ Pick<
49
+ SlasherConfig,
50
+ 'slashValidatorsAlways' | 'slashValidatorsNever' | 'slashExecuteRoundsLookBack' | 'slashMaxPayloadSize'
51
+ >;
51
52
 
52
53
  /**
53
- * The Tally Slasher client is responsible for managing slashable offenses using
54
+ * The Slasher client is responsible for managing slashable offenses using
54
55
  * the consensus-based slashing model where proposers vote on individual validator offenses.
55
56
  *
56
57
  * The client subscribes to several slash watchers that emit offenses and tracks them. When the slasher is the
@@ -74,22 +75,16 @@ export type TallySlasherClientConfig = SlashOffensesCollectorConfig &
74
75
  * - Validators that reach the quorum threshold are slashed. A vote for slashing N units is also considered
75
76
  * a vote for slashing N-1, N-2, ..., 1 units. The system slashes for the largest amount that reaches quorum.
76
77
  * - The client monitors executable rounds and triggers execution when appropriate.
77
- *
78
- * Differences from Empire model
79
- * - No fixed slash payloads - votes are for individual validator offenses encoded in bytes
80
- * - The L1 contract determines which offenses reach quorum rather than nodes agreeing on a payload
81
- * - Proposers vote directly on which validators to slash and by how much
82
- * - Uses a slash offset to vote on validators from past rounds (e.g., round N votes on round N-2)
83
78
  */
84
- export class TallySlasherClient implements ProposerSlashActionProvider, SlasherClientInterface {
79
+ export class SlasherClient implements ProposerSlashActionProvider, SlasherClientInterface {
85
80
  protected unwatchCallbacks: (() => void)[] = [];
86
81
  protected roundMonitor: SlashRoundMonitor;
87
82
  protected offensesCollector: SlashOffensesCollector;
88
83
 
89
84
  constructor(
90
- private config: TallySlasherClientConfig,
91
- private settings: TallySlasherSettings,
92
- private tallySlashingProposer: TallySlashingProposerContract,
85
+ private config: SlasherClientConfig,
86
+ private settings: SlasherSettings,
87
+ private slashingProposer: SlashingProposerContract,
93
88
  private slasher: SlasherContract,
94
89
  private rollup: RollupContract,
95
90
  watchers: Watcher[],
@@ -103,14 +98,14 @@ export class TallySlasherClient implements ProposerSlashActionProvider, SlasherC
103
98
  }
104
99
 
105
100
  public async start() {
106
- this.log.debug('Starting Tally Slasher client...');
101
+ this.log.debug('Starting slasher client...');
107
102
 
108
103
  this.roundMonitor.start();
109
104
  await this.offensesCollector.start();
110
105
 
111
106
  // Listen for RoundExecuted events
112
107
  this.unwatchCallbacks.push(
113
- this.tallySlashingProposer.listenToRoundExecuted(
108
+ this.slashingProposer.listenToRoundExecuted(
114
109
  ({ round, slashCount, l1BlockHash }) =>
115
110
  void this.handleRoundExecuted(round, slashCount, l1BlockHash).catch(err =>
116
111
  this.log.error('Error handling round executed', err),
@@ -121,15 +116,13 @@ export class TallySlasherClient implements ProposerSlashActionProvider, SlasherC
121
116
  // Check for round changes
122
117
  this.unwatchCallbacks.push(this.roundMonitor.listenToNewRound(round => this.handleNewRound(round)));
123
118
 
124
- this.log.info(`Started tally slasher client`);
119
+ this.log.info(`Started slasher client`);
125
120
  return Promise.resolve();
126
121
  }
127
122
 
128
- /**
129
- * Stop the tally slasher client
130
- */
123
+ /** Stop the slasher client */
131
124
  public async stop() {
132
- this.log.debug('Stopping Tally Slasher client...');
125
+ this.log.debug('Stopping slasher client...');
133
126
 
134
127
  for (const unwatchCallback of this.unwatchCallbacks) {
135
128
  unwatchCallback();
@@ -138,9 +131,7 @@ export class TallySlasherClient implements ProposerSlashActionProvider, SlasherC
138
131
  this.roundMonitor.stop();
139
132
  await this.offensesCollector.stop();
140
133
 
141
- // Sleeping to sidestep viem issue with unwatching events
142
- await sleep(2000);
143
- this.log.info('Tally Slasher client stopped');
134
+ this.log.info('Slasher client stopped');
144
135
  }
145
136
 
146
137
  /** Returns the current config */
@@ -155,11 +146,11 @@ export class TallySlasherClient implements ProposerSlashActionProvider, SlasherC
155
146
 
156
147
  /** Triggered on a time basis when we enter a new slashing round. Clears expired offenses. */
157
148
  protected async handleNewRound(round: bigint) {
158
- this.log.info(`Starting new tally slashing round ${round}`);
149
+ this.log.info(`Starting new slashing round ${round}`);
159
150
  await this.offensesCollector.handleNewRound(round);
160
151
  }
161
152
 
162
- /** Called when we see a RoundExecuted event on the TallySlashingProposer (just for logging). */
153
+ /** Called when we see a RoundExecuted event on the SlashingProposer (just for logging). */
163
154
  protected async handleRoundExecuted(round: bigint, slashCount: bigint, l1BlockHash: Hex) {
164
155
  const slashes = await this.rollup.getSlashEvents(l1BlockHash);
165
156
  this.log.info(`Slashing round ${round} has been executed with ${slashCount} slashes`, { slashes });
@@ -240,7 +231,7 @@ export class TallySlasherClient implements ProposerSlashActionProvider, SlasherC
240
231
  this.log.debug(`Testing if slashing round ${executableRound} is executable`, logData);
241
232
 
242
233
  try {
243
- const roundInfo = await this.tallySlashingProposer.getRound(executableRound);
234
+ const roundInfo = await this.slashingProposer.getRound(executableRound);
244
235
  logData = { ...logData, roundInfo };
245
236
  if (roundInfo.isExecuted) {
246
237
  this.log.verbose(`Round ${executableRound} has already been executed`, logData);
@@ -254,7 +245,7 @@ export class TallySlasherClient implements ProposerSlashActionProvider, SlasherC
254
245
  }
255
246
 
256
247
  // Check if round is ready to execute at the given slot
257
- const isReadyToExecute = await this.tallySlashingProposer.isRoundReadyToExecute(executableRound, slotNumber);
248
+ const isReadyToExecute = await this.slashingProposer.isRoundReadyToExecute(executableRound, slotNumber);
258
249
  if (!isReadyToExecute) {
259
250
  this.log.warn(
260
251
  `Round ${executableRound} is not ready to execute at slot ${slotNumber} according to contract check`,
@@ -264,14 +255,14 @@ export class TallySlasherClient implements ProposerSlashActionProvider, SlasherC
264
255
  }
265
256
 
266
257
  // Check if the round yields any slashing at all
267
- const { actions: slashActions, committees } = await this.tallySlashingProposer.getTally(executableRound);
258
+ const { actions: slashActions, committees } = await this.slashingProposer.getTally(executableRound);
268
259
  if (slashActions.length === 0) {
269
260
  this.log.verbose(`Round ${executableRound} does not resolve in any slashing`, logData);
270
261
  return undefined;
271
262
  }
272
263
 
273
264
  // Check if the slash payload is vetoed
274
- const payload = await this.tallySlashingProposer.getPayload(executableRound);
265
+ const payload = await this.slashingProposer.getPayload(executableRound);
275
266
  const isVetoed = await this.slasher.isPayloadVetoed(payload.address);
276
267
  if (isVetoed) {
277
268
  this.log.warn(`Round ${executableRound} payload is vetoed (skipping execution)`, {
@@ -281,8 +272,12 @@ export class TallySlasherClient implements ProposerSlashActionProvider, SlasherC
281
272
  return undefined;
282
273
  }
283
274
 
275
+ const slashActionsWithAmounts = slashActions.map(action => ({
276
+ validator: action.validator.toString(),
277
+ slashAmount: action.slashAmount.toString(),
278
+ }));
284
279
  this.log.info(`Round ${executableRound} is ready to execute with ${slashActions.length} slashes`, {
285
- slashActions,
280
+ slashActions: slashActionsWithAmounts,
286
281
  payloadAddress: payload.address.toString(),
287
282
  ...logData,
288
283
  });
@@ -357,11 +352,13 @@ export class TallySlasherClient implements ProposerSlashActionProvider, SlasherC
357
352
 
358
353
  const committees = await this.collectCommitteesActiveDuringRound(slashedRound);
359
354
  const epochsForCommittees = getEpochsForRound(slashedRound, this.settings);
355
+ const { slashMaxPayloadSize } = this.config;
360
356
  const votes = getSlashConsensusVotesFromOffenses(
361
357
  offensesToSlash,
362
358
  committees,
363
359
  epochsForCommittees.map(e => BigInt(e)),
364
- this.settings,
360
+ { ...this.settings, maxSlashedValidators: slashMaxPayloadSize },
361
+ this.log,
365
362
  );
366
363
  if (votes.every(v => v === 0)) {
367
364
  this.log.warn(`Computed votes for offenses are all zero. Skipping vote.`, {
@@ -399,17 +396,9 @@ export class TallySlasherClient implements ProposerSlashActionProvider, SlasherC
399
396
  );
400
397
  }
401
398
 
402
- /**
403
- * Get slash payloads is NOT SUPPORTED in tally model
404
- * @throws Error indicating this operation is not supported
405
- */
406
- public getSlashPayloads(): Promise<SlashPayloadRound[]> {
407
- return Promise.reject(new Error('Tally slashing model does not support slash payloads'));
408
- }
409
-
410
399
  /**
411
400
  * Gather offenses to be slashed on a given round.
412
- * In tally slashing, round N slashes validators from round N - slashOffsetInRounds.
401
+ * Round N slashes validators from round N - slashOffsetInRounds.
413
402
  * @param round - The round to get offenses for, defaults to current round
414
403
  * @returns Array of pending offenses for the round with offset applied
415
404
  */
@@ -422,9 +411,9 @@ export class TallySlasherClient implements ProposerSlashActionProvider, SlasherC
422
411
  return await this.offensesStore.getOffensesForRound(targetRound);
423
412
  }
424
413
 
425
- /** Returns all pending offenses stored */
426
- public getPendingOffenses(): Promise<Offense[]> {
427
- return this.offensesStore.getPendingOffenses();
414
+ /** Returns all offenses stored */
415
+ public getOffenses(): Promise<Offense[]> {
416
+ return this.offensesStore.getOffenses();
428
417
  }
429
418
 
430
419
  /**
@@ -1,14 +1,13 @@
1
1
  import { EpochCache } from '@aztec/epoch-cache';
2
- import type { ViemClient } from '@aztec/ethereum';
3
2
  import { RollupContract } from '@aztec/ethereum/contracts';
3
+ import type { ViemClient } from '@aztec/ethereum/types';
4
4
  import type { SlotNumber } from '@aztec/foundation/branded-types';
5
- import { EthAddress } from '@aztec/foundation/eth-address';
6
5
  import { createLogger } from '@aztec/foundation/log';
7
6
  import { DateProvider } from '@aztec/foundation/timer';
8
- import type { DataStoreConfig } from '@aztec/kv-store/config';
9
7
  import { AztecLMDBStoreV2 } from '@aztec/kv-store/lmdb-v2';
10
8
  import type { SlasherConfig } from '@aztec/stdlib/interfaces/server';
11
- import type { Offense, ProposerSlashAction, SlashPayloadRound } from '@aztec/stdlib/slashing';
9
+ import type { DataStoreConfig } from '@aztec/stdlib/kv-store';
10
+ import type { Offense, ProposerSlashAction } from '@aztec/stdlib/slashing';
12
11
 
13
12
  import { createSlasherImplementation } from './factory/create_implementation.js';
14
13
  import type { SlasherClientInterface } from './slasher_client_interface.js';
@@ -27,11 +26,11 @@ export class SlasherClientFacade implements SlasherClientInterface {
27
26
  private config: SlasherConfig & DataStoreConfig & { ethereumSlotDuration: number },
28
27
  private rollup: RollupContract,
29
28
  private l1Client: ViemClient,
30
- private slashFactoryAddress: EthAddress | undefined,
31
29
  private watchers: Watcher[],
32
30
  private epochCache: EpochCache,
33
31
  private dateProvider: DateProvider,
34
32
  private kvStore: AztecLMDBStoreV2,
33
+ private rollupRegisteredAtL2Slot: SlotNumber,
35
34
  private logger = createLogger('slasher'),
36
35
  ) {}
37
36
 
@@ -62,16 +61,12 @@ export class SlasherClientFacade implements SlasherClientInterface {
62
61
  this.watchers.forEach(watcher => watcher.updateConfig?.(config));
63
62
  }
64
63
 
65
- public getSlashPayloads(): Promise<SlashPayloadRound[]> {
66
- return this.client?.getSlashPayloads() ?? Promise.reject(new Error('Slasher client not initialized'));
67
- }
68
-
69
64
  public gatherOffensesForRound(round?: bigint): Promise<Offense[]> {
70
65
  return this.client?.gatherOffensesForRound(round) ?? Promise.reject(new Error('Slasher client not initialized'));
71
66
  }
72
67
 
73
- public getPendingOffenses(): Promise<Offense[]> {
74
- return this.client?.getPendingOffenses() ?? Promise.reject(new Error('Slasher client not initialized'));
68
+ public getOffenses(): Promise<Offense[]> {
69
+ return this.client?.getOffenses() ?? Promise.reject(new Error('Slasher client not initialized'));
75
70
  }
76
71
 
77
72
  public getProposerActions(slotNumber: SlotNumber): Promise<ProposerSlashAction[]> {
@@ -83,11 +78,11 @@ export class SlasherClientFacade implements SlasherClientInterface {
83
78
  this.config,
84
79
  this.rollup,
85
80
  this.l1Client,
86
- this.slashFactoryAddress,
87
81
  this.watchers,
88
82
  this.epochCache,
89
83
  this.dateProvider,
90
84
  this.kvStore,
85
+ this.rollupRegisteredAtL2Slot,
91
86
  this.logger,
92
87
  );
93
88
  }
@@ -1,11 +1,8 @@
1
1
  import type { SlotNumber } from '@aztec/foundation/branded-types';
2
2
  import type { SlasherConfig } from '@aztec/stdlib/interfaces/server';
3
- import type { Offense, ProposerSlashAction, SlashPayloadRound } from '@aztec/stdlib/slashing';
3
+ import type { Offense, ProposerSlashAction } from '@aztec/stdlib/slashing';
4
4
 
5
- /**
6
- * Common interface for slasher clients used by the Aztec node.
7
- * Both Empire and Consensus slasher clients implement this interface.
8
- */
5
+ /** Common interface for slasher clients used by the Aztec node. */
9
6
  export interface SlasherClientInterface {
10
7
  /** Start the slasher client */
11
8
  start(): Promise<void>;
@@ -13,25 +10,13 @@ export interface SlasherClientInterface {
13
10
  /** Stop the slasher client */
14
11
  stop(): Promise<void>;
15
12
 
16
- /**
17
- * Get slash payloads for the Empire model.
18
- * The Consensus model should throw an error when this is called.
19
- */
20
- getSlashPayloads(): Promise<SlashPayloadRound[]>;
21
-
22
- /**
23
- * Gather offenses for a given round, defaults to current.
24
- * Used by both Empire and Consensus models.
25
- */
13
+ /** Gather offenses for a given round, defaults to current. */
26
14
  gatherOffensesForRound(round?: bigint): Promise<Offense[]>;
27
15
 
28
- /** Returns all pending offenses */
29
- getPendingOffenses(): Promise<Offense[]>;
16
+ /** Returns all offenses */
17
+ getOffenses(): Promise<Offense[]>;
30
18
 
31
- /**
32
- * Update the configuration.
33
- * Used by both Empire and Consensus models.
34
- */
19
+ /** Update the configuration. */
35
20
  updateConfig(config: Partial<SlasherConfig>): void;
36
21
 
37
22
  /**
@@ -1,5 +1,5 @@
1
1
  import { createLogger } from '@aztec/aztec.js/log';
2
- import type { AztecAsyncKVStore, AztecAsyncMap, AztecAsyncMultiMap, AztecAsyncSet } from '@aztec/kv-store';
2
+ import type { AztecAsyncKVStore, AztecAsyncMap, AztecAsyncMultiMap } from '@aztec/kv-store';
3
3
  import {
4
4
  type Offense,
5
5
  type OffenseIdentifier,
@@ -14,10 +14,7 @@ export class SlasherOffensesStore {
14
14
  /** Map from offense key to offense data */
15
15
  private offenses: AztecAsyncMap<string, Buffer>;
16
16
 
17
- /** Map from offense key to whether the offense has been executed (only used for empire based slashing) */
18
- private offensesSlashed: AztecAsyncSet<string>;
19
-
20
- /** Multimap from round to offense keys (only used for consensus based slashing) */
17
+ /** Multimap from round to offense keys */
21
18
  private roundsOffenses: AztecAsyncMultiMap<string, string>;
22
19
 
23
20
  private log = createLogger('slasher:store:offenses');
@@ -32,18 +29,13 @@ export class SlasherOffensesStore {
32
29
  ) {
33
30
  this.offenses = kvStore.openMap('offenses');
34
31
  this.roundsOffenses = kvStore.openMultiMap('rounds-offenses');
35
- this.offensesSlashed = kvStore.openSet('offenses-slashed');
36
32
  }
37
33
 
38
- /** Returns all offenses not marked as slashed */
39
- public async getPendingOffenses(): Promise<Offense[]> {
34
+ /** Returns all offenses */
35
+ public async getOffenses(): Promise<Offense[]> {
40
36
  const offenses: Offense[] = [];
41
- for await (const [key, buffer] of this.offenses.entriesAsync()) {
42
- if (await this.offensesSlashed.hasAsync(key)) {
43
- continue; // Skip executed offenses
44
- }
45
- const offense = deserializeOffense(buffer);
46
- offenses.push(offense);
37
+ for await (const [, buffer] of this.offenses.entriesAsync()) {
38
+ offenses.push(deserializeOffense(buffer));
47
39
  }
48
40
  return offenses;
49
41
  }
@@ -61,35 +53,21 @@ export class SlasherOffensesStore {
61
53
  return offenses;
62
54
  }
63
55
 
64
- /** Returns whether an offense is pending (ie not marked as slashed) */
65
- public async hasPendingOffense(offense: OffenseIdentifier): Promise<boolean> {
66
- const key = this.getOffenseKey(offense);
67
- return (await this.offenses.getAsync(key)) !== undefined && !(await this.offensesSlashed.hasAsync(key));
68
- }
69
-
70
56
  /** Returns whether we have seen this offense */
71
57
  public async hasOffense(offense: OffenseIdentifier): Promise<boolean> {
72
58
  const key = this.getOffenseKey(offense);
73
59
  return (await this.offenses.getAsync(key)) !== undefined;
74
60
  }
75
61
 
76
- /** Adds a new offense (defaults to pending, but will be slashed if markAsSlashed had been called for it) */
77
- public async addPendingOffense(offense: Offense): Promise<void> {
62
+ /** Adds a new offense */
63
+ public async addOffense(offense: Offense): Promise<void> {
78
64
  const key = this.getOffenseKey(offense);
79
- await this.offenses.set(key, serializeOffense(offense));
80
65
  const round = getRoundForOffense(offense, this.settings);
81
- await this.roundsOffenses.set(this.getRoundKey(round), key);
82
- this.log.trace(`Adding pending offense ${key} for round ${round}`);
83
- }
84
-
85
- /** Marks the given offenses as slashed (regardless of whether they are known or not) */
86
- public async markAsSlashed(offenses: OffenseIdentifier[]): Promise<void> {
87
66
  await this.kvStore.transactionAsync(async () => {
88
- for (const offense of offenses) {
89
- const key = this.getOffenseKey(offense);
90
- await this.offensesSlashed.add(key);
91
- }
67
+ await this.offenses.set(key, serializeOffense(offense));
68
+ await this.roundsOffenses.set(this.getRoundKey(round), key);
92
69
  });
70
+ this.log.trace(`Adding pending offense ${key} for round ${round}`);
93
71
  }
94
72
 
95
73
  /** Prunes all offenses expired from the store */
@@ -123,7 +101,6 @@ export class SlasherOffensesStore {
123
101
  for (const key of expiredOffenseKeys) {
124
102
  this.log.trace(`Deleting offense ${key}`);
125
103
  await this.offenses.delete(key);
126
- await this.offensesSlashed.delete(key);
127
104
  }
128
105
  for (const roundKey of expiredRoundKeys) {
129
106
  this.log.trace(`Deleting round info for ${roundKey}`);
package/src/watcher.ts CHANGED
@@ -10,7 +10,7 @@ export interface WantToSlashArgs {
10
10
  validator: EthAddress;
11
11
  amount: bigint;
12
12
  offenseType: OffenseType;
13
- epochOrSlot: bigint; // Epoch number for epoch-based offenses, block number for block-based
13
+ epochOrSlot: bigint; // Epoch number for epoch-based offenses, slot number for slot-based
14
14
  }
15
15
 
16
16
  // Event map for specific, known events of a watcher
@@ -3,12 +3,12 @@ import { SlotNumber } from '@aztec/foundation/branded-types';
3
3
  import { merge, pick } from '@aztec/foundation/collection';
4
4
  import { type Logger, createLogger } from '@aztec/foundation/log';
5
5
  import {
6
- type InvalidBlockDetectedEvent,
7
- type L2BlockInfo,
6
+ type InvalidCheckpointDetectedEvent,
8
7
  type L2BlockSourceEventEmitter,
9
8
  L2BlockSourceEvents,
10
- type ValidateBlockNegativeResult,
9
+ type ValidateCheckpointNegativeResult,
11
10
  } from '@aztec/stdlib/block';
11
+ import type { CheckpointInfo } from '@aztec/stdlib/checkpoint';
12
12
  import { OffenseType } from '@aztec/stdlib/slashing';
13
13
 
14
14
  import EventEmitter from 'node:events';
@@ -32,19 +32,19 @@ type AttestationsBlockWatcherConfig = Pick<SlasherConfig, (typeof AttestationsBl
32
32
  export class AttestationsBlockWatcher extends (EventEmitter as new () => WatcherEmitter) implements Watcher {
33
33
  private log: Logger = createLogger('attestations-block-watcher');
34
34
 
35
- // Only keep track of the last N invalid blocks
36
- private maxInvalidBlocks = 100;
35
+ // Only keep track of the last N invalid checkpoints
36
+ private maxInvalidCheckpoints = 100;
37
37
 
38
38
  // All invalid archive roots seen
39
39
  private invalidArchiveRoots: Set<string> = new Set();
40
40
 
41
41
  private config: AttestationsBlockWatcherConfig;
42
42
 
43
- private boundHandleInvalidBlock = (event: InvalidBlockDetectedEvent) => {
43
+ private boundHandleInvalidCheckpoint = (event: InvalidCheckpointDetectedEvent) => {
44
44
  try {
45
- this.handleInvalidBlock(event);
45
+ this.handleInvalidCheckpoint(event);
46
46
  } catch (err) {
47
- this.log.error('Error handling invalid block', err, {
47
+ this.log.error('Error handling invalid checkpoint', err, {
48
48
  ...event.validationResult,
49
49
  reason: event.validationResult.reason,
50
50
  });
@@ -67,54 +67,61 @@ export class AttestationsBlockWatcher extends (EventEmitter as new () => Watcher
67
67
  }
68
68
 
69
69
  public start() {
70
- this.l2BlockSource.on(L2BlockSourceEvents.InvalidAttestationsBlockDetected, this.boundHandleInvalidBlock);
70
+ this.l2BlockSource.events.on(
71
+ L2BlockSourceEvents.InvalidAttestationsCheckpointDetected,
72
+ this.boundHandleInvalidCheckpoint,
73
+ );
71
74
  return Promise.resolve();
72
75
  }
73
76
 
74
77
  public stop() {
75
- this.l2BlockSource.removeListener(
76
- L2BlockSourceEvents.InvalidAttestationsBlockDetected,
77
- this.boundHandleInvalidBlock,
78
+ this.l2BlockSource.events.removeListener(
79
+ L2BlockSourceEvents.InvalidAttestationsCheckpointDetected,
80
+ this.boundHandleInvalidCheckpoint,
78
81
  );
79
82
  return Promise.resolve();
80
83
  }
81
84
 
82
- private handleInvalidBlock(event: InvalidBlockDetectedEvent): void {
85
+ /** Event handler for invalid checkpoints as reported by the archiver. Public for testing purposes. */
86
+ public handleInvalidCheckpoint(event: InvalidCheckpointDetectedEvent): void {
83
87
  const { validationResult } = event;
84
- const block = validationResult.block;
88
+ const checkpoint = validationResult.checkpoint;
85
89
 
86
- // Check if we already have processed this block, archiver may emit the same event multiple times
87
- if (this.invalidArchiveRoots.has(block.archive.toString())) {
88
- this.log.trace(`Already processed invalid block ${block.blockNumber}`);
90
+ // Check if we already have processed this checkpoint, archiver may emit the same event multiple times
91
+ if (this.invalidArchiveRoots.has(checkpoint.archive.toString())) {
92
+ this.log.trace(`Already processed invalid checkpoint ${checkpoint.checkpointNumber}`);
89
93
  return;
90
94
  }
91
95
 
92
- this.log.verbose(`Detected invalid block ${block.blockNumber}`, {
93
- ...block,
96
+ this.log.verbose(`Detected invalid checkpoint ${checkpoint.checkpointNumber}`, {
97
+ ...checkpoint,
94
98
  reason: validationResult.valid === false ? validationResult.reason : 'unknown',
95
99
  });
96
100
 
97
- // Store the invalid block
98
- this.addInvalidBlock(event.validationResult.block);
101
+ // Store the invalid checkpoint
102
+ this.addInvalidCheckpoint(event.validationResult.checkpoint);
99
103
 
100
- // Slash the proposer of the invalid block
104
+ // Slash the proposer of the invalid checkpoint
101
105
  this.slashProposer(event.validationResult);
102
106
 
103
- // Check if the parent of this block is invalid as well, if so, we will slash its attestors as well
107
+ // Check if the parent of this checkpoint is invalid as well, if so, we will slash its attestors as well
104
108
  this.slashAttestorsOnAncestorInvalid(event.validationResult);
105
109
  }
106
110
 
107
- private slashAttestorsOnAncestorInvalid(validationResult: ValidateBlockNegativeResult) {
108
- const block = validationResult.block;
111
+ private slashAttestorsOnAncestorInvalid(validationResult: ValidateCheckpointNegativeResult) {
112
+ const checkpoint = validationResult.checkpoint;
109
113
 
110
- const parentArchive = block.lastArchive.toString();
114
+ const parentArchive = checkpoint.lastArchive.toString();
111
115
  if (this.invalidArchiveRoots.has(parentArchive)) {
112
116
  const attestors = validationResult.attestors;
113
- this.log.info(`Want to slash attestors of block ${block.blockNumber} built on invalid block`, {
114
- ...block,
115
- ...attestors,
116
- parentArchive,
117
- });
117
+ this.log.info(
118
+ `Want to slash attestors of checkpoint ${checkpoint.checkpointNumber} built on invalid checkpoint`,
119
+ {
120
+ ...checkpoint,
121
+ ...attestors,
122
+ parentArchive,
123
+ },
124
+ );
118
125
 
119
126
  this.emit(
120
127
  WANT_TO_SLASH_EVENT,
@@ -122,20 +129,26 @@ export class AttestationsBlockWatcher extends (EventEmitter as new () => Watcher
122
129
  validator: attestor,
123
130
  amount: this.config.slashAttestDescendantOfInvalidPenalty,
124
131
  offenseType: OffenseType.ATTESTED_DESCENDANT_OF_INVALID,
125
- epochOrSlot: BigInt(SlotNumber(block.slotNumber)),
132
+ epochOrSlot: BigInt(SlotNumber(checkpoint.slotNumber)),
126
133
  })),
127
134
  );
128
135
  }
129
136
  }
130
137
 
131
- private slashProposer(validationResult: ValidateBlockNegativeResult) {
132
- const { reason, block } = validationResult;
133
- const blockNumber = block.blockNumber;
134
- const slot = SlotNumber(block.slotNumber);
135
- const proposer = this.epochCache.getProposerFromEpochCommittee(validationResult, slot);
138
+ private slashProposer(validationResult: ValidateCheckpointNegativeResult) {
139
+ const { reason, checkpoint } = validationResult;
140
+ const checkpointNumber = checkpoint.checkpointNumber;
141
+ const slot = checkpoint.slotNumber;
142
+ const epochCommitteeInfo = {
143
+ committee: validationResult.committee,
144
+ seed: validationResult.seed,
145
+ epoch: validationResult.epoch,
146
+ isEscapeHatchOpen: false,
147
+ };
148
+ const proposer = this.epochCache.getProposerFromEpochCommittee(epochCommitteeInfo, slot);
136
149
 
137
150
  if (!proposer) {
138
- this.log.warn(`No proposer found for block ${blockNumber} at slot ${slot}`);
151
+ this.log.warn(`No proposer found for checkpoint ${checkpointNumber} at slot ${slot}`);
139
152
  return;
140
153
  }
141
154
 
@@ -148,15 +161,15 @@ export class AttestationsBlockWatcher extends (EventEmitter as new () => Watcher
148
161
  epochOrSlot: BigInt(slot),
149
162
  };
150
163
 
151
- this.log.info(`Want to slash proposer of block ${blockNumber} due to ${reason}`, {
152
- ...block,
164
+ this.log.info(`Want to slash proposer of checkpoint ${checkpointNumber} due to ${reason}`, {
165
+ ...checkpoint,
153
166
  ...args,
154
167
  });
155
168
 
156
169
  this.emit(WANT_TO_SLASH_EVENT, [args]);
157
170
  }
158
171
 
159
- private getOffenseFromInvalidationReason(reason: ValidateBlockNegativeResult['reason']): OffenseType {
172
+ private getOffenseFromInvalidationReason(reason: ValidateCheckpointNegativeResult['reason']): OffenseType {
160
173
  switch (reason) {
161
174
  case 'invalid-attestation':
162
175
  return OffenseType.PROPOSED_INCORRECT_ATTESTATIONS;
@@ -169,11 +182,11 @@ export class AttestationsBlockWatcher extends (EventEmitter as new () => Watcher
169
182
  }
170
183
  }
171
184
 
172
- private addInvalidBlock(block: L2BlockInfo) {
173
- this.invalidArchiveRoots.add(block.archive.toString());
185
+ private addInvalidCheckpoint(checkpoint: CheckpointInfo) {
186
+ this.invalidArchiveRoots.add(checkpoint.archive.toString());
174
187
 
175
188
  // Prune old entries if we exceed the maximum
176
- if (this.invalidArchiveRoots.size > this.maxInvalidBlocks) {
189
+ if (this.invalidArchiveRoots.size > this.maxInvalidCheckpoints) {
177
190
  const oldestKey = this.invalidArchiveRoots.keys().next().value!;
178
191
  this.invalidArchiveRoots.delete(oldestKey);
179
192
  }