Skip to content

chore(api): Update generated file header text - #846

Open
apcha-oai wants to merge 1 commit into
mainfrom
castiron/promotions/pr-16-f16338a6e646
Open

chore(api): Update generated file header text#846
apcha-oai wants to merge 1 commit into
mainfrom
castiron/promotions/pr-16-f16338a6e646

Conversation

@apcha-oai

Copy link
Copy Markdown
Contributor

Summary

chore(api): Update generated file header text

Castiron-Internal-PR: openai/openai-java-internal#16
Castiron-Source-SHA: f16338a6e646f749d69c110cec1e5851c71d1510
Castiron-Public-Base-SHA: d01bea2
Copilot AI review requested due to automatic review settings August 7, 2026 02:56

Copilot AI 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.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@apcha-oai
apcha-oai marked this pull request as ready for review August 7, 2026 03:16
@apcha-oai
apcha-oai requested a review from a team as a code owner August 7, 2026 03:16
@openai-sdks

openai-sdks Bot commented Aug 7, 2026

Copy link
Copy Markdown

OkTest Summary

237/237 SDK tests passed in 16.743s for Java SDK PR #846.

Test results — 42 files
Test Result Time
tests/chat-completions-complex-body.test.ts ✅ Passed 776ms
tests/chat-completions-create.test.ts ✅ Passed 683ms
tests/chat-completions-stream.test.ts ✅ Passed 383ms
tests/files-content-binary.test.ts ✅ Passed 211ms
tests/files-create-multipart.test.ts ✅ Passed 233ms
tests/files-list-pagination.test.ts ✅ Passed 253ms
tests/initialize-config.test.ts ✅ Passed 163ms
tests/instance-isolation.test.ts ✅ Passed 164ms
tests/models-list.test.ts ✅ Passed 194ms
tests/responses-background-lifecycle.test.ts ✅ Passed 304ms
tests/responses-body-method-errors.test.ts ✅ Passed 537ms
tests/responses-cancel-timeout.test.ts ✅ Passed 258ms
tests/responses-cancel.test.ts ✅ Passed 331ms
tests/responses-compact-retries.test.ts ✅ Passed 511ms
tests/responses-compact.test.ts ✅ Passed 274ms
tests/responses-create-advanced-stream.test.ts ✅ Passed 582ms
tests/responses-create-advanced.test.ts ✅ Passed 1.548s
tests/responses-create-disconnect.test.ts ✅ Passed 1.143s
tests/responses-create-errors.test.ts ✅ Passed 344ms
tests/responses-create-malformed-api-responses.test.ts ✅ Passed 288ms
tests/responses-create-retries.test.ts ✅ Passed 271ms
tests/responses-create-stream-failures.test.ts ✅ Passed 265ms
tests/responses-create-stream-timeout.test.ts ✅ Passed 281ms
tests/responses-create-stream-wire.test.ts ✅ Passed 6.276s
tests/responses-create-stream.test.ts ✅ Passed 143ms
tests/responses-create-terminal-states.test.ts ✅ Passed 472ms
tests/responses-create-timeout.test.ts ✅ Passed 296ms
tests/responses-create.test.ts ✅ Passed 232ms
tests/responses-delete.test.ts ✅ Passed 240ms
tests/responses-input-items-errors.test.ts ✅ Passed 313ms
tests/responses-input-items-list.test.ts ✅ Passed 400ms
tests/responses-input-items-options.test.ts ✅ Passed 592ms
tests/responses-input-tokens-count-timeout.test.ts ✅ Passed 276ms
tests/responses-input-tokens-count.test.ts ✅ Passed 371ms
tests/responses-malformed-inputs.test.ts ✅ Passed 5.078s
tests/responses-not-found-errors.test.ts ✅ Passed 448ms
tests/responses-parse.test.ts ✅ Passed 614ms
tests/responses-retrieve-retries.test.ts ✅ Passed 830ms
tests/responses-retrieve.test.ts ✅ Passed 238ms
tests/responses-stored-method-errors.test.ts ✅ Passed 969ms
tests/retry-behavior.test.ts ✅ Passed 3.409s
tests/sdk-error-shape.test.ts ✅ Passed 378ms

View OkTest run #31143855066

SDK merge (9029048e2cb2) · head (9929654783ec) · base (d01bea2058ac) · OkTest (91635c6a2723)

@apcha-oai
apcha-oai enabled auto-merge (squash) August 7, 2026 03:55

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

The Castiron header replacement is clean across the generated Java/Kotlin files, but this regeneration contains an unrelated API-contract change in api_reference/openapi.transformed.yml: it removes minLength: 1 from both NamespaceToolParam.description and BetaNamespaceToolParam.description. The resulting fixture churn from "x" to "description" in four namespace-tool tests confirms that the generation semantics changed, rather than this being only header text. Please regenerate against the same input/transform behavior or restore those constraints so this PR is limited to the intended attribution update (plus generation metadata).

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.

3 participants