feat(tui): add full-screen thinking stream#1755
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces a "thinking stream" feature to the TUI, enabling a live, full-terminal overlay dedicated to reasoning steps. Key changes include the addition of the /thinking command (and /think alias), a new ThinkingOnly mode for the live transcript overlay, and integration into the UI to trigger this view when selecting active reasoning cells. Feedback suggests refactoring duplicated rendering logic in live_transcript.rs to improve maintainability, localizing hardcoded status and label strings to match the project's internationalization standards, and optimizing the inspection of the view stack to avoid inefficient pop and push operations.
Summary
Fixes #1750.
This adds a full-terminal live thinking stream so long reasoning output can be monitored as it arrives without being squeezed into the compact transcript pane.
What changed
/thinkingslash command, with/thinkas a short alias, that opens a thinking-only live overlay.LiveTranscriptOverlaywith aThinkingOnlymode that:HistoryCell::Thinkingentries only;End/G.Ctrl+OActivity Detail behavior for in-flight thinking: when the selected/current activity is still streaming, it opens the live thinking stream; finalized thinking still opens the existing reasoning timeline pager.Verification
cargo fmt --all -- --checkcargo test -p deepseek-tui live_transcript --all-featurescargo test -p deepseek-tui execute_thinking_opens_live_reasoning_stream --all-featurescargo test -p deepseek-tui activity_detail_opens_live_thinking_stream_for_streaming_thinking --all-featurescargo test -p deepseek-tui command_registry_has_unique_names_and_aliases --all-featurescargo test -p deepseek-tui localization --all-featurescargo clippy -p deepseek-tui --all-targets --all-featuresNote: clippy still reports existing warnings in
schema_migration.rsand existingneedless_returnwarnings inui.rs; no new blocking warnings were introduced by this change.