fix: aatif review

agglayer · Feb 12, 2025 · 4492d64 · 4492d64
1 parent 4396276
commit 4492d64
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/crates/aggchain-proof-builder/src/config.rs b/crates/aggchain-proof-builder/src/config.rs
@@ -11,7 +11,7 @@ pub(crate) const HTTP_RPC_NODE_BACKOFF_MAX_RETRIES: u32 = 64;
 #[derive(Serialize, Deserialize, Clone, Debug)]
 #[serde(rename_all = "kebab-case")]
 pub struct AggchainProofBuilderConfig {
-    /// Json rpc endpoint of the l1 node.
+    /// JSON-RPC endpoint of the l1 node.
     #[serde(default = "default_l1_url")]
     pub l1_rpc_endpoint: Url,
 

diff --git a/crates/aggchain-proof-builder/src/lib.rs b/crates/aggchain-proof-builder/src/lib.rs
@@ -38,10 +38,10 @@ pub struct AggchainProofBuilderResponse {
 
 #[derive(thiserror::Error, Debug)]
 pub enum Error {
-    #[error("Alloy error: {0}")]
+    #[error(transparent)]
     AlloyProviderError(anyhow::Error),
 
-    #[error("Alloy transport rpc error: {0:?}")]
+    #[error(transparent)]
     AlloyRpcTransportError(#[from] RpcError<TransportErrorKind>),
 
     #[error(transparent)]

diff --git a/crates/aggchain-proof-service/src/service.rs b/crates/aggchain-proof-service/src/service.rs
@@ -98,7 +98,7 @@ impl tower::Service<AggchainProofServiceRequest> for AggchainProofService {
             // TODO Aggchain proof should be available here from the
             // `aggchain_proof_builder_response`
             let aggchain_proof = AggchainProof::default();
-            debug!("The proof is {:?}", aggchain_proof);
+            debug!(?aggchain_proof);
 
             Ok(AggchainProofServiceResponse {
                 proof: AggchainProof::default(),