selfhostyourtech/cubbi
2
0
Fork 0
mirror of https://github.com/Monadical-SAS/cubbi.git synced 2025-12-20 20:29:06 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(mcp): fix UnboundLocalError: cannot access local variable 'container_name' where it is not associated with a value

Browse Source
This commit is contained in:
Mathieu Virbel
2025-04-01 19:00:34 -06:00
parent 150a6fe39c
commit a7630ace07
1 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
mcontainer/cli.py
View File

@@ -1037,6 +1037,9 @@ def start_mcp(
console.print(f"Starting {len(mcps)} MCP servers...") console.print(f"Starting {len(mcps)} MCP servers...")
# Keep track of MCP container names that were successfully started
mcp_container_names = []
for mcp in mcps: for mcp in mcps:
mcp_name = mcp.get("name") mcp_name = mcp.get("name")
if not mcp_name: if not mcp_name:
@@ -1045,6 +1048,8 @@ def start_mcp(
try: try:
with console.status(f"Starting MCP server '{mcp_name}'..."): with console.status(f"Starting MCP server '{mcp_name}'..."):
result = mcp_manager.start_mcp(mcp_name) result = mcp_manager.start_mcp(mcp_name)
container_name = mcp_manager.get_mcp_container_name(mcp_name)
mcp_container_names.append(container_name)
if result.get("status") == "running": if result.get("status") == "running":
console.print(f"[green]Started MCP server '{mcp_name}'[/green]") console.print(f"[green]Started MCP server '{mcp_name}'[/green]")
@@ -1061,6 +1066,9 @@ def start_mcp(
failed_count += 1 failed_count += 1
except Exception as e: except Exception as e:
console.print(f"[red]Error starting MCP server '{mcp_name}': {e}[/red]") console.print(f"[red]Error starting MCP server '{mcp_name}': {e}[/red]")
# Remove from the container names list if failed
if container_name in mcp_container_names:
mcp_container_names.remove(container_name)
failed_count += 1 failed_count += 1
# Show a summary # Show a summary