Skip to content

Honor Claude Code terminal error flags - #189

Draft
Qian-Cheng-nju wants to merge 1 commit into
mainfrom
fix/claude-result-error-flag
Draft

Qian-Cheng-nju wants to merge 1 commit into
mainfrom
fix/claude-result-error-flag

Conversation

@Qian-Cheng-nju

Copy link
Copy Markdown
Member

Claude Code can end a quota-blocked background turn with subtype: success and is_error: true. The classifier ignored the error flag and reported OK, allowing an interrupted attempt to be treated as a normal model outcome. Honor the final result's error flag so the existing evaluator records INFRA_ERROR and excludes it from genuine outcomes.

Preserve error_max_turns, runner timeout precedence, and recovery followed by a later successful result. This changes classification only; it does not add quota polling or automatic recovery.

Validation: 1,532 evaluator tests passed; Ruff lint, formatting, and diff checks passed. Three new regression cases fail on the base revision. Replaying 15 completed experiment streams changes only the observed quota-blocked case; the other 14 retain their classification. Original experiment artifacts remain unchanged.

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