chore(api): clarify decision creation documentation by jbeckwith-oai · Pull Request #4054 · openai/openai-python · GitHub
Skip to content

chore(api): clarify decision creation documentation - #4054

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

chore(api): clarify decision creation documentation#4054
jbeckwith-oai merged 1 commit into
mainfrom
castiron/promotions/pr-229

Conversation

@jbeckwith-oai

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

Copy link
Copy Markdown
Contributor

Summary

Clarifies the input formats and refusal results for decision creation.

Changes

  • Describe text and image inputs, the 128-image limit, and per-question refusal results.

@jbeckwith-oai
chore(api): clarify decision creation documentation
26da120 
Castiron-Internal-PR: openai/openai-python-internal#229
Castiron-Source-SHA: a72d57182d77c7f1e8a8360ba0e5124a36bc75f2
Castiron-Public-Base-SHA: 7352cf5
@jbeckwith-oai
jbeckwith-oai marked this pull request as ready for review October 11, 2026 02:58
@jbeckwith-oai
jbeckwith-oai requested a review from a team as a code owner October 11, 2026 02:58
@github-actions

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

Copy link
Copy Markdown
Contributor

@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-11T02:59:19.877210Z 26da120 Draft marked ready
🔒 Security Review ✅ Completed 2026-10-11T03:00:09.583455Z 26da120 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 9.229s for Python SDK PR #4054.

Test results — 42 files
Test Result Time
tests/chat-completions-complex-body.test.ts ✅ Passed 188ms
tests/chat-completions-create.test.ts ✅ Passed 257ms
tests/chat-completions-stream.test.ts ✅ Passed 291ms
tests/files-content-binary.test.ts ✅ Passed 198ms
tests/files-create-multipart.test.ts ✅ Passed 223ms
tests/files-list-pagination.test.ts ✅ Passed 199ms
tests/initialize-config.test.ts ✅ Passed 179ms
tests/instance-isolation.test.ts ✅ Passed 183ms
tests/models-list.test.ts ✅ Passed 277ms
tests/responses-background-lifecycle.test.ts ✅ Passed 230ms
tests/responses-body-method-errors.test.ts ✅ Passed 296ms
tests/responses-cancel-timeout.test.ts ✅ Passed 216ms
tests/responses-cancel.test.ts ✅ Passed 199ms
tests/responses-compact-retries.test.ts ✅ Passed 296ms
tests/responses-compact.test.ts ✅ Passed 308ms
tests/responses-create-advanced-stream.test.ts ✅ Passed 169ms
tests/responses-create-advanced.test.ts ✅ Passed 232ms
tests/responses-create-disconnect.test.ts ✅ Passed 1.227s
tests/responses-create-errors.test.ts ✅ Passed 198ms
tests/responses-create-malformed-api-responses.test.ts ✅ Passed 548ms
tests/responses-create-retries.test.ts ✅ Passed 245ms
tests/responses-create-stream-failures.test.ts ✅ Passed 233ms
tests/responses-create-stream-timeout.test.ts ✅ Passed 236ms
tests/responses-create-stream-wire.test.ts ✅ Passed 2.07s
tests/responses-create-stream.test.ts ✅ Passed 364ms
tests/responses-create-terminal-states.test.ts ✅ Passed 232ms
tests/responses-create-timeout.test.ts ✅ Passed 242ms
tests/responses-create.test.ts ✅ Passed 291ms
tests/responses-delete.test.ts ✅ Passed 189ms
tests/responses-input-items-errors.test.ts ✅ Passed 377ms
tests/responses-input-items-list.test.ts ✅ Passed 313ms
tests/responses-input-items-options.test.ts ✅ Passed 307ms
tests/responses-input-tokens-count-timeout.test.ts ✅ Passed 303ms
tests/responses-input-tokens-count.test.ts ✅ Passed 196ms
tests/responses-malformed-inputs.test.ts ✅ Passed 1.108s
tests/responses-not-found-errors.test.ts ✅ Passed 239ms
tests/responses-parse.test.ts ✅ Passed 274ms
tests/responses-retrieve-retries.test.ts ✅ Passed 255ms
tests/responses-retrieve.test.ts ✅ Passed 207ms
tests/responses-stored-method-errors.test.ts ✅ Passed 471ms
tests/retry-behavior.test.ts ✅ Passed 3.181s
tests/sdk-error-shape.test.ts ✅ Passed 308ms

View OkTest run #38106862931

SDK merge (c15ec6b7ba0f) · head (26da120cdb9c) · base (7352cf5c80cf) · OkTest (57341bb8c17e)

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 26da120cdb9cfaaa35a56809364c5e058acdfedf. No actionable findings.

The full schema changes only the POST /decisions description. The generated documentation agrees with that wording and preserves the existing input restrictions, question order, and refusal semantics. The executable Python AST is unchanged. The change adds no structural complexity.

Reported CI checks and external OkTest passed. Source and static analysis only; I ran no repository tests or generation.

@jbeckwith-oai
jbeckwith-oai added this pull request to the merge queue Oct 11, 2026
Merged via the queue into main with commit 16d3d3b Oct 11, 2026
28 checks passed
@jbeckwith-oai
jbeckwith-oai deleted the castiron/promotions/pr-229 branch October 11, 2026 03:08
@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