Skip to content

fix(openai): drop stop from Responses API payload - #38336

Merged
Mason Daugherty (mdrxy) merged 2 commits into
masterfrom
mdrxy/openai-drop-stop-responses-api
Jun 20, 2026
Merged

fix(openai): drop stop from Responses API payload#38336
Mason Daugherty (mdrxy) merged 2 commits into
masterfrom
mdrxy/openai-drop-stop-responses-api

Conversation

@mdrxy

@mdrxy Mason Daugherty (mdrxy) commented Jun 20, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Responses API requests now strip the Chat Completions-only stop parameter before sending the payload. This avoids request rejection while preserving stop for the Chat Completions API path.

@github-actions github-actions Bot added fix For PRs that implement a fix integration PR made that is related to a provider partner package integration internal openai `langchain-openai` package issues & PRs size: XS < 50 LOC labels Jun 20, 2026
open-swe[bot]
open-swe Bot reviewed Jun 20, 2026
View reviewed changes

@open-swe open-swe Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Open SWE Review: No issues found

Open SWE reviewed this PR and found no potential bugs to report.

Open in Web • View Open SWE trace

@mdrxy
Mason Daugherty (mdrxy) merged commit 7d1a383 into master Jun 20, 2026
109 of 110 checks passed
@mdrxy
Mason Daugherty (mdrxy) deleted the mdrxy/openai-drop-stop-responses-api branch June 20, 2026 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@open-swe open-swe[bot] open-swe[bot] left review comments
@ccurme ccurme (ccurme) Awaiting requested review from ccurme ccurme is a code owner

Assignees

No one assigned

Labels

fix For PRs that implement a fix integration PR made that is related to a provider partner package integration internal openai `langchain-openai` package issues & PRs size: XS < 50 LOC

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@mdrxy