mem0 icon indicating copy to clipboard operation
mem0 copied to clipboard

Published 20 hours ago • verified publisher icon mem0ai

Add support for FalkorDB MemoryGraph

Open gkorland opened this issue 1 month ago • 8 comments

Implement FalkorDB MemoryGraph support with tests

Description

  • FalkorDB added as a supported graph memory backend with configuration options, vector similarity search, entity/relationship storage, scoped CRUD operations, and reset capabilities.
  • Comprehensive tests covering initialization, add/search/get/delete/reset flows and utility behaviors.
  • Optional FalkorDB dependency added and provider registered in the graph store factory.

Fixes #1739

Type of change

  • [x] New feature (non-breaking change which adds functionality)
  • [x] Documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • [x] Unit Test

Checklist:

  • [x] My code follows the style guidelines of this project
  • [x] I have performed a self-review of my own code
  • [x] I have commented my code, particularly in hard-to-understand areas
  • [x] I have made corresponding changes to the documentation
  • [x] My changes generate no new warnings
  • [x] I have added tests that prove my fix is effective or that my feature works
  • [x] New and existing unit tests pass locally with my changes
  • [x] Any dependent changes have been merged and published in downstream modules
  • [x] I have checked my code and corrected any misspellings

Maintainer Checklist

  • [ ] closes #1739
  • [ ] Made sure Checks passed

gkorland avatar Oct 01 '25 04:10 gkorland

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Oct 01 '25 04:10 CLAassistant

@gkorland Hey thanks for the FalkorDB integration. Keeping this PR as open since this is on the roadmap but are planning to have it's support in the future releases.

parshvadaftari avatar Oct 01 '25 12:10 parshvadaftari

Ok thanks, meanwhile can you please trigger the CI so I can fix any issues I might have missed?

gkorland avatar Oct 01 '25 12:10 gkorland

Yes definitely!

parshvadaftari avatar Oct 01 '25 12:10 parshvadaftari

@parshvadaftari thanks it looks like it all green. But the CLA for some reason is asking Github Copilot to sign it...

gkorland avatar Oct 01 '25 23:10 gkorland

@gkorland the commit contains co pilot so it considers that as a user. Since it's co authored by code rabbit.

parshvadaftari avatar Oct 03 '25 13:10 parshvadaftari

@parshvadaftari

@gkorland the commit contains co pilot so it considers that as a user. Since it's co authored by code rabbit.

OK I fixed it

gkorland avatar Oct 05 '25 08:10 gkorland

@parshvadaftari can you please merge it?

gkorland avatar Oct 19 '25 14:10 gkorland