Fixing test errors

Author: matteius

crates/cli/src/commands/compare.rs

@@ -717,7 +717,7 @@ fn extract_functions_from_ast(ast: &smart_diff_parser::ASTNode) -> Vec<smart_dif
         let signature = FunctionSignature {
             name: name.clone(),
             parameters: Vec::new(), // Simplified for now
-            return_type: smart_diff_parser::Type::Primitive("void".to_string()),
+            return_type: smart_diff_parser::Type::new("void".to_string()),
             modifiers: Vec::new(),
             generic_parameters: Vec::new(),
         };
@@ -733,7 +733,7 @@ fn extract_functions_from_ast(ast: &smart_diff_parser::ASTNode) -> Vec<smart_dif
                 end_column: node.metadata.column,
             },
             dependencies: Vec::new(),
-            hash: 0, // Simplified for now
+            hash: "0".to_string(), // Simplified for now
         };
 
         functions.push(function);

crates/cli/src/commands/doctor.rs

@@ -110,7 +110,7 @@ async fn check_parser_system(term: &Term, fix: bool, quiet: bool) -> Result<(usi
     ];
 
     for (filename, expected) in test_cases {
-        let detected = language_detector.detect_from_path(std::path::Path::new(filename));
+        let detected = LanguageDetector::detect_from_path(std::path::Path::new(filename));
         if detected != expected {
             issues += 1;
             if !quiet {
@@ -235,7 +235,7 @@ async fn check_diff_engine(term: &Term, fix: bool, quiet: bool) -> Result<(usize
     // Test diff engine creation
     let languages = vec![Language::Java, Language::Python, Language::JavaScript, Language::Cpp, Language::C];
 
-    for lang in languages {
+    for lang in &languages {
         match std::panic::catch_unwind(|| DiffEngine::new()) {
             Ok(_engine) => {
                 if !quiet {

crates/cli/src/output.rs

@@ -341,6 +341,8 @@ impl OutputFormatter {
                             smart_diff_parser::ChangeType::Rename => change_desc.blue(),
                             smart_diff_parser::ChangeType::Move => change_desc.magenta(),
                             smart_diff_parser::ChangeType::CrossFileMove => change_desc.cyan(),
+                            smart_diff_parser::ChangeType::Split => change_desc.purple(),
+                            smart_diff_parser::ChangeType::Merge => change_desc.purple(),
                         };
                         output.push_str(&format!("{}\n", colored_desc));
                     }
@@ -515,6 +517,8 @@ impl OutputFormatter {
                         smart_diff_parser::ChangeType::Rename => "rename",
                         smart_diff_parser::ChangeType::Move => "move",
                         smart_diff_parser::ChangeType::CrossFileMove => "cross-file-move",
+                        smart_diff_parser::ChangeType::Split => "split",
+                        smart_diff_parser::ChangeType::Merge => "merge",
                     };
 
                     html.push_str(&format!("                    <li class=\"change-item {}\">\n", class_name));