Wire non-blocking ticket progress and terminal comments into the os-sdlc runner #517

New issue

Open

opened 2026-08-26 20:34:54 +00:00 by jared · 0 comments

jared commented

2026-08-26 20:34:54 +00:00

Owner

Context

Replaces #436 (shrunken scope after 2026-08-26 code-probe assessment). The escalation redesign (ADR-0152, #513) plus SKILL.md orchestrator instructions now cover the blocked-drive case, but ticket-writing is prose-only: TeaTracker#blocker_comment and #add_waiting_label (plugins/os-sdlc/lib/os_sdlc/runner/tea_tracker.rb:22-37) are dead code, and a true crash (no orchestrator turn) leaves the ticket silent. User direction 2026-08-26: progress history back to the ticket as work proceeds, and comments must never block the pipeline.

Tasks

Decide comment granularity: per agent increment vs per gate/terminal event (user leans per-increment plus a close comment with commit reference and summary)
Wire TeaTracker (or successor) into Implementation.fail!/escalate! and increment landings — best-effort: tracker unreachable or no real ticket (ad-hoc/investigative work) → warn or skip, never raise, never block the drive
Close path posts commit reference + summary on the worked ticket
Pull ticket comments (not just body) at implementation-open — verify current behavior first (#500 shipped body-at-open; confirm whether comments are included) and add --comments if absent
Tests for comment paths including unreachable-tracker and no-ticket fallbacks

Acceptance criteria

A failed or escalated implementation leaves a durable trace on its real ticket when the tracker is reachable
Tracker unavailability or ticketless work never fails or blocks a drive
Implementation-open brief includes existing ticket comments

Origin

Trigger: 2026-08-26 blocking-ticket assessment of #436; code-probe evidence that TeaTracker methods are never called and crash paths leave no ticket trace
Improvised this session: none
Chain: silent ticket on abort/crash ← ticket-writing delegated to orchestrator prose only (skills/implement/SKILL.md:108-113) ← TeaTracker wired nowhere (tea_tracker.rb:22-37) ← DESIGN (ADR-0131 era, predates ADR-0152 escalation redesign)
Root candidate: this ticket

## Context Replaces #436 (shrunken scope after 2026-08-26 code-probe assessment). The escalation redesign (ADR-0152, #513) plus SKILL.md orchestrator instructions now cover the blocked-drive case, but ticket-writing is prose-only: TeaTracker#blocker_comment and #add_waiting_label (plugins/os-sdlc/lib/os_sdlc/runner/tea_tracker.rb:22-37) are dead code, and a true crash (no orchestrator turn) leaves the ticket silent. User direction 2026-08-26: progress history back to the ticket as work proceeds, and comments must never block the pipeline. ## Tasks - [ ] Decide comment granularity: per agent increment vs per gate/terminal event (user leans per-increment plus a close comment with commit reference and summary) - [ ] Wire TeaTracker (or successor) into Implementation.fail!/escalate! and increment landings — best-effort: tracker unreachable or no real ticket (ad-hoc/investigative work) → warn or skip, never raise, never block the drive - [ ] Close path posts commit reference + summary on the worked ticket - [ ] Pull ticket comments (not just body) at implementation-open — verify current behavior first (#500 shipped body-at-open; confirm whether comments are included) and add --comments if absent - [ ] Tests for comment paths including unreachable-tracker and no-ticket fallbacks ## Acceptance criteria - [ ] A failed or escalated implementation leaves a durable trace on its real ticket when the tracker is reachable - [ ] Tracker unavailability or ticketless work never fails or blocks a drive - [ ] Implementation-open brief includes existing ticket comments ## Origin - Trigger: 2026-08-26 blocking-ticket assessment of #436; code-probe evidence that TeaTracker methods are never called and crash paths leave no ticket trace - Improvised this session: none - Chain: silent ticket on abort/crash ← ticket-writing delegated to orchestrator prose only (skills/implement/SKILL.md:108-113) ← TeaTracker wired nowhere (tea_tracker.rb:22-37) ← DESIGN (ADR-0131 era, predates ADR-0152 escalation redesign) - Root candidate: this ticket

jared added the

plugin/os-sdlc

label

2026-08-26 20:34:54 +00:00

jared referenced this issue

2026-08-26 20:35:01 +00:00

Make abort mark the ticket blocked with clear details (fail-loud follow-through) #436

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#517

No description provided.

Rows
Columns