From 8a5c0729e58c308066e3a9e4b4caa6dee3e81928 Mon Sep 17 00:00:00 2001 From: Kushagra Gupta Date: Thu, 2 Jul 2026 18:33:16 +0530 Subject: [PATCH] fix(server): do not forward empty-string entity ids as filters (#5992) Co-authored-by: Kartik --- server/main.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/server/main.py b/server/main.py index 5819d1192..4f4e5235c 100644 --- a/server/main.py +++ b/server/main.py @@ -426,7 +426,7 @@ def get_all_memories( # Admin all-memory listing is intentionally raw; scoped get_all below applies expiry visibility. return _list_all_memories(limit=top_k if top_k is not None else ALL_MEMORIES_LIMIT) filters = { - k: v for k, v in {"user_id": user_id, "run_id": run_id, "agent_id": agent_id}.items() if v is not None + k: v for k, v in {"user_id": user_id, "run_id": run_id, "agent_id": agent_id}.items() if v } params = {"filters": filters} if top_k is not None: @@ -456,7 +456,7 @@ def search_memories(search_req: SearchRequest, _auth=Depends(verify_auth)): deprecated_keys = [] for entity_key in ("user_id", "agent_id", "run_id"): entity_val = getattr(search_req, entity_key, None) - if entity_val is not None: + if entity_val: filters[entity_key] = entity_val deprecated_keys.append(entity_key) if deprecated_keys: @@ -535,7 +535,7 @@ def delete_all_memories( raise HTTPException(status_code=400, detail="At least one identifier is required.") try: params = { - k: v for k, v in {"user_id": user_id, "run_id": run_id, "agent_id": agent_id}.items() if v is not None + k: v for k, v in {"user_id": user_id, "run_id": run_id, "agent_id": agent_id}.items() if v } get_memory_instance().delete_all(**params) return MessageResponse(message="All relevant memories deleted")