fixes

Author: SozinM

crates/op-rbuilder/src/builders/flashblocks/payload.rs

@@ -630,7 +630,7 @@ where
             Ok(None) => error!(message = "Payload builder handle is not setup, skipping sending payload"),
             Err(e) => error!(
                 message = "Failed to get access to payload builder handle",
-                error = ?e.
+                error = ?e,
             ),
         }
     }

crates/op-rbuilder/src/builders/flashblocks/service.rs

@@ -56,7 +56,7 @@ impl FlashblocksServiceBuilder {
         let (payload_service, payload_builder) =
             PayloadBuilderService::new(payload_generator, ctx.provider().canonical_state_stream());
 
-        payload_builder_handle.lock()?.replace(payload_service.payload_events_handle());
+        payload_builder_handle.lock().expect("store payload builder handle in mutex").replace(payload_service.payload_events_handle());
 
         ctx.task_executor()
             .spawn_critical("custom payload builder service", Box::pin(payload_service));

crates/op-rbuilder/src/builders/generator.rs

@@ -197,7 +197,6 @@ where
             deadline,
             build_complete: None,
             cached_reads: self.maybe_pre_cached(parent_header.hash()),
-            payload_builder_handle: self.payload_builder_handle.clone(),
         };
 
         job.spawn_build_job();
@@ -263,7 +262,6 @@ where
     /// This is used to avoid reading the same state over and over again when new attempts are
     /// triggered, because during the building process we'll repeatedly execute the transactions.
     pub(crate) cached_reads: Option<CachedReads>,
-    pub(crate) payload_builder_handle: Arc<Mutex<Option<Sender<Events<OpEngineTypes>>>>>,
 }
 
 impl<Tasks, Builder> PayloadJob for BlockPayloadJob<Tasks, Builder>