Code-Review

Давайте на мгновение будем честны. Ваш любимый коллега не обязательно является любимым коллегой при проверке вашего кода. Тот старший разработчик, который утвердил ваш пул реквест в 17:50 в пятницу? Да, он не проводил глубокий архитектурный анализ. Он был в одном клике от свободы, и отладка через console.log не испортила бы ему выходные. Добро пожаловать в запутанную реальность проверок кода человеком: они предвзяты, непоследовательны и иногда жестоко честны, а иногда удобно забывчивы....

Будущее уже здесь, и оно генерирует код быстрее, чем ваша кофемашина варит эспрессо. Но есть нюанс: то, что ИИ может написать код за миллисекунды, не означает, что этот код готов к продакшену. На самом деле, считать код, сгенерированный ИИ, абсолютной истиной — это как доверять навигатору, который иногда решает, что дороги больше не существуют. Вы можете так делать, но, скорее всего, в итоге окажетесь в озере. Если вы интегрируете ИИ в свой рабочий процесс разработки — а давайте будем честными, большинство из нас это делает — вам нужна надёжная стратегия, чтобы гарантировать, что то, что попадает в продакшен, является надёжным, безопасным и не заставляет вашего будущего «я» хотеть перевернуть стол....

Иллюзия полезного робота-рецензента Вы знаете это чувство, когда кто-то говорит, что ваш код плох, но делает это самым профессиональным образом? Как будто получаете критику от робота в деловой одежде — нейтральный тон, отсутствие осуждения, максимальный эффект. Добро пожаловать в мир проверок кода с помощью ИИ, где нам как-то удалось сделать обратную связь одновременно менее эмоционально обременительной и бесконечно более напряжённой. Это аналог разработки программного обеспечения, когда вы обнаруживаете, что ваш партнёр по тренировкам теперь безжалостный надзиратель с ИИ, который никогда не устаёт....

Позвольте мне рассказать о самом дорогом «LGTM», который я когда-либо видел. Он был в запросе на включение, который выглядел достаточно безобидно — небольшое изменение в нашей логике обработки платежей. Рецензент, старший инженер, которого я уважал, одобрил его, оставив комментарий, который до сих пор преследует меня: «Отлично выглядит! Хорошая работа, что всё просто 👍». Этот «простой» код был запущен и тут же дважды списал средства за заказы 847 клиентов во время Чёрной пятницы....

Начну с неоднозначного утверждения, которое, вероятно, вызовет возмущение: ревью кода, как их проводит большинство команд, — колоссальная трата времени. Прежде чем хвататься за вилы и факелы, выслушайте меня. Я был по обе стороны баррикад — и в роли ревьювера, погрязшего в diff, и в роли разработчика, который ждёт одобрения, пока его блестящий код устаревает. Неудобная правда заключается в том, что мы превратили ревью кода в подобие карго-культа. Мы проводим их, потому что «лучшие практики» говорят, что так надо, а не потому, что они действительно улучшают наш код или делают наши команды продуктивнее....