fix: Resolve all clippy warnings in dev-tools crate

- Fixed unused variable warnings by prefixing with underscore
- Removed dead code with allow attribute for test function
- Fixed field assignment patterns using struct literals
- Added allow attributes for recursive functions with self parameter
- Cleaned up let-and-return pattern

All clippy warnings are now resolved while maintaining functionality.

Author: milliondreams

crates/codeprism-dev-tools/src/ast_visualizer.rs

@@ -239,7 +239,7 @@ impl AstVisualizer {
     }
 
     /// Visualize in list format
-    fn visualize_list_format(&self, node: &Node, source: &str) -> Result<String> {
+    fn visualize_list_format(&self, node: &Node, _source: &str) -> Result<String> {
         let mut output = String::new();
         let mut cursor = node.walk();
         let mut depth = 0;
@@ -494,6 +494,7 @@ impl AstVisualizer {
     }
 
     /// Recursively collect AST statistics
+    #[allow(clippy::only_used_in_recursion)]
     fn collect_statistics(&self, node: &Node, stats: &mut AstStatistics, depth: usize) {
         stats.total_nodes += 1;
         stats.max_depth = stats.max_depth.max(depth);
@@ -521,7 +522,7 @@ impl AstVisualizer {
     }
 
     /// Compare two ASTs and highlight differences
-    pub fn compare_asts(&self, old_node: &Node, new_node: &Node, source: &str) -> Result<String> {
+    pub fn compare_asts(&self, old_node: &Node, new_node: &Node, _source: &str) -> Result<String> {
         let mut output = String::new();
         output.push_str("=== AST Comparison ===\n\n");
 
@@ -614,6 +615,7 @@ mod tests {
     use super::*;
     use tree_sitter::Parser;
 
+    #[allow(dead_code)]
     fn create_test_parser() -> Parser {
         // For testing, we'll use a simple language grammar
         // In real usage, this would use the appropriate language
@@ -629,9 +631,11 @@ mod tests {
 
     #[test]
     fn test_custom_config() {
-        let mut config = VisualizationConfig::default();
-        config.max_depth = 10;
-        config.show_positions = false;
+        let config = VisualizationConfig {
+            max_depth: 10,
+            show_positions: false,
+            ..Default::default()
+        };
 
         let visualizer = AstVisualizer::with_config(config);
         assert_eq!(visualizer.config.max_depth, 10);
@@ -648,8 +652,10 @@ mod tests {
 
     #[test]
     fn test_format_node_type_without_colors() {
-        let mut config = VisualizationConfig::default();
-        config.use_colors = false;
+        let config = VisualizationConfig {
+            use_colors: false,
+            ..Default::default()
+        };
         let visualizer = AstVisualizer::with_config(config);
 
         let formatted = visualizer.format_node_type("function_definition");
@@ -658,11 +664,13 @@ mod tests {
 
     #[test]
     fn test_ast_statistics_display() {
-        let mut stats = AstStatistics::default();
-        stats.total_nodes = 100;
-        stats.named_nodes = 80;
-        stats.unnamed_nodes = 20;
-        stats.max_depth = 5;
+        let mut stats = AstStatistics {
+            total_nodes: 100,
+            named_nodes: 80,
+            unnamed_nodes: 20,
+            max_depth: 5,
+            ..Default::default()
+        };
         stats.node_type_counts.insert("function".to_string(), 10);
         stats.node_type_counts.insert("identifier".to_string(), 30);
 

crates/codeprism-dev-tools/src/diff_comparison.rs

@@ -124,7 +124,7 @@ impl AstDiff {
         &self,
         old_result: &ParseResult,
         new_result: &ParseResult,
-        source: &str,
+        _source: &str,
     ) -> Result<DiffReport> {
         let mut differences = Vec::new();
         let mut statistics = DiffStatistics::default();
@@ -392,6 +392,7 @@ impl AstDiff {
     }
 
     /// Calculate tree depth
+    #[allow(clippy::only_used_in_recursion)]
     fn calculate_tree_depth(&self, node: &tree_sitter::Node) -> usize {
         let mut max_depth = 0;
         for i in 0..node.child_count() {
@@ -613,7 +614,7 @@ mod tests {
     use codeprism_core::{NodeKind, Span};
     use std::path::PathBuf;
 
-    fn create_test_node(id: u64, name: &str, kind: NodeKind) -> codeprism_core::Node {
+    fn create_test_node(_id: u64, name: &str, kind: NodeKind) -> codeprism_core::Node {
         let path = PathBuf::from("test.rs");
         let span = Span::new(0, 10, 1, 1, 1, 10);
         let repo_id = "test_repo";
@@ -648,8 +649,10 @@ mod tests {
 
     #[test]
     fn test_node_key_ignoring_ids() {
-        let mut config = DiffConfig::default();
-        config.ignore_node_ids = true;
+        let config = DiffConfig {
+            ignore_node_ids: true,
+            ..Default::default()
+        };
         let diff = AstDiff::with_config(config);
 
         let node1 = create_test_node(1, "test", NodeKind::Function);

crates/codeprism-dev-tools/src/graphviz_export.rs

@@ -538,7 +538,7 @@ mod tests {
     use codeprism_core::{EdgeKind, Language, NodeKind, Span};
     use std::path::PathBuf;
 
-    fn create_test_node(id: u64, name: &str, kind: NodeKind) -> Node {
+    fn create_test_node(_id: u64, name: &str, kind: NodeKind) -> Node {
         let path = PathBuf::from("test.rs");
         let span = Span::new(0, 10, 1, 1, 1, 10);
         let repo_id = "test_repo";

crates/codeprism-dev-tools/src/parser_validator.rs

@@ -121,16 +121,17 @@ impl ParserValidator {
     pub fn validate_complete(
         &self,
         parse_result: &ParseResult,
-        source: &str,
+        _source: &str,
     ) -> Result<ValidationReport> {
         let start_time = std::time::Instant::now();
 
         let mut errors = Vec::new();
         let warnings = Vec::new();
-        let mut statistics = ValidationStatistics::default();
-
-        statistics.total_nodes = parse_result.nodes.len();
-        statistics.total_edges = parse_result.edges.len();
+        let mut statistics = ValidationStatistics {
+            total_nodes: parse_result.nodes.len(),
+            total_edges: parse_result.edges.len(),
+            ..Default::default()
+        };
 
         // Basic validation checks (simplified for initial implementation)
         if self.config.check_span_overlaps {
@@ -158,7 +159,7 @@ impl ParserValidator {
         errors: &mut Vec<ValidationError>,
     ) -> Result<()> {
         for (i, node1) in nodes.iter().enumerate() {
-            for (j, node2) in nodes.iter().enumerate().skip(i + 1) {
+            for (_j, node2) in nodes.iter().enumerate().skip(i + 1) {
                 let start1 = node1.span.start_byte;
                 let end1 = node1.span.end_byte;
                 let start2 = node2.span.start_byte;

crates/codeprism-dev-tools/src/performance_profiler.rs

@@ -339,18 +339,12 @@ impl PerformanceProfiler {
 
     /// Analyze performance for bottlenecks and trends
     fn analyze_performance(&self) -> PerformanceAnalysis {
-        let mut analysis = PerformanceAnalysis::default();
-
-        // Identify bottlenecks
-        analysis.bottlenecks = self.identify_bottlenecks();
-
-        // Analyze trends
-        analysis.trends = self.analyze_trends();
-
-        // Calculate efficiency score
-        analysis.efficiency_score = self.calculate_efficiency_score();
-
-        analysis
+        PerformanceAnalysis {
+            bottlenecks: self.identify_bottlenecks(),
+            trends: self.analyze_trends(),
+            efficiency_score: self.calculate_efficiency_score(),
+            ..Default::default()
+        }
     }
 
     /// Identify performance bottlenecks