Update with review comments

Author: rajarshimaitra

CHANGELOG.md

@@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ## [Unreleased]
 
 - Add distinct `key-value-db` and `sqlite-db` features, keep default as `key-value-db`
+- Reorganize existing codes in separate modules. Change crate type from lib to bin.
+- Rewrite relevant doc comments as `structopt` help document.
+- Update `bdk` and `bdk-reserves` to v0.19.0.
+- Change default database to `sqlite`.
 
 ## [0.5.0]
 

Cargo.toml

@@ -34,11 +34,11 @@ default = ["repl", "sqlite-db"]
 # To use the app in a REPL mode
 repl = ["regex", "rustyline", "fd-lock"]
 
-# Avaialable dataabse options
+# Avaialable databse options
 key-value-db = ["bdk/key-value-db"]
 sqlite-db = ["bdk/sqlite"]
 
-# Avialable blockchain backend options
+# Avialable blockchain client options
 rpc = ["bdk/rpc"]
 electrum = ["bdk/electrum"]
 compact_filters = ["bdk/compact_filters"]

src/handlers.rs

@@ -22,7 +22,7 @@ use crate::commands::OfflineWalletSubCommand::*;
 use crate::commands::OnlineWalletSubCommand::*;
 use crate::commands::*;
 use crate::utils::*;
-use crate::Backend;
+use crate::Nodes;
 use bdk::{database::BatchDatabase, wallet::AddressIndex, Error, FeeRate, KeychainKind, Wallet};
 
 use structopt::StructOpt;
@@ -559,7 +559,7 @@ pub fn get_outpoints_for_address(
 pub fn handle_command(
     cli_opts: CliOpts,
     network: Network,
-    _backend: Backend,
+    _backend: Nodes,
 ) -> Result<String, Error> {
     let result = match cli_opts.subcommand {
         #[cfg(any(

src/main.rs

@@ -10,12 +10,12 @@
 //!
 //! This module describes the app's main() function
 
-mod backend;
 mod commands;
 mod handlers;
+mod nodes;
 mod utils;
 
-use backend::Backend;
+use nodes::Nodes;
 
 use bitcoin::Network;
 
@@ -53,7 +53,7 @@ fn main() {
 
     #[cfg(feature = "regtest-bitcoin")]
     let backend = {
-        Backend::Bitcoin {
+        Nodes::Bitcoin {
             rpc_url: bitcoind.params.rpc_socket.to_string(),
             rpc_auth: bitcoind
                 .params
@@ -71,7 +71,7 @@ fn main() {
         let elect_exe =
             electrsd::downloaded_exe_path().expect("We should always have downloaded path");
         let electrsd = electrsd::ElectrsD::with_conf(elect_exe, &bitcoind, &elect_conf).unwrap();
-        let backend = Backend::Electrum {
+        let backend = Nodes::Electrum {
             electrum_url: electrsd.electrum_url.clone(),
         };
         (electrsd, backend)
@@ -84,17 +84,17 @@ fn main() {
         let elect_exe =
             electrsd::downloaded_exe_path().expect("Electrsd downloaded binaries not found");
         let electrsd = electrsd::ElectrsD::with_conf(elect_exe, &bitcoind, &elect_conf).unwrap();
-        let backend = Backend::Esplora {
+        let backend = Nodes::Esplora {
             esplora_url: electrsd
                 .esplora_url
                 .clone()
                 .expect("Esplora port not open in electrum"),
         };
-        (electrsd, backend)
+        (electrsd, nodes)
     };
 
     #[cfg(not(feature = "regtest-node"))]
-    let backend = Backend::None;
+    let backend = Nodes::None;
 
     match handle_command(cli_opts, network, backend) {
         Ok(result) => println!("{}", result),

src/backend.rs renamed to src/nodes.rs

@@ -6,14 +6,14 @@
 // You may not use this file except in accordance with one or both of these
 // licenses.
 
-//! The Backend
+//! The Node structures
 //!
-//! This module defines the Backend struct and associated operations
+//! This module defines the the backend node structures for `regtest-*` features
 
 #[allow(dead_code)]
-// Different Backend types activated with `regtest-*` mode.
+// Different regtest node types activated with `regtest-*` mode.
 // If `regtest-*` feature not activated, then default is `None`.
-pub enum Backend {
+pub enum Nodes {
     None,
     Bitcoin { rpc_url: String, rpc_auth: String },
     Electrum { electrum_url: String },

src/utils.rs

@@ -13,14 +13,14 @@
 use std::path::PathBuf;
 use std::str::FromStr;
 
+use crate::commands::WalletOpts;
 #[cfg(any(
     feature = "electrum",
     feature = "esplora",
     feature = "compact_filters",
     feature = "rpc"
 ))]
-use crate::backend::Backend;
-use crate::commands::WalletOpts;
+use crate::nodes::Nodes;
 use bdk::bitcoin::secp256k1::Secp256k1;
 use bdk::bitcoin::{Address, Network, OutPoint, Script};
 #[cfg(feature = "compact_filters")]
@@ -199,12 +199,12 @@ pub(crate) fn open_database(wallet_opts: &WalletOpts) -> Result<AnyDatabase, Err
 pub(crate) fn new_blockchain(
     _network: Network,
     wallet_opts: &WalletOpts,
-    _backend: &Backend,
+    _backend: &Nodes,
 ) -> Result<AnyBlockchain, Error> {
     #[cfg(feature = "electrum")]
     let config = {
         let url = match _backend {
-            Backend::Electrum { electrum_url } => electrum_url.to_owned(),
+            Nodes::Electrum { electrum_url } => electrum_url.to_owned(),
             _ => wallet_opts.electrum_opts.server.clone(),
         };
 
@@ -254,7 +254,7 @@ pub(crate) fn new_blockchain(
     #[cfg(feature = "rpc")]
     let config: AnyBlockchainConfig = {
         let (url, auth) = match _backend {
-            Backend::Bitcoin { rpc_url, rpc_auth } => (
+            Nodes::Bitcoin { rpc_url, rpc_auth } => (
                 rpc_url,
                 Auth::Cookie {
                     file: rpc_auth.into(),