Fix: _strip_ansi regex covers only a subset of CSI escape sequences (#52) (#55)

* Fix: _strip_ansi regex covers only a subset of CSI escape sequences (#52)

The test helper only stripped a narrow set of CSI final bytes, which made the suite fragile if Rich emitted other valid escape sequences.

Changes:
- Broadened `_strip_ansi` to strip the full CSI range
- Added a regression test for cursor-movement CSI sequences

Fixes #52

* Archive investigation for issue #52

---------

Co-authored-by: OpenCode <opencode@users.noreply.github.com>

Author: tbrandenburg

.agents/issues/completed/issue-52.md

@@ -0,0 +1,26 @@
+# Issue 52 Archive
+
+**Issue**: #52 - `test: _strip_ansi regex covers only a subset of CSI escape sequences`
+
+**Type**: BUG
+
+**Investigation Summary**
+
+The `_strip_ansi` helper in `tests/test_workflow_to_harness.py` used a narrow regex that only stripped a small set of CSI escape-sequence final bytes. That made the test suite fragile if Rich emitted other valid ANSI sequences.
+
+**Implementation Plan**
+
+1. Broaden `_strip_ansi` in `tests/test_workflow_to_harness.py` to match the full CSI range.
+2. Add a regression test for a cursor-movement CSI sequence.
+3. Keep the existing help-output contract assertions unchanged.
+
+**Validation**
+
+```bash
+make qa
+uv run --locked pytest tests/test_workflow_to_harness.py -k strip_ansi
+```
+
+**Result**
+
+Implemented on branch `fix/issue-52-strip-ansi-regex` and verified with `make qa` and the targeted pytest invocation.

tests/test_workflow_to_harness.py

@@ -38,7 +38,13 @@ def _strip_ansi(text: str) -> str:
     causing Rich to emit ANSI codes even in non-TTY subprocesses.  Stripping
     sequences here keeps assertions deterministic regardless of the host env.
     """
-    return re.sub(r"\x1b\[[0-9;]*[mGKHFJ]", "", text)
+    return re.sub(r"\x1b\[[0-?]*[ -/]*[@-~]", "", text)
+
+
+def test_strip_ansi_removes_common_csi_sequences() -> None:
+    assert _strip_ansi("before\x1b[2Cafter") == "beforeafter"
+    assert _strip_ansi("before\x1b[1Aafter") == "beforeafter"
+    assert _strip_ansi("before\x1b[?25lafter") == "beforeafter"
 
 
 EXPECTED_HELP = """\