Repository navigation
[pull] main from openai:main - #168
Merged
Merged
Conversation
…" (#3929) Reverts #3923. This reverts commit 40c334f. The generated SDK changes merged before their source-of-truth changes. This restores the preceding SDK definitions and generation metadata. The resulting tracked tree exactly matches the reverted commit’s parent. Validation: the complete patch is byte-for-byte the inverse of the named commit. Version manifests and release configuration are unchanged. Local test suites and new review rounds were not run for this urgent revert; current-head CI will validate this PR. Current CI: lint, build, Python 3.10/3.14/HTTPX2 tests, CodeQL and Castiron checks pass. [Breaking-change detection](https://github.com/openai/openai-python/actions/runs/35622466455) fails on the 11 removed `LocalShellCallOutput.call_id` attributes introduced by #3923. Those removals are the requested exact revert. The failure remains visible; no detector, release policy or version metadata was changed. CI diagnosis: the required [breaking-change detector](https://github.com/openai/openai-python/actions/runs/35622466455) compares against current PR base `40c334f028529c0b5e8dac65f30c76b314930767`. All 11 flagged `LocalShellCallOutput.call_id` attributes were introduced by #3923 and are deliberately removed by its exact inverse. The full tracked tree equals pre-promotion `69a2c1db6feacf32be6693809e7cab1c3b49cad7`; focused AST comparison also confirms every other field is unchanged. The existing workflow/detector has no supported unreleased-revert baseline selection. An explicit repository policy for validating unreleased reverts against their verified pre-promotion baseline is needed to reconcile the required check with this rollback. The detector failure remains visible; no attribute restoration, removal allowlist, disabled check, forced-success status, retry, or major-version marker was added.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )