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>
This commit is contained in:
@@ -1,17 +1,18 @@
|
|||||||
---
|
---
|
||||||
name: el-review
|
name: el-review
|
||||||
description: Use when пользователь запрашивает легкое code-review между двумя Git-ветками с диффом в стиле GitLab (от merge-base)
|
description: Use when пользователь запрашивает легкое/быстрое code-review между двумя Git-ветками с диффом в стиле GitLab (от merge-base). Триггеры: "быстрое ревью", "легкое ревью", "/el-review".
|
||||||
---
|
---
|
||||||
|
|
||||||
# Eltex Review
|
# Eltex Review
|
||||||
|
|
||||||
## Overview
|
## Overview
|
||||||
|
|
||||||
Легкое ревью диффа между двумя Git-ветками в стиле GitLab Merge Request (от merge-base). Запускает быстрый `/review` вместо тяжелого `/code-review`.
|
Лёгкое ревью диффа между двумя Git-ветками в стиле GitLab Merge Request (от merge-base). **Claude делает ревью сам, без multi-agent workflows** — читает дифф и выдаёт только критические проблемы с готовыми исправлениями.
|
||||||
|
|
||||||
## When to Use
|
## When to Use
|
||||||
|
|
||||||
- Пользователь просит "поревьюй ветку feature относительно master" быстро
|
- Пользователь просит "поревьюй ветку feature относительно master" быстро
|
||||||
|
- Пользователь говорит "быстрое ревью", "лёгкое ревью", "поверхностное ревью"
|
||||||
- Нужен дифф как на GitLab (без лишних изменений из целевой ветки)
|
- Нужен дифф как на GitLab (без лишних изменений из целевой ветки)
|
||||||
- Формат вызова: `/el-review <наша-ветка-с-изменениями> <целевая-ветка>`
|
- Формат вызова: `/el-review <наша-ветка-с-изменениями> <целевая-ветка>`
|
||||||
|
|
||||||
@@ -24,12 +25,12 @@ description: Use when пользователь запрашивает легко
|
|||||||
3. Если текущая checkout-ветка == `source`, выполни `git pull --ff-only origin source` — обнови локальную ветку
|
3. Если текущая checkout-ветка == `source`, выполни `git pull --ff-only origin source` — обнови локальную ветку
|
||||||
4. Построй дифф от merge-base: `git diff origin/target...origin/source > /tmp/el-review.diff` (три точки!)
|
4. Построй дифф от merge-base: `git diff origin/target...origin/source > /tmp/el-review.diff` (три точки!)
|
||||||
5. Прочитай файл `/tmp/el-review.diff`
|
5. Прочитай файл `/tmp/el-review.diff`
|
||||||
6. Запусти скилл `review` с этим файлом как аргумент
|
6. **Сделай инлайн-ревью сам** — не запускай `/review`, не спавни multi-agent workflows, не делай deep cross-file investigations:
|
||||||
7. Полученные findings переформатируй:
|
- Найди только критические проблемы: баги, утечки, падения, гонки, ошибки в бизнес-логике, нарушения инвариантов
|
||||||
- Пронумеруй проблемы
|
- **Не репорти:** форматирование, именование переменных, стиль кода, предложения по рефакторингу (если это не баг)
|
||||||
- Оставь только критические замечания
|
- Для каждой проблемы сразу пиши готовый diff с исправлением
|
||||||
- Для каждой предложи готовый дифф с исправлением
|
- Объясняй кратко, на русском, простым языком
|
||||||
- Объясни простым языком
|
- Если критических проблем нет — напиши **«Критических проблем не найдено»** и остановись
|
||||||
|
|
||||||
## Примеры вызова
|
## Примеры вызова
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user