Skip to content

[#856] Connection failure and streaming issues fixed in different versions – which version supports both?#869

Merged
benbrown merged 4 commits intomainfrom
southworks/update/webchat-status-ack
Jan 14, 2026
Merged

[#856] Connection failure and streaming issues fixed in different versions – which version supports both?#869
benbrown merged 4 commits intomainfrom
southworks/update/webchat-status-ack

Conversation

@ceciliaavila
Copy link
Copy Markdown
Collaborator

Fixes #856

Description

This PR improves the handling of connection acknowledgments for webchat, ensuring that the client remains responsive even when the startConversation method does not return activities.

Detailed Changes

  • Added once function to make sure the connection ack only happens once.
  • Added a second call for handleAcknowledgementOnce outside the forach loop to acknowledge even when no activities are received from the agent.
  • Replaced await 0 with await Promise.resolve() for clarity.

Testing

We tested the changes for both scenarios:

  • Agent with the StartConversation topic disabled.
image
  • Agent with the StartConversation topic enabled.
image

@ceciliaavila ceciliaavila requested a review from benbrown January 14, 2026 15:44
@benbrown benbrown enabled auto-merge (squash) January 14, 2026 18:26
@benbrown benbrown merged commit 8ca49ca into main Jan 14, 2026
7 checks passed
@benbrown benbrown deleted the southworks/update/webchat-status-ack branch January 14, 2026 18:29
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.

Connection failure and streaming issues fixed in different versions – which version supports both?

2 participants