Move the mutation patch run out of MutationCheck into its caller #1070

New issue

Open

opened 2026-10-17 01:09:43 +00:00 by jared · 0 comments

jared commented

2026-10-17 01:09:43 +00:00

Owner

Problem

Implementation::MutationCheck is a judge, and it runs commands to get its data. The CRC ruling R117 kept it in role model and recorded lens A11 as broken. The controller can run the patch and pass the result in.

Observed

ProjectShell.capture3 runs git apply at plugins/os-sdlc/lib/os_sdlc/implementation/mutation_check.rb:73. ProjectShell.capture2e runs the mutation command at line 79. R117 added an outward inward-lint baseline entry and a deviation line to allow this.

Reproduce

n/a: design ticket; the defect is the structure at the cited lines.

Approach

The caller (Implementation::Acceptance, plugins/os-sdlc/lib/os_sdlc/implementation/acceptance.rb:15) runs the patch and the mutation command, and passes the results to MutationCheck. MutationCheck only decides. The same change removes the R117 inward-lint baseline entry and the R117 deviation line.

Origin

Trigger: CRC os-sdlc run, ruling R117 (docs/crc/os-sdlc/rulings.md), follow-up named by ruling (n) of 2026-10-16
Improvised this session: none
Chain: judge runs a command ← role model kept by R117 so wave 4 changes names only ← principle 5 keeps a structure change out of the rename batch (DESIGN: docs/crc/os-sdlc/rulings.md R117)
Root candidate: none
Where: plugins/os-sdlc/lib/os_sdlc/implementation/mutation_check.rb:73
Session: 66f9640c-45bd-4f9b-a588-e7f8dabeae3d
Transcript: /home/jared/.claude/projects/-home-jared-dev-cc-os/66f9640c-45bd-4f9b-a588-e7f8dabeae3d.jsonl

VERDICT: CREATE

## Problem `Implementation::MutationCheck` is a judge, and it runs commands to get its data. The CRC ruling R117 kept it in role `model` and recorded lens A11 as broken. The controller can run the patch and pass the result in. ## Observed `ProjectShell.capture3` runs `git apply` at `plugins/os-sdlc/lib/os_sdlc/implementation/mutation_check.rb:73`. `ProjectShell.capture2e` runs the mutation command at line 79. R117 added an outward inward-lint baseline entry and a deviation line to allow this. ## Reproduce n/a: design ticket; the defect is the structure at the cited lines. ## Approach The caller (`Implementation::Acceptance`, `plugins/os-sdlc/lib/os_sdlc/implementation/acceptance.rb:15`) runs the patch and the mutation command, and passes the results to `MutationCheck`. `MutationCheck` only decides. The same change removes the R117 inward-lint baseline entry and the R117 deviation line. ## Origin - Trigger: CRC os-sdlc run, ruling R117 (docs/crc/os-sdlc/rulings.md), follow-up named by ruling (n) of 2026-10-16 - Improvised this session: none - Chain: judge runs a command ← role `model` kept by R117 so wave 4 changes names only ← principle 5 keeps a structure change out of the rename batch (DESIGN: docs/crc/os-sdlc/rulings.md R117) - Root candidate: none - Where: plugins/os-sdlc/lib/os_sdlc/implementation/mutation_check.rb:73 - Session: 66f9640c-45bd-4f9b-a588-e7f8dabeae3d - Transcript: /home/jared/.claude/projects/-home-jared-dev-cc-os/66f9640c-45bd-4f9b-a588-e7f8dabeae3d.jsonl VERDICT: CREATE

jared added the

plugin/os-sdlc

filed-by/user

category/code-change

labels

2026-10-17 01:09:43 +00:00

jared referenced this issue

2026-10-17 02:50:09 +00:00

os-sdlc: audit every lib file for ownership and naming (responsibility audit + ADR-0247 names) #835

jared referenced this issue

2026-10-20 14:05:46 +00:00

Invert the model-to-outer-layer edges that wave 4 unmasked #1212