Merge pull request #939 from Pipelex/release/v0.30.1

Release v0.30.1

Author: lchoquel

.badges/tests.json

@@ -1,7 +1,7 @@
 {
   "schemaVersion": 1,
   "label": "tests",
-  "message": "6111",
+  "message": "6114",
   "color": "blue",
   "cacheSeconds": 300
 }

CHANGELOG.md

@@ -1,5 +1,23 @@
 # Changelog
 
+## [v0.30.1] - 2026-05-26
+
+### Fixed
+
+- **`pipelex-agent` now silences every Python logger on stderr regardless of user TOML.** The agent CLI is machine-consumed: stdout is reserved for the structured success envelope (JSON / markdown) and stderr for the structured error envelope. Free-floating `log.*` calls — a `log.debug` from `telemetry_factory.py`, a `log.warning` from `validation_error_categorizer.py`, or an INFO/WARNING line from any third-party dep (`anthropic`, `httpx`, `botocore`, `openai`, anything a transitive dep configures) — would corrupt the stderr channel for downstream parsers (`mthds-js`'s `PipelexRunner` doing `JSON.parse(stderr)` on the validate hook). Two layers, defense-in-depth:
+  - **Layer 1 — pin pipelex's own logs off via config.** `make_pipelex_for_agent_cli` injects `config_overrides` into `Pipelex.make()` that pin `default_log_level = OFF` and `package_log_levels.pipelex = OFF` from the very first `log.configure` call. A user setting `[pipelex.log_config.package_log_levels] pipelex = "DEBUG"` in `~/.pipelex/pipelex.toml` can no longer leak its own DEBUG/INFO/WARNING lines.
+  - **Layer 2 — process-global cutoff that covers every logger.** New `silence_logging_for_agent_cli()` calls `logging.disable(sys.maxsize)` — a process-global threshold checked inside `Logger.isEnabledFor` BEFORE any per-logger level. No record gets created for any logger at any level (including custom levels above `CRITICAL`), regardless of which package emits or what level the user configured. Wired into the Typer `app_callback` so every subcommand — including `init` and `accept-gateway-terms`, which bypass `make_pipelex_for_agent_cli` — silences logging before any command body runs; idempotent per-call invocations remain inside `make_pipelex_for_agent_cli` and `agent_doctor_cmd` as belt-and-braces for direct library callers.
+
+  A new e2e regression test pins the contract by setting `anthropic`, `httpx`, `botocore`, `openai` to `DEBUG` in the user TOML and asserting both stdout and stderr stay clean.
+
+### Changed
+
+- **`pipelex-agent` no longer accepts `--log-level`.** Log suppression is unconditional by design — there is no verbosity setting on the agent CLI. The `log_level` parameter is removed from `make_pipelex_for_agent_cli()` and `apply_agent_cli_output_discipline()`, and the corresponding `ctx.obj["log_level"]` plumbing is gone from every caller. For verbose debugging, use the human `pipelex` CLI, which honors the user's TOML log config.
+
+- **`pipelex/cli/commands/doctor_cmd.py::setup_doctor_runtime` now `deep_update`s `log_config_overrides` into the loaded config** (was a flat-merge that silently replaced nested dicts like `package_log_levels`). With the flat merge, pinning `package_log_levels.pipelex = OFF` for the agent doctor path would have wiped out all the third-party package levels (`anthropic`, `asyncio`, `botocore`, ...) shipped in the default config. The deep merge preserves them.
+
+- **Removed `ctx: typer.Context` from 8 agent CLI commands that no longer used it.** `validate/{pipe,bundle,method}_cmd.py`, `inputs/{pipe,bundle,method}_cmd.py`, `models_cmd.py`, `check_model_cmd.py` had `ctx.obj["log_level"]` as their only ctx usage; with `--log-level` gone, the parameter became dead weight. The `run/` commands keep `ctx` because they still read `ctx.obj["runner"]`. Test callers (and the now-unused `agent_ctx` conftest fixture) updated to match.
+
 ## [v0.30.0] - 2026-05-25
 
 ### Fixed

TODOS.md

@@ -9,6 +9,7 @@
 from typing_extensions import override
 
 from pipelex.cli.agent_cli.commands.accept_gateway_terms_cmd import agent_accept_gateway_terms_cmd
+from pipelex.cli.agent_cli.commands.agent_cli_factory import silence_logging_for_agent_cli
 from pipelex.cli.agent_cli.commands.agent_output import CliOutputFormat, agent_error, set_agent_cli_error_format
 from pipelex.cli.agent_cli.commands.check_model_cmd import agent_check_model_cmd
 from pipelex.cli.agent_cli.commands.concept_cmd import concept_cmd
@@ -21,7 +22,6 @@
 from pipelex.cli.agent_cli.commands.pipe_cmd import pipe_cmd
 from pipelex.cli.agent_cli.commands.run.app import run_app
 from pipelex.cli.agent_cli.commands.validate.app import validate_app
-from pipelex.tools.log.log_levels import LogLevel
 from pipelex.tools.misc.package_utils import get_package_version
 
 
@@ -91,31 +91,35 @@ def app_callback(
             is_eager=True,
         ),
     ] = False,
-    log_level: Annotated[
-        str,
-        typer.Option("--log-level", help="Log verbosity level (debug, verbose, info, warning, error, critical)."),
-    ] = "warning",
     runner: Annotated[
         str,
         typer.Option("--runner", help="Runner to use: 'pipelex' (local) or 'api' (remote MTHDS API)."),
     ] = "pipelex",
 ) -> None:
-    """Agent CLI callback - no logo, minimal output."""
+    """Agent CLI callback - no logo, minimal output.
+
+    Note: there is no ``--log-level`` flag. ``pipelex-agent`` is machine-consumed:
+    stdout is reserved for the success envelope and stderr for the error envelope.
+    Free-floating logs would corrupt those channels, so Python's logging system is
+    cut off process-wide via ``silence_logging_for_agent_cli`` as the very first
+    action in this callback — covering every subcommand invocation (including ones
+    like ``init`` and ``accept-gateway-terms`` that bypass
+    ``make_pipelex_for_agent_cli``). The ``--version`` eager option short-circuits
+    before this body runs, but its callback only does ``typer.echo`` + ``Exit`` and
+    touches no log path. For verbose debugging, use the human ``pipelex`` CLI instead.
+    """
+    # Process-global logging cutoff, armed BEFORE any command body runs and BEFORE any
+    # error-format / runner-validation code below can route through ``log.*``. This is the
+    # primary armor for the stdout/stderr discipline; the per-call invocations inside
+    # ``make_pipelex_for_agent_cli`` and ``agent_doctor_cmd`` are kept as defense-in-depth
+    # for direct library callers that bypass this Typer entry point.
+    silence_logging_for_agent_cli()
     # Reset the error-format ContextVar at the single choke point every command passes through, so
     # a markdown command cannot leak markdown into a later JSON-only command in the same process.
     # --format / --error-format commands override this afterwards; JSON-only commands keep the JSON default.
     set_agent_cli_error_format(CliOutputFormat.JSON)
 
     ctx.ensure_object(dict)
-    try:
-        ctx.obj["log_level"] = LogLevel(log_level.upper())
-    except ValueError:
-        valid_values = ", ".join(level.value.lower() for level in LogLevel)
-        agent_error(
-            f"Invalid log level '{log_level}'. Valid values: {valid_values}",
-            "ArgumentError",
-        )
-
     try:
         ctx.obj["runner"] = RunnerType(runner)
     except ValueError: