feat(tui): animate reasoning effort transitions - #567
Open
yltx wants to merge 6 commits into
Open
Conversation
…y#507/ccch1mneyyy#492） 补充 ccch1mneyyy#548（已合并的退出鼠标残留闩锁）：probe/reassert 的 isUnmounted 守卫已由 ccch1mneyyy#548 落地，本 PR 补上它未覆盖的两条泄漏路径： - TerminalQuerier 加 disposed 标志：dispose 后 send()/flush() 入口拒绝 （resolve undefined 保持调用方 inert），不再 holdRawMode 拉回 raw mode （?2004h ?1004h >1u >4;2m）、不再写查询字节——ccch1mneyyy#507 实证的 DECRPM/ DA1/XTVERSION 回复在进程退出后无人消费、落入 shell 输入队列（实测 被整行执行的副作用）就此断根 - finishExit 在清理序列落地后 settle 150ms 再 drainStdin：detach 时点 的 drain 看不见清理落地后才到的在途回复与鼠标事件；150ms 覆盖 ssh 慢链路回复 RTT（ccch1mneyyy#522 即 ssh 环境），吞掉在途字节不落入 shell 回归：scripts/verify-exit-mouse-residue.tsx（挂 input-terminal 组， 与 ccch1mneyyy#548 的 verify-exit-mouse-cleanup 互补）——detached 后触发 probe/ reassert/querier 三条防线，断言零 ENABLE、零查询字节、零 raw mode 重开。rebase 于 ccch1mneyyy#548 之上，两个脚本全绿。
Owner
|
Check if the features of #386 effort ignition are redundant？ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
Why
The prior effort feedback made the top tiers look too similar, used fixed truecolor accents rather than the active theme, and ended completely after the one-shot transition. This makes each tier visually identifiable while keeping continuous animation bounded and inactive when hidden. The preview setting also makes every visual tier testable on models that do not advertise it.
Notes
previewEfforts is UI-only for levels the active adapter does not advertise: it does not persist that selection or send it to the model.