Skip to content

fix(chat): show loader when agent is still processing on the backend#6912

Merged
shaejaz merged 10 commits intomasterfrom
fix/chat-loader-visibility
Mar 5, 2026
Merged

fix(chat): show loader when agent is still processing on the backend#6912
shaejaz merged 10 commits intomasterfrom
fix/chat-loader-visibility

Conversation

@shaejaz
Copy link
Contributor

@shaejaz shaejaz commented Mar 3, 2026

FX-3715 FX-3727

Cases when loader will show in streaming state:

  • The last part in the message is text (we don't have to show a loader in this case since the text is streaming)
  • The last part is a tool with no tool output (as we can show anything yet)

Actions also only show up when streaming is done. Side effect of this is that they will also appear after suggestions are generated, which is not how it's done yet.

Screen.Recording.2026-03-04.at.12.29.12.mov

@codesandbox-ci
Copy link

codesandbox-ci bot commented Mar 3, 2026

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 21cf69f:

Sandbox Source
example-instantsearch-getting-started Configuration
example-react-instantsearch-getting-started Configuration
example-react-instantsearch-next-app-dir-example Configuration
example-react-instantsearch-next-routing-example Configuration
example-vue-instantsearch-getting-started Configuration

@shaejaz shaejaz marked this pull request as ready for review March 4, 2026 12:43
Copy link
Contributor

@Haroenv Haroenv left a comment

Choose a reason for hiding this comment

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

simplification possible I think

@shaejaz shaejaz requested a review from Haroenv March 5, 2026 11:17
Copy link
Contributor

@Haroenv Haroenv left a comment

Choose a reason for hiding this comment

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

some can be removed now

@shaejaz shaejaz merged commit 6d180e6 into master Mar 5, 2026
14 checks passed
@shaejaz shaejaz deleted the fix/chat-loader-visibility branch March 5, 2026 14:28
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.

2 participants