refactor: Make serde dependency optional

Serde should be optional in case downstream crates don't need it.

Author: uncomputable

Cargo.toml

@@ -10,11 +10,14 @@ description = "Inline Bitcoin scripts"
 license = "MIT"
 repository = "https://github.com/BitVM/rust-bitcoin-script"
 
+[features]
+serde = ["dep:serde", "bitcoin/serde"]
+
 [dependencies]
-bitcoin = { version = "0.32.5", features = ["serde"] }
+bitcoin = { version = "0.32.5" }
 script-macro = { path = "./macro" }
 stdext = "0.3.3"
-serde = { version = "1", features = ["derive"] }
+serde = { version = "1", features = ["derive"], optional = true }
 
 [dev-dependencies]
 bincode = "1.3.3"

src/builder.rs

@@ -3,12 +3,15 @@ use bitcoin::blockdata::script::{Instruction, PushBytes, PushBytesBuf, ScriptBuf
 use bitcoin::opcodes::{OP_0, OP_TRUE};
 use bitcoin::script::write_scriptint;
 use bitcoin::Witness;
-use serde::{Deserialize, Serialize};
 use std::collections::HashMap;
 use std::convert::TryFrom;
 use std::hash::{DefaultHasher, Hash, Hasher};
 
-#[derive(Clone, Debug, Hash, Serialize, Deserialize, PartialEq)]
+#[cfg(feature = "serde")]
+use serde::{Deserialize, Serialize};
+
+#[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
+#[derive(Clone, Debug, Hash, PartialEq)]
 pub enum Block {
     Call(u64),
     Script(ScriptBuf),
@@ -21,7 +24,8 @@ impl Block {
     }
 }
 
-#[derive(Clone, Debug, Serialize, Deserialize, PartialEq)]
+#[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
+#[derive(Clone, Debug, PartialEq)]
 pub struct StructuredScript {
     size: usize,
     pub debug_identifier: String,

tests/test.rs

@@ -304,6 +304,7 @@ fn test_push_witness() {
     );
 }
 
+#[cfg(feature = "serde")]
 #[test]
 fn test_serialization() {
     let script = script! {