Fix Codex 0.99.0 config schema and add workflow invocation docs - #47
Conversation
Co-authored-by: clduab11 <185000089+clduab11@users.noreply.github.com>
Reviewed the Codex 0.99.0 configuration updates and workflow documentation. Found one documentation clarity issue.
Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues. |
|
|
||
| **Automatic triggers:** | ||
| - Runs automatically on PR events: opened, synchronize, reopened, ready_for_review | ||
| - Only runs on non-draft PRs from non-fork branches (fork PRs are blocked to protect secrets) |
There was a problem hiding this comment.
The fork PR protection applies to both automatic and manual triggers, but the documentation only mentions it under "Automatic triggers". When manually triggering via workflow_dispatch, the workflow will still fail if the target PR is from a fork (see lines 36-54 in the workflow file where the fork check runs for both event types). This could confuse operators who attempt manual reviews on fork PRs expecting them to work.
Fix it with Roo Code or mention @roomote and request a fix.
Updated Codex configuration to match 0.99.0 schema requirements and added operator runbook section for workflow invocation.
Changes
.github/codex/configs/mcp-full.toml: Renamedsandbox_mode→sandbox_policyand removed deprecatedweb_searchtop-level key to align with Codex 0.99.0 schemadocs/runbooks/github-codex-action-mcp.md: Added "How to run" section covering:Before:
After:
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.
Continue Tasks:▶️ 5 queued — View all