[timeleft--]

Summary

• Root cause: propose_truth calling OpenRouter via asyncio.wait_for with 60s timeout, but client_args={"timeout": scalar} left the httpx pool timeout unset (infinite). OpenRouter accepted the connection then never responded — 12+ hour hang in session b3147494.
• Fix 1 (critical): @with_tool_timeout decorator on handle_call_tool — wraps every MCP tool call in asyncio.wait_for, reads tool_timeout_secs from config at call time. Default: 300s.
• Fix 2 (critical): asyncio.wait_for around the Trust Gate LLM call specifically, using configurable trust_gate_timeout_secs. Default: 120s. Returns structured error on timeout instead of hanging indefinitely.
• Fix 3 (defense in depth): Explicit httpx.Timeout(connect=10, read=N, write=N, pool=10) — sets all httpx timeout phases explicitly so no phase defaults to infinite.
• Fix 4 (observability): Rotating file handler on MCP server logger → ~/.fava-trails/logs/mcp-server.log. Without this, server-side hangs are undiagnosable since Claude Code doesn't capture MCP stderr to disk.

Configuration

Both timeouts are configurable in config.yaml, set to 0 to disable:

tool_timeout_secs: 300      # outermost guard on every tool call
trust_gate_timeout_secs: 120  # Trust Gate LLM call guard

Test plan

• test_propose_truth_timeout_returns_error — Trust Gate LLM timeout returns structured error
• TestWithToolTimeout::test_timeout_returns_error_dict — decorator fires after configurable timeout
• TestWithToolTimeout::test_passes_through_on_success — no interference on normal calls
• TestWithToolTimeout::test_disabled_when_zero — timeout disabled when tool_timeout_secs=0
• 420/420 tests pass

🤖 Generated with Claude Code