Fix locked notes reuse by ensuring retries after blockchain sync

Now, transactions will only be constructed with confirmed, available
notes, improving reliability and preventing conflicts.

Author: borngraced

mm2src/coins/Cargo.toml

@@ -57,6 +57,7 @@ hex = "0.4.2"
 http = "0.2"
 itertools = { version = "0.10", features = ["use_std"] }
 jsonrpc-core = "18.0.0"
+jubjub = "0.5.1"
 keys = { path = "../mm2_bitcoin/keys" }
 lazy_static = "1.4"
 libc = "0.2"
@@ -119,7 +120,6 @@ blake2b_simd = "0.5"
 ff = "0.8"
 futures-util = "0.3"
 instant = "0.1.12"
-jubjub = "0.5.1"
 js-sys = { version = "0.3.27" }
 mm2_db = { path = "../mm2_db" }
 mm2_metamask = { path = "../mm2_metamask" }

mm2src/coins/lp_coins.rs

@@ -84,7 +84,6 @@ use std::sync::Arc;
 use std::time::Duration;
 use std::{fmt, iter};
 use utxo_signer::with_key_pair::UtxoSignWithKeyPairError;
-use uuid::Uuid;
 use zcash_primitives::transaction::Transaction as ZTransaction;
 cfg_native! {
     use crate::lightning::LightningCoin;
@@ -1203,8 +1202,6 @@ pub trait SwapOps {
 
     fn maker_locktime_multiplier(&self) -> f64 { 2.0 }
 
-    async fn clean_up(&self, _uuid: Uuid) -> MmResult<(), String> { Ok(()) }
-
     /// Performs an action on Maker coin payment just before the Taker Swap payment refund begins
     /// Operation on maker coin from taker swap side
     /// Currently lightning specific