المهارات requesting-code-review
📝

requesting-code-review

آمن ⚙️ الأوامر الخارجية

طلب مراجعة الكود بالذكاء الاصطناعي

متاح أيضًا من: davila7,DMJGilbert,DYAI2025,Cygnusfear,Cycleaddict,CodingCossack,obra

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

يدعم: Claude Codex Code(CC)
🥉 74 برونزي
1

تنزيل ZIP المهارة

2

رفع في Claude

اذهب إلى Settings → Capabilities → Skills → Upload skill

3

فعّل وابدأ الاستخدام

اختبرها

استخدام "requesting-code-review". Request code review for my authentication module changes

النتيجة المتوقعة:

### Strengths
- Clean separation of concerns with auth service (auth.ts:1-50)
- Proper password hashing using bcrypt (hash.ts:15)
- Good test coverage for edge cases

### Issues

#### Important
1. **Missing token expiration check**
- File: auth.ts:45
- Issue: Tokens never expire, creating security risk
- Fix: Add expiration timestamp to JWT payload

#### Minor
1. **Inconsistent error messages**
- File: auth.ts:30-35

### Assessment
**Ready to merge: With fixes**

**Reasoning:** Core authentication logic is solid. Important issue with token expiration must be addressed before production use.

استخدام "requesting-code-review". Review my database migration changes

النتيجة المتوقعة:

### Strengths
- Proper foreign key constraints (migrations/002.sql:20)
- Good rollback strategy

### Issues

#### Critical
1. **No backup before destructive operation**
- File: migrations/003.sql:15
- Issue: DROP TABLE without backup strategy
- Fix: Add data migration script or backup step

### Assessment
**Ready to merge: With fixes**

**Reasoning:** Critical data safety issue must be resolved.

التدقيق الأمني

آمن
v1 • 2/24/2026

All 27 static findings are false positives. The skill uses legitimate git commands (git diff, git rev-parse) for code review - these are standard development operations. The detected 'weak cryptographic algorithm' patterns match the word 'security' in documentation headings, not actual crypto code. The 'system reconnaissance' matches are generic text. This is a safe code review workflow skill with no command injection risk - git commands use hardcoded arguments with template variable placeholders.

2
الملفات التي تم فحصها
253
الأسطر التي تم تحليلها
1
النتائج
1
إجمالي عمليات التدقيق

الأنماط المكتشفة

Git Command Execution (False Positive)Security Keyword Match (False Positive)Generic Text Pattern Match (False Positive)
تم تدقيقه بواسطة: claude

درجة الجودة

38
الهندسة المعمارية
100
قابلية الصيانة
87
المحتوى
50
المجتمع
100
الأمان
91
الامتثال للمواصفات

ماذا يمكنك بناءه

مراجعة الكود قبل الدمج

استخدم قبل دمج فروع الميزات لاكتشاف المشاكل مبكراً. يقوم الوكيل الفرعي بمراجعة جميع التغييرات مقابل المتطلبات ويقدم حكم واضح بالجاهزية/عدم الجاهزية.

تحقق من مهمة الوكيل الفرعي

بعد اكتمال كل مهمة يقوم بها الوكيل الفرعي في التطوير القائم على الوكلاء، وجه مراجع الكود للتحقق من العمل قبل الانتقال إلى المهمة التالية.

عالق في مشكلة

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

جرّب هذه الموجهات

طلب مراجعة كود أساسي
Request code review for the changes I just made. Use the requesting-code-review skill with: WHAT_WAS_IMPLEMENTED: [describe what you built], PLAN_OR_REQUIREMENTS: [link to plan or describe requirements], BASE_SHA: [starting commit], HEAD_SHA: [ending commit], DESCRIPTION: [brief summary]
مراجعة ميزة كاملة
Please review my latest feature implementation using the requesting-code-review skill. Compare against the requirements in docs/plans/[feature-name].md. Include architecture assessment and testing coverage evaluation.
مهمة ما بعد الوكيل الفرعي
Run the code-reviewer subagent to verify the task just completed. Check against the original task requirements and report any Critical or Important issues that need fixing before proceeding.
خط أساسي قبل إعادة الهيكلة
Request a code review of the current state of [component/module] before I refactor it. This will serve as a baseline to ensure the refactoring does not introduce regressions.

أفضل الممارسات

  • قدم متطلبات واضحة أو مرجع للخطة عند طلب المراجعة - يقارن الوكيل الفرعي التنفيذ بهذه
  • راجع بعد كل مهمة كبيرة في التطوير القائم على الوكلاء لاكتشاف المشاكل قبل تكدسها
  • أصلح المشاكل الحرجة المهمة قبل المتابعة؛ عالج المشاكل الثانوية في العمل اللاحق

تجنب

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

الأسئلة المتكررة

كيف تختلف هذه المهارة عن استخدام Claude مباشرة؟
تقوم هذه المهارة بتوجيه وكيل فرعي متخصص في مراجعة الكود مع قالب مراجعة منظم. يوفر مراجعات متسقة وشاملة مع تصنيف الخطورة وتقييم الجاهزية للإنتاج.
هل يمكنني استخدام هذا لأي لغة برمجة؟
نعم. تغطي قائمة التحقق من المراجعة مبادئ جودة الكود والهندسة والاختبار العامة التي تنطبق عبر اللغات. يكيّف الوكيل الفرعي ملاحظاته بناءً على الكود الذي يراجعه.
ماذا لو لم يكن لدي وثيقة متطلبات رسمية؟
قدم أي وثائق موجودة أو وصف التذكرة أو حتى ملخص واضح لما يجب أن تفعله الميزة. كلما زاد السياق الذي تقدمه، كانت المراجعة أفضل.
هل هذا يحل محل مراجعة الكود البشرية؟
لا. هذه مراجعة بمساعدة الذكاء الاصطناعي تكتشف العديد من المشاكل مبكراً. تظل المراجعة البشرية ذات قيمة للسياق واتفاقيات الفريق والمعرفة الخاصة بالمجال.
كيف أحصل على BASE_SHA و HEAD_SHA؟
استخدم أوامر Git: BASE_SHA=$(git rev-parse HEAD~1) أو تأكيد محدد، HEAD_SHA=$(git rev-parse HEAD) للأحدث. أو استخدم origin/main للمقارنة مع الفرع الرئيسي.
هل يمكنني تخصيص معايير المراجعة؟
يمكن تعديل قالب code-reviewer.md لإضافة فحوصات خاصة بالفريق أو إرشادات خاصة أو معايير مراجعة باللغة إضافية.

تفاصيل المطور

بنية الملفات