fix: Apply cargo fmt and update CI for Node.js deprecation

- Run cargo fmt to fix formatting across cli, core, and web crates
- Update actions/checkout from v4 to v5 to resolve Node.js 20 deprecation
- Update CI workflow trigger from master to main

Author: patsoffice

.github/workflows/ci.yml

@@ -14,7 +14,7 @@ jobs:
     name: Check
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v5
 
       - uses: dtolnay/rust-toolchain@stable
         with:

.github/workflows/release.yml

@@ -18,7 +18,7 @@ jobs:
     name: Docker Image
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v5
 
       - name: Log in to GHCR
         uses: docker/login-action@v3
@@ -58,7 +58,7 @@ jobs:
             os: macos-latest
             artifact: aliasman-aarch64-macos.tar.gz
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v5
 
       - uses: dtolnay/rust-toolchain@stable
         with:

crates/aliasman-cli/src/commands/alias.rs

@@ -209,16 +209,19 @@ pub async fn handle(
                     );
                 }
                 if let Some(ref desc) = description {
-                    println!(
-                        "  description: {:?} -> {:?}",
-                        existing.description, desc
-                    );
+                    println!("  description: {:?} -> {:?}", existing.description, desc);
                 }
             } else {
                 storage.open(false).await?;
-                let result =
-                    edit_alias(storage, email, alias, domain, new_addresses, description.clone())
-                        .await;
+                let result = edit_alias(
+                    storage,
+                    email,
+                    alias,
+                    domain,
+                    new_addresses,
+                    description.clone(),
+                )
+                .await;
                 let close_result = storage.close().await;
                 let updated = result?;
                 close_result?;

crates/aliasman-cli/src/commands/audit.rs

@@ -29,9 +29,7 @@ pub async fn handle(
 
     for diff in &result.diffs {
         match diff {
-            AuditDiff::StorageOnly {
-                alias, domain, ..
-            } => {
+            AuditDiff::StorageOnly { alias, domain, .. } => {
                 println!("  MISSING FROM EMAIL  {}@{}", alias, domain);
                 println!("    Active in storage but not found on email provider");
             }

crates/aliasman-core/src/lib.rs

@@ -118,10 +118,13 @@ pub async fn edit_alias(
     new_addresses: Option<Vec<String>>,
     new_description: Option<String>,
 ) -> Result<Alias> {
-    let mut existing = storage.get(alias, domain).await?.ok_or_else(|| Error::AliasNotFound {
-        alias: alias.to_string(),
-        domain: domain.to_string(),
-    })?;
+    let mut existing = storage
+        .get(alias, domain)
+        .await?
+        .ok_or_else(|| Error::AliasNotFound {
+            alias: alias.to_string(),
+            domain: domain.to_string(),
+        })?;
 
     let addresses_changed = new_addresses
         .as_ref()

crates/aliasman-web/src/routes.rs

@@ -7,8 +7,8 @@ use axum::{Form, Router};
 use rust_embed::Embed;
 use serde::Deserialize;
 
-use aliasman_core::model::{Alias, AliasFilter};
 use aliasman_core::build_alias;
+use aliasman_core::model::{Alias, AliasFilter};
 
 use crate::error::AppError;
 use crate::state::SharedState;
@@ -160,8 +160,14 @@ pub fn router(state: SharedState) -> Router {
     Router::new()
         .route("/", get(index_handler))
         .route("/aliases", get(aliases_handler))
-        .route("/aliases/create", get(create_form_handler).post(create_alias_handler))
-        .route("/aliases/edit", get(edit_form_handler).post(edit_alias_handler))
+        .route(
+            "/aliases/create",
+            get(create_form_handler).post(create_alias_handler),
+        )
+        .route(
+            "/aliases/edit",
+            get(edit_form_handler).post(edit_alias_handler),
+        )
         .route("/aliases/delete", post(delete_alias_handler))
         .route("/aliases/suspend", post(suspend_alias_handler))
         .route("/aliases/unsuspend", post(unsuspend_alias_handler))
@@ -275,9 +281,7 @@ async fn refresh_handler(
     })?))
 }
 
-async fn create_form_handler(
-    State(state): State<SharedState>,
-) -> Result<Html<String>, AppError> {
+async fn create_form_handler(State(state): State<SharedState>) -> Result<Html<String>, AppError> {
     let default_domain = state.active_default_domain().await.unwrap_or_default();
     let default_addresses = state
         .active_default_addresses()
@@ -319,16 +323,15 @@ async fn create_alias_handler(
     };
 
     let template = CreateResultTemplate { success, message };
-    let html = template.render().map_err(|e| {
-        AppError::Internal(format!("template render error: {}", e))
-    })?;
+    let html = template
+        .render()
+        .map_err(|e| AppError::Internal(format!("template render error: {}", e)))?;
 
     let mut response = Html(html).into_response();
     if success {
-        response.headers_mut().insert(
-            "HX-Trigger",
-            "alias-changed".parse().unwrap(),
-        );
+        response
+            .headers_mut()
+            .insert("HX-Trigger", "alias-changed".parse().unwrap());
     }
     Ok(response)
 }
@@ -424,16 +427,15 @@ fn alias_action_response(
     };
 
     let template = ActionResultTemplate { success, message };
-    let html = template.render().map_err(|e| {
-        AppError::Internal(format!("template render error: {}", e))
-    })?;
+    let html = template
+        .render()
+        .map_err(|e| AppError::Internal(format!("template render error: {}", e)))?;
 
     let mut response = Html(html).into_response();
     if success {
-        response.headers_mut().insert(
-            "HX-Trigger",
-            "alias-changed".parse().unwrap(),
-        );
+        response
+            .headers_mut()
+            .insert("HX-Trigger", "alias-changed".parse().unwrap());
     }
     Ok(response)
 }

crates/aliasman-web/src/state.rs

@@ -3,10 +3,10 @@ use std::sync::Arc;
 use tokio::sync::RwLock;
 
 use aliasman_core::config::AppConfig;
-use aliasman_core::{create_email_provider, create_storage_provider};
 use aliasman_core::error::Result as CoreResult;
 use aliasman_core::model::{Alias, AliasFilter};
 use aliasman_core::storage::StorageProvider;
+use aliasman_core::{create_email_provider, create_storage_provider};
 
 pub type SharedState = Arc<AppState>;