umputun / fya / 27921146628

86%

master: 86%

Build Type

Pull #18

github

Committed by Rishi Khan

Commit Message

fix: strip undeliverable control chars from prompts to prevent turn wedge

A prompt containing a raw control byte (ESC most notably) silently wedged
the turn: the byte is typed into Claude's interactive TUI rune-by-rune
where it is read as a control key (ESC clears the input line), so the
prompt never submits and the turn hangs until timeout, surfacing as
"select transcript: context error: context canceled".

Empirically every C0 byte (0x01-0x1F) and 0x7F DEL wedge the turn; tab is
unreliable even via bracketed paste; all Unicode >= U+0080 is delivered
fine. Sanitize at the single input boundary so the injected prompt stays
consistent with transcript matching:

- promptNormalizer now expands tab to four spaces (the only deliverable
  whitespace besides LF), alongside the existing CRLF/CR -> LF folding.
- stripUndeliverable removes C0 (except LF) and DEL, emitting a one-line
  stderr warning naming the dropped code points.
- wire input.Request.Warn to stderr in main.

Covered by input tests; full race suite, gofmt and go vet pass.

Pull Request Pull Request #18: fix: strip undeliverable control chars from prompts to prevent turn wedge

Coverage Stats

33 of 34 new or added lines in 2 files covered. (97.06%)

7 existing lines in 1 file now uncovered.

1713 of 1984 relevant lines covered (86.34%)

44.02 hits per line