Salastil/odysseus
1
0
Fork 0
mirror of https://github.com/pewdiepie-archdaemon/odysseus.git synced 2026-06-17 02:05:22 -04:00
Code Issues Packages Projects Releases Wiki Activity

Rebuild memory vector index from the full saved set, not just the audited owner (#1747)

Browse Source 
audit_memories saves final_entries merged with other owners' entries
(correct), but then rebuilt the shared vector collection from
final_entries alone — wiping every other owner from semantic search
until they happened to run their own audit. Keyword fallback masked
it, so it degraded silently. Capture saved_entries once and rebuild
from that.

Caught by #1747.
This commit is contained in:
pewdiepie-archdaemon
2026-06-03 11:36:24 +09:00
parent 8a10f271f7
commit 8e2b9baf19
1 changed files with 7 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
services/memory/memory_extractor.py
+7 -4
View File
@@ -547,17 +547,20 @@ async def audit_memories(
for e in all_entries: for e in all_entries:
if e.get("owner") is None and e["id"] not in audited_ids and e["id"] not in {o["id"] for o in other_entries}: if e.get("owner") is None and e["id"] not in audited_ids and e["id"] not in {o["id"] for o in other_entries}:
other_entries.append(e) other_entries.append(e)
memory_manager.save(final_entries + other_entries) saved_entries = final_entries + other_entries
else: else:
memory_manager.save(final_entries) saved_entries = final_entries
memory_manager.save(saved_entries)
logger.info( logger.info(
f"Memory audit complete: {before_count} -> {after_count} entries " f"Memory audit complete: {before_count} -> {after_count} entries "
f"({before_count - after_count} removed/merged)" f"({before_count - after_count} removed/merged)"
) )
# Rebuild vector index # Rebuild vector index from the full saved set, not just this owner's
# slice — otherwise the shared collection is wiped of every other
# owner's entries until they happen to run their own audit.
if memory_vector and memory_vector.healthy: if memory_vector and memory_vector.healthy:
memory_vector.rebuild(final_entries) memory_vector.rebuild(saved_entries)
# Persist the post-tidy fingerprint so the next call short-circuits # Persist the post-tidy fingerprint so the next call short-circuits
# if nothing has changed in the meantime. # if nothing has changed in the meantime.