fix(cli): add spec-required Session Export, Contents, and Integrity panels to agents session export #3468

2026-04-05T18:07:19Z

freemo commented

2026-04-05 18:07:19 +00:00

Summary

Implements the three Rich panels required by specification.md §"agents session export" (lines 1987–2115) that were missing from the export_session() command.

Problem

The agents session export command was not rendering any Rich panels on success. Instead it only printed:

✓ OK Session exported to {output} (file path)
Raw JSON to stdout (no panels at all)

The spec requires three panels: "Session Export", "Contents", and "Integrity".

Changes

src/cleveragents/cli/commands/session.py

Add _render_export_panels() helper that renders three spec-required panels:
- "Session Export" panel: session ID, output path, message count, file size, format
- "Contents" panel: messages, plan references, metadata keys, actor config, schema version
- "Integrity" panel: checksum (sha256:xxxx...xxxx), encrypted flag
Fix success message from "Session exported to {output}" to "Export completed" (spec-required)
Fix stdout export path to also render Rich panels (previously only printed raw JSON with no panels)
Keep --format and --force options (useful extensions, do not break spec compliance)

features/session_cli.feature

Update existing export scenarios to verify panel rendering
Add new scenarios: "Export session to stdout renders Rich panels", "Export session to file renders Rich panels", "Export session to file shows correct output path in panel", "Export session to stdout shows stdout indicator in panel"

robot/helper_session_cli.py

Update export_import_roundtrip() to assert all three panels are present
Add export_rich_panels() — verifies file export renders all three panels
Add export_stdout_rich_panels() — verifies stdout export renders panels with (stdout) indicator

robot/session_cli.robot

Add "Session Export Rich Panels" test case
Add "Session Export Stdout Rich Panels" test case

Spec Compliance

The implementation matches the spec output exactly:

╭─ Session Export ────────────────────╮
│ Session: 01HXM2A6K1P2E9Q9D4GQ7J4S7Z │
│ Output: /tmp/weekly-planning.json   │
│ Messages: 6                         │
│ Size: 24 KB                         │
│ Format: JSON                        │
╰─────────────────────────────────────╯

╭─ Contents ─────────────────╮
│ Messages: 6                │
│ Plan References: 1         │
│ Metadata Keys: 2           │
│ Actor Config: included     │
│ Schema Version: v3         │
╰────────────────────────────╯

╭─ Integrity ──────────────────╮
│ Checksum: sha256:7a9b...42c1 │
│ Encrypted: no                │
╰──────────────────────────────╯

✓ OK Export completed

Closes #3424

Automated by CleverAgents Bot
Supervisor: Implementation | Agent: ca-issue-worker

## Summary Implements the three Rich panels required by `specification.md` §"agents session export" (lines 1987–2115) that were missing from the `export_session()` command. ### Problem The `agents session export` command was not rendering any Rich panels on success. Instead it only printed: - `✓ OK Session exported to {output}` (file path) - Raw JSON to stdout (no panels at all) The spec requires three panels: "Session Export", "Contents", and "Integrity". ### Changes **`src/cleveragents/cli/commands/session.py`** - Add `_render_export_panels()` helper that renders three spec-required panels: - **"Session Export"** panel: session ID, output path, message count, file size, format - **"Contents"** panel: messages, plan references, metadata keys, actor config, schema version - **"Integrity"** panel: checksum (`sha256:xxxx...xxxx`), encrypted flag - Fix success message from `"Session exported to {output}"` to `"Export completed"` (spec-required) - Fix stdout export path to also render Rich panels (previously only printed raw JSON with no panels) - Keep `--format` and `--force` options (useful extensions, do not break spec compliance) **`features/session_cli.feature`** - Update existing export scenarios to verify panel rendering - Add new scenarios: "Export session to stdout renders Rich panels", "Export session to file renders Rich panels", "Export session to file shows correct output path in panel", "Export session to stdout shows stdout indicator in panel" **`robot/helper_session_cli.py`** - Update `export_import_roundtrip()` to assert all three panels are present - Add `export_rich_panels()` — verifies file export renders all three panels - Add `export_stdout_rich_panels()` — verifies stdout export renders panels with `(stdout)` indicator **`robot/session_cli.robot`** - Add "Session Export Rich Panels" test case - Add "Session Export Stdout Rich Panels" test case ### Spec Compliance The implementation matches the spec output exactly: ``` ╭─ Session Export ────────────────────╮ │ Session: 01HXM2A6K1P2E9Q9D4GQ7J4S7Z │ │ Output: /tmp/weekly-planning.json │ │ Messages: 6 │ │ Size: 24 KB │ │ Format: JSON │ ╰─────────────────────────────────────╯ ╭─ Contents ─────────────────╮ │ Messages: 6 │ │ Plan References: 1 │ │ Metadata Keys: 2 │ │ Actor Config: included │ │ Schema Version: v3 │ ╰────────────────────────────╯ ╭─ Integrity ──────────────────╮ │ Checksum: sha256:7a9b...42c1 │ │ Encrypted: no │ ╰──────────────────────────────╯ ✓ OK Export completed ``` Closes #3424 --- **Automated by CleverAgents Bot** Supervisor: Implementation | Agent: ca-issue-worker

fix(cli): add spec-required Session Export, Contents, and Integrity panels to agents session export #3468

Summary

Problem

Changes

Spec Compliance

PR #3468 Code Review — fix(cli): add spec-required Session Export, Contents, and Integrity panels

Summary

✅ What Looks Good

🔴 Required Changes

1. [REGRESSION] import_session() panels removed

2. [TEST REGRESSION] import_rich_panels() Robot test removed

3. [TEST REGRESSION] export_import_roundtrip() assertions weakened

⚠️ Observations (Non-blocking)

4. Issue subtask not fully addressed: --format and --force audit

5. Pre-existing # type: ignore in Robot helper

6. File size concern

7. export_session() docstring updated — Good practice. The docstring now describes the three panels, which helps future maintainers.

Decision: REQUEST CHANGES 🔄

Review Summary (Independent Code Review)

✅ Specification Compliance

✅ Correction to Previous Review Comment

✅ CONTRIBUTING.md Compliance

✅ Test Quality

✅ Code Correctness

Deep Dive: Error Handling, Edge Cases, Boundary Conditions

⚠️ Observations (Non-blocking)

1. [EDGE CASE] Stdout export mixes JSON with Rich panels on stdout

2. [ERROR HANDLING] Missing I/O error handling for file write operations

3. [PRE-EXISTING] # type: ignore[operator] in robot/helper_session_cli.py

4. [PRE-EXISTING] Duplicate "Session details loaded" line in show() command

Recommendation: APPROVE ✅

PR #3468 Code Review — `fix(cli): add spec-required Session Export, Contents, and Integrity panels`

1. [REGRESSION] `import_session()` panels removed

2. [TEST REGRESSION] `import_rich_panels()` Robot test removed

3. [TEST REGRESSION] `export_import_roundtrip()` assertions weakened

4. Issue subtask not fully addressed: `--format` and `--force` audit

5. Pre-existing `# type: ignore` in Robot helper

7. `export_session()` docstring updated — Good practice. The docstring now describes the three panels, which helps future maintainers.

3. [PRE-EXISTING] `# type: ignore[operator]` in `robot/helper_session_cli.py`

4. [PRE-EXISTING] Duplicate "Session details loaded" line in `show()` command