Skip to content

Conversation

@Moonlight-CL
Copy link
Contributor

@Moonlight-CL Moonlight-CL commented Sep 15, 2025

…ool and initiating it use default configuration

Description

Get the collection_name from the environment rather than use the 'mem0_memory' when use mem0-memory tool with the default configuration.

When use 'mem0_memory' as the collection name to create a collection in AWS Opensearch, it is not a valid collection name.

Related Issues

#250

Documentation PR

N/A, as there is no user-facing change on this PR.

Type of Change

Bug fix
New Tool
Breaking change
Documentation update
Other (please describe): Enhancement.

Testing

How have you tested the change? Verify that the changes do not break functionality or introduce warnings in consuming repositories: agents-docs, agents-tools, agents-cli

  • I ran hatch run prepare

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

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

…ool and initiating it use default configuration
@Moonlight-CL Moonlight-CL requested a review from a team as a code owner September 15, 2025 09:44
Copy link
Contributor Author

@Moonlight-CL Moonlight-CL left a comment

Choose a reason for hiding this comment

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

It looks fine , read collection_name from the environment and get the default value if user doesn't set the environment

@mehtarac mehtarac merged commit 55354a1 into strands-agents:main Sep 17, 2025
16 checks passed
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.

3 participants