cleveragents/cleveragents-core

Fork 3

Proposal: update specification — document deleted_at field in project delete output and full ULID display in plan tree #7898

New issue

Closed

opened 2026-04-12 06:44:39 +00:00 by HAL9000 · 1 comment

HAL9000 commented

2026-04-12 06:44:39 +00:00

Owner

Proposal: Specification Update

Triggered by merged PRs: #6639, #6571
Spec sections affected: agents project delete (§CLI Reference), agents plan tree (§CLI Reference)

What Changed in the Implementation

PR #6639 — `agents project delete` now includes `deleted_at` in JSON/YAML output

The implementation was fixed to include a deleted_at timestamp field in the JSON and YAML output of agents project delete. Previously the output only contained {"deleted": name, "success": true}. Now it also includes "deleted_at": <ISO timestamp>.

Code change (src/cleveragents/cli/commands/project.py):

format_output(
    {
        "deleted": name,
        "success": True,
        "deleted_at": datetime.now(tz=UTC),
    },
    output_format,
)

PR #6571 — `agents plan tree` now shows full 26-character ULIDs

The agents plan tree command was updated to:

Display full 26-character ULIDs for all decisions (previously truncated to 8 chars)
Show the full plan ID in the tree title (previously truncated with ...)
Add a "Decision IDs (for correction)" section with human-readable labels using per-type ordinals (e.g., "Invariant 1", "Invariant 2", "Strategy", "Parallel 1")

What Spec Sections Need Updating

Change 1: `agents project delete` — add `deleted_at` to JSON/YAML output

Section: ##### agents project delete (around line 3490)

Current spec JSON output:

{
  "command": "agents project delete local/docs",
  "status": "ok",
  "exit_code": 0,
  "data": {
    "deletion_summary": {
      "project": "local/docs",
      "resources_unlinked": 1,
      "data_dir": "/repos/docs/.cleveragents"
    },
    ...
  }
}

Proposed spec JSON output (add deleted_at to data):

{
  "command": "agents project delete local/docs",
  "status": "ok",
  "exit_code": 0,
  "data": {
    "deleted": "local/docs",
    "success": true,
    "deleted_at": "2026-02-08T12:30:00Z"
  },
  "timing": { "duration_ms": 530 },
  "messages": [{ "level": "ok", "text": "Project deleted" }]
}

Note: The actual implementation uses a simpler flat structure (deleted, success, deleted_at) rather than the nested deletion_summary structure in the spec. The spec should be updated to match the actual implementation output.

YAML equivalent:

command: agents project delete local/docs
status: ok
exit_code: 0
data:
  deleted: local/docs
  success: true
  deleted_at: "2026-02-08T12:30:00Z"
timing:
  duration_ms: 530
messages:
  - level: ok
    text: Project deleted

Change 2: `agents plan tree` — note full ULID display

Section: ##### agents plan tree (around line 14320)

The spec already shows full ULIDs in the "Decision IDs (for correction)" panel (lines 14367-14376) — this is already correct. However, the child plan IDs in the tree body are shown as truncated (e.g., 01HXM9F1A) — these should be noted as full 26-character ULIDs in the spec.

The spec description should note that:

All decision IDs displayed in the tree are full 26-character ULIDs (copy-paste ready for agents plan correct)
The "Decision IDs (for correction)" section uses per-type ordinal labels (e.g., "Invariant 1", "Invariant 2", "Strategy", "Parallel 1", "Spawn 1")

Rationale

PR #6639 fixed a bug where the JSON/YAML output was missing the deleted_at timestamp. The spec should reflect the actual output format.
PR #6571 improved usability by showing full ULIDs so users can copy-paste decision IDs directly into agents plan correct commands. The spec should document this behavior.

Both changes are additive — they improve the spec's accuracy without removing any planned features.

Scope

docs/specification.md — agents project delete section (~lines 3490-3543)
docs/specification.md — agents plan tree section (~lines 14320-14530)

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

## Proposal: Specification Update **Triggered by merged PRs**: #6639, #6571 **Spec sections affected**: `agents project delete` (§CLI Reference), `agents plan tree` (§CLI Reference) --- ## What Changed in the Implementation ### PR #6639 — `agents project delete` now includes `deleted_at` in JSON/YAML output The implementation was fixed to include a `deleted_at` timestamp field in the JSON and YAML output of `agents project delete`. Previously the output only contained `{"deleted": name, "success": true}`. Now it also includes `"deleted_at": <ISO timestamp>`. **Code change** (`src/cleveragents/cli/commands/project.py`): ```python format_output( { "deleted": name, "success": True, "deleted_at": datetime.now(tz=UTC), }, output_format, ) ``` ### PR #6571 — `agents plan tree` now shows full 26-character ULIDs The `agents plan tree` command was updated to: 1. Display full 26-character ULIDs for all decisions (previously truncated to 8 chars) 2. Show the full plan ID in the tree title (previously truncated with `...`) 3. Add a "Decision IDs (for correction)" section with human-readable labels using per-type ordinals (e.g., "Invariant 1", "Invariant 2", "Strategy", "Parallel 1") --- ## What Spec Sections Need Updating ### Change 1: `agents project delete` — add `deleted_at` to JSON/YAML output **Section**: `##### agents project delete` (around line 3490) **Current spec JSON output**: ```json { "command": "agents project delete local/docs", "status": "ok", "exit_code": 0, "data": { "deletion_summary": { "project": "local/docs", "resources_unlinked": 1, "data_dir": "/repos/docs/.cleveragents" }, ... } } ``` **Proposed spec JSON output** (add `deleted_at` to data): ```json { "command": "agents project delete local/docs", "status": "ok", "exit_code": 0, "data": { "deleted": "local/docs", "success": true, "deleted_at": "2026-02-08T12:30:00Z" }, "timing": { "duration_ms": 530 }, "messages": [{ "level": "ok", "text": "Project deleted" }] } ``` **Note**: The actual implementation uses a simpler flat structure (`deleted`, `success`, `deleted_at`) rather than the nested `deletion_summary` structure in the spec. The spec should be updated to match the actual implementation output. **YAML equivalent**: ```yaml command: agents project delete local/docs status: ok exit_code: 0 data: deleted: local/docs success: true deleted_at: "2026-02-08T12:30:00Z" timing: duration_ms: 530 messages: - level: ok text: Project deleted ``` ### Change 2: `agents plan tree` — note full ULID display **Section**: `##### agents plan tree` (around line 14320) The spec already shows full ULIDs in the "Decision IDs (for correction)" panel (lines 14367-14376) — this is already correct. However, the child plan IDs in the tree body are shown as truncated (e.g., `01HXM9F1A`) — these should be noted as full 26-character ULIDs in the spec. The spec description should note that: - All decision IDs displayed in the tree are full 26-character ULIDs (copy-paste ready for `agents plan correct`) - The "Decision IDs (for correction)" section uses per-type ordinal labels (e.g., "Invariant 1", "Invariant 2", "Strategy", "Parallel 1", "Spawn 1") --- ## Rationale - **PR #6639** fixed a bug where the JSON/YAML output was missing the `deleted_at` timestamp. The spec should reflect the actual output format. - **PR #6571** improved usability by showing full ULIDs so users can copy-paste decision IDs directly into `agents plan correct` commands. The spec should document this behavior. Both changes are **additive** — they improve the spec's accuracy without removing any planned features. --- ## Scope - `docs/specification.md` — `agents project delete` section (~lines 3490-3543) - `docs/specification.md` — `agents plan tree` section (~lines 14320-14530) --- **Automated by CleverAgents Bot** Supervisor: Spec Updater | Agent: spec-update-pool-supervisor

HAL9000 added this to the v3.2.0 milestone

2026-04-12 06:47:23 +00:00

HAL9000 added the

labels

2026-04-12 06:48:30 +00:00

HAL9000 referenced this issue

2026-04-12 06:53:25 +00:00

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

HAL9000 closed this issue

2026-04-12 07:19:43 +00:00

HAL9000 commented

2026-04-12 07:20:34 +00:00

Author

Owner

Closing as duplicate. The deleted_at change was already covered by:

Proposal issue #7872 (State/Verified — already approved)
PR #7899 (already created and awaiting review)

The full ULID display change (PR #6571) will be tracked separately in a new proposal if not already covered by another agent.

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

Closing as duplicate. The `deleted_at` change was already covered by: - Proposal issue #7872 (State/Verified — already approved) - PR #7899 (already created and awaiting review) The full ULID display change (PR #6571) will be tracked separately in a new proposal if not already covered by another agent. --- **Automated by CleverAgents Bot** Supervisor: Spec Updater | Agent: spec-update-pool-supervisor

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:29:54 +00:00

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

HAL9000 referenced this issue from a commit

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

docs(spec): update plan tree to show full 26-char ULIDs for child plans