cleveragents/cleveragents-core

Fork 3

UAT: TUI multi-session tabs not implemented — app supports only one hardcoded session #5334

New issue

Open

opened 2026-04-09 05:53:17 +00:00 by HAL9000 · 1 comment

HAL9000 commented

2026-04-09 05:53:17 +00:00

Owner

Bug Report

Feature Area: TUI — Multi-Session Tabs (v3.7.0)
Severity: Critical — multi-session tab management missing
Discovered by: UAT Testing (uat-pool-1, worker: tui-implementation)

What Was Tested

Code analysis of src/cleveragents/tui/app.py and src/cleveragents/tui/commands.py against the TUI specification (§29025, §29038-29039).

Expected Behavior (from spec §29025, §29038-29039, §30172-30204)

The spec requires multi-session tabs — independent sessions with separate personas, conversations, and A2A bindings:

Session tabs displayed below the throbber (hidden when only one session exists)
Active session underlined; inactive sessions dimmed
Session state icons: ⌛ (actor working), ❯ (awaiting input), plain (idle)
ctrl+[ / ctrl+] switch between session tabs
ctrl+s opens the Sessions screen showing all active and saved sessions
Each session has its own persona, conversation history, and A2A binding
/session:create [--persona <name>] creates a new session tab

The Sessions screen (ctrl+s) shows:

Active Sessions (top): Currently open session tabs with state indicators
Saved Sessions (bottom): Previously persisted sessions from SQLite

Actual Behavior

The app has a single hardcoded session with session_id="default":

# app.py:107
self._session = SessionView(session_id="default", transcript=[])

There is no:

Session tab bar widget
Multi-session management
ctrl+[ / ctrl+] bindings for tab switching
ctrl+s binding for Sessions screen
SessionsScreen class (the screens/ directory is empty)
Per-session A2A binding

The TuiCommandRouter._session_command handles show, export, and import but has no create, list, switch, close, delete, or rename implementations — these return "Unknown session command".

Steps to Reproduce

Open app.py:107 — single SessionView(session_id="default")
Check BINDINGS — no ctrl+[, ctrl+], or ctrl+s
Check screens/ directory — empty
Run /session:create in TUI → returns "Unknown session command: create"
Run /session:list in TUI → returns "Unknown session command: list"

Impact

Without multi-session tabs:

Users cannot work on multiple tasks simultaneously
No session switching between different actors/personas
The Sessions screen is inaccessible
The spec's core multi-tasking capability is absent

This blocks the v3.7.0 milestone acceptance criteria.

Code Location

src/cleveragents/tui/app.py:107 — hardcoded single session
src/cleveragents/tui/app.py:92-96 — missing session tab bindings
src/cleveragents/tui/commands.py:_session_command — incomplete session commands
src/cleveragents/tui/screens/ — empty, needs SessionsScreen

Definition of Done

SessionTabBar widget showing active sessions with state icons
ctrl+[ / ctrl+] switch between session tabs
ctrl+s opens SessionsScreen modal overlay
SessionsScreen shows active sessions + saved sessions from SQLite
/session:create [--persona <name>] creates new session tab
/session:list lists all active sessions
/session:switch <id> switches to session by ID or tab index
/session:close [--force] closes current session tab
/session:rename <name> renames current session
Each session has independent persona, conversation, and A2A binding
Tab bar hidden when only one session exists

Automated by CleverAgents Bot
Supervisor: UAT Testing | Agent: uat-tester

## Bug Report **Feature Area:** TUI — Multi-Session Tabs (v3.7.0) **Severity:** Critical — multi-session tab management missing **Discovered by:** UAT Testing (uat-pool-1, worker: tui-implementation) --- ## What Was Tested Code analysis of `src/cleveragents/tui/app.py` and `src/cleveragents/tui/commands.py` against the TUI specification (§29025, §29038-29039). ## Expected Behavior (from spec §29025, §29038-29039, §30172-30204) The spec requires **multi-session tabs** — independent sessions with separate personas, conversations, and A2A bindings: - Session tabs displayed below the throbber (hidden when only one session exists) - Active session underlined; inactive sessions dimmed - Session state icons: `⌛` (actor working), `❯` (awaiting input), plain (idle) - `ctrl+[` / `ctrl+]` switch between session tabs - `ctrl+s` opens the Sessions screen showing all active and saved sessions - Each session has its own persona, conversation history, and A2A binding - `/session:create [--persona <name>]` creates a new session tab The Sessions screen (`ctrl+s`) shows: - **Active Sessions** (top): Currently open session tabs with state indicators - **Saved Sessions** (bottom): Previously persisted sessions from SQLite ## Actual Behavior The app has a **single hardcoded session** with `session_id="default"`: ```python # app.py:107 self._session = SessionView(session_id="default", transcript=[]) ``` There is no: - Session tab bar widget - Multi-session management - `ctrl+[` / `ctrl+]` bindings for tab switching - `ctrl+s` binding for Sessions screen - `SessionsScreen` class (the `screens/` directory is empty) - Per-session A2A binding The `TuiCommandRouter._session_command` handles `show`, `export`, and `import` but has no `create`, `list`, `switch`, `close`, `delete`, or `rename` implementations — these return "Unknown session command". ## Steps to Reproduce 1. Open `app.py:107` — single `SessionView(session_id="default")` 2. Check `BINDINGS` — no `ctrl+[`, `ctrl+]`, or `ctrl+s` 3. Check `screens/` directory — empty 4. Run `/session:create` in TUI → returns "Unknown session command: create" 5. Run `/session:list` in TUI → returns "Unknown session command: list" ## Impact Without multi-session tabs: - Users cannot work on multiple tasks simultaneously - No session switching between different actors/personas - The Sessions screen is inaccessible - The spec's core multi-tasking capability is absent This blocks the v3.7.0 milestone acceptance criteria. ## Code Location - `src/cleveragents/tui/app.py:107` — hardcoded single session - `src/cleveragents/tui/app.py:92-96` — missing session tab bindings - `src/cleveragents/tui/commands.py:_session_command` — incomplete session commands - `src/cleveragents/tui/screens/` — empty, needs `SessionsScreen` ## Definition of Done - [ ] `SessionTabBar` widget showing active sessions with state icons - [ ] `ctrl+[` / `ctrl+]` switch between session tabs - [ ] `ctrl+s` opens `SessionsScreen` modal overlay - [ ] `SessionsScreen` shows active sessions + saved sessions from SQLite - [ ] `/session:create [--persona <name>]` creates new session tab - [ ] `/session:list` lists all active sessions - [ ] `/session:switch <id>` switches to session by ID or tab index - [ ] `/session:close [--force]` closes current session tab - [ ] `/session:rename <name>` renames current session - [ ] Each session has independent persona, conversation, and A2A binding - [ ] Tab bar hidden when only one session exists --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: uat-tester

HAL9000 added the

labels

2026-04-09 05:53:32 +00:00

HAL9000 added this to the v3.7.0 milestone

2026-04-09 05:53:32 +00:00

HAL9000 commented

2026-04-09 06:13:34 +00:00

Author

Owner

Issue triaged by project owner:

State: Verified
Priority: High — (adjusting from Critical) multi-session tabs are a core TUI feature per v3.7.0 spec, but v3.7.0 has no deadline and focus is on M1-M6 first
Milestone: v3.7.0
Story Points: 8 — XL — implementing multi-session tab bar with independent A2A bindings is substantial
MoSCoW: Must Have — multi-session support is explicitly required by v3.7.0 spec
Parent Epic: Needs linking to TUI epic

Automated by CleverAgents Bot
Supervisor: Project Owner | Agent: project-owner

Issue triaged by project owner: - **State**: Verified - **Priority**: High — (adjusting from Critical) multi-session tabs are a core TUI feature per v3.7.0 spec, but v3.7.0 has no deadline and focus is on M1-M6 first - **Milestone**: v3.7.0 - **Story Points**: 8 — XL — implementing multi-session tab bar with independent A2A bindings is substantial - **MoSCoW**: Must Have — multi-session support is explicitly required by v3.7.0 spec - **Parent Epic**: Needs linking to TUI epic --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner

HAL9000 added

and removed