UAT: agents project delete rich output missing spec-required structured panels (Deletion Summary, Index Cleanup, Backups) #1986

Open
opened 2026-04-03 00:31:24 +00:00 by freemo · 2 comments
Owner

Metadata

  • Branch: fix/project-delete-rich-output-panels
  • Commit Message: fix(cli): render Deletion Summary, Index Cleanup, and Backups panels for agents project delete
  • Milestone: v3.5.0
  • Parent Epic: #936

Background and Context

The agents project delete command's rich output format in src/cleveragents/cli/commands/project.py does not render the structured panels required by the specification. Per docs/specification.md lines 3417–3540 (agents project delete section), successful deletion must display three Rich panels — Deletion Summary, Index Cleanup, and Backups — followed by a ✓ OK Project deleted status line.

Note: Issue #1801 covers the incorrect --force flag behavior (blocking on linked resources instead of active plans). This issue covers the missing output panels on successful deletion.

Current Behavior

In src/cleveragents/cli/commands/project.py, the delete() function (approximately line 978) outputs a single one-liner:

console.print(f"[green]✓[/green] Project '{name}' deleted.")

This produces only: ✓ Project '<name>' deleted.

Missing output elements:

  1. Deletion Summary panel — Project name, Resources unlinked count, Data Dir path
  2. Index Cleanup panel — Text Index status, Vectors removed count, Graph Triples status, Storage Freed
  3. Backups panel — Snapshot path, Retention days
  4. ✓ OK Project deleted status message (correct spec format)

Expected Behavior

Per docs/specification.md lines 3439–3467, the rich output on successful deletion should render:

╭─ Deletion Summary ──────────────────╮
│ Project: local/docs                 │
│ Resources: 1 unlinked               │
│ Data Dir: /repos/docs/.cleveragents │
╰─────────────────────────────────────╯

╭─ Index Cleanup ────────╮
│ Text Index: cleared    │
│ Vectors: 240 removed   │
│ Graph Triples: none    │
│ Storage Freed: 12 MB   │
╰────────────────────────╯

╭─ Backups ────────────────────────────────────╮
│ Snapshot: /backups/local-docs-2026-02-08.tgz │
│ Retention: 7 days                            │
╰──────────────────────────────────────────────╯

✓ OK Project deleted

Acceptance Criteria

  • On successful deletion, the delete() command renders a Deletion Summary Rich panel containing: project name, resources unlinked count, and data directory path.
  • On successful deletion, the delete() command renders an Index Cleanup Rich panel containing: text index status, vectors removed count, graph triples status, and storage freed.
  • On successful deletion, the delete() command renders a Backups Rich panel containing: snapshot path and retention days.
  • The final status line reads ✓ OK Project deleted (matching spec format).
  • The single-line ✓ Project '<name>' deleted. output is replaced by the structured panel output.
  • All existing tests continue to pass.

Supporting Information

  • File: src/cleveragents/cli/commands/project.py
  • Function: delete()
  • Spec References:
    • docs/specification.md lines 3417–3540 (agents project delete section)
    • Lines 3439–3467: Rich output example showing all required panels
  • Related issue: #1801 (incorrect --force flag behavior — separate concern)

Subtasks

  • Implement Deletion Summary Rich panel in delete() (project name, resources unlinked, data dir)
  • Implement Index Cleanup Rich panel in delete() (text index, vectors removed, graph triples, storage freed)
  • Implement Backups Rich panel in delete() (snapshot path, retention days)
  • Replace single-line output with ✓ OK Project deleted status line
  • Tests (Behave): Add BDD scenarios verifying all three panels and status line appear on successful deletion
  • Tests (Robot): Add integration test verifying rich output structure for agents project delete
  • 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 (fix(cli): render Deletion Summary, Index Cleanup, and Backups panels for agents project delete), followed by a blank line, then additional lines providing relevant implementation details.
  • The commit is pushed to the remote on the branch matching the Branch in Metadata exactly (fix/project-delete-rich-output-panels).
  • The commit is submitted as a pull request to master, reviewed, and merged before this issue is marked done.
  • All nox stages pass.
  • Coverage >= 97%.

Automated by CleverAgents Bot
Supervisor: UAT Testing | Agent: ca-new-issue-creator

## Metadata - **Branch**: `fix/project-delete-rich-output-panels` - **Commit Message**: `fix(cli): render Deletion Summary, Index Cleanup, and Backups panels for agents project delete` - **Milestone**: v3.5.0 - **Parent Epic**: #936 ## Background and Context The `agents project delete` command's rich output format in `src/cleveragents/cli/commands/project.py` does not render the structured panels required by the specification. Per `docs/specification.md` lines 3417–3540 (agents project delete section), successful deletion must display three Rich panels — **Deletion Summary**, **Index Cleanup**, and **Backups** — followed by a `✓ OK Project deleted` status line. Note: Issue #1801 covers the incorrect `--force` flag behavior (blocking on linked resources instead of active plans). This issue covers the missing output panels on successful deletion. ## Current Behavior In `src/cleveragents/cli/commands/project.py`, the `delete()` function (approximately line 978) outputs a single one-liner: ```python console.print(f"[green]✓[/green] Project '{name}' deleted.") ``` This produces only: `✓ Project '<name>' deleted.` Missing output elements: 1. **Deletion Summary panel** — Project name, Resources unlinked count, Data Dir path 2. **Index Cleanup panel** — Text Index status, Vectors removed count, Graph Triples status, Storage Freed 3. **Backups panel** — Snapshot path, Retention days 4. **`✓ OK Project deleted`** status message (correct spec format) ## Expected Behavior Per `docs/specification.md` lines 3439–3467, the rich output on successful deletion should render: ``` ╭─ Deletion Summary ──────────────────╮ │ Project: local/docs │ │ Resources: 1 unlinked │ │ Data Dir: /repos/docs/.cleveragents │ ╰─────────────────────────────────────╯ ╭─ Index Cleanup ────────╮ │ Text Index: cleared │ │ Vectors: 240 removed │ │ Graph Triples: none │ │ Storage Freed: 12 MB │ ╰────────────────────────╯ ╭─ Backups ────────────────────────────────────╮ │ Snapshot: /backups/local-docs-2026-02-08.tgz │ │ Retention: 7 days │ ╰──────────────────────────────────────────────╯ ✓ OK Project deleted ``` ## Acceptance Criteria - [ ] On successful deletion, the `delete()` command renders a **Deletion Summary** Rich panel containing: project name, resources unlinked count, and data directory path. - [ ] On successful deletion, the `delete()` command renders an **Index Cleanup** Rich panel containing: text index status, vectors removed count, graph triples status, and storage freed. - [ ] On successful deletion, the `delete()` command renders a **Backups** Rich panel containing: snapshot path and retention days. - [ ] The final status line reads `✓ OK Project deleted` (matching spec format). - [ ] The single-line `✓ Project '<name>' deleted.` output is replaced by the structured panel output. - [ ] All existing tests continue to pass. ## Supporting Information - **File**: `src/cleveragents/cli/commands/project.py` - **Function**: `delete()` - **Spec References**: - `docs/specification.md` lines 3417–3540 (agents project delete section) - Lines 3439–3467: Rich output example showing all required panels - **Related issue**: #1801 (incorrect `--force` flag behavior — separate concern) ## Subtasks - [ ] Implement **Deletion Summary** Rich panel in `delete()` (project name, resources unlinked, data dir) - [ ] Implement **Index Cleanup** Rich panel in `delete()` (text index, vectors removed, graph triples, storage freed) - [ ] Implement **Backups** Rich panel in `delete()` (snapshot path, retention days) - [ ] Replace single-line output with `✓ OK Project deleted` status line - [ ] Tests (Behave): Add BDD scenarios verifying all three panels and status line appear on successful deletion - [ ] Tests (Robot): Add integration test verifying rich output structure for `agents project delete` - [ ] 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 (`fix(cli): render Deletion Summary, Index Cleanup, and Backups panels for agents project delete`), followed by a blank line, then additional lines providing relevant implementation details. - The commit is pushed to the remote on the branch matching the **Branch** in Metadata exactly (`fix/project-delete-rich-output-panels`). - The commit is submitted as a **pull request** to `master`, reviewed, and **merged** before this issue is marked done. - All nox stages pass. - Coverage >= 97%. --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: ca-new-issue-creator
freemo added this to the v3.5.0 milestone 2026-04-03 00:31:38 +00:00
Author
Owner

Issue triaged by project owner:

  • State: Verified
  • Priority: Medium — The agents project delete command works functionally but its rich output does not match the spec-required structured panels (Deletion Summary, Index Cleanup, Backups). This is a spec-alignment issue for output formatting.
  • Milestone: v3.5.0 (Autonomy Hardening — already assigned, confirmed correct since this is part of the Output Rendering Pipeline Epic #936)
  • MoSCoW: Should Have — The spec (docs/specification.md lines 3417–3540) explicitly defines the required output structure. While the command functions correctly, the output format is a spec requirement. This should be addressed for v3.5.0 compliance but is not blocking core autonomy features.
  • Parent Epic: #936 (Output Rendering Pipeline Integration — already linked, confirmed correct)

Clear issue with spec references and expected output format documented. Related to but distinct from #1801 (--force flag behavior).


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

Issue triaged by project owner: - **State**: Verified - **Priority**: Medium — The `agents project delete` command works functionally but its rich output does not match the spec-required structured panels (Deletion Summary, Index Cleanup, Backups). This is a spec-alignment issue for output formatting. - **Milestone**: v3.5.0 (Autonomy Hardening — already assigned, confirmed correct since this is part of the Output Rendering Pipeline Epic #936) - **MoSCoW**: Should Have — The spec (`docs/specification.md` lines 3417–3540) explicitly defines the required output structure. While the command functions correctly, the output format is a spec requirement. This should be addressed for v3.5.0 compliance but is not blocking core autonomy features. - **Parent Epic**: #936 (Output Rendering Pipeline Integration — already linked, confirmed correct) Clear issue with spec references and expected output format documented. Related to but distinct from #1801 (--force flag behavior). --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: ca-project-owner
Author
Owner

Issue triaged by project owner:

  • State: Verified
  • Priority: Medium — rich output panels are part of the spec's CLI output contract but the command functions correctly
  • Milestone: v3.5.0 (Server & Autonomy Infrastructure)
  • MoSCoW: Should Have — the spec requires structured panels for project delete output; missing panels degrade the user experience but don't break functionality
  • Parent Epic: #369 (Large Project Autonomy & Context)

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

Issue triaged by project owner: - **State**: Verified - **Priority**: Medium — rich output panels are part of the spec's CLI output contract but the command functions correctly - **Milestone**: v3.5.0 (Server & Autonomy Infrastructure) - **MoSCoW**: Should Have — the spec requires structured panels for project delete output; missing panels degrade the user experience but don't break functionality - **Parent Epic**: #369 (Large Project Autonomy & Context) --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: ca-project-owner
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.

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