Group help options into sections in command help (#2567)

Author: fnando

FULL_HELP_DOCS.md

@@ -17,6 +17,7 @@ use crate::{
     commands::{
         global,
         txn_result::{TxnEnvelopeResult, TxnResult},
+        HEADING_TRANSACTION,
     },
     config::{self, data, network},
     rpc::Error as SorobanRpcError,
@@ -91,7 +92,7 @@ pub struct Cmd {
     pub alias: Option<AliasName>,
 
     /// Build the transaction and only write the base64 xdr to stdout
-    #[arg(long)]
+    #[arg(long, help_heading = HEADING_TRANSACTION)]
     pub build_only: bool,
 }
 

cmd/soroban-cli/src/commands/contract/deploy/asset.rs

@@ -24,6 +24,7 @@ use crate::{
         contract::{self, arg_parsing, build, id::wasm::get_contract_id, upload},
         global,
         txn_result::{TxnEnvelopeResult, TxnResult},
+        HEADING_TRANSACTION,
     },
     config::{self, data, locator, network},
     print::Print,
@@ -66,7 +67,7 @@ pub struct Cmd {
     #[command(flatten)]
     pub resources: resources::Args,
     /// Build the transaction and only write the base64 xdr to stdout
-    #[arg(long)]
+    #[arg(long, help_heading = HEADING_TRANSACTION)]
     pub build_only: bool,
     /// If provided, will be passed to the contract's `__constructor` function with provided arguments for that function as `--arg-name value`
     #[arg(last = true, id = "CONTRACT_CONSTRUCTOR_ARGS")]

cmd/soroban-cli/src/commands/contract/deploy/wasm.rs

@@ -21,6 +21,7 @@ use crate::{
     commands::{
         global,
         txn_result::{TxnEnvelopeResult, TxnResult},
+        HEADING_TRANSACTION,
     },
     config::{self, data, locator, network},
     key, rpc, wasm, Pwd,
@@ -47,7 +48,7 @@ pub struct Cmd {
     pub resources: resources::Args,
 
     /// Build the transaction and only write the base64 xdr to stdout
-    #[arg(long)]
+    #[arg(long, help_heading = HEADING_TRANSACTION)]
     pub build_only: bool,
 }
 

cmd/soroban-cli/src/commands/contract/extend.rs

@@ -24,6 +24,7 @@ use crate::{
         global,
         tx::fetch::fee,
         txn_result::{TxnEnvelopeResult, TxnResult},
+        HEADING_TRANSACTION,
     },
     config::{self, data, locator, network},
     get_spec::{self, get_remote_contract_spec},
@@ -70,7 +71,7 @@ pub struct Cmd {
     pub send: Send,
 
     /// Build the transaction and only write the base64 xdr to stdout
-    #[arg(long)]
+    #[arg(long, help_heading = HEADING_TRANSACTION)]
     pub build_only: bool,
 }
 

cmd/soroban-cli/src/commands/contract/invoke.rs

@@ -20,6 +20,7 @@ use crate::{
         contract::extend,
         global,
         txn_result::{TxnEnvelopeResult, TxnResult},
+        HEADING_TRANSACTION,
     },
     config::{self, data, locator, network},
     key, rpc, wasm, Pwd,
@@ -46,7 +47,7 @@ pub struct Cmd {
     pub resources: crate::resources::Args,
 
     /// Build the transaction and only write the base64 xdr to stdout
-    #[arg(long)]
+    #[arg(long, help_heading = HEADING_TRANSACTION)]
     pub build_only: bool,
 }
 

cmd/soroban-cli/src/commands/contract/restore.rs

@@ -16,6 +16,7 @@ use crate::{
     commands::{
         global,
         txn_result::{TxnEnvelopeResult, TxnResult},
+        HEADING_TRANSACTION,
     },
     config::{self, data, network},
     key,
@@ -50,7 +51,7 @@ pub struct Cmd {
     pub ignore_checks: bool,
 
     /// Build the transaction and only write the base64 xdr to stdout
-    #[arg(long)]
+    #[arg(long, help_heading = HEADING_TRANSACTION)]
     pub build_only: bool,
 
     /// Package to build when --wasm is not provided

cmd/soroban-cli/src/commands/contract/upload.rs

@@ -5,7 +5,7 @@ use clap::Parser;
 use sha2::{Digest, Sha256};
 
 use crate::{
-    commands::global,
+    commands::{global, HEADING_SIGNING},
     config::{locator, secret},
     print::Print,
     signer::{self, Signer},
@@ -57,10 +57,15 @@ pub struct Cmd {
 
     // @dev: Ledger and Lab don't support signing arbitrary messages yet. Once they do, use `sign_with::Args` here.
     /// Sign with a local key or key saved in OS secure storage. Can be an identity (--sign-with-key alice), a secret key (--sign-with-key SC36…), or a seed phrase (--sign-with-key "kite urban…"). If using seed phrase, `--hd-path` defaults to the `0` path.
-    #[arg(long, env = "STELLAR_SIGN_WITH_KEY", hide_env_values = true)]
+    #[arg(
+        long,
+        env = "STELLAR_SIGN_WITH_KEY",
+        hide_env_values = true,
+        help_heading = HEADING_SIGNING
+    )]
     pub sign_with_key: String,
 
-    #[arg(long)]
+    #[arg(long, help_heading = HEADING_SIGNING)]
     /// If using a seed phrase to sign, sets which hierarchical deterministic path to use, e.g. `m/44'/148'/{hd_path}`. Example: `--hd-path 1`. Default: `0`
     pub hd_path: Option<usize>,
 

cmd/soroban-cli/src/commands/message/sign.rs

@@ -26,9 +26,11 @@ pub mod version;
 
 pub mod txn_result;
 
-pub const HEADING_RPC: &str = "Options (RPC)";
-pub const HEADING_ARCHIVE: &str = "Options (Archive)";
-pub const HEADING_GLOBAL: &str = "Options (Global)";
+pub const HEADING_RPC: &str = "RPC Options";
+pub const HEADING_ARCHIVE: &str = "Archive Options";
+pub const HEADING_GLOBAL: &str = "Global Options";
+pub const HEADING_SIGNING: &str = "Signing Options";
+pub const HEADING_TRANSACTION: &str = "Transaction Options";
 const ABOUT: &str =
     "Work seamlessly with Stellar accounts, contracts, and assets from the command line.
 

cmd/soroban-cli/src/commands/mod.rs

@@ -1,5 +1,5 @@
 use crate::{
-    commands::{global, txn_result::TxnEnvelopeResult},
+    commands::{global, txn_result::TxnEnvelopeResult, HEADING_TRANSACTION},
     config::{
         self,
         address::{self, UnresolvedMuxedAccount},
@@ -16,7 +16,7 @@ pub struct Args {
     #[clap(flatten)]
     pub config: config::Args,
     /// Build the transaction and only write the base64 xdr to stdout
-    #[arg(long)]
+    #[arg(long, help_heading = HEADING_TRANSACTION)]
     pub build_only: bool,
 }