fix(render): make NO_COLOR authoritative for color output

Author: almeidazs

internal/render/colors.go

@@ -2,7 +2,9 @@ package render
 
 import "os"
 
-var IS_COLOR_ENABLED = os.Getenv("NO_COLOR") == "" || os.Getenv("CI") != ""
+func IsColorEnabled() bool {
+	return os.Getenv("NO_COLOR") == ""
+}
 
 const (
 	Reset = "\033[0m"
@@ -16,7 +18,7 @@ const (
 )
 
 func Paint(content, code string, noColor bool) string {
-	if noColor || !IS_COLOR_ENABLED {
+	if noColor || !IsColorEnabled() {
 		return content
 	}
 

internal/render/utils.go

@@ -15,7 +15,7 @@ func Hyperlink(text, url string, noColor bool) string {
 }
 
 func supportsHyperlinks() bool {
-	return IS_COLOR_ENABLED &&
+	return IsColorEnabled() &&
 		os.Getenv("TERM") != "dumb" &&
 		term.IsTerminal(int(os.Stdout.Fd()))
 }

internal/style/style.go

@@ -175,7 +175,7 @@ func printJSONToWriter(w *os.File, data any, noColor bool) error {
 		payload += "\n"
 	}
 
-	if noColor || !render.IS_COLOR_ENABLED {
+	if noColor || !render.IsColorEnabled() {
 		_, err = fmt.Fprint(w, payload)
 		return err
 	}