From 5229e98ff3fa23887652464c49883a38a4391f82 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Peter=20St=C3=B6ckli?= Date: Fri, 24 Oct 2025 06:00:08 +0000 Subject: [PATCH 1/2] remove unecessary pass statements --- main.py | 1 - mcp_servers/logbook/logbook.py | 1 - mcp_servers/memcache/memcache_backend/dictionary_file.py | 1 - mcp_utils.py | 2 -- 4 files changed, 5 deletions(-) diff --git a/main.py b/main.py index 02bdfcf..8b289cd 100644 --- a/main.py +++ b/main.py @@ -229,7 +229,6 @@ async def mcp_session_task( logging.error(f"RuntimeError in mcp session task: {e}") except asyncio.CancelledError as e: logging.error(f"Timeout on main session task: {e}") - pass finally: mcp_servers.clear() diff --git a/mcp_servers/logbook/logbook.py b/mcp_servers/logbook/logbook.py index 199051e..c4ad597 100644 --- a/mcp_servers/logbook/logbook.py +++ b/mcp_servers/logbook/logbook.py @@ -26,7 +26,6 @@ def ensure_log(): with open(LOGBOOK, 'x') as logbook: logbook.write(json.dumps(LOG, indent=2)) logbook.flush() - pass except FileExistsError: pass diff --git a/mcp_servers/memcache/memcache_backend/dictionary_file.py b/mcp_servers/memcache/memcache_backend/dictionary_file.py index 8513f52..3238a43 100644 --- a/mcp_servers/memcache/memcache_backend/dictionary_file.py +++ b/mcp_servers/memcache/memcache_backend/dictionary_file.py @@ -16,7 +16,6 @@ def _ensure_memory(self): with open(self.memory, 'x') as memory: memory.write(json.dumps(self.memcache)) memory.flush() - pass except FileExistsError: pass def _deflate_memory(self): diff --git a/mcp_utils.py b/mcp_utils.py index c1fb40e..26bf4a5 100644 --- a/mcp_utils.py +++ b/mcp_utils.py @@ -309,7 +309,6 @@ def mcp_client_params(available_toolboxes: dict, requested_toolboxes: list): logging.critical(e) logging.info("Assuming toolbox has default configuration available") del env[k] - pass logging.debug(f"Tool call environment: {env}") if args and isinstance(args, list): for i, v in enumerate(args): @@ -396,7 +395,6 @@ def mcp_client_params(available_toolboxes: dict, requested_toolboxes: list): logging.critical(e) logging.info("Assuming toolbox has default configuration available") del env[k] - pass server_params['env'] = env case _: raise ValueError(f"Unsupported MCP transport {kind}") From 0940f41737dd1e0d6d8112b873f79555d1b88504 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Peter=20St=C3=B6ckli?= Date: Fri, 24 Oct 2025 10:03:43 +0000 Subject: [PATCH 2/2] leave main.py alone --- main.py | 1 + 1 file changed, 1 insertion(+) diff --git a/main.py b/main.py index 8b289cd..02bdfcf 100644 --- a/main.py +++ b/main.py @@ -229,6 +229,7 @@ async def mcp_session_task( logging.error(f"RuntimeError in mcp session task: {e}") except asyncio.CancelledError as e: logging.error(f"Timeout on main session task: {e}") + pass finally: mcp_servers.clear()