UAT: TUI MainScreen missing tab-based persona cycling (tab key not bound to cycle_persona) #1336

New issue

Open

opened 2026-04-02 16:53:46 +00:00 by freemo · 0 comments

freemo commented

2026-04-02 16:53:46 +00:00

Owner

Bug Report: [tui] — `tab` key is not bound to persona cycling in MainScreen

Severity Assessment

Impact: High. The tab key is the primary way to switch between personas in the TUI. Without this binding, users cannot cycle through their configured personas.
Likelihood: 100% reproducible — tab is not registered as a binding.
Priority: High

Location

File: src/cleveragents/tui/app.py — BINDINGS class variable (lines 90–93)

Description

The specification (§MainScreen — Prompt Focused) states:

tab — Cycle to next persona

The spec footer mockup shows:

F1 Help │ shift+tab Sidebar │ tab Persona │ ctrl+tab Preset │ ctrl+s Sessions │ ctrl+q Quit

The spec also describes persona cycling:

"tab cycles through personas in the configured cycle list."

tab:       persona_1 → persona_2 → persona_3 → persona_1 → ...

Actual Behavior

The BINDINGS class variable does not include tab:

BINDINGS: ClassVar[list[tuple[str, str, str]]] = [
    ("ctrl+q", "quit", "Quit"),
    ("f1", "help", "Help"),
    ("ctrl+t", "cycle_preset", "Cycle Preset"),
]

There is no action_cycle_persona() method in app.py. The PersonaState class does have a cycle_persona() method (in src/cleveragents/tui/persona/state.py), but it is never called from the app.

Expected Behavior (from spec)

The app should register a tab binding that:

Calls self._persona_state.cycle_persona(self._session.session_id)
Refreshes the PersonaBar to show the new active persona
Updates the session's actor binding for subsequent prompts

Steps to Reproduce

Configure multiple personas in ~/.config/cleveragents/personas/
Run the TUI
Press tab — no persona cycling occurs (Textual may use tab for focus cycling instead)

Additional Note

Since tab is Textual's default focus-cycling key, implementing persona cycling on tab requires careful handling to prevent conflicts with widget focus navigation. The spec implies this should work when the prompt is focused (not when navigating within the sidebar).

#694 (feat(tui): Textual MainScreen with sidebar states and Dracula theme)
#1290 (UAT: TUI help panel key bindings are inconsistent with actual app bindings)
#1330 (UAT: TUI MainScreen missing keyboard bindings)

References

Spec §MainScreen — Prompt Focused keyboard table
Spec §Persona Cycling
Spec §PersonaBar

## Bug Report: [tui] — `tab` key is not bound to persona cycling in MainScreen ### Severity Assessment - **Impact**: High. The `tab` key is the primary way to switch between personas in the TUI. Without this binding, users cannot cycle through their configured personas. - **Likelihood**: 100% reproducible — `tab` is not registered as a binding. - **Priority**: High ### Location - **File**: `src/cleveragents/tui/app.py` — `BINDINGS` class variable (lines 90–93) ### Description The specification (§MainScreen — Prompt Focused) states: > `tab` — Cycle to next persona The spec footer mockup shows: ``` F1 Help │ shift+tab Sidebar │ tab Persona │ ctrl+tab Preset │ ctrl+s Sessions │ ctrl+q Quit ``` The spec also describes persona cycling: > "`tab` cycles through personas in the configured cycle list." ``` tab: persona_1 → persona_2 → persona_3 → persona_1 → ... ``` ### Actual Behavior The `BINDINGS` class variable does not include `tab`: ```python BINDINGS: ClassVar[list[tuple[str, str, str]]] = [ ("ctrl+q", "quit", "Quit"), ("f1", "help", "Help"), ("ctrl+t", "cycle_preset", "Cycle Preset"), ] ``` There is no `action_cycle_persona()` method in `app.py`. The `PersonaState` class does have a `cycle_persona()` method (in `src/cleveragents/tui/persona/state.py`), but it is never called from the app. ### Expected Behavior (from spec) The app should register a `tab` binding that: 1. Calls `self._persona_state.cycle_persona(self._session.session_id)` 2. Refreshes the PersonaBar to show the new active persona 3. Updates the session's actor binding for subsequent prompts ### Steps to Reproduce 1. Configure multiple personas in `~/.config/cleveragents/personas/` 2. Run the TUI 3. Press `tab` — no persona cycling occurs (Textual may use `tab` for focus cycling instead) ### Additional Note Since `tab` is Textual's default focus-cycling key, implementing persona cycling on `tab` requires careful handling to prevent conflicts with widget focus navigation. The spec implies this should work when the prompt is focused (not when navigating within the sidebar). ### Related Issues - #694 (feat(tui): Textual MainScreen with sidebar states and Dracula theme) - #1290 (UAT: TUI help panel key bindings are inconsistent with actual app bindings) - #1330 (UAT: TUI MainScreen missing keyboard bindings) ### References - Spec §MainScreen — Prompt Focused keyboard table - Spec §Persona Cycling - Spec §PersonaBar

freemo added the

labels

2026-04-02 16:55:49 +00:00

freemo referenced this issue

2026-04-02 16:56:01 +00:00

UAT: TUI app uses ctrl+t for preset cycling instead of spec-required ctrl+tab #1340