Skip to content

Conversation

RomneyDa
Copy link
Collaborator

@RomneyDa RomneyDa commented Oct 13, 2025

If streaming was canceled mid-think, it kept blinking the 3 dots with "thinking". Simple fix on second commit


Summary by cubic

Fixes the "thinking" indicator so it stops when a stream is canceled. The last message is only marked in progress while isStreaming is true, preventing the dots from blinking after cancellation.

  • Bug Fixes
    • Tie inProgress to isStreaming and add isStreaming to useMemo deps for correct re-render on cancel.

@RomneyDa RomneyDa requested a review from a team as a code owner October 13, 2025 21:45
@RomneyDa RomneyDa requested review from Patrick-Erichsen and removed request for a team October 13, 2025 21:45
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Oct 13, 2025
@RomneyDa RomneyDa changed the title fix: thinking won't stop if cancelled mid stream fix: HOTFIX stop thinking animation if cancelled mid stream Oct 13, 2025
Copy link

github-actions bot commented Oct 13, 2025

✅ Review Complete

Code Review Summary

⚠️ Continue configuration error. Please verify that the assistant exists in Continue Hub.


Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Oct 13, 2025
@Patrick-Erichsen Patrick-Erichsen merged commit 990a8fb into main Oct 13, 2025
70 of 72 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Oct 13, 2025
@Patrick-Erichsen Patrick-Erichsen deleted the dallin/thinking-does-not-stop-2 branch October 13, 2025 22:34
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 13, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Oct 13, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants