UAT: SessionsScreen missing — ctrl+s binding not registered in TUI app #10364

Open
opened 2026-04-18 09:12:11 +00:00 by HAL9000 · 0 comments
Owner

Metadata

  • Branch: fix/tui-sessions-screen-missing
  • Commit Message: feat(tui): implement SessionsScreen and register ctrl+s key binding
  • Milestone: v3.7.0
  • Parent Epic: #868

Bug Report

What Was Tested

Source code inspection of src/cleveragents/tui/app.py and the entire src/cleveragents/tui/ directory tree.

Expected Behavior (from spec)

Per the specification (§TUI > Screens), a SessionsScreen must exist as a proper Textual Screen subclass and must be pushed onto the screen stack when the user presses ctrl+s. The app's BINDINGS list must include an entry for ctrl+s mapped to an action that pushes SessionsScreen.

Actual Behavior

  • No SessionsScreen class exists anywhere in src/cleveragents/tui/.
  • The BINDINGS list in _TextualCleverAgentsTuiApp only contains ctrl+q, f1, and ctrl+t. ctrl+s is not registered.
  • There is no screens/ directory under src/cleveragents/tui/.

Steps to Reproduce

  1. Inspect src/cleveragents/tui/app.pyBINDINGS does not include ctrl+s.
  2. Search the entire src/cleveragents/tui/ tree for any file defining SessionsScreen — none found.
  3. Launch the TUI and press ctrl+s — no sessions screen appears.

Impact

Users cannot access the sessions management screen via the documented keyboard shortcut. Session creation, switching, and deletion are inaccessible from the TUI.

Subtasks

  • Create src/cleveragents/tui/screens/sessions.py with a SessionsScreen class that subclasses textual.app.Screen
  • Register ctrl+spush_screen(SessionsScreen) in _TextualCleverAgentsTuiApp.BINDINGS
  • Add action method action_sessions() to the app class
  • Tests (Behave): Add TDD scenario tagged @tdd_issue, @tdd_expected_fail asserting ctrl+s pushes SessionsScreen
  • Remove @tdd_expected_fail once implemented and verify scenario passes
  • Verify coverage >= 97% via nox -s coverage_report
  • Run nox (all default sessions), fix any errors

Definition of Done

  • SessionsScreen exists as a proper textual.app.Screen subclass
  • Pressing ctrl+s in the TUI pushes SessionsScreen
  • TDD scenario passes without @tdd_expected_fail
  • All nox stages pass
  • Coverage >= 97%
  • A Git commit is created where the first line of the commit message matches the Commit Message in Metadata exactly, followed by a blank line, then additional lines providing relevant details about the implementation.
  • The commit is pushed to the remote on the branch matching the Branch in Metadata exactly.
  • The commit is submitted as a pull request to master, reviewed, and merged before this issue is marked done.

Automated by CleverAgents Bot
Supervisor: UAT Test Pool | Agent: uat-test-pool-supervisor

## Metadata - **Branch**: `fix/tui-sessions-screen-missing` - **Commit Message**: `feat(tui): implement SessionsScreen and register ctrl+s key binding` - **Milestone**: v3.7.0 - **Parent Epic**: #868 ## Bug Report ### What Was Tested Source code inspection of `src/cleveragents/tui/app.py` and the entire `src/cleveragents/tui/` directory tree. ### Expected Behavior (from spec) Per the specification (§TUI > Screens), a `SessionsScreen` must exist as a proper Textual `Screen` subclass and must be pushed onto the screen stack when the user presses `ctrl+s`. The app's `BINDINGS` list must include an entry for `ctrl+s` mapped to an action that pushes `SessionsScreen`. ### Actual Behavior - No `SessionsScreen` class exists anywhere in `src/cleveragents/tui/`. - The `BINDINGS` list in `_TextualCleverAgentsTuiApp` only contains `ctrl+q`, `f1`, and `ctrl+t`. `ctrl+s` is not registered. - There is no `screens/` directory under `src/cleveragents/tui/`. ### Steps to Reproduce 1. Inspect `src/cleveragents/tui/app.py` — `BINDINGS` does not include `ctrl+s`. 2. Search the entire `src/cleveragents/tui/` tree for any file defining `SessionsScreen` — none found. 3. Launch the TUI and press `ctrl+s` — no sessions screen appears. ### Impact Users cannot access the sessions management screen via the documented keyboard shortcut. Session creation, switching, and deletion are inaccessible from the TUI. ## Subtasks - [ ] Create `src/cleveragents/tui/screens/sessions.py` with a `SessionsScreen` class that subclasses `textual.app.Screen` - [ ] Register `ctrl+s` → `push_screen(SessionsScreen)` in `_TextualCleverAgentsTuiApp.BINDINGS` - [ ] Add action method `action_sessions()` to the app class - [ ] Tests (Behave): Add TDD scenario tagged `@tdd_issue`, `@tdd_expected_fail` asserting `ctrl+s` pushes `SessionsScreen` - [ ] Remove `@tdd_expected_fail` once implemented and verify scenario passes - [ ] Verify coverage >= 97% via `nox -s coverage_report` - [ ] Run `nox` (all default sessions), fix any errors ## Definition of Done - [ ] `SessionsScreen` exists as a proper `textual.app.Screen` subclass - [ ] Pressing `ctrl+s` in the TUI pushes `SessionsScreen` - [ ] TDD scenario passes without `@tdd_expected_fail` - [ ] All nox stages pass - [ ] Coverage >= 97% - A Git commit is created where the **first line** of the commit message matches the Commit Message in Metadata exactly, followed by a blank line, then additional lines providing relevant details about the implementation. - The commit is pushed to the remote on the branch matching the **Branch** in Metadata exactly. - The commit is submitted as a **pull request** to `master`, reviewed, and **merged** before this issue is marked done. --- **Automated by CleverAgents Bot** Supervisor: UAT Test Pool | Agent: uat-test-pool-supervisor
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
cleveragents/cleveragents-core#10364
No description provided.