Skip to content

Introduce async ChatEventDispatcher for non-blocking events - #1440

Merged
Oceania2018 merged 1 commit into
SciSharp:masterfrom
hchen2020:master
Sep 15, 2026
Merged

Oceania2018 merged 1 commit into
SciSharp:masterfrom
hchen2020:master

Conversation

@hchen2020

Copy link
Copy Markdown
Contributor

Added ChatEventDispatcher to decouple chat event production from SignalR delivery, ensuring producers are never blocked by slow clients. Registered dispatcher as singleton with detailed documentation. Refactored event emission to use dispatcher queue, updated observer to avoid awaiting sends. Added unit tests for dispatcher behavior and updated test project references.

Added ChatEventDispatcher to decouple chat event production from SignalR delivery, ensuring producers are never blocked by slow clients. Registered dispatcher as singleton with detailed documentation. Refactored event emission to use dispatcher queue, updated observer to avoid awaiting sends. Added unit tests for dispatcher behavior and updated test project references.
@qodo-code-review

Copy link
Copy Markdown
Contributor

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@Oceania2018
Oceania2018 merged commit e8aff60 into SciSharp:master Sep 15, 2026
3 of 4 checks passed
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