@lodestar/beacon-node 1.35.0-dev.21ba2777a9 → 1.35.0-dev.36a31f3a2f

package/src/chain/opPools/opPool.ts

@@ -1,462 +0,0 @@
-import {Id, Repository} from "@lodestar/db";
-import {
-  BLS_WITHDRAWAL_PREFIX,
-  ForkName,
-  ForkSeq,
-  MAX_ATTESTER_SLASHINGS,
-  MAX_ATTESTER_SLASHINGS_ELECTRA,
-  MAX_BLS_TO_EXECUTION_CHANGES,
-  MAX_PROPOSER_SLASHINGS,
-  MAX_VOLUNTARY_EXITS,
-} from "@lodestar/params";
-import {
-  CachedBeaconStateAllForks,
-  computeEpochAtSlot,
-  computeStartSlotAtEpoch,
-  getAttesterSlashableIndices,
-  isValidVoluntaryExit,
-} from "@lodestar/state-transition";
-import {
-  AttesterSlashing,
-  Epoch,
-  SignedBeaconBlock,
-  ValidatorIndex,
-  capella,
-  phase0,
-  sszTypesFor,
-} from "@lodestar/types";
-import {fromHex, toHex, toRootHex} from "@lodestar/utils";
-import {IBeaconDb} from "../../db/index.js";
-import {Metrics} from "../../metrics/metrics.js";
-import {SignedBLSToExecutionChangeVersioned} from "../../util/types.js";
-import {BlockType} from "../interface.js";
-import {BlockProductionStep} from "../produceBlock/produceBlockBody.js";
-import {isValidBlsToExecutionChangeForBlockInclusion} from "./utils.js";
-
-type HexRoot = string;
-type AttesterSlashingCached = {
-  attesterSlashing: AttesterSlashing;
-  intersectingIndices: number[];
-};
-
-export class OpPool {
-  /** Map of uniqueId(AttesterSlashing) -> AttesterSlashing */
-  private readonly attesterSlashings = new Map<HexRoot, AttesterSlashingCached>();
-  /** Map of to slash validator index -> ProposerSlashing */
-  private readonly proposerSlashings = new Map<ValidatorIndex, phase0.ProposerSlashing>();
-  /** Map of to exit validator index -> SignedVoluntaryExit */
-  private readonly voluntaryExits = new Map<ValidatorIndex, phase0.SignedVoluntaryExit>();
-  /** Set of seen attester slashing indexes. No need to prune */
-  private readonly attesterSlashingIndexes = new Set<ValidatorIndex>();
-  /** Map of validator index -> SignedBLSToExecutionChange */
-  private readonly blsToExecutionChanges = new Map<ValidatorIndex, SignedBLSToExecutionChangeVersioned>();
-
-  // Getters for metrics
-
-  get attesterSlashingsSize(): number {
-    return this.attesterSlashings.size;
-  }
-  get proposerSlashingsSize(): number {
-    return this.proposerSlashings.size;
-  }
-  get voluntaryExitsSize(): number {
-    return this.voluntaryExits.size;
-  }
-  get blsToExecutionChangeSize(): number {
-    return this.blsToExecutionChanges.size;
-  }
-
-  async fromPersisted(db: IBeaconDb): Promise<void> {
-    const [attesterSlashings, proposerSlashings, voluntaryExits, blsToExecutionChanges] = await Promise.all([
-      db.attesterSlashing.entries(),
-      db.proposerSlashing.values(),
-      db.voluntaryExit.values(),
-      db.blsToExecutionChange.values(),
-    ]);
-
-    for (const attesterSlashing of attesterSlashings) {
-      this.insertAttesterSlashing(ForkName.electra, attesterSlashing.value, attesterSlashing.key);
-    }
-    for (const proposerSlashing of proposerSlashings) {
-      this.insertProposerSlashing(proposerSlashing);
-    }
-    for (const voluntaryExit of voluntaryExits) {
-      this.insertVoluntaryExit(voluntaryExit);
-    }
-    for (const item of blsToExecutionChanges) {
-      this.insertBlsToExecutionChange(item.data, item.preCapella);
-    }
-  }
-
-  async toPersisted(db: IBeaconDb): Promise<void> {
-    await Promise.all([
-      persistDiff(
-        db.attesterSlashing,
-        Array.from(this.attesterSlashings.entries()).map(([key, value]) => ({
-          key: fromHex(key),
-          value: value.attesterSlashing,
-        })),
-        toHex
-      ),
-      persistDiff(
-        db.proposerSlashing,
-        Array.from(this.proposerSlashings.entries()).map(([key, value]) => ({key, value})),
-        (index) => index
-      ),
-      persistDiff(
-        db.voluntaryExit,
-        Array.from(this.voluntaryExits.entries()).map(([key, value]) => ({key, value})),
-        (index) => index
-      ),
-      persistDiff(
-        db.blsToExecutionChange,
-        Array.from(this.blsToExecutionChanges.entries()).map(([key, value]) => ({key, value})),
-        (index) => index
-      ),
-    ]);
-  }
-
-  // Use the opPool as seen cache for gossip validation
-
-  /** Returns false if at least one intersecting index has not been seen yet */
-  hasSeenAttesterSlashing(intersectingIndices: ValidatorIndex[]): boolean {
-    for (const validatorIndex of intersectingIndices) {
-      if (!this.attesterSlashingIndexes.has(validatorIndex)) {
-        return false;
-      }
-    }
-    return true;
-  }
-
-  hasSeenVoluntaryExit(validatorIndex: ValidatorIndex): boolean {
-    return this.voluntaryExits.has(validatorIndex);
-  }
-
-  hasSeenBlsToExecutionChange(validatorIndex: ValidatorIndex): boolean {
-    return this.blsToExecutionChanges.has(validatorIndex);
-  }
-
-  hasSeenProposerSlashing(validatorIndex: ValidatorIndex): boolean {
-    return this.proposerSlashings.has(validatorIndex);
-  }
-
-  /** Must be validated beforehand */
-  insertAttesterSlashing(fork: ForkName, attesterSlashing: AttesterSlashing, rootHash?: Uint8Array): void {
-    if (!rootHash) {
-      rootHash = sszTypesFor(fork).AttesterSlashing.hashTreeRoot(attesterSlashing);
-    }
-
-    // TODO: Do once and cache attached to the AttesterSlashing object
-    const intersectingIndices = getAttesterSlashableIndices(attesterSlashing);
-    this.attesterSlashings.set(toRootHex(rootHash), {
-      attesterSlashing,
-      intersectingIndices,
-    });
-    for (const index of intersectingIndices) {
-      this.attesterSlashingIndexes.add(index);
-    }
-  }
-
-  /** Must be validated beforehand */
-  insertProposerSlashing(proposerSlashing: phase0.ProposerSlashing): void {
-    this.proposerSlashings.set(proposerSlashing.signedHeader1.message.proposerIndex, proposerSlashing);
-  }
-
-  /** Must be validated beforehand */
-  insertVoluntaryExit(voluntaryExit: phase0.SignedVoluntaryExit): void {
-    this.voluntaryExits.set(voluntaryExit.message.validatorIndex, voluntaryExit);
-  }
-
-  /** Must be validated beforehand */
-  insertBlsToExecutionChange(blsToExecutionChange: capella.SignedBLSToExecutionChange, preCapella = false): void {
-    this.blsToExecutionChanges.set(blsToExecutionChange.message.validatorIndex, {
-      data: blsToExecutionChange,
-      preCapella,
-    });
-  }
-
-  /**
-   * Get proposer and attester slashings and voluntary exits and bls to execution change for inclusion in a block.
-   *
-   * This function computes both types of slashings and exits, because attester slashings and exits may be invalidated by
-   * slashings included earlier in the block.
-   */
-  getSlashingsAndExits(
-    state: CachedBeaconStateAllForks,
-    blockType: BlockType,
-    metrics: Metrics | null
-  ): [
-    AttesterSlashing[],
-    phase0.ProposerSlashing[],
-    phase0.SignedVoluntaryExit[],
-    capella.SignedBLSToExecutionChange[],
-  ] {
-    const {config} = state;
-    const stateEpoch = computeEpochAtSlot(state.slot);
-    const stateFork = config.getForkSeq(state.slot);
-    const toBeSlashedIndices = new Set<ValidatorIndex>();
-    const proposerSlashings: phase0.ProposerSlashing[] = [];
-
-    const stepsMetrics =
-      blockType === BlockType.Full
-        ? metrics?.executionBlockProductionTimeSteps
-        : metrics?.builderBlockProductionTimeSteps;
-
-    const endProposerSlashing = stepsMetrics?.startTimer();
-    for (const proposerSlashing of this.proposerSlashings.values()) {
-      const index = proposerSlashing.signedHeader1.message.proposerIndex;
-      const validator = state.validators.getReadonly(index);
-      if (!validator.slashed && validator.activationEpoch <= stateEpoch && stateEpoch < validator.withdrawableEpoch) {
-        proposerSlashings.push(proposerSlashing);
-        // Set of validators to be slashed, so we don't attempt to construct invalid attester slashings.
-        toBeSlashedIndices.add(index);
-        if (proposerSlashings.length >= MAX_PROPOSER_SLASHINGS) {
-          break;
-        }
-      }
-    }
-    endProposerSlashing?.({
-      step: BlockProductionStep.proposerSlashing,
-    });
-
-    const endAttesterSlashings = stepsMetrics?.startTimer();
-    const attesterSlashings: AttesterSlashing[] = [];
-    const maxAttesterSlashings = stateFork >= ForkSeq.electra ? MAX_ATTESTER_SLASHINGS_ELECTRA : MAX_ATTESTER_SLASHINGS;
-    attesterSlashing: for (const attesterSlashing of this.attesterSlashings.values()) {
-      /** Indices slashable in this attester slashing */
-      const slashableIndices = new Set<ValidatorIndex>();
-      for (let i = 0; i < attesterSlashing.intersectingIndices.length; i++) {
-        const index = attesterSlashing.intersectingIndices[i];
-        // If we already have a slashing for this index, we can continue on to the next slashing
-        if (toBeSlashedIndices.has(index)) {
-          continue attesterSlashing;
-        }
-
-        const validator = state.validators.getReadonly(index);
-        if (isSlashableAtEpoch(validator, stateEpoch)) {
-          slashableIndices.add(index);
-        }
-        if (attesterSlashings.length >= maxAttesterSlashings) {
-          break attesterSlashing;
-        }
-      }
-
-      // If there were slashable indices in this slashing
-      // Then include the slashing and count the slashable indices
-      if (slashableIndices.size > 0) {
-        attesterSlashings.push(attesterSlashing.attesterSlashing);
-        for (const index of slashableIndices) {
-          toBeSlashedIndices.add(index);
-        }
-      }
-    }
-    endAttesterSlashings?.({
-      step: BlockProductionStep.attesterSlashings,
-    });
-
-    const endVoluntaryExits = stepsMetrics?.startTimer();
-    const voluntaryExits: phase0.SignedVoluntaryExit[] = [];
-    for (const voluntaryExit of this.voluntaryExits.values()) {
-      if (
-        !toBeSlashedIndices.has(voluntaryExit.message.validatorIndex) &&
-        isValidVoluntaryExit(stateFork, state, voluntaryExit, false) &&
-        // Signature validation is skipped in `isValidVoluntaryExit(,,false)` since it was already validated in gossip
-        // However we must make sure that the signature fork is the same, or it will become invalid if included through
-        // a future fork.
-        isVoluntaryExitSignatureIncludable(
-          stateFork,
-          config.getForkSeq(computeStartSlotAtEpoch(voluntaryExit.message.epoch))
-        )
-      ) {
-        voluntaryExits.push(voluntaryExit);
-        if (voluntaryExits.length >= MAX_VOLUNTARY_EXITS) {
-          break;
-        }
-      }
-    }
-    endVoluntaryExits?.({
-      step: BlockProductionStep.voluntaryExits,
-    });
-
-    const endBlsToExecutionChanges = stepsMetrics?.startTimer();
-    const blsToExecutionChanges: capella.SignedBLSToExecutionChange[] = [];
-    for (const blsToExecutionChange of this.blsToExecutionChanges.values()) {
-      if (isValidBlsToExecutionChangeForBlockInclusion(state, blsToExecutionChange.data)) {
-        blsToExecutionChanges.push(blsToExecutionChange.data);
-        if (blsToExecutionChanges.length >= MAX_BLS_TO_EXECUTION_CHANGES) {
-          break;
-        }
-      }
-    }
-    endBlsToExecutionChanges?.({
-      step: BlockProductionStep.blsToExecutionChanges,
-    });
-
-    return [attesterSlashings, proposerSlashings, voluntaryExits, blsToExecutionChanges];
-  }
-
-  /** For beacon pool API */
-  getAllAttesterSlashings(): AttesterSlashing[] {
-    return Array.from(this.attesterSlashings.values()).map((attesterSlashings) => attesterSlashings.attesterSlashing);
-  }
-
-  /** For beacon pool API */
-  getAllProposerSlashings(): phase0.ProposerSlashing[] {
-    return Array.from(this.proposerSlashings.values());
-  }
-
-  /** For beacon pool API */
-  getAllVoluntaryExits(): phase0.SignedVoluntaryExit[] {
-    return Array.from(this.voluntaryExits.values());
-  }
-
-  /** For beacon pool API */
-  getAllBlsToExecutionChanges(): SignedBLSToExecutionChangeVersioned[] {
-    return Array.from(this.blsToExecutionChanges.values());
-  }
-
-  /**
-   * Prune all types of transactions given the latest head state
-   */
-  pruneAll(headBlock: SignedBeaconBlock, headState: CachedBeaconStateAllForks): void {
-    this.pruneAttesterSlashings(headState);
-    this.pruneProposerSlashings(headState);
-    this.pruneVoluntaryExits(headState);
-    this.pruneBlsToExecutionChanges(headBlock, headState);
-  }
-
-  /**
-   * Prune attester slashings for all slashed or withdrawn validators.
-   */
-  private pruneAttesterSlashings(headState: CachedBeaconStateAllForks): void {
-    const finalizedEpoch = headState.finalizedCheckpoint.epoch;
-    attesterSlashing: for (const [key, attesterSlashing] of this.attesterSlashings.entries()) {
-      // Slashings that don't slash any validators can be dropped
-      for (let i = 0; i < attesterSlashing.intersectingIndices.length; i++) {
-        const index = attesterSlashing.intersectingIndices[i];
-
-        // Declare that a validator is still slashable if they have not exited prior
-        // to the finalized epoch.
-        //
-        // We cannot check the `slashed` field since the `head` is not finalized and
-        // a fork could un-slash someone.
-        if (headState.validators.getReadonly(index).exitEpoch > finalizedEpoch) {
-          continue attesterSlashing;
-        }
-      }
-
-      // All intersecting indices are not slashable
-      this.attesterSlashings.delete(key);
-    }
-  }
-
-  /**
-   * Prune proposer slashings for validators which are exited in the finalized epoch.
-   */
-  private pruneProposerSlashings(headState: CachedBeaconStateAllForks): void {
-    const finalizedEpoch = headState.finalizedCheckpoint.epoch;
-    for (const [key, proposerSlashing] of this.proposerSlashings.entries()) {
-      const index = proposerSlashing.signedHeader1.message.proposerIndex;
-      if (headState.validators.getReadonly(index).exitEpoch <= finalizedEpoch) {
-        this.proposerSlashings.delete(key);
-      }
-    }
-  }
-
-  /**
-   * Call after finalizing
-   * Prune if validator has already exited at or before the finalized checkpoint of the head.
-   */
-  private pruneVoluntaryExits(headState: CachedBeaconStateAllForks): void {
-    const {config} = headState;
-    const headStateFork = config.getForkSeq(headState.slot);
-    const finalizedEpoch = headState.finalizedCheckpoint.epoch;
-
-    for (const [key, voluntaryExit] of this.voluntaryExits.entries()) {
-      // VoluntaryExit messages signed in the previous fork become invalid and can never be included in any future
-      // block, so just drop as the head state advances into the next fork.
-      if (config.getForkSeq(computeStartSlotAtEpoch(voluntaryExit.message.epoch)) < headStateFork) {
-        this.voluntaryExits.delete(key);
-      }
-
-      // TODO: Improve this simplistic condition
-      if (voluntaryExit.message.epoch <= finalizedEpoch) {
-        this.voluntaryExits.delete(key);
-      }
-    }
-  }
-
-  /**
-   * Prune BLS to execution changes that have been applied to the state more than 1 block ago.
-   * In the worse case where head block is reorged, the same BlsToExecutionChange message can be re-added
-   * to opPool once gossipsub seen cache TTL passes.
-   */
-  private pruneBlsToExecutionChanges(headBlock: SignedBeaconBlock, headState: CachedBeaconStateAllForks): void {
-    const {config} = headState;
-    const recentBlsToExecutionChanges =
-      config.getForkSeq(headBlock.message.slot) >= ForkSeq.capella
-        ? (headBlock as capella.SignedBeaconBlock).message.body.blsToExecutionChanges
-        : [];
-
-    const recentBlsToExecutionChangeIndexes = new Set(
-      recentBlsToExecutionChanges.map((blsToExecutionChange) => blsToExecutionChange.message.validatorIndex)
-    );
-
-    for (const [key, blsToExecutionChange] of this.blsToExecutionChanges.entries()) {
-      const {validatorIndex} = blsToExecutionChange.data.message;
-      if (!recentBlsToExecutionChangeIndexes.has(validatorIndex)) {
-        const validator = headState.validators.getReadonly(validatorIndex);
-        if (validator.withdrawalCredentials[0] !== BLS_WITHDRAWAL_PREFIX) {
-          this.blsToExecutionChanges.delete(key);
-        }
-      }
-    }
-  }
-}
-
-/**
- * Returns true if a pre-validated signature is still valid to be included in a specific block's fork
- */
-function isVoluntaryExitSignatureIncludable(stateFork: ForkSeq, voluntaryExitFork: ForkSeq): boolean {
-  if (stateFork >= ForkSeq.deneb) {
-    // Exists are perpetually valid https://eips.ethereum.org/EIPS/eip-7044
-    return true;
-  }
-  // Can only include exits from the current and previous fork
-  return voluntaryExitFork === stateFork || voluntaryExitFork === stateFork - 1;
-}
-
-function isSlashableAtEpoch(validator: phase0.Validator, epoch: Epoch): boolean {
-  return !validator.slashed && validator.activationEpoch <= epoch && epoch < validator.withdrawableEpoch;
-}
-
-/**
- * Persist target items `items` in `dbRepo` doing minimum put and delete writes.
- * Reads all keys in repository to compute the diff between current persisted data and target data.
- */
-async function persistDiff<K extends Id, V>(
-  dbRepo: Repository<K, V>,
-  items: {key: K; value: V}[],
-  serializeKey: (key: K) => number | string
-): Promise<void> {
-  const persistedKeys = await dbRepo.keys();
-  const itemsToPut: {key: K; value: V}[] = [];
-  const keysToDelete: K[] = [];
-
-  const persistedKeysSerialized = new Set(persistedKeys.map(serializeKey));
-  for (const item of items) {
-    if (!persistedKeysSerialized.has(serializeKey(item.key))) {
-      itemsToPut.push(item);
-    }
-  }
-
-  const targetKeysSerialized = new Set(items.map((item) => serializeKey(item.key)));
-  for (const persistedKey of persistedKeys) {
-    if (!targetKeysSerialized.has(serializeKey(persistedKey))) {
-      keysToDelete.push(persistedKey);
-    }
-  }
-
-  if (itemsToPut.length > 0) await dbRepo.batchPut(itemsToPut);
-  if (keysToDelete.length > 0) await dbRepo.batchDelete(keysToDelete);
-}

package/src/chain/opPools/syncCommitteeMessagePool.ts

@@ -1,167 +0,0 @@
-import {Signature, aggregateSignatures} from "@chainsafe/blst";
-import {BitArray} from "@chainsafe/ssz";
-import {SYNC_COMMITTEE_SIZE, SYNC_COMMITTEE_SUBNET_COUNT} from "@lodestar/params";
-import {Root, Slot, SubcommitteeIndex, SubnetID, altair} from "@lodestar/types";
-import {MapDef, toRootHex} from "@lodestar/utils";
-import {IClock} from "../../util/clock.js";
-import {InsertOutcome, OpPoolError, OpPoolErrorCode} from "./types.js";
-import {pruneBySlot, signatureFromBytesNoCheck} from "./utils.js";
-
-/**
- * SyncCommittee signatures are only useful during a single slot according to our peer's clocks
- */
-const SLOTS_RETAINED = 3;
-
-/**
- * The maximum number of distinct `ContributionFast` that will be stored in each slot.
- *
- * This is a DoS protection measure.
- */
-const MAX_ITEMS_PER_SLOT = 512;
-
-type ContributionFast = Omit<altair.SyncCommitteeContribution, "aggregationBits" | "signature"> & {
-  aggregationBits: BitArray;
-  signature: Signature;
-};
-
-/** Hex string of `contribution.beaconBlockRoot` */
-type BlockRootHex = string;
-
-/**
- * Preaggregate SyncCommitteeMessage into SyncCommitteeContribution
- * and cache seen SyncCommitteeMessage by slot + validator index.
- * This stays in-memory and should be pruned per slot.
- */
-export class SyncCommitteeMessagePool {
-  /**
-   * Each array item is respective to a subcommitteeIndex.
-   * Preaggregate into SyncCommitteeContribution.
-   * */
-  private readonly contributionsByRootBySubnetBySlot = new MapDef<
-    Slot,
-    MapDef<SubnetID, Map<BlockRootHex, ContributionFast>>
-  >(() => new MapDef<SubnetID, Map<BlockRootHex, ContributionFast>>(() => new Map<BlockRootHex, ContributionFast>()));
-  private lowestPermissibleSlot = 0;
-
-  constructor(
-    private readonly clock: IClock,
-    private readonly cutOffSecFromSlot: number,
-    private readonly preaggregateSlotDistance = 0
-  ) {}
-
-  /** Returns current count of unique ContributionFast by block root and subnet */
-  get size(): number {
-    let count = 0;
-    for (const contributionsByRootBySubnet of this.contributionsByRootBySubnetBySlot.values()) {
-      for (const contributionsByRoot of contributionsByRootBySubnet.values()) {
-        count += contributionsByRoot.size;
-      }
-    }
-    return count;
-  }
-
-  // TODO: indexInSubcommittee: number should be indicesInSyncCommittee
-  add(
-    subnet: SubnetID,
-    signature: altair.SyncCommitteeMessage,
-    indexInSubcommittee: number,
-    priority?: boolean
-  ): InsertOutcome {
-    const {slot, beaconBlockRoot} = signature;
-    const rootHex = toRootHex(beaconBlockRoot);
-    const lowestPermissibleSlot = this.lowestPermissibleSlot;
-
-    // Reject if too old.
-    if (slot < lowestPermissibleSlot) {
-      return InsertOutcome.Old;
-    }
-
-    // validator gets SyncCommitteeContribution at 2/3 of slot, it's no use to preaggregate later than that time
-    if (!priority && this.clock.secFromSlot(slot) > this.cutOffSecFromSlot) {
-      return InsertOutcome.Late;
-    }
-
-    // Limit object per slot
-    const contributionsByRoot = this.contributionsByRootBySubnetBySlot.getOrDefault(slot).getOrDefault(subnet);
-    if (contributionsByRoot.size >= MAX_ITEMS_PER_SLOT) {
-      throw new OpPoolError({code: OpPoolErrorCode.REACHED_MAX_PER_SLOT});
-    }
-
-    // Pre-aggregate the contribution with existing items
-    const contribution = contributionsByRoot.get(rootHex);
-    if (contribution) {
-      // Aggregate mutating
-      return aggregateSignatureInto(contribution, signature, indexInSubcommittee);
-    }
-
-    // Create new aggregate
-    contributionsByRoot.set(rootHex, signatureToAggregate(subnet, signature, indexInSubcommittee));
-    return InsertOutcome.NewData;
-  }
-
-  /**
-   * This is for the aggregator to produce ContributionAndProof.
-   */
-  getContribution(subnet: SubcommitteeIndex, slot: Slot, prevBlockRoot: Root): altair.SyncCommitteeContribution | null {
-    const contribution = this.contributionsByRootBySubnetBySlot.get(slot)?.get(subnet)?.get(toRootHex(prevBlockRoot));
-    if (!contribution) {
-      return null;
-    }
-
-    return {
-      ...contribution,
-      aggregationBits: contribution.aggregationBits,
-      signature: contribution.signature.toBytes(),
-    };
-  }
-
-  /**
-   * Prune per clock slot.
-   * SyncCommittee signatures are only useful during a single slot according to our peer's clocks
-   */
-  prune(clockSlot: Slot): void {
-    pruneBySlot(this.contributionsByRootBySubnetBySlot, clockSlot, SLOTS_RETAINED);
-    // by default preaggregateSlotDistance is 0, i.e only accept SyncCommitteeMessage in the same clock slot.
-    this.lowestPermissibleSlot = Math.max(clockSlot - this.preaggregateSlotDistance, 0);
-  }
-}
-
-/**
- * Aggregate a new signature into `contribution` mutating it
- */
-function aggregateSignatureInto(
-  contribution: ContributionFast,
-  signature: altair.SyncCommitteeMessage,
-  indexInSubcommittee: number
-): InsertOutcome {
-  if (contribution.aggregationBits.get(indexInSubcommittee) === true) {
-    return InsertOutcome.AlreadyKnown;
-  }
-
-  contribution.aggregationBits.set(indexInSubcommittee, true);
-  contribution.signature = aggregateSignatures([
-    contribution.signature,
-    signatureFromBytesNoCheck(signature.signature),
-  ]);
-  return InsertOutcome.Aggregated;
-}
-
-/**
- * Format `signature` into an efficient `contribution` to add more signatures in with aggregateSignatureInto()
- */
-function signatureToAggregate(
-  subnet: SubnetID,
-  signature: altair.SyncCommitteeMessage,
-  indexInSubcommittee: number
-): ContributionFast {
-  const indexesPerSubnet = Math.floor(SYNC_COMMITTEE_SIZE / SYNC_COMMITTEE_SUBNET_COUNT);
-  const aggregationBits = BitArray.fromSingleBit(indexesPerSubnet, indexInSubcommittee);
-
-  return {
-    slot: signature.slot,
-    beaconBlockRoot: signature.beaconBlockRoot,
-    subcommitteeIndex: subnet,
-    aggregationBits,
-    signature: signatureFromBytesNoCheck(signature.signature),
-  };
-}