chore: cargo fmt

Author: juacker

Makefile

@@ -1,4 +1,4 @@
-.PHONY: help dev build lint check test ci release-retry release-patch release-minor release-major tag-delete
+.PHONY: help dev build fmt fmt-check lint check test ci release-retry release-patch release-minor release-major tag-delete
 
 # Get current version from package.json
 VERSION := $(shell node -p "require('./package.json').version")
@@ -16,6 +16,12 @@ build: ## Build the application
 
 # CI/Linting targets
 
+fmt: ## Format code with cargo fmt
+	cd src-tauri && cargo fmt
+
+fmt-check: ## Check code formatting (same as CI)
+	cd src-tauri && cargo fmt -- --check
+
 lint: ## Run clippy with warnings as errors (same as CI)
 	cd src-tauri && cargo clippy -- -D warnings
 
@@ -25,7 +31,7 @@ check: ## Run cargo check (fast compilation check)
 test: ## Run cargo tests
 	cd src-tauri && cargo test
 
-ci: lint test ## Run all CI checks locally (lint + test)
+ci: fmt-check lint test ## Run all CI checks locally (fmt + lint + test)
 
 # Release management
 

src-tauri/src/api/client.rs

@@ -75,10 +75,7 @@ pub fn create_client() -> Client {
     );
 
     // Set Accept header to prefer JSON responses
-    default_headers.insert(
-        header::ACCEPT,
-        HeaderValue::from_static("application/json"),
-    );
+    default_headers.insert(header::ACCEPT, HeaderValue::from_static("application/json"));
 
     Client::builder()
         // No default timeout - set per-request based on endpoint type

src-tauri/src/api/error.rs

@@ -330,7 +330,10 @@ mod tests {
         let parsed: serde_json::Value = serde_json::from_str(&json).unwrap();
         assert_eq!(parsed["errorCode"], "unauthorized");
         assert_eq!(parsed["errorMsgKey"], "ErrUnauthorized");
-        assert_eq!(parsed["errorMessage"], "Authentication failed - please login again");
+        assert_eq!(
+            parsed["errorMessage"],
+            "Authentication failed - please login again"
+        );
     }
 
     #[test]

src-tauri/src/api/netdata.rs

@@ -85,10 +85,7 @@ impl NetdataApi {
                 .map_err(|e| ApiError::ParseError(e.to_string()))
         } else {
             // Try to parse the error response body to extract Netdata error details
-            let netdata_error: Option<NetdataErrorResponse> = response
-                .json()
-                .await
-                .ok(); // If parsing fails, we'll use defaults
+            let netdata_error: Option<NetdataErrorResponse> = response.json().await.ok(); // If parsing fails, we'll use defaults
 
             Err(ApiError::from_response(status, netdata_error))
         }
@@ -1016,7 +1013,10 @@ mod tests {
         let user: UserInfo = serde_json::from_str(json).unwrap();
         assert_eq!(user.id, "user123");
         assert_eq!(user.email, "test@example.com");
-        assert_eq!(user.avatar_url, Some("https://example.com/avatar.png".to_string()));
+        assert_eq!(
+            user.avatar_url,
+            Some("https://example.com/avatar.png".to_string())
+        );
     }
 
     #[test]
@@ -1030,6 +1030,9 @@ mod tests {
 
         let space: Space = serde_json::from_str(json).unwrap();
         assert_eq!(space.id, "space123");
-        assert_eq!(space.permissions, Some(vec!["read".to_string(), "write".to_string()]));
+        assert_eq!(
+            space.permissions,
+            Some(vec!["read".to_string(), "write".to_string()])
+        );
     }
 }

src-tauri/src/commands/autopilot.rs

@@ -87,13 +87,21 @@ pub async fn get_autopilot_status(
     // Determine status based on context
     if is_all_nodes_room {
         // In All Nodes room - can toggle directly
-        Ok(AutopilotStatus::available(all_nodes_enabled, has_credits, provider_info))
+        Ok(AutopilotStatus::available(
+            all_nodes_enabled,
+            has_credits,
+            provider_info,
+        ))
     } else if all_nodes_enabled {
         // In other room, but All Nodes is enabled - inherited, can't toggle here
         Ok(AutopilotStatus::via_all_nodes(has_credits, provider_info))
     } else {
         // In other room, All Nodes is not enabled - can toggle for this room
-        Ok(AutopilotStatus::available(current_room_enabled, has_credits, provider_info))
+        Ok(AutopilotStatus::available(
+            current_room_enabled,
+            has_credits,
+            provider_info,
+        ))
     }
 }
 

src-tauri/src/config/mod.rs

@@ -11,7 +11,9 @@
 
 pub mod types;
 
-pub use types::{AiProvider, AutopilotStatus, ClaiConfig, ProviderInfo, SpaceAutopilot, SpaceConfig};
+pub use types::{
+    AiProvider, AutopilotStatus, ClaiConfig, ProviderInfo, SpaceAutopilot, SpaceConfig,
+};
 
 use std::fs;
 use std::io::Write;
@@ -80,9 +82,7 @@ impl ConfigManager {
             source: e,
         })?;
 
-        serde_json::from_str(&contents).map_err(|e| ConfigError::Parse {
-            source: e,
-        })
+        serde_json::from_str(&contents).map_err(|e| ConfigError::Parse { source: e })
     }
 
     /// Saves the current config to disk.
@@ -95,9 +95,8 @@ impl ConfigManager {
 
     /// Saves config to file with atomic write.
     fn save_to_file(&self, config: &ClaiConfig) -> Result<(), ConfigError> {
-        let json = serde_json::to_string_pretty(config).map_err(|e| ConfigError::Serialize {
-            source: e,
-        })?;
+        let json = serde_json::to_string_pretty(config)
+            .map_err(|e| ConfigError::Serialize { source: e })?;
 
         // Write to temp file first
         let temp_path = self.config_path.with_extension("json.tmp");
@@ -107,10 +106,11 @@ impl ConfigManager {
             source: e,
         })?;
 
-        file.write_all(json.as_bytes()).map_err(|e| ConfigError::Io {
-            operation: "write temp config file".to_string(),
-            source: e,
-        })?;
+        file.write_all(json.as_bytes())
+            .map_err(|e| ConfigError::Io {
+                operation: "write temp config file".to_string(),
+                source: e,
+            })?;
 
         file.sync_all().map_err(|e| ConfigError::Io {
             operation: "sync temp config file".to_string(),
@@ -152,11 +152,7 @@ impl ConfigManager {
     /// Gets config for a space.
     pub fn get_space_config(&self, space_id: &str) -> SpaceConfig {
         let config = self.config.lock().unwrap();
-        config
-            .spaces
-            .get(space_id)
-            .cloned()
-            .unwrap_or_default()
+        config.spaces.get(space_id).cloned().unwrap_or_default()
     }
 
     // =========================================================================
@@ -176,10 +172,7 @@ impl ConfigManager {
     /// Enables auto-pilot for a room and saves config.
     pub fn enable_autopilot(&self, space_id: &str, room_id: &str) -> Result<(), ConfigError> {
         self.update(|config| {
-            let space_config = config
-                .spaces
-                .entry(space_id.to_string())
-                .or_default();
+            let space_config = config.spaces.entry(space_id.to_string()).or_default();
             space_config.autopilot.enable_room(room_id);
         })
     }
@@ -321,7 +314,12 @@ mod tests {
 
         // Load from file
         let loaded = ConfigManager::load_from_file(&manager.config_path).unwrap();
-        assert!(loaded.spaces.get("space-1").unwrap().autopilot.is_room_enabled("room-1"));
+        assert!(loaded
+            .spaces
+            .get("space-1")
+            .unwrap()
+            .autopilot
+            .is_room_enabled("room-1"));
     }
 
     #[test]

src-tauri/src/config/types.rs

@@ -94,7 +94,6 @@ pub struct SpaceConfig {
     /// Auto-pilot settings for this space.
     #[serde(default)]
     pub autopilot: SpaceAutopilot,
-
     // Future space-specific settings can be added here:
     // pub notifications: SpaceNotificationSettings,
     // pub default_room: Option<String>,
@@ -253,7 +252,6 @@ impl AutopilotStatus {
             message: Some("Requires AI credits".to_string()),
         }
     }
-
 }
 
 // =============================================================================
@@ -290,15 +288,22 @@ mod tests {
         let mut space_config = SpaceConfig::default();
         space_config.autopilot.enable_room("room-abc-123");
 
-        config.spaces.insert("space-xyz-456".to_string(), space_config);
+        config
+            .spaces
+            .insert("space-xyz-456".to_string(), space_config);
 
         let json = serde_json::to_string_pretty(&config).unwrap();
         assert!(json.contains("space-xyz-456"));
         assert!(json.contains("room-abc-123"));
 
         let parsed: ClaiConfig = serde_json::from_str(&json).unwrap();
         assert!(parsed.spaces.contains_key("space-xyz-456"));
-        assert!(parsed.spaces.get("space-xyz-456").unwrap().autopilot.is_room_enabled("room-abc-123"));
+        assert!(parsed
+            .spaces
+            .get("space-xyz-456")
+            .unwrap()
+            .autopilot
+            .is_room_enabled("room-abc-123"));
     }
 
     #[test]

src-tauri/src/mcp/bridge.rs

@@ -344,7 +344,9 @@ impl JsBridge {
                     Ok(response.result.unwrap_or(serde_json::Value::Null))
                 } else {
                     Err(BridgeError::ToolFailed(
-                        response.error.unwrap_or_else(|| "Unknown error".to_string()),
+                        response
+                            .error
+                            .unwrap_or_else(|| "Unknown error".to_string()),
                     ))
                 }
             }
@@ -418,10 +420,7 @@ mod tests {
 
     #[test]
     fn test_bridge_error_display() {
-        assert_eq!(
-            BridgeError::Timeout.to_string(),
-            "Tool request timed out"
-        );
+        assert_eq!(BridgeError::Timeout.to_string(), "Tool request timed out");
         assert_eq!(
             BridgeError::ChannelClosed.to_string(),
             "Response channel closed"

src-tauri/src/mcp/mod.rs

@@ -44,4 +44,3 @@ pub mod tools;
 
 // Re-export types used by commands/bridge.rs
 pub use bridge::{complete_pending_request, ToolResponse};
-