المهارات receiving-code-review
📦

receiving-code-review

آمن

التعامل مع ملاحظات مراجعة الكود بدقة تقنية

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

مراجعة الكود تتطلب تقييماً تقنياً، وليس أداءً عاطفياً. توجه هذه المهارة المساعدين الذكيين للتحقق من الملاحظات قبل التنفيذ، وطرح أسئلة توضيحية، وتقديم اعتراضات مبررة عندما تكون الاقتراحات غير سليمة تقنياً.

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

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

2

رفع في Claude

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

3

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

اختبرها

استخدام "receiving-code-review". المراجع: أزل مسار الكود القديم لإصدارات API القديمة

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

التحقق من متطلبات التوافق... هدف البناء هو 10.15+، لكن هذا API يحتاج 13+. المسار القديم لا يزال مطلوباً للتوافق العكسي. لكن، التنفيذ الحالي يحتوي على bundle ID خاطئ. يمكنني إصلاح bundle ID أو يمكننا إسقاط دعم ما قبل 13 بالكامل. أي اتجاه تفضل؟

استخدام "receiving-code-review". المراجع: أصلح العناصر 1-6 في الـ PR

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

أفهم العناصر 1 و 2 و 3 و 6. أحتاج لتوضيح حول العناصر 4 و 5 قبل التنفيذ. هل يمكنك التفصيل حول التغييرات المطلوبة في هذين العنصرين؟

استخدام "receiving-code-review". المراجع: أضف معالجة أخطاء مناسبة مع منطق إعادة المحاولة والتراجع الأسي

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

تم الإصلاح. أُضيف Try/Catch مع منطق إعادة المحاولة (3 محاولات، تراجع أسي بدءاً من 100ms) لاستدعاء API في api_service.rb:42-58. الاختبارات نجحت.

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

آمن
v1 • 2/24/2026

Static analysis flagged 24 instances of backtick patterns and 4 blocker patterns, but all are FALSE POSITIVES. The skill file (SKILL.md) is documentation-only markdown containing instructional text for AI behavior. No executable code, network calls, file operations, or cryptographic functions exist. The backtick patterns are markdown code blocks showing workflow examples, not shell execution. Blocker patterns are misidentified text fragments in documentation.

1
الملفات التي تم فحصها
219
الأسطر التي تم تحليلها
0
النتائج
1
إجمالي عمليات التدقيق
لا توجد مشكلات أمنية
تم تدقيقه بواسطة: claude

درجة الجودة

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

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

مساعد ذكي يستقبل مراجعة بشرية

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

مطور مبتدئ يتعلم الاستجابة للمراجعة

يتعلم مطور مبتدئ كيفية الاستجابة لملاحظات مطور أقدم بدقة تقنية بدلاً من الموافقة الأدائية، مع التحقق من الاقتراحات قبل التنفيذ الأعمى.

مساهم خارجي يتعامل مع ملاحظات المشرفين

يستقبل مساهم خارجي ملاحظات من مشرفي المشروع ويحتاج لتقييم ما إذا كانت الاقتراحات مناسبة لبنية المشروع قبل التنفيذ.

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

طلب توضيح أساسي 
I received code review feedback with 6 items. I understand items 1, 2, 3, and 6, but need clarification on items 4 and 5 before proceeding. Can you explain what changes you're looking for on those specific items?
اعتراض تقني مع أدلة 
Your suggestion to remove the legacy compatibility layer would break support for macOS 10.15. The current implementation targets 10.15+ but uses APIs that need 13+. Should I fix the bundle ID to maintain backward compatibility, or drop pre-13 support entirely?
فحص YAGNI قبل التنفيذ 
The reviewer suggested implementing full metrics tracking with database storage, date filters, and CSV export. I searched the codebase and found no callers for this endpoint. Should I remove it following YAGNI, or is there usage I'm missing?
تصحيح لطيف بعد اعتراض خاطئ 
I pushed back on your suggestion, but after checking the test results you were correct. My understanding was wrong because I missed the integration test failure. Implementing the fix now.

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

  • تحقق دائماً من الملاحظات مقابل قاعدة الكود الفعلية قبل تنفيذ أي تغييرات
  • نفذ التغييرات واحداً تلو الآخر واختبر كل منها بشكل فردي لاكتشاف التراجعات مبكراً
  • اعتراض بمنطق تقني عندما تكون الاقتراحات ستكسر الوظائف الموجودة

تجنب

  • قول 'أنت محق تماماً!' أو 'نقطة رائعة!' قبل التحقق من الملاحظات
  • تنفيذ الملاحظات بشكل أعمى دون التحقق مما إذا كانت تكسر الوظائف الموجودة
  • تنفيذ العناصر 1-3 و 6 مع تخطي العناصر غير الواضحة 4-5 دون السؤال أولاً

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

ماذا يجب أن أفعل إذا كانت ملاحظات مراجعة الكود غير واضحة؟
توقف واطلب التوضيح قبل تنفيذ أي شيء. قد تكون العناصر مرتبطة، والفهم الجزئي يؤدي لتنفيذ خاطئ. وضح بوضوح أي العناصر تفهمها وأيها يحتاج لتوضيح.
متى يكون الاعتراض على ملاحظات المراجع مناسباً؟
اعتراض عندما يكسر الاقتراح وظائف موجودة، أو يفتقر المراجع للسياق الكامل، أو ينتهك YAGNI لميزات غير مستخدمة، أو غير صحيح تقنياً لمكدسك، أو يتعارض مع قرارات معمارية.
كيف أعتراض دون أن أبدو دفاعياً؟
استخدم المنطق التقني، اطرح أسئلة محددة، أشر لاختبارات عاملة أو كود موجود، وأشرك شريكك البشري إذا كانت القضية معمارية. اذكر الحقائق، وليس العواطف.
ماذا لو اعتترض وكنت لاحقاً مخطئاً؟
اعترف بالتصحيح واقعياً: 'كنت محقاً - تحققت من X وهو يفعل Y. سأنفذ الآن.' تجنب الاعتذارات الطويلة أو الدفاع عن سبب اعتراضك مبدئياً.
لماذا يجب تجنب قول الشكر أو الأداء الأدائي؟
الأفعال أبلغ من الأقوال. فقط أصلح المشكلة وأظهر التغيير في الكود. الموافقة الأدائية تضيع الوقت وقد تخفي مخاوف تقنية فعلية تحتاج للمعالجة.
كيف أتعامل مع ملاحظات المراجعين الخارجيين بشكل مختلف عن شريكي البشري؟
كن أكثر تشككاً مع المراجعين الخارجيين. تحقق مما إذا كانت الاقتراحات صحيحة تقنياً لقاعدة كودك، ولن تكسر الوظائف الموجودة، وأن المراجع يفهم السياق الكامل. اعترض بالمنطق عند الحاجة.

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

المؤلف

sickn33

الترخيص

MIT

المستودع

https://github.com/sickn33/antigravity-awesome-skills/tree/main/skills/receiving-code-review

مرجع

main

بنية الملفات

📄 SKILL.md