Skip to content

[codex] Coordinate OpenAI docs sample with API key setup#21263

Merged
mifan-oai merged 1 commit into
mainfrom
codex/openai-docs-api-key-body
May 6, 2026
Merged

[codex] Coordinate OpenAI docs sample with API key setup#21263
mifan-oai merged 1 commit into
mainfrom
codex/openai-docs-api-key-body

Conversation

@mifan-oai
Copy link
Copy Markdown
Contributor

Summary

  • Add the same API key setup coordination guidance to the embedded OpenAI Docs sample skill in codex-rs/skills.
  • Keep the skill description/frontmatter unchanged; the coordination lives only in the body.
  • Preserve direct OpenAI Docs routing for docs-only questions, citations, model/API guidance, conceptual explanations, and non-building examples.

Why

The Codex repo carries its own OpenAI Docs skill variant under codex-rs/skills/src/assets/samples. This keeps that embedded sample aligned with the other OpenAI Docs variants patched in the related PRs.

Validation

  • cargo test -p codex-skills
  • git diff --check

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@mifan-oai mifan-oai force-pushed the codex/openai-docs-api-key-body branch 3 times, most recently from 6b05b11 to 094c306 Compare May 5, 2026 23:06
@mifan-oai mifan-oai force-pushed the codex/openai-docs-api-key-body branch from 094c306 to 826fecf Compare May 5, 2026 23:08
@mifan-oai
Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@mifan-oai mifan-oai marked this pull request as ready for review May 5, 2026 23:09
github-actions Bot added a commit that referenced this pull request May 5, 2026
@mifan-oai mifan-oai merged commit f2f5d6f into main May 6, 2026
48 of 51 checks passed
@mifan-oai mifan-oai deleted the codex/openai-docs-api-key-body branch May 6, 2026 17:46
@github-actions github-actions Bot locked and limited conversation to collaborators May 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants