Skip to content

Update mcp-agent apps list UI to use lists instead of tables #408

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Sep 4, 2025
+56 −64
Merged

Update mcp-agent apps list UI to use lists instead of tables #408

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
120 changes: 56 additions & 64 deletions src/mcp_agent/cli/cloud/commands/apps/list/main.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,7 @@
from typing import List, Optional

import typer
from rich.padding import Padding
from rich.panel import Panel
from rich.table import Table

from mcp_agent.cli.auth import load_api_key_credentials
from mcp_agent.cli.config import settings
Expand Down Expand Up @@ -73,15 +71,19 @@ async def parallel_requests():
print_info(f"Found {num_apps} deployed app(s):")
print_apps(list_apps_res.apps)
else:
console.print("\n[bold blue]📦 Deployed MCP Apps (0)[/bold blue]")
print_info("No deployed apps found.")

console.print("\n" + "─" * 80 + "\n")

if list_app_configs_res.appConfigurations:
num_configs = list_app_configs_res.totalCount or len(
list_app_configs_res.appConfigurations
)
print_info(f"Found {num_configs} configured app(s):")
print_app_configs(list_app_configs_res.appConfigurations)
else:
console.print("\n[bold blue]⚙️ Configured MCP Apps (0)[/bold blue]")
print_info("No configured apps found.")

except UnauthenticatedError as e:
Expand All @@ -106,81 +108,71 @@ def print_info_header() -> None:


def print_apps(apps: List[MCPApp]) -> None:
"""Print a summary table of the app information."""
table = Table(
title="Deployed MCP Apps",
expand=False,
border_style="blue",
padding=(0, 1),
)
"""Print a list of deployed apps in a clean, copyable format."""
console.print(f"\n[bold blue]📦 Deployed MCP Apps ({len(apps)})[/bold blue]")

table.add_column("Name", style="cyan", overflow="fold")
table.add_column("ID", style="bright_blue", no_wrap=True)
table.add_column("Description", style="cyan", overflow="fold")
table.add_column("Server URL", style="bright_blue", no_wrap=True)
table.add_column("Status", style="bright_blue", no_wrap=True)
table.add_column("Created", style="cyan", overflow="fold")

for idx, app in enumerate(apps):
is_last_row = idx == len(apps) - 1
table.add_row(
app.name,
app.appId,
app.description,
app.appServerInfo.serverUrl if app.appServerInfo else "",
_server_status_text(
(
app.appServerInfo.status
if app.appServerInfo
else "APP_SERVER_STATUS_OFFLINE"
),
is_last_row,
),
app.createdAt.strftime("%Y-%m-%d %H:%M:%S"),
for i, app in enumerate(apps):
if i > 0:
console.print()

status = _server_status_text(
app.appServerInfo.status
if app.appServerInfo
else "APP_SERVER_STATUS_OFFLINE"
)

console.print(table)
console.print(f"[bold cyan]{app.name or 'Unnamed'}[/bold cyan] {status}")
console.print(f" App ID: {app.appId}")

if app.appServerInfo and app.appServerInfo.serverUrl:
console.print(f" Server: {app.appServerInfo.serverUrl}")

if app.description:
console.print(f" Description: {app.description}")

console.print(f" Created: {app.createdAt.strftime('%Y-%m-%d %H:%M:%S')}")
Comment on lines +111 to +133
Copy link

@coderabbitai coderabbitai bot Sep 3, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🛠️ Refactor suggestion

⚠️ Potential issue

Guard against None createdAt and don’t default missing server info to “Offline”

  • createdAt may be None (seen on configurations; apps can be similar) → AttributeError risk.
  • Treat missing appServerInfo as Unknown, not Offline.
-        status = _server_status_text(
-            app.appServerInfo.status
-            if app.appServerInfo
-            else "APP_SERVER_STATUS_OFFLINE"
-        )
+        status = _server_status_text(
+            app.appServerInfo.status if app.appServerInfo else None
+        )
@@
-        console.print(f"  Created: {app.createdAt.strftime('%Y-%m-%d %H:%M:%S')}")
+        if app.createdAt:
+            console.print(f"  Created: {app.createdAt.strftime('%Y-%m-%d %H:%M:%S')}")
+        else:
+            console.print("  Created: -")

Additionally update the helper to accept Optional status (see suggestion below).

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
"""Print a list of deployed apps in a clean, copyable format."""
console.print(f"\n[bold blue]📦 Deployed MCP Apps ({len(apps)})[/bold blue]")
table.add_column("Name", style="cyan", overflow="fold")
table.add_column("ID", style="bright_blue", no_wrap=True)
table.add_column("Description", style="cyan", overflow="fold")
table.add_column("Server URL", style="bright_blue", no_wrap=True)
table.add_column("Status", style="bright_blue", no_wrap=True)
table.add_column("Created", style="cyan", overflow="fold")
for idx, app in enumerate(apps):
is_last_row = idx == len(apps) - 1
table.add_row(
app.name,
app.appId,
app.description,
app.appServerInfo.serverUrl if app.appServerInfo else "",
_server_status_text(
(
app.appServerInfo.status
if app.appServerInfo
else "APP_SERVER_STATUS_OFFLINE"
),
is_last_row,
),
app.createdAt.strftime("%Y-%m-%d %H:%M:%S"),
for i, app in enumerate(apps):
if i > 0:
console.print()
status = _server_status_text(
app.appServerInfo.status
if app.appServerInfo
else "APP_SERVER_STATUS_OFFLINE"
)
console.print(table)
console.print(f"[bold cyan]{app.name or 'Unnamed'}[/bold cyan] {status}")
console.print(f" App ID: {app.appId}")
if app.appServerInfo and app.appServerInfo.serverUrl:
console.print(f" Server: {app.appServerInfo.serverUrl}")
if app.description:
console.print(f" Description: {app.description}")
console.print(f" Created: {app.createdAt.strftime('%Y-%m-%d %H:%M:%S')}")
"""Print a list of deployed apps in a clean, copyable format."""
console.print(f"\n[bold blue]📦 Deployed MCP Apps ({len(apps)})[/bold blue]")
for i, app in enumerate(apps):
if i > 0:
console.print()
status = _server_status_text(
app.appServerInfo.status if app.appServerInfo else None
)
console.print(f"[bold cyan]{app.name or 'Unnamed'}[/bold cyan] {status}")
console.print(f" App ID: {app.appId}")
if app.appServerInfo and app.appServerInfo.serverUrl:
console.print(f" Server: {app.appServerInfo.serverUrl}")
if app.description:
console.print(f" Description: {app.description}")
if app.createdAt:
console.print(f" Created: {app.createdAt.strftime('%Y-%m-%d %H:%M:%S')}")
else:
console.print(" Created: -")



def print_app_configs(app_configs: List[MCPAppConfiguration]) -> None:
"""Print a summary table of the app configuration information."""
table = Table(title="Configured MCP Apps", expand=False, border_style="blue")

table.add_column("Name", style="cyan", overflow="fold")
table.add_column("ID", style="bright_blue", no_wrap=True)
table.add_column("App ID", style="bright_blue", overflow="fold")
table.add_column("Description", style="cyan", overflow="fold")
table.add_column("Server URL", style="bright_blue", no_wrap=True)
table.add_column("Status", style="bright_blue", no_wrap=True)
table.add_column("Created", style="cyan", overflow="fold")

for idx, config in enumerate(app_configs):
is_last_row = idx == len(app_configs) - 1
table.add_row(
config.app.name if config.app else "",
config.appConfigurationId,
config.app.appId if config.app else "",
config.app.description if config.app else "",
config.appServerInfo.serverUrl if config.appServerInfo else "",
_server_status_text(
(
config.appServerInfo.status
if config.appServerInfo
else "APP_SERVER_STATUS_OFFLINE"
),
is_last_row=is_last_row,
),
config.createdAt.strftime("%Y-%m-%d %H:%M:%S") if config.createdAt else "",
"""Print a list of configured apps in a clean, copyable format."""
console.print(
f"\n[bold blue]⚙️ Configured MCP Apps ({len(app_configs)})[/bold blue]"
)

for i, config in enumerate(app_configs):
if i > 0:
console.print()

status = _server_status_text(
config.appServerInfo.status
if config.appServerInfo
else "APP_SERVER_STATUS_OFFLINE"
)

console.print(
f"[bold cyan]{config.app.name if config.app else 'Unnamed'}[/bold cyan] {status}"
)
console.print(f" Config ID: {config.appConfigurationId}")

if config.app:
console.print(f" App ID: {config.app.appId}")
if config.app.description:
console.print(f" Description: {config.app.description}")

console.print(table)
if config.appServerInfo and config.appServerInfo.serverUrl:
console.print(f" Server: {config.appServerInfo.serverUrl}")

if config.createdAt:
console.print(
f" Created: {config.createdAt.strftime('%Y-%m-%d %H:%M:%S')}"
)


def _server_status_text(status: str, is_last_row: bool = False):
"""Convert server status code to emoji."""
if status == "APP_SERVER_STATUS_ONLINE":
return "🟢 Online"
return "[green]🟢 Online[/green]"
elif status == "APP_SERVER_STATUS_OFFLINE":
return Padding("🔴 Offline", (0, 0, 0 if is_last_row else 1, 0), style="red")
return "[red]🔴 Offline[/red]"
else:
return "❓ Unknown"
Loading