Skip to content

fix(agent): handle Codex stdin EOF status#488

Open
simfor99 wants to merge 2 commits into
kunchenguid:mainfrom
simfor99:test/codex-stdin-status-line
Open

fix(agent): handle Codex stdin EOF status#488
simfor99 wants to merge 2 commits into
kunchenguid:mainfrom
simfor99:test/codex-stdin-status-line

Conversation

@simfor99

@simfor99 simfor99 commented Jul 15, 2026

Copy link
Copy Markdown

What Changed

  • Prevent Codex agent runs from waiting on inherited stdin by supplying EOF, while accepting its informational stdin status output.
  • Add coverage for the Codex EOF path and isolate hook-based git tests from global hook configuration.

Risk Assessment

✅ Low: Die Änderung ergänzt nur einen korrekten Hinweis und einen begrenzten Regressionstest für das bereits bestehende EOF-Verhalten von cmd.Stdin = nil.

Testing

Completed 1 recorded test check.

  • Outcome: 🔧 1 issue found → auto-fixed ✅ across 2 runs (11m32s)

Pipeline

Updates from git push no-mistakes

⏭️ **intent** - skipped

✅ No issues found.

✅ **Rebase** - passed

✅ No issues found.

✅ **Review** - passed

✅ No issues found.

🔧 **Test** - 1 issue found → auto-fixed ✅
  • 🚨 tests failed with exit code 1
  • go test -race ./...

🔧 Fix: test: isolate hooks from global config
✅ Re-checked - no issues remain.

  • go test -race ./...
✅ **Document** - passed

✅ No issues found.

✅ **Lint** - passed

✅ No issues found.

✅ **Push** - passed

✅ No issues found.

@simfor99
simfor99 force-pushed the test/codex-stdin-status-line branch from 9d60166 to 1763120 Compare July 19, 2026 12:27
@simfor99 simfor99 changed the title test(agent): cover Codex stdin EOF handling fix(agent): handle Codex stdin EOF status Jul 19, 2026
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