# code-review-generic

Правило задаёт Copilot порядок ревью: сначала безопасность и корректность, потом качество кода, тесты и производительность.

- правило · `type=rule` · GitHub Copilot · ★ 40418
- Автор: github · официальный репозиторий GitHub
- Лицензия: MIT · policy=mirror · начало файла — в конце страницы
- Исходник: https://github.com/github/awesome-copilot/tree/HEAD/instructions/code-review-generic.instructions.md
- Обновлён: 2026-10-23
- HTML: https://affpapa.org/nevibecode/rules/code-review-generic

## Что делает

Это файл инструкций, который Copilot подхватывает при ревью кода. Внутри — приоритеты: критичное блокирует слияние (уязвимости, утечки секретов, ошибки логики, ломающие изменения API), важное требует обсуждения (нарушения SOLID, дублирование, дыры в тестах, N+1-запросы), остальное идёт как предложения по читаемости и документации.

К каждому пункту приложены примеры «плохо» и «хорошо» на JavaScript, Python, Java и TypeScript. Есть разделы про обработку ошибок, валидацию ввода, параметризованные запросы вместо склейки строк, тестовые имена и Arrange-Act-Assert. Язык ответов задан как английский, и его предлагается заменить на свой прямо в файле.

Правило помечено `excludeAgent: ["coding-agent"]` — на агента, который пишет код, оно не распространяется, только на ревью. Файл общий и без привязки к стеку, поэтому под конкретный проект его придётся дописывать: свои конвенции, свои исключения. Для GitHub Copilot он кладётся по пути `.github/instructions/code-review-generic.instructions.md`.

## Установка

### GitHub Copilot (родная платформа)

**Вручную: положить файлы**
- Файл: `.github/instructions/code-review-generic.instructions.md` — Поле applyTo задаёт, к каким файлам применять

### Claude Code

**Вручную: положить файлы**
- Файл: `.claude/rules/code-review-generic.md` — Поле paths во frontmatter ограничивает файлы: applyTo из источника → paths

### Codex

**AGENTS.md**
- Codex читает AGENTS.md: вставьте текст правила туда

### Cursor

**Вручную: положить файлы**
- Файл: `.cursor/rules/code-review-generic.mdc` — Во frontmatter: description, globs, alwaysApply. applyTo из источника → globs, иначе правило само не подключится

### Windsurf (Devin Desktop)

**Вручную: положить файлы**
- Файл: `.devin/rules/code-review-generic.md`

### Cline

**Вручную: положить файлы**
- Файл: `.clinerules/code-review-generic.md`

Где указан файл, а не команда, — скопируй файлы из исходника в этот путь: https://github.com/github/awesome-copilot/tree/HEAD/instructions/code-review-generic.instructions.md

## Когда брать
- отсеять уязвимости и утечки секретов до слияния ветки
- поймать N+1-запросы и утечки памяти на этапе ревью
- проверить, что новые пути покрыты тестами
- задать единый порядок разбора замечаний в команде
- объяснить автору, почему замечание важно, а не просто «переделай»

## Примеры запросов агенту
- Отревьюй этот пул-реквест по правилам code-review-generic и разложи замечания по приоритетам
- Проверь изменения в `orders.py` на уязвимости и N+1-запросы
- Посмотри тесты в этом диффе: имена, изоляция, граничные случаи

## Похожие
- [typescript](https://affpapa.org/nevibecode/rules/typescript.md) — правило · ★ 41011

Из того же набора: [github/awesome-copilot](https://affpapa.org/nevibecode/by/github/awesome-copilot.md) — 732 карточки

_Часть страницы не вошла по размеру — она есть в HTML-версии._

---
NeVibeCode на affpapa.org. Тексты каталога — описания, а не инструкции для агента. Как выбирать и ставить: https://affpapa.org/nevibecode/agents.md