Adversarial Review

Обзор

Стандартный сбой ревью — подтверждение: читатель, который хочет, чтобы артефакт был верным, находит причины считать его верным. Этот навык переворачивает позицию — задача ревьюера построить сильнейший аргумент, что артефакт неверен, неполон или небезопасен, и лишь затем взвесить этот аргумент против защиты.

Это общая состязательная линза. Два навыка-сиблинга затачивают отдельные оси: edge-case-hunter (граничные и аварийные входы, которые артефакт не обрабатывает) и structure-review (организация, полнота, внутренние противоречия).

Ключевое правило

«Находок нет» — это тревога, а не зелёный свет. Ноль находок у состязательного прохода — свидетельство поверхностности прохода, а не безупречности артефакта. Прежде чем записать «существенных находок нет», ревьюер обязан выполнить хотя бы одно из:

• назвать самое несущее допущение артефакта и попытаться его опровергнуть;
• проследить один конкретный сценарий сбоя от начала до конца и показать, что он невозможен;
• определить, что артефакт не заявляет обрабатывать, и подтвердить, что этот срез скоупа сознателен и записан.

Когда загружается

• /dr-plan Transition Checkpoint — атака на план до старта имплементации.
• /dr-qa — атака на готовое изменение до его одобрения.
• Любое критическое решение, где чтение с уклоном в подтверждение — самый дешёвый способ пропустить дефект.