From a85d432b544060795ff57796d713c597122f6293 Mon Sep 17 00:00:00 2001
From: Oleksii Pylypchuk <alpi@keemail.me>
Date: Sat, 18 Apr 2026 22:05:34 +0300
Subject: [PATCH] feat: add validation for missing name parameter in tools/call
 requests

---
 mempalace/mcp_server.py  |  6 ++++++
 tests/test_mcp_server.py | 14 ++++++++++++++
 2 files changed, 20 insertions(+)

diff --git a/mempalace/mcp_server.py b/mempalace/mcp_server.py
index 227e91a..25439f2 100644
--- a/mempalace/mcp_server.py
+++ b/mempalace/mcp_server.py
@@ -2007,6 +2007,12 @@ def handle_request(request):
             },
         }
     elif method == "tools/call":
+        if not isinstance(params, dict) or "name" not in params:
+            return {
+                "jsonrpc": "2.0",
+                "id": req_id,
+                "error": {"code": -32602, "message": "Invalid params: 'name' is required for tools/call"},
+            }
         tool_name = params.get("name")
         tool_args = params.get("arguments") or {}
         if tool_name not in TOOLS:
diff --git a/tests/test_mcp_server.py b/tests/test_mcp_server.py
index 0a020d3..fc56b34 100644
--- a/tests/test_mcp_server.py
+++ b/tests/test_mcp_server.py
@@ -148,6 +148,20 @@ class TestHandleRequest:
         )
         assert resp["error"]["code"] == -32601
 
+    def test_tools_call_missing_params(self):
+        from mempalace.mcp_server import handle_request
+
+        for bad_params in [None, {}, {"arguments": {}}]:
+            resp = handle_request(
+                {
+                    "method": "tools/call",
+                    "id": 15,
+                    "params": bad_params,
+                }
+            )
+            assert resp["error"]["code"] == -32602
+            assert "Invalid params" in resp["error"]["message"]
+
     def test_unknown_method(self):
         from mempalace.mcp_server import handle_request