Refactor tools integration and remove ToolsAdapter

- Deleted the ToolsAdapter class to streamline the tools integration process.
- Updated server and test files to directly utilize the new plugin system for tool registration and execution.
- Introduced a new YAML tools loader to handle dynamic tool implementations from YAML files.
- Adjusted tests to reflect changes in tool initialization and registration, ensuring compatibility with the new structure.
- Cleaned up obsolete entries in tools.yaml, marking deprecated tools as disabled.

Author: plusplusoneplusplus

mcp_core/tests/test_tools.py

@@ -18,8 +18,9 @@
 if parent_dir not in sys.path:
     sys.path.insert(0, parent_dir)
 
-# Import the ToolsAdapter
-from mcp_core.tools_adapter import ToolsAdapter
+# Import the direct tool system
+from mcp_tools.plugin import registry, discover_and_register_tools
+from mcp_tools.dependency import injector
 
 @pytest.fixture
 def yaml_data():
@@ -36,36 +37,39 @@ def yaml_data():
         return yaml.safe_load(f)
 
 @pytest.fixture
-def tools_adapter():
-    """Create a ToolsAdapter instance as a fixture"""
-    return ToolsAdapter()
+def tool_system():
+    """Initialize the tool system and return registry and injector"""
+    discover_and_register_tools()
+    injector.resolve_all_dependencies()
+    return {"registry": registry, "injector": injector}
 
 def test_tools_yaml_exists():
     """Test that tools.yaml exists in the server directory"""
     server_dir = Path(parent_dir) / "server"
     yaml_path = server_dir / "tools.yaml"
     assert yaml_path.exists(), f"tools.yaml not found at {yaml_path}"
 
-def test_tools_adapter_initialization(tools_adapter):
-    """Test that ToolsAdapter can be initialized"""
-    assert tools_adapter is not None
+def test_tool_system_initialization(tool_system):
+    """Test that the tool system can be initialized"""
+    assert tool_system["registry"] is not None
+    assert tool_system["injector"] is not None
 
     # Get all registered tools
-    tools = tools_adapter.get_tools()
+    tools = list(tool_system["injector"].instances.values())
     assert len(tools) > 0, "No tools were registered"
 
-def test_tools_from_yaml_are_registered(yaml_data, tools_adapter):
-    """Test that tools defined in tools.yaml are registered in the adapter"""
+def test_tools_from_yaml_are_registered(yaml_data, tool_system):
+    """Test that tools defined in tools.yaml are registered in the system"""
     # Get tools from yaml
     yaml_tool_names = yaml_data.get('tools', {}).keys()
     assert len(yaml_tool_names) > 0, "No tools found in tools.yaml"
 
     # Get registered tools
-    tools = tools_adapter.get_tools()
+    tools = list(tool_system["injector"].instances.values())
     registered_tool_names = [tool.name for tool in tools]
 
     # Verify at least some tools are registered
-    assert len(registered_tool_names) > 0, "No tools registered in the adapter"
+    assert len(registered_tool_names) > 0, "No tools registered in the system"
 
     # Count how many tools from yaml are registered
     registered_count = 0