single buffer size; fix lint

Author: cody-wang-cb

crates/op-rbuilder/src/args/op.rs

@@ -51,16 +51,10 @@ pub struct OpRbuilderArgs {
     /// Whether to enable TIPS Resource Metering
     #[arg(long = "builder.enable-resource-metering", default_value = "false")]
     pub enable_resource_metering: bool,
-    /// Whether to enable TIPS Resource Metering
-    #[arg(
-        long = "builder.resource-metering-buffer-size",
-        default_value = "10000"
-    )]
-    pub resource_metering_buffer_size: usize,
 
-    /// Buffer size for backrun bundles (LRU eviction when full)
-    #[arg(long = "builder.backrun-bundle-buffer-size", default_value = "10000")]
-    pub backrun_bundle_buffer_size: usize,
+    /// Buffer size for tx data store (LRU eviction when full)
+    #[arg(long = "builder.tx-data-store-buffer-size", default_value = "10000")]
+    pub tx_data_store_buffer_size: usize,
 
     /// Path to builder playgorund to automatically start up the node connected to it
     #[arg(

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

@@ -196,8 +196,7 @@ where
             gas_limiter_config: args.gas_limiter.clone(),
             tx_data_store: TxDataStore::new(
                 args.enable_resource_metering,
-                args.resource_metering_buffer_size
-                    .max(args.backrun_bundle_buffer_size),
+                args.tx_data_store_buffer_size,
             ),
             specific: S::try_from(args)?,
         })

crates/op-rbuilder/src/tx_data_store.rs

@@ -71,15 +71,15 @@ impl TxDataStore {
     }
 
     fn evict_if_needed(&self) {
-        if self.data.lru.is_full() {
-            if let Ok(evicted_hash) = self.data.lru.pop() {
-                self.data.by_tx_hash.remove(&evicted_hash);
-                debug!(
-                    target: "tx_data_store",
-                    evicted_tx = ?evicted_hash,
-                    "Evicted old transaction data"
-                );
-            }
+        if self.data.lru.is_full()
+            && let Ok(evicted_hash) = self.data.lru.pop()
+        {
+            self.data.by_tx_hash.remove(&evicted_hash);
+            debug!(
+                target: "tx_data_store",
+                evicted_tx = ?evicted_hash,
+                "Evicted old transaction data"
+            );
         }
     }