From 022003ac5e4a90f72ceb2b2a0b3125e67be972d8 Mon Sep 17 00:00:00 2001 From: Clippy Bot Date: Thu, 25 Jun 2026 20:09:18 +0200 Subject: [PATCH] fix: lints --- .../arbiter-server/src/actors/flow_coordinator/mod.rs | 4 ++-- server/crates/arbiter-server/src/crypto/integrity/v1.rs | 7 +++---- .../arbiter-server/src/peers/operator/session/handlers.rs | 8 ++++---- .../arbiter-server/src/peers/operator/session/mod.rs | 2 +- 4 files changed, 10 insertions(+), 11 deletions(-) diff --git a/server/crates/arbiter-server/src/actors/flow_coordinator/mod.rs b/server/crates/arbiter-server/src/actors/flow_coordinator/mod.rs index c9f8f5b..5a8dc0e 100644 --- a/server/crates/arbiter-server/src/actors/flow_coordinator/mod.rs +++ b/server/crates/arbiter-server/src/actors/flow_coordinator/mod.rs @@ -20,7 +20,7 @@ pub mod client_connect_approval; pub struct FlowCoordinator { pub clients: HashMap>, - /// Maps DB client_id → ActorId for fast connected-client lookup. + /// Maps DB `client_id` → `ActorId` for fast connected-client lookup. client_ids: HashMap, operator_registry: ActorRef, } @@ -94,7 +94,7 @@ impl FlowCoordinator { self.client_ids.contains_key(&client_id) } - /// Returns the DB client_ids of all currently connected SDK clients. + /// Returns the DB `client_ids` of all currently connected SDK clients. /// Used by operator sessions on startup to seed their approved-client set. #[message] pub fn get_connected_client_ids(&self) -> Vec { diff --git a/server/crates/arbiter-server/src/crypto/integrity/v1.rs b/server/crates/arbiter-server/src/crypto/integrity/v1.rs index a88506f..0b053d9 100644 --- a/server/crates/arbiter-server/src/crypto/integrity/v1.rs +++ b/server/crates/arbiter-server/src/crypto/integrity/v1.rs @@ -192,10 +192,9 @@ pub async fn verify_entity( Ok(false) => Err(Error::MacMismatch { entity_kind: E::KIND, }), - Err(SendError::HandlerError(vault::Error::Sealed)) - | Err(SendError::HandlerError(vault::Error::KeyVersionMismatch { .. })) => { - Ok(AttestationStatus::Unavailable) - } + Err(SendError::HandlerError( + vault::Error::Sealed | vault::Error::KeyVersionMismatch { .. }, + )) => Ok(AttestationStatus::Unavailable), Err(_) => Err(Error::VaultSend), } } diff --git a/server/crates/arbiter-server/src/peers/operator/session/handlers.rs b/server/crates/arbiter-server/src/peers/operator/session/handlers.rs index 4bad681..8240817 100644 --- a/server/crates/arbiter-server/src/peers/operator/session/handlers.rs +++ b/server/crates/arbiter-server/src/peers/operator/session/handlers.rs @@ -334,7 +334,7 @@ mod tests { /// Regression test: revocation must delete by access-entry `id`, not by `wallet_id`. /// - /// Before the fix, revoking entry_id=1 would delete all rows where wallet_id=1, + /// Before the fix, revoking `entry_id=1` would delete all rows where `wallet_id=1`, /// wiping out every client's access to wallet #1. #[tokio::test] async fn revoke_deletes_by_entry_id_not_wallet_id() { @@ -395,10 +395,10 @@ mod tests { assert_eq!(still_there, 1, "unrelated entry must not be deleted"); } - /// Regression test: when entry_id and wallet_id differ, only the correct row is removed. + /// Regression test: when `entry_id` and `wallet_id` differ, only the correct row is removed. /// - /// This specifically catches the case where entry.id=5 and wallet_id=1 are different values; - /// the old bug would delete by wallet_id, potentially matching a completely different entry. + /// This specifically catches the case where `entry.id=5` and `wallet_id=1` are different values; + /// the old bug would delete by `wallet_id`, potentially matching a completely different entry. #[tokio::test] async fn revoke_with_mismatched_wallet_and_entry_ids() { use crate::db::models::EvmWalletAccess; diff --git a/server/crates/arbiter-server/src/peers/operator/session/mod.rs b/server/crates/arbiter-server/src/peers/operator/session/mod.rs index eab628a..78bb3c4 100644 --- a/server/crates/arbiter-server/src/peers/operator/session/mod.rs +++ b/server/crates/arbiter-server/src/peers/operator/session/mod.rs @@ -54,7 +54,7 @@ pub struct OperatorSession { sender: Box>, pending_client_approvals: HashMap, PendingClientApproval>, - /// DB client_ids this operator session is allowed to sign for. + /// DB `client_ids` this operator session is allowed to sign for. /// Seeded from currently-connected clients on start, then updated as /// approvals are granted or denied during the session lifetime. approved_client_ids: HashSet,