hermes - 💡(How to fix) Fix CLI: completion_queue drain only runs on idle timeout, notify_on_complete notifications silently lost [1 pull requests]

Bug Description

In cli.py process_loop(), the completion_queue drain only ran when _pending_input.get(timeout=0.1) timed out — i.e., when the agent was idle. When the user sent a message, the drain was skipped entirely. If a background process exited during the agent's turn, the completion event sat in the queue until the post-turn drain. However, if the agent called process(action='poll') during the turn, _completion_consumed was set (see #10156 for the regression), and the post-turn drain skipped the event with continue. The notification was permanently lost.

This is a compounding factor to #10156. Even if #10156 is fixed, the idle-only drain window still delays notifications and creates races with wait()/log().

Root Cause

The drain logic was inside the except queue.Empty branch, meaning it only executed when no user input was pending. It should run on every loop iteration.

Proposed Fix

Move the completion_queue drain to run on every loop iteration, before the _pending_input.get() call.

Related Issues

• #10156 — _completion_consumed regression from PR #8228