Роль агента — це межа, а не титул
Reviewer, tester, debugger і documenter мають різні цілі та write boundary. Спільне слово agent не означає, що вони взаємозамінні. Перед викликом оберіть роль за результатом і явно обмежте поверхню змін.
Чим чутливіша область, тим важливіше розвести збір evidence, технічну перевірку та людське рішення.
| Роль | Мета | Межа запису |
|---|---|---|
| Reviewer | Diff, scope, ризики й edge cases | Нічого |
| Tester | Сценарії, відтворення й результати тестів | Тести лише за явним дозволом |
| Debugger | Пошук root cause і перевірка гіпотези | Зазвичай без редагування коду |
| Documenter | Документація на основі verified behavior | Документація в погодженій області |
Три шари review
| Шар | Хто або що | Питання |
|---|---|---|
| 1. Local review | Автор або reviewer agent | Чи є очевидні дефекти, scope drift і прогалини? |
| 2. Technical gate | CI, тести, build, lint, smoke | Чи підтверджує виконання технічний evidence? |
| 3. Team decision | Людина або owner чутливої області | Чи прийнятні ризик, дизайн і наслідки? |
Verification і review — різні дії
Verification збирає факти: запускає тест, читає diff, перевіряє маршрут або фіксує лог. Review інтерпретує ці факти та вирішує, чи достатньо їх для наступного кроку. Не називайте сам факт запуску тесту approval.
Наприклад, зелений build може підтвердити компіляцію, але не відповісти на питання про межі задачі, UX або міграційний ризик.
Спеціалізуйте review за ризиком
- security reviewer — permissions, секрети й межі доступу;
- performance reviewer — гарячі шляхи, latency і зайві операції;
- test-quality reviewer — покриття сценаріїв і слабкі assertions;
- architecture reviewer — залежності, boundaries і довгострокова ціна;
- frontend/UI reviewer — стани інтерфейсу, доступність і responsive поведінка.
Контрольний список перед рішенням
- Чи відповідає роль конкретному питанню?
- Чи має звіт file:line та evidence?
- Чи відділені факти від [hypothesis]?
- Чи виконано технічні gates, а не лише local review?
- Чи залишилося рішення за людиною, яка відповідає за область?
- Чи зафіксовано open questions і наступний крок?
Канонічне джерело уроку · JavaRush
Відкрити матеріал JavaRush