Skip to content

fix: update Mem0MemoryStore for Mem0 v3 + adapt Agent#466

Open
anakin87 wants to merge 4 commits intomainfrom
mem0-v3
Open

fix: update Mem0MemoryStore for Mem0 v3 + adapt Agent#466
anakin87 wants to merge 4 commits intomainfrom
mem0-v3

Conversation

@anakin87
Copy link
Copy Markdown
Member

@anakin87 anakin87 commented Apr 17, 2026

Related Issues

Proposed Changes:

  • Mem0 v3
    • remove async_mode parameter (removed in underlying API)
    • handle new results wrapping
    • adapt tests
  • Adapt Agent for Haystack 2.27.0

How did you test it?

CI

Checklist

@anakin87 anakin87 changed the title chore: update Mem0MemoryStore for Mem0 v3 fix: update Mem0MemoryStore for Mem0 v3 + adapt Agent Apr 17, 2026
@anakin87 anakin87 requested a review from sjrl April 17, 2026 07:00
def test_role_based_memories(self, memory_store):
store, user_id = memory_store
unique_agent_id = _get_unique_user_id()
messages = [
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sjrl could you review this change in particular?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hard for me to review since I don't fully understand Mem0. It looks reasonable to me in the sense that if that's what Mem0 requires then that's fine.

I don't think I really know of a use case for this. I.e. I don't know when I'd want to search for these two sets separately.

@anakin87 anakin87 marked this pull request as ready for review April 17, 2026 07:00
@anakin87 anakin87 requested a review from a team as a code owner April 17, 2026 07:00
@anakin87 anakin87 self-assigned this Apr 17, 2026
Copy link
Copy Markdown
Contributor

@sjrl sjrl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@anakin87 thanks for working on this! The changes looks fine to me, but if you'd like a more in depth review I'd recommend asking @Amnah199

@anakin87
Copy link
Copy Markdown
Member Author

@anakin87 thanks for working on this! The changes looks fine to me, but if you'd like a more in depth review I'd recommend asking @Amnah199

I'll ask her

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants