Compare commits
2
Commits
761fd9978b
...
4f034ea840
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
4f034ea840 | ||
|
|
6d302deb76 |
No files matched your search
@@ -31,6 +31,9 @@ description: >-
|
|||||||
- **постановка**: путь к задаче, спеке или критериям приёмки. Без неё линза
|
- **постановка**: путь к задаче, спеке или критериям приёмки. Без неё линза
|
||||||
«по задаче» работает вслепую — так и напиши;
|
«по задаче» работает вслепую — так и напиши;
|
||||||
- **команды проверки** и ожидаемый результат (например, число тестов);
|
- **команды проверки** и ожидаемый результат (например, число тестов);
|
||||||
|
- **границы**, если они есть: куда не ходить (приватные данные, рабочие
|
||||||
|
серверы, боевые скрипты), какие проверки слишком дороги, что в этой среде
|
||||||
|
заведомо не находка;
|
||||||
- **места, где ты сам не уверен** — вопросами, без своих обоснований:
|
- **места, где ты сам не уверен** — вопросами, без своих обоснований:
|
||||||
пересказ твоей логики лишает ревьюера независимости. То же с выводами
|
пересказ твоей логики лишает ревьюера независимости. То же с выводами
|
||||||
другого ревьюера, пока оба не закончили первый заход;
|
другого ревьюера, пока оба не закончили первый заход;
|
||||||
@@ -97,14 +100,20 @@ prompts/lens-….md > prompt.md`. Файлы `prompts/` лежат в катал
|
|||||||
## Разбор находок
|
## Разбор находок
|
||||||
|
|
||||||
Разбирает находки автор: у тебя контекст кода и задачи. Пользователю нужен
|
Разбирает находки автор: у тебя контекст кода и задачи. Пользователю нужен
|
||||||
итог в отчёте, а не участие в каждом споре.
|
итог в отчёте, а не участие в каждом споре. Так же разбирай и отзывы других
|
||||||
|
моделей, которые переслал пользователь.
|
||||||
|
|
||||||
По каждой находке проверь три вещи:
|
По каждой находке проверь три вещи:
|
||||||
|
|
||||||
1. **Верна ли она фактически.** Воспроизведи сценарий, прочти код, запусти
|
1. **Верна ли она фактически.** Воспроизведи сценарий, прочти код, запусти
|
||||||
тест. Ревьюеры ошибаются уверенно: выдумывают API, путают версии,
|
тест. Ревьюеры ошибаются уверенно: выдумывают API, путают версии,
|
||||||
приписывают коду то, чего в нём нет, требуют защиты от невозможных
|
приписывают коду то, чего в нём нет, требуют защиты от невозможных
|
||||||
состояний. Уверенный тон — не доказательство.
|
состояний. Автору легко поверить хорошо изложенной находке, но уверенный
|
||||||
|
тон — не доказательство. Проверяй и то, на чём находка держится:
|
||||||
|
утверждения о прогонах, истории и прошлых решениях сверяй с источником
|
||||||
|
так же, как утверждения о коде. Дешёвая починка от проверки не
|
||||||
|
освобождает: ложная находка, закрытая одной строкой, всё равно добавляет
|
||||||
|
лишнее.
|
||||||
2. **Применима ли к этому проекту** — его контрактам, масштабу, угрозам.
|
2. **Применима ли к этому проекту** — его контрактам, масштабу, угрозам.
|
||||||
3. **Соразмерна ли починка риску.** Предложенный способ — один из вариантов.
|
3. **Соразмерна ли починка риску.** Предложенный способ — один из вариантов.
|
||||||
Подтверждённый дефект закрывай лучшим путём, какой видишь; отказ от
|
Подтверждённый дефект закрывай лучшим путём, какой видишь; отказ от
|
||||||
|
|||||||
@@ -40,7 +40,9 @@
|
|||||||
ревьюер не увидит.
|
ревьюер не увидит.
|
||||||
- **`runtimeMode` не задавай.** Режимы с подтверждением остановят ревьюера
|
- **`runtimeMode` не задавай.** Режимы с подтверждением остановят ревьюера
|
||||||
на первом же действии и будут ждать человека в его разговоре. Ограничения
|
на первом же действии и будут ждать человека в его разговоре. Ограничения
|
||||||
задаёт мандат в поручении.
|
задаёт мандат в поручении. Режим наследуется от твоего разговора, а шире
|
||||||
|
своего задать нельзя: если ты сам работаешь с подтверждением, скажи
|
||||||
|
пользователю до запуска.
|
||||||
- После запуска заверши ход: уведомление о завершении придёт само.
|
- После запуска заверши ход: уведомление о завершении придёт само.
|
||||||
`task_status` нужен, только если результат требуется посреди хода.
|
`task_status` нужен, только если результат требуется посреди хода.
|
||||||
- Итоговое сообщение ревьюера приходит в `summary`; полный отчёт — в файле
|
- Итоговое сообщение ревьюера приходит в `summary`; полный отчёт — в файле
|
||||||
|
|||||||
Reference in new issue
Block a user