ticket: close ticket lifecycle feature
This commit is contained in:
parent
c0fb243760
commit
abbc8e16f0
|
|
@ -2,13 +2,13 @@
|
||||||
id: 20260607-033536-ticket-lifecycle-pod-feature
|
id: 20260607-033536-ticket-lifecycle-pod-feature
|
||||||
slug: ticket-lifecycle-pod-feature
|
slug: ticket-lifecycle-pod-feature
|
||||||
title: Ticket lifecycle pod feature
|
title: Ticket lifecycle pod feature
|
||||||
status: open
|
status: closed
|
||||||
kind: task
|
kind: task
|
||||||
priority: P1
|
priority: P1
|
||||||
labels: [ticket, pod-feature, tools, orchestration, workflow]
|
labels: [ticket, pod-feature, tools, orchestration, workflow]
|
||||||
workflow_state: ready
|
workflow_state: done
|
||||||
created_at: 2026-06-07T03:35:36Z
|
created_at: 2026-06-07T03:35:36Z
|
||||||
updated_at: 2026-06-07T04:02:48Z
|
updated_at: 2026-06-07T04:03:33Z
|
||||||
assignee: null
|
assignee: null
|
||||||
legacy_ticket: null
|
legacy_ticket: null
|
||||||
---
|
---
|
||||||
|
|
@ -0,0 +1,26 @@
|
||||||
|
Implemented, reviewed, merged, and validated.
|
||||||
|
|
||||||
|
Summary:
|
||||||
|
- Refined the existing Ticket-domain `builtin:ticket` feature rather than introducing an Orchestrator-specific feature.
|
||||||
|
- Added `TicketFeatureAccess::{ReadOnly, Lifecycle}`.
|
||||||
|
- Added explicit Ticket tool partitions:
|
||||||
|
- read-only/status tools: `TicketList`, `TicketShow`, `TicketDoctor`;
|
||||||
|
- mutating/lifecycle tools: create/comment/review/intake-ready/workflow-state/status/close.
|
||||||
|
- Feature descriptors and installation now expose only the tools allowed by the configured access level.
|
||||||
|
- Preserved the existing full lifecycle/default behavior for existing call sites.
|
||||||
|
- Updated `TicketWorkflowState` tool description with the queued acceptance sequencing contract.
|
||||||
|
- Preserved typed backend/tool transition enforcement.
|
||||||
|
|
||||||
|
Implementation:
|
||||||
|
- Child commit: `3d662bc pod: split ticket feature access levels`
|
||||||
|
- Merge commit: `merge: ticket lifecycle feature`
|
||||||
|
|
||||||
|
Review:
|
||||||
|
- External reviewer `ticket-lifecycle-feature-reviewer-20260607` approved with no blockers.
|
||||||
|
|
||||||
|
Validation after merge:
|
||||||
|
- `cargo test -p pod ticket --lib`
|
||||||
|
- `cargo test -p ticket tool --lib`
|
||||||
|
- `cargo fmt --check`
|
||||||
|
- `git diff --check`
|
||||||
|
- `cargo check --workspace --all-targets`
|
||||||
|
|
@ -164,3 +164,45 @@ Merge readiness:
|
||||||
- No requested changes.
|
- No requested changes.
|
||||||
|
|
||||||
---
|
---
|
||||||
|
|
||||||
|
<!-- event: state_changed author: hare at: 2026-06-07T04:03:33Z from: ready to: done reason: closed field: workflow_state -->
|
||||||
|
|
||||||
|
## State changed
|
||||||
|
|
||||||
|
Ticket closed; workflow_state set to done.
|
||||||
|
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
<!-- event: close author: hare at: 2026-06-07T04:03:33Z status: closed -->
|
||||||
|
|
||||||
|
## Closed
|
||||||
|
|
||||||
|
Implemented, reviewed, merged, and validated.
|
||||||
|
|
||||||
|
Summary:
|
||||||
|
- Refined the existing Ticket-domain `builtin:ticket` feature rather than introducing an Orchestrator-specific feature.
|
||||||
|
- Added `TicketFeatureAccess::{ReadOnly, Lifecycle}`.
|
||||||
|
- Added explicit Ticket tool partitions:
|
||||||
|
- read-only/status tools: `TicketList`, `TicketShow`, `TicketDoctor`;
|
||||||
|
- mutating/lifecycle tools: create/comment/review/intake-ready/workflow-state/status/close.
|
||||||
|
- Feature descriptors and installation now expose only the tools allowed by the configured access level.
|
||||||
|
- Preserved the existing full lifecycle/default behavior for existing call sites.
|
||||||
|
- Updated `TicketWorkflowState` tool description with the queued acceptance sequencing contract.
|
||||||
|
- Preserved typed backend/tool transition enforcement.
|
||||||
|
|
||||||
|
Implementation:
|
||||||
|
- Child commit: `3d662bc pod: split ticket feature access levels`
|
||||||
|
- Merge commit: `merge: ticket lifecycle feature`
|
||||||
|
|
||||||
|
Review:
|
||||||
|
- External reviewer `ticket-lifecycle-feature-reviewer-20260607` approved with no blockers.
|
||||||
|
|
||||||
|
Validation after merge:
|
||||||
|
- `cargo test -p pod ticket --lib`
|
||||||
|
- `cargo test -p ticket tool --lib`
|
||||||
|
- `cargo fmt --check`
|
||||||
|
- `git diff --check`
|
||||||
|
- `cargo check --workspace --all-targets`
|
||||||
|
|
||||||
|
---
|
||||||
Loading…
Reference in New Issue
Block a user