Skip to content

Let harness see every agent event and stop retrying what fails the same way - #43

Open
alexandre-daubois wants to merge 1 commit into
mainfrom
sandbox/harness-signals
Open

alexandre-daubois wants to merge 1 commit into
mainfrom
sandbox/harness-signals

Conversation

@alexandre-daubois

Copy link
Copy Markdown
Member

The agent wrapper only forwarded the session event, so RunSession never saw a lost conversation and never restarted it fresh, and StreamCmd never saw the error events it classifies as account errors. The retry now skips a model account refused for good and a spent budget. Codex gets its effort through Job.Effort, catalogued CWE ids such as cwe-787 are spelled canonically instead of dropped, GitHub reads retry 429, 502, 503 and 504, a test keeps the agent image pins in step, and CI vets the evals.

…me way

The agent wrapper only forwarded the session event, so RunSession never saw a lost conversation and never restarted it fresh, and StreamCmd never saw the error events it classifies as account errors. The retry now skips a model account refused for good and a spent budget. Codex gets its effort through Job.Effort, catalogued CWE ids such as cwe-787 are spelled canonically instead of dropped, GitHub reads retry 429, 502, 503 and 504, a test keeps the agent image pins in step, and CI vets the evals.
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