fix(clippy): fixing clippy CI build error with Box due to large enum variant

Author: aagbotemi

src/input.rs

@@ -1,6 +1,7 @@
 use alloc::sync::Arc;
 use alloc::vec::Vec;
 use core::fmt;
+use std::boxed::Box;
 
 use bitcoin::constants::COINBASE_MATURITY;
 use bitcoin::transaction::OutputsIndexError;
@@ -34,9 +35,9 @@ impl TxStatus {
 
 #[derive(Debug, Clone)]
 enum PlanOrPsbtInput {
-    Plan(Plan),
+    Plan(Box<Plan>),
     PsbtInput {
-        psbt_input: psbt::Input,
+        psbt_input: Box<psbt::Input>,
         sequence: Sequence,
         absolute_timelock: absolute::LockTime,
         satisfaction_weight: usize,
@@ -57,7 +58,7 @@ impl PlanOrPsbtInput {
             return Err(FromPsbtInputError::UtxoCheck);
         }
         Ok(Self::PsbtInput {
-            psbt_input,
+            psbt_input: Box::new(psbt_input),
             sequence,
             absolute_timelock: absolute::LockTime::ZERO,
             satisfaction_weight,
@@ -216,7 +217,7 @@ impl Input {
             prev_outpoint: OutPoint::new(tx.compute_txid(), output_index as _),
             prev_txout: tx.tx_out(output_index).cloned()?,
             prev_tx: Some(tx),
-            plan: PlanOrPsbtInput::Plan(plan),
+            plan: PlanOrPsbtInput::Plan(Box::new(plan)),
             status,
             is_coinbase,
         })
@@ -234,7 +235,7 @@ impl Input {
             prev_outpoint,
             prev_txout,
             prev_tx: None,
-            plan: PlanOrPsbtInput::Plan(plan),
+            plan: PlanOrPsbtInput::Plan(Box::new(plan)),
             status,
             is_coinbase,
         }

src/output.rs

@@ -1,3 +1,5 @@
+use std::boxed::Box;
+
 use bitcoin::{Amount, ScriptBuf, TxOut};
 use miniscript::bitcoin;
 
@@ -9,7 +11,7 @@ pub enum ScriptSource {
     /// bitcoin script
     Script(ScriptBuf),
     /// definite descriptor
-    Descriptor(DefiniteDescriptor),
+    Descriptor(Box<DefiniteDescriptor>),
 }
 
 impl From<ScriptBuf> for ScriptSource {
@@ -32,7 +34,7 @@ impl ScriptSource {
 
     /// From descriptor
     pub fn from_descriptor(descriptor: DefiniteDescriptor) -> Self {
-        Self::Descriptor(descriptor)
+        Self::Descriptor(Box::new(descriptor))
     }
 
     /// To ScriptBuf

src/selection.rs

@@ -1,5 +1,5 @@
 use core::fmt::{Debug, Display};
-use std::vec::Vec;
+use std::{boxed::Box, vec::Vec};
 
 use bdk_coin_select::FeeRate;
 use bitcoin::{absolute, transaction, Sequence};
@@ -67,9 +67,9 @@ pub enum CreatePsbtError {
     /// Attempted to mix locktime types.
     LockTypeMismatch,
     /// Missing tx for legacy input.
-    MissingFullTxForLegacyInput(Input),
+    MissingFullTxForLegacyInput(Box<Input>),
     /// Missing tx for segwit v0 input.
-    MissingFullTxForSegwitV0Input(Input),
+    MissingFullTxForSegwitV0Input(Box<Input>),
     /// Psbt error.
     Psbt(bitcoin::psbt::Error),
     /// Update psbt output with descriptor error.
@@ -196,16 +196,16 @@ impl Selection {
                 psbt_input.non_witness_utxo = plan_input.prev_tx().cloned();
                 if psbt_input.non_witness_utxo.is_none() {
                     if witness_version.is_none() {
-                        return Err(CreatePsbtError::MissingFullTxForLegacyInput(
+                        return Err(CreatePsbtError::MissingFullTxForLegacyInput(Box::new(
                             plan_input.clone(),
-                        ));
+                        )));
                     }
                     if params.mandate_full_tx_for_segwit_v0
                         && witness_version == Some(bitcoin::WitnessVersion::V0)
                     {
-                        return Err(CreatePsbtError::MissingFullTxForSegwitV0Input(
+                        return Err(CreatePsbtError::MissingFullTxForSegwitV0Input(Box::new(
                             plan_input.clone(),
-                        ));
+                        )));
                     }
                 }
                 continue;