Skip to content

chore(api): update generation metadata only - #138

Merged
apcha-oai merged 1 commit into
mainfrom
castiron/promotions/pr-43
Aug 25, 2026
Merged

chore(api): update generation metadata only#138
apcha-oai merged 1 commit into
mainfrom
castiron/promotions/pr-43

Conversation

@apcha-oai

Copy link
Copy Markdown
Contributor

Summary

Keep the CLI’s generation metadata aligned with the coordinated SDK update.

The CLI still does not expose Realtime call creation. This PR changes only generation metadata; commands, runtime behavior, and the public API reference are unchanged.

Castiron-Internal-PR: openai/openai-cli-internal#43
Castiron-Source-SHA: dcd04ea185a313e74ac12616aea998393eba3456
Castiron-Public-Base-SHA: e460582
Copilot AI lite review requested due to automatic review settings August 25, 2026 19:22
@github-actions

github-actions Bot commented Aug 25, 2026

Copy link
Copy Markdown

Castiron custom code

✅ No new custom-code files detected.

5 mixed files remain; 0 existing customizations changed.

Compared e4605829d7d6d96ac4e00637. Generated baselines verified.

5 existing customizations unchanged
  • pkg/cmd/adminorganizationcertificate.go
  • pkg/cmd/cmd.go
  • scripts/castiron/README.md
  • scripts/castiron/custom_code_report.py
  • scripts/castiron/test_custom_code_report.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 32890170126 --repo openai/openai-cli \
  --name castiron-custom-code-32890170126-1 --dir /tmp/castiron-custom-code-32890170126-1
git apply --stat /tmp/castiron-custom-code-32890170126-1/custom-code.patch
cat /tmp/castiron-custom-code-32890170126-1/custom-code.patch

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

git fetch --no-tags origin e4605829d7d63d2bb4dcc808b78b3ed76d1d0145 d96ac4e0063730c3d645048f85b7753f85f4c648
python3 scripts/castiron/custom_code_report.py report \
  --base e4605829d7d63d2bb4dcc808b78b3ed76d1d0145 \
  --head d96ac4e0063730c3d645048f85b7753f85f4c648 --fetch --require-head-hash --public \
  --out /tmp/castiron-custom-code-d96ac4e00637
cat /tmp/castiron-custom-code-d96ac4e00637/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

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.

Pull request overview

Updates Castiron generation metadata to stay aligned with the coordinated SDK/codegen update, without changing generated commands, runtime behavior, or the public API reference.

Changes:

  • Refresh generation_id to the new generation run.
  • Update config_hash, codegen_sha, and public_codegen_sha to match the coordinated codegen inputs/outputs.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@apcha-oai
apcha-oai marked this pull request as ready for review August 25, 2026 19:32
@apcha-oai
apcha-oai requested a review from a team as a code owner August 25, 2026 19:32
@apcha-oai
apcha-oai enabled auto-merge August 25, 2026 19:32

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

Verified the exact diff is limited to generation/config/codegen provenance fields in .castiron.stats.yml. OpenAPI content hashes, CLI commands, runtime behavior, authentication, and generated API surfaces are unchanged; CI/security checks pass. Approved.

@apcha-oai
apcha-oai added this pull request to the merge queue Aug 25, 2026
Merged via the queue into main with commit 4244a62 Aug 25, 2026
17 checks passed
@apcha-oai
apcha-oai deleted the castiron/promotions/pr-43 branch August 25, 2026 19:56
@openai-sdks openai-sdks Bot mentioned this pull request Aug 25, 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.

3 participants