Files

Виталий Никитенко 0e15033351 refactor: убрать multi-agent из /el-review — инлайн-ревью силами Claude

Убран вызов встроенного /review (multi-agent workflow). Теперь Claude
читает дифф сам и выдаёт только критические проблемы с готовыми diff-фиксами.

- Никаких sub-agent'ов и deep cross-file investigations
- Только баги, утечки, падения, гонки, нарушения инвариантов
- Исключены: форматирование, нейминг, стиль, рефакторинг
- Готовый diff на каждую проблему, коротко, на русском

Co-Authored-By: Claude <noreply@anthropic.com>

2026-06-23 09:19:39 +03:00

3.6 KiB

Raw Blame History

name: el-review description: Use when пользователь запрашивает легкое/быстрое code-review между двумя Git-ветками с диффом в стиле GitLab (от merge-base). Триггеры: "быстрое ревью", "легкое ревью", "/el-review".

Eltex Review

Overview

Лёгкое ревью диффа между двумя Git-ветками в стиле GitLab Merge Request (от merge-base). Claude делает ревью сам, без multi-agent workflows — читает дифф и выдаёт только критические проблемы с готовыми исправлениями.

When to Use

Пользователь просит "поревьюй ветку feature относительно master" быстро
Пользователь говорит "быстрое ревью", "лёгкое ревью", "поверхностное ревью"
Нужен дифф как на GitLab (без лишних изменений из целевой ветки)
Формат вызова: /el-review <наша-ветка-с-изменениями> <целевая-ветка>

Workflow

Определи аргументы:
- source — первая ветка (наша, с изменениями, которую ревьюим)
- target — вторая ветка (целевая, в которую мерж-реквест)
Выполни git fetch origin — обнови remote refs
Если текущая checkout-ветка == source, выполни git pull --ff-only origin source — обнови локальную ветку
Построй дифф от merge-base: git diff origin/target...origin/source > /tmp/el-review.diff (три точки!)
Прочитай файл /tmp/el-review.diff
Сделай инлайн-ревью сам — не запускай /review, не спавни multi-agent workflows, не делай deep cross-file investigations:
- Найди только критические проблемы: баги, утечки, падения, гонки, ошибки в бизнес-логике, нарушения инвариантов
- Не репорти: форматирование, именование переменных, стиль кода, предложения по рефакторингу (если это не баг)
- Для каждой проблемы сразу пиши готовый diff с исправлением
- Объясняй кратко, на русском, простым языком
- Если критических проблем нет — напиши «Критических проблем не найдено» и остановись

Примеры вызова

/el-review feature main
/el-review my-fix develop
/el-review TASK-123-payment-screen master

Ошибки и крайние случаи

Если ветка не существует на remote - сообщи пользователю и не продолжай
Если дифф пустой - сообщи "изменений между ветками нет" и не продолжай
Если аргументы не указаны - попроси уточнить: /el-review <source-ветка> <target-ветка>

Формат вывода

## Проблема N: [краткое название]

**Где:** `file.kt:123`

**Что не так:** простым языком, зачем это плохо

**Исправление:**
```diff
- старая строка
+ новая строка

3.6 KiB Raw Blame History Unescape Escape