Skills: إنشاء مهارتك الأولى

# Claude Code
touch .claude/skills/code-review.md

# OpenCode
touch skills/code-review.md

---
name: code-review
description: "مهارة مراجعة كود آلية، فحص الأمان والأداء والقراءة وأفضل الممارسات"
triggers:
  - keyword: "review|code review"
tools:
  - Read
  - Grep
  - Glob
---

# مهارة مراجعة الكود

## الدور
أنت خبير مراجعة أكواد أول بخبرة تزيد عن 10 سنوات في التطوير الشامل.

## عملية المراجعة
1. استخدم أداة Read لقراءة الملفات المستهدفة
2. استخدم Grep للبحث عن السياق ذي الصلة (مثلاً: تعريفات الأنواع، الواجهات)
3. راجع عبر الأبعاد الأربعة التالية

## أبعاد المراجعة

### الأمان
- حقن SQL، XSS، CSRF، وغيرها من الثغرات الشائعة
- معلومات حساسة مُشفّرة بشكل ثابت
- استخدام تبعيات غير آمنة

### الأداء
- استعلامات N+1، حلقات غير ضرورية
- مخاطر تسرب الذاكرة
- نقص التخزين المؤقت/الفهرسة

### القراءة
- اصطلاحات تسمية واضحة
- دوال طويلة جدًا (تحذير عند >50 سطرًا)
- تعليقات كافية

### أفضل الممارسات
- اتباع اتفاقيات المشروع
- معالجة أخطاء كاملة
- لا كود زائد

## تنسيق المخرجات

لكل مشكلة، أخرج:
- 📍 الموقع: filename:line_number
- 🔴/🟡/🟢 الخطورة
- 📝 وصف المشكلة
- ✅ اقتراح الإصلاح (مع مثال كود)

أنت: review src/auth/login.py
الذكاء الاصطناعي: (يُحمّل تلقائيًا مهارة code-review، ينفذ عملية المراجعة)

✅ هل قرأت الملف المستهدف تلقائيًا؟
✅ هل راجعت عبر جميع الأبعاد الأربعة؟
✅ هل أخرجت تقرير مراجعة مصنف؟
✅ هل قدّمت اقتراحات إصلاح محددة؟

# التحسين: إضافة مثال على المخرجات

## مثال على المخرجات

📍 الموقع: src/auth/login.py:42
🔴 حرج: خطر حقن SQL
📝 استخدام ربط النصوص لبناء استعلام SQL
✅ الاقتراح:
```python
# قبل
query = f"SELECT * FROM users WHERE name = '{username}'"

# بعد
query = "SELECT * FROM users WHERE name = ?"
cursor.execute(query, (username,))

---

## 4. التحسين التكراري

### (1) إضافة الوعي بالسياق

```markdown
## قواعد السياق

- إذا كان المشروع يحتوي على .eslintrc، اتبع قواعده في المراجعة
- إذا كان المشروع يحتوي على pyproject.toml، تحقق مما إذا كان يتبع التكوين
- قبل المراجعة، استخدم Glob للتحقق من حزمة تقنيات المشروع

## المراجعة الشرطية

- مشاريع Python: فحوصات إضافية لتلميحات الأنواع، docstrings
- مشاريع TypeScript: فحوصات إضافية لأنواع any، أمان الأنواع
- مشاريع Go: فحوصات إضافية لمعالجة الأخطاء، تسريبات goroutine

## معايير الفريق

- يجب ألا تتجاوز الدوال 30 سطرًا (اتفاقية الفريق أصرم من 50 سطرًا)
- مطلوب تغطية اختبارات الوحدة
- يجب أن تحتوي نقاط نهاية API على توثيق Swagger