From 9528bf20365fcac4ce2b6f6dc8b22e3cdf7c861f Mon Sep 17 00:00:00 2001 From: ScriptedAlchemy Date: Thu, 3 Sep 2026 21:47:54 +0000 Subject: [PATCH] docs(agents): make the pull-request review rule neutral about who reviews Drop the "never solicit external reviews" bullet. The section now says only what the merge gate is: CI green, every review thread answered whoever opened it, squash-merge, follow-up PR for threads on merged PRs. --- AGENTS.md | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/AGENTS.md b/AGENTS.md index 28ddb24b5..830568761 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -72,11 +72,9 @@ the PR description under a "Self-review" section (reviewer model, findings, disposition), run the reviewer once more after fixes, then merge on green CI. -- Never solicit external reviews: no `@codex review` comments, no waiting for - the connector, no unreviewed-SHA bookkeeping. Review comments that arrive - on their own (Codex or human) are still addressed — fix in the same PR or - reply with a precise reason — and re-checked after each push until none - remain. +- CI green first. Address every review thread, whoever opened it — fix it in + the same PR or reply with a precise reason — and re-check for new threads + after each push until none remain. Only then merge. - PRs are squash-merged. Review threads left on an already-merged PR must still be answered, in a follow-up PR.