Close protocol: enforced three-part resolution, merge-subsumes-review, recurring label #257

New issue

Closed

opened 2026-08-04 11:44:35 +00:00 by jared · 2 comments

jared commented

2026-08-04 11:44:35 +00:00

Owner

Spec: #250

Slice

Close protocol in os-backlog rules/skills (design doc §Close): three-part ## Resolution comment on AI closes, enforced at the os-backlog surface (a close path that refuses without a valid resolution comment); merge approval subsumes review for pipeline-worked tickets (recognized by a PR/branch link in the start/resolution comment); create and document the recurring label as the recurring flavor's mechanical marker — occurrence outcomes commented, never closed.

Acceptance criteria

Given an agent close through os-backlog, when the resolution comment is missing a part (done/evidence/follow-ups), then the close is refused with the missing part named.
Given a follow-up surfaced at close, when the resolution is written, then it is either captured via /os-backlog:capture or explicitly declared dropped.
Given a recurring-labelled issue, when an occurrence completes, then the outcome is commented and the issue stays open.
The recurring label exists on the tracker; artifacts pass os-aidd-lint.

Non-goals

Policing human closes; policing direct-API closes (out of protocol by definition).

Spec: #250 ## Slice Close protocol in os-backlog rules/skills (design doc §Close): three-part `## Resolution` comment on AI closes, enforced at the os-backlog surface (a close path that refuses without a valid resolution comment); merge approval subsumes `review` for pipeline-worked tickets (recognized by a PR/branch link in the start/resolution comment); create and document the `recurring` label as the recurring flavor's mechanical marker — occurrence outcomes commented, never closed. ## Acceptance criteria - Given an agent close through os-backlog, when the resolution comment is missing a part (done/evidence/follow-ups), then the close is refused with the missing part named. - Given a follow-up surfaced at close, when the resolution is written, then it is either captured via `/os-backlog:capture` or explicitly declared dropped. - Given a `recurring`-labelled issue, when an occurrence completes, then the outcome is commented and the issue stays open. - The `recurring` label exists on the tracker; artifacts pass os-aidd-lint. ## Non-goals Policing human closes; policing direct-API closes (out of protocol by definition).

jared added the

ready-for-agent

label

2026-08-04 11:44:35 +00:00

jared commented

2026-08-04 13:50:03 +00:00

Author

Owner

Work started (session 2026-08-04).

jared referenced this issue

2026-08-04 13:55:40 +00:00

Clear pre-existing aidd-lint findings in os-backlog SKILL.md files (hard-wrap, sentence length) #259

jared commented

2026-08-04 13:55:47 +00:00

Author

Owner

Resolution

Done: Close protocol implemented: os-backlog close/occurrence subcommands with enforced three-part ## Resolution (refuses naming missing parts; refuses outright on recurring label), merge-subsumes-review documented in session rules, recurring label created on tracker (id 138). 108 tests green; bin/refresh-plugins run.

Evidence: commit c1727e8; label https://forgejo.swansoncloud.com/jared/cc-os/issues?labels=138

Follow-ups: captured #259 (pre-existing aidd-lint debt in os-backlog SKILL.md files); none dropped

## Resolution **Done:** Close protocol implemented: os-backlog close/occurrence subcommands with enforced three-part ## Resolution (refuses naming missing parts; refuses outright on recurring label), merge-subsumes-review documented in session rules, recurring label created on tracker (id 138). 108 tests green; bin/refresh-plugins run. **Evidence:** commit c1727e8; label https://forgejo.swansoncloud.com/jared/cc-os/issues?labels=138 **Follow-ups:** captured #259 (pre-existing aidd-lint debt in os-backlog SKILL.md files); none dropped

jared closed this issue

2026-08-04 13:55:47 +00:00

jared referenced this issue

2026-08-04 14:42:31 +00:00

Harden the close protocol: idempotent close, Resolution.missing_parts, mechanical merge-subsumes-review gate #261

jared referenced this issue from a commit

2026-08-04 15:57:58 +00:00

os-backlog close protocol: enforced three-part resolution, merge-subsumes-review, recurring label (#257)

jared referenced this issue from a commit

2026-08-04 15:57:58 +00:00

Record #257 close protocol in os-backlog status leaf