Repository navigation
docs: define agent issue handling policy - #751
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. |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
hibiki-Logbie
left a comment
There was a problem hiding this comment.
Approving.
Docs/contributing/issue-policy.mdis coherent. The I1/I2/I3 ranking defaults upward when facts are missing (I2) or a sensitive trigger is plausible (I3), and security goes to private handling first. The I1 completion gates require all required checks on the final head (a missing run blocks merge), independent review of the final diff, and no bypass or self-dismissal. Gate 4 correctly notes that a comment-only bot review doesn't satisfy the ruleset's required approving review.- The GOVERNANCE.md §3.9, decision-rights row, and §4 step 6 changes, plus the AI_POLICY.md, CLAUDE.md, CONTRIBUTING.md, and index updates, are consistent with the policy. The
testing.md#5-change-risk-classesanchor resolves. - The
ci.yml/wfl-config-lint.ymlchange is byte-identical to #748 (same blobs 4e8ad701 / be60c3cd), so it merges cleanly after #748. - All 12 required checks are green on dffa69d. The only thread (Devin, dev CI coverage) is resolved and was fixed by dffa69d.
Note: #749 edits the same GOVERNANCE.md and AI_POLICY.md lines with a different merge-authority model. I've asked #749 to reconcile against this PR.
Summary
dev, where the active ruleset requires their status checks. The policy now explicitly blocks autonomous merge if a required check has no run for that PR.Test evidence
devstarts the workflows that produce its required checks, and missing checks block merge.dffa69d3, PR docs: define agent issue handling policy #751 had zero pull-request workflow runs on head1a657e7f.dffa69d3, both CI and WFL Config Lint started for thisdevPR. CI created all 11 jobs named in the branch ruleset; Config Lint supplies the twelfth. Final results are pending.git diff --cached --checkpassed.testing.md§12, plus the ordinary governance review for this policy change.Direct push to
devwas rejected by the active repository rule requiring a pull request and 12 status checks. This PR targetsdev.