Add message-id to messages list in API response#86
Open
sweep-ai-deprecated[bot] wants to merge 26 commits intomainfrom
Open
Add message-id to messages list in API response#86sweep-ai-deprecated[bot] wants to merge 26 commits intomainfrom
sweep-ai-deprecated[bot] wants to merge 26 commits intomainfrom
Conversation
Contributor
Author
Rollback Files For Sweep
|
Contributor
Author
Apply Sweep Rules to your PR?
|
2 tasks
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.
PR Feedback (click)
Description
This PR adds a unique message-id to each message in the API response of the
/sessions/{session-id}/memoryendpoint in the Motorhead memory server. The message-id is a UUID and facilitates the ability to delete or update a specific message content.Summary of Changes
message_idfield to theMemoryMessagestruct insrc/models.rs.get_memoryfunction insrc/memory.rsto include themessage_idin the response.post_memoryfunction insrc/memory.rsto generate and store a uniquemessage_idfor each new message.delete_memoryfunction insrc/memory.rsto delete messages based on themessage_id.handle_compactionfunction insrc/reducer.rsto handle the compaction of messages based on themessage_id.Fixes #85.
🎉 Latest improvements to Sweep:
💡 To get Sweep to edit this pull request, you can: