diff --git a/home-configs/claude/skills/el-review/SKILL.md b/home-configs/claude/skills/el-review/SKILL.md index 98ccf39..84499bd 100644 --- a/home-configs/claude/skills/el-review/SKILL.md +++ b/home-configs/claude/skills/el-review/SKILL.md @@ -1,17 +1,18 @@ --- 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 ## Overview -Легкое ревью диффа между двумя Git-ветками в стиле GitLab Merge Request (от merge-base). Запускает быстрый `/review` вместо тяжелого `/code-review`. +Лёгкое ревью диффа между двумя Git-ветками в стиле GitLab Merge Request (от merge-base). **Claude делает ревью сам, без multi-agent workflows** — читает дифф и выдаёт только критические проблемы с готовыми исправлениями. ## When to Use - Пользователь просит "поревьюй ветку feature относительно master" быстро +- Пользователь говорит "быстрое ревью", "лёгкое ревью", "поверхностное ревью" - Нужен дифф как на GitLab (без лишних изменений из целевой ветки) - Формат вызова: `/el-review <наша-ветка-с-изменениями> <целевая-ветка>` @@ -24,12 +25,12 @@ description: Use when пользователь запрашивает легко 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` с этим файлом как аргумент -7. Полученные findings переформатируй: - - Пронумеруй проблемы - - Оставь только критические замечания - - Для каждой предложи готовый дифф с исправлением - - Объясни простым языком +6. **Сделай инлайн-ревью сам** — не запускай `/review`, не спавни multi-agent workflows, не делай deep cross-file investigations: + - Найди только критические проблемы: баги, утечки, падения, гонки, ошибки в бизнес-логике, нарушения инвариантов + - **Не репорти:** форматирование, именование переменных, стиль кода, предложения по рефакторингу (если это не баг) + - Для каждой проблемы сразу пиши готовый diff с исправлением + - Объясняй кратко, на русском, простым языком + - Если критических проблем нет — напиши **«Критических проблем не найдено»** и остановись ## Примеры вызова