Skip to content

Populate resolved_model from OpenAI Chat response model - #1

Open
Wisernage wants to merge 1 commit into
mainfrom
fix/openai-resolved-model-1681
Open

Wisernage wants to merge 1 commit into
mainfrom
fix/openai-resolved-model-1681

Conversation

@Wisernage

@Wisernage Wisernage commented Sep 19, 2026

Copy link
Copy Markdown

Summary

Implements simonw#1681.

Call response.set_resolved_model() from the OpenAI Chat/AsyncChat response model field when it is present and differs from the requested model_id. Zero extra API calls.

OpenJobs

Note: upstream PR simonw#1682 (adity982) is a separate broader draft. This PR stays scoped to Chat/AsyncChat as requested in simonw#1681 / the OpenJobs listing.

Test plan

  • pytest tests/test_openai_resolved_model.py (20 passed)

Call response.set_resolved_model() from the OpenAI response model field
when it is nonempty and differs from the configured model id. Covers
Chat, AsyncChat, Responses, AsyncResponses, and Completion. Fixes simonw#1681.
@Wisernage
Wisernage force-pushed the fix/openai-resolved-model-1681 branch from 2077465 to 1b47e45 Compare September 19, 2026 14:35
@Wisernage Wisernage changed the title Populate resolved_model from OpenAI Chat response model fix(openai): record provider-resolved model provenance (#1681) Sep 19, 2026
@Wisernage
Wisernage force-pushed the fix/openai-resolved-model-1681 branch 2 times, most recently from 2077465 to 1b47e45 Compare September 19, 2026 14:36
@Wisernage Wisernage changed the title fix(openai): record provider-resolved model provenance (#1681) Populate resolved_model from OpenAI Chat response model Sep 19, 2026
@Wisernage
Wisernage force-pushed the fix/openai-resolved-model-1681 branch from 1b47e45 to 2077465 Compare September 19, 2026 14:36
@Wisernage Wisernage changed the title Populate resolved_model from OpenAI Chat response model fix(openai): record provider-resolved model provenance (#1681) Sep 19, 2026
@Wisernage
Wisernage force-pushed the fix/openai-resolved-model-1681 branch 2 times, most recently from 1b47e45 to 2077465 Compare September 19, 2026 14:37
@Wisernage Wisernage changed the title fix(openai): record provider-resolved model provenance (#1681) Populate resolved_model from OpenAI Chat response model Sep 19, 2026
@Wisernage
Wisernage force-pushed the fix/openai-resolved-model-1681 branch 3 times, most recently from 2077465 to 1b47e45 Compare September 19, 2026 16:07
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