chore(api): document decision choice constraints by jbeckwith-oai · Pull Request #4055 · openai/openai-python · GitHub
Skip to content

chore(api): document decision choice constraints - #4055

Merged
jbeckwith-oai merged 1 commit into
mainfrom
castiron/promotions/pr-230
Oct 11, 2026
Merged

chore(api): document decision choice constraints#4055
jbeckwith-oai merged 1 commit into
mainfrom
castiron/promotions/pr-230

Conversation

@jbeckwith-oai

@jbeckwith-oai jbeckwith-oai commented Oct 11, 2026

Copy link
Copy Markdown
Contributor

Summary

Clarifies the number and uniqueness requirements for decision choices.

Changes

  • Document that choice questions require between 2 and 255 unique choices.

@jbeckwith-oai
chore(api): document decision choice constraints
c126c10 
Castiron-Internal-PR: openai/openai-python-internal#230
Castiron-Source-SHA: d428118ffdb796e6dff24df7c9bb0b256b1767c0
Castiron-Public-Base-SHA: 16d3d3b
@github-actions

github-actions Bot commented Oct 11, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

markstuart-oai
markstuart-oai approved these changes Oct 11, 2026
View reviewed changes

@markstuart-oai markstuart-oai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Reviewed all three changed files at c126c1082bb33ea503dc76668721a7847515d502. No actionable findings.

The full schema comparison adds only the choice description: provide 2–255 choices, with each choice unique. The Python field documentation matches it. Executable Python code, type declarations and schema constraints are unchanged. The public generator snapshot resolves and has no .github/workflows directory.

Current-head CI passed. This was a source-only review; I ran no repository tests, builds or generation. I did not verify a public OkTest run for this draft PR.

@jbeckwith-oai
jbeckwith-oai marked this pull request as ready for review October 11, 2026 04:54
@jbeckwith-oai
jbeckwith-oai requested a review from a team as a code owner October 11, 2026 04:54
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 11, 2026 •
edited
Loading

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-11T04:55:23.531374Z c126c10 Draft marked ready
🔒 Security Review ✅ Completed 2026-10-11T04:56:16.551829Z c126c10 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@openai-sdks

openai-sdks Bot commented Oct 11, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

OkTest Summary

✅ 236/236 SDK tests passed in 8.444s for Python SDK PR #4055.

Test results — 42 files
Test Result Time
tests/chat-completions-complex-body.test.ts ✅ Passed 246ms
tests/chat-completions-create.test.ts ✅ Passed 231ms
tests/chat-completions-stream.test.ts ✅ Passed 270ms
tests/files-content-binary.test.ts ✅ Passed 214ms
tests/files-create-multipart.test.ts ✅ Passed 255ms
tests/files-list-pagination.test.ts ✅ Passed 195ms
tests/initialize-config.test.ts ✅ Passed 143ms
tests/instance-isolation.test.ts ✅ Passed 176ms
tests/models-list.test.ts ✅ Passed 166ms
tests/responses-background-lifecycle.test.ts ✅ Passed 225ms
tests/responses-body-method-errors.test.ts ✅ Passed 313ms
tests/responses-cancel-timeout.test.ts ✅ Passed 226ms
tests/responses-cancel.test.ts ✅ Passed 162ms
tests/responses-compact-retries.test.ts ✅ Passed 274ms
tests/responses-compact.test.ts ✅ Passed 239ms
tests/responses-create-advanced-stream.test.ts ✅ Passed 320ms
tests/responses-create-advanced.test.ts ✅ Passed 175ms
tests/responses-create-disconnect.test.ts ✅ Passed 1.193s
tests/responses-create-errors.test.ts ✅ Passed 305ms
tests/responses-create-malformed-api-responses.test.ts ✅ Passed 417ms
tests/responses-create-retries.test.ts ✅ Passed 408ms
tests/responses-create-stream-failures.test.ts ✅ Passed 184ms
tests/responses-create-stream-timeout.test.ts ✅ Passed 219ms
tests/responses-create-stream-wire.test.ts ✅ Passed 1.833s
tests/responses-create-stream.test.ts ✅ Passed 77ms
tests/responses-create-terminal-states.test.ts ✅ Passed 210ms
tests/responses-create-timeout.test.ts ✅ Passed 216ms
tests/responses-create.test.ts ✅ Passed 147ms
tests/responses-delete.test.ts ✅ Passed 174ms
tests/responses-input-items-errors.test.ts ✅ Passed 324ms
tests/responses-input-items-list.test.ts ✅ Passed 235ms
tests/responses-input-items-options.test.ts ✅ Passed 231ms
tests/responses-input-tokens-count-timeout.test.ts ✅ Passed 276ms
tests/responses-input-tokens-count.test.ts ✅ Passed 144ms
tests/responses-malformed-inputs.test.ts ✅ Passed 1.04s
tests/responses-not-found-errors.test.ts ✅ Passed 191ms
tests/responses-parse.test.ts ✅ Passed 193ms
tests/responses-retrieve-retries.test.ts ✅ Passed 239ms
tests/responses-retrieve.test.ts ✅ Passed 146ms
tests/responses-stored-method-errors.test.ts ✅ Passed 521ms
tests/retry-behavior.test.ts ✅ Passed 2.825s
tests/sdk-error-shape.test.ts ✅ Passed 640ms

View OkTest run #38113050875

SDK merge (c53d15800162) · head (c126c1082bb3) · base (16d3d3b3c33b) · OkTest (59813f410948)

@jbeckwith-oai
jbeckwith-oai added this pull request to the merge queue Oct 11, 2026
Merged via the queue into main with commit b7a1990 Oct 11, 2026
28 checks passed
@jbeckwith-oai
jbeckwith-oai deleted the castiron/promotions/pr-230 branch October 11, 2026 05:03
@openai-sdks openai-sdks Bot mentioned this pull request Oct 11, 2026
Merged
opensource-fork-sync Bot pushed a commit to cla7aye15I4nd/opensource-openai__openai-python that referenced this pull request Oct 11, 2026
@openai-sdks
release: 3.29.0 (openai#4048)
7e1643a 
Automated Release PR
---


##
[3.29.0](openai/openai-python@v3.28.0...v3.29.0)
(2026-10-11)


### Features

* **api:** add billing_not_active session error code
([openai#4052](openai#4052))
([c88b320](openai@c88b320))
* **api:** add shida to named Live voices
([openai#4053](openai#4053))
([7352cf5](openai@7352cf5))
* **api:** add vault updates and metadata filters
([openai#4047](openai#4047))
([e72f493](openai@e72f493))
* **api:** align beta pagination limits and non-null parameters
([openai#4057](openai#4057))
([800899e](openai@800899e))


### Chores

* **api:** clarify decision creation documentation
([openai#4054](openai#4054))
([16d3d3b](openai@16d3d3b))
* **api:** clarify Decisions question and answer descriptions
([openai#4051](openai#4051))
([895d617](openai@895d617))
* **api:** document 128-character safety identifiers
([openai#4050](openai#4050))
([2c1ae52](openai@2c1ae52))
* **api:** document decision choice constraints
([openai#4055](openai#4055))
([b7a1990](openai@b7a1990))
* **api:** document Files API rate-limit responses
([openai#4049](openai#4049))
([f6a3459](openai@f6a3459))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: openai-sdks[bot] <284451331+openai-sdks[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@markstuart-oai markstuart-oai markstuart-oai approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jbeckwith-oai @markstuart-oai