Код пишется быстро, а што с ревью
Я как-то уже бухтел на тему код-ревью. Но мир так быстро меняется, что хочется поднять её снова.
Код теперь генерируют агенты. Увидеть пиар на +4623 −200 – да запросто. А дальше у всех по-разному.
Вижу проекты, где код всё так же по старинке дотошно и вдумчиво смотрят люди. Ревью становится бутылочным горлышком: код пишется быстро, пиары висят днями.
Где-то к просмотру глазами подключают агента: «Сделай ревью и смотри, чтобы там всё хорошо было» – тоже вариант.
На одном из проектов мы тоже столкнулись с этой проблемой. Задач много, бежать нужно быстро, ещё и у внешних контрибьюторов чешутся руки – дайте занести кода к вам. Пиары летят, ревью загибается.
Мы решили сделать накопленный опыт ревью доступным агентам ещё до создания пиара. Для этого:
- собрали все-все замечания из прошлых пиаров – проект не новый, истории хватало
- прошерстили личные чаты с агентами и вытащили оттуда замечания к их работе
- на этой основе сформировали гайдлайны по написанию и тестированию кода – что-то положили в AGENTS.md, что-то обернули в скиллы
Так агент может ещё во время написания кода учесть, как у нас всё устроено.
Дальше, уже в CI, добавили дополнительные валидации и ревью агентом.
В итоге обычно смотрим пиары глазами по минимуму: многие замечания уже учтены по умолчанию. А иногда и не смотрим вовсе :)
Пока проблему с очередью из пиаров удалось решить. Как этот подход скажется на качестве кода на дистанции, ещё предстоит понять. По результатам будем тюнить процесс. Чтобы держать руку на пульсе, периодически внимательно смотрим код: как связаны части проекта, не расползается ли структура.
Расскажите, как у вас устроено код-ревью? Что хорошо работает?
Остаётся, конечно, ещё один вопрос: кто в этой схеме отвечает за все эти тонны кода? Но это уже совсем другая история.
Я как-то уже бухтел на тему код-ревью. Но мир так быстро меняется, что хочется поднять её снова.
Код теперь генерируют агенты. Увидеть пиар на +4623 −200 – да запросто. А дальше у всех по-разному.
Вижу проекты, где код всё так же по старинке дотошно и вдумчиво смотрят люди. Ревью становится бутылочным горлышком: код пишется быстро, пиары висят днями.
Где-то к просмотру глазами подключают агента: «Сделай ревью и смотри, чтобы там всё хорошо было» – тоже вариант.
На одном из проектов мы тоже столкнулись с этой проблемой. Задач много, бежать нужно быстро, ещё и у внешних контрибьюторов чешутся руки – дайте занести кода к вам. Пиары летят, ревью загибается.
Мы решили сделать накопленный опыт ревью доступным агентам ещё до создания пиара. Для этого:
- собрали все-все замечания из прошлых пиаров – проект не новый, истории хватало
- прошерстили личные чаты с агентами и вытащили оттуда замечания к их работе
- на этой основе сформировали гайдлайны по написанию и тестированию кода – что-то положили в AGENTS.md, что-то обернули в скиллы
Так агент может ещё во время написания кода учесть, как у нас всё устроено.
Дальше, уже в CI, добавили дополнительные валидации и ревью агентом.
В итоге обычно смотрим пиары глазами по минимуму: многие замечания уже учтены по умолчанию. А иногда и не смотрим вовсе :)
Пока проблему с очередью из пиаров удалось решить. Как этот подход скажется на качестве кода на дистанции, ещё предстоит понять. По результатам будем тюнить процесс. Чтобы держать руку на пульсе, периодически внимательно смотрим код: как связаны части проекта, не расползается ли структура.
Расскажите, как у вас устроено код-ревью? Что хорошо работает?
Остаётся, конечно, ещё один вопрос: кто в этой схеме отвечает за все эти тонны кода? Но это уже совсем другая история.