do not allow unsupported server URLs as arguments

Author: jtcorbett

src/mcp_agent/cli/cloud/commands/logger/tail/main.py

@@ -28,7 +28,7 @@
 
 def tail_logs(
     app_identifier: str = typer.Argument(
-        help="Server ID, URL, or app configuration ID to retrieve logs for"
+        help="App ID or app configuration ID to retrieve logs for"
     ),
     since: Optional[str] = typer.Option(
         None,
@@ -83,7 +83,7 @@ def tail_logs(
         mcp-agent cloud logger tail app_abc123 --limit 50
         
         # Stream logs continuously
-        mcp-agent cloud logger tail https://app.mcpac.dev/abc123 --follow
+        mcp-agent cloud logger tail app_abc123 --follow
         
         # Show logs from the last hour with error filtering
         mcp-agent cloud logger tail app_abc123 --since 1h --grep "ERROR|WARN"
@@ -129,14 +129,17 @@ def tail_logs(
         console.print("[red]Error: --format must be 'text', 'json', or 'yaml'[/red]")
         raise typer.Exit(6)
 
-    app_id, config_id, server_url = parse_app_identifier(app_identifier)
+    try:
+        app_id, config_id = parse_app_identifier(app_identifier)
+    except ValueError as e:
+        console.print(f"[red]Error: {e}[/red]")
+        raise typer.Exit(6)
 
     try:
         if follow:
             asyncio.run(_stream_logs(
                 app_id=app_id,
                 config_id=config_id,
-                server_url=server_url,
                 credentials=credentials,
                 grep_pattern=grep,
                 app_identifier=app_identifier,
@@ -146,7 +149,6 @@ def tail_logs(
             asyncio.run(_fetch_logs(
                 app_id=app_id,
                 config_id=config_id,
-                server_url=server_url,
                 credentials=credentials,
                 since=since,
                 grep_pattern=grep,
@@ -168,7 +170,6 @@ def tail_logs(
 async def _fetch_logs(
     app_id: Optional[str],
     config_id: Optional[str], 
-    server_url: Optional[str],
     credentials: UserCredentials,
     since: Optional[str],
     grep_pattern: Optional[str],
@@ -241,17 +242,15 @@ async def _fetch_logs(
 
 async def _stream_logs(
     app_id: Optional[str],
-    config_id: Optional[str],
-    server_url: Optional[str], 
+    config_id: Optional[str], 
     credentials: UserCredentials,
     grep_pattern: Optional[str],
     app_identifier: str,
     format: str,
 ) -> None:
     """Stream logs continuously via SSE."""
 
-    if not server_url:
-        server_url = await resolve_server_url(app_id, config_id, credentials)
+    server_url = await resolve_server_url(app_id, config_id, credentials)
 
     parsed = urlparse(server_url)
     stream_url = f"{parsed.scheme}://{parsed.netloc}/logs"

src/mcp_agent/cli/cloud/commands/servers/delete/main.py

@@ -17,7 +17,7 @@
 
 @handle_server_api_errors
 def delete_server(
-    id_or_url: str = typer.Argument(..., help="Server ID or URL to delete"),
+    id_or_url: str = typer.Argument(..., help="Server ID or app configuration ID to delete"),
     force: bool = typer.Option(False, "--force", "-f", help="Force deletion without confirmation prompt"),
 ) -> None:
     """Delete a specific MCP Server."""

src/mcp_agent/cli/cloud/commands/servers/describe/main.py

@@ -19,7 +19,7 @@
 
 @handle_server_api_errors 
 def describe_server(
-    id_or_url: str = typer.Argument(..., help="Server ID or URL to describe"),
+    id_or_url: str = typer.Argument(..., help="Server ID or app configuration ID to describe"),
     format: Optional[str] = typer.Option("text", "--format", help="Output format (text|json|yaml)"),
 ) -> None:
     """Describe a specific MCP Server."""

src/mcp_agent/cli/cloud/commands/servers/utils.py

@@ -51,11 +51,11 @@ def validate_output_format(format: str) -> None:
 
 
 def resolve_server(client: MCPAppClient, id_or_url: str) -> Union[MCPApp, MCPAppConfiguration]:
-    """Resolve server from ID or URL.
+    """Resolve server from ID.
     
     Args:
         client: Authenticated MCP App client
-        id_or_url: Server identifier (ID, config ID, or URL)
+        id_or_url: Server identifier (app ID or app config ID)
         
     Returns:
         Server object (MCPApp or MCPAppConfiguration)
@@ -64,15 +64,15 @@ def resolve_server(client: MCPAppClient, id_or_url: str) -> Union[MCPApp, MCPApp
         CLIError: If server resolution fails
     """
     try:
-        app_id, config_id, server_url = parse_app_identifier(id_or_url)
+        app_id, config_id = parse_app_identifier(id_or_url)
 
-        if server_url:
-            return run_async(client.get_app(server_url=server_url))
-        elif config_id:
+        if config_id:
             return run_async(client.get_app_configuration(app_config_id=config_id))
         else:
             return run_async(client.get_app(app_id=app_id))
 
+    except ValueError as e:
+        raise CLIError(str(e)) from e
     except Exception as e:
         raise CLIError(f"Failed to resolve server '{id_or_url}': {str(e)}") from e
 

src/mcp_agent/cli/core/utils.py

@@ -26,19 +26,26 @@ def run_async(coro):
         raise
 
 
-def parse_app_identifier(identifier: str) -> Tuple[Optional[str], Optional[str], Optional[str]]:
-    """Parse app identifier to extract app ID, config ID, and server URL."""
+def parse_app_identifier(identifier: str) -> Tuple[Optional[str], Optional[str]]:
+    """Parse app identifier to extract app ID and config ID.
     
-    if identifier.startswith(('http://', 'https://')):
-        return None, None, identifier
+    Args:
+        identifier: App identifier (must be app_... or apcnf_...)
+        
+    Returns:
+        Tuple of (app_id, config_id)
+        
+    Raises:
+        ValueError: If identifier format is not recognized
+    """
 
     if identifier.startswith('apcnf_'):
-        return None, identifier, None
+        return None, identifier
 
     if identifier.startswith('app_'):
-        return identifier, None, None
+        return identifier, None
 
-    return identifier, None, None
+    raise ValueError(f"Invalid identifier format: '{identifier}'. Must be an app ID (app_...) or app configuration ID (apcnf_...)")
 
 
 async def resolve_server_url(