Skip to content

fix(provider): wire reasoningEffort through OpenCode Go and Zen responses - #1555

Merged
TheGreatAxios merged 1 commit into
mainfrom
cl-11323-wire-reasoningeffort-through-opencode-go-and-zen-openai
Oct 23, 2026
Merged

TheGreatAxios merged 1 commit into
mainfrom
cl-11323-wire-reasoningeffort-through-opencode-go-and-zen-openai

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Oct 23, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Set effortOption: "reasoning_effort" on hostQuirks.reasoning in src/provider/openai-responses.ts so @corbits/openai-responses serializes reasoning.effort when providerOptions.reasoning_effort is present
  • Forward fields.reasoningEffort into defaults.providerOptions.reasoning_effort on the openai-responses branch of buildGoSource and buildZenSource in src/config/index.ts

Verification

  • bun run typecheck, bun run build, and bun run test pass
  • Unit tests in src/provider/openai-responses.test.ts and src/config/inference-sources.test.ts verify reasoning.effort serialization and source option wiring

Fixes CL-11323

@linear-code

linear-code Bot commented Oct 23, 2026

Copy link
Copy Markdown

CL-11323

@corbits-triage

corbits-triage Bot commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Triage: Pass

No changes needed from you. A maintainer will review this pull request.

@TheGreatAxios
TheGreatAxios force-pushed the cl-11323-wire-reasoningeffort-through-opencode-go-and-zen-openai branch from afd75ca to 7fd2bbe Compare October 23, 2026 16:41
@TheGreatAxios
TheGreatAxios merged commit 5c675ca into main Oct 23, 2026
14 checks passed
@TheGreatAxios
TheGreatAxios deleted the cl-11323-wire-reasoningeffort-through-opencode-go-and-zen-openai branch October 23, 2026 19:59
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