prefer fields

Author: zerosnacks

crates/wallets/src/wallet_browser/mod.rs

@@ -67,7 +67,7 @@ mod tests {
         connect_wallet(&client, &server, Connection::new(ALICE, 1)).await;
 
         // Check connection state
-        let Connection(address, chain_id) =
+        let Connection { address, chain_id } =
             server.get_connection().expect("expected an active wallet connection");
         assert_eq!(address, ALICE);
         assert_eq!(chain_id, 1);
@@ -82,7 +82,7 @@ mod tests {
         connect_wallet(&client, &server, Connection::new(BOB, 42)).await;
 
         // Check connection state
-        let Connection(address, chain_id) =
+        let Connection { address, chain_id } =
             server.get_connection().expect("expected an active wallet connection");
         assert_eq!(address, BOB);
         assert_eq!(chain_id, 42);
@@ -99,14 +99,14 @@ mod tests {
 
         // Connect Alice, assert connected
         connect_wallet(&client, &server, Connection::new(ALICE, 1)).await;
-        let Connection(address, chain_id) =
+        let Connection { address, chain_id } =
             server.get_connection().expect("expected an active wallet connection");
         assert_eq!(address, ALICE);
         assert_eq!(chain_id, 1);
 
         // Connect Bob, assert switched
         connect_wallet(&client, &server, Connection::new(BOB, 42)).await;
-        let Connection(address, chain_id) =
+        let Connection { address, chain_id } =
             server.get_connection().expect("expected an active wallet connection");
         assert_eq!(address, BOB);
         assert_eq!(chain_id, 42);
@@ -317,7 +317,7 @@ mod tests {
         let mut server = create_server();
         let client = client_with_token(&server);
         server.start().await.unwrap();
-        connect_wallet(&client, &server, Connection(ALICE, 1)).await;
+        connect_wallet(&client, &server, Connection::new(ALICE, 1)).await;
 
         // Create a browser transaction request
         let (tx_request_id, tx_request) = create_browser_transaction_request();

crates/wallets/src/wallet_browser/signer.rs

@@ -44,7 +44,7 @@ impl BrowserSigner {
         let start = Instant::now();
 
         loop {
-            if let Some(Connection(address, chain_id)) = server.get_connection() {
+            if let Some(Connection { address, chain_id }) = server.get_connection() {
                 let _ = sh_println!("Wallet connected: {}", address);
                 let _ = sh_println!("Chain ID: {}", chain_id);
 

crates/wallets/src/wallet_browser/types.rs

@@ -112,11 +112,16 @@ pub(crate) struct BrowserSignResponse {
 
 /// Represents an active connection to a browser wallet.
 #[derive(Debug, Clone, Copy, PartialEq, Eq, Serialize, Deserialize)]
-pub struct Connection(pub Address, pub ChainId);
+pub struct Connection {
+    /// The address of the connected wallet.
+    pub address: Address,
+    /// The chain ID of the connected wallet.
+    pub chain_id: ChainId,
+}
 
 impl Connection {
     /// Create a new connection instance.
     pub fn new(address: Address, chain_id: ChainId) -> Self {
-        Self(address, chain_id)
+        Self { address, chain_id }
     }
 }