Skip to content

feat(tui): add pairing password toggle - #40934

Open
kitlangton wants to merge 1 commit into
v2from
pair-password-reveal
Open

feat(tui): add pairing password toggle#40934
kitlangton wants to merge 1 commit into
v2from
pair-password-reveal

Conversation

@kitlangton

Copy link
Copy Markdown
Contributor

What

Make pairing-password visibility discoverable and keyboard accessible.

Before / After

Before: The masked password could only be revealed by clicking the password text, with no visible affordance.

After: The dialog shows a configurable space show password / space hide password action that works from the keyboard or mouse.

How

  • Register a modal dialog.pair.toggle_password command.
  • Default the command to space in the V2 TUI keybind definitions.
  • Reuse one toggle function for keyboard and mouse activation.

Scope

The password remains masked by default. Pairing credentials and transport behavior are unchanged.

Testing

  • bun typecheck from packages/tui
  • bun run test test/keybind.test.ts test/keymap.test.tsx
  • Repository pre-push typecheck
  • git diff --check

@kitlangton

Copy link
Copy Markdown
Contributor Author

Demo

The recording uses the fixed fixture credential demo-password; no real credential is present.

Before: There is no visible password action, and pressing Space has no effect.

pair-before.mp4

After: The visible Space action reveals the fixture password and changes to hide password.

pair-after.mp4

@kitlangton
kitlangton enabled auto-merge (squash) August 6, 2026 23:25
@kitlangton
kitlangton force-pushed the pair-password-reveal branch from da15694 to 536b99b Compare August 6, 2026 23:58
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