Add Snafu error types and proper Rust typed functions

Cargo.toml

@@ -13,6 +13,7 @@ bitcoin = { version =  "0.31.2" }
 bitcoin_hashes = "0.14.0"
 hex = "0.4.3"
 miniscript = "11.0.0"
+snafu = "0.8.4"
 
 [dev-dependencies]
 pretty_assertions = "1.4.0"

src/error.rs

@@ -1,8 +1,63 @@
-#[derive(Debug, PartialEq, Eq)]
-pub enum Bip322Error {
-  InvalidAddress, // for legacy addresses 1... (p2pkh) not supported, also any non taproot
-  Invalid,        // Address no key; pubkey not recovered, invalid signature
-  MalformedSignature, // wrong length, etc.
-  InvalidSigHash, // only sighash All and Default supported
-  NotKeyPathSpend, // only single key path spend supported
+use super::*;
+
+#[derive(Debug, Snafu)]
+#[snafu(context(suffix(false)), visibility(pub))]
+pub enum Error {
+  #[snafu(display("Failed to parse address `{address}`"))]
+  AddressParse {
+    source: bitcoin::address::ParseError,
+    address: String,
+  },
+  #[snafu(display("Failed to parse private key"))]
+  PrivateKeyParse { source: bitcoin::key::Error },
+  #[snafu(display("Unsuported address `{address}`, only P2TR allowed"))]
+  UnsupportedAddress { address: String },
+  #[snafu(display("Decode error for signature `{signature}`"))]
+  SignatureDecode {
+    source: base64::DecodeError,
+    signature: String,
+  },
+  #[snafu(display("Transaction encode error"))]
+  TransactionEncode { source: std::io::Error },
+  #[snafu(display("Transaction extract error"))]
+  TransactionExtract {
+    source: bitcoin::psbt::ExtractTxError,
+  },
+  #[snafu(display("To sign transaction invalid"))]
+  ToSignInvalid,
+  #[snafu(display("PSBT extract error"))]
+  PsbtExtract { source: bitcoin::psbt::Error },
+  #[snafu(display("Base64 decode error for transaction `{transaction}`"))]
+  TransactionBase64Decode {
+    source: base64::DecodeError,
+    transaction: String,
+  },
+  #[snafu(display("Consensus decode error for transaction `{transaction}`"))]
+  TransactionConsensusDecode {
+    source: bitcoin::consensus::encode::Error,
+    transaction: String,
+  },
+  #[snafu(display("Witness malformed"))]
+  WitnessMalformed {
+    source: bitcoin::consensus::encode::Error,
+  },
+  #[snafu(display("Witness empty"))]
+  WitnessEmpty,
+  #[snafu(display("Encode witness error"))]
+  WitnessEncoding { source: std::io::Error },
+  #[snafu(display("Signature of wrong length `{length}`"))]
+  SignatureLength {
+    length: usize,
+    encoded_signature: Vec<u8>,
+  },
+  #[snafu(display("Invalid signature"))]
+  SignatureInvalid { source: bitcoin::secp256k1::Error },
+  #[snafu(display("Invalid sighash"))]
+  SigHashTypeInvalid {
+    source: bitcoin::sighash::InvalidSighashTypeError,
+  },
+  #[snafu(display("Unsupported sighash type `{sighash_type}`"))]
+  SigHashTypeUnsupported { sighash_type: String },
+  #[snafu(display("Not key path spend"))]
+  NotKeyPathSpend,
 }