Improve memory - #227
Conversation
|
Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull Request Overview
This PR improves memory functionality by enhancing user information management, conversation history handling, and database fact extraction in the QueryWeaver application. The changes focus on more comprehensive context preservation and personalized query processing.
- Enhanced memory tool to update user information and handle conversation history
- Modified analysis agent to handle personal queries more robustly with missing information checks
- Improved database facts search with episode content extraction
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| api/routes/graphs.py | Modified memory tool call to include conversation history |
| api/memory/graphiti_tool.py | Major refactoring of memory management with new user information updates, conversation history handling, and improved database facts search |
| api/agents/analysis_agent.py | Enhanced personal query handling with stricter validation for missing user information |
Comments suppressed due to low confidence (1)
api/memory/graphiti_tool.py:1
- This query parameter references
f\"User {self.user_id}\"but based on line 85, the user node name format was changed to justf\"{user_id}\"without the "User " prefix. This inconsistency will cause the pinned user condition to not match any existing user nodes.
"""
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
No description provided.