From PR #196 review (Issue #1).
When update_episode() or mark_memories_summarized() fails after _store_summary_memory() succeeds, the summary memory is orphaned in the vector store — the episode still points to the old summary.
Current behavior: Error is logged with orphan info but no cleanup.
Recommended fix: Add cleanup that deletes the orphaned summary memory from the vector store on metadata update failure, or implement a periodic orphan cleanup job.
Part of Episode Memories (#190).
From PR #196 review (Issue #1).
When
update_episode()ormark_memories_summarized()fails after_store_summary_memory()succeeds, the summary memory is orphaned in the vector store — the episode still points to the old summary.Current behavior: Error is logged with orphan info but no cleanup.
Recommended fix: Add cleanup that deletes the orphaned summary memory from the vector store on metadata update failure, or implement a periodic orphan cleanup job.
Part of Episode Memories (#190).