Skip to content

Document native provider-hosted tools - #5578

Merged
markstuart-oai merged 1 commit into
feat/codex-harnessfrom
markstuart/native-hosted-tools-docs
Nov 7, 2026
Merged

markstuart-oai merged 1 commit into
feat/codex-harnessfrom
markstuart/native-hosted-tools-docs

Conversation

@markstuart-oai

@markstuart-oai markstuart-oai commented Nov 7, 2026 •

Copy link
Copy Markdown
Contributor

Summary

The native hosted tools added in #5576 need a user guide. This docs-only PR explains configuration, provider results and resource ownership, history and restore, and progress limits. It also links to runnable examples.

This PR targets feat/codex-harness and documents the native APIs and examples merged in #5576.

Test plan

  • Executed the example: all three tool constructors and Agent initialization passed. No provider calls were made.
  • Independent review checked implementation claims and Japanese, Korean, and Chinese translation risks. Corrected the required instructions argument and clarified the Responses Lite restriction.
  • make build-docs and git diff --check passed. Only docs/sdk/tools.md changes.

Issue number

None.

Checks

  • Verified the documented behavior against the implementation.
  • Passed the applicable content-tier checks in AGENTS.md.
  • Kept the guide separate from the feature PR.

Base automatically changed from markstuart/native-hosted-tools to feat/codex-harness November 7, 2026 06:45
@markstuart-oai
markstuart-oai force-pushed the markstuart/native-hosted-tools-docs branch from 90948b8 to d83022d Compare November 7, 2026 06:46
@markstuart-oai
markstuart-oai marked this pull request as ready for review November 7, 2026 06:47
@markstuart-oai
markstuart-oai requested review from a team, rm-openai and seratch as code owners November 7, 2026 06:47
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 7, 2026 •

Copy link
Copy Markdown
Codex Review Summary: ✅ Completed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-07T06:48:08.995627Z d83022d Draft marked ready
🔒 Security Review ✅ Completed 2026-11-07T06:48:45.753261Z d83022d Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

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

Reviewed all 36 added documentation lines against the exact-head merged hosted-tools implementation and examples. Constructor signatures, captured tool selection, provider restrictions, result fields, container ownership, image editing, observation limits, and restore behavior match the implementation. No actionable findings.

Coordinator and independent static review; no candidate code/tests or documentation build run locally. Hosted docs build and lint passed; remaining hosted checks are pending.

@markstuart-oai
markstuart-oai merged commit 0d67248 into feat/codex-harness Nov 7, 2026
25 checks passed
@markstuart-oai
markstuart-oai deleted the markstuart/native-hosted-tools-docs branch November 7, 2026 06:51
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