Use context var to define unconditional verbose logger

Author: petersonbill64

LLMS.txt

@@ -1406,7 +1406,7 @@ Example usage:
 
 ### src/mcp_agent/cli/main.py
 
-**Function:** `main(verbose: bool = typer.Option(False, '--verbose', '-v', help='Enable verbose mode'), quiet: bool = typer.Option(False, '--quiet', '-q', help='Disable output'), color: bool = typer.Option(True, '--color/--no-color', help='Enable/disable color output'))`
+**Function:** `main(verbose: bool = typer.Option(False, '--verbose', '-v', help='Enable verbose mode'), color: bool = typer.Option(True, '--color/--no-color', help='Enable/disable color output'))`
 
 - **Description**: Main entry point for the MCP Agent CLI.
 - **Parameters**

docs/cli-reference.mdx

@@ -26,7 +26,6 @@ Available for all commands:
 | Option | Description |
 | :----- | :---------- |
 | `--verbose, -v` | Enable verbose output |
-| `--quiet, -q` | Minimize output |
 | `--format <type>` | Output format (`text`, `json`, `yaml`) |
 | `--color/--no-color` | Enable/disable colored output |
 | `--version` | Show version and exit |

src/mcp_agent/cli/cloud/commands/configure/main.py

@@ -35,6 +35,7 @@
     print_configuration_header,
     print_info,
     print_success,
+    LOG_VERBOSE,
 )
 
 
@@ -85,6 +86,12 @@ def configure_app(
         help="API key for authentication. Defaults to MCP_API_KEY environment variable.",
         envvar=ENV_API_KEY,
     ),
+    verbose: bool = typer.Option(
+        False,
+        "--verbose",
+        "-v",
+        help="Enable verbose output for this command",
+    ),
 ) -> str:
     """Configure an MCP app with the required params (e.g. user secrets).
 
@@ -99,6 +106,9 @@ def configure_app(
     Returns:
         Configured app ID.
     """
+    if verbose:
+        LOG_VERBOSE.set(True)
+
     # Check what params the app requires (doubles as an access check)
     if not app_server_url:
         raise CLIError("You must provide a server URL to configure.")
@@ -214,7 +224,7 @@ def configure_app(
             print_success("User secrets processed successfully")
 
         except Exception as e:
-            if ctx.obj.get("verbose", False):
+            if LOG_VERBOSE.get():
                 import traceback
 
                 typer.echo(traceback.format_exc())

src/mcp_agent/cli/cloud/commands/deploy/main.py

@@ -31,6 +31,8 @@
     print_error,
     print_info,
     print_success,
+    LOG_VERBOSE,
+    print_verbose,
 )
 from mcp_agent.cli.utils.git_utils import (
     get_git_metadata,
@@ -130,6 +132,12 @@ def deploy_config(
         file_okay=True,
         resolve_path=True,
     ),
+    verbose: bool = typer.Option(
+        False,
+        "--verbose",
+        "-v",
+        help="Enable verbose output for this command",
+    ),
 ) -> str:
     """Deploy an mcp-agent using the specified configuration.
 
@@ -152,6 +160,9 @@ def deploy_config(
     Returns:
         Newly-deployed MCP App ID
     """
+    if verbose:
+        LOG_VERBOSE.set(True)
+
     try:
         if config_dir is None:
             resolved_config_dir = working_dir
@@ -209,15 +220,13 @@ def deploy_config(
                 retriable=False,
             )
 
-        if ctx.obj.get("verbose", False):
-            print_info(f"Using API at {effective_api_url}")
+        print_verbose(f"Using API at {effective_api_url}")
 
         mcp_app_client = MCPAppClient(
             api_url=effective_api_url, api_key=effective_api_key
         )
 
-        if ctx.obj.get("verbose", False):
-            print_info(f"Checking for existing app ID for '{app_name}'...")
+        print_verbose(f"Checking for existing app ID for '{app_name}'...")
 
         try:
             app_id = run_async(mcp_app_client.get_app_id_by_name(app_name))
@@ -232,8 +241,7 @@ def deploy_config(
                 )
                 app_id = app.appId
                 print_success(f"Created new app '{app_name}'")
-                if ctx.obj.get("verbose", False):
-                    print_info(f"New app id: `{app_id}`")
+                print_verbose(f"New app id: `{app_id}`")
             else:
                 short_id = f"{app_id[:8]}…"
                 print_success(f"Found existing app '{app_name}' (ID: `{short_id}`)")
@@ -243,8 +251,7 @@ def deploy_config(
                         default=True,
                     )
                     if use_existing:
-                        if ctx.obj.get("verbose", False):
-                            print_info(f"Will deploy an update to app ID: `{app_id}`")
+                        print_verbose(f"Will deploy an update to app ID: `{app_id}`")
                     else:
                         print_error(
                             "Cancelling deployment. Please choose a different app name."
@@ -261,8 +268,7 @@ def deploy_config(
                     update_payload["unauthenticated_access"] = unauthenticated_access
 
                 if update_payload:
-                    if ctx.obj.get("verbose", False):
-                        print_info("Updating app settings before deployment...")
+                    print_verbose("Updating app settings before deployment...")
                     run_async(
                         mcp_app_client.update_app(
                             app_id=app_id,
@@ -280,10 +286,9 @@ def deploy_config(
         # If a deployed secrets file already exists, determine if it should be used or overwritten
         if deployed_secrets_file:
             if secrets_file:
-                if ctx.obj.get("verbose", False):
-                    print_info(
-                        f"Both '{MCP_SECRETS_FILENAME}' and '{MCP_DEPLOYED_SECRETS_FILENAME}' found in {config_dir}."
-                    )
+                print_verbose(
+                    f"Both '{MCP_SECRETS_FILENAME}' and '{MCP_DEPLOYED_SECRETS_FILENAME}' found in {config_dir}."
+                )
                 if non_interactive:
                     print_info(
                         "Running in non-interactive mode — reusing previously deployed secrets."
@@ -320,10 +325,9 @@ def deploy_config(
             )
 
             print_success("Secrets file processed successfully")
-            if ctx.obj.get("verbose", False):
-                print_info(
-                    f"Transformed secrets file written to {secrets_transformed_path}"
-                )
+            print_verbose(
+                f"Transformed secrets file written to {secrets_transformed_path}"
+            )
 
         else:
             print_info("Skipping secrets processing...")
@@ -367,7 +371,6 @@ def deploy_config(
                 mcp_app_client=mcp_app_client,
                 retry_count=retry_count,
                 ignore=ignore_path,
-                verbose=ctx.obj.get("verbose", False),
             )
         )
 
@@ -390,7 +393,7 @@ def deploy_config(
         return app_id
 
     except Exception as e:
-        if ctx.obj.get("verbose", False):
+        if LOG_VERBOSE.get():
             import traceback
 
             typer.echo(traceback.format_exc())
@@ -404,7 +407,6 @@ async def _deploy_with_retry(
     mcp_app_client: MCPAppClient,
     retry_count: int,
     ignore: Optional[Path],
-    verbose: bool = False,
 ):
     """Execute the deployment operations with retry logic.
 
@@ -425,7 +427,6 @@ async def _deploy_with_retry(
             api_key=api_key,
             project_dir=project_dir,
             ignore_file=ignore,
-            verbose=verbose,
         )
     except Exception as e:
         raise CLIError(f"Bundling failed: {str(e)}") from e
@@ -485,8 +486,7 @@ async def _perform_api_deployment():
                 raise
 
     if retry_count > 1:
-        if verbose:
-            print_info(f"Deployment API configured with up to {retry_count} attempts")
+        print_verbose(f"Deployment API configured with up to {retry_count} attempts")
 
     try:
         return await retry_async_with_exponential_backoff(

src/mcp_agent/cli/cloud/commands/deploy/wrangler_wrapper.py

@@ -11,7 +11,13 @@
 
 from mcp_agent.cli.config import settings
 from mcp_agent.cli.core.constants import MCP_SECRETS_FILENAME
-from mcp_agent.cli.utils.ux import console, print_error, print_warning, print_info
+from mcp_agent.cli.utils.ux import (
+    console,
+    print_error,
+    print_warning,
+    print_info,
+    print_verbose,
+)
 from mcp_agent.cli.utils.git_utils import (
     get_git_metadata,
     compute_directory_fingerprint,
@@ -116,7 +122,6 @@ def wrangler_deploy(
     api_key: str,
     project_dir: Path,
     ignore_file: Path | None = None,
-    verbose: bool = False,
 ) -> None:
     """Bundle the MCP Agent using Wrangler.
 
@@ -188,8 +193,7 @@ def wrangler_deploy(
             else:
                 print_info(f"Using ignore patterns from {ignore_file}")
         else:
-            if verbose:
-                print_info("No ignore file provided; applying default excludes only")
+            print_verbose("No ignore file provided; applying default excludes only")
 
         # Copy the entire project to temp directory, excluding unwanted directories and the live secrets file
         def ignore_patterns(path_str, names):
@@ -299,10 +303,9 @@ def ignore_patterns(path_str, names):
                 },
             )
             meta_vars.update({"MCP_DEPLOY_WORKSPACE_HASH": bundle_hash})
-            if verbose:
-                print_info(
-                    f"Deploying from non-git workspace (hash {bundle_hash[:12]}…)"
-                )
+            print_verbose(
+                f"Deploying from non-git workspace (hash {bundle_hash[:12]}…)"
+            )
 
         # Write a breadcrumb file into the project so it ships with the bundle.
         # Use a Python file for guaranteed inclusion without renaming.

src/mcp_agent/cli/commands/build.py

@@ -19,6 +19,7 @@
 from rich.panel import Panel
 from rich.progress import Progress, SpinnerColumn, TextColumn
 
+from mcp_agent.cli.utils.ux import LOG_VERBOSE
 from mcp_agent.config import get_settings, Settings
 
 
@@ -200,6 +201,9 @@ def build(
     ),
 ) -> None:
     """Run comprehensive preflight checks and generate build manifest."""
+    if verbose:
+        LOG_VERBOSE.set(True)
+    verbose = LOG_VERBOSE.get()
 
     console.print("\n[bold cyan]🔍 MCP-Agent Build Preflight Checks[/bold cyan]\n")
 

src/mcp_agent/cli/commands/config.py

@@ -17,6 +17,7 @@
 from rich.prompt import Prompt, Confirm
 from rich.progress import Progress, SpinnerColumn, TextColumn
 
+from mcp_agent.cli.utils.ux import LOG_VERBOSE
 from mcp_agent.config import Settings, get_settings
 
 
@@ -126,6 +127,10 @@ def check(
     ),
 ) -> None:
     """Check and summarize configuration status."""
+    if verbose:
+        LOG_VERBOSE.set(True)
+    verbose = LOG_VERBOSE.get()
+
     cfg = _find_config_file()
     sec = _find_secrets_file()
 

src/mcp_agent/cli/commands/keys.py

@@ -18,6 +18,7 @@
 from rich.prompt import Prompt, Confirm
 from rich.progress import Progress, SpinnerColumn, TextColumn
 
+from mcp_agent.cli.utils.ux import LOG_VERBOSE
 
 app = typer.Typer(help="Manage provider API keys")
 console = Console()
@@ -172,6 +173,10 @@ def show(
     """Show configured API keys and their status."""
     from mcp_agent.config import get_settings
 
+    if verbose:
+        LOG_VERBOSE.set(True)
+    verbose = LOG_VERBOSE.get()
+
     console.print("\n[bold cyan]🔑 API Key Status[/bold cyan]\n")
 
     settings = get_settings()
@@ -451,6 +456,10 @@ def test(
 
     console.print("\n[bold cyan]🧪 Testing API Keys[/bold cyan]\n")
 
+    if verbose:
+        LOG_VERBOSE.set(True)
+    verbose = LOG_VERBOSE.get()
+
     settings = get_settings()
 
     # Determine which providers to test

src/mcp_agent/cli/commands/server.py

@@ -12,6 +12,7 @@
 from rich.table import Table
 from rich.prompt import Confirm
 
+from mcp_agent.cli.utils.ux import LOG_VERBOSE
 from mcp_agent.config import Settings, MCPServerSettings, MCPSettings, get_settings
 from mcp_agent.cli.utils.importers import import_servers_from_mcp_json
 from mcp_agent.core.context import cleanup_context
@@ -694,6 +695,10 @@ def test(
     from mcp_agent.app import MCPApp
     from mcp_agent.agents.agent import Agent
 
+    if verbose:
+        LOG_VERBOSE.set(True)
+    verbose = LOG_VERBOSE.get()
+
     async def _probe():
         app_obj = MCPApp(name="server-test")
         async with app_obj.run():

src/mcp_agent/cli/config/settings.py

@@ -10,6 +10,7 @@
     ENV_API_BASE_URL,
     ENV_API_KEY,
 )
+from mcp_agent.cli.utils.ux import LOG_VERBOSE
 
 
 class Settings(BaseSettings):
@@ -38,3 +39,6 @@ class Settings(BaseSettings):
 
 # Create a singleton settings instance
 settings = Settings()
+
+# Set LOG_VERBOSE context var based on VERBOSE setting
+LOG_VERBOSE.set(settings.VERBOSE)