Skip to content

[BUG] Statek counts one terminal harness error multiple times when DI… - #203

Merged
wskozlowski merged 3 commits into
mainfrom
feature/issue_1506
Sep 9, 2026
Merged

wskozlowski merged 3 commits into
mainfrom
feature/issue_1506

Conversation

@KikiMcArron

Copy link
Copy Markdown
Contributor

…RECT-mode LLM turns share the same console_pos

…RECT-mode LLM turns share the same console_pos
Comment thread statek/executors/job.py Outdated
if isinstance(item.tool_log, str):
return 0
return sum(1 for entry in item.tool_log if isinstance(entry, ToolError))
def _is_harness_limit_error(message: str) -> bool:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rozpoznawanie typu błędu po tekście brzmi strasznie amatorsko, to astra implementowała ??

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TO 5.6

Comment thread statek/pyenv.py Outdated
# Diagnostics by console-entry index, including
# harness-limit messages retained for display but excluded from execution counters.
exceptions: Dict[int, str] = None
# Console-entry indexes explicitly recorded as terminal harness diagnostics.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

co to za pole "diagnostics", dlaczego zostało dodane do obiektu memo !?

@wskozlowski
wskozlowski merged commit 0b77e08 into main Sep 9, 2026
4 checks passed
@wskozlowski
wskozlowski deleted the feature/issue_1506 branch September 9, 2026 15:38
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.

2 participants