Skip to content

feat(api): share Decisions cache breakpoint parameter type - #4092

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

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

Conversation

@jbeckwith-oai

Copy link
Copy Markdown
Contributor

Summary

Reuse a shared prompt-cache breakpoint parameter type for Decisions text and image inputs.

Changes

  • Expose a common Decisions prompt-cache breakpoint parameter type.
  • Preserve existing breakpoint parameter imports and request fields.

@github-actions

github-actions Bot commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Castiron custom code

Evaluated main: 6ea95a6d1f250840eda4cb99737a90fef31c7a14.

Trusted main policy: 6ea95a6d1f250840eda4cb99737a90fef31c7a14.

Mixed files: 36 → 38

2 newly customized · 0 customizations removed · 0 existing customizations changed · 3 generated baselines changed

Compared 6ea95a6d1f25 → b4ae04e5b1ca. Generated baselines verified.

File Result Current custom patch
api.md Generated baseline changed +119 / −8
src/openai/types/decision_input_image_param.py Generated baseline changed +11 / −1
src/openai/types/decision_input_text_param.py Generated baseline changed +11 / −1
35 existing customizations unchanged
  • src/openai/init.py
  • src/openai/_client.py
  • src/openai/resources/audio/transcriptions.py
  • src/openai/resources/audio/translations.py
  • src/openai/resources/beta/agents/environments/files.py
  • src/openai/resources/beta/agents/sessions/artifacts.py
  • src/openai/resources/beta/agents/sessions/sessions.py
  • src/openai/resources/beta/beta.py
  • src/openai/resources/beta/responses/responses.py
  • src/openai/resources/beta/threads/runs/runs.py
  • src/openai/resources/beta/threads/threads.py
  • src/openai/resources/chat/completions/completions.py
  • src/openai/resources/embeddings.py
  • src/openai/resources/files.py
  • src/openai/resources/live/forks.py
  • src/openai/resources/live/live.py
  • src/openai/resources/live/sideband.py
  • src/openai/resources/realtime/api.md
  • src/openai/resources/realtime/realtime.py
  • src/openai/resources/responses/responses.py
  • src/openai/resources/uploads/uploads.py
  • src/openai/resources/vector_stores/file_batches.py
  • src/openai/resources/vector_stores/files.py
  • src/openai/resources/videos.py
  • src/openai/resources/webhooks/init.py
  • src/openai/resources/webhooks/webhooks.py
  • src/openai/types/fine_tuning/fine_tuning_job_integration.py
  • src/openai/types/realtime/conversation_item_input_audio_transcription_delta_event.py
  • src/openai/types/realtime/realtime_error_event.py
  • src/openai/types/responses/tool.py
  • src/openai/types/responses/tool_param.py
  • src/openai/types/webhooks/init.py
  • src/openai/types/webhooks/live_call_incoming_webhook_event.py
  • src/openai/types/webhooks/unwrap_webhook_event.py
  • tests/api_resources/test_videos.py

A changed generated baseline means this report cannot reliably identify which handwritten lines changed.

Inspect the custom-code diff

Download the exact patch produced by this run (requires repository access):

gh run download 38967967675 --repo openai/openai-python \
  --name castiron-custom-code-38967967675-1 --dir /tmp/castiron-custom-code-38967967675-1
git apply --stat /tmp/castiron-custom-code-38967967675-1/custom-code.patch
cat /tmp/castiron-custom-code-38967967675-1/custom-code.patch

Or reproduce it from an SDK checkout containing the vendored reporter:

git fetch --no-tags origin 6ea95a6d1f250840eda4cb99737a90fef31c7a14 b4ae04e5b1cab57e66bab02893b2aa6a80cfeefe
python3 scripts/castiron/custom_code_report.py report \
  --base 6ea95a6d1f250840eda4cb99737a90fef31c7a14 \
  --head b4ae04e5b1cab57e66bab02893b2aa6a80cfeefe --fetch --require-head-hash --public \
  --out /tmp/castiron-custom-code-b4ae04e5b1ca
cat /tmp/castiron-custom-code-b4ae04e5b1ca/custom-code.patch

This is the current full custom patch for mixed files, not an attribution of only the handwritten lines changed by this PR.

Full report and patch

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

chatgpt-codex-connector Bot commented Oct 16, 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-16T21:16:03.390809Z b4ae04e New commits
🔒 Security Review ✅ Completed 2026-10-16T21:17:09.936185Z b4ae04e New commits
ℹ️ 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 16, 2026 •

Copy link
Copy Markdown
Contributor

OkTest Summary

✅ 236/236 SDK tests passed in 9.756s for Python SDK PR #4092.

Test results — 42 files
Test Result Time
tests/chat-completions-complex-body.test.ts ✅ Passed 254ms
tests/chat-completions-create.test.ts ✅ Passed 236ms
tests/chat-completions-stream.test.ts ✅ Passed 293ms
tests/files-content-binary.test.ts ✅ Passed 219ms
tests/files-create-multipart.test.ts ✅ Passed 223ms
tests/files-list-pagination.test.ts ✅ Passed 264ms
tests/initialize-config.test.ts ✅ Passed 252ms
tests/instance-isolation.test.ts ✅ Passed 187ms
tests/models-list.test.ts ✅ Passed 214ms
tests/responses-background-lifecycle.test.ts ✅ Passed 234ms
tests/responses-body-method-errors.test.ts ✅ Passed 292ms
tests/responses-cancel-timeout.test.ts ✅ Passed 266ms
tests/responses-cancel.test.ts ✅ Passed 255ms
tests/responses-compact-retries.test.ts ✅ Passed 311ms
tests/responses-compact.test.ts ✅ Passed 199ms
tests/responses-create-advanced-stream.test.ts ✅ Passed 196ms
tests/responses-create-advanced.test.ts ✅ Passed 226ms
tests/responses-create-disconnect.test.ts ✅ Passed 1.21s
tests/responses-create-errors.test.ts ✅ Passed 196ms
tests/responses-create-malformed-api-responses.test.ts ✅ Passed 225ms
tests/responses-create-retries.test.ts ✅ Passed 241ms
tests/responses-create-stream-failures.test.ts ✅ Passed 891ms
tests/responses-create-stream-timeout.test.ts ✅ Passed 222ms
tests/responses-create-stream-wire.test.ts ✅ Passed 2.138s
tests/responses-create-stream.test.ts ✅ Passed 244ms
tests/responses-create-terminal-states.test.ts ✅ Passed 245ms
tests/responses-create-timeout.test.ts ✅ Passed 283ms
tests/responses-create.test.ts ✅ Passed 616ms
tests/responses-delete.test.ts ✅ Passed 312ms
tests/responses-input-items-errors.test.ts ✅ Passed 407ms
tests/responses-input-items-list.test.ts ✅ Passed 332ms
tests/responses-input-items-options.test.ts ✅ Passed 286ms
tests/responses-input-tokens-count-timeout.test.ts ✅ Passed 282ms
tests/responses-input-tokens-count.test.ts ✅ Passed 240ms
tests/responses-malformed-inputs.test.ts ✅ Passed 1.207s
tests/responses-not-found-errors.test.ts ✅ Passed 240ms
tests/responses-parse.test.ts ✅ Passed 251ms
tests/responses-retrieve-retries.test.ts ✅ Passed 221ms
tests/responses-retrieve.test.ts ✅ Passed 202ms
tests/responses-stored-method-errors.test.ts ✅ Passed 459ms
tests/retry-behavior.test.ts ✅ Passed 3.158s
tests/sdk-error-shape.test.ts ✅ Passed 325ms

View OkTest run #38967924357

SDK merge (6fe9cdb09252) · head (b4ae04e5b1ca) · base (6ea95a6d1f25) · OkTest (a23aae9eb06f)

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: fd201d7a6a

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/openai/types/decision_prompt_cache_breakpoint_param.py

@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 fd201d7a6a9d9c226ea755653b473e58da87760a. No actionable findings.

The shared breakpoint TypedDict accepts the same values as both legacy types. Preserving the existing classes and field annotations also preserves their runtime identity and import paths. I do not see a compatibility defect in that choice. The sync and async request tests cover explicit values, nulls, and omission.

The legacy checkpoint inventory fallback accepts only the audited checker digest; manifest ownership checks and the existing budget remain enforced. The trusted report accounts for the compatibility overlays rather than treating them as unmodified generated code.

Source review only. Hosted lint, build, Python 3.10/3.14, HTTPX2, baseline, and custom-code budget checks passed on this head. Candidate code was not executed locally.

Castiron-Internal-PR: openai/openai-python-internal#262
Castiron-Source-SHA: 885a3077d71810f12cb6f77ec421b0c6d4535403
Castiron-Public-Base-SHA: 6ea95a6
@jbeckwith-oai
jbeckwith-oai force-pushed the castiron/promotions/pr-262 branch from fd201d7 to b4ae04e Compare October 16, 2026 21:13
@jbeckwith-oai
jbeckwith-oai added this pull request to the merge queue Oct 16, 2026
Merged via the queue into main with commit f7cefda Oct 16, 2026
31 checks passed
@jbeckwith-oai
jbeckwith-oai deleted the castiron/promotions/pr-262 branch October 16, 2026 21:28
@openai-sdks openai-sdks Bot mentioned this pull request Oct 16, 2026
opensource-fork-sync Bot pushed a commit to cla7aye15I4nd/opensource-openai__openai-python that referenced this pull request Oct 17, 2026
Automated Release PR
---


##
[3.33.0](openai/openai-python@v3.32.0...v3.33.0)
(2026-10-16)


### Features

* **api:** add safety identifiers to image requests and responses
([openai#4093](openai#4093))
([c969e32](openai@c969e32))
* **api:** share Decisions cache breakpoint parameter type
([openai#4092](openai#4092))
([f7cefda](openai@f7cefda))

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