Skip to content

PRD-2078: Stop response generation#6

Merged
veliseev93 merged 11 commits into
developmentfrom
PRD-2078-stop-response-generation
Dec 4, 2025
Merged

PRD-2078: Stop response generation#6
veliseev93 merged 11 commits into
developmentfrom
PRD-2078-stop-response-generation

Conversation

@Tomass673
Copy link
Copy Markdown
Collaborator

@Tomass673
Copy link
Copy Markdown
Collaborator Author

@veliseev93 could you please take a look?

@veliseev93
Copy link
Copy Markdown
Contributor

@Tomass673 The basic logic works well, but could you please make a few changes:

  • Instead of a loader, show a "stop" icon (in the same button)
  • After the response generation is stopped, allow the user to send messages to the chat again

@veliseev93 veliseev93 assigned Tomass673 and unassigned veliseev93 Dec 2, 2025
@Tomass673 Tomass673 assigned veliseev93 and unassigned Tomass673 Dec 3, 2025
/>
{isResponseGenerating ? (
<IconButton
iconName='stopCircle'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@Tomass673 Could you please use this icon to keep the button style consistent?
stop.svg.zip
So, the button should look lie this:
Screenshot 2025-12-04 at 08 55 14

field.onChange(text);
};

const onStopGenerationPress = async (): Promise<void> => {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

After stopping response generation, we should also mark the last message as done: true. We use this flag to determine whether a response is still being generated or not. This can be done by patching the chat like this: patch-new-chat.ts.
We also patch this field for messages when retrieving the chat in chat api ( there is similar logic in the web application ).

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I'm already doing that here

@veliseev93 veliseev93 assigned Tomass673 and unassigned veliseev93 Dec 4, 2025
@Tomass673 Tomass673 assigned veliseev93 and unassigned Tomass673 Dec 4, 2025
@veliseev93 veliseev93 merged commit e007748 into development Dec 4, 2025
1 check passed
@veliseev93 veliseev93 deleted the PRD-2078-stop-response-generation branch December 4, 2025 10:23
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.

3 participants