Clear some obsolete or wrong asserts

why It is wrong to depend on a particular state of other modules (e.g. `FC` module) unless they are fully controlled by this one.
status-im · Mar 10, 2025 · a946773 · a946773
1 parent e2ae3fb
commit a946773
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 6 deletions.
diff --git a/execution_chain/sync/beacon/worker/blocks_staged.nim b/execution_chain/sync/beacon/worker/blocks_staged.nim
@@ -363,7 +363,6 @@ proc blocksStagedImport*(
           finHash = if nBn < ctx.layout.final: nthHash
                     else: ctx.layout.finalHash
 
-        doAssert nBn == ctx.chain.latestNumber()
         ctx.pool.chain.forkChoice(nthHash, finHash).isOkOr:
           ctx.poolMode = true
           warn info & ": fork choice error (reorg triggered)", n, iv,

diff --git a/execution_chain/sync/beacon/worker/headers_staged.nim b/execution_chain/sync/beacon/worker/headers_staged.nim
@@ -116,9 +116,6 @@ proc headersStagedCollect*(
 
     # End block: `fetchHeadersBody`
 
-  # The cache `antecedent` must match variable `D` (aka dangling)
-  doAssert ctx.hdrCache.fcHeaderAntecedent().number <= ctx.layout.dangling
-
   let nHeaders = nDeterministic + nOpportunistic.uint64
   if nHeaders == 0:
     return false
@@ -149,8 +146,6 @@ proc headersStagedProcess*(buddy: BeaconBuddyRef; info: static[string]) =
     let qItem = ctx.hdr.staged.le(high BlockNumber).valueOr:
       break # all done
 
-    doAssert qItem.key == qItem.data.revHdrs[0].number
-
     let
       minNum = qItem.data.revHdrs[^1].number
       maxNum = qItem.data.revHdrs[0].number