Repository navigation
fix: return latest conversation messages from native backend - #8
Merged
Merged
Conversation
Contributor
Author
|
Reviewed current Go route and regression: ownership is checked before either query; the omitted-offset branch selects newest records with deterministic created/id ordering and returns chronological order; explicit offset retains the history query. Linux and Windows backend checks, frontend gate, CAS contract/build and validation all passed on this head. |
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
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.
Carry the latest-message window contract into the current Go backend: omitted offset returns the newest 50 messages in chronological order; explicit offset preserves history pagination. Regression exercises the production route with 60 messages and verifies account ownership. Reproduced the oldest-window failure before the fix; Linux full Go suite passed after it. Closes #5.