docs: clarify short-term memory behavior in the memory example#483
docs: clarify short-term memory behavior in the memory example#483azarboon wants to merge 2 commits intostrands-agents:mainfrom
Conversation
|
Hi @azarboon, thanks for the contribution! I did some digging into the mem0 default behavior before merging to make sure the note is accurate. Looking at the example's source code ( I verified this by tracing through the mem0 source:
So the original wording ("Storing user-specific information persistently") appears to be accurate for the default config. The proposed note saying memory is "short-term and scoped to the current session" could mislead users into thinking they need extra setup for persistence, when persistence is actually the default. That said, your instinct that users might want alternative options is helpful. Would you be open to revising the note to something like:
...and please accept our apologies for sitting on this PR! |
thank you @ryanycoleman for digging into this and for the comment. I have updated the commit accordingly. I think it is ready for merge. |
Documentation Preview FailedThe documentation deployment encountered an error. Please check the deployment logs for more details. |
|
Assessment: Approve ✅ This is a helpful documentation improvement that clarifies mem0's default memory persistence behavior. The content has been verified by a maintainer and the external link is valid. Review Details
One minor suggestion for syntax consistency has been left as an inline comment, but this is non-blocking. Thanks for the contribution! |
|
it looks like there's a conflict here? |
Yes, there appears to be an issue, but I don’t fully understand it. My changes were minimal. Could you help me resolve it? |
Description
What changed:
This change clarifies that the memory used in the memory agent workflow is short-term, using the same terminology as the mem0 documentation, and is scoped to the current session by default. It also directs users to the relevant configuration and documentation for enabling persistence across sessions. Links to the appropriate mem0 documentation are added to improve discoverability and navigation.
Why this change is needed:
The current documentation is largely silent and somewhat misleading regarding memory persistence. This ambiguity is compounded by the fact that mem0’s own documentation is vague on this topic. I had to spend time verifying the actual behavior, and this change aims to save that effort for future readers. The updated text aligns expectations with the default behavior and provides clear pointers to configuration options for enabling persistence.
Related Issues
Type of Change
Checklist
mkdocs serveBy submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.