Fixes

Signed-off-by: Alina Buzachis <[email protected]>

Author: alinabuzachis

plugins/connection/mcp.py

@@ -4,6 +4,7 @@
 
 
 DOCUMENTATION = r"""
+---
 name: mcp
 author:
     - Alina Buzachis (@alinabuzachis)
@@ -147,10 +148,6 @@ def __init__(self, play_context, new_stdin, *args, **kwargs):
         self._client: MCPClient | None = None
         self._connected = False
 
-        # Persistent options
-        self._timeout = self.get_option("persistent_command_timeout")
-        self._log_messages_enabled = self.get_option("persistent_log_messages")
-
     @property
     def connected(self) -> bool:
         """Return True if connected to MCP server."""

plugins/plugin_utils/mcp.py

@@ -17,16 +17,6 @@
 from ansible.module_utils.urls import open_url
 
 
-class MCPError(Exception):
-    """Base exception class for MCP related errors.
-
-    This exception is raised when MCP operations fail, such as initialization,
-    tool listing, tool execution, or validation errors.
-    """
-
-    pass
-
-
 class MCPError(Exception):
     """Base exception class for MCP related errors.
 

tests/unit/plugins/connection/test_mcp.py

@@ -66,6 +66,9 @@ def get_option(key):
         "mcp_bearer_token": "token123",
         "mcp_validate_certs": True,
         "mcp_manifest_path": str(manifest_file),
+        "persistent_connect_timeout": 15,
+        "persistent_command_timeout": 15,
+        "persistent_log_messages": False,
     }
     conn.get_option = get_option