@aztec/validator-ha-signer 0.0.1-commit.ffe5b04ea → 0.0.1-commit.fff30aa

package/dest/metrics.d.ts

@@ -1,51 +0,0 @@
-import { type TelemetryClient } from '@aztec/telemetry-client';
-export type HACleanupType = 'stuck' | 'old' | 'outdated_rollup';
-/**
- * Metrics for HA signer tracking signing operations, lock acquisition, and cleanup.
- */
-export declare class HASignerMetrics {
-    private nodeId;
-    private signingDuration;
-    private signingSuccessCount;
-    private dutyAlreadySignedCount;
-    private slashingProtectionCount;
-    private signingErrorCount;
-    private lockAcquiredCount;
-    private cleanupStuckDutiesCount;
-    private cleanupOldDutiesCount;
-    private cleanupOutdatedRollupDutiesCount;
-    constructor(client: TelemetryClient, nodeId: string, name?: string);
-    /**
-     * Record a successful signing operation.
-     * @param dutyType - The type of duty signed
-     * @param durationMs - Duration from start of signWithProtection to completion
-     */
-    recordSigningSuccess(dutyType: string, durationMs: number): void;
-    /**
-     * Record a DutyAlreadySignedError (expected in HA; another node signed first).
-     * @param dutyType - The type of duty
-     */
-    recordDutyAlreadySigned(dutyType: string): void;
-    /**
-     * Record a SlashingProtectionError (attempted to sign different data for same duty).
-     * @param dutyType - The type of duty
-     */
-    recordSlashingProtection(dutyType: string): void;
-    /**
-     * Record a signing function failure (lock will be deleted for retry).
-     * @param dutyType - The type of duty
-     */
-    recordSigningError(dutyType: string): void;
-    /**
-     * Record lock acquisition.
-     * @param acquired - Whether a new lock was acquired (true) or existing record found (false)
-     */
-    recordLockAcquire(acquired: boolean): void;
-    /**
-     * Record cleanup metrics.
-     * @param type - Type of cleanup
-     * @param count - Number of duties cleaned up
-     */
-    recordCleanup(type: HACleanupType, count: number): void;
-}
-//# sourceMappingURL=data:application/json;base64,eyJ2ZXJzaW9uIjozLCJmaWxlIjoibWV0cmljcy5kLnRzIiwic291cmNlUm9vdCI6IiIsInNvdXJjZXMiOlsiLi4vc3JjL21ldHJpY3MudHMiXSwibmFtZXMiOltdLCJtYXBwaW5ncyI6IkFBQUEsT0FBTyxFQUlMLEtBQUssZUFBZSxFQUdyQixNQUFNLHlCQUF5QixDQUFDO0FBRWpDLE1BQU0sTUFBTSxhQUFhLEdBQUcsT0FBTyxHQUFHLEtBQUssR0FBRyxpQkFBaUIsQ0FBQztBQUVoRTs7R0FFRztBQUNILHFCQUFhLGVBQWU7SUFrQnhCLE9BQU8sQ0FBQyxNQUFNO0lBaEJoQixPQUFPLENBQUMsZUFBZSxDQUFZO0lBQ25DLE9BQU8sQ0FBQyxtQkFBbUIsQ0FBZ0I7SUFDM0MsT0FBTyxDQUFDLHNCQUFzQixDQUFnQjtJQUM5QyxPQUFPLENBQUMsdUJBQXVCLENBQWdCO0lBQy9DLE9BQU8sQ0FBQyxpQkFBaUIsQ0FBZ0I7SUFHekMsT0FBTyxDQUFDLGlCQUFpQixDQUFnQjtJQUd6QyxPQUFPLENBQUMsdUJBQXVCLENBQWdCO0lBQy9DLE9BQU8sQ0FBQyxxQkFBcUIsQ0FBZ0I7SUFDN0MsT0FBTyxDQUFDLGdDQUFnQyxDQUFnQjtJQUV4RCxZQUNFLE1BQU0sRUFBRSxlQUFlLEVBQ2YsTUFBTSxFQUFFLE1BQU0sRUFDdEIsSUFBSSxTQUFvQixFQXFCekI7SUFFRDs7OztPQUlHO0lBQ0ksb0JBQW9CLENBQUMsUUFBUSxFQUFFLE1BQU0sRUFBRSxVQUFVLEVBQUUsTUFBTSxHQUFHLElBQUksQ0FPdEU7SUFFRDs7O09BR0c7SUFDSSx1QkFBdUIsQ0FBQyxRQUFRLEVBQUUsTUFBTSxHQUFHLElBQUksQ0FNckQ7SUFFRDs7O09BR0c7SUFDSSx3QkFBd0IsQ0FBQyxRQUFRLEVBQUUsTUFBTSxHQUFHLElBQUksQ0FNdEQ7SUFFRDs7O09BR0c7SUFDSSxrQkFBa0IsQ0FBQyxRQUFRLEVBQUUsTUFBTSxHQUFHLElBQUksQ0FNaEQ7SUFFRDs7O09BR0c7SUFDSSxpQkFBaUIsQ0FBQyxRQUFRLEVBQUUsT0FBTyxHQUFHLElBQUksQ0FPaEQ7SUFFRDs7OztPQUlHO0lBQ0ksYUFBYSxDQUFDLElBQUksRUFBRSxhQUFhLEVBQUUsS0FBSyxFQUFFLE1BQU0sR0FBRyxJQUFJLENBWTdEO0NBQ0YifQ==

package/dest/metrics.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"metrics.d.ts","sourceRoot":"","sources":["../src/metrics.ts"],"names":[],"mappings":"AAAA,OAAO,EAIL,KAAK,eAAe,EAGrB,MAAM,yBAAyB,CAAC;AAEjC,MAAM,MAAM,aAAa,GAAG,OAAO,GAAG,KAAK,GAAG,iBAAiB,CAAC;AAEhE;;GAEG;AACH,qBAAa,eAAe;IAkBxB,OAAO,CAAC,MAAM;IAhBhB,OAAO,CAAC,eAAe,CAAY;IACnC,OAAO,CAAC,mBAAmB,CAAgB;IAC3C,OAAO,CAAC,sBAAsB,CAAgB;IAC9C,OAAO,CAAC,uBAAuB,CAAgB;IAC/C,OAAO,CAAC,iBAAiB,CAAgB;IAGzC,OAAO,CAAC,iBAAiB,CAAgB;IAGzC,OAAO,CAAC,uBAAuB,CAAgB;IAC/C,OAAO,CAAC,qBAAqB,CAAgB;IAC7C,OAAO,CAAC,gCAAgC,CAAgB;IAExD,YACE,MAAM,EAAE,eAAe,EACf,MAAM,EAAE,MAAM,EACtB,IAAI,SAAoB,EAqBzB;IAED;;;;OAIG;IACI,oBAAoB,CAAC,QAAQ,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,GAAG,IAAI,CAOtE;IAED;;;OAGG;IACI,uBAAuB,CAAC,QAAQ,EAAE,MAAM,GAAG,IAAI,CAMrD;IAED;;;OAGG;IACI,wBAAwB,CAAC,QAAQ,EAAE,MAAM,GAAG,IAAI,CAMtD;IAED;;;OAGG;IACI,kBAAkB,CAAC,QAAQ,EAAE,MAAM,GAAG,IAAI,CAMhD;IAED;;;OAGG;IACI,iBAAiB,CAAC,QAAQ,EAAE,OAAO,GAAG,IAAI,CAOhD;IAED;;;;OAIG;IACI,aAAa,CAAC,IAAI,EAAE,aAAa,EAAE,KAAK,EAAE,MAAM,GAAG,IAAI,CAY7D;CACF"}

package/dest/metrics.js

@@ -1,103 +0,0 @@
-import { Attributes, Metrics, createUpDownCounterWithDefault } from '@aztec/telemetry-client';
-/**
- * Metrics for HA signer tracking signing operations, lock acquisition, and cleanup.
- */ export class HASignerMetrics {
-    nodeId;
-    // Signing lifecycle metrics
-    signingDuration;
-    signingSuccessCount;
-    dutyAlreadySignedCount;
-    slashingProtectionCount;
-    signingErrorCount;
-    // Lock acquisition metrics
-    lockAcquiredCount;
-    // Cleanup metrics
-    cleanupStuckDutiesCount;
-    cleanupOldDutiesCount;
-    cleanupOutdatedRollupDutiesCount;
-    constructor(client, nodeId, name = 'HASignerMetrics'){
-        this.nodeId = nodeId;
-        const meter = client.getMeter(name);
-        // Signing lifecycle
-        this.signingDuration = meter.createHistogram(Metrics.HA_SIGNER_SIGNING_DURATION);
-        this.signingSuccessCount = createUpDownCounterWithDefault(meter, Metrics.HA_SIGNER_SIGNING_SUCCESS_COUNT);
-        this.dutyAlreadySignedCount = createUpDownCounterWithDefault(meter, Metrics.HA_SIGNER_DUTY_ALREADY_SIGNED_COUNT);
-        this.slashingProtectionCount = createUpDownCounterWithDefault(meter, Metrics.HA_SIGNER_SLASHING_PROTECTION_COUNT);
-        this.signingErrorCount = createUpDownCounterWithDefault(meter, Metrics.HA_SIGNER_SIGNING_ERROR_COUNT);
-        // Lock acquisition
-        this.lockAcquiredCount = createUpDownCounterWithDefault(meter, Metrics.HA_SIGNER_LOCK_ACQUIRED_COUNT);
-        // Cleanup
-        this.cleanupStuckDutiesCount = createUpDownCounterWithDefault(meter, Metrics.HA_SIGNER_CLEANUP_STUCK_DUTIES_COUNT);
-        this.cleanupOldDutiesCount = createUpDownCounterWithDefault(meter, Metrics.HA_SIGNER_CLEANUP_OLD_DUTIES_COUNT);
-        this.cleanupOutdatedRollupDutiesCount = createUpDownCounterWithDefault(meter, Metrics.HA_SIGNER_CLEANUP_OUTDATED_ROLLUP_DUTIES_COUNT);
-    }
-    /**
-   * Record a successful signing operation.
-   * @param dutyType - The type of duty signed
-   * @param durationMs - Duration from start of signWithProtection to completion
-   */ recordSigningSuccess(dutyType, durationMs) {
-        const attributes = {
-            [Attributes.HA_DUTY_TYPE]: dutyType,
-            [Attributes.HA_NODE_ID]: this.nodeId
-        };
-        this.signingSuccessCount.add(1, attributes);
-        this.signingDuration.record(durationMs, attributes);
-    }
-    /**
-   * Record a DutyAlreadySignedError (expected in HA; another node signed first).
-   * @param dutyType - The type of duty
-   */ recordDutyAlreadySigned(dutyType) {
-        const attributes = {
-            [Attributes.HA_DUTY_TYPE]: dutyType,
-            [Attributes.HA_NODE_ID]: this.nodeId
-        };
-        this.dutyAlreadySignedCount.add(1, attributes);
-    }
-    /**
-   * Record a SlashingProtectionError (attempted to sign different data for same duty).
-   * @param dutyType - The type of duty
-   */ recordSlashingProtection(dutyType) {
-        const attributes = {
-            [Attributes.HA_DUTY_TYPE]: dutyType,
-            [Attributes.HA_NODE_ID]: this.nodeId
-        };
-        this.slashingProtectionCount.add(1, attributes);
-    }
-    /**
-   * Record a signing function failure (lock will be deleted for retry).
-   * @param dutyType - The type of duty
-   */ recordSigningError(dutyType) {
-        const attributes = {
-            [Attributes.HA_DUTY_TYPE]: dutyType,
-            [Attributes.HA_NODE_ID]: this.nodeId
-        };
-        this.signingErrorCount.add(1, attributes);
-    }
-    /**
-   * Record lock acquisition.
-   * @param acquired - Whether a new lock was acquired (true) or existing record found (false)
-   */ recordLockAcquire(acquired) {
-        if (acquired) {
-            const attributes = {
-                [Attributes.HA_NODE_ID]: this.nodeId
-            };
-            this.lockAcquiredCount.add(1, attributes);
-        }
-    }
-    /**
-   * Record cleanup metrics.
-   * @param type - Type of cleanup
-   * @param count - Number of duties cleaned up
-   */ recordCleanup(type, count) {
-        const attributes = {
-            [Attributes.HA_NODE_ID]: this.nodeId
-        };
-        if (type === 'stuck') {
-            this.cleanupStuckDutiesCount.add(count, attributes);
-        } else if (type === 'old') {
-            this.cleanupOldDutiesCount.add(count, attributes);
-        } else if (type === 'outdated_rollup') {
-            this.cleanupOutdatedRollupDutiesCount.add(count, attributes);
-        }
-    }
-}

package/src/db/lmdb.ts

@@ -1,264 +0,0 @@
-/**
- * LMDB implementation of SlashingProtectionDatabase
- *
- * Provides local (single-node) double-signing protection using LMDB as the backend.
- * Suitable for nodes that do NOT run in a high-availability multi-node setup.
- *
- * The LMDB store is single-writer, making setIfNotExists inherently atomic.
- * This means we get crash-restart protection without needing an external database.
- */
-import { SlotNumber } from '@aztec/foundation/branded-types';
-import { randomBytes } from '@aztec/foundation/crypto/random';
-import { EthAddress } from '@aztec/foundation/eth-address';
-import { type Logger, createLogger } from '@aztec/foundation/log';
-import type { DateProvider } from '@aztec/foundation/timer';
-import type { AztecAsyncKVStore, AztecAsyncMap } from '@aztec/kv-store';
-
-import type { SlashingProtectionDatabase, TryInsertOrGetResult } from '../types.js';
-import {
-  type CheckAndRecordParams,
-  DutyStatus,
-  DutyType,
-  type StoredDutyRecord,
-  getBlockIndexFromDutyIdentifier,
-  recordFromFields,
-} from './types.js';
-
-function dutyKey(
-  rollupAddress: string,
-  validatorAddress: string,
-  slot: string,
-  dutyType: string,
-  blockIndexWithinCheckpoint: number,
-): string {
-  return `${rollupAddress}:${validatorAddress}:${slot}:${dutyType}:${blockIndexWithinCheckpoint}`;
-}
-
-/**
- * LMDB-backed implementation of SlashingProtectionDatabase.
- *
- * Provides single-node double-signing protection that survives crashes and restarts.
- * Does not provide cross-node coordination (that requires the PostgreSQL implementation).
- */
-export class LmdbSlashingProtectionDatabase implements SlashingProtectionDatabase {
-  public static readonly SCHEMA_VERSION = 1;
-
-  private readonly duties: AztecAsyncMap<string, StoredDutyRecord>;
-  private readonly log: Logger;
-
-  constructor(
-    private readonly store: AztecAsyncKVStore,
-    private readonly dateProvider: DateProvider,
-  ) {
-    this.log = createLogger('slashing-protection:lmdb');
-    this.duties = store.openMap<string, StoredDutyRecord>('signing-protection-duties');
-  }
-
-  /**
-   * Atomically try to insert a new duty record, or get the existing one if present.
-   *
-   * LMDB is single-writer so the read-then-write inside transactionAsync is naturally atomic.
-   */
-  public async tryInsertOrGetExisting(params: CheckAndRecordParams): Promise<TryInsertOrGetResult> {
-    const blockIndexWithinCheckpoint = getBlockIndexFromDutyIdentifier(params);
-    const key = dutyKey(
-      params.rollupAddress.toString(),
-      params.validatorAddress.toString(),
-      params.slot.toString(),
-      params.dutyType,
-      blockIndexWithinCheckpoint,
-    );
-
-    const lockToken = randomBytes(16).toString('hex');
-    const now = this.dateProvider.now();
-
-    const result = await this.store.transactionAsync(async () => {
-      const existing = await this.duties.getAsync(key);
-      if (existing) {
-        return { isNew: false as const, record: { ...existing, lockToken: '' } };
-      }
-
-      const newRecord: StoredDutyRecord = {
-        rollupAddress: params.rollupAddress.toString(),
-        validatorAddress: params.validatorAddress.toString(),
-        slot: params.slot.toString(),
-        blockNumber: params.blockNumber.toString(),
-        blockIndexWithinCheckpoint,
-        dutyType: params.dutyType,
-        status: DutyStatus.SIGNING,
-        messageHash: params.messageHash,
-        nodeId: params.nodeId,
-        lockToken,
-        startedAtMs: now,
-      };
-      await this.duties.set(key, newRecord);
-      return { isNew: true as const, record: newRecord };
-    });
-
-    if (result.isNew) {
-      this.log.debug(`Acquired lock for duty ${params.dutyType} at slot ${params.slot}`, {
-        validatorAddress: params.validatorAddress.toString(),
-        nodeId: params.nodeId,
-      });
-    }
-
-    return { isNew: result.isNew, record: recordFromFields(result.record) };
-  }
-
-  /**
-   * Update a duty to 'signed' status with the signature.
-   * Only succeeds if the lockToken matches.
-   */
-  public updateDutySigned(
-    rollupAddress: EthAddress,
-    validatorAddress: EthAddress,
-    slot: SlotNumber,
-    dutyType: DutyType,
-    signature: string,
-    lockToken: string,
-    blockIndexWithinCheckpoint: number,
-  ): Promise<boolean> {
-    const key = dutyKey(
-      rollupAddress.toString(),
-      validatorAddress.toString(),
-      slot.toString(),
-      dutyType,
-      blockIndexWithinCheckpoint,
-    );
-
-    return this.store.transactionAsync(async () => {
-      const existing = await this.duties.getAsync(key);
-      if (!existing) {
-        this.log.warn('Failed to update duty to signed: duty not found', {
-          rollupAddress: rollupAddress.toString(),
-          validatorAddress: validatorAddress.toString(),
-          slot: slot.toString(),
-          dutyType,
-          blockIndexWithinCheckpoint,
-        });
-        return false;
-      }
-
-      if (existing.lockToken !== lockToken) {
-        this.log.warn('Failed to update duty to signed: invalid token', {
-          rollupAddress: rollupAddress.toString(),
-          validatorAddress: validatorAddress.toString(),
-          slot: slot.toString(),
-          dutyType,
-          blockIndexWithinCheckpoint,
-        });
-        return false;
-      }
-
-      await this.duties.set(key, {
-        ...existing,
-        status: DutyStatus.SIGNED,
-        signature,
-        completedAtMs: this.dateProvider.now(),
-      });
-
-      return true;
-    });
-  }
-
-  /**
-   * Delete a duty record.
-   * Only succeeds if the lockToken matches.
-   */
-  public deleteDuty(
-    rollupAddress: EthAddress,
-    validatorAddress: EthAddress,
-    slot: SlotNumber,
-    dutyType: DutyType,
-    lockToken: string,
-    blockIndexWithinCheckpoint: number,
-  ): Promise<boolean> {
-    const key = dutyKey(
-      rollupAddress.toString(),
-      validatorAddress.toString(),
-      slot.toString(),
-      dutyType,
-      blockIndexWithinCheckpoint,
-    );
-
-    return this.store.transactionAsync(async () => {
-      const existing = await this.duties.getAsync(key);
-      if (!existing || existing.lockToken !== lockToken) {
-        this.log.warn('Failed to delete duty: invalid token or duty not found', {
-          rollupAddress: rollupAddress.toString(),
-          validatorAddress: validatorAddress.toString(),
-          slot: slot.toString(),
-          dutyType,
-          blockIndexWithinCheckpoint,
-        });
-        return false;
-      }
-
-      await this.duties.delete(key);
-      return true;
-    });
-  }
-
-  /**
-   * Cleanup own stuck duties (SIGNING status older than maxAgeMs).
-   */
-  public cleanupOwnStuckDuties(nodeId: string, maxAgeMs: number): Promise<number> {
-    const cutoffMs = this.dateProvider.now() - maxAgeMs;
-
-    return this.store.transactionAsync(async () => {
-      const keysToDelete: string[] = [];
-      for await (const [key, record] of this.duties.entriesAsync()) {
-        if (record.nodeId === nodeId && record.status === DutyStatus.SIGNING && record.startedAtMs < cutoffMs) {
-          keysToDelete.push(key);
-        }
-      }
-      for (const key of keysToDelete) {
-        await this.duties.delete(key);
-      }
-      return keysToDelete.length;
-    });
-  }
-
-  /**
-   * Cleanup duties with outdated rollup address.
-   *
-   * This is always a no-op for the LMDB implementation: the underlying store is created via
-   * DatabaseVersionManager (in factory.ts), which already resets the entire data directory at
-   * startup whenever the rollup address changes.
-   */
-  public cleanupOutdatedRollupDuties(_currentRollupAddress: EthAddress): Promise<number> {
-    return Promise.resolve(0);
-  }
-
-  /**
-   * Cleanup old signed duties older than maxAgeMs.
-   */
-  public cleanupOldDuties(maxAgeMs: number): Promise<number> {
-    const cutoffMs = this.dateProvider.now() - maxAgeMs;
-
-    return this.store.transactionAsync(async () => {
-      const keysToDelete: string[] = [];
-      for await (const [key, record] of this.duties.entriesAsync()) {
-        if (
-          record.status === DutyStatus.SIGNED &&
-          record.completedAtMs !== undefined &&
-          record.completedAtMs < cutoffMs
-        ) {
-          keysToDelete.push(key);
-        }
-      }
-      for (const key of keysToDelete) {
-        await this.duties.delete(key);
-      }
-      return keysToDelete.length;
-    });
-  }
-
-  /**
-   * Close the underlying LMDB store.
-   */
-  public async close(): Promise<void> {
-    await this.store.close();
-    this.log.debug('LMDB slashing protection database closed');
-  }
-}

package/src/metrics.ts

@@ -1,138 +0,0 @@
-import {
-  Attributes,
-  type Histogram,
-  Metrics,
-  type TelemetryClient,
-  type UpDownCounter,
-  createUpDownCounterWithDefault,
-} from '@aztec/telemetry-client';
-
-export type HACleanupType = 'stuck' | 'old' | 'outdated_rollup';
-
-/**
- * Metrics for HA signer tracking signing operations, lock acquisition, and cleanup.
- */
-export class HASignerMetrics {
-  // Signing lifecycle metrics
-  private signingDuration: Histogram;
-  private signingSuccessCount: UpDownCounter;
-  private dutyAlreadySignedCount: UpDownCounter;
-  private slashingProtectionCount: UpDownCounter;
-  private signingErrorCount: UpDownCounter;
-
-  // Lock acquisition metrics
-  private lockAcquiredCount: UpDownCounter;
-
-  // Cleanup metrics
-  private cleanupStuckDutiesCount: UpDownCounter;
-  private cleanupOldDutiesCount: UpDownCounter;
-  private cleanupOutdatedRollupDutiesCount: UpDownCounter;
-
-  constructor(
-    client: TelemetryClient,
-    private nodeId: string,
-    name = 'HASignerMetrics',
-  ) {
-    const meter = client.getMeter(name);
-
-    // Signing lifecycle
-    this.signingDuration = meter.createHistogram(Metrics.HA_SIGNER_SIGNING_DURATION);
-    this.signingSuccessCount = createUpDownCounterWithDefault(meter, Metrics.HA_SIGNER_SIGNING_SUCCESS_COUNT);
-    this.dutyAlreadySignedCount = createUpDownCounterWithDefault(meter, Metrics.HA_SIGNER_DUTY_ALREADY_SIGNED_COUNT);
-    this.slashingProtectionCount = createUpDownCounterWithDefault(meter, Metrics.HA_SIGNER_SLASHING_PROTECTION_COUNT);
-    this.signingErrorCount = createUpDownCounterWithDefault(meter, Metrics.HA_SIGNER_SIGNING_ERROR_COUNT);
-
-    // Lock acquisition
-    this.lockAcquiredCount = createUpDownCounterWithDefault(meter, Metrics.HA_SIGNER_LOCK_ACQUIRED_COUNT);
-
-    // Cleanup
-    this.cleanupStuckDutiesCount = createUpDownCounterWithDefault(meter, Metrics.HA_SIGNER_CLEANUP_STUCK_DUTIES_COUNT);
-    this.cleanupOldDutiesCount = createUpDownCounterWithDefault(meter, Metrics.HA_SIGNER_CLEANUP_OLD_DUTIES_COUNT);
-    this.cleanupOutdatedRollupDutiesCount = createUpDownCounterWithDefault(
-      meter,
-      Metrics.HA_SIGNER_CLEANUP_OUTDATED_ROLLUP_DUTIES_COUNT,
-    );
-  }
-
-  /**
-   * Record a successful signing operation.
-   * @param dutyType - The type of duty signed
-   * @param durationMs - Duration from start of signWithProtection to completion
-   */
-  public recordSigningSuccess(dutyType: string, durationMs: number): void {
-    const attributes = {
-      [Attributes.HA_DUTY_TYPE]: dutyType,
-      [Attributes.HA_NODE_ID]: this.nodeId,
-    };
-    this.signingSuccessCount.add(1, attributes);
-    this.signingDuration.record(durationMs, attributes);
-  }
-
-  /**
-   * Record a DutyAlreadySignedError (expected in HA; another node signed first).
-   * @param dutyType - The type of duty
-   */
-  public recordDutyAlreadySigned(dutyType: string): void {
-    const attributes = {
-      [Attributes.HA_DUTY_TYPE]: dutyType,
-      [Attributes.HA_NODE_ID]: this.nodeId,
-    };
-    this.dutyAlreadySignedCount.add(1, attributes);
-  }
-
-  /**
-   * Record a SlashingProtectionError (attempted to sign different data for same duty).
-   * @param dutyType - The type of duty
-   */
-  public recordSlashingProtection(dutyType: string): void {
-    const attributes = {
-      [Attributes.HA_DUTY_TYPE]: dutyType,
-      [Attributes.HA_NODE_ID]: this.nodeId,
-    };
-    this.slashingProtectionCount.add(1, attributes);
-  }
-
-  /**
-   * Record a signing function failure (lock will be deleted for retry).
-   * @param dutyType - The type of duty
-   */
-  public recordSigningError(dutyType: string): void {
-    const attributes = {
-      [Attributes.HA_DUTY_TYPE]: dutyType,
-      [Attributes.HA_NODE_ID]: this.nodeId,
-    };
-    this.signingErrorCount.add(1, attributes);
-  }
-
-  /**
-   * Record lock acquisition.
-   * @param acquired - Whether a new lock was acquired (true) or existing record found (false)
-   */
-  public recordLockAcquire(acquired: boolean): void {
-    if (acquired) {
-      const attributes = {
-        [Attributes.HA_NODE_ID]: this.nodeId,
-      };
-      this.lockAcquiredCount.add(1, attributes);
-    }
-  }
-
-  /**
-   * Record cleanup metrics.
-   * @param type - Type of cleanup
-   * @param count - Number of duties cleaned up
-   */
-  public recordCleanup(type: HACleanupType, count: number): void {
-    const attributes = {
-      [Attributes.HA_NODE_ID]: this.nodeId,
-    };
-
-    if (type === 'stuck') {
-      this.cleanupStuckDutiesCount.add(count, attributes);
-    } else if (type === 'old') {
-      this.cleanupOldDutiesCount.add(count, attributes);
-    } else if (type === 'outdated_rollup') {
-      this.cleanupOutdatedRollupDutiesCount.add(count, attributes);
-    }
-  }
-}