Remove undocumented ledger positional shorthand.

fnando · fnando · commit cc17850afbf9 · 2026-05-05T15:11:52.000-07:00
diff --git a/FULL_HELP_DOCS.md b/FULL_HELP_DOCS.md
@@ -1173,7 +1173,7 @@ Fund an identity on a test network
 
 ###### **Arguments:**
 
-- `<NAME>` — Name of identity to lookup, default test identity used if not provided
+- `<NAME>` — Name of identity to lookup. Required unless `--ledger` is provided
 
 ###### **Options:**
 
diff --git a/cmd/crates/soroban-test/tests/it/integration/ledger/entry.rs b/cmd/crates/soroban-test/tests/it/integration/ledger/entry.rs
@@ -605,7 +605,7 @@ fn claimable_balance_tx_env(sender: &str, destination: &str) -> TransactionEnvel
 
     let source: UnresolvedMuxedAccount = sender.parse().unwrap();
     let resolved_source = source
-        .resolve_muxed_account_sync(&locator::Args::default(), None)
+        .resolve_muxed_account(&locator::Args::default(), None)
         .unwrap();
 
     xdr::Transaction::new_tx(resolved_source, 1000, 1, create_op).into()
@@ -642,7 +642,7 @@ fn liquidity_pool_tx_env(
 
     let source: UnresolvedMuxedAccount = test_account_address.parse().unwrap();
     let resolved_source = source
-        .resolve_muxed_account_sync(&locator::Args::default(), None)
+        .resolve_muxed_account(&locator::Args::default(), None)
         .unwrap();
 
     let tx = xdr::Transaction::new_tx(resolved_source, 1000, 1, op).into();
diff --git a/cmd/soroban-cli/src/commands/contract/deploy/asset.rs b/cmd/soroban-cli/src/commands/contract/deploy/asset.rs
@@ -147,7 +147,7 @@ impl Cmd {
             .verify_network_passphrase(Some(&network.network_passphrase))
             .await?;
 
-        let source_account = config.source_account().await?;
+        let source_account = config.source_account()?;
 
         // Get the account sequence number
         // TODO: use symbols for the method names (both here and in serve)
diff --git a/cmd/soroban-cli/src/commands/contract/deploy/wasm.rs b/cmd/soroban-cli/src/commands/contract/deploy/wasm.rs
@@ -352,7 +352,7 @@ impl Cmd {
         };
 
         let client = network.rpc_client()?;
-        let MuxedAccount::Ed25519(bytes) = config.source_account().await? else {
+        let MuxedAccount::Ed25519(bytes) = config.source_account()? else {
             return Err(Error::OnlyEd25519AccountsAllowed);
         };
         let source_account = AccountId(PublicKey::PublicKeyTypeEd25519(bytes));
diff --git a/cmd/soroban-cli/src/commands/contract/extend.rs b/cmd/soroban-cli/src/commands/contract/extend.rs
@@ -196,7 +196,7 @@ impl Cmd {
         tracing::trace!(?network);
         let keys = self.key.parse_keys(&config.locator, &network)?;
         let client = network.rpc_client()?;
-        let source_account = config.source_account().await?;
+        let source_account = config.source_account()?;
         let extend_to = self.ledgers_to_extend(&client).await?;
 
         // Get the account sequence number
diff --git a/cmd/soroban-cli/src/commands/contract/id.rs b/cmd/soroban-cli/src/commands/contract/id.rs
@@ -18,10 +18,10 @@ pub enum Error {
 }
 
 impl Cmd {
-    pub async fn run(&self) -> Result<(), Error> {
+    pub fn run(&self) -> Result<(), Error> {
         match &self {
             Cmd::Asset(asset) => asset.run()?,
-            Cmd::Wasm(wasm) => wasm.run().await?,
+            Cmd::Wasm(wasm) => wasm.run()?,
         }
         Ok(())
     }
diff --git a/cmd/soroban-cli/src/commands/contract/id/wasm.rs b/cmd/soroban-cli/src/commands/contract/id/wasm.rs
@@ -29,12 +29,12 @@ pub enum Error {
     OnlyEd25519AccountsAllowed,
 }
 impl Cmd {
-    pub async fn run(&self) -> Result<(), Error> {
+    pub fn run(&self) -> Result<(), Error> {
         let salt: [u8; 32] = soroban_spec_tools::utils::padded_hex_from_str(&self.salt, 32)
             .map_err(|_| Error::CannotParseSalt(self.salt.clone()))?
             .try_into()
             .map_err(|_| Error::CannotParseSalt(self.salt.clone()))?;
-        let source_account = match self.config.source_account().await? {
+        let source_account = match self.config.source_account()? {
             xdr::MuxedAccount::Ed25519(uint256) => stellar_strkey::ed25519::PublicKey(uint256.0),
             xdr::MuxedAccount::MuxedEd25519(_) => return Err(Error::OnlyEd25519AccountsAllowed),
         };
diff --git a/cmd/soroban-cli/src/commands/contract/invoke.rs b/cmd/soroban-cli/src/commands/contract/invoke.rs
@@ -316,7 +316,7 @@ impl Cmd {
                 .await?;
 
             client
-                .get_account(&config.source_account().await?.to_string())
+                .get_account(&config.source_account()?.to_string())
                 .await?
         } else {
             if should_send == ShouldSend::DefaultNo {
diff --git a/cmd/soroban-cli/src/commands/contract/mod.rs b/cmd/soroban-cli/src/commands/contract/mod.rs
@@ -169,7 +169,7 @@ impl Cmd {
             Cmd::Extend(extend) => extend.run(global_args).await?,
             Cmd::Alias(alias) => alias.run(global_args)?,
             Cmd::Deploy(deploy) => deploy.run(global_args).await?,
-            Cmd::Id(id) => id.run().await?,
+            Cmd::Id(id) => id.run()?,
             Cmd::Info(info) => info.run(global_args).await?,
             Cmd::Init(init) => init.run(global_args)?,
             Cmd::Inspect(inspect) => {
diff --git a/cmd/soroban-cli/src/commands/contract/restore.rs b/cmd/soroban-cli/src/commands/contract/restore.rs
@@ -165,7 +165,7 @@ impl Cmd {
         tracing::trace!(?network);
         let entry_keys = self.key.parse_keys(&config.locator, &network)?;
         let client = network.rpc_client()?;
-        let source_account = config.source_account().await?;
+        let source_account = config.source_account()?;
 
         // Get the account sequence number
         let account_details = client
diff --git a/cmd/soroban-cli/src/commands/contract/upload.rs b/cmd/soroban-cli/src/commands/contract/upload.rs
@@ -237,7 +237,7 @@ impl Cmd {
         }
 
         // Get the account sequence number
-        let source_account = config.source_account().await?;
+        let source_account = config.source_account()?;
 
         let account_details = client
             .get_account(&source_account.clone().to_string())
diff --git a/cmd/soroban-cli/src/commands/keys/public_key.rs b/cmd/soroban-cli/src/commands/keys/public_key.rs
@@ -1,13 +1,17 @@
 use crate::{
     commands::config::{address, locator},
     config::UnresolvedMuxedAccount,
+    signer::ledger,
 };
 
 #[derive(thiserror::Error, Debug)]
 pub enum Error {
     #[error(transparent)]
     Address(#[from] address::Error),
 
+    #[error(transparent)]
+    Ledger(#[from] ledger::Error),
+
     #[error("--hd-path {0} is out of range for a Ledger account index")]
     HdPathOutOfRange(usize),
 }
@@ -44,19 +48,14 @@ impl Cmd {
         if self.ledger {
             let raw = self.hd_path.unwrap_or(0);
             let index: u32 = raw.try_into().map_err(|_| Error::HdPathOutOfRange(raw))?;
-            return Ok(public_key_from_muxed(
-                UnresolvedMuxedAccount::Ledger(index)
-                    .resolve_muxed_account(&self.locator, None)
-                    .await?,
-            ));
+            return Ok(ledger::new(index).await?.public_key().await?);
         }
         let name = self
             .name
             .as_ref()
             .expect("clap requires `name` unless --ledger is set");
         Ok(public_key_from_muxed(
-            name.resolve_muxed_account(&self.locator, self.hd_path)
-                .await?,
+            name.resolve_muxed_account(&self.locator, self.hd_path)?,
         ))
     }
 }
diff --git a/cmd/soroban-cli/src/commands/snapshot/create.rs b/cmd/soroban-cli/src/commands/snapshot/create.rs
@@ -487,9 +487,7 @@ impl Cmd {
     // G-address or a key name (as in `stellar keys address NAME`).
     fn resolve_account_sync(&self, address: &str) -> Option<AccountId> {
         let address: UnresolvedMuxedAccount = address.parse().ok()?;
-        let muxed_account = address
-            .resolve_muxed_account_sync(&self.locator, None)
-            .ok()?;
+        let muxed_account = address.resolve_muxed_account(&self.locator, None).ok()?;
         Some(muxed_account.account_id())
     }
 
diff --git a/cmd/soroban-cli/src/commands/tx/args.rs b/cmd/soroban-cli/src/commands/tx/args.rs
@@ -54,7 +54,7 @@ pub enum Error {
 
 impl Args {
     pub async fn tx(&self, body: impl Into<xdr::OperationBody>) -> Result<xdr::Transaction, Error> {
-        let source_account = self.source_account().await?;
+        let source_account = self.source_account()?;
         let seq_num = self
             .config
             .next_sequence_number(source_account.clone().account_id())
@@ -121,39 +121,36 @@ impl Args {
         Ok(TxnEnvelopeResult::Res(txn_resp))
     }
 
-    pub async fn source_account(&self) -> Result<xdr::MuxedAccount, Error> {
-        Ok(self.config.source_account().await?)
+    pub fn source_account(&self) -> Result<xdr::MuxedAccount, Error> {
+        Ok(self.config.source_account()?)
     }
 
     pub fn resolve_muxed_address(
         &self,
         address: &UnresolvedMuxedAccount,
     ) -> Result<xdr::MuxedAccount, Error> {
-        Ok(address.resolve_muxed_account_sync(&self.config.locator, self.config.hd_path())?)
+        Ok(address.resolve_muxed_account(&self.config.locator, self.config.hd_path())?)
     }
 
     pub fn resolve_account_id(
         &self,
         address: &UnresolvedMuxedAccount,
     ) -> Result<xdr::AccountId, Error> {
         Ok(address
-            .resolve_muxed_account_sync(&self.config.locator, self.config.hd_path())?
+            .resolve_muxed_account(&self.config.locator, self.config.hd_path())?
             .account_id())
     }
 
-    pub async fn add_op(
+    pub fn add_op(
         &self,
         op_body: impl Into<xdr::OperationBody>,
         tx_env: xdr::TransactionEnvelope,
         op_source: Option<&address::UnresolvedMuxedAccount>,
     ) -> Result<xdr::TransactionEnvelope, Error> {
         let mut source_account = None;
         if let Some(account) = op_source {
-            source_account = Some(
-                account
-                    .resolve_muxed_account(&self.config.locator, self.config.hd_path())
-                    .await?,
-            );
+            source_account =
+                Some(account.resolve_muxed_account(&self.config.locator, self.config.hd_path())?);
         }
         let op = xdr::Operation {
             source_account,
diff --git a/cmd/soroban-cli/src/commands/tx/mod.rs b/cmd/soroban-cli/src/commands/tx/mod.rs
@@ -90,7 +90,7 @@ impl Cmd {
             Cmd::Hash(cmd) => cmd.run(global_args)?,
             Cmd::New(cmd) => cmd.run(global_args).await?,
             Cmd::Edit(cmd) => cmd.run(global_args)?,
-            Cmd::Operation(cmd) => cmd.run(global_args).await?,
+            Cmd::Operation(cmd) => cmd.run(global_args)?,
             Cmd::Send(cmd) => cmd.run(global_args).await?,
             Cmd::Sign(cmd) => cmd.run(global_args).await?,
             Cmd::Simulate(cmd) => cmd.run(global_args).await?,
diff --git a/cmd/soroban-cli/src/commands/tx/op/add/mod.rs b/cmd/soroban-cli/src/commands/tx/op/add/mod.rs
@@ -135,7 +135,7 @@ impl TryFrom<&Cmd> for OperationBody {
 
 impl Cmd {
     #[allow(clippy::too_many_lines)]
-    pub async fn run(&self, _: &global::Args) -> Result<(), Error> {
+    pub fn run(&self, _: &global::Args) -> Result<(), Error> {
         let op = OperationBody::try_from(self)?;
         let res = match self {
             Cmd::AccountMerge(cmd) => cmd.op.tx.add_op(
@@ -248,8 +248,7 @@ impl Cmd {
                 tx_envelope_from_input(&cmd.args.tx_xdr)?,
                 cmd.args.source(),
             ),
-        }
-        .await?;
+        }?;
         println!("{}", res.to_xdr_base64(crate::xdr::Limits::none())?);
         Ok(())
     }
diff --git a/cmd/soroban-cli/src/commands/tx/op/mod.rs b/cmd/soroban-cli/src/commands/tx/op/mod.rs
@@ -16,9 +16,9 @@ pub enum Error {
 }
 
 impl Cmd {
-    pub async fn run(&self, global_args: &global::Args) -> Result<(), Error> {
+    pub fn run(&self, global_args: &global::Args) -> Result<(), Error> {
         match self {
-            Cmd::Add(cmd) => cmd.run(global_args).await?,
+            Cmd::Add(cmd) => cmd.run(global_args)?,
         }
         Ok(())
     }
diff --git a/cmd/soroban-cli/src/config/address.rs b/cmd/soroban-cli/src/config/address.rs
diff --git a/cmd/soroban-cli/src/config/mod.rs b/cmd/soroban-cli/src/config/mod.rs
diff --git a/cmd/soroban-cli/src/tx/builder/asset.rs b/cmd/soroban-cli/src/tx/builder/asset.rs

Original file line number	Diff line number	Diff line change
`@@ -18,10 +18,10 @@ pub enum Error {`
`18`	`18`	`}`
`19`	`19`
`20`	`20`	`impl Cmd {`
`21`		`- pub async fn run(&self) -> Result<(), Error> {`
	`21`	`+ pub fn run(&self) -> Result<(), Error> {`
`22`	`22`	`match &self {`
`23`	`23`	`Cmd::Asset(asset) => asset.run()?,`
`24`		`- Cmd::Wasm(wasm) => wasm.run().await?,`
	`24`	`+ Cmd::Wasm(wasm) => wasm.run()?,`
`25`	`25`	`}`
`26`	`26`	`Ok(())`
`27`	`27`	`}`