Skip to content

fix(tui): expand active tool run summaries#3818

Open
cyq1017 wants to merge 4 commits into
Hmbown:mainfrom
cyq1017:codex/codewhale-3256-active-tool-run-expand
Open

fix(tui): expand active tool run summaries#3818
cyq1017 wants to merge 4 commits into
Hmbown:mainfrom
cyq1017:codex/codewhale-3256-active-tool-run-expand

Conversation

@cyq1017

@cyq1017 cyq1017 commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Include active in-flight entries when resolving dense tool-run expansion.
  • Add a regression test for toggling an active tool-run summary before it flushes into history.

This covers a small #3256 edge case: the transcript can already collapse active dense tool runs, but the expansion lookup only considered committed history.

Verification

  • cargo fmt --check
  • git diff --check
  • DEEPSEEK_BUILD_SHA=localtest cargo test -p codewhale-tui --bin codewhale-tui tool_run --locked passed on the same two-file patch before replaying it onto a fresh clone.

@Hmbown

Hmbown commented Jul 3, 2026

Copy link
Copy Markdown
Owner

Thanks @cyq1017. I am leaving this out of 0.8.67 because it is still draft and now conflicts with current main. The feature is still valuable; once the release cut is past, please rebase/undraft and we can review it on the 0.8.68 lane with your authorship preserved.

…active-tool-run-expand

# Conflicts:
#	crates/tui/src/core/engine/tests.rs
@cyq1017 cyq1017 marked this pull request as ready for review July 5, 2026 01:48
@cyq1017 cyq1017 requested a review from Hmbown as a code owner July 5, 2026 01:48
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