feat(api): add Cyber access programs to Responses - #757
Conversation
Castiron-Internal-PR: openai/openai-ruby-internal#148 Castiron-Source-SHA: 44a8ceeb015778cfd5aec989cd3393e76f34245a Castiron-Public-Base-SHA: 9e18acf
Castiron custom codeMixed files: 70 → 82 12 newly customized · 0 customizations removed · 0 existing customizations changed · 20 generated baselines changed Compared
62 existing customizations unchanged
22 more in the full report. A changed generated baseline means this report cannot reliably identify which handwritten lines changed. Inspect the custom-code diffDownload the exact patch produced by this run (requires repository access): gh run download 35940671784 --repo openai/openai-ruby \
--name castiron-custom-code-35940671784-1 --dir /tmp/castiron-custom-code-35940671784-1
git apply --stat /tmp/castiron-custom-code-35940671784-1/custom-code.patch
cat /tmp/castiron-custom-code-35940671784-1/custom-code.patchOr reproduce it from an SDK checkout containing the vendored reporter: git fetch --no-tags origin 9e18acfc93ba6fbedaae37bb89956b1a4cc24b49 04b8ba0a5a7b63106fcf105cdfda15da4b5a5f97
python3 scripts/castiron/custom_code_report.py report \
--base 9e18acfc93ba6fbedaae37bb89956b1a4cc24b49 \
--head 04b8ba0a5a7b63106fcf105cdfda15da4b5a5f97 --fetch --require-head-hash --public \
--out /tmp/castiron-custom-code-04b8ba0a5a7b
cat /tmp/castiron-custom-code-04b8ba0a5a7b/custom-code.patchThis is the current full custom patch for mixed files, not an attribution of only the handwritten lines changed by this PR. |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
markstuart-oai
left a comment
There was a problem hiding this comment.
Reviewed all 28 changed files at 04b8ba0, including the schema, GA/Beta HTTP and WebSocket models, RBI/RBS declarations, streaming paths and response tests. The optional request selection and required-but-nullable response field match the schema, and the three Cyber values flow through the existing serializers without adding client-side selection logic. No substantive correctness or structural issues found.
Validation was source-only plus hosted CI. Ruby 3.3/3.4/4.0 tests, Bedrock tests, RBI/RBS checks, lint, packaging, and Castiron consistency/budget checks pass on this head. The linked OkTest job also passed; its hosted log confirms all 236 SDK tests passed for the merge candidate containing this head. I did not run repository workloads or live Cyber entitlement/API checks.
Summary
Adds typed Responses API request and response fields for Cyber access program selection.
Changes
access_programs.cyberto response creation parametersstandard,daybreak_blue, anddaybreak_redvalues