Repository navigation
docs: carry generated-file ownership guidance to next - #3064
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. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 98caa435dd
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cf8fdc62ea
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d2eb40de49
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
🛡️ Codex Security Review · Automatically triggeredSecurity review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
Addresses the late review finding on #3064: the next branch still retains its strict ownership checker until canonical regeneration removes the legacy tooling. Qualify the future guidance in AGENTS, CONTRIBUTING, and the security model, and correct dependency-policy citations. No checks or runtime behavior change. Validation: basic code review, citation inspection, and git diff --check passed.
Summary
Bring the reviewed contributor and security guidance from #3061 onto next before its first regeneration with the simplified compiler. Generated files are replaced; SDK-specific handwritten behavior belongs in separate helpers. The retired customization budget and reporter are no longer part of the contributor workflow.
Only AGENTS.md, .github/CONTRIBUTING.md, and docs/architecture/security-model.md change. The guidance is based on merged main e7afc14, with explicit transition wording for next: legacy workflow files and their publisher trust boundaries remain in scope until canonical regeneration removes them. Generated tooling removal will come through canonical generation separately.
Validation
Basic review, comparison with reviewed main and inspection of the retained legacy workflow boundaries, changed-file scope check, and git diff --check passed. No runtime, workflow, generation metadata, inventory, dependency, release, or branch-protection changes.