كتابة برومبت نظام لمراجعة الكود آلياً بالذكاء الاصطناعي
برومبت نظام لمراجعة كود آلية: صحة وأمان قبل الأسلوب، بحالة فشل وإصلاح أصغر، ووسم ما يحتاج مراجعة بشرية.
بواسطة Ahmed EidClaude0 نسخة
متى تستخدم هذا البرومبت
- عند ربط مراجعة آلية بمستودعك.
- عند إغراق المراجعة الآلية فريقك بملاحظات أسلوبية.
- قبل الاعتماد على مراجعة آلية في مسار الدمج.
نص البرومبت
اكتب برومبت نظام لمساعد يراجع طلبات الدمج آلياً في مستودعنا. اللغات: {{اللغات}}. أولوياتنا: {{الأولويات}}. معاييرنا: {{المعايير}}.
يجب أن يشمل: ترتيب المراجعة (صحة · أمان · أداء · قراءة · اتساق مع المعايير)، ومنع الملاحظات الأسلوبية التي يغطيها المنسّق الآلي، وأن لكل ملاحظة: الملف والسطر، وحالة الفشل المحددة، والإصلاح المقترح كأقل تغيير ممكن، ومنع الملاحظات الظنية («قد يسبب مشكلة») بلا حالة محددة، وسلوكاً عند نقص السياق (يسأل ولا يفترض).
أضف: لا يوافق على الدمج ولا يرفضه؛ يخرج بملاحظات وتقدير خطورة. ولا يقترح إعادة كتابة واسعة في مراجعة طلب صغير. ويوسم ما يحتاج مراجعة بشرية (تغييرات أمنية، ترحيل بيانات، تغيير عقود واجهات).
أعطني البرومبت ونموذج تقرير مراجعة.
{{اللغات}}{{الأولويات}}{{المعايير}}املأ المتغيرات ثم انسخ
اكتب برومبت نظام لمساعد يراجع طلبات الدمج آلياً في مستودعنا. اللغات: {{اللغات}}. أولوياتنا: {{الأولويات}}. معاييرنا: {{المعايير}}.
يجب أن يشمل: ترتيب المراجعة (صحة · أمان · أداء · قراءة · اتساق مع المعايير)، ومنع الملاحظات الأسلوبية التي يغطيها المنسّق الآلي، وأن لكل ملاحظة: الملف والسطر، وحالة الفشل المحددة، والإصلاح المقترح كأقل تغيير ممكن، ومنع الملاحظات الظنية («قد يسبب مشكلة») بلا حالة محددة، وسلوكاً عند نقص السياق (يسأل ولا يفترض).
أضف: لا يوافق على الدمج ولا يرفضه؛ يخرج بملاحظات وتقدير خطورة. ولا يقترح إعادة كتابة واسعة في مراجعة طلب صغير. ويوسم ما يحتاج مراجعة بشرية (تغييرات أمنية، ترحيل بيانات، تغيير عقود واجهات).
أعطني البرومبت ونموذج تقرير مراجعة.كيف تستخدمه
- انسخ البرومبت بالزر أو املأ المتغيرات أولاً.
- الصقه في ChatGPT أو Claude أو Gemini.
- عدّل النتيجة أو أعد الطلب بتغيير المتغيرات.
مثال على النتيجة
نموذج التقرير: خطورة عالية (2): 1. `src/db/query.ts:41` — المُدخل يُدرج في الاستعلام نصاً. الفشل: قيمة تحتوي علامة اقتباس تكسر الاستعلام وتسمح بتنفيذ غير مقصود. الإصلاح: معاملات مُهيّأة (سطر واحد). 2. `src/api/user.ts:88` — لا تحقق من ملكية السجل قبل التعديل. الفشل: مستخدم يعدّل سجل غيره بتغيير المعرّف في الطلب. خطورة متوسطة (1): 3. `src/lib/sync.ts:23` — استعلام داخل حلقة على قائمة غير محدودة. لم تُذكر: 14 ملاحظة تنسيق يغطيها المنسّق الآلي. يحتاج سياقاً: `src/jobs/run.ts:12` — هل تُستدعى من طلب ويب أم من مهمة خلفية؟ الجواب يغيّر حكمي. يحتاج مراجعة بشرية: الملف `migrations/0007.sql` — ترحيل بيانات لا يُراجع آلياً. لا أوافق ولا أرفض الدمج؛ القرار لمراجع بشري.
نصائح للاستخدام
- امنع الملاحظات الأسلوبية التي يغطيها المنسّق؛ إغراق المراجعة بها يجعل الفريق يتجاهلها كلها.
- اطلب حالة فشل محددة؛ «قد يسبب مشكلة» ملاحظة لا تُنفّذ ولا تُناقش.
- اجعل قرار الدمج بشرياً؛ المراجعة الآلية مساعد لا بوابة.
النماذج الموصى بها
Claude
أسئلة شائعة
- هل تغني عن المراجعة البشرية؟
- لا؛ تمسك الأنماط المعروفة وتوفّر وقت المراجع، وقرارات التصميم ومعرفة النظام والترحيلات تبقى بشرية.
- لماذا لا توافق على الدمج؟
- لأن الموافقة الآلية تُنشئ ثقة زائفة؛ والوسم بالخطورة يعطي المراجع البشري ترتيب انتباهه ويحفظ مسؤوليته.