feat: add accessible flag and environment variable for screen reader friendly forms by srtaalej · Pull Request #455 · slackapi/slack-cli

srtaalej · 2026-03-31T20:12:29Z

Changelog

Interactive forms become screen reader friendly when the ACCESSIBLE environment variable is set or when using the --accessible flag.

Summary

Related: #454
This PR adds a global --accessible flag that switches huh interactive prompts to accessible mode. In accessible mode, select prompts render as numbered lists with "Enter a number between 1 and N" input, confirm prompts accept y/n text, and input prompts use plain line-by-line I/O. This makes the CLI usable with screen readers by avoiding the TUI-based rendering

Test plan

Run slack create --accessible and verify prompts render as numbered lists
Run slack login --accessible and verify confirm prompt accepts y/n
Run slack create (without flag) and verify normal TUI prompts still work
go test ./internal/iostreams/ -run TestFormsAccessible

Requirements

I've read and understood the Contributing Guidelines and have done my best effort to follow them.
I've read and agree to the Code of Conduct.

codecov · 2026-03-31T20:16:42Z

Codecov Report

❌ Patch coverage is 87.50000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.27%. Comparing base (7c7394c) to head (f53744f).

Files with missing lines	Patch %	Lines
cmd/root.go	0.00%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #455      +/-   ##
==========================================
+ Coverage   71.26%   71.27%   +0.01%     
==========================================
  Files         222      222              
  Lines       18682    18696      +14     
==========================================
+ Hits        13314    13326      +12     
- Misses       4188     4190       +2     
  Partials     1180     1180

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

zimeg

@srtaalej So amazing to find this feature arriving here! 🍀 ✨

Before approving I want to leave a few comments, but think we can follow up with some other improvements to the edges:

Environment Variable: The ACCESSIBLE environment variable is standard I understand and we should include support alongside these changes! This makes preferring these prompts easier with an export added to the shell profile 🔬
Blank Selections: The "select" prompts request a number to be entered but accept blank input for the first option! This might be an upstream issue 🐛
Input Default: Adjacent for the "input" prompt I find the default value of the create command name isn't showing 🧪

The first note is most important for these changes IMHO! I'd be curious if the latter two can be proven in unit tests, but please don't consider those blocking 🙏

zimeg · 2026-04-07T06:32:40Z

 	SlackTestTraceFlag      bool
 	TeamFlag                string
 	TokenFlag               string
+	Accessible              bool


🌲 thought: As we're introducing this, should we include the environment variable similar?

ACCESSIBLE

We recommend setting this through an environment variable or configuration option to allow the user to control accessibility.

🔗 https://github.com/charmbracelet/huh?tab=readme-ov-file#accessibility

🔍 ramble: We might find this adjacent file most useful!

slack-cli/internal/config/dotenv.go

Lines 23 to 24 in f39d175

// LoadEnvironmentVariables sets flags based on their environment variable value

func (c *Config) LoadEnvironmentVariables() error {

Co-authored-by: Eden Zimbelman <eden.zimbelman@salesforce.com>

zimeg

@srtaalej Thanks for these updates! I'm hoping we can land a few more to bring this to a polished state for release 🔬 ✨

I tested some more cases that I'll share here:

One selection option: The hint is solid IMHO! I share more comments on the title format later but I'm a fan of the hint.

Multiple select options: This highlights the ":" format a bit more for following comments.

Invalid select options: Super cool that this validates!

Confirmation default: This appears as a kind callout.

Input prompts without default: These might favor the trailing ":" also?

The formatting issue I callout in comments below and some screenshots above is that I think form titles should end with a ":" to signal input! Some workarounds might be needed in code but this outputs well I think.

Let me know if more testing can be requested 🫡

Co-authored-by: Eden Zimbelman <eden.zimbelman@salesforce.com>

zimeg · 2026-04-14T05:57:02Z

🤖 CI error: https://github.com/slackapi/slack-cli/actions/runs/24351525876/job/71211294566#step:4:39

  Running [/Users/runner/golangci-lint-2.11.4-darwin-arm64/golangci-lint config path] in [/Users/runner/work/slack-cli/slack-cli] ...
  Running [/Users/runner/golangci-lint-2.11.4-darwin-arm64/golangci-lint config verify] in [/Users/runner/work/slack-cli/slack-cli] ...
  Running [/Users/runner/golangci-lint-2.11.4-darwin-arm64/golangci-lint run  --timeout=5m] in [/Users/runner/work/slack-cli/slack-cli] ...
  internal/iostreams/forms.go:1: : # github.com/slackapi/slack-cli/internal/iostreams [github.com/slackapi/slack-cli/internal/iostreams.test]
  Error: internal/iostreams/forms.go:53:44: undefined: strings
  Error: internal/iostreams/forms.go:113:50: undefined: strings (typecheck)
  // Copyright 2022-2026 Salesforce, Inc.
  1 issues:
  * typecheck: 1
  
  Error: issues found

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

zimeg

@srtaalej A few more comments of polish before this lands but it's feeling good in testing 💾

The comments below hint at variable suffixes and placement and also standard line endings to these prompts. It'd be awesome to update the preview in the PR description and perhaps include test case also for the password prompt before merge 🙏

zimeg · 2026-05-06T19:40:17Z

 	SlackTestTraceFlag      bool
 	TeamFlag                string
 	TokenFlag               string
+	Accessible              bool


Suggested change

Accessible bool

AccessibleFlag bool

🧮 quibble: We might want to match adjacent configurations and perhaps alphabetical order this? The following NoColor option can perhaps find similar change in a follow up PR I think too

zimeg · 2026-05-06T19:50:53Z

+	if io != nil && io.config.Accessible && cfg.Placeholder != "" {
+		title = fmt.Sprintf("%s (default: %s):", strings.TrimSuffix(message, ":"), cfg.Placeholder)
+	}


📸 suggestion: Let's add a missing ":" to inputs without placeholder too:

With interactive prompts - Unsure if this requires a change but might be nice? If it doesn't feel right we can hold off on update here!

Without interactive prompts - Feels more meaningful to have a split between output and input

📣 I suggest the form following:

Variable name:

🔬 note: Similar comment might be applied to password forms and others?

feat: add --accessible flag to huh interactive prompts

9c26894

srtaalej self-assigned this Mar 31, 2026

srtaalej requested a review from a team as a code owner March 31, 2026 20:12

srtaalej added enhancement M-T: A feature request for new functionality semver:minor Use on pull requests to describe the release version increment labels Mar 31, 2026

srtaalej marked this pull request as draft April 1, 2026 15:13

srtaalej and others added 2 commits April 6, 2026 14:24

accessible implies no-color

7a747ba

Merge branch 'main' into ale-accessibility-flag

f39d175

srtaalej marked this pull request as ready for review April 6, 2026 18:28

srtaalej requested a review from zimeg April 6, 2026 18:40

zimeg reviewed Apr 7, 2026

View reviewed changes

srtaalej and others added 7 commits April 7, 2026 13:06

Update cmd/root_test.go

6dc0c1a

Co-authored-by: Eden Zimbelman <eden.zimbelman@salesforce.com>

add accessible env var support

504bfcf

Merge branch 'main' into ale-accessibility-flag

09c76d0

linter

0417f40

Merge branch 'main' into ale-accessibility-flag

907f5ec

add default placeholders to accessible prompts

2318e16

tests

95a713e

zimeg added this to the Next Release milestone Apr 11, 2026

zimeg added the changelog Use on updates to be included in the release notes label Apr 11, 2026

Merge branch 'main' into ale-accessibility-flag

4f13099

zimeg changed the title ~~feat: add --accessible flag to huh interactive prompts~~ feat: add accessible flag and environment variable for screen reader friendly forms Apr 11, 2026

zimeg reviewed Apr 11, 2026

View reviewed changes

Comment thread internal/iostreams/forms.go Outdated

Comment thread internal/iostreams/forms.go Outdated

srtaalej and others added 2 commits April 13, 2026 11:21

Update internal/iostreams/forms.go

54996c9

Co-authored-by: Eden Zimbelman <eden.zimbelman@salesforce.com>

Update internal/iostreams/forms.go

da077ee

Co-authored-by: Eden Zimbelman <eden.zimbelman@salesforce.com>

srtaalej and others added 3 commits May 5, 2026 12:21

fix: add missing strings import and update test assertion

9652400

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

chore: retrigger CI

25927f7

Merge branch 'main' into ale-accessibility-flag

f53744f

zimeg self-requested a review May 6, 2026 19:33

zimeg reviewed May 6, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

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

feat: add accessible flag and environment variable for screen reader friendly forms#455
srtaalej wants to merge 16 commits intomainfrom
ale-accessibility-flag

srtaalej commented Mar 31, 2026 •

edited by zimeg

Loading

Uh oh!

codecov Bot commented Mar 31, 2026 •

edited

Loading

Uh oh!

zimeg left a comment

Uh oh!

Uh oh!

zimeg Apr 7, 2026

Uh oh!

zimeg Apr 7, 2026

Uh oh!

zimeg left a comment

Uh oh!

Uh oh!

Uh oh!

zimeg commented Apr 14, 2026

Uh oh!

zimeg left a comment

Uh oh!

zimeg May 6, 2026

Uh oh!

zimeg May 6, 2026

Uh oh!

zimeg May 6, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

	// LoadEnvironmentVariables sets flags based on their environment variable value
	func (c *Config) LoadEnvironmentVariables() error {

Conversation

srtaalej commented Mar 31, 2026 • edited by zimeg Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changelog

Summary

Test plan

Requirements

Uh oh!

codecov Bot commented Mar 31, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

zimeg left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

zimeg Apr 7, 2026

Choose a reason for hiding this comment

Uh oh!

zimeg Apr 7, 2026

Choose a reason for hiding this comment

Uh oh!

zimeg left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

zimeg commented Apr 14, 2026

Uh oh!

zimeg left a comment

Choose a reason for hiding this comment

Uh oh!

zimeg May 6, 2026

Choose a reason for hiding this comment

Uh oh!

zimeg May 6, 2026

Choose a reason for hiding this comment

Uh oh!

zimeg May 6, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

srtaalej commented Mar 31, 2026 •

edited by zimeg

Loading

codecov Bot commented Mar 31, 2026 •

edited

Loading