Repository navigation
docs: clarify generated-file ownership - #1331
Conversation
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. |
jbeckwith-oai
left a comment
There was a problem hiding this comment.
Approved after coordinator and independent static review of 136c4b9. Ownership guidance correctly routes schema, generator and handwritten fixes; BedrockAuth and example-directory ownership match the inventory. Links/anchors resolve, and the consolidated guidance preserves generated-output and inventory protections. No actionable findings. Hosted checks remain pending with no failures observed. No candidate code or local tests were executed.
markstuart-oai
left a comment
There was a problem hiding this comment.
Reviewed 136c4b918e0571d7575583c25e9d70a8c6023893; no actionable findings. The consolidated ownership guidance preserves generated-file protections and correctly routes schema, generator and handwritten fixes. The Bedrock authentication helper and examples match the pinned inventory.
Checked both changed documents, links/anchors and pinned source ownership. Source-only; no local tests, generation or rendering checks ran. Hosted checks have no reported failures, with OkTest still pending.
Consolidate generated-file ownership guidance in the contributor guide and link to it from
AGENTS.md. Explain how to trace a fix to the API schema, SDK generator, or a handwritten file, with a repository-specific example.Verified the documentation links and section anchors and ran
git diff --check.