Skip to content

Reject stalled input probes and incomplete benchmark runs - #73

Merged
chrissotraidis merged 1 commit into
mainfrom
codex/benchmark-process-guards
Oct 13, 2026
Merged

chrissotraidis merged 1 commit into
mainfrom
codex/benchmark-process-guards

Conversation

@chrissotraidis

Copy link
Copy Markdown
Owner

The controller input helper could hang indefinitely when a FIFO existed without a reader, despite its requested timeout. It now bounds the connection wait while preserving blocking command writes after connection.

The benchmark could also append a successful CSV row when the runner exited during settling or partway through measurement. It now checks the runner throughout both intervals, reports its exit status and rejects incomplete runs.

Validation: both failures reproduced before the fixes; eight real-FIFO/synthetic-runner regressions pass under Apple Python 3.9, along with source bootstrap and the complete repository suite. No runtime or player defaults change. The remaining timing fields, traversal routes and physical gameplay checks stay open; no performance improvement is claimed.

@chrissotraidis
chrissotraidis merged commit ad95d72 into main Oct 13, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant