From 0f35bf93bd7d75e30a2c8a2f923ea13710084e95 Mon Sep 17 00:00:00 2001 From: aivibe <93047788+axisrow@users.noreply.github.com> Date: Mon, 2 Mar 2026 11:55:17 +0700 Subject: [PATCH] fix: disable startup banner that pollutes CLI output The startup banner runs at module-import time before Typer parses arguments, so it pollutes --version, --help, and every subcommand output. In chat mode it is immediately erased by clear_screen() anyway. Co-Authored-By: Claude Opus 4.6 --- src/mcp_cli/main.py | 20 ++++++++++++-------- 1 file changed, 12 insertions(+), 8 deletions(-) diff --git a/src/mcp_cli/main.py b/src/mcp_cli/main.py index ccb353a5..f1496302 100644 --- a/src/mcp_cli/main.py +++ b/src/mcp_cli/main.py @@ -1733,14 +1733,18 @@ async def _ping_wrapper(**params): direct_registered.append("ping") -# Show what we actually registered -all_registered = registry_registered + direct_registered -output.success("✓ MCP CLI ready") -if all_registered: - output.info(f" Available commands: {', '.join(sorted(all_registered))}") -else: - output.warning(" Warning: No commands were successfully registered!") -output.hint(" Use --help to see all options") +# NOTE: Startup banner disabled — it runs at module-import time (before +# Typer parses arguments), so it pollutes the output of --version, --help, +# and every subcommand. In chat mode it is immediately erased by +# clear_screen(). Consider removing this block entirely. +# +# all_registered = registry_registered + direct_registered +# output.success("✓ MCP CLI ready") +# if all_registered: +# output.info(f" Available commands: {', '.join(sorted(all_registered))}") +# else: +# output.warning(" Warning: No commands were successfully registered!") +# output.hint(" Use --help to see all options") # ──────────────────────────────────────────────────────────────────────────────