mcp-proxy-adapter 2.1.9__tar.gz → 2.1.10__tar.gz

{mcp_proxy_adapter-2.1.9/mcp_proxy_adapter.egg-info → mcp_proxy_adapter-2.1.10}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mcp-proxy-adapter
-Version: 2.1.9
+Version: 2.1.10
 Summary: Adapter for exposing Command Registry commands as tools for AI models via MCP Proxy.
 Home-page: https://github.com/vasilyvz/mcp-proxy-adapter
 Author: Vasiliy VZ

{mcp_proxy_adapter-2.1.9 → mcp_proxy_adapter-2.1.10}/mcp_proxy_adapter/adapter.py

@@ -493,13 +493,6 @@ class MCPProxyAdapter:
                             "details": "Request requires 'command', 'method' or 'params' field"
                         }
                     }
-                # Подмена help -> __help
-                if command == "help":
-                    command = "__help"
-                    # Переименовываем параметр внутри params
-                    if "command" in params:
-                        params["cmdname"] = params.pop("command")
-                logger.info(f"[DEBUG] MCP CMD: command={command}, params={params}")
                 
                 # Check if command exists
                 if command not in self.registry.dispatcher.get_valid_commands():
@@ -508,14 +501,12 @@ class MCPProxyAdapter:
                         "error": {
                             "code": 404,
                             "message": f"Unknown command: {command}",
-                            "details": f"Unknown command: {command}. Available commands: {', '.join(self.registry.dispatcher.get_valid_commands())}"
+                            "details": f"Command '{command}' not found in registry. Available commands: {', '.join(self.registry.dispatcher.get_valid_commands())}"
                         }
                     }
                 
-                logger.info(f"[DEBUG] MCP CMD: command={command}, params={params}")
                 # Check for required parameters
                 command_info = self.registry.dispatcher.get_command_info(command)
-                logger.info(f"[DEBUG] MCP CMD: command_info={command_info}")
                 if command_info and "params" in command_info:
                     missing_params = []
                     for param_name, param_info in command_info["params"].items():
@@ -531,7 +522,7 @@ class MCPProxyAdapter:
                                 "details": f"Command '{command}' requires following parameters: {', '.join(missing_params)}"
                             }
                         }
-                logger.info(f"[DEBUG] MCP CMD: command={command}, params={params}")
+                
                 # Check parameter types
                 type_errors = self._validate_param_types(command, params)
                 if type_errors:
@@ -543,10 +534,9 @@ class MCPProxyAdapter:
                             "details": "Check parameter types and try again"
                         }
                     }
-                logger.info(f"[DEBUG] MCP CMD: command={command}, params={params}")
+                
                 # Execute the command
                 try:
-                    logger.info(f"[DEBUG] MCP CMD: executing command={command}, params={params}")
                     result = self.registry.dispatcher.execute(command, **params)
                     
                     # Return result in MCP Proxy format

{mcp_proxy_adapter-2.1.9 → mcp_proxy_adapter-2.1.10}/mcp_proxy_adapter/dispatchers/json_rpc_dispatcher.py

@@ -41,7 +41,7 @@ class JsonRpcDispatcher(BaseDispatcher):
             description="Returns information about available commands",
             summary="Command help",
             params={
-                "cmdname": {
+                "command": {
                     "type": "string",
                     "description": "Command name for detailed information",
                     "required": False
@@ -157,26 +157,31 @@ class JsonRpcDispatcher(BaseDispatcher):
     def _help_command(self, params: Dict[str, Any] = None) -> Dict[str, Any]:
         """
         Built-in help command for getting command information.
+        
         Args:
-            params: Command parameters (dict)
-                cmdname: Command name for detailed information
+            params: Command parameters
+                command: Command name for detailed information
+                
         Returns:
             Dict[str, Any]: Command help information
         """
-        if params is None:
+        if not params:
             params = {}
+            
         # If specific command is specified, return information only about it
-        cmdname = params.get("cmdname")
-        if cmdname:
-            if cmdname not in self._metadata:
+        if "command" in params and params["command"]:
+            command = params["command"]
+            if command not in self._metadata:
                 return {
-                    "error": f"Command '{cmdname}' not found",
+                    "error": f"Command '{command}' not found",
                     "available_commands": list(self._metadata.keys())
                 }
+            
             return {
-                "cmdname": cmdname,
-                "info": self._metadata[cmdname]
+                "command": command,
+                "info": self._metadata[command]
             }
+        
         # Otherwise return brief information about all commands
         commands_info = {}
         for cmd, info in self._metadata.items():
@@ -185,8 +190,9 @@ class JsonRpcDispatcher(BaseDispatcher):
                 "description": info["description"],
                 "params_count": len(info["params"])
             }
+        
         return {
             "commands": commands_info,
             "total": len(commands_info),
-            "note": "Use the 'cmdname' parameter to get detailed information about a specific command"
+            "note": "Use the 'command' parameter to get detailed information about a specific command"
         } 

{mcp_proxy_adapter-2.1.9 → mcp_proxy_adapter-2.1.10}/mcp_proxy_adapter/examples/help_best_practices.py

@@ -12,10 +12,10 @@ Run:
 """
 import os
 import sys
-sys.path.insert(0, os.path.abspath(os.path.dirname(os.path.dirname(__file__))))
+sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '../..')))
 from typing import Any, Dict
 from mcp_proxy_adapter.adapter import MCPProxyAdapter
-from tests.test_mcp_proxy_adapter import MockRegistry
+from mcp_proxy_adapter.testing_utils import MockRegistry
 
 # --- Setup registry and adapter ---
 registry = MockRegistry()

{mcp_proxy_adapter-2.1.9 → mcp_proxy_adapter-2.1.10}/mcp_proxy_adapter/examples/help_usage.py

@@ -12,12 +12,12 @@ Run:
 """
 import os
 import sys
-sys.path.insert(0, os.path.abspath(os.path.dirname(os.path.dirname(__file__))))
+sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '../..')))
 from typing import Any, Dict
 
 # Assume MCPProxyAdapter and MockRegistry are available from src and tests
 from mcp_proxy_adapter.adapter import MCPProxyAdapter
-from tests.test_mcp_proxy_adapter import MockRegistry
+from mcp_proxy_adapter.testing_utils import MockRegistry
 
 # --- Setup registry and adapter ---
 registry = MockRegistry()

{mcp_proxy_adapter-2.1.9 → mcp_proxy_adapter-2.1.10/mcp_proxy_adapter.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mcp-proxy-adapter
-Version: 2.1.9
+Version: 2.1.10
 Summary: Adapter for exposing Command Registry commands as tools for AI models via MCP Proxy.
 Home-page: https://github.com/vasilyvz/mcp-proxy-adapter
 Author: Vasiliy VZ

{mcp_proxy_adapter-2.1.9 → mcp_proxy_adapter-2.1.10}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "mcp-proxy-adapter"
-version = "2.1.9"
+version = "2.1.10"
 description = "Adapter for exposing Command Registry commands as tools for AI models via MCP Proxy."
 readme = "README.md"
 requires-python = ">=3.9"

{mcp_proxy_adapter-2.1.9 → mcp_proxy_adapter-2.1.10}/setup.py

@@ -14,7 +14,7 @@ with open(os.path.join(here, 'requirements.txt'), encoding='utf-8') as f:
 
 setup(
     name="mcp-proxy-adapter",
-    version="2.1.9",
+    version="2.1.10",
     description="Adapter for exposing Command Registry commands as tools for AI models via MCP Proxy.",
     long_description=long_description,
     long_description_content_type="text/markdown",

{mcp_proxy_adapter-2.1.9 → mcp_proxy_adapter-2.1.10}/tests/test_adapter.py

@@ -29,8 +29,6 @@ except ImportError:
     from src.adapter import MCPProxyAdapter, configure_logger
     from src.registry import CommandRegistry
 
-print('LOADED test_adapter.py')
-
 # Test command functions
 def success_command(value: int = 1) -> dict:
     """Test command that completes successfully."""
@@ -65,7 +63,7 @@ class BaseDispatcher:
         raise NotImplementedError("Method must be overridden in subclass")
 
 # Mock for command dispatcher
-class MockDispatcherXXX(BaseDispatcher):
+class MockDispatcher(BaseDispatcher):
     """Mock for command dispatcher in tests."""
     
     def __init__(self):
@@ -73,8 +71,7 @@ class MockDispatcherXXX(BaseDispatcher):
             "success": success_command,
             "error": error_command,
             "param": param_command,
-            "execute": self.execute_from_params,
-            "__help": self.help_command
+            "execute": self.execute_from_params
         }
         self.commands_info = {
             "success": {
@@ -117,16 +114,6 @@ class MockDispatcherXXX(BaseDispatcher):
                         "required": False
                     }
                 }
-            },
-            "__help": {
-                "description": "Show help for commands",
-                "params": {
-                    "cmdname": {
-                        "type": "string",
-                        "description": "Command name for detailed info",
-                        "required": False
-                    }
-                }
             }
         }
     
@@ -134,8 +121,6 @@ class MockDispatcherXXX(BaseDispatcher):
         """Executes command based on parameters."""
         if "query" in params and params["query"] in self.commands:
             command = params.pop("query")
-            if command == "help":
-                command = "__help"
             return self.execute(command, **params)
         return {
             "available_commands": self.get_valid_commands(),
@@ -143,48 +128,25 @@ class MockDispatcherXXX(BaseDispatcher):
         }
     
     def execute(self, command, **params):
-        print(f"[DEBUG] execute called: command={command}, params={params}")
-        if command == "help":
-            command = "__help"
         if command not in self.commands:
             raise KeyError(f"Unknown command: {command}")
-        if command == "__help":
-            return self.commands[command](params=params)
         return self.commands[command](**params)
     
     def get_valid_commands(self):
         return list(self.commands.keys())
     
     def get_command_info(self, command):
-        if command == "help":
-            command = "__help"
         return self.commands_info.get(command)
     
     def get_commands_info(self):
         return self.commands_info
 
-    def help_command(self, params=None):
-        import traceback; print(f"[DEBUG] help_command called with params: {params}"); print(f"[DEBUG] commands_info: {self.commands_info}"); traceback.print_stack()
-        if params is None:
-            params = {}
-        cmdname = params.get("cmdname")
-        if cmdname:
-            if cmdname in self.commands_info:
-                return {"cmdname": cmdname, "info": self.commands_info[cmdname]}
-            else:
-                return {"error": f"Command '{cmdname}' not found", "available_commands": list(self.commands_info.keys())}
-        return {
-            "commands": {cmd: {"description": info["description"], "params": info["params"]} for cmd, info in self.commands_info.items()},
-            "total": len(self.commands_info),
-            "note": "Use the 'cmdname' parameter to get detailed information about a specific command"
-        }
-
 # Mock for CommandRegistry
 class MockRegistry:
     """Mock for CommandRegistry in tests."""
     
     def __init__(self):
-        self.dispatcher = MockDispatcherXXX()
+        self.dispatcher = MockDispatcher()
         self.generators = []
     
     def get_commands_info(self):
@@ -457,9 +419,9 @@ def test_save_config_to_file(adapter, tmp_path):
     
     assert "version" in config
     assert "tools" in config
-    assert len(config["tools"]) == 5  # Теперь 5 команд (включая help)
+    assert len(config["tools"]) == 4  # Corrected: now we have 4 commands
     assert "routes" in config
-    assert len(config["routes"]) == 5  # Теперь 5 маршрутов
+    assert len(config["routes"]) == 4  # Corrected: now we have 4 routes, corresponding to commands
 
 def test_parameter_type_validation(test_app):
     """Test parameter type validation."""
@@ -564,62 +526,4 @@ def test_params_only_format(test_app):
     data = response.json()
     assert "result" in data
     assert "available_commands" in data["result"]
-    assert "received_params" in data["result"]
-
-def test_jsonrpc_proxy_and_help_conflict(test_app):
-    """Test JSON-RPC proxying and help/command param conflict."""
-    # Обычный JSON-RPC вызов существующей команды
-    response = test_app.post("/cmd", json={
-        "jsonrpc": "2.0",
-        "method": "success",
-        "params": {"value": 3},
-        "id": 42
-    })
-    assert response.status_code == 200
-    data = response.json()
-    assert "result" in data
-    assert data["result"] == {"result": 6}
-
-    # Вызов help по конкретной команде (не должно быть multiple values for argument 'command')
-    response = test_app.post("/cmd", json={
-        "jsonrpc": "2.0",
-        "method": "help",
-        "params": {"cmdname": "success"},
-        "id": 43
-    })
-    assert response.status_code == 200
-    data = response.json()
-    assert "result" in data
-    assert data["result"]["cmdname"] == "success"
-    assert "info" in data["result"]
-    assert "description" in data["result"]["info"]
-
-    # Вызов help по несуществующей команде
-    response = test_app.post("/cmd", json={
-        "jsonrpc": "2.0",
-        "method": "help",
-        "params": {"cmdname": "not_a_command"},
-        "id": 44
-    })
-    assert response.status_code == 200
-    data = response.json()
-    assert "result" in data
-    # Ошибка может быть либо в data['result']['error'], либо в data['error']['message']
-    found = False
-    if "error" in data["result"] and "not found" in data["result"]["error"]:
-        found = True
-    if "error" in data and "message" in data["error"] and "not found" in data["error"]["message"]:
-        found = True
-    assert found, f"No 'not found' in error: {data}"
-
-    # Вызов несуществующей команды
-    response = test_app.post("/cmd", json={
-        "jsonrpc": "2.0",
-        "method": "not_a_command",
-        "params": {},
-        "id": 45
-    })
-    assert response.status_code == 200
-    data = response.json()
-    assert "error" in data
-    assert "Unknown command" in data["error"]["message"] 
+    assert "received_params" in data["result"] 

{mcp_proxy_adapter-2.1.9 → mcp_proxy_adapter-2.1.10}/tests/test_examples.py

@@ -3,7 +3,7 @@ import sys
 import os
 import pytest
 
-EXAMPLES_DIR = os.path.join(os.path.dirname(__file__), "..", "examples")
+EXAMPLES_DIR = os.path.join(os.path.dirname(__file__), "..", "mcp_proxy_adapter", "examples")
 
 @pytest.mark.parametrize("script,expected", [
     ("help_usage.py", ["Project help", "Adapter help"]),