Skip to content

fix(api): clarify audio upload metadata requirements - #857

Merged
jbeckwith-oai merged 1 commit into
mainfrom
castiron/openapi-577fa922
Aug 11, 2026
Merged

fix(api): clarify audio upload metadata requirements#857
jbeckwith-oai merged 1 commit into
mainfrom
castiron/openapi-577fa922

Conversation

@jbeckwith-oai

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

Copy link
Copy Markdown
Contributor

Summary

Regenerates the Java SDK from the latest OpenAPI input for the audio upload documentation change.

  • Clarifies that transcription and translation uploads must include enough metadata for the file format to be identified.
  • Recommends an extension-bearing filename and an appropriate content type.
  • Updates generated Kotlin parameter documentation, the transformed OpenAPI document, and Castiron provenance.

This is documentation-only; runtime behavior and exported type shapes are unchanged.

Source

Validation

  • Rebased directly onto the current public main
  • Every changed file matches the Castiron-generated candidate
  • git diff --check
  • Thermo-nuclear code-quality review

@HAYDEN-OAI HAYDEN-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 the generated audio upload metadata documentation against the approved source change. No substantive issues found.

@jbeckwith-oai
jbeckwith-oai force-pushed the castiron/openapi-577fa922 branch from 9d8b536 to 77562a0 Compare August 11, 2026 16:43
@jbeckwith-oai
jbeckwith-oai changed the base branch from castiron/promotions/pr-18-c40ca3098202 to main August 11, 2026 16:44
@jbeckwith-oai
jbeckwith-oai marked this pull request as ready for review August 11, 2026 16:57
@jbeckwith-oai
jbeckwith-oai requested a review from a team as a code owner August 11, 2026 16:57
@openai-sdks

openai-sdks Bot commented Aug 11, 2026

Copy link
Copy Markdown

OkTest Summary

237/237 SDK tests passed in 16.499s for Java SDK PR #857.

Test results — 42 files
Test Result Time
tests/chat-completions-complex-body.test.ts ✅ Passed 618ms
tests/chat-completions-create.test.ts ✅ Passed 506ms
tests/chat-completions-stream.test.ts ✅ Passed 407ms
tests/files-content-binary.test.ts ✅ Passed 310ms
tests/files-create-multipart.test.ts ✅ Passed 262ms
tests/files-list-pagination.test.ts ✅ Passed 262ms
tests/initialize-config.test.ts ✅ Passed 194ms
tests/instance-isolation.test.ts ✅ Passed 224ms
tests/models-list.test.ts ✅ Passed 178ms
tests/responses-background-lifecycle.test.ts ✅ Passed 353ms
tests/responses-body-method-errors.test.ts ✅ Passed 510ms
tests/responses-cancel-timeout.test.ts ✅ Passed 232ms
tests/responses-cancel.test.ts ✅ Passed 392ms
tests/responses-compact-retries.test.ts ✅ Passed 499ms
tests/responses-compact.test.ts ✅ Passed 290ms
tests/responses-create-advanced-stream.test.ts ✅ Passed 686ms
tests/responses-create-advanced.test.ts ✅ Passed 1.418s
tests/responses-create-disconnect.test.ts ✅ Passed 1.128s
tests/responses-create-errors.test.ts ✅ Passed 343ms
tests/responses-create-malformed-api-responses.test.ts ✅ Passed 340ms
tests/responses-create-retries.test.ts ✅ Passed 307ms
tests/responses-create-stream-failures.test.ts ✅ Passed 216ms
tests/responses-create-stream-timeout.test.ts ✅ Passed 332ms
tests/responses-create-stream-wire.test.ts ✅ Passed 6.249s
tests/responses-create-stream.test.ts ✅ Passed 118ms
tests/responses-create-terminal-states.test.ts ✅ Passed 414ms
tests/responses-create-timeout.test.ts ✅ Passed 216ms
tests/responses-create.test.ts ✅ Passed 282ms
tests/responses-delete.test.ts ✅ Passed 256ms
tests/responses-input-items-errors.test.ts ✅ Passed 302ms
tests/responses-input-items-list.test.ts ✅ Passed 304ms
tests/responses-input-items-options.test.ts ✅ Passed 480ms
tests/responses-input-tokens-count-timeout.test.ts ✅ Passed 232ms
tests/responses-input-tokens-count.test.ts ✅ Passed 401ms
tests/responses-malformed-inputs.test.ts ✅ Passed 4.955s
tests/responses-not-found-errors.test.ts ✅ Passed 320ms
tests/responses-parse.test.ts ✅ Passed 595ms
tests/responses-retrieve-retries.test.ts ✅ Passed 624ms
tests/responses-retrieve.test.ts ✅ Passed 380ms
tests/responses-stored-method-errors.test.ts ✅ Passed 983ms
tests/retry-behavior.test.ts ✅ Passed 3.459s
tests/sdk-error-shape.test.ts ✅ Passed 411ms

View OkTest run #31515666158

SDK merge (b411fa069e04) · head (77562a00423b) · base (5b9ba3a75cbb) · OkTest (91635c6a2723)

@jbeckwith-oai jbeckwith-oai reopened this Aug 11, 2026
@jbeckwith-oai
jbeckwith-oai enabled auto-merge (squash) August 11, 2026 17:04
@jbeckwith-oai
jbeckwith-oai merged commit 593f04c into main Aug 11, 2026
10 checks passed
@openai-sdks openai-sdks Bot mentioned this pull request Aug 11, 2026
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