Remove unnecessary Boxed values

src/cli/apply.rs

@@ -58,9 +58,9 @@ pub fn apply(args: ApplyArgs) -> CliResult {
 
     // Bind for later use.
     let confirmed_count = result.confirmed.len();
-    let mut out: Box<dyn Write> = match &args.output {
-        Some(path) => Box::new(File::create(path)?),
-        None => Box::new(stdout()),
+    let mut out: &mut dyn Write = match &args.output {
+        Some(path) => &mut File::create(path)?,
+        None => &mut stdout(),
     };
     let msg_f = || {
         let out_description = args

src/cli/cleanup_allowlist.rs

@@ -35,9 +35,9 @@ pub fn cleanup_allowlist(args: CleanupAllowlistArgs) -> CliResult {
         .for_each(|rule| {
             rule.remove("allowlist");
         });
-    let mut out: Box<dyn Write> = match args.output {
-        Some(path) => Box::new(File::create(path)?),
-        None => Box::new(stdout()),
+    let mut out: &mut dyn Write = match args.output {
+        Some(path) => &mut File::create(path)?,
+        None => &mut stdout(),
     };
     write!(&mut out, "{doc}")?;
     SUCCESS

src/cli/cleanup_rule.rs

@@ -73,9 +73,9 @@ pub fn cleanup_rule(args: CleanupRuleArgs) -> CliResult {
         !contain
     });
 
-    let mut out: Box<dyn Write> = match args.output {
-        Some(path) => Box::new(File::create(path)?),
-        None => Box::new(stdout()),
+    let mut out: &mut dyn Write = match args.output {
+        Some(path) => &mut File::create(path)?,
+        None => &mut stdout(),
     };
     write!(&mut out, "{doc}")?;
     SUCCESS

src/cli/diff.rs

@@ -62,9 +62,9 @@ pub fn diff(args: DiffArgs) -> CliResult {
         .with_context(|| format!("Failed to parse JSON: {}", after_path.display()))?;
 
     let diffs = compute_diff(befores, afters);
-    let mut out: Box<dyn Write> = match args.output.clone() {
-        Some(path) => Box::new(File::create(path)?),
-        None => Box::new(stdout()),
+    let mut out: &mut dyn Write = match args.output.clone() {
+        Some(path) => &mut File::create(path)?,
+        None => &mut stdout(),
     };
 
     match args.format {

src/cli/extract_allowlist.rs

@@ -1,6 +1,6 @@
 use std::{
     fs::{self, read_to_string},
-    io,
+    io::{self, Write},
     path::PathBuf,
 };
 
@@ -51,9 +51,9 @@ pub fn extract_allowlist(args: ExtractAllowlistArgs) -> CliResult {
         .collect::<Vec<_>>();
     let config = ConfigRoot::new(allowlists);
 
-    let mut out: Box<dyn io::Write> = match args.output {
-        Some(path) => Box::new(fs::File::create(path)?),
-        None => Box::new(io::stdout()),
+    let mut out: &mut dyn Write = match args.output {
+        Some(path) => &mut fs::File::create(path)?,
+        None => &mut io::stdout(),
     };
     write!(&mut out, "{}", toml::to_string(&config)?)?;
     SUCCESS

src/cli/format.rs

@@ -22,9 +22,9 @@ pub struct FormatArgs {
 pub fn format(args: FormatArgs) -> CliResult {
     let contents = read_to_string(args.source)?;
     let config: ConfigRoot = toml::from_str(&contents)?;
-    let mut out: Box<dyn Write> = match args.output {
-        Some(path) => Box::new(File::create(path)?),
-        None => Box::new(stdout()),
+    let mut out: &mut dyn Write = match args.output {
+        Some(path) => &mut File::create(path)?,
+        None => &mut stdout(),
     };
     write!(&mut out, "{}", toml::to_string(&config)?)?;
     writeln!(out)?;

src/cli/review.rs

@@ -91,9 +91,9 @@ pub fn review(args: ReviewArgs) -> CliResult {
         .map(|report| filter.apply_report(report))
         .collect::<Vec<FilterResult>>();
 
-    let mut out: Box<dyn Write> = match args.output.as_ref() {
-        Some(path) => Box::new(File::create(path)?),
-        None => Box::new(stdout()),
+    let mut out: &mut dyn Write = match args.output.as_ref() {
+        Some(path) => &mut File::create(path)?,
+        None => &mut stdout(),
     };
     match args.mode {
         Mode::Summary => print_summary(&results, &filter, &mut out)?,