stateTransition is not async

dgcoffman · Nov 17, 2022 · 013d173 · 013d173
1 parent 2942065
commit 013d173
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 6 deletions.
diff --git a/packages/beacon-node/src/api/impl/beacon/state/utils.ts b/packages/beacon-node/src/api/impl/beacon/state/utils.ts
@@ -221,7 +221,7 @@ async function getFinalizedState(
 
   // process blocks up to the requested slot
   for await (const block of db.blockArchive.valuesStream({gt: state.slot, lte: slot})) {
-    state = await stateTransition(state, block, undefined, {
+    state = stateTransition(state, block, undefined, {
       verifyStateRoot: false,
       verifyProposer: false,
       verifySignatures: false,

diff --git a/packages/beacon-node/src/chain/blocks/verifyBlocksStateTransitionOnly.ts b/packages/beacon-node/src/chain/blocks/verifyBlocksStateTransitionOnly.ts
@@ -33,7 +33,7 @@ export async function verifyBlocksStateTransitionOnly(
     // STFN - per_slot_processing() + per_block_processing()
     // NOTE: `regen.getPreState()` should have dialed forward the state already caching checkpoint states
     const useBlsBatchVerify = !opts?.disableBlsBatchVerify;
-    const postState = await stateTransition(
+    const postState = stateTransition(
       preState,
       block,
       undefined,

diff --git a/packages/beacon-node/src/chain/produceBlock/computeNewStateRoot.ts b/packages/beacon-node/src/chain/produceBlock/computeNewStateRoot.ts
@@ -21,7 +21,7 @@ export async function computeNewStateRoot(
   // Set signature to zero to re-use stateTransition() function which requires the SignedBeaconBlock type
   const blockEmptySig = {message: block, signature: ZERO_HASH} as allForks.FullOrBlindedSignedBeaconBlock;
 
-  const postState = await stateTransition(
+  const postState = stateTransition(
     state,
     blockEmptySig,
     undefined,

diff --git a/packages/beacon-node/src/chain/regen/regen.ts b/packages/beacon-node/src/chain/regen/regen.ts
@@ -169,7 +169,7 @@ export class StateRegenerator implements IStateRegenerator {
       try {
         // Only advances state trusting block's signture and hashes.
         // We are only running the state transition to get a specific state's data.
-        state = await stateTransition(
+        state = stateTransition(
           state,
           block,
           undefined,

diff --git a/packages/state-transition/src/stateTransition.ts b/packages/state-transition/src/stateTransition.ts
@@ -36,13 +36,13 @@ export type StateTransitionOpts = EpochProcessOpts & {
 /**
  * Implementation Note: follows the optimizations in protolambda's eth2fastspec (https://github.com/protolambda/eth2fastspec)
  */
-export async function stateTransition(
+export function stateTransition(
   state: CachedBeaconStateAllForks,
   signedBlock: allForks.FullOrBlindedSignedBeaconBlock,
   blobsSidecar?: BlobsSidecar,
   options?: StateTransitionOpts,
   metrics?: IBeaconStateTransitionMetrics | null
-): Promise<CachedBeaconStateAllForks> {
+): CachedBeaconStateAllForks {
   const {verifyStateRoot = true, verifyProposer = true, verifySignatures = true, verifyBlobs = true} = options || {};
 
   const block = signedBlock.message;