fix(bin): Fix Compute Pending Block Docs

bin/reth/src/args/payload_builder_args.rs

@@ -41,6 +41,10 @@ pub struct PayloadBuilderArgs {
     /// to save resources and not leak txs from the tx-pool,
     /// this flag enables computing of the pending block
     /// from the tx-pool instead.
+    ///
+    /// If `compute_pending_block` is not enabled, the payload builder
+    /// will use the payload attributes from the latest block. Note
+    /// that this flag is not yet functional.
     #[cfg(feature = "optimism")]
     #[arg(long = "rollup.compute-pending-block")]
     pub compute_pending_block: bool,
@@ -94,7 +98,7 @@ impl TypedValueParser for ExtradataValueParser {
                 format!(
                     "Payload builder extradata size exceeds {MAXIMUM_EXTRA_DATA_SIZE}bytes limit"
                 ),
-            ))
+            ));
         }
         Ok(val.to_string())
     }

bin/reth/src/node/mod.rs

@@ -191,14 +191,14 @@ pub struct NodeCommand<Ext: RethCliExt = ()> {
     #[clap(flatten)]
     pub pruning: PruningArgs,
 
-    /// Additional cli arguments
-    #[clap(flatten)]
-    pub ext: Ext::Node,
-
     /// Rollup related arguments
     #[cfg(feature = "optimism")]
     #[clap(flatten)]
-    rollup: crate::args::RollupArgs,
+    pub rollup: crate::args::RollupArgs,
+
+    /// Additional cli arguments
+    #[clap(flatten)]
+    pub ext: Ext::Node,
 }
 
 impl<Ext: RethCliExt> NodeCommand<Ext> {
@@ -238,9 +238,9 @@ impl<Ext: RethCliExt> NodeCommand<Ext> {
             db,
             dev,
             pruning,
-            ext,
             #[cfg(feature = "optimism")]
             rollup,
+            ext,
         }
     }
 
@@ -554,6 +554,12 @@ impl<Ext: RethCliExt> NodeCommand<Ext> {
 
         self.ext.on_node_started(&components)?;
 
+        // If `enable_genesis_walkback` is set to true, the rollup client will need to
+        // perform the derivation pipeline from genesis, validating the data dir.
+        // When set to false, set the finalized, safe, and unsafe head block hashes
+        // on the rollup client using a fork choice update. This prevents the rollup
+        // client from performing the derivation pipeline from genesis, and instead
+        // starts syncing from the current tip in the DB.
         #[cfg(feature = "optimism")]
         if self.chain.optimism && !self.rollup.enable_genesis_walkback {
             let client = _rpc_server_handles.auth.http_client();
@@ -788,15 +794,15 @@ impl<Ext: RethCliExt> NodeCommand<Ext> {
         // try to look up the header in the database
         if let Some(header) = header {
             info!(target: "reth::cli", ?tip, "Successfully looked up tip block in the database");
-            return Ok(header.seal_slow())
+            return Ok(header.seal_slow());
         }
 
         info!(target: "reth::cli", ?tip, "Fetching tip block from the network.");
         loop {
             match get_single_header(&client, tip).await {
                 Ok(tip_header) => {
                     info!(target: "reth::cli", ?tip, "Successfully fetched tip");
-                    return Ok(tip_header)
+                    return Ok(tip_header);
                 }
                 Err(error) => {
                     error!(target: "reth::cli", %error, "Failed to fetch the tip. Retrying...");