Запрос ревью кода

Отправьте субагента-ревьюера, чтобы поймать проблемы до того, как они каскадом разойдутся. Ревьюер получает точно подготовленный контекст для оценки — никогда не историю вашей сессии. Это держит ревьюера сфокусированным на продукте работы, а не на вашем ходе мыслей, и сохраняет ваш собственный контекст для продолжения работы.

Главный принцип: ревьюйте рано, ревьюйте часто.

Когда запрашивать ревью

Обязательно:

• После каждой задачи в разработке через субагентов
• После завершения крупной функции
• Перед слиянием в main

Необязательно, но полезно:

• Когда застряли (свежий взгляд)
• Перед рефакторингом (проверка базовой точки)
• После исправления сложного бага

Как запросить

1. Получите git SHA:

BASE_SHA=$(git rev-parse HEAD~1)  # or origin/main
HEAD_SHA=$(git rev-parse HEAD)

2. Отправьте субагента-ревьюера кода:

Отправьте субагента типа general-purpose, заполнив шаблон из code-reviewer.md

Плейсхолдеры:

• {DESCRIPTION} — краткое резюме того, что вы построили
• {PLAN_OR_REQUIREMENTS} — что оно должно делать
• {BASE_SHA} — начальный коммит
• {HEAD_SHA} — конечный коммит

3. Действуйте по обратной связи:

• Critical-проблемы исправляйте немедленно
• Important-проблемы исправляйте до продолжения
• Minor-проблемы отметьте на потом
• Возражайте, если ревьюер неправ (с обоснованием)

Пример

[Just completed Task 2: Add verification function]

You: Let me request code review before proceeding.

BASE_SHA=$(git log --oneline | grep "Task 1" | head -1 | awk '{print $1}')
HEAD_SHA=$(git rev-parse HEAD)

[Dispatch code reviewer subagent]
  DESCRIPTION: Added verifyIndex() and repairIndex() with 4 issue types
  PLAN_OR_REQUIREMENTS: Task 2 from docs/superpowers/plans/deployment-plan.md
  BASE_SHA: a7981ec
  HEAD_SHA: 3df7661

[Subagent returns]:
  Strengths: Clean architecture, real tests
  Issues:
    Important: Missing progress indicators
    Minor: Magic number (100) for reporting interval
  Assessment: Ready to proceed

You: [Fix progress indicators]
[Continue to Task 3]

Интеграция с рабочими процессами

Разработка через субагентов:

• Ревью после КАЖДОЙ задачи
• Ловите проблемы до того, как они накопятся
• Исправляйте до перехода к следующей задаче

Исполнение планов:

• Ревью после каждой задачи или в естественных контрольных точках
• Получите обратную связь, примените, продолжайте

Ad-hoc разработка:

• Ревью перед слиянием
• Ревью, когда застряли

Красные флаги

Никогда:

• Не пропускайте ревью, потому что «это просто»
• Не игнорируйте Critical-проблемы
• Не продолжайте с неисправленными Important-проблемами
• Не спорьте с верной технической обратной связью

Если ревьюер неправ:

• Возражайте с техническим обоснованием
• Покажите код/тесты, доказывающие, что это работает
• Запросите уточнение

См. шаблон: code-reviewer.md