Sub-agents stream `assistant.reasoning_delta` but never emit the `assistant.reasoning` close event documented as always-sent
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 45/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Quiet
- Tech stack
- javascript, python
Research direction
Start in sdk/index.js around createAgentCallbackBridge (~L4554) and compare it with the main-agent path near ~L4541. Review session-events.schema.json lines 520-522 and the README contract, then determine whether sub-agents should emit a matching assistant.reasoning event or document the assistant.message shape. Done means the chosen behavior is implemented consistently and verified against the described reasoningId event sequence.
Written by the indexing model from the issue text.
Description
Summary
Per the Python SDK README:
Note:
assistant.messageandassistant.reasoning(final events) are always sent regardless of streaming setting.
And per the schema docstring on AssistantReasoningDeltaData.reasoningId (session-events.schema.json line 520-522):
Reasoning block ID this delta belongs to, matching the corresponding
assistant.reasoningevent.
In practice, sub-agents stream assistant.reasoning_delta events (with agentId set on the event), but the matching assistant.reasoning close is never emitted. Instead the reasoning content lands in the sub-agent's assistant.message.reasoningText / reasoningOpaque fields. Consumers that rely on the documented per-reasoningId close event end up with permanently-orphaned streaming reasoning blocks for every sub-agent turn.
Environment
@github/copilot(CLI bundle):1.0.49github-copilot-sdk(Python):1.0.0b4- Default provider (not BYOK)
- Reasoning model in use (Claude Opus / Sonnet via the default backend)
What I'd expect (per docs)
For every sub-agent reasoning block: one or more assistant.reasoning_delta events with a given reasoningId, followed by exactly one assistant.reasoning event carrying that same reasoningId and the complete content — symmetric with master-agent reasoning.
What actually happens
Code path (built bundle, sdk/index.js ~ L4554, inside createAgentCallbackBridge):
case "reasoning":
u.reasoningId && u.deltaContent &&
this.emitEphemeral("assistant.reasoning_delta",
{ reasoningId: u.reasoningId, deltaContent: u.deltaContent }, r);
// ...later, when the sub-agent's response completes:
this.emit("assistant.message",
{ parentToolCallId: r, messageId: kr(), ..., reasoningOpaque, reasoningText }, r);
// No paired emit("assistant.reasoning", ...) here.
Compare to the main-agent path (sdk/index.js ~ L4541) which does:
this.emit("assistant.message", { ... });
Ho && this.emitEphemeral("assistant.reasoning", { reasoningId: Ho, content: Yi ?? "" });
So the close emit is conditional on the main-agent code path and absent from the sub-agent bridge.
Empirical evidence
Across six debug-mode session JSONL transcripts (~17K reasoning_delta events total) captured from a production run with multiple custom sub-agents (planner, implementer, profiler_agent):
- Every
assistant.reasoning_deltacarrying a sub-agent'sagentIdis never followed by a matchingassistant.reasoning. - Sub-agent
assistant.messageevents for those same turns do carry non-emptyreasoningText(often several KB) andreasoningOpaque, so the data is being retained — it just travels via the message payload instead of the documented close event. - The two log files in the same run that contain no sub-agent activity (master-only) have zero orphan reasoning streams. The four files with sub-agent activity have 25 orphan streams collectively. The pattern is deterministic and 1:1 with sub-agent boundaries.
Suggested fixes (either or both)
- Behavior fix: also emit
assistant.reasoningfrom the sub-agent bridge afterendCurrentStreamingMessage, mirroring the main-agent path. This keeps the documented contract (1:1reasoning_delta→assistant.reasoning) and matches downstream consumer expectations. - Documentation fix: if the embed-in-
assistant.messageshape is intentional for sub-agents, document it explicitly onAssistantReasoningDeltaData(e.g. "for sub-agent reasoning, the closingassistant.messageevent for the sameagentIdcarries the complete content inreasoningText/reasoningOpaque; no separateassistant.reasoningevent is emitted") and in the streaming-events guide referenced by #922.
Related: #922 (CLOSED — "Extended thinking/reasoning events not firing for OpenAI and Anthropic models via Copilot SDK"), #1064 (CLOSED — "BYOK Anthropic provider missing multiple session events ... reasoning ..."). Same family of bug but neither covers this specific main-vs-sub-agent asymmetry on the default provider.
Happy to provide the full JSONL fixtures if helpful.
- Dominant language
- Java
- Stars
- 10.5k
- Forks
- 1.5k
- Avg merge
- 1d 9h
- Merged PRs (30d)
- 131
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from github/copilot-sdk
-
agentic-workflows
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
github/copilot-sdk#2709 · 1 comment ·
-
Difficulty 1/5 Under an hour Newbie friendliness 78/100
github/copilot-sdk#2673 ·
-
bug testing
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
github/copilot-sdk#2628 ·
-
agentic-workflows
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
github/copilot-sdk#2627 · 1 comment ·
-
agentic-workflows
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
github/copilot-sdk#2493 ·
All issues in github/copilot-sdk
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
elastic/gradle-plugins#157 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
cryptomator/hub#497 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
johanhaleby/occurrent#1120 ·