cleveragents/cleveragents-core

Fork 3

Proposal: update specification — document full ULID display in agents plan tree output #7911

New issue

Closed

opened 2026-04-12 07:24:32 +00:00 by HAL9000 · 1 comment

HAL9000 commented

2026-04-12 07:24:32 +00:00

Owner

Proposal: Specification Update

Triggered by merged PR: #6571 — fix(cli): display full ULIDs in plan tree output for CLI usability (merged 2026-04-10)
Spec section affected: agents plan tree (§CLI Reference, ~line 14320)

What Changed in the Implementation

PR #6571 updated agents plan tree to:

Display full 26-character ULIDs for all decisions in the tree (previously truncated to 8 characters like 01HXM9A0)
Show the full plan ID in the tree title (previously shown as Plan 01HXM8C2...)
Add a "Decision IDs (for correction)" section with human-readable per-type ordinal labels:
- Root: <full-ULID>
- Invariant 1: <full-ULID>, Invariant 2: <full-ULID>, ...
- Strategy: <full-ULID>
- Parallel 1: <full-ULID>, Parallel 2: <full-ULID>, ...
- Spawn 1: <full-ULID>, Spawn 2: <full-ULID>, ...

Code changes (src/cleveragents/cli/commands/plan.py):

Table column ID width changed from max_width=8 to max_width=26
d.decision_id[:8] → d.decision_id (full ULID in table rows)
Plan {plan_id[:8]}... → Plan {plan_id} (full plan ID in tree title)
Added _get_decision_label() helper for per-type ordinal labels
Added "Decision IDs (for correction)" panel in both rich and plain formats

What Spec Section Needs Updating

Section: ##### agents plan tree (around line 14320 of docs/specification.md)

The spec already shows full ULIDs in the "Decision IDs (for correction)" panel — this is already correct. However, the Arguments section should note that all displayed IDs are full 26-character ULIDs, and the description should mention the "Decision IDs (for correction)" section.

Proposed Addition to Arguments/Description

Add a note to the Purpose or Arguments section:

**Note**: All decision IDs displayed in the tree output are full 26-character ULIDs,
copy-paste ready for use in `agents plan correct <DECISION_ID>`. The "Decision IDs
(for correction)" section at the bottom of the output lists all decisions with
human-readable per-type ordinal labels (e.g., "Invariant 1", "Strategy", "Parallel 1").

Rationale

The implementation is better than the spec — showing full ULIDs eliminates the usability gap where users had to manually reconstruct IDs for agents plan correct. The spec should document this behavior so users know they can copy IDs directly from the tree output.

This is an additive change — no existing spec content needs to be removed.

Scope

docs/specification.md — agents plan tree section (~lines 14320-14530)
Change type: Minor — additive note about full ULID display behavior

Automated by CleverAgents Bot
Supervisor: Spec Updater | Agent: spec-update-pool-supervisor

## Proposal: Specification Update **Triggered by merged PR**: #6571 — `fix(cli): display full ULIDs in plan tree output for CLI usability` (merged 2026-04-10) **Spec section affected**: `agents plan tree` (§CLI Reference, ~line 14320) --- ## What Changed in the Implementation PR #6571 updated `agents plan tree` to: 1. **Display full 26-character ULIDs** for all decisions in the tree (previously truncated to 8 characters like `01HXM9A0`) 2. **Show the full plan ID** in the tree title (previously shown as `Plan 01HXM8C2...`) 3. **Add a "Decision IDs (for correction)" section** with human-readable per-type ordinal labels: - `Root: <full-ULID>` - `Invariant 1: <full-ULID>`, `Invariant 2: <full-ULID>`, ... - `Strategy: <full-ULID>` - `Parallel 1: <full-ULID>`, `Parallel 2: <full-ULID>`, ... - `Spawn 1: <full-ULID>`, `Spawn 2: <full-ULID>`, ... **Code changes** (`src/cleveragents/cli/commands/plan.py`): - Table column `ID` width changed from `max_width=8` to `max_width=26` - `d.decision_id[:8]` → `d.decision_id` (full ULID in table rows) - `Plan {plan_id[:8]}...` → `Plan {plan_id}` (full plan ID in tree title) - Added `_get_decision_label()` helper for per-type ordinal labels - Added "Decision IDs (for correction)" panel in both rich and plain formats --- ## What Spec Section Needs Updating **Section**: `##### agents plan tree` (around line 14320 of `docs/specification.md`) The spec already shows full ULIDs in the "Decision IDs (for correction)" panel — this is already correct. However, the **Arguments** section should note that all displayed IDs are full 26-character ULIDs, and the description should mention the "Decision IDs (for correction)" section. ### Proposed Addition to Arguments/Description Add a note to the **Purpose** or **Arguments** section: ``` **Note**: All decision IDs displayed in the tree output are full 26-character ULIDs, copy-paste ready for use in `agents plan correct <DECISION_ID>`. The "Decision IDs (for correction)" section at the bottom of the output lists all decisions with human-readable per-type ordinal labels (e.g., "Invariant 1", "Strategy", "Parallel 1"). ``` --- ## Rationale The implementation is **better than the spec** — showing full ULIDs eliminates the usability gap where users had to manually reconstruct IDs for `agents plan correct`. The spec should document this behavior so users know they can copy IDs directly from the tree output. This is an additive change — no existing spec content needs to be removed. --- ## Scope - `docs/specification.md` — `agents plan tree` section (~lines 14320-14530) - Change type: Minor — additive note about full ULID display behavior --- **Automated by CleverAgents Bot** Supervisor: Spec Updater | Agent: spec-update-pool-supervisor

HAL9000 added the

labels

2026-04-12 07:25:29 +00:00

HAL9000 added this to the v3.2.0 milestone

2026-04-12 07:25:30 +00:00

HAL9000 referenced this issue

2026-04-12 07:26:42 +00:00

[AUTO-SPEC] Status: Specification Update (Cycle 1) #7892

HAL9000 referenced this issue

2026-04-12 07:27:32 +00:00

[AUTO-SPEC] Status: Specification Update (Cycle 1) #7892

HAL9000 referenced this issue

2026-04-12 07:46:23 +00:00

[AUTO-EPIC] Epic Planning Update (Cycle 27) #7923

HAL9000 commented

2026-04-12 07:46:37 +00:00

Author

Owner

Closing as superseded. The architecture-pool-supervisor has already incorporated the full ULID display change into PR #7899 (commit cc12e991). The PR now covers both:

agents project delete — deleted_at field (from PR #6639)
agents plan tree — full 26-char ULID display (from PR #6571)

PR #7899 is open and awaiting human review.

Automated by CleverAgents Bot
Supervisor: Spec Updater | Agent: spec-update-pool-supervisor

Closing as superseded. The architecture-pool-supervisor has already incorporated the full ULID display change into PR #7899 (commit `cc12e991`). The PR now covers both: 1. `agents project delete` — `deleted_at` field (from PR #6639) 2. `agents plan tree` — full 26-char ULID display (from PR #6571) PR #7899 is open and awaiting human review. --- **Automated by CleverAgents Bot** Supervisor: Spec Updater | Agent: spec-update-pool-supervisor

HAL9000 closed this issue

2026-04-12 07:46:38 +00:00

HAL9000 referenced this issue

2026-04-12 07:48:15 +00:00

[AUTO-SPEC] Status: Specification Update (Cycle 1) #7892

HAL9000 referenced this issue

2026-04-12 07:48:23 +00:00

[AUTO-OWNR] Project Owner Report (Cycle 1) #7896

HAL9000 referenced this issue

2026-04-12 07:48:54 +00:00

[AUTO-SPEC] Status: Specification Update (Cycle 1) #7892