feat(hwi): set hwi as top level command

- add ledger and coldcard integration
- update hwi as top level command
- update CHANGELOG

rebase and fix clippy issues

Author: tvpeter

CHANGELOG.md

@@ -7,6 +7,8 @@ page. See [DEVELOPMENT_CYCLE.md](DEVELOPMENT_CYCLE.md) for more details.
 - Add wallet subcommand `config` to save wallet configs
 - Add `wallets` command to list all wallets saved configs
 
+- Add `hwi` top level command with subcommands: `devices`, `register`, `address` and `sign` transaction
+
 ## [2.0.0]
 
 - Removed MSRV and bumped Rust Edition to 2024
@@ -20,6 +22,7 @@ page. See [DEVELOPMENT_CYCLE.md](DEVELOPMENT_CYCLE.md) for more details.
 - Renamed `BuilderError` to `KyotoBuilderError` and added `KyotoUpdateError`
 - Updated `bdk_electrum` to 0.23.0
 - Added `just` example for starting, connecting and funding a wallet in regtest
+- Add `--pretty` top level flag for formatting commands output in a tabular format
 
 ## [1.0.0]
 

Cargo.lock

@@ -247,6 +247,23 @@ pub struct WalletOpts {
     pub compactfilter_opts: CompactFilterOpts,
 }
 
+/// HWI specific options
+#[cfg(feature = "hwi")]
+#[derive(Clone, Debug, PartialEq, Eq, Args)]
+pub struct HwiOpts {
+    /// Wallet name
+    #[arg(env = "WALLET", short = 'w', long = "wallet")]
+    pub wallet: Option<String>,
+
+    /// External descriptor
+    #[arg(env = "EXT_DESCRIPTOR", short = 'e', long = "ext_descriptor")]
+    pub ext_descriptor: Option<String>,
+
+    /// Database type
+    #[arg(short = 'd', long = "database_type")]
+    pub database_type: Option<DatabaseType>,
+}
+
 /// Options to configure a SOCKS5 proxy for a blockchain client connection.
 #[cfg(any(feature = "electrum", feature = "esplora"))]
 #[derive(Debug, Args, Clone, PartialEq, Eq)]

src/commands.rs

@@ -1314,7 +1314,7 @@ pub(crate) async fn handle_command(cli_opts: CliOpts) -> Result<String, Error> {
                     &wallet_opts,
                     &cli_opts,
                     offline_subcommand.clone(),
-                )?
+                )
             };
             Ok(result?)
         }
@@ -1424,6 +1424,15 @@ pub(crate) async fn handle_command(cli_opts: CliOpts) -> Result<String, Error> {
             let descriptor = handle_descriptor_command(cli_opts.network, desc_type, key, pretty)?;
             Ok(descriptor)
         }
+
+        #[cfg(feature = "hwi")]
+        CliSubCommand::Hwi {
+            hwi_opts,
+            subcommand,
+        } => {
+            let result = handle_hwi_subcommand(network, &hwi_opts, subcommand).await?;
+            Ok(serde_json::to_string_pretty(&result).map_err(|e| Error::SerdeJson(e))?)
+        }
     };
     result
 }

src/handlers.rs

@@ -38,8 +38,9 @@ use bdk_wallet::{
 use cli_table::{Cell, CellStruct, Style, Table};
 #[cfg(feature = "hwi")]
 use {
+    crate::commands::HwiOpts,
     async_hwi::jade::{self, Jade},
-    async_hwi::ledger::{HidApi, LedgerSimulator},
+    async_hwi::ledger::{HidApi, Ledger, LedgerSimulator, TransportHID},
     async_hwi::specter::{Specter, SpecterSimulator},
     async_hwi::{HWI, coldcard},
     async_hwi::{