Skip to content

Fix resume waiter leaks and long-session TUI stalls - #612

Merged
zhnt merged 3 commits into
mainfrom
fix/tui-resume-context-freeze
Sep 24, 2026
Merged

zhnt merged 3 commits into
mainfrom
fix/tui-resume-context-freeze

Conversation

@zhnt

@zhnt zhnt commented Sep 24, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Cancel the pending terminal render waiter when input returns or is cancelled, preventing the Python 3.14 Event.wait() task warning after resume.
  • Defer synchronous context-usage refresh until the active TUI run completes so Bash tool execution does not pause Working animation or input.
  • After compaction, avoid rebuilding a pre-compaction model request for provider-usage display. Reuse the already replayed context when counting messages.

Validation

  • 63 focused Harness, transcript, compaction, model-call, and Coding session tests passed on local main.
  • Focused TUI and Coding TUI playback tests passed during the original fixes.
  • Ruff check passed for the changed Harness files; git diff --check origin/main..main passed.
  • Local make check-changed passed docs and architecture checks, then was stopped during its long Coding suite. The selected GitHub checks are the merge gate.

Contract notes

AgentTranscriptInspector.message_counts() now accepts an optional prebuilt message sequence. Existing callers remain compatible. No configuration changes.

@zhnt
zhnt merged commit 49ccf9e into main Sep 24, 2026
37 checks passed
@zhnt
zhnt deleted the fix/tui-resume-context-freeze branch September 24, 2026 07:09
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