chore(api): document Files API rate-limit responses by jbeckwith-oai · Pull Request #4049 · openai/openai-python · GitHub
Skip to content

chore(api): document Files API rate-limit responses - #4049

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

jbeckwith-oai merged 1 commit into
mainfrom
castiron/promotions/pr-223

Conversation

@jbeckwith-oai

Copy link
Copy Markdown
Contributor

Summary

Documents Files API rate-limit responses for agent environment file uploads and session creation.

Changes

  • Describe HTTP 429 responses for environment file uploads and session creation.

Castiron-Internal-PR: openai/openai-python-internal#223
Castiron-Source-SHA: 7a3653a8d8c67d8a222ad71e09b44b365054b493
Castiron-Public-Base-SHA: e72f493
@github-actions

github-actions Bot commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

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

chatgpt-codex-connector Bot commented Oct 10, 2026 •

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-10T19:29:26.156412Z 6d0c0e0 Draft marked ready
🔒 Security Review ✅ Completed 2026-10-10T19:30:35.627720Z 6d0c0e0 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 10, 2026 •

Copy link
Copy Markdown
Contributor

OkTest Summary

✅ 236/236 SDK tests passed in 9.28s for Python SDK PR #4049.

Test results — 42 files
Test Result Time
tests/chat-completions-complex-body.test.ts ✅ Passed 204ms
tests/chat-completions-create.test.ts ✅ Passed 210ms
tests/chat-completions-stream.test.ts ✅ Passed 274ms
tests/files-content-binary.test.ts ✅ Passed 281ms
tests/files-create-multipart.test.ts ✅ Passed 223ms
tests/files-list-pagination.test.ts ✅ Passed 226ms
tests/initialize-config.test.ts ✅ Passed 198ms
tests/instance-isolation.test.ts ✅ Passed 205ms
tests/models-list.test.ts ✅ Passed 219ms
tests/responses-background-lifecycle.test.ts ✅ Passed 205ms
tests/responses-body-method-errors.test.ts ✅ Passed 264ms
tests/responses-cancel-timeout.test.ts ✅ Passed 272ms
tests/responses-cancel.test.ts ✅ Passed 192ms
tests/responses-compact-retries.test.ts ✅ Passed 416ms
tests/responses-compact.test.ts ✅ Passed 189ms
tests/responses-create-advanced-stream.test.ts ✅ Passed 257ms
tests/responses-create-advanced.test.ts ✅ Passed 214ms
tests/responses-create-disconnect.test.ts ✅ Passed 1.205s
tests/responses-create-errors.test.ts ✅ Passed 207ms
tests/responses-create-malformed-api-responses.test.ts ✅ Passed 196ms
tests/responses-create-retries.test.ts ✅ Passed 198ms
tests/responses-create-stream-failures.test.ts ✅ Passed 820ms
tests/responses-create-stream-timeout.test.ts ✅ Passed 235ms
tests/responses-create-stream-wire.test.ts ✅ Passed 2.013s
tests/responses-create-stream.test.ts ✅ Passed 142ms
tests/responses-create-terminal-states.test.ts ✅ Passed 243ms
tests/responses-create-timeout.test.ts ✅ Passed 270ms
tests/responses-create.test.ts ✅ Passed 204ms
tests/responses-delete.test.ts ✅ Passed 211ms
tests/responses-input-items-errors.test.ts ✅ Passed 521ms
tests/responses-input-items-list.test.ts ✅ Passed 333ms
tests/responses-input-items-options.test.ts ✅ Passed 291ms
tests/responses-input-tokens-count-timeout.test.ts ✅ Passed 256ms
tests/responses-input-tokens-count.test.ts ✅ Passed 193ms
tests/responses-malformed-inputs.test.ts ✅ Passed 1.025s
tests/responses-not-found-errors.test.ts ✅ Passed 281ms
tests/responses-parse.test.ts ✅ Passed 254ms
tests/responses-retrieve-retries.test.ts ✅ Passed 453ms
tests/responses-retrieve.test.ts ✅ Passed 238ms
tests/responses-stored-method-errors.test.ts ✅ Passed 465ms
tests/retry-behavior.test.ts ✅ Passed 3.038s
tests/sdk-error-shape.test.ts ✅ Passed 282ms

View OkTest run #38079896657

SDK merge (e1d73aed0295) · head (6d0c0e0a7442) · base (e72f4931f16a) · OkTest (57341bb8c17e)

@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.

Approved at 6d0c0e0a74422d7ce211d306002a97aae28de0f6. No actionable findings.

The API reference adds only the two documented HTTP 429 responses for environment file uploads and session creation. Both use the existing error schema. I verified that the rest of the parsed reference is unchanged, the transformed-spec checksum matches, and the public generation snapshot resolves. This diff changes documentation and generation metadata only.

Hosted CI is green for this head. I reviewed source and hosted checks; I did not run repository tests or generation.

@jbeckwith-oai
jbeckwith-oai added this pull request to the merge queue Oct 10, 2026
Merged via the queue into main with commit f6a3459 Oct 10, 2026
28 checks passed
@jbeckwith-oai
jbeckwith-oai deleted the castiron/promotions/pr-223 branch October 10, 2026 19:40
@openai-sdks openai-sdks Bot mentioned this pull request Oct 10, 2026
opensource-fork-sync Bot pushed a commit to cla7aye15I4nd/opensource-openai__openai-python that referenced this pull request Oct 11, 2026
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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants