ticket: close yoi ticket cli
This commit is contained in:
parent
4ba1b2fb95
commit
ec525a3d70
|
|
@ -2,12 +2,12 @@
|
||||||
id: 20260605-203006-yoi-ticket-cli-parity
|
id: 20260605-203006-yoi-ticket-cli-parity
|
||||||
slug: yoi-ticket-cli-parity
|
slug: yoi-ticket-cli-parity
|
||||||
title: Yoi ticket CLI parity
|
title: Yoi ticket CLI parity
|
||||||
status: open
|
status: closed
|
||||||
kind: task
|
kind: task
|
||||||
priority: P1
|
priority: P1
|
||||||
labels: [ticket, cli, backend]
|
labels: [ticket, cli, backend]
|
||||||
created_at: 2026-06-05T20:30:06Z
|
created_at: 2026-06-05T20:30:06Z
|
||||||
updated_at: 2026-06-05T20:56:42Z
|
updated_at: 2026-06-05T20:58:21Z
|
||||||
assignee: null
|
assignee: null
|
||||||
legacy_ticket: null
|
legacy_ticket: null
|
||||||
---
|
---
|
||||||
|
|
@ -0,0 +1,45 @@
|
||||||
|
Yoi ticket CLI parity is complete and merged.
|
||||||
|
|
||||||
|
Implementation:
|
||||||
|
|
||||||
|
- `4d5068b feat: add yoi ticket CLI`
|
||||||
|
- merge commit: `4ba1b2f merge: add yoi ticket cli`
|
||||||
|
|
||||||
|
Summary:
|
||||||
|
|
||||||
|
- Added `yoi ticket ...` subcommands to the product `yoi` binary.
|
||||||
|
- Implemented:
|
||||||
|
- `yoi ticket create`
|
||||||
|
- `yoi ticket list`
|
||||||
|
- `yoi ticket show`
|
||||||
|
- `yoi ticket comment`
|
||||||
|
- `yoi ticket review`
|
||||||
|
- `yoi ticket status`
|
||||||
|
- `yoi ticket close`
|
||||||
|
- `yoi ticket doctor`
|
||||||
|
- The CLI uses `crates/ticket` backend APIs directly.
|
||||||
|
- The CLI does not shell out to `tickets.sh`.
|
||||||
|
- Active storage remains unchanged for this ticket: absent `.yoi/ticket.config.toml`, the backend defaults to `<workspace>/work-items`.
|
||||||
|
- `yoi ticket status ... closed` is intentionally rejected with guidance to use `yoi ticket close`, because status-only close would not write `resolution.md`.
|
||||||
|
|
||||||
|
Review:
|
||||||
|
|
||||||
|
- External sibling reviewer approved with no blockers.
|
||||||
|
- Non-blocker follow-ups:
|
||||||
|
- `yoi ticket doctor` currently hides warning-only diagnostics when there are no errors.
|
||||||
|
- `show`, `list`, and doctor diagnostic output are not explicitly bounded.
|
||||||
|
- Body-source error text is generic and can mention `--resolution` for comment/review commands.
|
||||||
|
|
||||||
|
Post-merge validation passed:
|
||||||
|
|
||||||
|
- `cargo test -p yoi ticket`
|
||||||
|
- `cargo test -p ticket`
|
||||||
|
- `cargo check --workspace --all-targets`
|
||||||
|
- `cargo fmt --check`
|
||||||
|
- `git diff --check`
|
||||||
|
- `./tickets.sh doctor`
|
||||||
|
- `cargo build -p yoi`
|
||||||
|
- `target/debug/yoi ticket doctor`
|
||||||
|
- `nix build .#yoi --no-link`
|
||||||
|
|
||||||
|
This clears the CLI prerequisite for the next migration step: `builtin-yoi-local-ticket-backend-config`.
|
||||||
|
|
@ -265,4 +265,57 @@ Reviewer-rerun validation passed:
|
||||||
Yes.
|
Yes.
|
||||||
|
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
<!-- event: close author: hare at: 2026-06-05T20:58:21Z status: closed -->
|
||||||
|
|
||||||
|
## Closed
|
||||||
|
|
||||||
|
Yoi ticket CLI parity is complete and merged.
|
||||||
|
|
||||||
|
Implementation:
|
||||||
|
|
||||||
|
- `4d5068b feat: add yoi ticket CLI`
|
||||||
|
- merge commit: `4ba1b2f merge: add yoi ticket cli`
|
||||||
|
|
||||||
|
Summary:
|
||||||
|
|
||||||
|
- Added `yoi ticket ...` subcommands to the product `yoi` binary.
|
||||||
|
- Implemented:
|
||||||
|
- `yoi ticket create`
|
||||||
|
- `yoi ticket list`
|
||||||
|
- `yoi ticket show`
|
||||||
|
- `yoi ticket comment`
|
||||||
|
- `yoi ticket review`
|
||||||
|
- `yoi ticket status`
|
||||||
|
- `yoi ticket close`
|
||||||
|
- `yoi ticket doctor`
|
||||||
|
- The CLI uses `crates/ticket` backend APIs directly.
|
||||||
|
- The CLI does not shell out to `tickets.sh`.
|
||||||
|
- Active storage remains unchanged for this ticket: absent `.yoi/ticket.config.toml`, the backend defaults to `<workspace>/work-items`.
|
||||||
|
- `yoi ticket status ... closed` is intentionally rejected with guidance to use `yoi ticket close`, because status-only close would not write `resolution.md`.
|
||||||
|
|
||||||
|
Review:
|
||||||
|
|
||||||
|
- External sibling reviewer approved with no blockers.
|
||||||
|
- Non-blocker follow-ups:
|
||||||
|
- `yoi ticket doctor` currently hides warning-only diagnostics when there are no errors.
|
||||||
|
- `show`, `list`, and doctor diagnostic output are not explicitly bounded.
|
||||||
|
- Body-source error text is generic and can mention `--resolution` for comment/review commands.
|
||||||
|
|
||||||
|
Post-merge validation passed:
|
||||||
|
|
||||||
|
- `cargo test -p yoi ticket`
|
||||||
|
- `cargo test -p ticket`
|
||||||
|
- `cargo check --workspace --all-targets`
|
||||||
|
- `cargo fmt --check`
|
||||||
|
- `git diff --check`
|
||||||
|
- `./tickets.sh doctor`
|
||||||
|
- `cargo build -p yoi`
|
||||||
|
- `target/debug/yoi ticket doctor`
|
||||||
|
- `nix build .#yoi --no-link`
|
||||||
|
|
||||||
|
This clears the CLI prerequisite for the next migration step: `builtin-yoi-local-ticket-backend-config`.
|
||||||
|
|
||||||
|
|
||||||
---
|
---
|
||||||
Loading…
Reference in New Issue
Block a user