feat: add accessible flag and environment variable for screen reader friendly forms

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())
 

internal/config/config.go

@@ -58,6 +58,7 @@ type Config struct {
 	SlackTestTraceFlag      bool
 	TeamFlag                string
 	TokenFlag               string
+	Accessible              bool
 	NoColor                 bool
 
 	// Feature experiments

internal/config/flags.go

@@ -37,6 +37,7 @@ func (c *Config) InitializeGlobalFlags(cmd *cobra.Command) {
 	cmd.PersistentFlags().BoolVarP(&c.DeprecatedDevFlag, "dev", "d", false, "use dev apis")                                                // Can be removed after v0.25.0
 	cmd.PersistentFlags().StringVarP(&c.DeprecatedWorkspaceFlag, "workspace", "", "", "select workspace or organization by domain name or team ID")
 	cmd.PersistentFlags().StringSliceVarP(&c.ExperimentsFlag, "experiment", "e", nil, "use the experiment(s) in the command")
+	cmd.PersistentFlags().BoolVarP(&c.Accessible, "accessible", "", false, "use accessible prompts for screen readers")
 	cmd.PersistentFlags().BoolVarP(&c.ForceFlag, "force", "f", false, "ignore warnings and continue executing command")
 	cmd.PersistentFlags().BoolVarP(&c.NoColor, "no-color", "", false, "remove styles and formatting from outputs")
 	cmd.PersistentFlags().BoolVarP(&c.SkipUpdateFlag, "skip-update", "s", false, "skip checking for latest version of CLI")

internal/iostreams/forms.go

@@ -37,6 +37,9 @@ func newForm(io *IOStreams, field huh.Field) *huh.Form {
 	} else {
 		form = form.WithTheme(style.ThemeSurvey())
 	}
+	if io != nil && io.config.Accessible {
+		form = form.WithAccessible(true)
+	}
 	return form
 }
 

internal/iostreams/forms_test.go

@@ -415,6 +415,54 @@ func TestFormsUseSlackTheme(t *testing.T) {
 	})
 }
 
+func TestFormsAccessible(t *testing.T) {
+	fsMock := slackdeps.NewFsMock()
+	osMock := slackdeps.NewOsMock()
+	osMock.AddDefaultMocks()
+	cfg := config.NewConfig(fsMock, osMock)
+	cfg.Accessible = true
+	io := NewIOStreams(cfg, fsMock, osMock)
+
+	t.Run("select form accepts valid numbered input", func(t *testing.T) {
+		var selected string
+		f := buildSelectForm(io, "Pick one", []string{"A", "B", "C"}, SelectPromptConfig{}, &selected)
+
+		var out strings.Builder
+		err := f.WithOutput(&out).WithInput(strings.NewReader("2\n")).Run()
+
+		assert.NoError(t, err)
+		assert.Equal(t, "B", selected)
+		assert.Contains(t, out.String(), "1. A")
+		assert.Contains(t, out.String(), "2. B")
+		assert.Contains(t, out.String(), "3. C")
+		assert.Contains(t, out.String(), "Enter a number between 1 and 3")
+	})
+
+	t.Run("confirm form accepts yes/no input", func(t *testing.T) {
+		var choice bool
+		f := buildConfirmForm(io, "Continue?", &choice)
+
+		var out strings.Builder
+		err := f.WithOutput(&out).WithInput(strings.NewReader("y\n")).Run()
+
+		assert.NoError(t, err)
+		assert.True(t, choice)
+		assert.Contains(t, out.String(), "Continue?")
+	})
+
+	t.Run("input form accepts text input", func(t *testing.T) {
+		var input string
+		f := buildInputForm(io, "Name?", InputPromptConfig{}, &input)
+
+		var out strings.Builder
+		err := f.WithOutput(&out).WithInput(strings.NewReader("my-app\n")).Run()
+
+		assert.NoError(t, err)
+		assert.Equal(t, "my-app", input)
+		assert.Contains(t, out.String(), "Name?")
+	})
+}
+
 func TestFormsUseSurveyTheme(t *testing.T) {
 	t.Run("multi-select uses survey prefix without lipgloss", func(t *testing.T) {
 		var selected []string