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

#1
open+2~14proposed by gardener · Aug 12, 2026 · based on v1
Proposed changes · v1 → suggestion
+2~14
Поведение соответствует требованиямChanged

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

subtasks changed
Решение остаётся простым по принципу KISSChanged

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

subtasks changed
Знания не дублируются по принципу DRYChanged

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

subtasks changed
Случайное сходство не превращено в преждевременную абстракциюRecommendedChanged

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

subtasks changed
Спекулятивная функциональность отсутствует по принципу YAGNIChanged

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

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

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

subtasks changed
Зависимости направлены к устойчивым контрактамRecommendedChanged

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

subtasks changed
Контракты и ошибки выражены явноChanged

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

subtasks changed
Критические сценарии и границы покрыты тестамиChanged

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

subtasks changed
Автоматические проверки проходят воспроизводимоChanged

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

subtasks changed
Имена и структура соответствуют соглашениям проектаRecommendedChanged

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

subtasks changed
Документация объясняет актуальные решенияRecommendedChanged

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

subtasks changed
Границы доверия и привилегии ограниченыChanged

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

subtasks changed
Принципы применены как компромиссы, а не догмыRecommendedChanged

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

subtasks changed
Логи не содержат конфиденциальной информацииAdded

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

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

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

Review