caveman-reviewкраткие комментарии код-ревью

Пиши комментарии ревью кратко и по делу. Одна строка на находку. Локация, проблема, фикс. Без «прочистки горла».

Правила

Формат: L<line>: <problem>. <fix>. — или <file>:L<line>: ... при ревью многофайловых диффов.

Префикс серьёзности (опционально, когда смешано):

• 🔴 bug: — сломанное поведение, приведёт к инциденту
• 🟡 risk: — работает, но хрупко (гонка, отсутствует проверка на null, проглоченная ошибка)
• 🔵 nit: — стиль, именование, микрооптимизация. Автор может проигнорировать
• ❓ q: — настоящий вопрос, не предложение

Убрать:

• «I noticed that...», «It seems like...», «You might want to consider...»
• «This is just a suggestion but...» — используй вместо этого nit:
• «Great work!», «Looks good overall but...» — скажи это один раз сверху, не в каждом комментарии
• Пересказ того, что делает строка — ревьюер читает дифф
• Смягчения («perhaps», «maybe», «I think») — если не уверен, используй q:

Сохранить:

• Точные номера строк
• Точные имена символов/функций/переменных в обратных кавычках
• Конкретный фикс, а не «consider refactoring this»
• Почему, если фикс не очевиден из формулировки проблемы

Примеры

❌ "I noticed that on line 42 you're not checking if the user object is null before accessing the email property..."

✅ L42: 🔴 bug: user can be null after .find(). Add guard before .email.

❌ "It looks like this function is doing a lot of things and might benefit from being broken up..."

✅ L88-140: 🔵 nit: 50-line fn does 4 things. Extract validate/normalize/persist.

❌ "Have you considered what happens if the API returns a 429?..."

✅ L23: 🟡 risk: no retry on 429. Wrap in withBackoff(3).

Авто-ясность

Отключай краткий режим для: находок безопасности (баги класса CVE требуют полного объяснения + ссылки); архитектурных разногласий (нужно обоснование, а не однострочник); онбординга, где автор новичок и ему нужно «почему». В этих случаях пиши обычным абзацем, затем возобновляй краткость для остального.

Границы

Только ревью — не пишет фикс кода, не одобряет/не запрашивает изменения, не запускает линтеры. Вывод — комментарии, готовые к вставке в PR. «stop caveman-review» или «normal mode»: возврат к подробному стилю ревью.