fix(llm): keep OpenAI-compatible reasoning streams active#89440
Conversation
|
ClawSweeper status: review started. I am starting a fresh review of this pull request: fix(llm): keep OpenAI-compatible reasoning streams active This is item 1/1 in the current shard. Shard 0/1. This placeholder means the worker is alive and reading the current context. I will edit this same comment with the actual review when the claws are done clicking. Crustacean status: shell secured, claws on keyboard, evidence pebbles being sorted. |
|
Land-ready proof for exact head Local proof:
CI proof:
Known gap: no live reporter Vertex sidecar was available; regression models the reported OpenAI-compatible SSE shape with positive |
Summary:
completion_tokens_details.reasoning_tokensas reasoning activity when the chunk otherwise emits no assistant stream event.thinking_deltapattern used by the native Google transport so the LLM idle watchdog sees progress without leaking visible text.Fixes #84384
Verification:
node scripts/run-vitest.mjs src/agents/openai-transport-stream.test.ts src/agents/embedded-agent-runner/run/llm-idle-timeout.test.tsnode ./node_modules/.bin/oxfmt --check src/agents/openai-transport-stream.ts src/agents/openai-transport-stream.test.ts && git diff --checkpnpm check:test-types.agents/skills/autoreview/scripts/autoreview --mode local(clean: no accepted/actionable findings)