Skip to content

DIAG: Codex limit card (do not merge) - #42

Closed
codedev-david wants to merge 8 commits into
mainfrom
diag/codex-card
Closed

codedev-david wants to merge 8 commits into
mainfrom
diag/codex-card

Conversation

@codedev-david

Copy link
Copy Markdown
Owner

Throwaway diagnostic run, closed after.

🤖 Generated with Claude Code

David Engelhart and others added 8 commits November 7, 2026 19:23
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ell still echoes the launch line

On Windows, Git Bash echoes Termpolis's long launch line in pieces, and can print one more prompt after
the line is entered and before the echo. A look in that moment saw a prompt and nothing (or part of the
line) after it, latched the agent as exited, and the limit card never appeared (Windows e2e, Codex).
sinceStarted now reads the line's start at the foot as still echoing, and agentGone ignores a prompt
with nothing printed before it in the moment the shell takes to start the agent.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…not counted yet

The renderer's loop asked main about a terminal only when its screen changed. Main's word on the agent
can change while the screen stays the same (the turn from the Enter, a start recorded as the shell
echoes the line), so a loop that asked once too early never opened the chooser. A screen showing a stop
main did not count is now looked at again, up to 10 times, 2 s apart.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…with it

Main judges its own parse of the screen, which can trail the raw output the
2 s loop keys on. On Windows CI the Codex limit screen was looked at once,
main's parse did not show the limit yet, and the unchanged screen was never
looked at again. Any look that is not a counted stop now gets up to
UNSETTLED_LOOKS more looks of the same screen; a new screen starts over.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@codedev-david

Copy link
Copy Markdown
Owner Author

Diagnosis done: root causes fixed on main (a3d7af6, ed39ef8, 793dd52).

@codedev-david
codedev-david deleted the diag/codex-card branch November 8, 2026 01:17
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