From 7c805b5c25a4ca650b7782dea87d63309b8d6876 Mon Sep 17 00:00:00 2001
From: Nicola Busanello <nicola.busanello@gmail.com>
Date: Fri, 8 Mar 2024 13:46:27 +0100
Subject: [PATCH] add missing serde feature cfg attributes

---
 src/descriptor.rs | 18 +++++++++---------
 src/runtime.rs    |  7 ++++++-
 2 files changed, 15 insertions(+), 10 deletions(-)

diff --git a/src/descriptor.rs b/src/descriptor.rs
index 75107e2..cf924d2 100644
--- a/src/descriptor.rs
+++ b/src/descriptor.rs
@@ -50,8 +50,8 @@ pub trait DescriptorRgb<K = XpubDerivable, V = ()>: Descriptor<K, V> {
 }
 
 #[derive(Copy, Clone, Ord, PartialOrd, Eq, PartialEq, Hash, Debug, Display)]
-#[derive(Serialize, Deserialize)]
-#[serde(crate = "serde_crate", rename_all = "camelCase")]
+#[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
+#[cfg_attr(feature = "serde", serde(crate = "serde_crate", rename_all = "camelCase"))]
 #[repr(u8)]
 pub enum RgbKeychain {
     #[display("0", alt = "0")]
@@ -106,15 +106,15 @@ impl From<RgbKeychain> for Keychain {
     fn from(keychain: RgbKeychain) -> Self { Keychain::from(keychain as u8) }
 }
 
-#[serde_as]
+#[cfg_attr(feature = "serde", serde_as)]
 #[derive(Clone, Eq, PartialEq, Debug)]
-#[derive(Serialize, Deserialize)]
-#[serde(crate = "serde_crate", rename_all = "camelCase")]
+#[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
+#[cfg_attr(feature = "serde", serde(crate = "serde_crate", rename_all = "camelCase"))]
 pub struct TapretKey<K: DeriveXOnly = XpubDerivable> {
     pub internal_key: K,
     // TODO: Allow multiple tweaks per index by introducing derivation using new Terminal trait
     // TODO: Change serde implementation for both Terminal and TapretCommitment
-    #[serde_as(as = "HashMap<serde_with::DisplayFromStr, serde_with::DisplayFromStr>")]
+    #[cfg_attr(feature = "serde", serde_as(as = "HashMap<serde_with::DisplayFromStr, serde_with::DisplayFromStr>"))]
     pub tweaks: HashMap<Terminal, TapretCommitment>,
 }
 
@@ -224,15 +224,15 @@ impl<K: DeriveXOnly> DescriptorRgb<K> for TapretKey<K> {
 }
 
 #[derive(Clone, Eq, PartialEq, Debug, From)]
-#[derive(Serialize, Deserialize)]
-#[serde(
+#[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
+#[cfg_attr(feature = "serde", serde(
     crate = "serde_crate",
     rename_all = "camelCase",
     bound(
         serialize = "S::Compr: serde::Serialize, S::XOnly: serde::Serialize",
         deserialize = "S::Compr: serde::Deserialize<'de>, S::XOnly: serde::Deserialize<'de>"
     )
-)]
+))]
 #[non_exhaustive]
 pub enum RgbDescr<S: DeriveSet = XpubDerivable> {
     #[from]
diff --git a/src/runtime.rs b/src/runtime.rs
index ef4040e..dbc8df9 100644
--- a/src/runtime.rs
+++ b/src/runtime.rs
@@ -24,6 +24,7 @@
 use std::collections::HashMap;
 use std::convert::Infallible;
 use std::io;
+#[cfg(feature = "serde")]
 use std::io::ErrorKind;
 use std::ops::{Deref, DerefMut};
 use std::path::PathBuf;
@@ -31,6 +32,7 @@ use std::path::PathBuf;
 use amplify::IoError;
 use bpstd::{Network, XpubDerivable};
 use bpwallet::Wallet;
+#[cfg(feature = "serde")]
 use rgbfs::StockFs;
 use rgbstd::containers::{Contract, LoadError, Transfer};
 use rgbstd::interface::{
@@ -43,7 +45,9 @@ use rgbstd::persistence::{
 use rgbstd::resolvers::ResolveHeight;
 use rgbstd::validation::{self, ResolveWitness};
 use rgbstd::{AssignmentWitness, ContractId, WitnessId, XChain, XOutpoint};
-use strict_types::encoding::{DecodeError, DeserializeError, Ident, SerializeError, TypeName};
+use strict_types::encoding::{DeserializeError, Ident, SerializeError, TypeName};
+#[cfg(feature = "serde")]
+use strict_types::encoding::DecodeError;
 
 use crate::{DescriptorRgb, RgbDescr};
 
@@ -111,6 +115,7 @@ pub enum RuntimeError {
     #[display(doc_comments)]
     ResolverError(crate::AnyResolverError),
 
+    #[cfg(feature = "serde")]
     #[from]
     Yaml(serde_yaml::Error),