Skip to content

docs(openai): note reasoning is not carried across turns in stateless responses mode - #5294

Closed
yonib05 wants to merge 1 commit into
strands-agents:mainfrom
yonib05:docs/responses-reasoning-limitation
Closed

yonib05 wants to merge 1 commit into
strands-agents:mainfrom
yonib05:docs/responses-reasoning-limitation

Conversation

@yonib05

@yonib05 yonib05 commented Oct 20, 2026

Copy link
Copy Markdown
Member

Human Overview

Description

Users running OpenAI reasoning models through the Responses provider in the default stateless mode lose the model's earlier reasoning on every model call in a tool loop. Both SDKs drop reasoning blocks from history with only a log warning and never request reasoning.encrypted_content, so nothing on the docs page tells users this happens or how to avoid it.

This adds a short "Reasoning Models" section to the OpenAI Responses page that shows how to set reasoning effort through params, states the stateless-mode limitation for both SDKs, points to server-side conversation state (stateful), which chains on previous_response_id with store: true and so keeps prior reasoning on OpenAI's side, and links the tracking issue. The note can be removed once #3389 lands.

Related Issues

#3389

Documentation PR

This PR is the documentation change.

Type of Change

Documentation update

Testing

Checked the claims against strands-ts/src/models/openai/responses-adapter.ts (formatResponsesMessages, formatResponsesRequest) and strands-py/src/strands/models/openai_responses.py (_format_request_messages, _format_request). Ran npm run build in site/ (no broken links) and confirmed the section and <Syntax> toggle render. npm run typecheck passes; npm run format:check and npm test report the same pre-existing failures as clean main.

  • I ran hatch run prepare

Checklist

  • I have read the CONTRIBUTING document
  • I have reviewed and understand every line of code in this PR, including any generated by AI tools, and I can explain why it works
  • My change is focused and reasonably small; I have split unrelated work into separate PRs
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@github-actions github-actions Bot added documentation Documentation changes, improvements, additions, content updates, site improvements, examples, guides area-model Related to models or model providers size/xs labels Oct 20, 2026
@yonib05

yonib05 commented Oct 20, 2026

Copy link
Copy Markdown
Member Author

Closing: we're merging the fix itself (#3419) instead of documenting the limitation.

@yonib05 yonib05 closed this Oct 20, 2026

This branch is waiting to be deployed

1 waiting deployment
manual-approval — 2190f13a Waiting Oct 20, 2026 by yonib05 via Build and Deploy Preview #2959
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-model Related to models or model providers documentation Documentation changes, improvements, additions, content updates, site improvements, examples, guides size/xs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants