Update app.py

app.py

@@ -15,7 +15,7 @@ import sys
 import os
 import random
 from io import StringIO
-from typing import List, Dict, Tuple, Annotated, Literal
+from typing import List, Dict, Tuple, Annotated, Literal, Optional
 
 import gradio as gr
 import requests
@@ -1104,11 +1104,11 @@ kokoro_interface = gr.Interface(
 
 def Memory_Tool(
     action: Annotated[Literal["save","list","search","delete"], "Action to perform: save | list | search | delete"],
-    text: Annotated[str, "Text content (used when action=save)"] = "",
-    tags: Annotated[str, "Comma-separated tags (used when action=save)"] = "",
-    query: Annotated[str, "Search query terms (used when action=search)"] = "",
+    text: Annotated[Optional[str], "Text content (used when action=save)"] = None,
+    tags: Annotated[Optional[str], "Comma-separated tags (used when action=save)"] = None,
+    query: Annotated[Optional[str], "Search query terms (used when action=search)"] = None,
     limit: Annotated[int, "Max results for list/search (1–200)"] = 20,
-    memory_id: Annotated[str, "Full UUID or unique prefix (used when action=delete)"] = "",
+    memory_id: Annotated[Optional[str], "Full UUID or unique prefix (used when action=delete)"] = None,
     include_tags: Annotated[bool, "Include tags in list/search output"] = True,
 ) -> str:
     """Manage lightweight local JSON “memories” (save | list | search | delete) in one MCP tool.
@@ -1159,13 +1159,26 @@ def Memory_Tool(
         Plaintext JSON; do not store secrets, credentials, or regulated personal data.
     """
     act = (action or "").lower().strip()
+
+    # Normalize None -> "" for internal helpers
+    text = text or ""
+    tags = tags or ""
+    query = query or ""
+    memory_id = memory_id or ""
+
     if act == "save":
+        if not text.strip():
+            return "Error: 'text' is required when action=save."
         return _mem_save(text=text, tags=tags)
     if act == "list":
         return _mem_list(limit=limit, include_tags=include_tags)
     if act == "search":
+        if not query.strip():
+            return "Error: 'query' is required when action=search."
         return _mem_search(query=query, limit=limit)
     if act == "delete":
+        if not memory_id.strip():
+            return "Error: 'memory_id' is required when action=delete."
         return _mem_delete(memory_id=memory_id)
     return "Error: invalid action (use save|list|search|delete)."