feat(ui): Add thinking/reasoning block visibility toggle#9139
feat(ui): Add thinking/reasoning block visibility toggle#9139sauerdaniel wants to merge 2 commits intoanomalyco:devfrom
Conversation
|
The following comment was made by an LLM, it may be inaccurate: Potential Duplicate FoundPR #7868: feat(app): add /thinking command to toggle reasoning visibility Why it's related: Both PRs address the same core feature - toggling thinking/reasoning block visibility. However, PR #7868 uses a Also related (but older/different scope):
|
00637c0 to
71e0ba2
Compare
f1ae801 to
08fa7f7
Compare
6312cd5 to
8b6d80e
Compare
|
Rebased onto latest dev and resolved merge conflicts.\n\nNo functional change beyond the thinking-visibility toggle; also typed mouse event handlers to satisfy TS.\n\nPlease re-run checks / re-review when convenient. |
33464c8 to
546aa0b
Compare
|
Re: potential overlap with #7868 — different approach. #7868 adds a |
546aa0b to
5084ad0
Compare
Add toggle button to show/hide thinking blocks after response completion. Currently, reasoning blocks are only visible during streaming and disappear once the response is complete. Changes: - Add showReasoning state to session turn store - Add hasReasoningParts() memo to detect if turn has reasoning content - Show toggle button in response header when reasoning parts exist - Add reasoning section at bottom of response when toggled on - Keep reasoning visible during streaming (existing behavior) Fixes anomalyco#7866
5084ad0 to
a5ec579
Compare
Summary
Add a keyboard shortcut
Ctrl+ito toggle thinking/reasoning block visibility in the TUI.Fixes #13789
Problem
No quick way to toggle thinking block visibility while using the TUI — users must navigate through menus.
Solution
Add
Ctrl+ikeybinding that toggles thinking mode visibility in the session turn component, consistent with other TUI quick-settings shortcuts.Changes
packages/ui/src/components/session-turn.tsx— AddCtrl+itoggle for thinking block visibility with proper mouse event typingTesting
bun turbo typecheck)AI-Assisted
Yes
Design review note
This touches TUI UI behavior; per CONTRIBUTING.md, maintainer design review is requested before merge.