fix: use bon as builder for ClientOption struct

Author: iamquang95

Cargo.lock

@@ -25,6 +25,7 @@ serde.workspace = true
 serde_json.workspace = true
 hex.workspace = true
 k256.workspace = true
+bon.workspace = true
 charon-cluster = { workspace = true }
 charon-k1util = { workspace = true }
 charon-crypto = { workspace = true }

crates/charon/Cargo.toml

@@ -5,6 +5,7 @@
 
 use std::time::Duration;
 
+use bon::Builder;
 use charon_cluster::lock::Lock;
 use reqwest::{Method, StatusCode};
 use url::Url;
@@ -28,25 +29,12 @@ pub struct Client {
 }
 
 /// Options for configuring the Obol API client.
-#[derive(Debug, Default, Clone)]
+#[derive(Debug, Default, Clone, Builder)]
 pub struct ClientOptions {
-    /// HTTP request timeout (defaults to 10 seconds).
+    /// Optional HTTP request timeout override (defaults to 10 seconds).
     pub timeout: Option<Duration>,
 }
 
-impl ClientOptions {
-    /// Creates new default client options.
-    pub fn new() -> Self {
-        Self::default()
-    }
-
-    /// Sets the HTTP request timeout for all Client calls.
-    pub fn timeout(mut self, timeout: Duration) -> Self {
-        self.timeout = Some(timeout);
-        self
-    }
-}
-
 impl Client {
     /// Creates a new Obol API client.
     pub fn new(url_str: &str, options: ClientOptions) -> Result<Self> {
@@ -231,7 +219,15 @@ mod tests {
 
     #[test]
     fn test_new_client_valid_url() {
-        assert!(Client::new("https://api.obol.tech", ClientOptions::default()).is_ok());
+        assert!(
+            Client::new(
+                "https://api.obol.tech",
+                ClientOptions::builder()
+                    .timeout(Duration::from_secs(10))
+                    .build()
+            )
+            .is_ok()
+        );
     }
 
     #[test]