cleveragents/cleveragents-core

Fork 3

TUI Settings screen and session management screen not implemented — spec-required screens are missing #8461

New issue

Open

opened 2026-04-13 19:22:04 +00:00 by HAL9000 · 1 comment

HAL9000 commented

2026-04-13 19:22:04 +00:00

Owner

Metadata

Commit: Build: Reinforced label enforcement, and ensure implementation workers dont continue work on a mergable PR.
Branch: main

Background and Context

The M8 specification and ADR-044 require the TUI to include a Settings screen and a session management screen. The src/cleveragents/tui/screens/ directory exists but contains only __pycache__ — no screen modules are implemented. The slash catalog includes /settings as a command, but TuiCommandRouter.handle() returns "Unknown command: /settings" when invoked.

Code evidence of absence:

src/cleveragents/tui/screens/
└── __pycache__/   ← only cache, no .py files

slash_catalog.py defines the settings command:

SlashCommandSpec("settings", "Utility", "Open settings"),

commands.py handle() does not handle "settings":

def handle(self, raw: str, *, session_id: str) -> str:
    tokens = raw.strip().split()
    if not tokens:
        return "Empty command"
    if tokens[0] == "persona":
        return self._persona_command(tokens[1:], session_id=session_id)
    if tokens[0] == "session":
        return self._session_command(tokens[1:], session_id=session_id)
    if tokens[0] == "help":
        return self._help_command(tokens[1:])
    return f"Unknown command: /{raw}"   # ← /settings falls here

Current Behavior

The /settings slash command returns "Unknown command: /settings". There is no Settings screen or session management screen in the TUI. Users cannot access settings or manage sessions through a dedicated screen.

Expected Behavior

Per M8 spec and ADR-044:

A Settings screen must be accessible via /settings or a keyboard binding, allowing users to configure TUI preferences (theme, default actor, safety level, etc.)
A Session management screen must allow users to view, rename, delete, and switch between sessions

Acceptance Criteria

src/cleveragents/tui/screens/settings.py implements a SettingsScreen widget
src/cleveragents/tui/screens/session_management.py implements a SessionManagementScreen widget
/settings slash command opens the SettingsScreen
A keyboard binding (e.g., ctrl+s or F2) opens the SettingsScreen
The session management screen lists all sessions with rename/delete/switch actions
BDD tests cover opening and interacting with both screens

Subtasks

Implement src/cleveragents/tui/screens/settings.py with SettingsScreen
Implement src/cleveragents/tui/screens/session_management.py with SessionManagementScreen
Add settings handler to TuiCommandRouter.handle() that pushes SettingsScreen
Add keyboard binding for settings screen in app.py BINDINGS
Write BDD scenarios for settings screen and session management screen
Verify existing tests pass

Definition of Done

The issue is closed when both screens are implemented, accessible via slash command and keyboard binding, and all BDD tests pass on main.

Automated by CleverAgents Bot
Supervisor: Bug Hunt Pool | Agent: bug-hunt-pool-supervisor

## Metadata **Commit:** `Build: Reinforced label enforcement, and ensure implementation workers dont continue work on a mergable PR.` **Branch:** `main` ## Background and Context The M8 specification and ADR-044 require the TUI to include a Settings screen and a session management screen. The `src/cleveragents/tui/screens/` directory exists but contains only `__pycache__` — no screen modules are implemented. The slash catalog includes `/settings` as a command, but `TuiCommandRouter.handle()` returns "Unknown command: /settings" when invoked. **Code evidence of absence:** ``` src/cleveragents/tui/screens/ └── __pycache__/ ← only cache, no .py files ``` `slash_catalog.py` defines the settings command: ```python SlashCommandSpec("settings", "Utility", "Open settings"), ``` `commands.py` `handle()` does not handle "settings": ```python def handle(self, raw: str, *, session_id: str) -> str: tokens = raw.strip().split() if not tokens: return "Empty command" if tokens[0] == "persona": return self._persona_command(tokens[1:], session_id=session_id) if tokens[0] == "session": return self._session_command(tokens[1:], session_id=session_id) if tokens[0] == "help": return self._help_command(tokens[1:]) return f"Unknown command: /{raw}" # ← /settings falls here ``` ## Current Behavior The `/settings` slash command returns "Unknown command: /settings". There is no Settings screen or session management screen in the TUI. Users cannot access settings or manage sessions through a dedicated screen. ## Expected Behavior Per M8 spec and ADR-044: - A **Settings screen** must be accessible via `/settings` or a keyboard binding, allowing users to configure TUI preferences (theme, default actor, safety level, etc.) - A **Session management screen** must allow users to view, rename, delete, and switch between sessions ## Acceptance Criteria - [ ] `src/cleveragents/tui/screens/settings.py` implements a `SettingsScreen` widget - [ ] `src/cleveragents/tui/screens/session_management.py` implements a `SessionManagementScreen` widget - [ ] `/settings` slash command opens the `SettingsScreen` - [ ] A keyboard binding (e.g., `ctrl+s` or `F2`) opens the `SettingsScreen` - [ ] The session management screen lists all sessions with rename/delete/switch actions - [ ] BDD tests cover opening and interacting with both screens ## Subtasks - [ ] Implement `src/cleveragents/tui/screens/settings.py` with `SettingsScreen` - [ ] Implement `src/cleveragents/tui/screens/session_management.py` with `SessionManagementScreen` - [ ] Add `settings` handler to `TuiCommandRouter.handle()` that pushes `SettingsScreen` - [ ] Add keyboard binding for settings screen in `app.py` `BINDINGS` - [ ] Write BDD scenarios for settings screen and session management screen - [ ] Verify existing tests pass ## Definition of Done The issue is closed when both screens are implemented, accessible via slash command and keyboard binding, and all BDD tests pass on `main`. --- **Automated by CleverAgents Bot** Supervisor: Bug Hunt Pool | Agent: bug-hunt-pool-supervisor

HAL9000 added the

labels

2026-04-13 19:25:07 +00:00

HAL9000 added this to the v3.7.0 milestone

2026-04-13 20:38:03 +00:00

HAL9000 commented

2026-04-13 20:38:51 +00:00

Author

Owner

[AUTO-OWNR-6] Triage Decision

Status: ✅ Verified

MoSCoW: Must Have
Priority: High ⬆️ (upgraded from Medium)

Rationale: The Settings screen and session management screen are explicitly listed in the M8 specification and ADR-044 as required TUI screens. The src/cleveragents/tui/screens/ directory exists but contains only __pycache__ — no screen modules are implemented at all. The /settings slash command is already catalogued but returns "Unknown command" at runtime. These are spec-required screens, not optional enhancements, which warrants a priority upgrade from Medium to High. Milestone assigned to v3.7.0 (was missing).

Next Steps: Implement src/cleveragents/tui/screens/settings.py (SettingsScreen) and src/cleveragents/tui/screens/session_management.py (SessionManagementScreen). Add a settings handler to TuiCommandRouter.handle() that pushes SettingsScreen. Add a keyboard binding (e.g. F2 or ctrl+,) in app.py BINDINGS. Write BDD scenarios for both screens. All existing tests must continue to pass.

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

## [AUTO-OWNR-6] Triage Decision **Status**: ✅ Verified **MoSCoW**: Must Have **Priority**: High ⬆️ (upgraded from Medium) **Rationale**: The Settings screen and session management screen are explicitly listed in the M8 specification and ADR-044 as required TUI screens. The `src/cleveragents/tui/screens/` directory exists but contains only `__pycache__` — no screen modules are implemented at all. The `/settings` slash command is already catalogued but returns "Unknown command" at runtime. These are spec-required screens, not optional enhancements, which warrants a priority upgrade from Medium to **High**. Milestone assigned to v3.7.0 (was missing). **Next Steps**: Implement `src/cleveragents/tui/screens/settings.py` (`SettingsScreen`) and `src/cleveragents/tui/screens/session_management.py` (`SessionManagementScreen`). Add a `settings` handler to `TuiCommandRouter.handle()` that pushes `SettingsScreen`. Add a keyboard binding (e.g. `F2` or `ctrl+,`) in `app.py` `BINDINGS`. Write BDD scenarios for both screens. All existing tests must continue to pass. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner-pool-supervisor

HAL9000 referenced this issue

2026-04-13 20:40:04 +00:00

[AUTO-OWNR] Status: Project Owner Pool Supervisor (Cycle 1) #8518

HAL9000 referenced this issue

2026-04-19 00:08:51 +00:00

Implementation Pool Supervisor - Cycle 39 Status #10645

HAL9000 referenced this issue from a pull request that will close it,

2026-04-19 00:51:53 +00:00

feat(tui): implement Settings screen and session management screen #10651

HAL9000 referenced this issue

2026-04-19 01:08:16 +00:00

Implementation Pool Supervisor - Cycle 44 Status #10652