fix(fomatting): missing formatting

Author: Alex Holmberg

src/agent/history.rs

@@ -705,7 +705,10 @@ mod tests {
         // to_messages should still work and include summary
         let messages = history.to_messages();
         if history.summary_frame.is_some() {
-            assert!(!messages.is_empty(), "Should still have summary in messages");
+            assert!(
+                !messages.is_empty(),
+                "Should still have summary in messages"
+            );
         }
     }
 
@@ -736,7 +739,10 @@ mod tests {
                 history_preserve.summary_frame.is_some(),
                 "preserve should keep summary"
             );
-            assert!(history_clear.summary_frame.is_none(), "clear removes summary");
+            assert!(
+                history_clear.summary_frame.is_none(),
+                "clear removes summary"
+            );
         }
 
         // Both should have no turns

src/agent/mod.rs

@@ -317,30 +317,35 @@ pub async fn run_interactive(
                         println!();
 
                         // Try to restore from history_snapshot (new format with full context)
-                        let restored_from_snapshot =
-                            if let Some(history_json) = &record.history_snapshot {
-                                match ConversationHistory::from_json(history_json) {
-                                    Ok(restored) => {
-                                        conversation_history = restored;
-                                        // Rebuild raw_chat_history from restored conversation_history
-                                        raw_chat_history = conversation_history.to_messages();
-                                        println!(
+                        let restored_from_snapshot = if let Some(history_json) =
+                            &record.history_snapshot
+                        {
+                            match ConversationHistory::from_json(history_json) {
+                                Ok(restored) => {
+                                    conversation_history = restored;
+                                    // Rebuild raw_chat_history from restored conversation_history
+                                    raw_chat_history = conversation_history.to_messages();
+                                    println!(
                                             "{}",
                                             "  ✓ Restored full conversation context (including compacted history)".green()
                                         );
-                                        true
-                                    }
-                                    Err(e) => {
-                                        eprintln!(
-                                            "{}",
-                                            format!("  Warning: Failed to restore history snapshot: {}", e).yellow()
-                                        );
-                                        false
-                                    }
+                                    true
                                 }
-                            } else {
-                                false
-                            };
+                                Err(e) => {
+                                    eprintln!(
+                                        "{}",
+                                        format!(
+                                            "  Warning: Failed to restore history snapshot: {}",
+                                            e
+                                        )
+                                        .yellow()
+                                    );
+                                    false
+                                }
+                            }
+                        } else {
+                            false
+                        };
 
                         // Fallback: Load from messages (old format or if snapshot failed)
                         if !restored_from_snapshot {
@@ -359,13 +364,13 @@ pub async fn run_interactive(
                                     persistence::MessageRole::Assistant => {
                                         raw_chat_history
                                             .push(rig::completion::Message::Assistant {
-                                                id: Some(msg.id.clone()),
-                                                content: rig::one_or_many::OneOrMany::one(
-                                                    rig::completion::message::AssistantContent::text(
-                                                        &msg.content,
-                                                    ),
+                                            id: Some(msg.id.clone()),
+                                            content: rig::one_or_many::OneOrMany::one(
+                                                rig::completion::message::AssistantContent::text(
+                                                    &msg.content,
                                                 ),
-                                            });
+                                            ),
+                                        });
                                     }
                                     persistence::MessageRole::System => {}
                                 }

src/agent/session/commands.rs

@@ -9,9 +9,9 @@
 //! - `/resume` - Browse and select a session to resume
 //! - `/sessions` - List available sessions
 
+use super::ChatSession;
 use super::plan_mode::find_incomplete_plans;
 use super::providers::{get_available_models, prompt_api_key};
-use super::ChatSession;
 use crate::agent::{AgentResult, ProviderType};
 use crate::config::{load_agent_config, save_agent_config};
 use colored::Colorize;
@@ -700,7 +700,7 @@ pub fn handle_plans_command(session: &ChatSession) -> AgentResult<()> {
 /// Handle /resume command - browse and select a session to resume
 /// Returns true if a session was loaded and should be displayed
 pub fn handle_resume_command(session: &mut ChatSession) -> AgentResult<bool> {
-    use crate::agent::persistence::{browse_sessions, format_relative_time, SessionSelector};
+    use crate::agent::persistence::{SessionSelector, browse_sessions, format_relative_time};
 
     let selector = SessionSelector::new(&session.project_path);
     let sessions = selector.list_sessions();
@@ -747,7 +747,7 @@ pub fn handle_resume_command(session: &mut ChatSession) -> AgentResult<bool> {
 
 /// Handle /sessions command - list available sessions
 pub fn handle_list_sessions_command(session: &ChatSession) {
-    use crate::agent::persistence::{format_relative_time, SessionSelector};
+    use crate::agent::persistence::{SessionSelector, format_relative_time};
 
     let selector = SessionSelector::new(&session.project_path);
     let sessions = selector.list_sessions();

src/agent/session/mod.rs

@@ -15,7 +15,7 @@ mod providers;
 mod ui;
 
 // Re-exports for backward compatibility
-pub use plan_mode::{find_incomplete_plans, IncompletePlan, PlanMode};
+pub use plan_mode::{IncompletePlan, PlanMode, find_incomplete_plans};
 pub use providers::{get_available_models, get_configured_providers, prompt_api_key};
 
 use crate::agent::commands::TokenUsage;

src/agent/session/providers.rs

@@ -113,8 +113,7 @@ pub fn has_api_key(provider: ProviderType) -> bool {
             ProviderType::Bedrock => {
                 if let Some(bedrock) = &profile.bedrock
                     && (bedrock.profile.is_some()
-                        || (bedrock.access_key_id.is_some()
-                            && bedrock.secret_access_key.is_some()))
+                        || (bedrock.access_key_id.is_some() && bedrock.secret_access_key.is_some()))
                 {
                     return true;
                 }
@@ -148,8 +147,7 @@ pub fn has_api_key(provider: ProviderType) -> bool {
             ProviderType::Bedrock => {
                 if let Some(bedrock) = &profile.bedrock
                     && (bedrock.profile.is_some()
-                        || (bedrock.access_key_id.is_some()
-                            && bedrock.secret_access_key.is_some()))
+                        || (bedrock.access_key_id.is_some() && bedrock.secret_access_key.is_some()))
                 {
                     return true;
                 }
@@ -410,8 +408,7 @@ pub(crate) fn run_bedrock_setup_wizard() -> AgentResult<String> {
         }
         "3" => {
             // Use existing env vars
-            if std::env::var("AWS_ACCESS_KEY_ID").is_err()
-                && std::env::var("AWS_PROFILE").is_err()
+            if std::env::var("AWS_ACCESS_KEY_ID").is_err() && std::env::var("AWS_PROFILE").is_err()
             {
                 println!("{}", "No AWS credentials found in environment!".yellow());
                 println!("Set AWS_ACCESS_KEY_ID + AWS_SECRET_ACCESS_KEY or AWS_PROFILE");

src/agent/session/ui.rs

@@ -2,7 +2,7 @@
 //!
 //! Contains display functions for help, logo, and welcome banner.
 
-use super::{find_incomplete_plans, ChatSession};
+use super::{ChatSession, find_incomplete_plans};
 use crate::agent::commands::SLASH_COMMANDS;
 use crate::agent::ui::ansi;
 use colored::Colorize;

src/agent/tools/analyze.rs

@@ -155,10 +155,7 @@ Returns a compressed summary with key findings. Full analysis is stored and can
                 }
 
                 let json_value = serde_json::to_value(&analysis).map_err(|e| {
-                    AnalyzeError(format!(
-                        "Failed to serialize analysis results: {}",
-                        e
-                    ))
+                    AnalyzeError(format!("Failed to serialize analysis results: {}", e))
                 })?;
 
                 // Use smart compression with RAG retrieval pattern
@@ -186,10 +183,7 @@ Returns a compressed summary with key findings. Full analysis is stored and can
                 } else if error_str.contains("not found") || error_str.contains("No such file") {
                     (
                         ErrorCategory::FileNotFound,
-                        vec![
-                            "Verify the path exists",
-                            "Use list_directory to explore",
-                        ],
+                        vec!["Verify the path exists", "Use list_directory to explore"],
                     )
                 } else {
                     (
@@ -287,6 +281,10 @@ mod tests {
 
         let result = tool.call(args).await.unwrap();
         // Should return error formatted for LLM
-        assert!(result.contains("error") || result.contains("not found") || result.contains("Path not found"));
+        assert!(
+            result.contains("error")
+                || result.contains("not found")
+                || result.contains("Path not found")
+        );
     }
 }

src/agent/tools/dclint.rs

@@ -475,7 +475,10 @@ USAGE:
                 return Ok(format_error_for_llm(
                     "dclint",
                     ErrorCategory::ValidationFailed,
-                    &format!("Invalid Docker Compose YAML syntax: {}", result.parse_errors.join(", ")),
+                    &format!(
+                        "Invalid Docker Compose YAML syntax: {}",
+                        result.parse_errors.join(", ")
+                    ),
                     Some(vec![
                         "Check YAML indentation (use spaces, not tabs)",
                         "Verify key-value pair syntax (key: value)",

src/agent/tools/error.rs

@@ -361,8 +361,10 @@ mod tests {
 
     #[test]
     fn test_tool_error_context() {
-        let result: Result<(), std::io::Error> =
-            Err(std::io::Error::new(std::io::ErrorKind::NotFound, "file missing"));
+        let result: Result<(), std::io::Error> = Err(std::io::Error::new(
+            std::io::ErrorKind::NotFound,
+            "file missing",
+        ));
 
         let with_context = result.with_tool_context("read_file", "reading config");
         assert!(with_context.is_err());