Skip to content

fix(core): request default OpenAI reasoning summaries#36648

Closed
opencode-agent[bot] wants to merge 1 commit into
v2from
thought-toggle
Closed

fix(core): request default OpenAI reasoning summaries#36648
opencode-agent[bot] wants to merge 1 commit into
v2from
thought-toggle

Conversation

@opencode-agent

@opencode-agent opencode-agent Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Summary

  • apply OpenAI Responses defaults when V2 resolves catalog-backed @ai-sdk/openai models
  • request reasoning.summary: "auto" for default GPT-5 reasoning models, not only explicit variants
  • retain stateless encrypted reasoning defaults through the shared OpenAI option normalizer

Root cause

V2 maps catalog-backed OpenAI models directly onto the native Responses route but skipped the OpenAI model default normalizer. Therefore the normal/default GPT-5 selection sent no reasoning.summary request. Explicit reasoning variants happened to add reasoningSummary: "auto", but default model use produced no visible summary events for the TUI.

Checks

  • bun test test/session-runner-model.test.ts (22 pass)
  • bun typecheck in packages/core
  • bun typecheck in packages/llm

@opencode-agent opencode-agent Bot changed the title fix(tui): reveal title-only reasoning summaries fix(core): request default OpenAI reasoning summaries Jul 13, 2026
@opencode-agent

Copy link
Copy Markdown
Contributor Author

Closing after clarification: OpenAI is returning the available reasoning summary headline; V2 is not dropping a fuller thought payload. The proposed change incorrectly treated provider behavior as a product bug.

@opencode-agent opencode-agent Bot closed this Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant