Migrate to latest revision of the Zcash crates

Cargo.toml

@@ -11,6 +11,7 @@ anyhow = "1"
 bip0039 = { version = "0.12", features = ["std", "all-languages"] }
 futures-util = "0.3"
 gumdrop = "0.8"
+hex = "0.4"
 prost = "0.13"
 rayon = "1.7"
 rusqlite = { version = "0.32", features = ["time"] }
@@ -23,6 +24,7 @@ toml = "0.8"
 tonic = { version = "0.12", features = ["gzip", "tls-webpki-roots"] }
 tracing = "0.1"
 tracing-subscriber = { version = "0.3", features = ["env-filter", "fmt"] }
+uuid = "1"
 
 orchard = { version = "0.10", default-features = false }
 sapling = { package = "sapling-crypto", version = "0.3" }
@@ -33,6 +35,7 @@ zcash_keys = { version = "0.5", features = ["unstable", "orchard"] }
 zcash_primitives = "0.20"
 zcash_proofs = "0.20"
 zcash_protocol = "0.4"
+zip32 = "0.1"
 zip321 = "0.2"
 
 # Seed encryption
@@ -61,3 +64,13 @@ tui = [
     "dep:tokio-util",
     "dep:tui-logger",
 ]
+
+[patch.crates-io]
+zcash_address = { git = "https://github.com/zcash/librustzcash.git", rev = "e0f04e6c7749751e7f590b2c25275f1fa3421d50" }
+zcash_client_backend = { git = "https://github.com/zcash/librustzcash.git", rev = "e0f04e6c7749751e7f590b2c25275f1fa3421d50" }
+zcash_client_sqlite = { git = "https://github.com/zcash/librustzcash.git", rev = "e0f04e6c7749751e7f590b2c25275f1fa3421d50" }
+zcash_keys = { git = "https://github.com/zcash/librustzcash.git", rev = "e0f04e6c7749751e7f590b2c25275f1fa3421d50" }
+zcash_primitives = { git = "https://github.com/zcash/librustzcash.git", rev = "e0f04e6c7749751e7f590b2c25275f1fa3421d50" }
+zcash_proofs = { git = "https://github.com/zcash/librustzcash.git", rev = "e0f04e6c7749751e7f590b2c25275f1fa3421d50" }
+zcash_protocol = { git = "https://github.com/zcash/librustzcash.git", rev = "e0f04e6c7749751e7f590b2c25275f1fa3421d50" }
+zip321 = { git = "https://github.com/zcash/librustzcash.git", rev = "e0f04e6c7749751e7f590b2c25275f1fa3421d50" }

src/commands/balance.rs

@@ -65,17 +65,17 @@ impl Command {
             }
             println!("    Balance: {}", printer.format(balance.total()));
             println!(
-                "  Sapling Spendable: {}",
+                "     Sapling Spendable: {}",
                 printer.format(balance.sapling_balance().spendable_value()),
             );
             println!(
-                "  Orchard Spendable: {}",
+                "     Orchard Spendable: {}",
                 printer.format(balance.orchard_balance().spendable_value()),
             );
             #[cfg(feature = "transparent-inputs")]
             println!(
-                "         Unshielded: {}",
-                printer.format(balance.unshielded()),
+                "  Unshielded Spendable: {}",
+                printer.format(balance.unshielded_balance().spendable_value()),
             );
         } else {
             println!("Insufficient information to build a wallet summary.");

src/commands/import_ufvk.rs

@@ -3,12 +3,13 @@ use gumdrop::Options;
 
 use zcash_address::unified::{self, Encoding};
 use zcash_client_backend::{
-    data_api::{AccountBirthday, AccountPurpose, WalletWrite},
+    data_api::{AccountBirthday, AccountPurpose, WalletWrite, Zip32Derivation},
     proto::service,
 };
 use zcash_client_sqlite::WalletDb;
 use zcash_keys::keys::UnifiedFullViewingKey;
 use zcash_primitives::consensus;
+use zip32::fingerprint::SeedFingerprint;
 
 use crate::{
     data::get_db_paths,
@@ -19,14 +20,23 @@ use crate::{
 // Options accepted for the `import-ufvk` command
 #[derive(Debug, Options)]
 pub(crate) struct Command {
+    #[options(help = "a name for the account")]
+    name: String,
+
     #[options(free, required, help = "The Unified Full Viewing Key to import")]
     ufvk: String,
 
     #[options(free, required, help = "the UFVK's birthday")]
     birthday: u32,
 
-    #[options(help = "can the wallet omit information needed to spend funds (default is false)")]
-    view_only: bool,
+    #[options(
+        help = "hex encoding of the ZIP 32 fingerprint for the seed from which the UFVK was derived",
+        parse(try_from_str = "hex::decode")
+    )]
+    seed_fingerprint: Option<Vec<u8>>,
+
+    #[options(help = "ZIP 32 account index corresponding to the UFVK")]
+    hd_account_index: Option<u32>,
 
     #[options(
         help = "the server to initialize with (default is \"ecc\")",
@@ -84,16 +94,24 @@ impl Command {
                 .map_err(error::Error::from)?
         };
 
+        let purpose = match (self.seed_fingerprint, self.hd_account_index) {
+            (Some(seed_fingerprint), Some(hd_account_index)) => Ok(AccountPurpose::Spending {
+                derivation: Some(Zip32Derivation::new(
+                    SeedFingerprint::from_bytes(
+                        seed_fingerprint
+                            .as_slice()
+                            .try_into()
+                            .map_err(|_| anyhow!("Incorrect seed_fingerprint length"))?,
+                    ),
+                    zip32::AccountId::try_from(hd_account_index)?,
+                )),
+            }),
+            (None, None) => Ok(AccountPurpose::ViewOnly),
+            _ => Err(anyhow!("Need either both (for spending) or neither (for view-only) of seed_fingerprint and hd_account_index")),
+        }?;
+
         // Import the UFVK.
-        db_data.import_account_ufvk(
-            &ufvk,
-            &birthday,
-            if self.view_only {
-                AccountPurpose::ViewOnly
-            } else {
-                AccountPurpose::Spending
-            },
-        )?;
+        db_data.import_account_ufvk(&self.name, &ufvk, &birthday, purpose, None)?;
 
         Ok(())
     }

src/commands/init.rs

@@ -20,6 +20,9 @@ use crate::{
 // Options accepted for the `init` command
 #[derive(Debug, Options)]
 pub(crate) struct Command {
+    #[options(help = "a name for the account")]
+    name: String,
+
     #[options(help = "age identity file to encrypt the mnemonic phrase to")]
     identity: String,
 
@@ -29,9 +32,6 @@ pub(crate) struct Command {
     #[options(help = "the wallet's birthday (default is current chain height)")]
     birthday: Option<u32>,
 
-    #[options(help = "the number of accounts to initialise the wallet with (default is 1)")]
-    accounts: Option<usize>,
-
     #[options(
         help = "the network the wallet will be used with: \"test\" or \"main\" (default is \"test\")",
         parse(try_from_str = "Network::parse")
@@ -110,9 +110,10 @@ impl Command {
         Self::init_dbs(
             params,
             wallet_dir.as_ref(),
+            &opts.name,
             &seed,
             birthday,
-            opts.accounts.unwrap_or(1),
+            None,
         )
     }
 
@@ -137,17 +138,16 @@ impl Command {
     pub(crate) fn init_dbs(
         params: impl Parameters + 'static,
         wallet_dir: Option<&String>,
+        account_name: &str,
         seed: &SecretVec<u8>,
         birthday: AccountBirthday,
-        accounts: usize,
+        key_source: Option<&str>,
     ) -> Result<(), anyhow::Error> {
         // Initialise the block and wallet DBs.
         let mut db_data = init_dbs(params, wallet_dir)?;
 
-        // Add accounts.
-        for _ in 0..accounts {
-            db_data.create_account(seed, &birthday)?;
-        }
+        // Add account.
+        db_data.create_account(account_name, seed, &birthday, key_source)?;
 
         Ok(())
     }