Missing RedactingFormatter import causing gateway crashes

Fixes on record

Merged to main. A stamp counts the issues a fix is linked to.

• #8103 fix(gateway): add missing RedactingFormatter import (salvage #8046)

  merged to main

  files: gateway/run.py, tests/gateway/test_runner_startup_failures.py

  Show the diff

  • gateway/run.py

    @@ -8560,6 +8560,8 @@ async def start_gateway(config: Optional[GatewayConfig] = None, replace: bool =
         # verbosity=1    (-v):         INFO and above
         # verbosity=2+   (-vv/-vvv):   DEBUG
         if verbosity is not None:
    +        from agent.redact import RedactingFormatter
    +
             _stderr_level = {0: logging.WARNING, 1: logging.INFO}.get(verbosity, logging.DEBUG)
             _stderr_handler = logging.StreamHandler()
             _stderr_handler.setLevel(_stderr_level)

  • tests/gateway/test_runner_startup_failures.py

    @@ -132,6 +132,39 @@ async def test_runner_records_connected_platform_state_on_success(monkeypatch, t
         assert state["platforms"]["discord"]["error_message"] is None
     
     
    [redacted:email]
    +async def test_start_gateway_verbosity_imports_redacting_formatter(monkeypatch, tmp_path):
    +    """Verbosity != None must not crash with NameError on RedactingFormatter (#8044)."""
    +    monkeypatch.setenv("HERMES_HOME", str(tmp_path))
    +
    +    class _CleanExitRunner:
    +        def __init__(self, config):
    +            self.config = config
    +            self.should_exit_cleanly = True
    +            self.exit_reason = None
    +            self.adapters = {}
    +
    +        async def start(self):
    +            return True
    +
    +        async def stop(self):
    +            return None
    +
    +    monkeypatch.setattr("gateway.status.get_running_pid", lambda: None)
    +    monkeypatch.setattr("tools.skills_sync.sync_skills", lambda quiet=True: None)
    +    monkeypatch.setattr("hermes_logging.setup_logging", lambda hermes_home, mode: tmp_path)
    +    monkeypatch.setattr("hermes_logging._add_rotating_handler", lambda *args, **kwargs: None)
    +    monkeypatch.setattr("gateway.run.GatewayRunner", _CleanExitRunner)
    +
    +    from gateway.run import start_gateway
    +
    +    # verbosity=1 triggers the code path that uses RedactingFormatter.
    +    # Before the fix this raised NameError.
    +    ok = await start_gateway(config=GatewayConfig(), replace=False, verbosity=1)
    +
    +    assert ok is True
    +
    +
     @pytest.mark.asyncio
     async def test_start_gateway_replace_force_uses_terminate_pid(monkeypatch, tmp_path):
         monkeypatch.setenv("HERMES_HOME", str(tmp_path))