From ab7293bed6529b4f28f3b4109cfe96e5e62fc3e0 Mon Sep 17 00:00:00 2001 From: hjc-puro Date: Mon, 17 Nov 2025 18:39:16 -0500 Subject: [PATCH] don't log exit code !=0 as terminal failure --- batch_runner.py | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/batch_runner.py b/batch_runner.py index 9d21aebc..2487d9fb 100644 --- a/batch_runner.py +++ b/batch_runner.py @@ -98,10 +98,9 @@ def _extract_tool_stats(messages: List[Dict[str, Any]]) -> Dict[str, Dict[str, i # Terminal wraps its response in a "content" field if "content" in content_json and isinstance(content_json["content"], dict): inner_content = content_json["content"] - # Check for actual error (non-null error field or non-zero exit code) - has_error = (inner_content.get("error") is not None or - inner_content.get("exit_code", 0) != 0) - if has_error: + # Check for actual error (non-null error field) + # Note: non-zero exit codes are not failures - the model can self-correct + if inner_content.get("error") is not None: is_success = False # Check for "success": false pattern used by some tools