fix: respect CLICOLOR_FORCE (#3607)

Fixes #1836

src/config/settings.rs

@@ -155,7 +155,13 @@ impl Settings {
         if !settings.color {
             console::set_colors_enabled(false);
             console::set_colors_enabled_stderr(false);
-        } else if ci_info::is_ci() && !cfg!(test) && *env::CLICOLOR != Some(false) {
+        } else if *env::CLICOLOR_FORCE == Some(true) {
+            console::set_colors_enabled(true);
+            console::set_colors_enabled_stderr(true);
+        } else if *env::CLICOLOR == Some(false) {
+            console::set_colors_enabled(false);
+            console::set_colors_enabled_stderr(false);
+        } else if ci_info::is_ci() && !cfg!(test) {
             console::set_colors_enabled_stderr(true);
         }
         if settings.ci {

src/env.rs

@@ -200,8 +200,11 @@ pub static GITHUB_TOKEN: Lazy<Option<String>> = Lazy::new(|| {
 pub static TEST_TRANCHE: Lazy<usize> = Lazy::new(|| var_u8("TEST_TRANCHE") as usize);
 pub static TEST_TRANCHE_COUNT: Lazy<usize> = Lazy::new(|| var_u8("TEST_TRANCHE_COUNT") as usize);
 
+pub static CLICOLOR_FORCE: Lazy<Option<bool>> =
+    Lazy::new(|| var("CLICOLOR_FORCE").ok().map(|v| v != "0"));
+
 pub static CLICOLOR: Lazy<Option<bool>> = Lazy::new(|| {
-    if var("CLICOLOR_FORCE").is_ok_and(|v| v != "0") {
+    if *CLICOLOR_FORCE == Some(true) {
         Some(true)
     } else if let Ok(v) = var("CLICOLOR") {
         Some(v != "0")

src/ui/table.rs

@@ -52,6 +52,9 @@ impl MiseTable {
         table
             .load_preset(comfy_table::presets::NOTHING)
             .set_content_arrangement(ContentArrangement::Dynamic);
+        if console::colors_enabled() {
+            table.enforce_styling();
+        }
         if !no_header && console::user_attended() {
             let headers = headers.iter().map(Self::header).collect_vec();
             table.set_header(headers);