За годы код-ревью я видел всякое, но один пул-реквест от мидла заставил меня проверить, не взломали ли наш гит. Задача была простой. Починить падение внешней интеграции в корзине. буквально 20 строк кода. Вместо этого автор принёс полторы тысячи строк изменений в десяти модулях. Он решил, что раз уж залез в код, то заодно перепишет всю логику оформления заказа, обновит старые библиотеки и внедрит модный паттерн, о котором только утром прочитал в интернете. Внутри этого хаоса творился настоящий ад. Вместо аккуратного перехвата ошибок там красовалась конструкция из пяти вложенных try-catch, которая просто глушила любые исключения, а в логи выводила лаконичное «Ой». Вдобавок для оптимизации автор воткнул сырой SQL-запрос с дырой для инъекций прямо внутрь цикла, из-за чего тестовый стенд мгновенно ложился под минимальной нагрузкой. Но вишенкой на торте стали упавшие тесты. Вместо того чтобы разобраться, почему его глобальный рефакторинг всё сломал, разработчик просто закомментировал все м