Skip to content

Conversation

@andy-k-improving
Copy link
Contributor

@andy-k-improving andy-k-improving commented Aug 26, 2025

Description

This PR adds support for optional Neptune Analytics graph backend settings in the mem0_memory tool through environment variables, allowing users to configure additional Neptune Analytics graph backend to enhance the memory search.

Documentation PR

strands-agents/docs#222

Changes made

  • Add support for NEPTUNE_ANALYTICS_HOST for mem0 tool
  • Associated test to assert the client setup
  • Documentation to explain the usage of NEPTUNE_ANALYTICS_HOST

Testing

hatch test tests/test_mem0.py -v  # All mem0 tests pass
hatch test                       # Full test suite passes
hatch run prepare     

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@andy-k-improving
Copy link
Contributor Author

@mehtarac wonder would that be possible to have a look on this?
This is similar to the past mem0 PR, ex: #221
Thanks!

@andy-k-improving
Copy link
Contributor Author

@Unshure would you mind to have a look?
This should be a quick one, thanks!

Signed-off-by: Andy Kwok <[email protected]>
Signed-off-by: Andy Kwok <[email protected]>
Signed-off-by: Andy Kwok <[email protected]>
Signed-off-by: Andy Kwok <[email protected]>
Signed-off-by: Andy Kwok <[email protected]>
Signed-off-by: Andy Kwok <[email protected]>
Signed-off-by: Andy Kwok <[email protected]>
Signed-off-by: Andy Kwok <[email protected]>
@andy-k-improving
Copy link
Contributor Author

Hi @mehtarac , I believe this build failure is caused by #237
and I have rebase my PR for this the purpose, would you mind to trigger the build again and review the PR?
Thansk!

@mehtarac
Copy link
Member

mehtarac commented Sep 9, 2025

Adding for visibility: The documentation found for Neptune analytics with mem0 memory - https://docs.mem0.ai/open-source/graph_memory/overview#initialize-neptune-analytics

@andy-k-improving
Copy link
Contributor Author

@mehtarac Thx for the review, regarding the next step, would you mind to trigger the integration test?

@zastrowm zastrowm merged commit bc1aaff into strands-agents:main Sep 10, 2025
15 of 16 checks passed
@andy-k-improving
Copy link
Contributor Author

@zastrowm Thx, would you mind to also merge strands-agents/docs#222 which is the doc this PR.

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.

4 participants