Skip to main content
المدونة

مراجعة الكود بالذكاء: ما ينجح فعلا

٢٨‏/٣‏/١٤٤٨ هـ

٩‏/٩‏/٢٠٢٦ م

مراجعة الذكاء السيئة تزحم الطلبات بملاحظات تنسيق وAPIs مهلوسة وتعليقات حسّن المعالجة الغامضة. والجيدة تجد await مفقودا على مدفوعات وwebhooks غير متحقق منها وأخطاء RTL عربية واستعلامات N+1 مع ملف وسطر وإصلاح.

وهذا هو سير العمل الذي خفض وقت مراجعتنا 30 بالمئة مع بقاء البشر ملاكا.

1. النطاق للفرق لا المستودع

راجع الأسطر المتغيرة فقط مع 20 سطر سياق بحد 400 سطر لكل تشغيل. والفروقات الأكبر تقسم حسب الملف. ومراجعات الملف الكامل تكرر دينا معروفا وتغرق الإشارة. ويجب أن يقول الموجه راجع هذا الفرق فقط ولا تعلق على كود لم يتغير إلا إذا كسر التغيير.

واشترط خطورة لكل تعليق. وحاجب لفقد بيانات أو أمن ورئيسي لأخطاء وأداء وثانوي لمقروئية. وامنع تعليقات التنسيق. واترك التنسيق لـ ESLint. وتعليق تنسيق واحد يقوض الثقة فيتجاهل المطورون البوت.

2. قواعد مخصصة تطابق ستاكك

القواعد العامة تنتج مراجعات عامة. وشفر قواعد ستاكك السعودية. ولا any لحمولات النظام وتحقق من كل webhook بـ Zod وعالج RTL العربية بخصائص منطقية واستخدم مفاتيح منع تكرار لميسر وفاتورة وافحص تقريب ضريبة نصف لأعلى.

وأعط أنماط ملفات. وللمدفوعات افحص ازدواج نقرة مزدوجة. ولـ Next.js افحص سوء مكون خادم مقابل عميل. وللنماذج العربية افحص dir auto على مدخلات الجوال. ويجب أن يستشهد البوت بمعرف قاعدة مثل PAY-02 في كل تعليق ليتعلم الكتاب.

وحدث القواعد من دروس الحوادث. وعندما يسبب await مفقود التقاطا مزدوجا أضف قاعدة مع رابط حادث. وملف القواعد يعيش في المستودع ويُراجع ككود.

3. اطلب إعادة إنتاج مع إصلاح

كل تعليق رئيسي يحتاج لماذا يكسر مع مثال سعودي مع فرق مقترح. وتعليق سيئ ضع حالات حدية. وتعليق جيد order.items[0] يتعطل عندما تحوي السلة عينة مجانية مع مصفوفة فارغة من طلب اختبار 4521 وأصلح إذا فارغة ارجع مبكرا مع حالة اختبار.

واشترط على النموذج اقتباس الأسطر التي يشير إليها حرفيا. وإذا لم يقتبس يجب ألا يعلق. وهذه القاعدة وحدها تقتل معظم الهلوسات.

4. تركيز أمن وأداء

أقسام موجه لأسرار في الفرق وحقن SQL عبر دمج نصوص وXSS عبر dangerouslySetInnerHTML مع محتوى عربي وتحويل مفتوح بعد دخول. وللأداء علّم N+1 على بنود طلبات وغياب ترقيم على بحث عربي وجلب نظام غير مخزن في حلقات.

ولـ RTL علّم أصناف ml وmr التي تكسر تخطيط العربية وleft وright المكتوبة في CSS وأرقام جوال بدون dir ltr. وهذه غير مرئية لمراجعين إنجليز فقط لكنها تكسر تجربة سعودية.

5. ملكية بشرية ومقاييس

البوت لا يدمج أبدا. والبشري يعتمد. وتتبع نسبة قبول تعليقات البوت ونسبة الإيجاب الكاذب عبر عدم إعجاب وزمن فتح لدمج. واستهدف فوق 50 بالمئة مفيدا. وراجع قواعد البوت شهريا مع الفريق واحذف المزعج سريعا.

وشغل مراجعة على مسودات لإشارة مبكرة وكاملة عند جاهز. واحتفظ بسجلات أوامر وفروقات شهرا لتصحيح نصائح سيئة.

والخلاصة فروقات محددة وقواعد ستاك سعودية مخصصة وإصلاحات مقتبسة ودمج بشري. وافعل ذلك ويصبح مراجع الذكاء مراجعا مبتدئا لا ينام لا إزعاجا.