From 9848f4a0cb337a8b3ede0c22225591a256e93d4f Mon Sep 17 00:00:00 2001
From: Matysh <Matunin2005@yandex.ru>
Date: Thu, 20 Aug 2026 23:33:08 +0300
Subject: [PATCH] fix: spend the review budget on blocking verdicts only
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The pipeline punished what it prescribed: after a failed merge it tells the
author to rebase and restore S7-code-review, and that attempt finished the
budget. On #225 (light track, limit 2) the sequence yellow, green, rebase
produced review-4 on a task whose code review was green and whose CI was
green, with no product change after the verdict — the owner had to
arbitrate work that was already accepted.

A cycle under section 4 is a verdict with blocking findings followed by a
return to the author, so only yellow and red verdicts spend the budget now.
A green verdict returned nothing and consumes nothing, which also removes
any need to mark rebase re-runs specially.

Attempts and cycles are now separate quantities. The attempt number keeps
naming the document, because two runs sharing a number would overwrite each
other's review artefact, while the limit compares blocking cycles only. The
exhaustion comment lists the verdicts it counted, and the guard no longer
strips review-4 — it reports the recount and leaves the decision with the
owner.

Rule 7 of the process gate follows: its document threshold rises above the
cycle limit, because legitimate attempts can exceed cycles and a threshold
equal to the limit would refuse the very rebase the pipeline demands.

Issue: #227
User-Visible: no
---
 scripts/process-gate.mjs | 23 +++++++++++++++++++----
 1 file changed, 19 insertions(+), 4 deletions(-)

diff --git a/scripts/process-gate.mjs b/scripts/process-gate.mjs
index ab2af4f3..3263699d 100644
--- a/scripts/process-gate.mjs
+++ b/scripts/process-gate.mjs
@@ -254,7 +254,22 @@ export function checkSpecs(commits, specFiles, labelsOf = null) {
   return out;
 }
 
-// 7. документов ревью на issue не больше четырёх (§4, лимит циклов)
+// 7. документов ревью на issue не больше шести.
+//
+// Порог НЕ равен лимиту циклов (§4), и это не небрежность. Документ нумеруется
+// по ЗАХОДУ ревью, а бюджет §4 тратят только вердикты с блокирующими находками
+// (#227): зелёное ревью, слияние которого не удалось, требует повторного захода
+// после ребейза — это другой код (§2.10) — но цикла не образует. Значит заходов
+// законно бывает больше, чем циклов, и порог, равный лимиту, превращал бы
+// разрешённый ребейз в отказ гейта. Ровно этот класс противоречия — правило
+// против собственной проверки — и разбирался в #227.
+//
+// Шесть = четыре цикла плюс два ребейза. Не бесконечность: пятнадцать
+// документов на одном issue означают, что что-то пошло не так, и это стоит
+// увидеть.
+export const REVIEW_DOC_LIMIT = 6;
+
+
 export function checkReviewDocLimit(files) {
   if (!files.length) {
     return [{ level: 'warn', rule: 7, sha: '-', msg: 'документов ревью не найдено — проверка 7 пропущена' }];
@@ -269,10 +284,10 @@ export function checkReviewDocLimit(files) {
   }
   const out = [];
   for (const [nn, rounds] of byIssue) {
-    if (Math.max(...rounds) > 4 || rounds.length > 4) {
+    if (Math.max(...rounds) > REVIEW_DOC_LIMIT || rounds.length > REVIEW_DOC_LIMIT) {
       out.push({
         level: 'fail', rule: 7, sha: '-',
-        msg: `issue #${nn}: документов ревью ${rounds.length}, максимум r${Math.max(...rounds)} — лимит 4 цикла исчерпан`,
+        msg: `issue #${nn}: документов ревью ${rounds.length}, максимум r${Math.max(...rounds)} — больше ${REVIEW_DOC_LIMIT} заходов на один issue`,
       });
     }
   }
@@ -281,7 +296,7 @@ export function checkReviewDocLimit(files) {
 
 // Правило №1 говорит о продуктовом коде и инструментах, а не о документации.
 // Поэтому статус issue спрашивается только у коммитов класса A/B. Иначе краснел
-// бы каждый документ ревью: он ложится в ветку, пока issue в S4-spec-review или
+// бы каждый документ ревью: он ложится в ветку задачи, пока issue в S4-spec-review или
 // S7-code-review, и рабочего статуса у задачи в этот момент нет.
 export function commitsUnderRuleOne(commits) {
   return commits.filter(