Если вы умеете делать хороший code review, вы умеете работать с AI-агентами

[Zoolander]

Я в agents.md записал - сначала создаём файл задачи с описанием, анализом существующего кода и пишем два варианта решения - одно предельно KISS с минимальными правками, второе по уму

И знаете, ни разу не было, чтобы он в Kiss варианте предложил какой-то ООП головного мозга. Часто, особенно с багами, в первом варианте он дает реально решения в несколько строчек

Пользуйтесь.

Ps это уже не гворя о том, что оба варианта я с ним обсуждаю, прежде чем разрешить работать.

[wanhelsing]

Интересный подход. Я сначала его пытаю в Plan Mode, и только когда всё устроит - нажимаю принять все правки без обсуждений.

[kuza2000]

Мне что-то Plan Mode не зашёл. Но план использую всегда, просто прошу агента написать план и задать вопросы.

[kuza2000]

Никогда не пытаюсь написать "промпт". Просто обсуждаю задачу с агентом в форме диалога, потом прошу написать план, задать вопросы. После этого корректирую план, отвечаю на вопросы. Когда все устраивает, разрешаю реализацию. С топовыми сетями типа опуса 4.6 получается идеальная реализация и работает почти всегда сразу, без отладки. С агентами похуже нужно больше направлять его и больше детализировать план, но тоже получаю хороший результат, только с большими усилиями.

Самое важное - ревьюить план, а не код. В коде ошибается очень редко, а в плане - да, может пойти не по тому пути. В этом я согласен с автором статьи.

[ToniDoni]

А вы пробовали написать хороший reviewer.md и ревьюить тоже иишкой, раз уж взялись за систематизацию знаний о ревью?

[wanhelsing]

написать хороший reviewer.md

У меня пока такой (для Claude):

---

name: my.code-review

description: Perform a code review

disable-model-invocation: true

---

Perform a code review of the last commit in this branch. Flag unused/dead code, duplication, public methods that should be private, mutation of input parameters, missing readonly annotations, common code smells. it('test description') - 'test description' should correspond to the tested functionality. Check if any new code could be replaced with existing Fabric.js built-in functions or third-party geometric 2D libraries (think thoroughly, search the web). Check if vector operations could be replaced with Fabric.js util.* methods. Check every new method and make sure there are no existing methods that do the same thing.

тулза для 2D рисования