Skip to content

schemas: allow extra fields in extra_body and tools#266

Merged
lowpolyneko merged 4 commits into
argonne-lcf:mainfrom
lowpolyneko:allow-extra-body
Jul 20, 2026
Merged

schemas: allow extra fields in extra_body and tools#266
lowpolyneko merged 4 commits into
argonne-lcf:mainfrom
lowpolyneko:allow-extra-body

Conversation

@lowpolyneko

Copy link
Copy Markdown
Collaborator

This PR allows the bare minimum of extra fields to the OpenAI Chat Completion schema validation to allow Codex to work with FIRST.

@lowpolyneko lowpolyneko self-assigned this Jul 13, 2026

@masalim2 masalim2 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks a pretty focused and conservative change, which is great. Thanks!

@lowpolyneko
lowpolyneko merged commit 8e45871 into argonne-lcf:main Jul 20, 2026
1 check passed
@lowpolyneko
lowpolyneko deleted the allow-extra-body branch July 20, 2026 19:11
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.

2 participants