Check pt fixing bugs

Author: matteius

crates/cli/src/commands/analyze.rs

@@ -344,7 +344,10 @@ fn display_analysis_summary(
 
     let total_files = results.len();
     let total_lines: usize = results.iter().map(|r| r.line_count).sum();
-    let total_functions: usize = results.iter().map(|_r| 0).sum(); // Would need to count functions from symbol table
+    let total_functions: usize = results.iter().map(|r| {
+        let stats = r.symbols.get_statistics();
+        stats.function_count + stats.method_count
+    }).sum();
 
     term.write_line(&format!("Files analyzed: {}", total_files.to_string().bold()))?;
     term.write_line(&format!("Total lines: {}", total_lines.to_string().bold()))?;

crates/cli/src/output.rs

@@ -198,7 +198,8 @@ impl OutputFormatter {
             html.push_str(&format!("        <h2>{}</h2>\n", html_escape(&result.file_path.to_string_lossy())));
             html.push_str(&format!("        <p><strong>Language:</strong> {:?}</p>\n", result.language));
             html.push_str(&format!("        <p><strong>Lines:</strong> {}</p>\n", result.line_count));
-            html.push_str(&format!("        <p><strong>Functions:</strong> {}</p>\n", 0)); // Would need to count functions from symbol table
+            let stats = result.symbols.get_statistics();
+            html.push_str(&format!("        <p><strong>Functions:</strong> {}</p>\n", stats.function_count + stats.method_count));
             html.push_str("    </div>\n");
         }
 
@@ -218,7 +219,8 @@ impl OutputFormatter {
             xml.push_str(&format!("    <path>{}</path>\n", xml_escape(&result.file_path.to_string_lossy())));
             xml.push_str(&format!("    <language>{:?}</language>\n", result.language));
             xml.push_str(&format!("    <lines>{}</lines>\n", result.line_count));
-            xml.push_str(&format!("    <functions>{}</functions>\n", 0)); // Would need to count functions from symbol table
+            let stats = result.symbols.get_statistics();
+            xml.push_str(&format!("    <functions>{}</functions>\n", stats.function_count + stats.method_count));
             xml.push_str("  </file>\n");
         }
 
@@ -256,8 +258,9 @@ impl OutputFormatter {
             md.push_str(&format!("## File {}: {}\n\n", index + 1, result.file_path.display()));
             md.push_str(&format!("- **Language**: {:?}\n", result.language));
             md.push_str(&format!("- **Lines of Code**: {}\n", result.line_count));
-            md.push_str(&format!("- **Functions**: {}\n", 0)); // Would need to count functions from symbol table
-            md.push_str(&format!("- **Variables**: {}\n", 0)); // Would need to count variables from symbol table
+            let stats = result.symbols.get_statistics();
+            md.push_str(&format!("- **Functions**: {}\n", stats.function_count + stats.method_count));
+            md.push_str(&format!("- **Variables**: {}\n", stats.variable_count));
             md.push_str(&format!("- **Processing Time**: {}\n\n", Self::format_duration(result.processing_time)));
         }
 

crates/parser/src/ast_builder.rs

@@ -172,10 +172,9 @@ impl ASTBuilder {
             original_text: if text.len() <= self.config.max_text_length {
                 text.to_string()
             } else {
-                format!(
-                    "{}...",
-                    &text[..self.config.max_text_length.min(text.len())]
-                )
+                // Use Unicode-aware truncation to avoid panics on multi-byte characters
+                let truncated = self.truncate_text_safely(text, self.config.max_text_length);
+                format!("{}...", truncated)
             },
             attributes,
         }
@@ -479,6 +478,21 @@ impl ASTBuilder {
         }
         None
     }
+
+    /// Safely truncate text at Unicode character boundaries
+    fn truncate_text_safely<'a>(&self, text: &'a str, max_bytes: usize) -> &'a str {
+        if text.len() <= max_bytes {
+            return text;
+        }
+
+        // Find the largest valid UTF-8 boundary within max_bytes
+        let mut boundary = max_bytes;
+        while boundary > 0 && !text.is_char_boundary(boundary) {
+            boundary -= 1;
+        }
+
+        &text[..boundary]
+    }
 }
 
 /// Builder pattern for AST construction with fluent interface