docs(adr): ADR-011 — error handling / Result pattern (Accepted) - #2
Merged
Conversation
Decision-level MADR. Recommends custom Result/Result<T> + Error + ErrorType taxonomy with a single Result -> RFC 9457 problem-details mapper via IProblemDetailsService. Traces the "slot already taken" failure end to end; taxonomy -> HTTP status table; Results-vs-exceptions boundary. Verified 2026-07-21: RFC 9457 obsoletes 7807; .NET 10 problem details APIs; ErrorOr/FluentResults both MIT (ErrorOr = off-ramp). Status: Proposed. First Phase 1 ADR. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Incorporate review edits: note 422 considered/rejected for validation (400 = ASP.NET default); set problem-details type explicitly to about:blank (RFC 9457 default) with future type-URI catalog deferred; pin the plural field-level validation `errors` contract that ADR-009 / Signal Forms consume. Fix ROADMAP RFC 7807 -> RFC 9457 (3 mentions). Status: Proposed -> Accepted; update index. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
First Phase 1 ADR. Accepted after review.
Decision: custom
Result/Result<T>+Error+ErrorType; single Result → RFC 9457 problem-details mapper viaIProblemDetailsService.type: about:blank(9457 default); validation carries field-levelerrorsdict (Signal Forms contract)RFC 7807→RFC 9457(3 mentions)🤖 Generated with Claude Code