Skip to content

Add client result unions for happy-path and problem responses#60

Merged
tiksn merged 1 commit into
mainfrom
add-union-type-for-client
May 20, 2026
Merged

Add client result unions for happy-path and problem responses#60
tiksn merged 1 commit into
mainfrom
add-union-type-for-client

Conversation

@tiksn
Copy link
Copy Markdown
Contributor

@tiksn tiksn commented May 20, 2026

Summary

  • Add ClientResult<'T> and ClientUnitResult beside ProblemDetailsModel for typed happy-path and problem responses
  • Update IHttpRequestSender, IHttpTransport, and all bridge clients to return result unions instead of raw payloads or unit tasks
  • Classify 2xx responses as success and deserialize non-2xx bodies into ProblemDetailsModel
  • Add tests for result helpers, transport behavior, and status-code handling

Testing

  • dotnet test passed
  • ./pack.ps1 passed and produced NuGet and NPM package artifacts

@tiksn tiksn self-assigned this May 20, 2026
@tiksn tiksn added the enhancement New feature or request label May 20, 2026
@tiksn tiksn merged commit 09e0c77 into main May 20, 2026
12 of 22 checks passed
@tiksn tiksn deleted the add-union-type-for-client branch May 20, 2026 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant