UAT: `agents session import` CLI output missing spec-required Validation and Merge panels, and --format option #1349

New issue

Open

opened 2026-04-02 16:57:09 +00:00 by freemo · 0 comments

freemo commented

2026-04-02 16:57:09 +00:00

Owner

Bug Report

Feature Area: Session export/import — CLI import output format

What Was Tested

The import_session CLI command in src/cleveragents/cli/commands/session.py (lines 488–536) was compared against the specification's documented output format.

Expected Behavior (from spec)

The spec defines three Rich output panels for agents session import:

Panel 1 — Session Import:

╭─ Session Import ────────────────────────╮
│ Input: /tmp/weekly-planning.json        │
│ Session ID: 01HXM3D3B2W4CQYQ3P4ZB8A5T1  │
│ Messages: 6                             │
│ Schema: v3                              │
╰─────────────────────────────────────────╯

Panel 2 — Validation:

╭─ Validation ────────────╮
│ Checksum: verified      │
│ Schema: compatible      │
│ Actor Ref: resolved     │
╰─────────────────────────╯

Panel 3 — Merge:

╭─ Merge ──────────────╮
│ Existing: none       │
│ Strategy: create new │
╰──────────────────────╯

Actual Behavior

The current implementation (session.py lines 514–525) outputs only:

╭─ Session Imported ─────────────────────────╮
│ Session ID: <id>                           │
│ Actor: (none)                              │
│ Messages: <count>                          │
│ Namespace: local                           │
╰────────────────────────────────────────────╯
✓ OK Session imported

Missing Elements

Input file path not shown in the Session Import panel
Schema version not shown in the Session Import panel
Validation panel entirely absent — no checksum verification status, schema compatibility, or actor ref resolution shown to user
Merge panel entirely absent — no existing session conflict info or merge strategy shown
--format option missing — the spec shows a JSON output format for the import command; the current implementation has no --format flag

Code Location

src/cleveragents/cli/commands/session.py lines 488–536 (import_session function)

Severity

Medium — The import command is functionally correct (data is imported with checksum validation) but the user-facing output omits important feedback about the validation and merge steps that the spec requires. The missing --format option also prevents machine-readable output of the import operation result.

## Bug Report **Feature Area:** Session export/import — CLI import output format ### What Was Tested The `import_session` CLI command in `src/cleveragents/cli/commands/session.py` (lines 488–536) was compared against the specification's documented output format. ### Expected Behavior (from spec) The spec defines three Rich output panels for `agents session import`: **Panel 1 — Session Import:** ``` ╭─ Session Import ────────────────────────╮ │ Input: /tmp/weekly-planning.json │ │ Session ID: 01HXM3D3B2W4CQYQ3P4ZB8A5T1 │ │ Messages: 6 │ │ Schema: v3 │ ╰─────────────────────────────────────────╯ ``` **Panel 2 — Validation:** ``` ╭─ Validation ────────────╮ │ Checksum: verified │ │ Schema: compatible │ │ Actor Ref: resolved │ ╰─────────────────────────╯ ``` **Panel 3 — Merge:** ``` ╭─ Merge ──────────────╮ │ Existing: none │ │ Strategy: create new │ ╰──────────────────────╯ ``` ### Actual Behavior The current implementation (`session.py` lines 514–525) outputs only: ``` ╭─ Session Imported ─────────────────────────╮ │ Session ID: <id> │ │ Actor: (none) │ │ Messages: <count> │ │ Namespace: local │ ╰────────────────────────────────────────────╯ ✓ OK Session imported ``` ### Missing Elements 1. **Input file path** not shown in the Session Import panel 2. **Schema version** not shown in the Session Import panel 3. **Validation panel** entirely absent — no checksum verification status, schema compatibility, or actor ref resolution shown to user 4. **Merge panel** entirely absent — no existing session conflict info or merge strategy shown 5. **`--format` option** missing — the spec shows a JSON output format for the import command; the current implementation has no `--format` flag ### Code Location `src/cleveragents/cli/commands/session.py` lines 488–536 (`import_session` function) ### Severity **Medium** — The import command is functionally correct (data is imported with checksum validation) but the user-facing output omits important feedback about the validation and merge steps that the spec requires. The missing `--format` option also prevents machine-readable output of the import operation result.

Rows
Columns

UAT: agents session import CLI output missing spec-required Validation and Merge panels, and --format option #1349