Skip to content

fix: surface subscription handler overflow - #10

Merged
smiggleworth merged 1 commit into
mainfrom
fix/client-notification-overflow
Aug 25, 2026
Merged

fix: surface subscription handler overflow#10
smiggleworth merged 1 commit into
mainfrom
fix/client-notification-overflow

Conversation

@smiggleworth

Copy link
Copy Markdown
Contributor

Summary

  • configure detached callback queue capacity independently from request admission and callback concurrency
  • terminate all six notification subscription domains with typed AsyncHandlerOverflowException instead of silently dropping work
  • expose subscription completion and propagate terminal overflow through public async iterators while retaining RPC protocol backpressure
  • export Schedule ERR_BACKEND_ERROR (7010) and classify it as retryable for replay-safe operations

This is the .NET parity companion to cntryl/fitz-ts#22. Stream error decoding already matched the coded-READ and plain-non-READ contract.

Validation

  • dotnet format Fitz.sln --verify-no-changes --no-restore --severity warn
  • dotnet build Fitz.sln -c Release --no-restore
  • dotnet test tests/Core/Core.Tests.csproj -c Release --no-build --filter FullyQualifiedName!~Integration
  • dotnet pack Fitz.sln -c Release --no-restore --output artifacts/packages
  • four-leg TCP/WebSocket and anonymous/valid-JWT conformance matrix
  • domain workflow and invalid-JWT integration gates

Copilot AI lite review requested due to automatic review settings August 25, 2026 14:16

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@smiggleworth
smiggleworth merged commit 1c96c49 into main Aug 25, 2026
14 of 15 checks passed
@smiggleworth
smiggleworth deleted the fix/client-notification-overflow branch August 25, 2026 14:54
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