review: add thiserror to remote-provers

miden-remote-provers/Cargo.toml

@@ -35,6 +35,7 @@ miden-objects = { workspace = true, default-features = false }
 miden-tx = { workspace = true, default-features = false }
 prost = { version = "0.13", default-features = false, features = ["derive"] }
 serde = { version = "1.0", features = ["derive"] }
+thiserror = "2.0"
 
 [build-dependencies]
 miette = { version = "7.2", features = ["fancy"] }

miden-remote-provers/src/lib.rs

@@ -2,6 +2,8 @@ extern crate alloc;
 
 use alloc::string::String;
 
+use thiserror::Error;
+
 #[cfg(feature = "tx-prover")]
 pub mod generated;
 
@@ -16,26 +18,18 @@ pub const PROTO_MESSAGES: &str = include_str!("../proto/api.proto");
 /// ERRORS
 /// ===============================================================================================
 
-#[derive(Debug)]
+#[derive(Debug, Error)]
 pub enum RemoteProverError {
     /// Indicates that the provided gRPC server endpoint is invalid.
+    #[error("invalid uri {0}")]
     InvalidEndpoint(String),
-
+    #[error("failed to connect to prover {0}")]
     /// Indicates that the connection to the server failed.
     ConnectionFailed(String),
 }
 
-impl std::fmt::Display for RemoteProverError {
-    fn fmt(&self, f: &mut std::fmt::Formatter) -> std::fmt::Result {
-        match self {
-            RemoteProverError::InvalidEndpoint(endpoint) => {
-                write!(f, "Invalid endpoint: {}", endpoint)
-            },
-            RemoteProverError::ConnectionFailed(endpoint) => {
-                write!(f, "Failed to connect to remote prover at: {}", endpoint)
-            },
-        }
+impl From<RemoteProverError> for String {
+    fn from(err: RemoteProverError) -> Self {
+        err.to_string()
     }
 }
-
-impl core::error::Error for RemoteProverError {}