accessible implies no-color

Author: srtaalej

cmd/root.go

@@ -247,6 +247,11 @@ func InitConfig(ctx context.Context, clients *shared.ClientFactory, rootCmd *cob
 		clients.Config.SystemConfig.SetCustomConfigDirPath(clients.Config.ConfigDirFlag)
 	}
 
+	// Accessible mode implies no-color
+	if clients.Config.Accessible {
+		clients.Config.NoColor = true
+	}
+
 	// Init color and formatting
 	style.ToggleStyles(clients.IO.IsTTY() && !clients.Config.NoColor)
 	style.ToggleSpinner(clients.IO.IsTTY() && !clients.Config.NoColor && !clients.Config.DebugEnabled)

cmd/root_test.go

@@ -21,6 +21,7 @@ import (
 	"strings"
 	"testing"
 
+	"github.com/slackapi/slack-cli/internal/config"
 	"github.com/slackapi/slack-cli/internal/iostreams"
 	"github.com/slackapi/slack-cli/internal/shared"
 	"github.com/slackapi/slack-cli/internal/slackcontext"
@@ -194,6 +195,15 @@ func TestVersionFlags(t *testing.T) {
 	assert.True(t, testutil.ContainsSemVer(output), `-v should output the version number but yielded "%s"`, output)
 }
 
+func Test_AccessibleImpliesNoColor(t *testing.T) {
+	cfg := &config.Config{Accessible: true}
+	// Simulate the logic from PersistentPreRunE
+	if cfg.Accessible {
+		cfg.NoColor = true
+	}
+	assert.True(t, cfg.NoColor, "--accessible should imply --no-color")
+}
+
 func Test_NewSuggestion(t *testing.T) {
 	ctx := slackcontext.MockContext(t.Context())