Fix cargo fmt and clippy formatting issues

Co-authored-by: larp0 <[email protected]>

Author: Copilot

src/services/audit_service.rs

@@ -299,7 +299,10 @@ impl AuditService {
                 self.coordinator
                     .generate_markdown_summary(&report, &markdown_path)
                     .map_err(|e| {
-                        AuditError::OutputError(format!("Failed to generate Markdown summary: {}", e))
+                        AuditError::OutputError(format!(
+                            "Failed to generate Markdown summary: {}",
+                            e
+                        ))
                     })?;
 
                 if request.verbose > 0 {

src/utils/audit.rs

@@ -1276,15 +1276,24 @@ impl AuditCoordinator {
                 ai_successes,
                 ai_failures
             );
-            
+
             // Provide user-facing feedback about AI analysis status
             if ai_failures > ai_successes {
-                println!("⚠️  AI analysis partially unavailable ({} failures, {} successes)", ai_failures, ai_successes);
+                println!(
+                    "⚠️  AI analysis partially unavailable ({} failures, {} successes)",
+                    ai_failures, ai_successes
+                );
                 println!("   Manual review is recommended for enhanced security insights");
             } else if ai_failures > 0 {
-                println!("ℹ️  AI analysis completed with some failures ({} failures, {} successes)", ai_failures, ai_successes);
+                println!(
+                    "ℹ️  AI analysis completed with some failures ({} failures, {} successes)",
+                    ai_failures, ai_successes
+                );
             } else {
-                println!("✅ AI analysis completed successfully for {} findings", ai_successes);
+                println!(
+                    "✅ AI analysis completed successfully for {} findings",
+                    ai_successes
+                );
             }
         }
 
@@ -5787,7 +5796,11 @@ impl AuditCoordinator {
     }
 
     /// Generate Markdown summary using template system
-    pub fn generate_markdown_summary(&self, report: &AuditReport, output_path: &Path) -> Result<()> {
+    pub fn generate_markdown_summary(
+        &self,
+        report: &AuditReport,
+        output_path: &Path,
+    ) -> Result<()> {
         self.template_generator
             .generate_markdown_summary(report, output_path)
     }

src/utils/audit_modular.rs

@@ -165,7 +165,9 @@ impl FindingIdAllocator {
             .as_nanos()
             .hash(&mut hasher);
         std::thread::current().id().hash(&mut hasher);
-        FINDING_ID_COUNTER.fetch_add(1, Ordering::SeqCst).hash(&mut hasher);
+        FINDING_ID_COUNTER
+            .fetch_add(1, Ordering::SeqCst)
+            .hash(&mut hasher);
         category.hash(&mut hasher);
 
         let hash = hasher.finish();
@@ -191,7 +193,9 @@ impl FindingIdAllocator {
             .as_nanos()
             .hash(&mut hasher);
         std::thread::current().id().hash(&mut hasher);
-        FINDING_ID_COUNTER.fetch_add(1, Ordering::SeqCst).hash(&mut hasher);
+        FINDING_ID_COUNTER
+            .fetch_add(1, Ordering::SeqCst)
+            .hash(&mut hasher);
 
         let hash = hasher.finish();
         format!("OSVM-{:016x}", hash)
@@ -572,10 +576,21 @@ impl SolanaSecurityCheck {
     fn is_likely_solana_key(&self, value: &str, context: &str) -> bool {
         // Skip common false positives
         let false_positive_indicators = [
-            "test", "mock", "example", "dummy", "placeholder", "lorem", "ipsum",
-            "comment", "doc", "readme", "license", "copyright", "author"
+            "test",
+            "mock",
+            "example",
+            "dummy",
+            "placeholder",
+            "lorem",
+            "ipsum",
+            "comment",
+            "doc",
+            "readme",
+            "license",
+            "copyright",
+            "author",
         ];
-        
+
         let context_lower = context.to_lowercase();
         for indicator in &false_positive_indicators {
             if context_lower.contains(indicator) {
@@ -585,11 +600,26 @@ impl SolanaSecurityCheck {
 
         // Look for Solana-specific context clues
         let solana_indicators = [
-            "pubkey", "program_id", "account", "signer", "authority", "mint", 
-            "token", "pda", "system_program", "spl_token", "metaplex", "anchor",
-            "lamports", "rent", "solana", "devnet", "mainnet", "testnet"
+            "pubkey",
+            "program_id",
+            "account",
+            "signer",
+            "authority",
+            "mint",
+            "token",
+            "pda",
+            "system_program",
+            "spl_token",
+            "metaplex",
+            "anchor",
+            "lamports",
+            "rent",
+            "solana",
+            "devnet",
+            "mainnet",
+            "testnet",
         ];
-        
+
         for indicator in &solana_indicators {
             if context_lower.contains(indicator) {
                 return true;
@@ -613,8 +643,9 @@ impl SolanaSecurityCheck {
             // Check for potential base58 encoded Solana public keys
             if Self::is_valid_base58_pubkey(&string_lit.value) {
                 // Reduce false positives by checking context
-                let is_likely_key = self.is_likely_solana_key(&string_lit.value, &string_lit.context);
-                
+                let is_likely_key =
+                    self.is_likely_solana_key(&string_lit.value, &string_lit.context);
+
                 if is_likely_key {
                     findings.push(AuditFinding {
                         id: FindingIdAllocator::next_category_id("solana"),
@@ -1168,7 +1199,7 @@ mod tests {
         assert!(id1.starts_with("OSVM-") && id1.len() == 21); // OSVM- + 16 hex chars
         assert!(id2.starts_with("OSVM-") && id2.len() == 21);
         assert!(id3.starts_with("OSVM-SOL-") && id3.len() == 25); // OSVM-SOL- + 16 hex chars
-        
+
         // IDs should be different
         assert_ne!(id1, id2);
         assert_ne!(id1, id3);

src/utils/audit_templates.rs

@@ -232,7 +232,7 @@ impl EnhancedAIErrorHandler {
     pub fn new() -> Self {
         Self {
             last_error_log: std::sync::Arc::new(std::sync::Mutex::new(
-                std::time::Instant::now() - std::time::Duration::from_secs(60)
+                std::time::Instant::now() - std::time::Duration::from_secs(60),
             )),
             error_log_threshold: std::time::Duration::from_secs(30), // Log at most once every 30 seconds
         }
@@ -254,12 +254,16 @@ impl EnhancedAIErrorHandler {
     }
 
     /// Handle AI analysis error with comprehensive but rate-limited logging
-    pub fn handle_ai_error_with_context(&self, error: &anyhow::Error, context: &str) -> Option<String> {
+    pub fn handle_ai_error_with_context(
+        &self,
+        error: &anyhow::Error,
+        context: &str,
+    ) -> Option<String> {
         // Always log the first occurrence or after the threshold period
         if self.should_log_error() {
             log::error!("AI Analysis Error in {}: {}", context, error);
             log::warn!("AI error logging rate-limited to prevent log flooding");
-            
+
             // Log the full error chain for detailed diagnostics
             let mut current_error = error.source();
             let mut error_level = 1;