adding csv support

Cargo.toml

@@ -25,8 +25,10 @@ path = "src/lib.rs"
 
 [dependencies]
 anyhow = { version = "1.0.75", features = [] } # for error handling
+chrono = "0.4.38"
 clap = { version = "4.2.1", features = ["derive"] } # cli
 clap_derive = "4.2.0" # cli
+csv = "1.3.0"
 itertools = "0.13.0" # tools for iterating over iterable things
 jwalk = "0.8.1" # for walking the file tree
 path-clean = "1.0.1" # Pathname#cleaname in Ruby

src/packs.rs

@@ -10,6 +10,7 @@ pub(crate) mod checker_configuration;
 pub(crate) mod configuration;
 pub(crate) mod constant_resolver;
 pub(crate) mod creator;
+pub(crate) mod csv;
 pub(crate) mod dependencies;
 pub(crate) mod ignored;
 pub(crate) mod monkey_patch_detection;
@@ -38,6 +39,7 @@ pub(crate) use self::parsing::ruby::zeitwerk::get_zeitwerk_constant_resolver;
 pub(crate) use self::parsing::ParsedDefinition;
 pub(crate) use self::parsing::UnresolvedReference;
 use anyhow::bail;
+use cli::OutputFormat;
 pub(crate) use configuration::Configuration;
 pub(crate) use package_todo::PackageTodo;
 
@@ -68,14 +70,24 @@ pub fn create(
 
 pub fn check(
     configuration: &Configuration,
+    output_format: OutputFormat,
     files: Vec<String>,
 ) -> anyhow::Result<()> {
     let result = checker::check_all(configuration, files)
         .context("Failed to check files")?;
-    println!("{}", result);
-    if result.has_violations() {
-        bail!("Violations found!")
+
+    match output_format {
+        OutputFormat::Packwerk => {
+            println!("{}", result);
+            if result.has_violations() {
+                bail!("Violations found!")
+            }
+        }
+        OutputFormat::CSV => {
+            csv::write_csv(&result)?;
+        }
     }
+
     Ok(())
 }
 

src/packs/checker.rs

@@ -64,9 +64,9 @@ pub(crate) trait ValidatorInterface {
 
 #[derive(Debug, PartialEq)]
 pub struct CheckAllResult {
-    reportable_violations: HashSet<Violation>,
-    stale_violations: Vec<ViolationIdentifier>,
-    strict_mode_violations: HashSet<Violation>,
+    pub reportable_violations: HashSet<Violation>,
+    pub stale_violations: Vec<ViolationIdentifier>,
+    pub strict_mode_violations: HashSet<Violation>,
 }
 
 impl CheckAllResult {

src/packs/cli.rs

@@ -1,7 +1,7 @@
 use crate::packs;
 
 use crate::packs::file_utils::get_absolute_path;
-use clap::{Parser, Subcommand};
+use clap::{Parser, Subcommand, ValueEnum};
 use clap_derive::Args;
 use std::path::PathBuf;
 use tracing::debug;
@@ -70,6 +70,9 @@ enum Command {
         #[arg(long)]
         ignore_recorded_violations: bool,
 
+        #[arg(short, long, default_value = "packwerk")]
+        output_format: OutputFormat,
+
         files: Vec<String>,
     },
 
@@ -152,6 +155,12 @@ enum Command {
     ListDefinitions(ListDefinitionsArgs),
 }
 
+#[derive(ValueEnum, Copy, Clone, Debug, PartialEq, Eq)]
+pub enum OutputFormat {
+    Packwerk,
+    CSV,
+}
+
 #[derive(Debug, Args)]
 struct ListDefinitionsArgs {
     /// Show constants with multiple definitions only
@@ -242,11 +251,12 @@ pub fn run() -> anyhow::Result<()> {
         Command::ListIncludedFiles => packs::list_included_files(configuration),
         Command::Check {
             ignore_recorded_violations,
+            output_format,
             files,
         } => {
             configuration.ignore_recorded_violations =
                 ignore_recorded_violations;
-            packs::check(&configuration, files)
+            packs::check(&configuration, output_format, files)
         }
         Command::CheckContents {
             ignore_recorded_violations,
@@ -257,7 +267,7 @@ pub fn run() -> anyhow::Result<()> {
 
             let absolute_path = get_absolute_path(file.clone(), &configuration);
             configuration.stdin_file_path = Some(absolute_path);
-            packs::check(&configuration, vec![file])
+            packs::check(&configuration, OutputFormat::Packwerk, vec![file])
         }
         Command::Update => packs::update(&configuration),
         Command::Validate => {

src/packs/csv.rs

@@ -0,0 +1,56 @@
+use std::fs::File;
+
+use chrono::Utc;
+use itertools::chain;
+
+use crate::packs::bin_locater;
+
+use super::checker::CheckAllResult;
+
+pub fn write_csv(result: &CheckAllResult) -> anyhow::Result<()> {
+    if !&result.reportable_violations.is_empty()
+        || !&result.strict_mode_violations.is_empty()
+    {
+        let utc = Utc::now().naive_utc().to_string().replace(' ', "");
+        let reportable_file = format!("{}-violations.csv", utc);
+        let file = File::create(&reportable_file)?;
+        let mut wtr = csv::Writer::from_writer(file);
+
+        let all = chain!(
+            &result.reportable_violations,
+            &result.strict_mode_violations
+        );
+        wtr.write_record([
+            "Violation",
+            "Strict?",
+            "File",
+            "Constant",
+            "Referencing Pack",
+            "Defining Pack",
+            "Message",
+        ])?;
+        for violation in all {
+            let identifier = &violation.identifier;
+            wtr.serialize((
+                &identifier.violation_type,
+                &identifier.strict,
+                &identifier.file,
+                &identifier.constant_name,
+                &identifier.referencing_pack_name,
+                &identifier.defining_pack_name,
+                &violation.message,
+            ))?;
+        }
+        wtr.flush()?;
+
+        println!("Violations written to '{}'", &reportable_file);
+    }
+
+    if !&result.stale_violations.is_empty() {
+        println!(
+            "There were stale violations found, please run `{} update`",
+            bin_locater::packs_bin_name(),
+        );
+    }
+    Ok(())
+}