fix: remove arc and clones

Author: prestwich

bin/builder.rs

@@ -6,7 +6,6 @@ use builder::{
 use init4_bin_base::{deps::tracing, utils::from_env::FromEnv};
 use signet_sim::SimCache;
 use signet_types::constants::SignetSystemConstants;
-use std::sync::Arc;
 use tokio::select;
 use tracing::info_span;
 
@@ -44,12 +43,12 @@ async fn main() -> eyre::Result<()> {
     let sim_items = SimCache::new();
     let slot_calculator = config.slot_calculator;
 
-    let sim = Arc::new(Simulator::new(&config, ru_provider.clone(), slot_calculator));
+    let sim = Simulator::new(&config, ru_provider.clone(), slot_calculator);
 
     let (basefee_jh, sim_cache_jh) =
         sim.spawn_cache_tasks(tx_receiver, bundle_receiver, sim_items.clone());
 
-    let build_jh = sim.clone().spawn_simulator_task(constants, sim_items.clone(), submit_channel);
+    let build_jh = sim.spawn_simulator_task(constants, sim_items.clone(), submit_channel);
 
     let server = serve_builder(([0, 0, 0, 0], config.builder_port));
 

src/tasks/block.rs

@@ -208,12 +208,12 @@ impl Simulator {
     ///
     /// A `JoinHandle` for the spawned task.
     pub fn spawn_simulator_task(
-        self: Arc<Self>,
+        self,
         constants: SignetSystemConstants,
         cache: SimCache,
         submit_sender: mpsc::UnboundedSender<BuiltBlock>,
     ) -> JoinHandle<()> {
-        debug!("starting builder task");
+        debug!("starting simulator task");
 
         tokio::spawn(async move { self.run_simulator(constants, cache, submit_sender).await })
     }
@@ -233,7 +233,7 @@ impl Simulator {
     /// - `cache`: The simulation cache containing transactions and bundles.
     /// - `submit_sender`: A channel sender used to submit built blocks.
     async fn run_simulator(
-        self: Arc<Self>,
+        self,
         constants: SignetSystemConstants,
         cache: SimCache,
         submit_sender: mpsc::UnboundedSender<BuiltBlock>,