code-review-excellence
Освоение лучших практик ревью кода
Ревью кода часто становятся узким местом или источником напряжённости в команде. Этот навык предоставляет структурированные техники для предоставления конструктивной обратной связи, проведения тщательных ревью и превращения каждого ревью в возможность обмена знаниями, которая улучшает качество кода и командное взаимодействие.
Скачать ZIP навыка
Загрузить в Claude
Перейдите в Settings → Capabilities → Skills → Upload skill
Включите и начните использовать
Протестировать
Использование «code-review-excellence». Как мне предоставить обратную связь на этот код, в котором есть уязвимость безопасности?
Ожидаемый результат:
Сформулируй свою обратную связь, используя паттерн контекст-проблема-решение:
[Контекст] Я заметил, что логика обработки платежей находится в контроллере.
[Конкретная проблема] Функция calculateTotal() использует конкатенацию строк для SQL-запросов, что уязвимо к инъекциям.
[Полезное решение] Рассмотри использование параметризованных запросов вместо этого. Вот пример паттерна...
Использование «code-review-excellence». Какие ключевые моменты нужно проверить при ревью pull request?
Ожидаемый результат:
Используй 4-фазный процесс ревью:
1. Сбор контекста: описание PR, статус CI, бизнес-требования
2. Ревью высокого уровня: архитектура, дизайн, организация файлов, стратегия тестирования
3. Построчное ревью: корректность логики, безопасность, производительность, поддерживаемость
4. Резюме: чёткое решение с приоритизированной обратной связью
Аудит безопасности
Низкий рискStatic analysis detected patterns in documentation that appear concerning but are false positives. The skill is a code review guide containing examples of vulnerabilities and code patterns for educational purposes. All detected patterns (eval references, backticks, URLs, fetch calls) are documentation content, not executable code. No actual security risks found. Safe to publish.
Факторы риска
⚡ Содержит скрипты
🌐 Доступ к сети
⚙️ Внешние команды
Оценка качества
Что вы можете построить
Обучение эффективному ревью кода
Начинающие разработчики или новые члены команды, которые учатся проводить конструктивные ревью кода, улучшающие качество кода при сохранении позитивной динамики в команде.
Улучшение процессов ревью
Команды, желающие установить или улучшить свои практики ревью кода с помощью систематических подходов, чек-листов и фреймворков обратной связи.
Помощь в ревью кода
При подготовке к проведению ревью кода или в процессе его проведения используйте этот навык для структурирования обратной связи, приоритизации проблем и обеспечения всестороннего охвата.
Попробуйте эти промпты
Помоги мне провести ревью этого pull request. Я хочу: 1) Понять общий подход и архитектуру, 2) Проверить наличие проблем безопасности, 3) Выявить проблемы производительности, 4) Убедиться, что код поддерживаемый. На что мне следует обратить внимание и какие вопросы задать автору?
Проверь этот код конкретно на проблемы безопасности. Проверь: валидацию входных данных, риски SQL-инъекций, уязвимости XSS, пробелы в аутентификации, захардкоженные секреты и правильную обработку ошибок. Какие проблемы безопасности мне следует поднять?
Проверь этот Python-код на специфичные для Python проблемы, такие как изменяемые аргументы по умолчанию, голые except-блоки, корректная обработка исключений и питоновские паттерны. Какие улучшения ты бы предложил?
Я нашёл несколько проблем в этом коде. Помоги мне составить обратную связь, которая: конкретна и выполнима, образовательна, а не осуждающа, сбалансирована с похвалой за то, что сделано хорошо, и чётко приоритизирована по степени важности.
Лучшие практики
- Проводите ревью кода оперативно в течение 24 часов, чтобы поддерживать скорость работы команды и проявлять уважение к работе автора
- Используйте модифицированный метод сэндвича: сначала контекст, затем конкретная проблема, затем полезное решение
- Чётко различайте серьёзность с помощью меток, таких как [blocking], [important], [nit] и [praise]
Избегать
- Придирки к вопросам форматирования, которые должны отлавливаться линтерами
- Расплывчатость или неясность относительно того, что нужно изменить
- Обратная связь, направленная на человека, а не на код
- Пропуск позитивной обратной связи, когда код выполнен хорошо
Часто задаваемые вопросы
Что делает этот навык?
Может ли этот навык автоматически анализировать мой код?
Какие типы ревью кода это охватывает?
Как дать обратную связь, которую разработчики примут?
Сколько времени должно занимать ревью кода?
Это работает с конкретными платформами, такими как GitHub?
Сведения для разработчиков
Автор
wshobsonЛицензия
MIT
Репозиторий
https://github.com/wshobson/agents/tree/main/plugins/developer-essentials/skills/code-review-excellenceСсылка
main
Структура файлов
📄 SKILL.md