Skip to content

docs(codex): add Approval Policy & Auto-review section#719

Merged
thepagent merged 1 commit intomainfrom
docs/codex-auto-review
May 3, 2026
Merged

docs(codex): add Approval Policy & Auto-review section#719
thepagent merged 1 commit intomainfrom
docs/codex-auto-review

Conversation

@chaodu-agent
Copy link
Copy Markdown
Collaborator

Summary

Add an Approval Policy & Auto-review section to docs/codex.md, placed above Troubleshooting.

What's included

  • Comparison table of the three approval modes (Manual / Auto-review / Full Access)
  • Recommendation to use Auto-review for OpenAB deployments (unattended, long-running agents)
  • config.toml and Helm configuration examples
  • What Auto-review does and its limitations
  • Link to the OpenAI Alignment Blog post

Why

OpenAB agents run as background processes without a human at the terminal. The existing docs mention --dangerously-bypass-approvals-and-sandbox for smoke tests but don't explain the proper approval policy options for production. Auto-review is the right fit for this use case.

@chaodu-agent chaodu-agent requested a review from thepagent as a code owner May 3, 2026 18:41
@github-actions github-actions Bot added pending-screening PR awaiting automated screening closing-soon PR missing Discord Discussion URL — will auto-close in 3 days labels May 3, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 3, 2026

⚠️ This PR is missing a Discord Discussion URL in the body.

All PRs must reference a prior Discord discussion to ensure community alignment before implementation.

Please edit the PR description to include a link like:

Discord Discussion URL: https://discord.com/channels/...

This PR will be automatically closed in 3 days if the link is not added.

@thepagent thepagent merged commit 5d30769 into main May 3, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

closing-soon PR missing Discord Discussion URL — will auto-close in 3 days pending-screening PR awaiting automated screening

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants