From 724e616e4d7f95e62878baf28ec2824a35543696 Mon Sep 17 00:00:00 2001 From: Alexandre Franke Date: Wed, 7 Feb 2024 21:39:07 +0100 Subject: [PATCH] feat: add Code Climate output format --- crates/typos-cli/src/bin/typos-cli/args.rs | 2 + crates/typos-cli/src/bin/typos-cli/report.rs | 92 ++++++++++++++++++++ 2 files changed, 94 insertions(+) diff --git a/crates/typos-cli/src/bin/typos-cli/args.rs b/crates/typos-cli/src/bin/typos-cli/args.rs index 2b93086c5..0490e50ed 100644 --- a/crates/typos-cli/src/bin/typos-cli/args.rs +++ b/crates/typos-cli/src/bin/typos-cli/args.rs @@ -10,6 +10,7 @@ pub enum Format { #[default] Long, Json, + CodeClimate, } impl Format { @@ -19,6 +20,7 @@ impl Format { Format::Brief => Box::new(crate::report::PrintBrief), Format::Long => Box::new(crate::report::PrintLong), Format::Json => Box::new(crate::report::PrintJson), + Format::CodeClimate => Box::new(crate::report::PrintCodeClimate), } } } diff --git a/crates/typos-cli/src/bin/typos-cli/report.rs b/crates/typos-cli/src/bin/typos-cli/report.rs index 31eef2220..9704413ce 100644 --- a/crates/typos-cli/src/bin/typos-cli/report.rs +++ b/crates/typos-cli/src/bin/typos-cli/report.rs @@ -280,6 +280,98 @@ impl Report for PrintJson { } } +pub struct PrintCodeClimate; + +impl Report for PrintCodeClimate { + fn report(&self, msg: Message) -> Result<(), std::io::Error> { + let mut climate: CodeClimate = Default::default(); + climate.r#type = "issue"; + climate.check_name = "typos"; + match msg { + Message::BinaryFile(_msg) => { + climate.severity = "info"; + } + Message::Typo(msg) => { + match msg.corrections { + typos::Status::Valid => {} + typos::Status::Invalid => { + climate.description = format!("`{}` is disallowed", msg.typo,); + } + typos::Status::Corrections(corrections) => { + climate.description = format!( + "`{}` should be {}", + msg.typo, + itertools::join(corrections.iter().map(|s| format!("`{}`", s)), ", ") + ); + } + } + climate.categories = "Clarity"; + climate.severity = "minor"; + climate.location = Location::from(msg.context.unwrap()); + } + Message::Error(msg) => { + climate.severity = "error"; + climate.description = context_display(&msg.context).to_string(); + } + _ => { + climate.severity = "minor"; + } + } + writeln!( + stdout().lock(), + "{}", + serde_json::to_string(&climate).unwrap() + )?; + Ok(()) + } +} + +#[derive(Default, serde::Serialize)] +struct CodeClimate<'l> { + check_name: &'l str, + description: String, + categories: &'l str, + location: Location<'l>, + severity: &'l str, + r#type: &'l str, +} + +#[derive(serde::Serialize)] +struct Location<'l> { + path: &'l std::path::Path, + lines: Lines, +} + +impl<'l> Default for Location<'l> { + fn default() -> Self { + Self { + path: std::path::Path::new("-"), + lines: Lines { begin: 0 }, + } + } +} + +impl<'l> From> for Location<'l> { + fn from(context: Context<'l>) -> Self { + match context { + Context::File(c) => Self { + path: c.path, + lines: Lines { begin: c.line_num }, + }, + Context::Path(c) => Self { + path: c.path, + ..Default::default() + }, + _ => Default::default(), + } + } +} + +#[derive(Default, serde::Serialize)] +struct Lines { + begin: usize, +} + #[cfg(test)] mod tests { use super::*;