From 817221347f7b5b34bf54c5be581719388bc69267 Mon Sep 17 00:00:00 2001
From: lanceyq <1982376970@qq.com>
Date: Wed, 4 Mar 2026 17:57:58 +0800
Subject: [PATCH] [fix] Preserve full result dict and default status to
 "unknown" instead of "success".

---
 api/app/services/memory_api_service.py | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/api/app/services/memory_api_service.py b/api/app/services/memory_api_service.py
index ad0a8164..f86fbed8 100644
--- a/api/app/services/memory_api_service.py
+++ b/api/app/services/memory_api_service.py
@@ -154,13 +154,17 @@ class MemoryAPIService:
             logger.info(f"Memory write successful for end_user: {end_user_id}")
             
             # result may be a string "success" or a dict with a "status" key
+            # Preserve the full dict so callers don't silently lose extra fields
+            # (e.g. error codes, metadata) returned by MemoryAgentService.
             if isinstance(result, dict):
-                status = result.get("status", "success")
-            else:
-                status = result if isinstance(result, str) else "success"
+                return {
+                    **result,
+                    "status": result.get("status", "unknown"),
+                    "end_user_id": end_user_id,
+                }
             return {
-                "status": status,
-                "end_user_id": end_user_id
+                "status": result if isinstance(result, str) else "success",
+                "end_user_id": end_user_id,
             }
             
         except ConfigurationError as e: