fix(tui): add regression test verifying viewport auto-follows streaming answer (#5456)#5476
fix(tui): add regression test verifying viewport auto-follows streaming answer (#5456)#5476ttmouse wants to merge 1 commit into
Conversation
…ng answer (esengine#5456) Add a regression test that verifies the transcript viewport auto-follows streaming answer text when the user is scrolled to the bottom. This complements the existing TestRunningStreamPreservesScrolledReadingPosition which asserts the inverse — scrolled-up readers are not yanked back. The test: 1. Fills the viewport with overflow content (banner + 40 notice lines) 2. Verifies the viewport is at bottom 3. Streams event.Text content (simulating model response) 4. Verifies the viewport stays pinned to the bottom after each chunk
d6b68f0 to
0ff0a74
Compare
|
Thanks for adding coverage here. I do not think this PR should be treated as the fix or regression guard for #5456 in its current form. The issue's root cause was the removal of I also checked the proposed test against the v1.12.0 code plus this test file: it still passes there, so it would not have caught the reported #5456 regression. If we keep this PR, I think it should be scoped as supplemental coverage for answer-stream auto-follow while already pinned to the bottom, and the title/body should remove the #5456 fix implication. Otherwise, I think it is reasonable to close this as superseded by #5378/#5400. |
修复内容
为 Issue #5456 添加回归测试,验证视口在流式输出时自动跟随。
变更说明
验证