refactor: format imports using cargo fmt (#258)

Signed-off-by: Lorenzo Delgado <[email protected]>

Author: LNSD

.github/workflows/tests.yml

@@ -11,13 +11,15 @@ jobs:
   fmt:
     name: cargo fmt
     runs-on: ubuntu-latest
-    container:
-      image: rust:1.83-bookworm
     steps:
-      - uses: actions/checkout@v3
-      - run: |
-          rustup component add rustfmt
-          cargo fmt --all -- --check
+      - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
+      - uses: actions-rust-lang/setup-rust-toolchain@11df97af8e8102fd60b60a77dfbf58d40cd843b8 # v1
+        with:
+          # Imports formatting is an unstable feature, so we need to use nightly
+          toolchain: nightly
+          components: rustfmt
+          cache: false
+      - run: cargo +nightly fmt --all -- --check
 
   clippy:
     name: cargo clippy

rustfmt.toml

@@ -1 +1,7 @@
 edition = "2021"
+
+# Enable nightly features required for formatting imports
+unstable_features = true
+imports_granularity = "Crate"
+group_imports = "StdExternalCrate"
+reorder_modules = true

tap_aggregator/src/aggregator.rs

@@ -137,9 +137,9 @@ mod tests {
 
     use alloy::{dyn_abi::Eip712Domain, primitives::Address, signers::local::PrivateKeySigner};
     use rstest::*;
+    use tap_core::{receipt::Receipt, signed_message::EIP712SignedMessage, tap_eip712_domain};
 
     use crate::aggregator;
-    use tap_core::{receipt::Receipt, signed_message::EIP712SignedMessage, tap_eip712_domain};
 
     #[fixture]
     fn keys() -> (PrivateKeySigner, Address) {

tap_aggregator/src/main.rs

@@ -9,9 +9,8 @@ use alloy::{dyn_abi::Eip712Domain, primitives::Address, signers::local::PrivateK
 use anyhow::Result;
 use clap::Parser;
 use log::{debug, info};
-use tap_core::tap_eip712_domain;
-
 use tap_aggregator::{metrics, server};
+use tap_core::tap_eip712_domain;
 
 #[derive(Parser, Debug)]
 #[command(author, version, about, long_about = None)]

tap_aggregator/src/server.rs

@@ -388,21 +388,19 @@ fn create_json_rpc_service(
 #[cfg(test)]
 #[allow(clippy::too_many_arguments)]
 mod tests {
-    use std::collections::HashSet;
-    use std::str::FromStr;
+    use std::{collections::HashSet, str::FromStr};
 
     use alloy::{dyn_abi::Eip712Domain, primitives::Address, signers::local::PrivateKeySigner};
     use jsonrpsee::{core::client::ClientT, http_client::HttpClientBuilder, rpc_params};
-    use rand::prelude::*;
-    use rand::seq::SliceRandom;
+    use rand::{prelude::*, seq::SliceRandom};
     use rstest::*;
-
-    use crate::server;
     use tap_core::{
         rav::ReceiptAggregateVoucher, receipt::Receipt, signed_message::EIP712SignedMessage,
         tap_eip712_domain,
     };
 
+    use crate::server;
+
     #[derive(Clone)]
     struct Keys {
         wallet: PrivateKeySigner,

tap_aggregator/tests/aggregate_test.rs

@@ -4,7 +4,6 @@
 use std::{collections::HashSet, str::FromStr};
 
 use alloy::{primitives::Address, signers::local::PrivateKeySigner};
-
 use jsonrpsee::{core::client::ClientT, http_client::HttpClientBuilder, rpc_params};
 use tap_aggregator::{
     grpc::{tap_aggregator_client::TapAggregatorClient, RavRequest},

tap_core/benches/timeline_aggretion_protocol_benchmark.rs

@@ -10,13 +10,11 @@
 
 use std::str::FromStr;
 
-use alloy::dyn_abi::Eip712Domain;
-use alloy::primitives::Address;
-use alloy::signers::local::PrivateKeySigner;
+use alloy::{dyn_abi::Eip712Domain, primitives::Address, signers::local::PrivateKeySigner};
 use criterion::{black_box, criterion_group, criterion_main, Criterion};
-use tap_core::tap_eip712_domain;
 use tap_core::{
     rav::ReceiptAggregateVoucher, receipt::Receipt, signed_message::EIP712SignedMessage,
+    tap_eip712_domain,
 };
 
 pub fn create_and_sign_receipt(

tap_core/src/error.rs

@@ -4,11 +4,13 @@
 //! Module containing Error type and Result typedef
 //!
 
-use crate::{rav::ReceiptAggregateVoucher, receipt::ReceiptError};
-use alloy::primitives::{Address, SignatureError};
 use std::result::Result as StdResult;
+
+use alloy::primitives::{Address, SignatureError};
 use thiserror::Error as ThisError;
 
+use crate::{rav::ReceiptAggregateVoucher, receipt::ReceiptError};
+
 /// Error type for the TAP protocol
 #[derive(ThisError, Debug)]
 pub enum Error {

tap_core/src/manager/context/memory.rs

@@ -6,17 +6,21 @@
 //! This module provides an in-memory implementation of the TAP manager context.
 //! It is useful for testing and development purposes.
 
+use std::{
+    collections::HashMap,
+    ops::RangeBounds,
+    sync::{Arc, RwLock},
+};
+
+use alloy::primitives::Address;
+use async_trait::async_trait;
+
 use crate::{
     manager::adapters::*,
     rav::SignedRAV,
     receipt::{checks::StatefulTimestampCheck, state::Checking, ReceiptWithState},
     signed_message::MessageId,
 };
-use alloy::primitives::Address;
-use async_trait::async_trait;
-use std::ops::RangeBounds;
-use std::sync::RwLock;
-use std::{collections::HashMap, sync::Arc};
 
 pub type EscrowStorage = Arc<RwLock<HashMap<Address, u128>>>;
 pub type QueryAppraisals = Arc<RwLock<HashMap<MessageId, u128>>>;
@@ -259,6 +263,13 @@ impl EscrowHandler for InMemoryContext {
 }
 
 pub mod checks {
+    use std::{
+        collections::{HashMap, HashSet},
+        sync::{Arc, RwLock},
+    };
+
+    use alloy::{dyn_abi::Eip712Domain, primitives::Address};
+
     use crate::{
         receipt::{
             checks::{Check, CheckError, CheckResult, ReceiptCheck},
@@ -267,11 +278,6 @@ pub mod checks {
         },
         signed_message::MessageId,
     };
-    use alloy::{dyn_abi::Eip712Domain, primitives::Address};
-    use std::{
-        collections::{HashMap, HashSet},
-        sync::{Arc, RwLock},
-    };
 
     pub fn get_full_list_of_checks(
         domain_separator: Eip712Domain,

tap_core/src/rav.rs

@@ -41,12 +41,10 @@ mod request;
 
 use std::cmp;
 
-use alloy::primitives::Address;
-use alloy::sol;
+use alloy::{primitives::Address, sol};
 use serde::{Deserialize, Serialize};
 
-use crate::Error;
-use crate::{receipt::Receipt, signed_message::EIP712SignedMessage};
+use crate::{receipt::Receipt, signed_message::EIP712SignedMessage, Error};
 
 /// EIP712 signed message for ReceiptAggregateVoucher
 pub type SignedRAV = EIP712SignedMessage<ReceiptAggregateVoucher>;