UAT: TUI /session command missing 6 of 9 required subcommands #10328

Open
opened 2026-04-18 08:46:34 +00:00 by HAL9000 · 0 comments
Owner

Metadata

  • Issue Type: Type/Bug
  • Priority: Priority/Critical
  • MoSCoW: MoSCoW/Must Have
  • Branch: bugfix/m130-tui-session-subcommands
  • Commit Message: fix(tui): implement missing session subcommands create/list/switch/close/delete/rename

Description

During UAT testing of the TUI Session Persistence feature, only 3 of the 9 required /session subcommands are implemented in the TUI.

Expected behaviour (from spec):
All 9 session subcommands must work in the TUI: create, list, switch, close, delete, rename, show, export, import.

Actual behaviour:
Inspecting src/cleveragents/tui/commands.py, the _session_command() method only handles 3 subcommands:

  • show — returns current session ID
  • export — calls _session_export()
  • import — calls _session_import()

The following 6 subcommands are not implemented and fall through to "Unknown session command: ...":

  • create — not handled
  • list — not handled
  • switch — not handled
  • close — not handled
  • delete — not handled
  • rename — not handled

Reproduction steps:

  1. Inspect src/cleveragents/tui/commands.py, method _session_command().
  2. Observe that only show, export, and import are handled.
  3. Any other subcommand returns "Unknown session command: <subcommand>".

Acceptance Criteria

  • /session create creates a new session and switches to it.
  • /session list lists all available sessions.
  • /session switch <id> switches the active session.
  • /session close closes the current session.
  • /session delete <id> deletes a session.
  • /session rename <id> <name> renames a session.
  • All 6 new subcommands are covered by Behave and Robot Framework tests.

Subtasks

  • Implement /session create in TuiCommandRouter._session_command().
  • Implement /session list in TuiCommandRouter._session_command().
  • Implement /session switch <id> in TuiCommandRouter._session_command().
  • Implement /session close in TuiCommandRouter._session_command().
  • Implement /session delete <id> in TuiCommandRouter._session_command().
  • Implement /session rename <id> <name> in TuiCommandRouter._session_command().
  • Write Behave scenarios for each new subcommand.
  • Write Robot Framework tests for each new subcommand.
  • Verify coverage ≥97% via nox -s coverage_report.
  • Run nox (all default sessions), fix any errors.

Definition of Done

This issue is complete when:

  • All subtasks above are completed and checked off.
  • 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 - **Issue Type:** Type/Bug - **Priority:** Priority/Critical - **MoSCoW:** MoSCoW/Must Have - **Branch:** bugfix/m130-tui-session-subcommands - **Commit Message:** fix(tui): implement missing session subcommands create/list/switch/close/delete/rename ## Description During UAT testing of the TUI Session Persistence feature, only 3 of the 9 required `/session` subcommands are implemented in the TUI. **Expected behaviour (from spec):** All 9 session subcommands must work in the TUI: `create`, `list`, `switch`, `close`, `delete`, `rename`, `show`, `export`, `import`. **Actual behaviour:** Inspecting `src/cleveragents/tui/commands.py`, the `_session_command()` method only handles 3 subcommands: - `show` — returns current session ID - `export` — calls `_session_export()` - `import` — calls `_session_import()` The following 6 subcommands are **not implemented** and fall through to `"Unknown session command: ..."`: - `create` — not handled - `list` — not handled - `switch` — not handled - `close` — not handled - `delete` — not handled - `rename` — not handled **Reproduction steps:** 1. Inspect `src/cleveragents/tui/commands.py`, method `_session_command()`. 2. Observe that only `show`, `export`, and `import` are handled. 3. Any other subcommand returns `"Unknown session command: <subcommand>"`. ## Acceptance Criteria - [ ] `/session create` creates a new session and switches to it. - [ ] `/session list` lists all available sessions. - [ ] `/session switch <id>` switches the active session. - [ ] `/session close` closes the current session. - [ ] `/session delete <id>` deletes a session. - [ ] `/session rename <id> <name>` renames a session. - [ ] All 6 new subcommands are covered by Behave and Robot Framework tests. ## Subtasks - [ ] Implement `/session create` in `TuiCommandRouter._session_command()`. - [ ] Implement `/session list` in `TuiCommandRouter._session_command()`. - [ ] Implement `/session switch <id>` in `TuiCommandRouter._session_command()`. - [ ] Implement `/session close` in `TuiCommandRouter._session_command()`. - [ ] Implement `/session delete <id>` in `TuiCommandRouter._session_command()`. - [ ] Implement `/session rename <id> <name>` in `TuiCommandRouter._session_command()`. - [ ] Write Behave scenarios for each new subcommand. - [ ] Write Robot Framework tests for each new subcommand. - [ ] Verify coverage ≥97% via `nox -s coverage_report`. - [ ] Run `nox` (all default sessions), fix any errors. ## Definition of Done This issue is complete when: - All subtasks above are completed and checked off. - 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#10328
No description provided.