Address PR review comments

1. Fix string consistency in FT.AGGREGATE query parameters - now all
   numeric parameters are explicitly converted to strings using str()

2. Handle tool messages and other unknown roles in memory_prompt API -
   treat non-user/assistant roles as assistant messages for MCP compatibility
   since MCP base only supports UserMessage and AssistantMessage types

Author: abrookins

agent_memory_server/api.py

@@ -637,8 +637,13 @@ async def memory_prompt(
             for msg in recent_messages:
                 if msg.role == "user":
                     msg_class = base.UserMessage
+                elif msg.role == "assistant":
+                    msg_class = base.AssistantMessage
                 else:
+                    # For tool messages or other roles, treat as assistant for MCP compatibility
+                    # since MCP base only supports UserMessage and AssistantMessage
                     msg_class = base.AssistantMessage
+
                 _messages.append(
                     msg_class(
                         content=TextContent(type="text", text=msg.content),

agent_memory_server/long_term_memory.py

@@ -329,21 +329,21 @@ async def compact_long_term_memories(
                 index_name,
                 filter_str,
                 "GROUPBY",
-                "1",
+                str(1),
                 "@memory_hash",
                 "REDUCE",
                 "COUNT",
-                "0",
+                str(0),
                 "AS",
                 "count",
                 "FILTER",
                 "@count>1",  # Only groups with more than 1 memory
                 "SORTBY",
-                "2",
+                str(2),
                 "@count",
                 "DESC",
                 "LIMIT",
-                "0",
+                str(0),
                 str(limit),
             ]