Respect terminal color capabilities in forc output

Detect NO_COLOR, FORC_TERM_COLOR, and non-interactive streams before
emitting ANSI

Refactor formatting logic into a pure helper and cover it with unit
tests

Drop the unused tracing-test dev dependency now that log capture isn’t
needed

Author: tritao

Cargo.lock

@@ -352,10 +352,8 @@ where
                 true => format!(" {}", pkg.source),
                 false => String::new(),
             };
-            println_action_red(
-                "Removing",
-                &format!("{}{src}", ansiterm::Style::new().bold().paint(&pkg.name)),
-            );
+            let pkg_name = format_pkg_name(&pkg.name);
+            println_action_red("Removing", &format!("{pkg_name}{src}"));
         }
     }
 }
@@ -370,14 +368,20 @@ where
                 true => format!(" {}", pkg.source),
                 false => "".to_string(),
             };
-            println_action_green(
-                "Adding",
-                &format!("{}{src}", ansiterm::Style::new().bold().paint(&pkg.name)),
-            );
+            let pkg_name = format_pkg_name(&pkg.name);
+            println_action_green("Adding", &format!("{pkg_name}{src}"));
         }
     }
 }
 
+fn format_pkg_name(name: &str) -> String {
+    if forc_tracing::ansi_output_enabled() {
+        format!("{}", ansiterm::Style::new().bold().paint(name))
+    } else {
+        name.to_owned()
+    }
+}
+
 #[cfg(test)]
 mod tests {
     use sway_core::fuel_prelude::fuel_tx;

forc-pkg/src/lock.rs

@@ -2334,11 +2334,23 @@ fn print_pkg_summary_header(built_pkg: &BuiltPackage) {
     // type and name around the 10th column ourselves.
     let padded_ty_str = format!("{prog_ty_str:>10}");
     let padding = &padded_ty_str[..padded_ty_str.len() - prog_ty_str.len()];
-    let ty_ansi = ansiterm::Colour::Green.bold().paint(prog_ty_str);
-    let name_ansi = ansiterm::Style::new()
-        .bold()
-        .paint(&built_pkg.descriptor.name);
-    debug!("{padding}{ty_ansi} {name_ansi}");
+    let colors_enabled = forc_tracing::ansi_output_enabled();
+    let ty_fmt = if colors_enabled {
+        format!("{}", ansiterm::Colour::Green.bold().paint(prog_ty_str))
+    } else {
+        prog_ty_str.to_string()
+    };
+    let name_fmt = if colors_enabled {
+        format!(
+            "{}",
+            ansiterm::Style::new()
+                .bold()
+                .paint(&built_pkg.descriptor.name)
+        )
+    } else {
+        built_pkg.descriptor.name.clone()
+    };
+    debug!("{padding}{ty_fmt} {name_fmt}");
 }
 
 /// Returns the ContractId of a built_package contract with specified `salt`.

forc-pkg/src/pkg.rs

@@ -205,10 +205,12 @@ impl source::Fetch for Pinned {
         {
             let _guard = lock.write()?;
             if !repo_path.exists() {
-                println_action_green(
-                    "Fetching",
-                    &format!("{} {}", ansiterm::Style::new().bold().paint(ctx.name), self),
-                );
+                let pkg_name = if forc_tracing::ansi_output_enabled() {
+                    format!("{}", ansiterm::Style::new().bold().paint(ctx.name))
+                } else {
+                    ctx.name.to_string()
+                };
+                println_action_green("Fetching", &format!("{pkg_name} {}", self));
                 fetch(ctx.fetch_id(), ctx.name(), self)?;
             }
         }

forc-pkg/src/source/git/mod.rs

@@ -74,10 +74,12 @@ impl source::Fetch for Pinned {
         {
             let _guard = lock.write()?;
             if !repo_path.exists() {
-                println_action_green(
-                    "Fetching",
-                    &format!("{} {}", ansiterm::Style::new().bold().paint(ctx.name), self),
-                );
+                let pkg_name = if forc_tracing::ansi_output_enabled() {
+                    format!("{}", ansiterm::Style::new().bold().paint(ctx.name))
+                } else {
+                    ctx.name.to_string()
+                };
+                println_action_green("Fetching", &format!("{pkg_name} {}", self));
                 let cid = self.0.clone();
                 let ipfs_node = ctx.ipfs_node().clone();
                 let ipfs_client = ipfs_client();

forc-pkg/src/source/ipfs.rs

@@ -381,14 +381,12 @@ impl source::Fetch for Pinned {
         {
             let _guard = lock.write()?;
             if !path.exists() {
-                println_action_green(
-                    "Fetching",
-                    &format!(
-                        "{} {}",
-                        ansiterm::Style::new().bold().paint(ctx.name),
-                        self.source.version
-                    ),
-                );
+                let pkg_name = if forc_tracing::ansi_output_enabled() {
+                    format!("{}", ansiterm::Style::new().bold().paint(ctx.name))
+                } else {
+                    ctx.name.to_string()
+                };
+                println_action_green("Fetching", &format!("{pkg_name} {}", self.source.version));
                 let pinned = self.clone();
                 let fetch_id = ctx.fetch_id();
                 let ipfs_node = ctx.ipfs_node().clone();

forc-pkg/src/source/reg/mod.rs

@@ -14,8 +14,5 @@ regex.workspace = true
 tracing.workspace = true
 tracing-subscriber = { workspace = true, features = ["ansi", "env-filter", "json"] }
 
-[dev-dependencies]
-tracing-test.workspace = true
-
 [lints]
 workspace = true