wip: refactor request.rs to allow type safety

src-xmr-btc-swap/swap/Cargo.toml

@@ -27,17 +27,31 @@ data-encoding = "2.6"
 dialoguer = "0.11"
 digest = "0.10.7"
 directories-next = "2"
-ecdsa_fun = { version = "0.10", default-features = false, features = [ "libsecp_compat", "serde", "adaptor" ] }
+ecdsa_fun = { version = "0.10", default-features = false, features = [
+    "libsecp_compat",
+    "serde",
+    "adaptor",
+] }
 ed25519-dalek = "1"
+erased-serde = "0.4.5"
 futures = { version = "0.3", default-features = false }
 hex = "0.4"
+hyper = "0.14.20"
 itertools = "0.13"
 jsonrpsee = { version = "0.16.2", features = [ "server" ] }
-tower-http = { version = "0.3.4", features = ["full"] }
-tower = { version = "0.4.13", features = ["full"] }
-hyper = "0.14.20"
 jsonrpsee-core = "0.16.2"
-libp2p = { version = "0.42.2", default-features = false, features = [ "tcp-tokio", "yamux", "mplex", "dns-tokio", "noise", "request-response", "websocket", "ping", "rendezvous", "identify" ] }
+libp2p = { version = "0.42.2", default-features = false, features = [
+    "tcp-tokio",
+    "yamux",
+    "mplex",
+    "dns-tokio",
+    "noise",
+    "request-response",
+    "websocket",
+    "ping",
+    "rendezvous",
+    "identify",
+] }
 monero = { version = "0.12", features = [ "serde_support" ] }
 monero-rpc = { path = "../monero-rpc" }
 pem = "3.0"
@@ -45,7 +59,12 @@ proptest = "1"
 qrcode = "0.14"
 rand = "0.8"
 rand_chacha = "0.3"
-reqwest = { version = "0.12", features = [ "http2", "rustls-tls", "stream", "socks" ], default-features = false }
+reqwest = { version = "0.12", features = [
+    "http2",
+    "rustls-tls",
+    "stream",
+    "socks",
+], default-features = false }
 rust_decimal = { version = "1", features = [ "serde-float" ] }
 rust_decimal_macros = "1"
 serde = { version = "1", features = [ "derive" ] }
@@ -53,22 +72,52 @@ serde_cbor = "0.11"
 serde_json = "1"
 serde_with = { version = "1", features = [ "macros" ] }
 sha2 = "0.10"
-sigma_fun = { version = "0.7", default-features = false, features = [ "ed25519", "serde", "secp256k1", "alloc" ] }
-sqlx = { version = "0.6.3", features = [ "sqlite", "runtime-tokio-rustls", "offline" ] }
+sigma_fun = { version = "0.7", default-features = false, features = [
+    "ed25519",
+    "serde",
+    "secp256k1",
+    "alloc",
+] }
+sqlx = { version = "0.6.3", features = [
+    "sqlite",
+    "runtime-tokio-rustls",
+    "offline",
+] }
 structopt = "0.3"
 strum = { version = "0.26", features = [ "derive" ] }
 thiserror = "1"
 time = "0.3"
-tokio = { version = "1", features = [ "rt-multi-thread", "time", "macros", "sync", "process", "fs", "net", "parking_lot" ] }
+tokio = { version = "1", features = [
+    "rt-multi-thread",
+    "time",
+    "macros",
+    "sync",
+    "process",
+    "fs",
+    "net",
+    "parking_lot",
+] }
 tokio-socks = "0.5"
 tokio-tungstenite = { version = "0.15", features = [ "rustls-tls" ] }
 tokio-util = { version = "0.7", features = [ "io", "codec" ] }
 toml = "0.8"
-torut = { version = "0.2", default-features = false, features = [ "v3", "control" ] }
+torut = { version = "0.2", default-features = false, features = [
+    "v3",
+    "control",
+] }
+tower = { version = "0.4.13", features = [ "full" ] }
+tower-http = { version = "0.3.4", features = [ "full" ] }
 tracing = { version = "0.1", features = [ "attributes" ] }
 tracing-appender = "0.2"
 tracing-futures = { version = "0.2", features = [ "std-future", "futures-03" ] }
-tracing-subscriber = { version = "0.3", default-features = false, features = [ "fmt", "ansi", "env-filter", "time", "tracing-log", "json" ] }
+tracing-subscriber = { version = "0.3", default-features = false, features = [
+    "fmt",
+    "ansi",
+    "env-filter",
+    "time",
+    "tracing-log",
+    "json",
+] }
 url = { version = "2", features = [ "serde" ] }
 uuid = { version = "1.9", features = [ "serde", "v4" ] }
 void = "1"
@@ -96,4 +145,8 @@ testcontainers = "0.15"
 
 [build-dependencies]
 anyhow = "1"
-vergen = { version = "8.3", default-features = false, features = [ "build", "git", "git2" ] }
+vergen = { version = "8.3", default-features = false, features = [
+    "build",
+    "git",
+    "git2",
+] }

src-xmr-btc-swap/swap/src/api.rs

@@ -375,6 +375,7 @@ pub mod api_test {
     use crate::api::request::{Method, Request};
 
     use libp2p::Multiaddr;
+    use request::BuyXmrArgs;
     use std::str::FromStr;
     use uuid::Uuid;
 
@@ -431,12 +432,12 @@ pub mod api_test {
                 }
             };
 
-            Request::new(Method::BuyXmr {
+            Request::new(Method::BuyXmr(BuyXmrArgs {
                 seller,
                 bitcoin_change_address,
                 monero_receive_address,
                 swap_id: Uuid::new_v4(),
-            })
+            }))
         }
 
         pub fn resume() -> Request {

src-xmr-btc-swap/swap/src/bin/swap.rs

@@ -12,9 +12,12 @@
 #![forbid(unsafe_code)]
 #![allow(non_snake_case)]
 
+use crate::{
+    cli::command::{parse_args_and_apply_defaults, ParseResult},
+    common::check_latest_version,
+};
 use anyhow::Result;
 use std::env;
-use crate::{cli::command::{parse_args_and_apply_defaults, ParseResult}, common::check_latest_version};
 
 #[tokio::main]
 pub async fn main() -> Result<()> {

src-xmr-btc-swap/swap/src/cli/command.rs

@@ -1,4 +1,7 @@
-use crate::api::request::{Method, Request};
+use crate::api::request::{
+    BalanceArgs, BuyXmrArgs, CancelAndRefundArgs, ListSellersArgs, Method, MoneroRecoveryArgs,
+    Request, ResumeArgs, StartDaemonArgs, WithdrawBtcArgs,
+};
 use crate::api::Context;
 use crate::bitcoin::{bitcoin_address, Amount};
 use crate::monero;
@@ -74,12 +77,12 @@ where
             let bitcoin_change_address =
                 bitcoin_address::validate_is_testnet(bitcoin_change_address, is_testnet)?;
 
-            let request = Request::new(Method::BuyXmr {
+            let request = Request::new(Method::BuyXmr(BuyXmrArgs {
                 seller,
                 bitcoin_change_address,
                 monero_receive_address,
                 swap_id: Uuid::new_v4(),
-            });
+            }));
 
             let context = Context::build(
                 Some(bitcoin),
@@ -109,9 +112,9 @@ where
             (context, request)
         }
         CliCommand::Balance { bitcoin } => {
-            let request = Request::new(Method::Balance {
+            let request = Request::new(Method::Balance(BalanceArgs {
                 force_refresh: true,
-            });
+            }));
 
             let context = Context::build(
                 Some(bitcoin),
@@ -132,7 +135,7 @@ where
             monero,
             tor,
         } => {
-            let request = Request::new(Method::StartDaemon { server_address });
+            let request = Request::new(Method::StartDaemon(StartDaemonArgs { server_address }));
 
             let context = Context::build(
                 Some(bitcoin),
@@ -153,7 +156,7 @@ where
             address,
         } => {
             let address = bitcoin_address::validate_is_testnet(address, is_testnet)?;
-            let request = Request::new(Method::WithdrawBtc { amount, address });
+            let request = Request::new(Method::WithdrawBtc(WithdrawBtcArgs { amount, address }));
 
             let context = Context::build(
                 Some(bitcoin),
@@ -174,7 +177,7 @@ where
             monero,
             tor,
         } => {
-            let request = Request::new(Method::Resume { swap_id });
+            let request = Request::new(Method::Resume(ResumeArgs { swap_id }));
 
             let context = Context::build(
                 Some(bitcoin),
@@ -194,7 +197,7 @@ where
             bitcoin,
             tor,
         } => {
-            let request = Request::new(Method::CancelAndRefund { swap_id });
+            let request = Request::new(Method::CancelAndRefund(CancelAndRefundArgs { swap_id }));
 
             let context = Context::build(
                 Some(bitcoin),
@@ -213,7 +216,7 @@ where
             rendezvous_point,
             tor,
         } => {
-            let request = Request::new(Method::ListSellers { rendezvous_point });
+            let request = Request::new(Method::ListSellers(ListSellersArgs { rendezvous_point }));
 
             let context =
                 Context::build(None, None, Some(tor), data, is_testnet, debug, json, None).await?;
@@ -239,7 +242,7 @@ where
         CliCommand::MoneroRecovery {
             swap_id: SwapId { swap_id },
         } => {
-            let request = Request::new(Method::MoneroRecovery { swap_id });
+            let request = Request::new(Method::MoneroRecovery(MoneroRecoveryArgs { swap_id }));
 
             let context =
                 Context::build(None, None, None, data, is_testnet, debug, json, None).await?;

src-xmr-btc-swap/swap/src/lib.rs

@@ -18,6 +18,7 @@
 
 pub mod api;
 pub mod asb;
+pub mod bin;
 pub mod bitcoin;
 pub mod cli;
 pub mod common;
@@ -27,14 +28,13 @@ pub mod fs;
 pub mod kraken;
 pub mod libp2p_ext;
 pub mod monero;
+mod monero_ext;
 pub mod network;
 pub mod protocol;
 pub mod rpc;
 pub mod seed;
 pub mod tor;
 pub mod tracing_ext;
-pub mod bin;
-mod monero_ext;
 
 #[cfg(test)]
 mod proptest;

src-xmr-btc-swap/swap/src/protocol/bob/swap.rs

@@ -338,12 +338,18 @@ async fn next_state(
                     }
                 }
                 Ok(Rejected { reason, .. }) => {
-                    tracing::error!(?reason, "Alice rejected our request for cooperative XMR redeem");
+                    tracing::error!(
+                        ?reason,
+                        "Alice rejected our request for cooperative XMR redeem"
+                    );
                     return Err(reason)
                         .context("Alice rejected our request for cooperative XMR redeem");
                 }
                 Err(error) => {
-                    tracing::error!(?error, "Failed to request cooperative XMR redeem from Alice");
+                    tracing::error!(
+                        ?error,
+                        "Failed to request cooperative XMR redeem from Alice"
+                    );
                     return Err(error)
                         .context("Failed to request cooperative XMR redeem from Alice");
                 }

src-xmr-btc-swap/swap/src/rpc.rs

@@ -1,11 +1,11 @@
 use crate::api::Context;
 use std::{net::SocketAddr, sync::Arc};
 use thiserror::Error;
-use tower_http::cors::{Any, CorsLayer};
+use tower_http::cors::CorsLayer;
 
 use jsonrpsee::{
-	core::server::host_filtering::AllowHosts,
-	server::{RpcModule, ServerBuilder, ServerHandle},
+    core::server::host_filtering::AllowHosts,
+    server::{RpcModule, ServerBuilder, ServerHandle},
 };
 
 pub mod methods;
@@ -21,10 +21,13 @@ pub async fn run_server(
     context: Arc<Context>,
 ) -> anyhow::Result<(SocketAddr, ServerHandle)> {
     let cors = CorsLayer::permissive();
-	let middleware = tower::ServiceBuilder::new().layer(cors);
+    let middleware = tower::ServiceBuilder::new().layer(cors);
 
-    let server = ServerBuilder::default().set_host_filtering(AllowHosts::Any)
-    .set_middleware(middleware).build(server_address).await?;
+    let server = ServerBuilder::default()
+        .set_host_filtering(AllowHosts::Any)
+        .set_middleware(middleware)
+        .build(server_address)
+        .await?;
     let mut modules = RpcModule::new(());
     {
         modules

src-xmr-btc-swap/swap/src/rpc/methods.rs

@@ -1,4 +1,7 @@
-use crate::api::request::{Method, Request};
+use crate::api::request::{
+    BalanceArgs, BuyXmrArgs, CancelAndRefundArgs, GetSwapInfoArgs, ListSellersArgs, Method,
+    MoneroRecoveryArgs, Request, ResumeArgs, WithdrawBtcArgs,
+};
 use crate::api::Context;
 use crate::bitcoin::bitcoin_address;
 use crate::monero::monero_address;
@@ -29,7 +32,12 @@ pub fn register_modules(context: Arc<Context>) -> Result<RpcModule<Arc<Context>>
         let swap_id = as_uuid(swap_id)
             .ok_or_else(|| jsonrpsee_core::Error::Custom("Could not parse swap_id".to_string()))?;
 
-        execute_request(params_raw, Method::GetSwapInfo { swap_id }, &context).await
+        execute_request(
+            params_raw,
+            Method::GetSwapInfo(GetSwapInfoArgs { swap_id }),
+            &context,
+        )
+        .await
     })?;
 
     module.register_async_method("get_bitcoin_balance", |params_raw, context| async move {
@@ -45,7 +53,12 @@ pub fn register_modules(context: Arc<Context>) -> Result<RpcModule<Arc<Context>>
                 jsonrpsee_core::Error::Custom("force_refesh is not a boolean".to_string())
             })?;
 
-        execute_request(params_raw, Method::Balance { force_refresh }, &context).await
+        execute_request(
+            params_raw,
+            Method::Balance(BalanceArgs { force_refresh }),
+            &context,
+        )
+        .await
     })?;
 
     module.register_async_method("get_history", |params, context| async move {
@@ -66,7 +79,7 @@ pub fn register_modules(context: Arc<Context>) -> Result<RpcModule<Arc<Context>>
         let swap_id = as_uuid(swap_id)
             .ok_or_else(|| jsonrpsee_core::Error::Custom("Could not parse swap_id".to_string()))?;
 
-        execute_request(params_raw, Method::Resume { swap_id }, &context).await
+        execute_request(params_raw, Method::Resume(ResumeArgs { swap_id }), &context).await
     })?;
 
     module.register_async_method("cancel_refund_swap", |params_raw, context| async move {
@@ -79,7 +92,12 @@ pub fn register_modules(context: Arc<Context>) -> Result<RpcModule<Arc<Context>>
         let swap_id = as_uuid(swap_id)
             .ok_or_else(|| jsonrpsee_core::Error::Custom("Could not parse swap_id".to_string()))?;
 
-        execute_request(params_raw, Method::CancelAndRefund { swap_id }, &context).await
+        execute_request(
+            params_raw,
+            Method::CancelAndRefund(CancelAndRefundArgs { swap_id }),
+            &context,
+        )
+        .await
     })?;
 
     module.register_async_method(
@@ -95,7 +113,12 @@ pub fn register_modules(context: Arc<Context>) -> Result<RpcModule<Arc<Context>>
                 jsonrpsee_core::Error::Custom("Could not parse swap_id".to_string())
             })?;
 
-            execute_request(params_raw, Method::MoneroRecovery { swap_id }, &context).await
+            execute_request(
+                params_raw,
+                Method::MoneroRecovery(MoneroRecoveryArgs { swap_id }),
+                &context,
+            )
+            .await
         },
     )?;
 
@@ -123,10 +146,10 @@ pub fn register_modules(context: Arc<Context>) -> Result<RpcModule<Arc<Context>>
 
         execute_request(
             params_raw,
-            Method::WithdrawBtc {
+            Method::WithdrawBtc(WithdrawBtcArgs {
                 amount,
                 address: withdraw_address,
-            },
+            }),
             &context,
         )
         .await
@@ -165,12 +188,12 @@ pub fn register_modules(context: Arc<Context>) -> Result<RpcModule<Arc<Context>>
 
         execute_request(
             params_raw,
-            Method::BuyXmr {
+            Method::BuyXmr(BuyXmrArgs {
                 bitcoin_change_address,
                 monero_receive_address,
                 seller,
                 swap_id: Uuid::new_v4(),
-            },
+            }),
             &context,
         )
         .await
@@ -192,9 +215,9 @@ pub fn register_modules(context: Arc<Context>) -> Result<RpcModule<Arc<Context>>
 
         execute_request(
             params_raw,
-            Method::ListSellers {
+            Method::ListSellers(ListSellersArgs {
                 rendezvous_point: rendezvous_point.clone(),
-            },
+            }),
             &context,
         )
         .await