Runner commit leaves ticket files uncommitted when the programmer handoff items omit them #1116

New issue

Closed

opened 2026-10-18 17:23:18 +00:00 by jared · 1 comment

jared commented

2026-10-18 17:23:18 +00:00

Owner

Problem

The runner commit can leave ticket files uncommitted. The programmer handoff items did not name every changed file, so Implementation::Commit staged only the named files.

Observed

Implementation 67 of #1107 passed Required work checks R2, R3, and R7. The programmer created plugins/os-sdlc/agents/structure-changer.md and edited plugins/os-sdlc/skills/implement/SKILL.md. Both files are named by those checks. The handoff items listed only the map and the two test files. Implementation::Commit staged those paths and reported the two other files as dirty_paths. The orchestrator committed them by hand.

Reproduce

n/a: observed in implementation 67 of #1107; no standalone command.

Approach

Choose one fix:

Implementation::Commit stages every path that a passing Required work check named.
The programmer gate fails when a changed path under the target is absent from the handoff items.

Add a test for the chosen fix.

Origin

Trigger: implementation 67 of #1107
Improvised this session: the orchestrator committed the two files by hand
Chain: uncommitted ticket files <- handoff items omit two paths <- Commit stages only handoff items. Stop: DESIGN (Implementation::Commit)
Root candidate: none
Where: Implementation::Commit in plugins/os-sdlc
Session: 26783407-6911-48d5-a818-55fbfbc70108
Transcript: /home/jared/.claude/projects/-home-jared-dev-cc-os/26783407-6911-48d5-a818-55fbfbc70108.jsonl

## Problem The runner commit can leave ticket files uncommitted. The programmer handoff items did not name every changed file, so `Implementation::Commit` staged only the named files. ## Observed Implementation 67 of #1107 passed Required work checks R2, R3, and R7. The programmer created `plugins/os-sdlc/agents/structure-changer.md` and edited `plugins/os-sdlc/skills/implement/SKILL.md`. Both files are named by those checks. The handoff items listed only the map and the two test files. `Implementation::Commit` staged those paths and reported the two other files as `dirty_paths`. The orchestrator committed them by hand. ## Reproduce n/a: observed in implementation 67 of #1107; no standalone command. ## Approach Choose one fix: 1. `Implementation::Commit` stages every path that a passing Required work check named. 2. The programmer gate fails when a changed path under the target is absent from the handoff items. Add a test for the chosen fix. ## Origin - Trigger: implementation 67 of #1107 - Improvised this session: the orchestrator committed the two files by hand - Chain: uncommitted ticket files <- handoff items omit two paths <- Commit stages only handoff items. Stop: DESIGN (Implementation::Commit) - Root candidate: none - Where: Implementation::Commit in plugins/os-sdlc - Session: 26783407-6911-48d5-a818-55fbfbc70108 - Transcript: /home/jared/.claude/projects/-home-jared-dev-cc-os/26783407-6911-48d5-a818-55fbfbc70108.jsonl

jared added the

plugin/os-sdlc

filed-by/user

category/code-change

labels

2026-10-18 17:23:18 +00:00

jared referenced this issue

2026-10-18 18:58:38 +00:00

Runner commit omits production files that no handoff lists #955

jared commented

2026-10-18 18:59:15 +00:00

Author