requesting-code-review
طلب مراجعة الكود بالذكاء الاصطناعي
متاح أيضًا من: davila7,DMJGilbert,DYAI2025,Cygnusfear,Cycleaddict,CodingCossack,obra
غالباً ما يفقد المطورون المشاكل في كودهم الخاص لأنهم قريبون جداً من التنفيذ. تقوم هذه المهارة بتوجيه وكيل فرعي بالذكاء الاصطناعي لمراجعة تغييرات الكود مقابل المتطلبات، مع التحقق من الجودة والهندسة والاختبارات والاستعداد للإنتاج قبل الدمج.
تنزيل ZIP المهارة
رفع في Claude
اذهب إلى Settings → Capabilities → Skills → Upload skill
فعّل وابدأ الاستخدام
اختبرها
استخدام "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.
التدقيق الأمني
آمن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.
عوامل الخطر
⚙️ الأوامر الخارجية (5)
الأنماط المكتشفة
درجة الجودة
ماذا يمكنك بناءه
مراجعة الكود قبل الدمج
استخدم قبل دمج فروع الميزات لاكتشاف المشاكل مبكراً. يقوم الوكيل الفرعي بمراجعة جميع التغييرات مقابل المتطلبات ويقدم حكم واضح بالجاهزية/عدم الجاهزية.
تحقق من مهمة الوكيل الفرعي
بعد اكتمال كل مهمة يقوم بها الوكيل الفرعي في التطوير القائم على الوكلاء، وجه مراجع الكود للتحقق من العمل قبل الانتقال إلى المهمة التالية.
عالق في مشكلة
عند العالق في خطأ أو ميزة معقدة، اطلب مراجعة الكود للحصول على منظور جديد من الوكيل الفرعي بالذكاء الاصطناعي حول المشاكل المحتملة أو الأساليب الأفضل.
جرّب هذه الموجهات
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؟
هل يمكنني تخصيص معايير المراجعة؟
تفاصيل المطور
المؤلف
ZhanlinCuiالترخيص
MIT
المستودع
https://github.com/ZhanlinCui/Ultimate-Agent-Skills-Collection/tree/main/requesting-code-reviewمرجع
main
بنية الملفات