Skip to content

fix(tui): distinguish disconnect actions - #40937

Open
kitlangton wants to merge 1 commit into
v2from
group-disconnect
Open

fix(tui): distinguish disconnect actions#40937
kitlangton wants to merge 1 commit into
v2from
group-disconnect

Conversation

@kitlangton

Copy link
Copy Markdown
Contributor

What

Separate connected accounts from connection methods and visibly mark Disconnect as destructive.

Before / After

Before: Disconnect entries appeared alongside connection methods with the same primary focus styling.

After: Existing accounts appear under Connected accounts, and focused Disconnect actions use destructive colors.

How

  • Categorize credential-backed disconnect options.
  • Supply destructive focused foreground and background colors to the select option.

Scope

Connection, credential removal, and environment-discovered integrations behave as before.

Testing

  • bun typecheck from packages/tui
  • bun run test test/cli/cmd/tui/integration-options.test.ts
  • Repository pre-push typecheck
  • git diff --check

@kitlangton

Copy link
Copy Markdown
Contributor Author

Demo

The recording uses a fixed Work fixture connection.

Before: Disconnect is mixed with connection methods and uses the normal focused style.

disconnect-before.mp4

After: Disconnect appears under Connected accounts and uses destructive focused colors.

disconnect-after.mp4

@kitlangton
kitlangton enabled auto-merge (squash) August 6, 2026 23:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant