devpilot-agentic-cli 1.0.0__tar.gz → 1.0.2__tar.gz

{devpilot_agentic_cli-1.0.0 → devpilot_agentic_cli-1.0.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: devpilot-agentic-cli
-Version: 1.0.0
+Version: 1.0.2
 Summary: Autonomous AI coding agent for your terminal — Claude, GPT-4o, Groq, Ollama, and more
 Author: Thijesh Praveen V
 License: MIT

{devpilot_agentic_cli-1.0.0 → devpilot_agentic_cli-1.0.2}/agent/cli.py

@@ -161,14 +161,29 @@ async def main_async() -> None:
 
     if config.a2a_enabled:
         import uvicorn
-        a2a_cfg = uvicorn.Config(
-            app=a2a_app,
-            host="0.0.0.0",
-            port=config.a2a_port,
-            log_level="error",
-        )
-        a2a_server = uvicorn.Server(a2a_cfg)
-        a2a_task   = asyncio.create_task(a2a_server.serve())
+        import socket
+        
+        # Check if port is available
+        port_in_use = False
+        with socket.socket(socket.AF_INET, socket.SOCK_STREAM) as s:
+            try:
+                s.bind(("0.0.0.0", config.a2a_port))
+            except OSError:
+                port_in_use = True
+                
+        if port_in_use:
+            UI.print_error(f"Port {config.a2a_port} is already in use. A2A server disabled. Use --a2a-port <port> to change.")
+            a2a_server = None
+            a2a_task = None
+        else:
+            a2a_cfg = uvicorn.Config(
+                app=a2a_app,
+                host="0.0.0.0",
+                port=config.a2a_port,
+                log_level="error",
+            )
+            a2a_server = uvicorn.Server(a2a_cfg)
+            a2a_task   = asyncio.create_task(a2a_server.serve())
     else:
         a2a_server = None
         a2a_task   = None
@@ -217,12 +232,13 @@ async def main_async() -> None:
         config=config,
         repo_context=repo_context,
     )
-    await app.run_async()
-
-    if a2a_server and a2a_task:
-        a2a_server.should_exit = True
-        await a2a_task
-    await mcp_manager.close()
+    try:
+        await app.run_async()
+    finally:
+        if a2a_server and a2a_task:
+            a2a_server.should_exit = True
+            await a2a_task
+        await mcp_manager.close()
 
 
 def main() -> None:

devpilot_agentic_cli-1.0.2/agent/mcp_client.py

@@ -0,0 +1,130 @@
+"""
+agent/mcp_client.py
+───────────────────
+MCP Client Integration (Sprint 3).
+Connects to servers defined in mcp_servers.json, discovers tools,
+and registers them into the ToolRegistry.
+"""
+
+import json
+import asyncio
+from pathlib import Path
+
+from mcp.client.session import ClientSession
+from mcp.client.stdio import StdioServerParameters, stdio_client
+from mcp.types import TextContent
+
+from agent.tools import ToolRegistry, ToolResult
+from agent.ui import UI
+
+
+class MCPManager:
+    """Manages connections to multiple MCP servers."""
+
+    def __init__(self, config_path: Path):
+        self.config_path = config_path
+        self.sessions: dict[str, ClientSession] = {}
+        self._tasks: list[asyncio.Task] = []
+
+    async def _run_server(
+        self, 
+        name: str, 
+        command: str, 
+        args: list[str], 
+        env: dict | None, 
+        registry: ToolRegistry, 
+        ready_event: asyncio.Event
+    ) -> None:
+        try:
+            server_params = StdioServerParameters(command=command, args=args, env=env)
+            async with stdio_client(server_params) as (read, write):
+                async with ClientSession(read, write) as session:
+                    await session.initialize()
+                    self.sessions[name] = session
+
+                    # Fetch and register tools
+                    tools_response = await session.list_tools()
+                    for mcp_tool in tools_response.tools:
+                        canonical_schema = {
+                            "name": mcp_tool.name,
+                            "description": mcp_tool.description or "",
+                            "input_schema": mcp_tool.inputSchema,
+                            "_mcp_server_id": name,
+                        }
+
+                        def make_executor(session_ref: ClientSession, tool_name: str):
+                            async def _executor(tool_input: dict) -> ToolResult:
+                                try:
+                                    result = await session_ref.call_tool(tool_name, tool_input)
+                                    text_contents = [c.text for c in result.content if isinstance(c, TextContent)]
+                                    output = "\n".join(text_contents)
+                                    return ToolResult(output, is_error=result.isError)
+                                except Exception as e:
+                                    return ToolResult(f"MCP execution error: {e}", is_error=True)
+                            return _executor
+
+                        registry.register_mcp_tool(canonical_schema, make_executor(session, mcp_tool.name))
+
+                    UI.print_info(f"Connected to MCP server: {name} ({len(tools_response.tools)} tools)")
+                    ready_event.set()
+
+                    try:
+                        await asyncio.Event().wait()
+                    except asyncio.CancelledError:
+                        pass
+        except Exception as e:
+            UI.print_error(f"Failed to connect to MCP server '{name}': {e}")
+            registry.deregister_mcp_tools(name)
+        finally:
+            ready_event.set()
+
+    async def connect_all(self, registry: ToolRegistry) -> None:
+        """Connect to all servers in mcp_servers.json and register tools."""
+        if not self.config_path.exists():
+            return
+
+        try:
+            with open(self.config_path, "r", encoding="utf-8") as f:
+                data = json.load(f)
+                servers = data.get("mcpServers", data.get("servers", {}))
+        except (json.JSONDecodeError, OSError) as e:
+            UI.print_error(f"Failed to read mcp_servers.json: {e}")
+            return
+
+        if isinstance(servers, dict):
+            server_items = servers.items()
+        else:
+            server_items = [(s.get("name", f"server_{i}"), s) for i, s in enumerate(servers)]
+
+        events = []
+        for name, server_config in server_items:
+            if server_config.get("enabled", True) is False:
+                continue
+            
+            command = server_config.get("command")
+            args = server_config.get("args", [])
+
+            if not command:
+                UI.print_error(f"MCP server '{name}' missing 'command'. Skipping.")
+                continue
+
+            ready_event = asyncio.Event()
+            events.append(ready_event)
+            task = asyncio.create_task(
+                self._run_server(name, command, args, server_config.get("env"), registry, ready_event)
+            )
+            self._tasks.append(task)
+
+        if events:
+            await asyncio.gather(*(e.wait() for e in events))
+
+    async def close(self) -> None:
+        """Close all connections."""
+        for task in self._tasks:
+            task.cancel()
+        
+        if self._tasks:
+            await asyncio.gather(*self._tasks, return_exceptions=True)
+            
+        self.sessions.clear()
+        self._tasks.clear()

{devpilot_agentic_cli-1.0.0 → devpilot_agentic_cli-1.0.2}/agent/ui.py

@@ -171,7 +171,7 @@ class UI:
         if UI._tui_app:
             UI._tui_app.post_message(ErrorEvent(msg))
             return
-        console.print(f"[danger]❌ {msg}[/danger]")
+        console.print(f"[danger]Error: {msg}[/danger]")
 
     @staticmethod
     def print_info(msg: str) -> None:
@@ -179,7 +179,7 @@ class UI:
         if UI._tui_app:
             UI._tui_app.post_message(InfoEvent(msg))
             return
-        console.print(f"[info]ℹ️ {msg}[/info]")
+        console.print(f"[info]Info: {msg}[/info]")
 
     @staticmethod
     def print_success(msg: str) -> None:
@@ -187,7 +187,7 @@ class UI:
         if UI._tui_app:
             UI._tui_app.post_message(SuccessEvent(msg))
             return
-        console.print(f"[bold green]✓ {msg}[/bold green]")
+        console.print(f"[bold green]Success: {msg}[/bold green]")
 
     @staticmethod
     def print_thinking_block(thinking_text: str) -> None:

{devpilot_agentic_cli-1.0.0 → devpilot_agentic_cli-1.0.2}/devpilot_agentic_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: devpilot-agentic-cli
-Version: 1.0.0
+Version: 1.0.2
 Summary: Autonomous AI coding agent for your terminal — Claude, GPT-4o, Groq, Ollama, and more
 Author: Thijesh Praveen V
 License: MIT

{devpilot_agentic_cli-1.0.0 → devpilot_agentic_cli-1.0.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "devpilot-agentic-cli"
-version = "1.0.0"
+version = "1.0.2"
 description = "Autonomous AI coding agent for your terminal — Claude, GPT-4o, Groq, Ollama, and more"
 authors = [{ name = "Thijesh Praveen V" }]
 requires-python = ">=3.11"

devpilot_agentic_cli-1.0.0/agent/mcp_client.py

@@ -1,104 +0,0 @@
-"""
-agent/mcp_client.py
-───────────────────
-MCP Client Integration (Sprint 3).
-Connects to servers defined in mcp_servers.json, discovers tools,
-and registers them into the ToolRegistry.
-"""
-
-import json
-from contextlib import AsyncExitStack
-from pathlib import Path
-
-from mcp.client.session import ClientSession
-from mcp.client.stdio import StdioServerParameters, stdio_client
-from mcp.types import TextContent
-
-from agent.tools import ToolRegistry, ToolResult
-from agent.ui import UI
-
-
-class MCPManager:
-    """Manages connections to multiple MCP servers."""
-
-    def __init__(self, config_path: Path):
-        self.config_path = config_path
-        self.exit_stack = AsyncExitStack()
-        self.sessions: dict[str, ClientSession] = {}
-
-    async def connect_all(self, registry: ToolRegistry) -> None:
-        """Connect to all servers in mcp_servers.json and register tools."""
-        if not self.config_path.exists():
-            return
-
-        try:
-            with open(self.config_path, "r", encoding="utf-8") as f:
-                data = json.load(f)
-                servers = data.get("mcpServers", data.get("servers", {}))
-        except (json.JSONDecodeError, OSError) as e:
-            UI.print_error(f"Failed to read mcp_servers.json: {e}")
-            return
-
-        # Handle both list of dicts and dict of dicts formats for mcp_servers.json
-        if isinstance(servers, dict):
-            # In official MCP config format, it's a dict mapping name to config
-            server_items = servers.items()
-        else:
-            # Fallback if it's a list
-            server_items = [(s.get("name", f"server_{i}"), s) for i, s in enumerate(servers)]
-
-        for name, server_config in server_items:
-            if server_config.get("enabled", True) is False:
-                continue
-            
-            command = server_config.get("command")
-            args = server_config.get("args", [])
-
-            if not command:
-                UI.print_error(f"MCP server '{name}' missing 'command'. Skipping.")
-                continue
-
-            try:
-                server_params = StdioServerParameters(command=command, args=args, env=server_config.get("env"))
-                stdio_transport = await self.exit_stack.enter_async_context(stdio_client(server_params))
-                read, write = stdio_transport
-                session = await self.exit_stack.enter_async_context(ClientSession(read, write))
-                await session.initialize()
-
-                self.sessions[name] = session
-
-                # Fetch and register tools
-                tools_response = await session.list_tools()
-                for mcp_tool in tools_response.tools:
-                    # Convert to canonical schema format
-                    canonical_schema = {
-                        "name": mcp_tool.name,
-                        "description": mcp_tool.description or "",
-                        "input_schema": mcp_tool.inputSchema,
-                        "_mcp_server_id": name,
-                    }
-
-                    # Create closure for execution
-                    def make_executor(session_ref: ClientSession, tool_name: str):
-                        async def _executor(tool_input: dict) -> ToolResult:
-                            try:
-                                result = await session_ref.call_tool(tool_name, tool_input)
-                                # Flatten result text
-                                text_contents = [c.text for c in result.content if isinstance(c, TextContent)]
-                                output = "\n".join(text_contents)
-                                return ToolResult(output, is_error=result.isError)
-                            except Exception as e:
-                                return ToolResult(f"MCP execution error: {e}", is_error=True)
-                        return _executor
-
-                    registry.register_mcp_tool(canonical_schema, make_executor(session, mcp_tool.name))
-
-                UI.print_info(f"Connected to MCP server: {name} ({len(tools_response.tools)} tools)")
-            except Exception as e:
-                UI.print_error(f"Failed to connect to MCP server '{name}': {e}")
-                registry.deregister_mcp_tools(name)
-
-    async def close(self) -> None:
-        """Close all connections."""
-        await self.exit_stack.aclose()
-        self.sessions.clear()