refactor(conf): remove unused code, change builderProposalDelayTolerance type from int to uint16

Author: meta-duck

beacon_chain/conf.nim

@@ -686,7 +686,7 @@ type
       builderProposalDelayTolerance* {.
         desc: "Timeout for builder proposal delay tolerance in milliseconds"
         defaultValue: 1500
-        name: "builder-proposal-delay-tolerance" .}: int
+        name: "builder-proposal-delay-tolerance" .}: uint16
 
       historyMode* {.
         desc: "Retention strategy for historical data (archive/prune)"

beacon_chain/spec/mev/electra_mev.nim

@@ -8,7 +8,6 @@
 {.push raises: [].}
 
 import ".."/datatypes/[altair, bellatrix, electra]
-# import ../../conf
 
 from stew/byteutils import to0xHex
 from ".."/datatypes/phase0 import AttesterSlashing
@@ -100,7 +99,6 @@ const
 
   # Spec is 1 second, but mev-boost indirection can induce delay when the relay
   # itself has already consumed the entire second.
-  # BUILDER_PROPOSAL_DELAY_TOLERANCE* = builderProposalDelayTolerance.milliseconds
 
 func shortLog*(v: BlindedBeaconBlock): auto =
   (

beacon_chain/validators/beacon_validators.nim

@@ -996,9 +996,7 @@ proc collectBids(
     engineBlockFut = makeBeaconBlockForHeadAndSlot(
       EPS, node, randao, validator_index, graffitiBytes, head, slot)
 
-  # getBuilderBid times out after BUILDER_PROPOSAL_DELAY_TOLERANCE, with 1 more
-  # second for remote validators. makeBeaconBlockForHeadAndSlot times out after
-  # 1 second.
+
   await allFutures(payloadBuilderBidFut, engineBlockFut)
   doAssert payloadBuilderBidFut.finished and engineBlockFut.finished