Fixed the typo. For now it backwards compatible. (#310)

## 📝 Summary

It was supposed to be sbundle_mergeable_signers but was
sbundle_mergeabe_signers.

## 💡 Motivation and Context

- My eyes start to bleed everytime I see that typo.

## ✅ I have completed the following steps:

* [X] Run `make lint`
* [X] Run `make test`
* [ ] Added tests (if applicable)

config-backtest-example.toml

@@ -8,7 +8,7 @@ backtest_fetch_eth_rpc_parallel = 400
 backtest_fetch_output_file = "~/.rbuilder/backtest/main.sqlite"
 backtest_fetch_mempool_data_dir = "~/.rbuilder/mempool-data"
 
-sbundle_mergeabe_signers = []
+sbundle_mergeable_signers = []
 
 backtest_builders = ["mp-ordering", "mgp-ordering"]
 

config-live-example.toml

@@ -31,7 +31,7 @@ max_concurrent_seals = 4
 
 # genesis_fork_version = "0x00112233"
 
-sbundle_mergeabe_signers = []
+sbundle_mergeable_signers = []
 live_builders = ["mp-ordering", "mgp-ordering", "merging"]
 
 [[relays]]

config-optimism-local.toml

@@ -21,7 +21,7 @@ dry_run_validation_url = "http://localhost:8545"
 
 ignore_cancellable_orders = true
 
-sbundle_mergeabe_signers = []
+sbundle_mergeable_signers = []
 live_builders = ["mp-ordering"]
 
 [[relays]]

config-playground.toml

@@ -24,7 +24,7 @@ dry_run_validation_url = "http://localhost:8545"
 
 ignore_cancellable_orders = true
 
-sbundle_mergeabe_signers = []
+sbundle_mergeable_signers = []
 live_builders = ["mp-ordering", "mgp-ordering", "parallel"]
 
 [[relays]]

crates/rbuilder/src/backtest/backtest_build_block.rs

@@ -108,7 +108,7 @@ where
         chain_spec.clone(),
         cli.block_building_time_ms,
         config.base_config().blocklist()?,
-        &config.base_config().sbundle_mergeabe_signers(),
+        &config.base_config().sbundle_mergeable_signers(),
         config.base_config().coinbase_signer()?,
     )?;
 

crates/rbuilder/src/backtest/backtest_build_range.rs

@@ -178,7 +178,7 @@ where
                         builders_names,
                         &config,
                         blocklist,
-                        &config.base_config().sbundle_mergeabe_signers(),
+                        &config.base_config().sbundle_mergeable_signers(),
                     ) {
                         Ok(ok) => Some(ok),
                         Err(err) => {

crates/rbuilder/src/backtest/redistribute/mod.rs

@@ -1010,7 +1010,7 @@ where
         base_config.backtest_builders.clone(),
         config,
         base_config.blocklist()?,
-        &base_config.sbundle_mergeabe_signers(),
+        &base_config.sbundle_mergeable_signers(),
     )?
     .builder_outputs
     .into_iter()

crates/rbuilder/src/live_builder/base_config.rs

@@ -34,7 +34,7 @@ use std::{
     time::Duration,
 };
 use tokio::sync::mpsc;
-use tracing::warn;
+use tracing::{error, warn};
 
 use super::SlotSource;
 
@@ -80,6 +80,9 @@ pub struct BaseConfig {
     pub extra_data: String,
 
     /// mev-share bundles coming from this address are treated in a special way(see [`ShareBundleMerger`])
+    pub sbundle_mergeable_signers: Option<Vec<Address>>,
+
+    /// Backwards compatible typo soon to be removed.
     pub sbundle_mergeabe_signers: Option<Vec<Address>>,
 
     /// Number of threads used for incoming order simulation
@@ -232,9 +235,7 @@ impl BaseConfig {
 
             orderpool_sender,
             orderpool_receiver,
-            sbundle_merger_selected_signers: Arc::new(
-                self.sbundle_mergeabe_signers.clone().unwrap_or_default(),
-            ),
+            sbundle_merger_selected_signers: Arc::new(self.sbundle_mergeable_signers()),
         })
     }
 
@@ -254,12 +255,19 @@ impl BaseConfig {
         chain_value_parser(&self.chain)
     }
 
-    pub fn sbundle_mergeabe_signers(&self) -> Vec<Address> {
-        if self.sbundle_mergeabe_signers.is_none() {
-            warn!("Defaulting sbundle_mergeabe_signers to empty. We may not comply with order flow rules.");
+    pub fn sbundle_mergeable_signers(&self) -> Vec<Address> {
+        if let Some(sbundle_mergeable_signers) = &self.sbundle_mergeable_signers {
+            if self.sbundle_mergeabe_signers.is_some() {
+                error!("sbundle_mergeable_signers and sbundle_mergeabe_signers found. Will use bundle_mergeable_signers");
+            }
+            sbundle_mergeable_signers.clone()
+        } else if let Some(sbundle_mergeable_signers) = &self.sbundle_mergeabe_signers {
+            warn!("sbundle_mergeable_signers missing but found sbundle_mergeabe_signers. sbundle_mergeabe_signers will be used but this will be deprecated soon");
+            sbundle_mergeable_signers.clone()
+        } else {
+            warn!("Defaulting sbundle_mergeable_signers to empty. We may not comply with order flow rules.");
+            Vec::default()
         }
-
-        self.sbundle_mergeabe_signers.clone().unwrap_or_default()
     }
 
     /// Open reth db and DB should be opened once per process but it can be cloned and moved to different threads.
@@ -455,6 +463,7 @@ impl Default for BaseConfig {
             backtest_builders: Vec::new(),
             live_builders: vec!["mgp-ordering".to_string(), "mp-ordering".to_string()],
             simulation_threads: 1,
+            sbundle_mergeable_signers: None,
             sbundle_mergeabe_signers: None,
         }
     }