0e15033351
Убран вызов встроенного /review (multi-agent workflow). Теперь Claude читает дифф сам и выдаёт только критические проблемы с готовыми diff-фиксами. - Никаких sub-agent'ов и deep cross-file investigations - Только баги, утечки, падения, гонки, нарушения инвариантов - Исключены: форматирование, нейминг, стиль, рефакторинг - Готовый diff на каждую проблему, коротко, на русском Co-Authored-By: Claude <noreply@anthropic.com>
64 lines
3.6 KiB
Markdown
64 lines
3.6 KiB
Markdown
---
|
||
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
|
||
|
||
1. Определи аргументы:
|
||
- `source` — первая ветка (наша, с изменениями, которую ревьюим)
|
||
- `target` — вторая ветка (целевая, в которую мерж-реквест)
|
||
2. Выполни `git fetch origin` — обнови remote refs
|
||
3. Если текущая checkout-ветка == `source`, выполни `git pull --ff-only origin source` — обнови локальную ветку
|
||
4. Построй дифф от merge-base: `git diff origin/target...origin/source > /tmp/el-review.diff` (три точки!)
|
||
5. Прочитай файл `/tmp/el-review.diff`
|
||
6. **Сделай инлайн-ревью сам** — не запускай `/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
|
||
- старая строка
|
||
+ новая строка
|
||
```
|
||
```
|