Skip to content

feat(llm-worker): record terminal observation coverage - #3301

Closed
Xore wants to merge 1 commit into
mainfrom
issue-3292-coder
Closed

Xore wants to merge 1 commit into
mainfrom
issue-3292-coder

Conversation

@Xore

@Xore Xore commented Sep 25, 2026

Copy link
Copy Markdown
Owner

Summary

  • add bounded terminal-observation and capture-coverage values to SessionAccumulator
  • preserve close, idle, and still-open distinctions without inferring a termination reason
  • expose covered/excluded session counts in cycle status and document the denominator
  • add synthetic coverage tests and ambiguous-terminal-absence documentation

Validation

  • python -m unittest discover -s llm-worker/tests -v (77 tests, OK)
  • python llm-worker/worker.py --selftest (PASS)

Closes #3292

@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@Xore

Xore commented Sep 25, 2026

Copy link
Copy Markdown
Owner Author

Closing as already-landed. Rebasing onto main after #3302 merged shows this branch is empty: the terminal_observation / capture_coverage fields and the ambiguity wording from this PR are present on main, carried in by #3302 (offline engagement aggregate), which was built on this branch's schema and included it. The only conflict was this PR's doc/test sections that #3302 had already superseded. Verified on main: TERMINAL_OBSERVATION_CLOSE and capture_coverage both in llm-worker/worker.py; 82/82 llm-worker tests pass.

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.

llm worker: preserve session terminal observation and capture coverage in state summaries

1 participant