fix: Resolve CI formatting and clippy errors

- Fix formatting for if-let chain patterns in downloader.rs and indexer.rs
- Remove unused imports and functions in integration_tests.rs
- Add #[allow(dead_code)] to TaskResult enum
- Use inlined format args in println! statements

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: snowmead

rust-docs-mcp/src/cache/downloader.rs

@@ -216,10 +216,11 @@ impl CrateDownloader {
 
             // Report progress if callback provided and total size known
             if let Some(ref callback) = progress_callback
-                && total_bytes > 0 {
-                    let percent = ((downloaded_bytes * 100) / total_bytes).min(100) as u8;
-                    callback(percent);
-                }
+                && total_bytes > 0
+            {
+                let percent = ((downloaded_bytes * 100) / total_bytes).min(100) as u8;
+                callback(percent);
+            }
         }
 
         // Extract the crate

rust-docs-mcp/src/search/indexer.rs

@@ -177,10 +177,11 @@ impl SearchIndexer {
 
             // Report progress every 50 items during document creation (0-70%)
             if let Some(ref callback) = progress_callback
-                && (i % 50 == 0 || i == total_items - 1) {
-                    let percent = ((i * 70) / total_items.max(1)).min(70) as u8;
-                    callback(percent);
-                }
+                && (i % 50 == 0 || i == total_items - 1)
+            {
+                let percent = ((i * 70) / total_items.max(1)).min(70) as u8;
+                callback(percent);
+            }
         }
 
         // Then add all documents to the writer
@@ -190,10 +191,11 @@ impl SearchIndexer {
 
             // Report progress during writing (70-95%)
             if let Some(ref callback) = progress_callback
-                && (i % 50 == 0 || i == documents.len() - 1) {
-                    let percent = (70 + ((i * 25) / documents.len().max(1))).min(95) as u8;
-                    callback(percent);
-                }
+                && (i % 50 == 0 || i == documents.len() - 1)
+            {
+                let percent = (70 + ((i * 25) / documents.len().max(1))).min(95) as u8;
+                callback(percent);
+            }
         }
 
         writer.commit()?;

rust-docs-mcp/tests/integration_tests.rs

@@ -11,7 +11,7 @@ use rust_docs_mcp::RustDocsService;
 use rust_docs_mcp::analysis::outputs::StructureOutput;
 use rust_docs_mcp::analysis::tools::AnalyzeCrateStructureParams;
 use rust_docs_mcp::cache::outputs::{
-    CacheCrateOutput, CacheTaskStartedOutput, GetCratesMetadataOutput, ListCrateVersionsOutput,
+    CacheTaskStartedOutput, GetCratesMetadataOutput, ListCrateVersionsOutput,
 };
 use rust_docs_mcp::cache::tools::{
     CacheCrateParams, CacheOperationsParams, CrateMetadataQuery, GetCratesMetadataParams,
@@ -29,7 +29,6 @@ use rust_docs_mcp::docs::tools::{
 };
 use rust_docs_mcp::search::outputs::SearchItemsFuzzyOutput;
 use rust_docs_mcp::search::tools::SearchItemsFuzzyParams;
-use std::collections::HashMap;
 use std::time::Duration;
 use tempfile::TempDir;
 
@@ -49,20 +48,6 @@ fn parse_cache_task_started(response: &str) -> Result<CacheTaskStartedOutput> {
     })
 }
 
-fn parse_cache_response(response: &str) -> Result<CacheCrateOutput> {
-    serde_json::from_str(response)
-        .map_err(|e| anyhow::anyhow!("Failed to parse cache response: {e}\nResponse: {response}"))
-}
-
-fn is_binary_only_response(response: &str) -> bool {
-    // Binary-only packages will return an error with this message
-    if let Ok(output) = parse_cache_response(response) {
-        matches!(output, CacheCrateOutput::Error { error } if error.contains("binary-only") || error.contains("no library"))
-    } else {
-        false
-    }
-}
-
 /// Helper to create a test service with temporary cache
 fn create_test_service() -> Result<(RustDocsService, TempDir)> {
     let temp_dir = TempDir::new()?;
@@ -72,6 +57,7 @@ fn create_test_service() -> Result<(RustDocsService, TempDir)> {
 
 /// Result of waiting for a caching task
 #[derive(Debug)]
+#[allow(dead_code)]
 enum TaskResult {
     Success,
     WorkspaceDetected(String), // Contains workspace info
@@ -1518,7 +1504,7 @@ async fn test_step_tracking() -> Result<()> {
                 .map(|(c, t, _)| *c == current && *t == total)
                 .unwrap_or(false);
             if !last_matches {
-                println!("Step update: {} of {} {:?}", current, total, desc);
+                println!("Step update: {current} of {total} {desc:?}");
                 step_updates.push((current, total, desc));
             }
         }
@@ -1535,7 +1521,7 @@ async fn test_step_tracking() -> Result<()> {
         tokio::time::sleep(poll_interval).await;
     }
 
-    println!("Step updates observed: {:?}", step_updates);
+    println!("Step updates observed: {step_updates:?}");
 
     // Verify step tracking requirements: