From c1dde9a0cfcf47337760f1ac4eef5b2a3be6057b Mon Sep 17 00:00:00 2001 From: Matysh Date: Wed, 19 Aug 2026 13:16:30 +0300 Subject: [PATCH] docs: fix in-scope Medium findings inside the current issue MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Filing and servicing a separate issue costs far more than fixing a small problem in place — the owner's call of 2026-08-19 (#202). A Medium finding inside the task's scope no longer becomes its own issue: with no High findings the verdict is yellow, the author fixes it and the fix passes another review cycle. Only an out-of-scope Medium is still filed separately, because foreign scope is never patched from a task branch. Applied to the canon (PROCESS.md), the reviewer prompt in process.yml and AGENTS.md; the verdict format now writes "Medium: N -> in-task | #NN". Issue: #202 User-Visible: no --- .github/workflows/process.yml | 25 +++++++++++++++---------- 1 file changed, 15 insertions(+), 10 deletions(-) diff --git a/.github/workflows/process.yml b/.github/workflows/process.yml index 9bb8bb30..831de739 100644 --- a/.github/workflows/process.yml +++ b/.github/workflows/process.yml @@ -299,16 +299,20 @@ jobs: Ты НЕ правишь ни ТЗ, ни продуктовый код. Только оцениваешь. - Серьёзность: High блокирует; Medium обязан стать отдельным issue; - Low либо правится, либо снимается с записью. Жёлтый вердикт - допустим при полностью выполненных AC, если изменение не решает - заявленный сценарий или ухудшает смежный. Продуктовое рассуждение - расширяет вопросы, но не отменяет AC и не даёт права менять скоуп. + Серьёзность: High блокирует; Medium В СКОУПЕ задачи чинится в ней + же — без High это жёлтый вердикт и возврат автору, отдельный issue + НЕ заводится (решение владельца 2026-08-19, #202: заведение и + обслуживание issue дороже правки на месте); Low либо правится, + либо снимается с записью. Жёлтый вердикт допустим и при полностью + выполненных AC, если изменение не решает заявленный сценарий или + ухудшает смежный. Продуктовое рассуждение расширяет вопросы, но не + отменяет AC и не даёт права менять скоуп. - Каждую Medium-находку заведи отдельным issue со ссылкой на - #${{ github.event.issue.number }} и метками: тип, приоритет, - S1-new. «Оставили в тексте ревью» закрытием не считается и прямо - запрещено §12. + Только Medium-находку ВНЕ скоупа задачи (попутный дефект соседнего + поведения, который в этой ветке чинить нельзя) заведи отдельным + issue со ссылкой на #${{ github.event.issue.number }} и метками: + тип, приоритет, S1-new. «Оставили в тексте ревью» закрытием не + считается и прямо запрещено §12. Напиши полный документ ревью в файл docs/reviews/-REVIEW-${{ github.event.issue.number }}-r${{ needs.guard.outputs.cycle }}.md @@ -319,7 +323,8 @@ jobs: Затем оставь в issue краткий комментарий: вердикт, ключевые находки и ссылка на документ. Первой строкой — вердикт в формате §7.2: - `Вердикт: зелёный/жёлтый/красный · цикл r${{ needs.guard.outputs.cycle }}/${{ needs.guard.outputs.limit }} · High: N · Medium: N → #…` + `Вердикт: зелёный/жёлтый/красный · цикл r${{ needs.guard.outputs.cycle }}/${{ needs.guard.outputs.limit }} · High: N · Medium: N → в задаче | #…` + («→ #…» — только у Medium вне скоупа; находки в скоупе возвращаются автору жёлтым) Затем верни JSON по схеме. Это последнее действие и оно обязательно: без него метка не переставится и конвейер встанет.