fix(api): correct tool and tool-output schema tags - #3924
markstuart-oai wants to merge 1 commit into
Conversation
Castiron custom code✅ No new custom-code files detected. 47 mixed files remain; 0 existing customizations changed. Compared 47 existing customizations unchanged
7 more in the full report. A changed generated baseline means this report cannot reliably identify which handwritten lines changed. Inspect the custom-code diffDownload the exact patch produced by this run (requires repository access): gh run download 35576165858 --repo openai/openai-python \
--name castiron-custom-code-35576165858-1 --dir /tmp/castiron-custom-code-35576165858-1
git apply --stat /tmp/castiron-custom-code-35576165858-1/custom-code.patch
cat /tmp/castiron-custom-code-35576165858-1/custom-code.patchOr reproduce it from an SDK checkout containing the vendored reporter: git fetch --no-tags origin 69a2c1db6feacf32be6693809e7cab1c3b49cad7 323a092d0129408c586918d922f90bea0e390720
python3 scripts/castiron/custom_code_report.py report \
--base 69a2c1db6feacf32be6693809e7cab1c3b49cad7 \
--head 323a092d0129408c586918d922f90bea0e390720 --fetch --require-head-hash --public \
--out /tmp/castiron-custom-code-323a092d0129
cat /tmp/castiron-custom-code-323a092d0129/custom-code.patchThis is the current full custom patch for mixed files, not an attribution of only the handwritten lines changed by this PR. |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Castiron-Internal-PR: openai/openai-python-internal#129 Castiron-Source-SHA: 31e15a5dd49639e813ecd0d07880c054d9e7dd4c Castiron-Public-Base-SHA: 69a2c1d
27b7c2a to
323a092
Compare
Summary
Clarifies tool-output content tags and the apply-patch tool tag while preserving existing tool types.
Changes