fix new chain not using configured reorg check start block (#173)

### TL;DR

Fixed reorg handler logic to properly handle cases where the configured start block is ahead of the latest committed block.

### What changed?

- Refactored `getInitialCheckedBlockNumber` function to improve readability by using more descriptive variable names and restructuring the conditional logic
- Added a new check in `getReorgCheckRange` to handle the case where the latest checked block is ahead of the latest committed block, preventing unnecessary processing

### How to test?

1. Configure a reorg handler with a start block that is ahead of the latest committed block
2. Verify that the reorg handler logs the message "Committing has not reached the configured reorg check start block" and continues without error
3. Confirm that the reorg handler correctly uses the configured block when appropriate based on the ForceFromBlock setting

### Why make this change?

This change prevents the reorg handler from attempting to process blocks that haven't been committed yet, which could lead to errors or unexpected behavior. The refactoring also improves code readability and maintainability by using clearer variable names and more logical flow.

Author: iuwqyir

internal/orchestrator/reorg_handler.go

@@ -49,22 +49,22 @@ func NewReorgHandler(rpc rpc.IRPCClient, storage storage.IStorage) *ReorgHandler
 }
 
 func getInitialCheckedBlockNumber(storage storage.IStorage, chainId *big.Int) *big.Int {
-	bn := big.NewInt(int64(config.Cfg.ReorgHandler.FromBlock))
-	if !config.Cfg.ReorgHandler.ForceFromBlock {
-		storedFromBlock, err := storage.OrchestratorStorage.GetLastReorgCheckedBlockNumber(chainId)
-		if err != nil {
-			log.Debug().Err(err).Msgf("Error getting last reorg checked block number, using configured: %s", bn)
-			return bn
-		}
-		if storedFromBlock.Sign() <= 0 {
-			log.Debug().Msgf("Last reorg checked block number not found, using configured: %s", bn)
-			return bn
-		}
-		log.Debug().Msgf("Last reorg checked block number found, using: %s", storedFromBlock)
-		return storedFromBlock
+	configuredBn := big.NewInt(int64(config.Cfg.ReorgHandler.FromBlock))
+	if config.Cfg.ReorgHandler.ForceFromBlock {
+		log.Debug().Msgf("Force from block reorg check flag set, using configured: %s", configuredBn)
+		return configuredBn
+	}
+	storedBn, err := storage.OrchestratorStorage.GetLastReorgCheckedBlockNumber(chainId)
+	if err != nil {
+		log.Debug().Err(err).Msgf("Error getting last reorg checked block number, using configured: %s", configuredBn)
+		return configuredBn
+	}
+	if storedBn.Sign() <= 0 {
+		log.Debug().Msgf("Last reorg checked block number not found, using configured: %s", configuredBn)
+		return configuredBn
 	}
-	log.Debug().Msgf("Force from block reorg check flag set, using configured: %s", bn)
-	return bn
+	log.Debug().Msgf("Last reorg checked block number found, using: %s", storedBn)
+	return storedBn
 }
 
 func (rh *ReorgHandler) Start(ctx context.Context) {
@@ -148,6 +148,11 @@ func (rh *ReorgHandler) getReorgCheckRange(latestCheckedBlock *big.Int) (*big.In
 	if err != nil {
 		return nil, nil, fmt.Errorf("error getting latest committed block: %w", err)
 	}
+	if latestCheckedBlock.Cmp(latestCommittedBlock) > 0 {
+		log.Debug().Msgf("Committing has not reached the configured reorg check start block: %s (reorg start) > %s (last committed)", latestCheckedBlock.String(), latestCommittedBlock.String())
+		return latestCheckedBlock, latestCheckedBlock, nil
+	}
+
 	if new(big.Int).Sub(latestCommittedBlock, latestCheckedBlock).Cmp(big.NewInt(int64(rh.blocksPerScan))) < 0 {
 		// diff between latest committed and latest checked is less than blocksPerScan, so we will look back from the latest committed block
 		fromBlock := new(big.Int).Sub(latestCommittedBlock, big.NewInt(int64(rh.blocksPerScan)))