AgentSkillsCN

code-review-checklist

结构化的代码审查清单：涵盖缺陷、代码风格、性能、安全性与文档编写等方面。

SKILL.md
--- frontmatter
name: code-review-checklist
description: Структурированный чек-лист для ревью кода: баги, стиль, производительность, безопасность, документация.
version: 1.1

Чек-лист Code Review

1. Соответствие Задаче

  • • Требования: Выполнены ли все пункты "Описание Изменений/Changes Description"?
  • • Критерии Приемки: Достигнуты?
  • • Use Cases: Основной сценарий работает?

2. Качество Реализации

  • • Top-Down/Stubs:
    • •Задача Stub: Возвращает хардкодные значения? НЕТ логики? E2E проверяет хардкод?
    • •Задача Impl: Реальная логика заменяет заглушку? E2E обновлен?
  • • Нет Дублирования: использованы существующие методы/хелперы?
  • • Обработка Ошибок: Исключения перехвачены и залогированы?
  • • Code Smells: Нет магических чисел, понятные имена?

3. Документация "Прежде Всего"

  • • Документация Директорий: .AGENTS.md обновлен в каждой затронутой директории?
  • • Docstrings: Присутствуют для новых классов/методов? (Google/JSDoc)
  • • Документация Проекта: README обновлен, если изменилась архитектура?

4. Тестирование

  • • E2E: Проходит? Проверяет основной сценарий?
  • • Регрессия: Все тесты проходят?
  • • Unit: Граничные случаи покрыты?
  • • Никаких Моков: Реальные DB/LLM использованы в интеграционных тестах (или VCR)?

5. Согласованность

  • • Обратная Совместимость: Существующие потребители не сломаны?
  • • Архитектура: Следует слоям (Service -> Repo)?
  • • Стиль: Соответствует конвенциям проекта?

Протокол Критичности

  • •🔴 BLOCKING: Задача не сделана, Тесты падают, Сломана совместимость, Нарушение Stub (Логика в задаче Stub).
  • •🟡 MAJOR: Отсутствует документация, Дублирование, Плохие имена.
  • •🟢 MINOR: Стилевые придирки.