chore(cli): fix clap deprecated warnings (#6872)

Co-authored-by: Alexey Shekhirin <a.shekhirin@gmail.com>

bin/reth/src/cli/mod.rs

@@ -32,7 +32,7 @@ pub use crate::core::cli::*;
 #[command(author, version = SHORT_VERSION, long_version = LONG_VERSION, about = "Reth", long_about = None)]
 pub struct Cli<Ext: RethCliExt = ()> {
     /// The command to run
-    #[clap(subcommand)]
+    #[command(subcommand)]
     command: Commands<Ext>,
 
     /// The chain this node is running.
@@ -64,7 +64,7 @@ pub struct Cli<Ext: RethCliExt = ()> {
     #[arg(long, value_name = "INSTANCE", global = true, default_value_t = 1, value_parser = value_parser!(u16).range(..=200))]
     instance: u16,
 
-    #[clap(flatten)]
+    #[command(flatten)]
     logs: LogArgs,
 }
 

bin/reth/src/commands/db/diff.rs

@@ -31,7 +31,7 @@ pub struct Command {
     secondary_datadir: PlatformPath<DataDirPath>,
 
     /// Arguments for the second database
-    #[clap(flatten)]
+    #[command(flatten)]
     second_db: DatabaseArgs,
 
     /// The table name to diff. If not specified, all tables are diffed.

bin/reth/src/commands/db/get.rs

@@ -12,7 +12,7 @@ use tracing::error;
 /// The arguments for the `reth db get` command
 #[derive(Parser, Debug)]
 pub struct Command {
-    #[clap(subcommand)]
+    #[command(subcommand)]
     subcommand: Subcommand,
 }
 
@@ -31,7 +31,7 @@ enum Subcommand {
         subkey: Option<String>,
 
         /// Output bytes instead of human-readable decoded value
-        #[clap(long)]
+        #[arg(long)]
         raw: bool,
     },
     /// Gets the content of a static file segment for the given key
@@ -43,7 +43,7 @@ enum Subcommand {
         key: String,
 
         /// Output bytes instead of human-readable decoded value
-        #[clap(long)]
+        #[arg(long)]
         raw: bool,
     },
 }
@@ -209,7 +209,7 @@ mod tests {
     /// A helper type to parse Args more easily
     #[derive(Parser)]
     struct CommandParser<T: Args> {
-        #[clap(flatten)]
+        #[command(flatten)]
         args: T,
     }
 

bin/reth/src/commands/db/mod.rs

@@ -56,10 +56,10 @@ pub struct Command {
     )]
     chain: Arc<ChainSpec>,
 
-    #[clap(flatten)]
+    #[command(flatten)]
     db: DatabaseArgs,
 
-    #[clap(subcommand)]
+    #[command(subcommand)]
     command: Subcommands,
 }
 

bin/reth/src/commands/debug_cmd/build_block.rs

@@ -81,7 +81,7 @@ pub struct Command {
     chain: Arc<ChainSpec>,
 
     /// Database arguments.
-    #[clap(flatten)]
+    #[command(flatten)]
     db: DatabaseArgs,
 
     /// Overrides the KZG trusted setup by reading from the supplied file.

bin/reth/src/commands/debug_cmd/execution.rs

@@ -71,10 +71,10 @@ pub struct Command {
     )]
     chain: Arc<ChainSpec>,
 
-    #[clap(flatten)]
+    #[command(flatten)]
     network: NetworkArgs,
 
-    #[clap(flatten)]
+    #[command(flatten)]
     db: DatabaseArgs,
 
     /// The maximum block height.

bin/reth/src/commands/debug_cmd/in_memory_merkle.rs

@@ -60,10 +60,10 @@ pub struct Command {
     )]
     chain: Arc<ChainSpec>,
 
-    #[clap(flatten)]
+    #[command(flatten)]
     db: DatabaseArgs,
 
-    #[clap(flatten)]
+    #[command(flatten)]
     network: NetworkArgs,
 
     /// The number of retries per request

bin/reth/src/commands/debug_cmd/merkle.rs

@@ -67,10 +67,10 @@ pub struct Command {
     )]
     chain: Arc<ChainSpec>,
 
-    #[clap(flatten)]
+    #[command(flatten)]
     db: DatabaseArgs,
 
-    #[clap(flatten)]
+    #[command(flatten)]
     network: NetworkArgs,
 
     /// The number of retries per request

bin/reth/src/commands/debug_cmd/mod.rs

@@ -13,7 +13,7 @@ mod replay_engine;
 /// `reth debug` command
 #[derive(Debug, Parser)]
 pub struct Command {
-    #[clap(subcommand)]
+    #[command(subcommand)]
     command: Subcommands,
 }
 

bin/reth/src/commands/debug_cmd/replay_engine.rs

@@ -68,10 +68,10 @@ pub struct Command {
     )]
     chain: Arc<ChainSpec>,
 
-    #[clap(flatten)]
+    #[command(flatten)]
     db: DatabaseArgs,
 
-    #[clap(flatten)]
+    #[command(flatten)]
     network: NetworkArgs,
 
     /// The path to read engine API messages from.

bin/reth/src/commands/import.rs

@@ -61,7 +61,7 @@ pub struct ImportCommand {
     )]
     chain: Arc<ChainSpec>,
 
-    #[clap(flatten)]
+    #[command(flatten)]
     db: DatabaseArgs,
 
     /// The path to a block file for import.

bin/reth/src/commands/init_cmd.rs

@@ -40,7 +40,7 @@ pub struct InitCommand {
     )]
     chain: Arc<ChainSpec>,
 
-    #[clap(flatten)]
+    #[command(flatten)]
     db: DatabaseArgs,
 }
 

bin/reth/src/commands/node/mod.rs

@@ -85,45 +85,44 @@ pub struct NodeCommand<Ext: RethCliExt = ()> {
     pub trusted_setup_file: Option<PathBuf>,
 
     /// All networking related arguments
-    #[clap(flatten)]
+    #[command(flatten)]
     pub network: NetworkArgs,
 
     /// All rpc related arguments
-    #[clap(flatten)]
+    #[command(flatten)]
     pub rpc: RpcServerArgs,
 
     /// All txpool related arguments with --txpool prefix
-    #[clap(flatten)]
+    #[command(flatten)]
     pub txpool: TxPoolArgs,
 
     /// All payload builder related arguments
-    #[clap(flatten)]
+    #[command(flatten)]
     pub builder: PayloadBuilderArgs,
 
     /// All debug related arguments with --debug prefix
-    #[clap(flatten)]
+    #[command(flatten)]
     pub debug: DebugArgs,
 
     /// All database related arguments
-    #[clap(flatten)]
+    #[command(flatten)]
     pub db: DatabaseArgs,
 
     /// All dev related arguments with --dev prefix
-    #[clap(flatten)]
+    #[command(flatten)]
     pub dev: DevArgs,
 
     /// All pruning related arguments
-    #[clap(flatten)]
+    #[command(flatten)]
     pub pruning: PruningArgs,
 
     /// Rollup related arguments
     #[cfg(feature = "optimism")]
-    #[clap(flatten)]
+    #[command(flatten)]
     pub rollup: crate::args::RollupArgs,
 
     /// Additional cli arguments
-    #[clap(flatten)]
-    #[clap(next_help_heading = "Extension")]
+    #[command(flatten, next_help_heading = "Extension")]
     pub ext: Ext::Node,
 }
 

bin/reth/src/commands/p2p/mod.rs

@@ -73,10 +73,10 @@ pub struct Command {
     #[arg(long, default_value = "any")]
     nat: NatResolver,
 
-    #[clap(flatten)]
+    #[command(flatten)]
     db: DatabaseArgs,
 
-    #[clap(subcommand)]
+    #[command(subcommand)]
     command: Subcommands,
 }
 

bin/reth/src/commands/recover/mod.rs

@@ -8,7 +8,7 @@ mod storage_tries;
 /// `reth recover` command
 #[derive(Debug, Parser)]
 pub struct Command {
-    #[clap(subcommand)]
+    #[command(subcommand)]
     command: Subcommands,
 }
 

bin/reth/src/commands/stage/drop.rs

@@ -45,7 +45,7 @@ pub struct Command {
     )]
     chain: Arc<ChainSpec>,
 
-    #[clap(flatten)]
+    #[command(flatten)]
     db: DatabaseArgs,
 
     stage: StageEnum,

bin/reth/src/commands/stage/dump/mod.rs

@@ -58,10 +58,10 @@ pub struct Command {
     )]
     chain: Arc<ChainSpec>,
 
-    #[clap(flatten)]
+    #[command(flatten)]
     db: DatabaseArgs,
 
-    #[clap(subcommand)]
+    #[command(subcommand)]
     command: Stages,
 }
 

bin/reth/src/commands/stage/mod.rs

@@ -10,7 +10,7 @@ pub mod unwind;
 /// `reth stage` command
 #[derive(Debug, Parser)]
 pub struct Command {
-    #[clap(subcommand)]
+    #[command(subcommand)]
     command: Subcommands,
 }
 

bin/reth/src/commands/stage/run.rs

@@ -63,7 +63,7 @@ pub struct Command {
     /// Enable Prometheus metrics.
     ///
     /// The metrics will be served at the given interface and port.
-    #[clap(long, value_name = "SOCKET")]
+    #[arg(long, value_name = "SOCKET")]
     metrics: Option<SocketAddr>,
 
     /// The name of the stage to run
@@ -90,10 +90,10 @@ pub struct Command {
     #[arg(long, short)]
     skip_unwind: bool,
 
-    #[clap(flatten)]
+    #[command(flatten)]
     network: NetworkArgs,
 
-    #[clap(flatten)]
+    #[command(flatten)]
     db: DatabaseArgs,
 
     /// Commits the changes in the database. WARNING: potentially destructive.

bin/reth/src/commands/stage/unwind.rs

@@ -42,10 +42,10 @@ pub struct Command {
     )]
     chain: Arc<ChainSpec>,
 
-    #[clap(flatten)]
+    #[command(flatten)]
     db: DatabaseArgs,
 
-    #[clap(subcommand)]
+    #[command(subcommand)]
     command: Subcommands,
 }
 
@@ -87,10 +87,10 @@ impl Command {
 #[derive(Subcommand, Debug, Eq, PartialEq)]
 enum Subcommands {
     /// Unwinds the database until the given block number (range is inclusive).
-    #[clap(name = "to-block")]
+    #[command(name = "to-block")]
     ToBlock { target: BlockHashOrNumber },
     /// Unwinds the given number of blocks from the database.
-    #[clap(name = "num-blocks")]
+    #[command(name = "num-blocks")]
     NumBlocks { amount: u64 },
 }
 

bin/reth/src/commands/test_vectors/mod.rs

@@ -7,7 +7,7 @@ mod tables;
 /// Generate test-vectors for different data types.
 #[derive(Debug, Parser)]
 pub struct Command {
-    #[clap(subcommand)]
+    #[command(subcommand)]
     command: Subcommands,
 }
 

crates/node-core/src/args/database_args.rs

@@ -5,7 +5,7 @@ use reth_interfaces::db::LogLevel;
 
 /// Parameters for database configuration
 #[derive(Debug, Args, PartialEq, Default, Clone, Copy)]
-#[clap(next_help_heading = "Database")]
+#[command(next_help_heading = "Database")]
 pub struct DatabaseArgs {
     /// Database logging level. Levels higher than "notice" require a debug build.
     #[arg(long = "db.log-level", value_enum)]
@@ -20,7 +20,7 @@ mod tests {
     /// A helper type to parse Args more easily
     #[derive(Parser)]
     struct CommandParser<T: Args> {
-        #[clap(flatten)]
+        #[command(flatten)]
         args: T,
     }
 

crates/node-core/src/args/debug_args.rs

@@ -6,7 +6,7 @@ use std::path::PathBuf;
 
 /// Parameters for debugging purposes
 #[derive(Debug, Clone, Args, PartialEq, Default)]
-#[clap(next_help_heading = "Debug")]
+#[command(next_help_heading = "Debug")]
 pub struct DebugArgs {
     /// Prompt the downloader to download blocks one at a time.
     ///
@@ -74,7 +74,7 @@ mod tests {
     /// A helper type to parse Args more easily
     #[derive(Parser)]
     struct CommandParser<T: Args> {
-        #[clap(flatten)]
+        #[command(flatten)]
         args: T,
     }
 

crates/node-core/src/args/dev_args.rs

@@ -7,7 +7,7 @@ use humantime::parse_duration;
 
 /// Parameters for Dev testnet configuration
 #[derive(Debug, Args, PartialEq, Default, Clone, Copy)]
-#[clap(next_help_heading = "Dev testnet")]
+#[command(next_help_heading = "Dev testnet")]
 pub struct DevArgs {
     /// Start the node in dev mode
     ///
@@ -49,7 +49,7 @@ mod tests {
     /// A helper type to parse Args more easily
     #[derive(Parser)]
     struct CommandParser<T: Args> {
-        #[clap(flatten)]
+        #[command(flatten)]
         args: T,
     }