Upgrade OpenAI Go SDK to v3.54.0 - #952
Conversation
There was a problem hiding this comment.
Copilot review overview
Review tier: Lite
Findings: 2
New issues introduced by this change (2)
| Severity | Finding |
|---|---|
provider/openaiprovider/responses.go — call_id is optional in the Responses API; always setting it to an empty string can change request… |
|
provider/foundryprovider/agent.go — For Foundry project endpoints, api-version must be removed. Currently… |
What changed in this PR
This PR upgrades the OpenAI Go SDK (github.com/openai/openai-go/v3) to v3.54.0 and updates the OpenAI Responses and Foundry providers to remain compatible with SDK changes around function-call correlation, MCP error shapes, and Azure authentication/endpoint behavior.
Changes:
- Upgrade
github.com/openai/openai-go/v3from v3.52.0 to v3.54.0. - Add a helper to set Responses function-call output
call_id(now optional in the SDK) and update call sites to preserve tool-call correlation. - Refactor Foundry agent client setup to satisfy the SDK’s stricter Azure endpoint/token-credential behavior while keeping Foundry routing and project/server-agent differences.
| File | Description |
|---|---|
| provider/openaiprovider/responses.go | Adds helper for function-call output items and sets optional call_id for correlation. |
| provider/openaiprovider/responses_test.go | Updates MCP error fixtures to match the SDK’s structured error representation. |
| provider/foundryprovider/agent.go | Refactors Foundry OpenAI client setup (endpoint options + middleware) for SDK Azure contract changes. |
| provider/foundryprovider/agent_test.go | Updates Foundry request assertions (api-version omission for project mode; User-Agent check). |
| go.mod | Bumps OpenAI Go SDK dependency to v3.54.0. |
| go.sum | Updates checksums (including new transitive deps). |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This comment has been minimized.
This comment has been minimized.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
|
Scope: internal-only This PR upgrades
|

Summary
github.com/openai/openai-go/v3from v3.52.0 to v3.54.0.call_idAPI while preserving call correlation.Partially supersedes #943 by extracting its OpenAI SDK upgrade and compatibility fixes. #943 still contains the remaining dependency updates.
Testing
go test ./provider/openaiprovider ./provider/foundryprovider -count=1go mod tidy -diffgit diff --check