Design the close process and resolution comments #244

Closed
opened 2026-08-03 14:43:00 +00:00 by jared · 1 comment
Owner

Map: #239

Question

What structure should closing carry? Candidates from #56: a resolution comment recording what was decided/done, evidence links, and explicit out-of-scope; plus how this composes with the existing review-label flow and the never-close rule for recurring issues.

Map: #239 ## Question What structure should closing carry? Candidates from #56: a resolution comment recording what was decided/done, evidence links, and explicit out-of-scope; plus how this composes with the existing review-label flow and the never-close rule for recurring issues.
jared self-assigned this 2026-08-04 10:33:42 +00:00
Author
Owner

Resolution

Decided (grilling session 2026-08-04):

  1. Three-part resolution comment, mandatory for agent closes. Every AI-performed close carries ## Resolution with: (a) what was decided/done; (b) evidence links — commits, PR, test output, ADRs, linked never pasted; (c) explicit out-of-scope/follow-ups, each either linked to a captured issue (/os-backlog:capture) or declared dropped. A follow-up mentioned in prose but not captured is a lifecycle leak — (c) is the enforcement point. Human closes: encouraged, not policed.
  2. Merge approval = sign-off. Pipeline-worked tickets (os-sdlc merge gate) close at merge with the resolution comment; the human's merge approval subsumes the review label. review + stays-open is reserved for work with no merge gate (pure decisions, docs, ops changes).
  3. Corollary (recurring issues): occurrence outcomes are recorded with the same three-part comment shape, without closing — the never-closed-by-AI rule stands.

Composes with prior decisions unchanged: sweep-gate closes decision children (#240); refine may close duplicate/done captures with evidence (#242).

## Resolution Decided (grilling session 2026-08-04): 1. **Three-part resolution comment, mandatory for agent closes.** Every AI-performed close carries `## Resolution` with: (a) what was decided/done; (b) evidence links — commits, PR, test output, ADRs, linked never pasted; (c) explicit out-of-scope/follow-ups, each either linked to a captured issue (`/os-backlog:capture`) or declared dropped. A follow-up mentioned in prose but not captured is a lifecycle leak — (c) is the enforcement point. Human closes: encouraged, not policed. 2. **Merge approval = sign-off.** Pipeline-worked tickets (os-sdlc merge gate) close at merge with the resolution comment; the human's merge approval subsumes the `review` label. `review` + stays-open is reserved for work with no merge gate (pure decisions, docs, ops changes). 3. **Corollary (recurring issues):** occurrence outcomes are recorded with the same three-part comment shape, without closing — the never-closed-by-AI rule stands. Composes with prior decisions unchanged: sweep-gate closes decision children (#240); refine may close duplicate/done captures with evidence (#242).
jared closed this issue 2026-08-04 10:35:02 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
jared/cc-os#244
No description provided.