Токсичный код и токсичные люди: как чистить команду от «звезд», которые валят проект

Привет, интернет. На связи Алексей Прохоров — директор по развитию технологий искусственного интеллекта, и вы в подкасте «Токсичный код и токсичные люди: как чистить команду от «звезд», которые валят проект». Сегодня без лирики. Будем говорить о хронической проблеме. Нет, не только о той, что в коде, хотя и о ней тоже. О той, что сидит в кресле напротив и называет себя "хардкорным сеньором". Вы знаете этих ребят. Они коммитят по ночам, пишут идеальный код на голом энтузиазме и… методично валят ваш проект.Давайте сразу признаемся. Нас с детства учат любить звёзд. В футболе Месси решает исход матча, в музыке Фредди Меркьюри делает шоу. В IT работает другая физика. Если разработчик называет себя "рок-звездой" или прячет архитектуру у себя в голове — это не талант. Это деструктивное поведение.Я веду речь о типе "Серебряная критическая зависимость". Этот человек пишет гениальный, сжатый, нечитаемый код. Он правит баги в 3 часа ночи. Он снизил время ответа API на пол секунды. Казалось бы — бери и плати два оклада. Но вот вопрос: почему в команде, где есть такая "звезда", остальные превращаются в овощей? Потому что звезда не светит, она греет.Чем токсичный код отличается от просто кривого? Кривой код можно починить рефакторингом. Токсичный код — это инструмент саботажа. Это "умные" однострочники без документации. Это макросы, которые понятны только автору. Это архитектура "на вырост", где вырос только эго разработчика.Я видел проект, где "звезда" написал свой велосипедный DI-контейнер за выходные. Работало быстро. Через три месяца, когда он ушел в отпуск, релиз встал. Потому что никто не смог доказать, почему контейнер падает именно при сотом запросе. Автор вернулся, почекал полдня, поправил одну строчку и сказал: "Ну вы же тупые, это очевидно". Это не очевидно. Это токсичный код, написанный для самоутверждения.Самое страшное происходит не в репозитории, а в головах команды. Когда в коллективе есть "Незаменимый", менеджмент впадает в зависимость. Ему страшно сделать замечание — вдруг уволится и всё рухнет? А он знает этот страх и использует его.Такие люди создают "феодальные вотчины". Они не дают джуниорам задавать вопросы. Они переписывают пул-реквесты так, что новичок плачет в третьей попытке. Они говорят: "Я переделаю это сам, быстрее". И через месяц вся база знаний лежит у одного человека в голове. Пока этот человек в сознании — бизнес жив. Как только он устал, обиделся или нашёл оффер за 300 тысяч — всё. Вы ловите стоп фактор.Запомните: Звезда — это актив, который всегда в процессе увольнения.Хватит диагностики. Как резать? У нас нет времени на разбор полетов с психологом. Есть три шага:Первый. Документируем "магию".Вы объявляете Code Review обязательным для всех, включая звезду. Правило простое: "Любая магия, которая не объяснена в комментарии или доке, отправляется на доработку". Звезда начнёт беситься, скажет, что это бюрократия. Прекрасно. Это фильтр.Второй. Тихий эксперимент "Отпуск".Попросите звезду уйти в оплачиваемый отпуск на две недели прямо перед релизом. Шучу, конечно. Но метод рабочий. Если без него команда стопится и не может сделать пуш в прод — значит, вы наняли не сотрудника, а костыль для своей трусости. Результат эксперимента: либо звезда пишет документацию, либо вы понимаете, что его пора выпиливать.Третий. Холодный расчёт.Когда вы решаетесь на увольнение токсичной звезды, готовьтесь к драме. Он скажет: "Всё посыплется", "Вы без меня ноль". Не верьте. За первые две недели без него будет хаос. Это абстинентный синдром команды. На третьей неделе джуниоры начнут дышать. На четвёртой — кто-то возьмёт на себя его задачи. Через два месяца вы удивитесь, насколько стабильно работает система, которую вы ненавидели.