Unexpected (?) behaviour of LocalVectorStore's addMemories

[victorcrrd]

I think that the current implementation of addMemories for the LocalVectorStore class is not what we would expect. Namely, the actual code updates the previous state's memories as prevState.orderedMemories + memories.groupBy { it.conversationId }, however, for a given conversationId it would just replace the list of Memory to the one in value memories, "forgetting" about the ones in prevState.orderedMemories for that same conversationId. The following image shows the problem:

[javipacheco]

Yes @victorcrrd

I saw the same problem this morning in my PR. I'm working on it