WIP ignore the clippy-annotation-reporter crate

Author: ekump

.github/actions/clippy-annotation-reporter/Cargo.toml

@@ -9,7 +9,6 @@ clap = { version = "4.3", features = ["derive", "env"] }
 octocrab = "0.44"
 anyhow = "1.0"
 regex = "1.9"
-# TODO: EK - fix features
 tokio = { version = "1", features = ["macros", "rt-multi-thread"] }
 serde_json = "1.0"
 log = "0.4"

.github/actions/clippy-annotation-reporter/src/report_generator.rs

@@ -18,21 +18,16 @@ pub fn generate_report(
     base_branch: &str,
     head_branch: &str,
 ) -> String {
+    // TODO: This is hardcoded for now, as the presence of the clippy-annotation-reporter crate in
+    // the repo is temporary. A future feature can allow users to specify crates to ignore.
+    const IGNORED_CRATE: &str = "clippy-annotation-reporter";
+
     let mut report = String::new();
 
-    // Add header and branch information
     add_header(&mut report, repository, base_branch, head_branch);
-
-    // Add rule summary section
     add_rule_summary(&mut report, analysis, rules);
-
-    // Add file-level section
-    add_file_level_section(&mut report, analysis);
-
-    // Add crate-level section
-    add_crate_level_section(&mut report, analysis);
-
-    // Add explanation
+    add_file_level_section(&mut report, analysis, IGNORED_CRATE);
+    add_crate_level_section(&mut report, analysis, IGNORED_CRATE);
     add_explanation(&mut report);
 
     report
@@ -79,7 +74,7 @@ fn add_rule_summary(report: &mut String, analysis: &AnalysisResult, rules: &[Str
 }
 
 /// Add section showing annotation counts by file
-fn add_file_level_section(report: &mut String, analysis: &AnalysisResult) {
+fn add_file_level_section(report: &mut String, analysis: &AnalysisResult, ignored_crate: &str) {
     if analysis.changed_files.is_empty() {
         return;
     }
@@ -98,8 +93,15 @@ fn add_file_level_section(report: &mut String, analysis: &AnalysisResult) {
 
     // Add row for each file
     for file in all_files {
-        let base_count = *base_file_counts.get(&file).unwrap_or(&0);
-        let head_count = *head_file_counts.get(&file).unwrap_or(&0);
+        // Skip files that belong to the ignored crate
+        if !ignored_crate.is_empty()
+            && file.contains(&format!(".github/actions/{}/", ignored_crate))
+        {
+            continue;
+        }
+
+        let base_count = *base_file_counts.get(&Rc::new(file.clone())).unwrap_or(&0);
+        let head_count = *head_file_counts.get(&Rc::new(file.clone())).unwrap_or(&0);
 
         // Skip files with no annotations in either branch
         if base_count == 0 && head_count == 0 {
@@ -113,7 +115,7 @@ fn add_file_level_section(report: &mut String, analysis: &AnalysisResult) {
 }
 
 /// Add section showing annotation stats by crate
-fn add_crate_level_section(report: &mut String, analysis: &AnalysisResult) {
+fn add_crate_level_section(report: &mut String, analysis: &AnalysisResult, ignored_crate: &str) {
     report.push_str("### Annotation Stats by Crate\n\n");
     report.push_str("| Crate | Base Branch | PR Branch | Change |\n");
     report.push_str("|-------|------------|-----------|--------|\n");
@@ -126,6 +128,11 @@ fn add_crate_level_section(report: &mut String, analysis: &AnalysisResult) {
 
     // Add row for each crate
     for crate_name in all_crates {
+        // Skip the ignored crate
+        if !ignored_crate.is_empty() && *crate_name == ignored_crate {
+            continue;
+        }
+
         let base_count = *analysis.base_crate_counts.get(&crate_name).unwrap_or(&0);
         let head_count = *analysis.head_crate_counts.get(&crate_name).unwrap_or(&0);