vitaly
6120f34199
- Установить часовую зону Europe/Helsinki в ai-api-helpers.sh - Добавить примеры вызова обоих skill'ов - Документировать обработку ошибок (отсутствие веток, пустой дифф) Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
63 lines
2.8 KiB
Markdown
63 lines
2.8 KiB
Markdown
---
|
||
name: el-review-heavy
|
||
description: Use when пользователь запрашивает тяжелое code-review между двумя Git-ветками с диффом в стиле GitLab (от merge-base)
|
||
---
|
||
|
||
# Eltex Review Heavy
|
||
|
||
## Overview
|
||
|
||
Тяжелое ревью диффа между двумя Git-ветками в стиле GitLab Merge Request (от merge-base). Запускает полный `/code-review` с кучей агентов.
|
||
|
||
## When to Use
|
||
|
||
- Пользователь просит "поревьюй ветку feature относительно master" тщательно
|
||
- Нужен дифф как на GitLab (без лишних изменений из целевой ветки)
|
||
- Формат вызова: `/el-review-heavy <наша-ветка-с-изменениями> <целевая-ветка>`
|
||
|
||
## 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-heavy.diff` (три точки!)
|
||
5. Прочитай файл `/tmp/el-review-heavy.diff`
|
||
6. Запусти скилл `code-review` с этим файлом как аргумент
|
||
7. Полученные findings переформатируй:
|
||
- Пронумеруй проблемы
|
||
- Оставь только критические замечания
|
||
- Для каждой предложи готовый дифф с исправлением
|
||
- Объясни простым языком
|
||
|
||
## Примеры вызова
|
||
|
||
```
|
||
/el-review-heavy feature main
|
||
/el-review-heavy my-fix develop
|
||
/el-review-heavy TASK-123-payment-screen master
|
||
```
|
||
|
||
## Ошибки и крайние случаи
|
||
|
||
- Если ветка не существует на remote - сообщи пользователю и не продолжай
|
||
- Если дифф пустой - сообщи "изменений между ветками нет" и не продолжай
|
||
- Если аргументы не указаны - попроси уточнить: `/el-review-heavy <source-ветка> <target-ветка>`
|
||
|
||
## Формат вывода
|
||
|
||
```
|
||
## Проблема N: [краткое название]
|
||
|
||
**Где:** `file.kt:123`
|
||
|
||
**Что не так:** простым языком, зачем это плохо
|
||
|
||
**Исправление:**
|
||
```diff
|
||
- старая строка
|
||
+ новая строка
|
||
```
|
||
```
|