From d8e611d9bda8b8decce290e0409fd318cff4ebd9 Mon Sep 17 00:00:00 2001 From: Sarah Wooders Date: Fri, 14 Mar 2025 09:43:03 -0700 Subject: [PATCH] feat: fix MCP-related logs format and add Docker tests (#1280) --- .github/workflows/docker-integration-tests.yaml | 3 ++- letta/server/server.py | 8 ++++---- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/.github/workflows/docker-integration-tests.yaml b/.github/workflows/docker-integration-tests.yaml index a6683446..77ddb3a0 100644 --- a/.github/workflows/docker-integration-tests.yaml +++ b/.github/workflows/docker-integration-tests.yaml @@ -32,7 +32,8 @@ jobs: LETTA_PG_PORT: 8888 OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }} - run: docker compose -f dev-compose.yaml up --build -d + run: | + docker compose -f dev-compose.yaml up --build -d #- name: "Setup Python, Poetry and Dependencies" # uses: packetcoders/action-setup-cache-python-poetry@v1.2.0 # with: diff --git a/letta/server/server.py b/letta/server/server.py index af6adbfb..65e01832 100644 --- a/letta/server/server.py +++ b/letta/server/server.py @@ -353,8 +353,8 @@ class SyncServer(Server): for server_name, client in self.mcp_clients.items(): logger.info(f"Attempting to fetch tools from MCP server: {server_name}") mcp_tools = client.list_tools() - logger.info(f"MCP tools connected: {", ".join([t.name for t in mcp_tools])}") - logger.debug(f"MCP tools: {"\n".join([str(t) for t in mcp_tools])}") + logger.info(f"MCP tools connected: {', '.join([t.name for t in mcp_tools])}") + logger.debug(f"MCP tools: {', '.join([str(t) for t in mcp_tools])}") def load_agent(self, agent_id: str, actor: User, interface: Union[AgentInterface, None] = None) -> Agent: """Updated method to load agents from persisted storage""" @@ -1366,8 +1366,8 @@ class SyncServer(Server): # Print out the tools that are connected logger.info(f"Attempting to fetch tools from MCP server: {server_config.server_name}") new_mcp_tools = new_mcp_client.list_tools() - logger.info(f"MCP tools connected: {", ".join([t.name for t in new_mcp_tools])}") - logger.debug(f"MCP tools: {"\n".join([str(t) for t in new_mcp_tools])}") + logger.info(f"MCP tools connected: {', '.join([t.name for t in new_mcp_tools])}") + logger.debug(f"MCP tools: {', '.join([str(t) for t in new_mcp_tools])}") # Now that we've confirmed the config is working, let's add it to the client list self.mcp_clients[server_config.server_name] = new_mcp_client