mr-review

MR Review — ревью чужого Merge Request

Полный цикл code review для MR в GitLab: получить diff, проанализировать код, опубликовать замечания как inline threads.

Подключения — из project-index.md в auto-memory проекта и ~/.claude/memory/infra-{group}.md.

Использование

/mr-review 42
/mr-review 42 --focus security
/mr-review 42 --focus performance

Flow

FETCH MR → READ CODE → REVIEW → SUMMARY → ⏸ HITL → PUBLISH

---

1. FETCH — получить MR

# Детали MR (title, description, author, source/target branch, diff_refs)
pcurl @{gl_profile} 'https://{gl_host}/api/v4/projects/{gl_project_id}/merge_requests/{mr_iid}' -s

# Diff (список изменённых файлов с содержимым)
pcurl @{gl_profile} 'https://{gl_host}/api/v4/projects/{gl_project_id}/merge_requests/{mr_iid}/changes' -s

# Коммиты MR
pcurl @{gl_profile} 'https://{gl_host}/api/v4/projects/{gl_project_id}/merge_requests/{mr_iid}/commits' -s

# Существующие discussions (чтобы не дублировать замечания)
pcurl @{gl_profile} 'https://{gl_host}/api/v4/projects/{gl_project_id}/merge_requests/{mr_iid}/discussions?per_page=100' -s

Извлечь:

• diff_refs (base_sha, start_sha, head_sha) — нужны для inline comments
• Список изменённых файлов и их diff
• Описание MR и связанную задачу (из title или description)
• Уже существующие замечания (не дублировать)

2. READ CODE — прочитать контекст

Для каждого изменённого файла:

• Прочитать полный файл локально (не только diff) — для понимания контекста
• Если ветка MR доступна локально — git fetch origin && git checkout {source_branch}
• Если нет — читать через API или анализировать diff as-is

Дополнительно:

• Прочитать связанную задачу из YouTrack (если ID в title/description)
• Понять что должен делать MR — сравнить intent с реализацией

3. REVIEW — анализ кода

Запустить /go-review в режиме MR: изменения — diff из шага 1 (или git diff origin/{target_branch}...origin/{source_branch}), preflight локальной базы не нужен. 6 ревьюеров:

1. Common — соответствие задаче, полнота
2. Architecture (Dave Cheney) — структура, зависимости
3. Code (Rob Pike) — простота, читаемость
4. Security (Filippo Valsorda) — уязвимости, authz
5. Tests (Mitchell Hashimoto) — покрытие, качество тестов
6. Operability (Peter Bourgon) — логирование, метрики, graceful degradation