Pipeline cannot delete files: programmer role and Runner commit omit deletions #916

New issue

Open

opened 2026-10-02 18:07:48 +00:00 by jared · 3 comments

jared commented

2026-10-02 18:07:48 +00:00

Owner

Problem

A ticket that requires a file deletion or move cannot pass its gate, because the programmer role has no way to delete or move a file. The Runner commit step also leaves staged deletions out of the commit.

Observed

Implementation 18 (#913): a test required three .md files to be gone. The programmer role could not delete them. Three programmer-repair dispatches each failed in about 15 s. The run escalated with bound_exhausted:programmer-repair/fail. The orchestrator removed the files with git rm and rerouted. The Runner commit abd8e57c then left the three deletions staged and uncommitted. The payload listed them under dirty_paths. The orchestrator committed them by hand as d9257a3a.

Implementation 11 (#908): the Runner commit 127395d1 added the moved files. It left four file deletions staged and uncommitted. The completion payload reported dirty_paths: []. The orchestrator committed the deletions by hand as 52cd78a5.

The two cases differ in the dirty_paths report: it was wrong (empty) in implementation 11 and correct in implementation 18.

Ticket #906 covers a different cause: uppercase CHANGE: tags that the commit item parser does not read, so refactorer-only new files are not staged. It does not cover deletions.

Reproduce

n/a: observed in pipeline runs (implementation 11 and implementation 18); no standalone command.

Origin

Trigger: /os-sdlc:implement 913 (implementation 18) and /os-sdlc:implement 908 (implementation 11), 2026-10-02
Improvised this session: manual git rm and rerouting (implementation 18); manual commits 52cd78a5 and d9257a3a
Chain: staged deletions absent from the Runner commit ← the commit takes its paths from handoff items (ADR-0171) ← DESIGN: ADR-0171; programmer cannot delete files ← DESIGN: programmer role toolset (Read, Write, Glob, Grep, Bash per its agent definition)
Root candidate: none
Where: plugins/os-sdlc/lib/os_sdlc/command/implementation/commit.rb and plugins/os-sdlc/agents/programmer.md
Session: 9ee7805f-a4e4-470e-af50-5020e5c524a4
Transcript: /home/jared/.claude/projects/-home-jared-dev-cc-os/9ee7805f-a4e4-470e-af50-5020e5c524a4.jsonl

## Problem A ticket that requires a file deletion or move cannot pass its gate, because the programmer role has no way to delete or move a file. The Runner commit step also leaves staged deletions out of the commit. ## Observed Implementation 18 (#913): a test required three `.md` files to be gone. The programmer role could not delete them. Three programmer-repair dispatches each failed in about 15 s. The run escalated with `bound_exhausted:programmer-repair/fail`. The orchestrator removed the files with `git rm` and rerouted. The Runner commit abd8e57c then left the three deletions staged and uncommitted. The payload listed them under `dirty_paths`. The orchestrator committed them by hand as d9257a3a. Implementation 11 (#908): the Runner commit 127395d1 added the moved files. It left four file deletions staged and uncommitted. The completion payload reported `dirty_paths: []`. The orchestrator committed the deletions by hand as 52cd78a5. The two cases differ in the `dirty_paths` report: it was wrong (empty) in implementation 11 and correct in implementation 18. Ticket #906 covers a different cause: uppercase `CHANGE:` tags that the commit item parser does not read, so refactorer-only new files are not staged. It does not cover deletions. ## Reproduce n/a: observed in pipeline runs (implementation 11 and implementation 18); no standalone command. ## Origin - Trigger: /os-sdlc:implement 913 (implementation 18) and /os-sdlc:implement 908 (implementation 11), 2026-10-02 - Improvised this session: manual `git rm` and rerouting (implementation 18); manual commits 52cd78a5 and d9257a3a - Chain: staged deletions absent from the Runner commit ← the commit takes its paths from handoff items (ADR-0171) ← DESIGN: ADR-0171; programmer cannot delete files ← DESIGN: programmer role toolset (Read, Write, Glob, Grep, Bash per its agent definition) - Root candidate: none - Where: plugins/os-sdlc/lib/os_sdlc/command/implementation/commit.rb and plugins/os-sdlc/agents/programmer.md - Session: 9ee7805f-a4e4-470e-af50-5020e5c524a4 - Transcript: /home/jared/.claude/projects/-home-jared-dev-cc-os/9ee7805f-a4e4-470e-af50-5020e5c524a4.jsonl

jared added the

plugin/os-sdlc

filed-by/user

labels

2026-10-02 18:07:48 +00:00

jared added the

category/umbrella

label

2026-10-10 13:31:51 +00:00

jared commented

2026-10-19 12:33:32 +00:00

Author

Owner

Occurrence: Runner implementation_complete commit omits the deleted side of a moved file
Seen: plugins/os-sdlc Runner commit at implementation_complete (Commit#staged_paths, ADR-0171)
Run: implementation 75 (#1153), session 22b441ac-a301-4637-8de9-a9e0f0970fae
Adds: a third occurrence; cops/inward_dependency/{ledger,layer_map}.rb moved to cops/layer_ledger/, commit.paths listed only the new files, the two staged deletions were committed by hand (d3e0d4d3).

Occurrence: Runner implementation_complete commit omits the deleted side of a moved file Seen: plugins/os-sdlc Runner commit at implementation_complete (Commit#staged_paths, ADR-0171) Run: implementation 75 (#1153), session 22b441ac-a301-4637-8de9-a9e0f0970fae Adds: a third occurrence; cops/inward_dependency/{ledger,layer_map}.rb moved to cops/layer_ledger/, commit.paths listed only the new files, the two staged deletions were committed by hand (d3e0d4d3).

jared referenced this issue

2026-10-19 12:34:30 +00:00

Split Sdlc/Structural/InwardDependency into one cop per offense kind #1153

jared referenced this issue

2026-10-19 20:20:38 +00:00

Cut Lint -> Gate: move Gate::ExitClassification into Lint #1165

jared commented

2026-10-19 20:20:38 +00:00

Author

Owner

Occurrence: runner commit leaves a staged deletion out of the commit
Seen: implementation 79 (#1165), commit f8a06e1d left plugins/os-sdlc/lib/os_sdlc/gate/exit_classification.rb staged as D
Run: session 2e167079-f404-4139-9cfa-6c0c937ce9f0, os-sdlc CRC run 3
Adds: a second occurrence on the structure-change map; the deletion was committed by hand in c67ca2b3

Occurrence: runner commit leaves a staged deletion out of the commit Seen: implementation 79 (#1165), commit f8a06e1d left plugins/os-sdlc/lib/os_sdlc/gate/exit_classification.rb staged as D Run: session 2e167079-f404-4139-9cfa-6c0c937ce9f0, os-sdlc CRC run 3 Adds: a second occurrence on the structure-change map; the deletion was committed by hand in c67ca2b3

jared commented

2026-10-20 01:40:32 +00:00

Author

Owner

Occurrence: Runner commit leaves git mv deletions staged and uncommitted
Seen: implementation 98 (#1147), Runner commit 052849c5; dirty_paths listed plugins/os-sdlc/lib/os_sdlc/argv_option.rb and tests/argv_option_test.rb (staged D); orchestrator committed them in 0d04165e
Run: session 2e167079-f404-4139-9cfa-6c0c937ce9f0, implementation 98, dispatch 790
Adds: the defect recurs for a structure-change move made with git mv (old path staged as deleted), not only for plain deletions; also seen on #1177 (71051dff)

Occurrence: Runner commit leaves git mv deletions staged and uncommitted Seen: implementation 98 (#1147), Runner commit 052849c5; dirty_paths listed plugins/os-sdlc/lib/os_sdlc/argv_option.rb and tests/argv_option_test.rb (staged D); orchestrator committed them in 0d04165e Run: session 2e167079-f404-4139-9cfa-6c0c937ce9f0, implementation 98, dispatch 790 Adds: the defect recurs for a structure-change move made with git mv (old path staged as deleted), not only for plain deletions; also seen on #1177 (71051dff)

jared referenced this issue

2026-10-25 20:06:32 +00:00

Add os-sdlc routes for deletion tickets and search-check criteria over pinned tests #1339