Перейти к содержимому

🧙 Садовник: уточнил шаги, добавил проверки и обоснования. Примите, если полезно.

#1
открыто+2~14предложил gardener · 12 авг. 2026 г. · на основе v1
Предлагаемые изменения · v1 → предложение
+2~14
Поведение соответствует требованиямИзменён

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

subtasks изменён
Решение остаётся простым по принципу KISSИзменён

Проверить на ревью, что основной путь выполнения легко прослеживается, уровни абстракции оправданы, а более простая реализация не обеспечивает те же требования.

subtasks изменён
Знания не дублируются по принципу DRYИзменён

Найти бизнес-правила, формулы, схемы и конфигурацию, представленные в нескольких местах, и убедиться, что у каждого знания есть единый источник истины.

subtasks изменён
Случайное сходство не превращено в преждевременную абстракциюРекомендуетсяИзменён

Проверить, что объединённый код действительно меняется по одной причине; допустимое структурное повторение оставлено раздельным, если предметные правила независимы.

subtasks изменён
Спекулятивная функциональность отсутствует по принципу YAGNIИзменён

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

subtasks изменён
Ответственности разделены и связность модулей высокаИзменён

Проверить, что модуль объединяет поведение вокруг одной причины изменения, а бизнес-логика не смешана с вводом-выводом, хранением, интерфейсом или инфраструктурой.

subtasks изменён
Зависимости направлены к устойчивым контрактамРекомендуетсяИзменён

Проверить, что существенная бизнес-логика не зависит напрямую от нестабильных деталей, внешние интеграции изолированы, а интерфейсы вводятся в местах реальной заменяемости или тестового шва.

subtasks изменён
Контракты и ошибки выражены явноИзменён

Проверить, что входные условия, возвращаемые значения, состояния ошибок и побочные эффекты понятны из типов, интерфейсов, тестов или документации, а ошибки не подавляются молча.

subtasks изменён
Критические сценарии и границы покрыты тестамиИзменён

Сопоставить тесты с критериями приёмки и проверить наличие успешных, граничных и ошибочных сценариев; тесты должны проверять наблюдаемое поведение, а не внутреннее устройство.

subtasks изменён
Автоматические проверки проходят воспроизводимоИзменён

Убедиться, что обязательный конвейер CI на актуальном коммите успешно выполняет тесты, статический анализ, форматирование, сборку и применимые проверки безопасности в чистом окружении.

subtasks изменён
Имена и структура соответствуют соглашениям проектаРекомендуетсяИзменён

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

subtasks изменён
Документация объясняет актуальные решенияРекомендуетсяИзменён

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

subtasks изменён
Границы доверия и привилегии ограниченыИзменён

Проверить валидацию недоверенных данных, авторизацию операций, хранение секретов, безопасные значения по умолчанию и минимально необходимые права компонентов.

subtasks изменён
Принципы применены как компромиссы, а не догмыРекомендуетсяИзменён

Проверить, что отступления от KISS, DRY, YAGNI или SOLID объясняются измеримыми требованиями, а применение одного принципа не создаёт больший риск для понятности, корректности или изменений.

subtasks изменён
Логи не содержат конфиденциальной информацииДобавлен

Проверить, что в логах нет чувствительных данных, таких как пароли, токены, персональные данные пользователей.

Обработка исключений не нарушает бизнес-логикуДобавлен

Проверить, что при обработке исключений программа не теряет критически важные данные и не переходит в некорректное состояние.

Ревью