Fixing test errors

Author: matteius

crates/cli/src/commands/compare.rs

@@ -13,11 +13,10 @@ use smart_diff_engine::{
     SimilarityScorer, ChangeClassifier, CrossFileTracker
 };
 use std::collections::HashMap;
-use std::fs;
 use std::path::{Path, PathBuf};
 use std::time::{Instant, Duration};
 use tokio::fs as async_fs;
-use tracing::{info, warn, error, debug};
+use tracing::{info, warn, debug};
 
 pub async fn run(cli: Cli) -> Result<()> {
     if let Commands::Compare {
@@ -403,10 +402,10 @@ async fn process_file_pair(
         .with_context(|| format!("Failed to analyze target file: {}", target_file.display()))?;
 
     // Initialize diff engine components
-    let mut diff_engine = DiffEngine::new();
+    let diff_engine = DiffEngine::new();
 
     // Configure similarity scorer
-    let mut similarity_scorer = SimilarityScorer::new(detected_language, smart_diff_engine::SimilarityScoringConfig::default());
+    let similarity_scorer = SimilarityScorer::new(detected_language, smart_diff_engine::SimilarityScoringConfig::default());
     if ignore_whitespace {
         // Configure to ignore whitespace - would need to add this to SimilarityScorer
         debug!("Ignoring whitespace in similarity calculation");
@@ -546,7 +545,7 @@ fn calculate_function_similarities(
     target_symbols: &SymbolTable,
     similarity_scorer: &SimilarityScorer,
 ) -> Result<HashMap<String, f64>> {
-    let mut similarities: HashMap<String, f64> = HashMap::new();
+    let similarities: HashMap<String, f64> = HashMap::new();
 
     // Would need to iterate over functions from symbol table - simplified for now
     let similarities = HashMap::new();
@@ -704,7 +703,7 @@ fn format_duration(duration: Duration) -> String {
 
 /// Extract functions from AST for comparison
 fn extract_functions_from_ast(ast: &smart_diff_parser::ASTNode) -> Vec<smart_diff_parser::Function> {
-    use smart_diff_parser::{Function, FunctionSignature, Parameter, Type, NodeType};
+    use smart_diff_parser::{Function, FunctionSignature, NodeType};
 
     let mut functions = Vec::new();
 

crates/cli/src/commands/config.rs

@@ -1,15 +1,11 @@
 //! Config command implementation
 
 use crate::cli::{Cli, Commands, ConfigAction};
-use anyhow::{Result, Context, bail};
+use anyhow::{Result, bail};
 use colored::*;
 use console::Term;
 use serde::{Deserialize, Serialize};
-use smart_diff_engine::RefactoringDetectionConfig;
-use std::collections::HashMap;
-use std::fs;
-use std::path::{Path, PathBuf};
-use tracing::{info, warn};
+use std::path::PathBuf;
 
 /// Application configuration structure
 #[derive(Debug, Clone, Serialize, Deserialize)]

crates/cli/src/commands/doctor.rs

@@ -1,14 +1,12 @@
 //! Doctor command implementation for system validation
 
 use crate::cli::{Cli, Commands};
-use anyhow::{Result, Context};
+use anyhow::Result;
 use colored::*;
 use console::Term;
 use smart_diff_parser::{tree_sitter::TreeSitterParser, Parser, LanguageDetector, Language};
 use smart_diff_semantic::SemanticAnalyzer;
 use smart_diff_engine::{DiffEngine, RefactoringDetector, SimilarityScorer};
-use std::collections::HashMap;
-use tracing::{info, warn, error};
 
 pub async fn run(cli: Cli) -> Result<()> {
     if let Commands::Doctor { component, fix } = cli.command {
@@ -95,7 +93,7 @@ async fn check_parser_system(term: &Term, fix: bool, quiet: bool) -> Result<(usi
     }
 
     let mut issues = 0;
-    let mut fixes = 0;
+    let fixes = 0;
 
     // Test language detector
     let language_detector = LanguageDetector;
@@ -170,7 +168,7 @@ async fn check_semantic_system(term: &Term, fix: bool, quiet: bool) -> Result<(u
     }
 
     let mut issues = 0;
-    let mut fixes = 0;
+    let fixes = 0;
 
     // Test semantic analyzer creation
     let languages = vec![Language::Java, Language::Python, Language::JavaScript, Language::Cpp, Language::C];
@@ -230,7 +228,7 @@ async fn check_diff_engine(term: &Term, fix: bool, quiet: bool) -> Result<(usize
     }
 
     let mut issues = 0;
-    let mut fixes = 0;
+    let fixes = 0;
 
     // Test diff engine creation
     let languages = vec![Language::Java, Language::Python, Language::JavaScript, Language::Cpp, Language::C];
@@ -294,8 +292,8 @@ async fn check_language_support(term: &Term, fix: bool, quiet: bool) -> Result<(
         term.write_line(&format!("{}", "Checking Language Support...".bold()))?;
     }
 
-    let mut issues = 0;
-    let mut fixes = 0;
+    let issues = 0;
+    let fixes = 0;
 
     let supported_languages = vec![
         (Language::Java, vec!["java"]),
@@ -322,7 +320,7 @@ async fn check_configuration(term: &Term, fix: bool, quiet: bool) -> Result<(usi
     }
 
     let mut issues = 0;
-    let mut fixes = 0;
+    let fixes = 0;
 
     // Check if we can create default configurations
     let config_tests = vec![

crates/cli/src/output.rs

@@ -1,9 +1,9 @@
 //! Output formatting utilities
 
 use crate::cli::OutputFormat;
-use anyhow::{Result, Context};
+use anyhow::Result;
 use colored::*;
-use serde::{Deserialize, Serialize};
+use serde::Serialize;
 use smart_diff_parser::{Language, ASTNode};
 use smart_diff_engine::{
     DiffResult, RefactoringPattern, DetailedChangeClassification, FunctionMove