• fix(opencode-compat): preserve chronological part ordering across interleaved tool and text streams #133
• fix: delay idle event until turn finishes #132 👈 (View in Graphite)
• main

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

• merge queue - adds this PR to the back of the merge queue
• hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

This stack of pull requests is managed by Graphite. Learn more about stacking.

Merge activity

• Feb 7, 9:09 AM UTC: NathanFlurry added this pull request to the Graphite merge queue.
• Feb 7, 9:11 AM UTC: Merged by the Graphite merge queue.

Code Review

Issue: Idle events may be permanently lost when suppressed

Location: opencode_compat.rs:1827-1832

Problem: When turn.completed or session.idle events arrive while open_tool_calls is non-empty, the code returns early (line 1832), successfully suppressing premature idle event emission. However, when tool calls are subsequently removed from open_tool_calls (at lines 2304 and 2576), there is no mechanism to check whether the set has become empty and emit the previously-suppressed idle event.

Impact: The session.status (idle) and session.idle events will be permanently lost for that turn. The OpenCode client will never receive the idle signal, potentially leaving the UI stuck in a "busy" or "thinking" state indefinitely until the session ends.

Root cause: The PR implements the "defer" mechanism (suppress when tool calls are open) but is missing the "resume" mechanism (emit when tool calls drain).

Suggested fix: Track whether an idle event was deferred, then emit it when open_tool_calls becomes empty:

1. Add a deferred_idle: bool field to OpenCodeSessionRuntime
2. At line 1831, instead of returning early, set runtime.deferred_idle = true
3. At lines 2304 and 2576 after removing from open_tool_calls, check:

   if runtime.open_tool_calls.is_empty() && runtime.deferred_idle {
       runtime.deferred_idle = false;
       // emit the session.status (idle) and session.idle events here
   }

This ensures the idle event is delayed until tool calls complete, but not lost entirely.