diff --git a/command-restructure/CLI_AUDIT.md b/command-restructure/CLI_AUDIT.md index ed47a28..d29066f 100644 --- a/command-restructure/CLI_AUDIT.md +++ b/command-restructure/CLI_AUDIT.md @@ -37,11 +37,10 @@ | 29 | *help* | — | — | — | (clap built-in) | | | **Hidden/deprecated:** | | | | | | 30 | `list` | — | `` | 14 | deprecated, use issues/mrs | -| 31 | `show` | — | ` ` | 1 | deprecated, use issues/mrs | -| 32 | `auth-test` | — | — | 0 | deprecated, use auth | -| 33 | `sync-status` | — | — | 0 | deprecated, use status | -| 34 | `backup` | — | — | 0 | Stub (not implemented) | -| 35 | `reset` | — | — | 1 | Stub (not implemented) | +| 31 | `auth-test` | — | — | 0 | deprecated, use auth | +| 32 | `sync-status` | — | — | 0 | deprecated, use status | +| 33 | `backup` | — | — | 0 | Stub (not implemented) | +| 34 | `reset` | — | — | 1 | Stub (not implemented) | --- diff --git a/docs/plan-expose-discussion-ids.feedback-3.md b/docs/plan-expose-discussion-ids.feedback-3.md index 1af5e42..879677e 100644 --- a/docs/plan-expose-discussion-ids.feedback-3.md +++ b/docs/plan-expose-discussion-ids.feedback-3.md @@ -1,5 +1,5 @@ 1. **Make `gitlab_note_id` explicit in all note-level payloads without breaking existing consumers** -Rationale: Your Bridge Contract already requires `gitlab_note_id`, but current plan keeps `gitlab_id` only in `notes` list while adding `gitlab_note_id` only in `show`. That forces agents to special-case commands. Add `gitlab_note_id` as an alias field everywhere note-level data appears, while keeping `gitlab_id` for compatibility. +Rationale: Your Bridge Contract already requires `gitlab_note_id`, but current plan keeps `gitlab_id` only in `notes` list while adding `gitlab_note_id` only in detail views. That forces agents to special-case commands. Add `gitlab_note_id` as an alias field everywhere note-level data appears, while keeping `gitlab_id` for compatibility. ```diff @@ Bridge Contract (Cross-Cutting) diff --git a/docs/plan-expose-discussion-ids.md b/docs/plan-expose-discussion-ids.md index 39e6424..782436b 100644 --- a/docs/plan-expose-discussion-ids.md +++ b/docs/plan-expose-discussion-ids.md @@ -43,7 +43,7 @@ construct API calls without a separate project-ID lookup, even after path change **Back-compat rule**: Note payloads in the `notes` list command continue exposing `gitlab_id` for existing consumers, but **MUST also** expose `gitlab_note_id` with the same value. This ensures agents can use a single field name (`gitlab_note_id`) across all commands — `notes`, -`show`, and `discussions --include-notes` — without special-casing by command. +`issues `/`mrs `, and `discussions --include-notes` — without special-casing by command. This contract exists so agents can deterministically construct `glab api` write calls without cross-referencing multiple commands. Each workstream below must satisfy these fields in its diff --git a/plans/work-item-status-graphql.md b/plans/work-item-status-graphql.md index b995b4b..8154ca9 100644 --- a/plans/work-item-status-graphql.md +++ b/plans/work-item-status-graphql.md @@ -107,12 +107,12 @@ Each criterion is independently testable. Implementation is complete when ALL pa ### AC-7: Show Issue Display (E2E) -**Human (`lore show issue 123`):** +**Human (`lore issues 123`):** - [ ] New line after "State": `Status: In progress` (colored by `status_color` hex → nearest terminal color) - [ ] Status line only shown when `status_name IS NOT NULL` - [ ] Category shown in parens when available, lowercased: `Status: In progress (in_progress)` -**Robot (`lore --robot show issue 123`):** +**Robot (`lore --robot issues 123`):** - [ ] JSON includes `status_name`, `status_category`, `status_color`, `status_icon_name`, `status_synced_at` fields - [ ] Fields are `null` (not absent) when status not available - [ ] `status_synced_at` is integer (ms epoch UTC) or `null` — enables freshness checks by consumers