Skip to content

add debounce for working shimmer#304285

Merged
justschen merged 3 commits intomainfrom
justin/bisharp
Mar 23, 2026
Merged

add debounce for working shimmer#304285
justschen merged 3 commits intomainfrom
justin/bisharp

Conversation

@justschen
Copy link
Collaborator

fix #304050

Copilot AI review requested due to automatic review settings March 23, 2026 22:08
@vs-code-engineering vs-code-engineering bot added this to the 1.114.0 milestone Mar 23, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR addresses chat response streaming jitter (Issue #304050) by debouncing when the “Working” shimmer/progress affordance is shown while progressively rendering markdown content.

Changes:

  • Add a small debounce window (WORKING_CAUGHT_UP_DEBOUNCE_MS) before showing “Working” after the renderer has caught up.
  • Gate the “Working” display for the markdownContent && !moreContentAvailable case behind a new hasBeenCaughtUpLongEnough(...) check.

justschen and others added 2 commits March 23, 2026 15:15
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@justschen justschen enabled auto-merge (squash) March 23, 2026 22:30
@justschen justschen merged commit 5785d32 into main Mar 23, 2026
17 of 19 checks passed
@justschen justschen deleted the justin/bisharp branch March 23, 2026 22:45
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.

Chat: A lot of jitter when streaming responses

3 participants