lsm / neokai / 27051408741

82%

push

github

fix(daemon): include reasoning.summary_text in OpenAI Responses bridge (#2070)

* fix(daemon): include reasoning.summary_text in OpenAI Responses bridge requests

The OpenAI Responses API only streams reasoning summary text deltas
(response.reasoning_summary_text.delta) when 'reasoning.summary_text' is
present in the request's 'include' array. The bridge previously only
included 'reasoning.encrypted_content', which is sufficient for multi-turn
stateless continuation but does not trigger streaming of reasoning summaries.

Without reasoning.summary_text in include, the bridge receives no reasoning
events from the upstream API, so no Anthropic thinking SSE blocks are emitted,
and thinking blocks never appear in the UI for Codex sessions.

Fix: add 'reasoning.summary_text' to the include array alongside the existing
'reasoning.encrypted_content' whenever reasoning is enabled or cached reasoning
items are being continued.

Update unit tests that assert on the include array.

* fix(daemon): override non-enabled SDK thinking with session side-channel config

The SDK can send body.thinking = {type:'adaptive'} internally. The bridge's
injection guard only checked !body.thinking, so the session's explicit
{type:'enabled', budget_tokens:N} config was ignored when the SDK sent an
adaptive payload. mapThinkingToReasoningEffort then returned undefined
(because type !== 'enabled'), so no reasoning or include fields reached the
OpenAI API and thinking blocks never appeared.

Fix: inject the session thinking config whenever body.thinking is absent or
its type is not 'enabled'. This preserves request-level enabled thinking
while overriding adaptive/disabled payloads from the SDK.

Add unit test covering the adaptive-payload override path.

* fix(daemon): gate reasoning.summary_text behind non-ChatGPT OAuth path

The ChatGPT Codex endpoint (chatgpt.com/backend-api/codex) rejects
'reasoning.summary_text' in the include array with:
  400 Invalid value: 'reasoning.summary_... (continued)

10076 of 14751 branches covered (68.31%)

Branch coverage included in aggregate %.

10 of 10 new or added lines in 1 file covered. (100.0%)

85080 of 100129 relevant lines covered (84.97%)

293.95 hits per line