Skip to content

fix(ai): default responses to encrypted reasoning - #44863

Merged
rekram1-node merged 3 commits into
v2from
encrypted-reasoning-defaults
Aug 25, 2026
Merged

rekram1-node merged 3 commits into
v2from
encrypted-reasoning-defaults

Conversation

@rekram1-node

Copy link
Copy Markdown
Collaborator

Summary

  • default the provider-neutral Open Responses route to store: false and include: ["reasoning.encrypted_content"]
  • explicitly apply the same defaults to the independent OpenAI and xAI Responses routes
  • preserve request-level storage overrides and include: [] opt-outs
  • update inherited route assertions, WebSocket payload expectations, and affected recorded request fixtures

Why

Stateless Responses conversations can replay prior reasoning only when the provider returns encrypted reasoning state. Requesting it consistently prevents reasoning from being dropped on tool-result and follow-up requests.

Verification

  • bun run typecheck
  • 123 focused OpenAI, compatible Responses, and xAI tests
  • 9 affected recorded/provider tests
  • 2 inherited provider-default tests

Unrelated existing Anthropic provider-package and recorded PDF failures reproduce on the unmodified baseline and are outside this PR.

Default Open Responses, OpenAI, and xAI routes to stateless requests that include encrypted reasoning. Preserve explicit caller overrides and update recorded request fixtures for inherited provider routes.
@rekram1-node
rekram1-node merged commit 1b30098 into v2 Aug 25, 2026
6 of 7 checks passed
@rekram1-node
rekram1-node deleted the encrypted-reasoning-defaults branch August 25, 2026 04:43
jinhuang712 pushed a commit to jinhuang712/opencode that referenced this pull request Sep 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant