fix(hermes): wire config.yaml through to AIAgent + fix client is_closed false positive

[saulmc]

Summary

• Config passthrough: AgentRunner was creating AIAgent without passing provider_routing, fallback_model, or reasoning_config from config.yaml. The upstream gateway and CLI both read these sections and pass them through — we were silently ignoring them. This meant provider_routing.only was dead config and OpenRouter could route to any provider.
• Client is_closed fix: Monkey-patches _is_openai_client_closed to fix an upstream bug (reported at [Bug]: _is_openai_client_closed false positive — openai SDK is_closed is a method, not a property NousResearch/hermes-agent#4377) where the openai SDK's is_closed method is checked via bool(getattr(client, "is_closed", False)). Since is_closed is a method (not a property), the bound method object is always truthy — every API call falsely detects the shared client as closed and recreates it, adding ~2 unnecessary TCP+TLS round trips per call.

Why

Investigating slow API call latency. Two root causes found:

1. Provider routing config was never reaching OpenRouter, so it could route to any (potentially slow) provider
2. Every API call was churning through 2 extra OpenAI client creations + TCP+TLS handshakes due to a false-positive closed-client detection

🤖 Generated with Claude Code

Note

Wire config.yaml into AIAgent and fix is_closed false positive in AgentRunner

• AgentRunner._create_agent now loads HERMES_HOME/config.yaml and passes provider routing options (only/ignore/order/sort/require_parameters/data_collection), fallback model, and reasoning config into the AIAgent constructor.
• Fixes a false positive in the OpenAI client closed-status check by overriding _is_openai_client_closed to handle cases where is_closed is a bound method (not a bool), inspect the underlying http_client._client.is_closed, and correctly handle unittest.mock.Mock instances.
• Changes the hermes-agent install extras from [all] to [cron,mcp,pty,homeassistant] in both the Dockerfile and install-deps.sh, removing unintended optional dependencies from the image.
• honcho_session_key is no longer passed to AIAgent.

^{Macroscope summarized 73d6c8a.}

[macroscopeapp]

Approvability

Verdict: Needs human review

This PR wires provider_routing, fallback_model, and reasoning_config from config.yaml through to AIAgent, enabling configuration-driven provider selection that wasn't previously active. Additionally, it changes Docker dependency installation from [all] to explicit extras. These changes affect runtime behavior and warrant human review.

^{You can customize Macroscope's approvability policy. Learn more.}