chore: Init Schema #18
Merged
Conversation
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
Contributor
WalkthroughThis pull request restructures the domain model by replacing the Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant API as Web.Api
participant Exception as GlobalExceptionMiddleware
participant Service as Application Service
Client->>API: Request
API->>Service: Execute business logic
alt BusinessRuleException thrown
Service-->>Exception: Throw BusinessRuleException
else ValidationException thrown
Service-->>Exception: Throw ValidationException
else ApplicationException thrown
Service-->>Exception: Throw ApplicationException (wrapped in RequestProcessingException)
end
Exception->>Exception: Detect RequestProcessingException
Exception->>Exception: Unwrap inner exception
Exception->>Exception: GetExceptionDetails(inner exception)
alt Known custom exception
Exception->>Exception: Map to status code & message
else Standard .NET exception
Exception->>Exception: Map to status code & message
else Unknown
Exception->>Exception: Return 500 Internal Server Error
end
Exception-->>Client: HTTP Response + structured Error details
sequenceDiagram
participant App as ConversationAggregate
participant Msg as Message Entity
participant Constants as MessageRoles
App->>App: AddMessage(senderId, content, type, isFromBot)
alt isFromBot = true
App->>Constants: Get MessageRoles.Assistant
else isFromBot = false
App->>Constants: Get MessageRoles.User
end
App->>Msg: Create Message with Role property
Msg->>Msg: Role initialized (Assistant or User)
App->>App: Add to _messages collection
Note over Msg: Old: IsFromBot property<br/>New: Role = "assistant" or "user"
Estimated code review effort🎯 4 (Complex) | ⏱️ ~50 minutes
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Free 📒 Files selected for processing (31)
💤 Files with no reviewable changes (10)
Note 🎁 Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login. Comment |
taiphanvan2k3
added a commit
that referenced
this pull request
Dec 22, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary by CodeRabbit
New Features
Bug Fixes
Chores
Refactor