From e0f7dafcea7dd623b79ddb778f84fb2c53bb8f85 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Fri, 15 May 2026 13:41:34 +0100 Subject: [PATCH] docs: require codex review before landing --- AGENTS.md | 1 + 1 file changed, 1 insertion(+) diff --git a/AGENTS.md b/AGENTS.md index dee735c6a34..20bf6aa6791 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -67,6 +67,7 @@ Skills own workflows; root owns hard policy and routing. - Full suites, broad changed gates, Docker/package/E2E/live/cross-OS proof, or anything that bogs down the Mac: Crabbox/Testbox. - One/few files local. If a local command fans out, stop and move broad proof to Crabbox/Testbox. - Before handoff/push: prove touched surface. Before landing to `main`: issue proof plus appropriate full/broad proof unless scope is clearly narrow. +- Pre-land/pre-commit code changes: use `$codex-review` until no accepted/actionable findings remain, unless equivalent manual review already done, trivial/docs-only, or user opts out. - If proof is blocked, say exactly what is missing and why. - Do not land related failing format/lint/type/build/tests. If unrelated on latest `origin/main`, say so with scoped proof. - Docs/changelog-only and CI/workflow metadata-only: `git diff --check` plus relevant docs/workflow sanity; escalate only if scripts/config/generated/package/runtime behavior changed.