fix: force store for openai-responses providers by EdwardWu7 · Pull Request #22441 · openclaw/openclaw

EdwardWu7 · 2026-02-21T05:27:11Z

Summary

Force store: true for all openai-responses providers, not just direct OpenAI hosts.
Prevents previous_response_id failures on Azure/OpenAI-compatible endpoints when store defaults to false.

Notes

Responses API multi-turn requires persisted items; Azure returns 400 when store is false.

Greptile Summary

Simplifies shouldForceResponsesStore to force store=true for all openai-responses API providers, removing the previous restriction to direct OpenAI hosts. This prevents 400 errors on Azure and OpenAI-compatible endpoints that require persisted items for multi-turn conversations.

The existing test at src/agents/pi-embedded-runner-extraparams.e2e.test.ts:267-279 expects store=false for proxy URLs and will need to be updated
Comment on line 13 should be updated to reflect the new behavior (no longer limited to "direct" OpenAI)

Confidence Score: 4/5

This PR is safe to merge after updating the test and comment
The logic change is straightforward and addresses a real bug with Azure/compatible endpoints. However, there's a test that will break and a comment that needs updating, which are easy fixes but should be addressed before merging.
Check that the existing test at src/agents/pi-embedded-runner-extraparams.e2e.test.ts:267-279 is updated to match the new behavior

_{Last reviewed commit: f3e1ad3}

_{(2/5) Greptile learns from your feedback when you react with thumbs up/down!}

greptile-apps

_{1 file reviewed, 2 comments}

_{Edit Code Review Agent Settings | Greptile}

greptile-apps · 2026-02-21T05:29:52Z

Additional Comments (2)

src/agents/pi-embedded-runner/extra-params.ts
comment is outdated - it says "only force for direct OpenAI Responses" but the code now forces store for all openai-responses providers regardless of baseURL

// NOTE: Force `store=true` for all OpenAI Responses API providers (OpenAI, Azure, compatible endpoints).
// Codex responses (chatgpt.com/backend-api/codex/responses) require `store=false`.

Prompt To Fix With AI

This is a comment left during a code review.
Path: src/agents/pi-embedded-runner/extra-params.ts
Line: 13-14

Comment:
comment is outdated - it says "only force for *direct* OpenAI Responses" but the code now forces store for all `openai-responses` providers regardless of baseURL

```suggestion
// NOTE: Force `store=true` for all OpenAI Responses API providers (OpenAI, Azure, compatible endpoints).
// Codex responses (chatgpt.com/backend-api/codex/responses) require `store=false`.
```

How can I resolve this? If you propose a fix, please make it concise.

src/agents/pi-embedded-runner/extra-params.ts
removing the provider and baseURL checks will break the existing test on line 267-279 of src/agents/pi-embedded-runner-extraparams.e2e.test.ts that expects store=false for non-OpenAI base URLs (proxies)

the test should be updated or removed to match the new behavior

Prompt To Fix With AI

This is a comment left during a code review.
Path: src/agents/pi-embedded-runner/extra-params.ts
Line: 128-140

Comment:
removing the provider and baseURL checks will break the existing test on line 267-279 of `src/agents/pi-embedded-runner-extraparams.e2e.test.ts` that expects `store=false` for non-OpenAI base URLs (proxies)

the test should be updated or removed to match the new behavior

How can I resolve this? If you propose a fix, please make it concise.

, #22441, #25088) Landed from contributor PRs #16930, #22441, and #25088. Co-authored-by: liweiguang <[email protected]> Co-authored-by: EdwardWu7 <[email protected]> Co-authored-by: MoerAI <[email protected]>

steipete · 2026-02-27T16:16:14Z

Landed on main as 8da3a9a92d9c2c6db6fa6caa881e9310d1792e93.

Rewrite summary:

Kept fix: force store for openai-responses providers #22441 intent (Responses store safety) but merged with stricter direct-host and compat guards.
Unified with fix: treat empty baseUrl as non-direct for OpenAI Responses #16930 + fix(agents): skip forced store=true when model declares supportsStore=false #25088 into one OpenAI Responses context wrapper.
Added automatic OpenAI server-side context_management compaction injection for compatible direct OpenAI models.
Added tests, docs, and one unified changelog entry.

SHA notes:

Original PR commit: f3e1ad36870c9d1f9149745d604b34e68fe2e5f2
Landed commit: 8da3a9a92d9c2c6db6fa6caa881e9310d1792e93

Thanks @EdwardWu7.

…nclaw#16930, openclaw#22441, openclaw#25088) Landed from contributor PRs openclaw#16930, openclaw#22441, and openclaw#25088. Co-authored-by: liweiguang <[email protected]> Co-authored-by: EdwardWu7 <[email protected]> Co-authored-by: MoerAI <[email protected]>

…nclaw#16930, openclaw#22441, openclaw#25088) Landed from contributor PRs openclaw#16930, openclaw#22441, and openclaw#25088. Co-authored-by: liweiguang <[email protected]> Co-authored-by: EdwardWu7 <[email protected]> Co-authored-by: MoerAI <[email protected]> (cherry picked from commit c38be8f)

…nclaw#16930, openclaw#22441, openclaw#25088) Landed from contributor PRs openclaw#16930, openclaw#22441, and openclaw#25088. Co-authored-by: liweiguang <[email protected]> Co-authored-by: EdwardWu7 <[email protected]> Co-authored-by: MoerAI <[email protected]>

fix: force store for openai-responses providers

f3e1ad3

openclaw-barnacle bot added agents Agent runtime and tooling size: XS labels Feb 21, 2026

greptile-apps bot reviewed Feb 21, 2026

View reviewed changes

steipete self-assigned this Feb 27, 2026

This was referenced Feb 27, 2026

fix: treat empty baseUrl as non-direct for OpenAI Responses #16930

Closed

fix(agents): skip forced store=true when model declares supportsStore=false #25088

Closed

steipete closed this Feb 27, 2026

gemini-code-assist bot mentioned this pull request Mar 1, 2026

chore(sync): replay rollup fixes on upstream/main MillionthOdin16/openclaw#111

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix: force store for openai-responses providers#22441

fix: force store for openai-responses providers#22441
EdwardWu7 wants to merge 1 commit intoopenclaw:mainfrom
EdwardWu7:fix/responses-store-all-providers

EdwardWu7 commented Feb 21, 2026 •

edited by greptile-apps bot

Loading

Uh oh!

greptile-apps bot left a comment

Uh oh!

greptile-apps bot commented Feb 21, 2026

Uh oh!

steipete commented Feb 27, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

EdwardWu7 commented Feb 21, 2026 • edited by greptile-apps bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Notes

Greptile Summary

Confidence Score: 4/5

Uh oh!

greptile-apps bot left a comment

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot commented Feb 21, 2026

Uh oh!

steipete commented Feb 27, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

EdwardWu7 commented Feb 21, 2026 •

edited by greptile-apps bot

Loading