UAT: agents session delete rich output is a one-liner — missing spec-required Deletion Summary and Cleanup panels #1550

Closed
opened 2026-04-02 20:54:47 +00:00 by freemo · 1 comment
Owner

Metadata

  • Branch: fix/uat-session-delete-rich-output-panels
  • Commit Message: fix(cli): add Deletion Summary and Cleanup panels to session delete rich output
  • Milestone: v3.7.0
  • Parent Epic: #374

Background and Context

The specification (docs/specification.md, agents session delete section, line ~1872) defines a structured Rich output format for agents session delete. The current implementation in src/cleveragents/cli/commands/session.py (delete() function) only prints a single success line, missing all required structured panels.

Current Behavior

Running agents session delete <SESSION_ID> --yes renders:

✓ OK Session <id> deleted

Expected Behavior

Per the specification, the Rich output must render two panels plus a success message:

╭─ Deletion Summary ──────────────────╮
│ Session: 01HXM2A6K1P2E9Q9D4GQ7J4S7Z │
│ ID: 01HXM2A6K1P2E9Q9D4GQ7J4S7Z      │
│ Messages: 6 removed                 │
│ Storage: 18 KB freed                │
│ Plans Orphaned: 0                   │
╰─────────────────────────────────────╯

╭─ Cleanup ───────────╮
│ Backups: none       │
│ Logs: preserved     │
│ Context: cleared    │
│ Checkpoints: none   │
╰─────────────────────╯

✓ OK Session deleted

Specifically, the following are missing:

  1. Deletion Summary panel — must show Session ID, Messages removed, Storage freed, Plans Orphaned
  2. Cleanup panel — must show Backups, Logs, Context, Checkpoints status
  3. Success message format — spec requires ✓ OK Session deleted (not ✓ OK Session <id> deleted)

Acceptance Criteria

  • agents session delete rich output renders a Deletion Summary panel with Session, ID, Messages removed, Storage freed, Plans Orphaned
  • A Cleanup panel is rendered showing Backups, Logs, Context, Checkpoints
  • The success message reads ✓ OK Session deleted
  • All existing tests pass; new tests cover the corrected Rich output format

Subtasks

  • Audit delete() in src/cleveragents/cli/commands/session.py against the spec
  • Add Deletion Summary panel with required fields
  • Add Cleanup panel with required fields
  • Fix success message format to ✓ OK Session deleted
  • Write / update unit tests for the corrected Rich output
  • Manually verify output matches the spec example exactly

Definition of Done

  • All subtasks above are completed
  • agents session delete rich output matches the spec format exactly (two panels + success message)
  • No regressions in other agents session subcommands
  • Commit message matches: fix(cli): add Deletion Summary and Cleanup panels to session delete rich output
  • Pull Request merged into the target branch
  • All nox stages pass
  • Coverage >= 97%

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

## Metadata - **Branch**: `fix/uat-session-delete-rich-output-panels` - **Commit Message**: `fix(cli): add Deletion Summary and Cleanup panels to session delete rich output` - **Milestone**: v3.7.0 - **Parent Epic**: #374 ## Background and Context The specification (`docs/specification.md`, `agents session delete` section, line ~1872) defines a structured Rich output format for `agents session delete`. The current implementation in `src/cleveragents/cli/commands/session.py` (`delete()` function) only prints a single success line, missing all required structured panels. ## Current Behavior Running `agents session delete <SESSION_ID> --yes` renders: ``` ✓ OK Session <id> deleted ``` ## Expected Behavior Per the specification, the Rich output must render **two panels** plus a success message: ``` ╭─ Deletion Summary ──────────────────╮ │ Session: 01HXM2A6K1P2E9Q9D4GQ7J4S7Z │ │ ID: 01HXM2A6K1P2E9Q9D4GQ7J4S7Z │ │ Messages: 6 removed │ │ Storage: 18 KB freed │ │ Plans Orphaned: 0 │ ╰─────────────────────────────────────╯ ╭─ Cleanup ───────────╮ │ Backups: none │ │ Logs: preserved │ │ Context: cleared │ │ Checkpoints: none │ ╰─────────────────────╯ ✓ OK Session deleted ``` Specifically, the following are missing: 1. **`Deletion Summary` panel** — must show Session ID, Messages removed, Storage freed, Plans Orphaned 2. **`Cleanup` panel** — must show Backups, Logs, Context, Checkpoints status 3. **Success message format** — spec requires `✓ OK Session deleted` (not `✓ OK Session <id> deleted`) ## Acceptance Criteria - [ ] `agents session delete` rich output renders a `Deletion Summary` panel with Session, ID, Messages removed, Storage freed, Plans Orphaned - [ ] A `Cleanup` panel is rendered showing Backups, Logs, Context, Checkpoints - [ ] The success message reads `✓ OK Session deleted` - [ ] All existing tests pass; new tests cover the corrected Rich output format ## Subtasks - [ ] Audit `delete()` in `src/cleveragents/cli/commands/session.py` against the spec - [ ] Add `Deletion Summary` panel with required fields - [ ] Add `Cleanup` panel with required fields - [ ] Fix success message format to `✓ OK Session deleted` - [ ] Write / update unit tests for the corrected Rich output - [ ] Manually verify output matches the spec example exactly ## Definition of Done - [ ] All subtasks above are completed - [ ] `agents session delete` rich output matches the spec format exactly (two panels + success message) - [ ] No regressions in other `agents session` subcommands - [ ] Commit message matches: `fix(cli): add Deletion Summary and Cleanup panels to session delete rich output` - [ ] Pull Request merged into the target branch - All nox stages pass - Coverage >= 97% --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: ca-uat-tester
freemo self-assigned this 2026-04-02 20:58:56 +00:00
Author
Owner

Implementation Complete

Added spec-required Deletion Summary and Cleanup panels to agents session delete rich output.

Changes:

  • Deletion Summary panel with Session, ID, Messages removed, Storage freed, Plans Orphaned
  • Cleanup panel with Backups, Logs, Context, Checkpoints status
  • Fixed success message format to ✓ OK Session deleted
  • Added --format option for output format control

PR: #1569 (MERGED)
Commit: cc9ce3d4


Automated by CleverAgents Bot
Supervisor: Product Builder | Agent: product-builder

✅ **Implementation Complete** Added spec-required `Deletion Summary` and `Cleanup` panels to `agents session delete` rich output. **Changes:** - Deletion Summary panel with Session, ID, Messages removed, Storage freed, Plans Orphaned - Cleanup panel with Backups, Logs, Context, Checkpoints status - Fixed success message format to `✓ OK Session deleted` - Added `--format` option for output format control **PR**: #1569 (MERGED) **Commit**: cc9ce3d4 --- **Automated by CleverAgents Bot** Supervisor: Product Builder | Agent: product-builder
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#1550
No description provided.