Skip to content

Switch client result DTOs to discriminated unions#67

Merged
tiksn merged 1 commit into
mainfrom
use-union-type-for-client-results
Jun 8, 2026
Merged

Switch client result DTOs to discriminated unions#67
tiksn merged 1 commit into
mainfrom
use-union-type-for-client-results

Conversation

@tiksn

@tiksn tiksn commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Convert ClientResult and ClientUnitResult from record DTOs to discriminated unions with Success / Failure cases.
  • Update helper constructors to return the new union cases.
  • Rewrite tests and transport stubs to pattern match on the new API instead of reading Succeeded, Value, and Problem fields.

Testing

  • dotnet build .\src\Bridge\Bridge.fsproj -c Release
  • dotnet test .\tests\Bridge.Tests\Bridge.Tests.fsproj -c Release
  • . pack.ps1

@tiksn tiksn self-assigned this Jun 8, 2026
@tiksn tiksn added the enhancement New feature or request label Jun 8, 2026
@tiksn tiksn merged commit 1615319 into main Jun 8, 2026
3 of 4 checks passed
@tiksn tiksn deleted the use-union-type-for-client-results branch June 8, 2026 03:38
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