refactor: drop duplicate convenience tools (#25)

Author: julienld

README.md

@@ -330,12 +330,9 @@ claude mcp add-json home-assistant '{
 | `ha_config_get_automation` | Get automation configuration | `ha_config_get_automation("automation.id")` |
 | `ha_config_remove_automation` | Delete automations | `ha_config_remove_automation("automation.id")` |
 
-### Convenience Tools
+### History & Insights Tools
 | Tool | Description | Example |
 |------|-------------|---------|
-| `ha_activate_scene` | Activate a Home Assistant scene | `ha_activate_scene("scene.movie_time")` |
-| `ha_get_weather` | Get current weather information | `ha_get_weather()` |
-| `ha_get_energy` | Get energy usage information | `ha_get_energy()` |
 | `ha_get_logbook` | Access historical logbook entries | `ha_get_logbook(hours_back=24)` |
 
 ### Backup & Restore Tools
@@ -386,7 +383,7 @@ For comprehensive testing documentation, see **[tests/README.md](tests/README.md
 - [x] Smart search tools with fuzzy matching
 - [x] Optimized tool documentation to reduce tool call errors
 - [x] WebSocket async device control
-- [x] Convenience tools for scenes and automations
+- [x] Logbook history and operational insights
 - [x] Comprehensive test suite
 - [x] Home Assistant Add-on support
 - [x] Docker images with multi-mode support

homeassistant-addon/DOCS.md

@@ -286,10 +286,7 @@ The add-on provides 20+ MCP tools for controlling Home Assistant:
 - `ha_get_operation_status` - Check operation status
 - `ha_get_bulk_status` - Check multiple operations
 
-### Convenience
-- `ha_activate_scene` - Activate scenes
-- `ha_get_weather` - Weather information
-- `ha_get_energy` - Energy usage data
+### History & Insights
 - `ha_get_logbook` - Historical events
 
 ### Backup & Restore

src/ha_mcp/server.py

@@ -11,7 +11,6 @@
 from .config import get_global_settings
 from .prompts.enhanced import EnhancedPromptsMixin
 from .tools.enhanced import EnhancedToolsMixin
-from .tools.convenience import create_convenience_tools
 from .tools.device_control import create_device_control_tools
 from .tools.smart_search import create_smart_search_tools
 from .prompts.manager import register_mcp_prompts
@@ -37,7 +36,6 @@ def __init__(self, client: HomeAssistantClient | None = None):
         # Initialize smart tools
         self.smart_tools = create_smart_search_tools(self.client)
         self.device_tools = create_device_control_tools(self.client)
-        self.convenience_tools = create_convenience_tools(self.client)
 
         # Initialize tools registry
         self.tools_registry = ToolsRegistry(self)
@@ -111,7 +109,7 @@ async def start(self) -> None:
             logger.error(f"❌ Error testing connection: {e}")
 
         # Log available tools count
-        logger.info("🔧 Smart server with 20+ enhanced tools loaded")
+        logger.info("🔧 Smart server with enhanced tools loaded")
 
         # Run the MCP server with async compatibility
         await self.mcp.run_async()

src/ha_mcp/tools/__init__.py

@@ -1,8 +1,5 @@
-"""
-Custom tools for Home Assistant MCP server.
-"""
+"""Custom tools for the Home Assistant MCP server."""
 
-from .convenience import ConvenienceTools, create_convenience_tools
 from .device_control import DeviceControlTools, create_device_control_tools
 from .smart_search import SmartSearchTools, create_smart_search_tools
 
@@ -11,6 +8,4 @@
     "create_smart_search_tools",
     "DeviceControlTools",
     "create_device_control_tools",
-    "ConvenienceTools",
-    "create_convenience_tools",
 ]