monty-code-review
Révision de code Django avec vérifications strictes de correction
Les revues de code manquent souvent des problèmes subtils de correction, en particulier concernant la sécurité multi-locataire et l'intégrité des données. Cette compétence fournit un cadre de révision structuré et hyper-pointilleux qui détecte les bugs de correction, les violations SOLID et les risques de sécurité avant la mise en production.
تنزيل ZIP المهارة
رفع في Claude
اذهب إلى Settings → Capabilities → Skills → Upload skill
فعّل وابدأ الاستخدام
اختبرها
استخدام "monty-code-review". Review this PR with monty-code-review
النتيجة المتوقعة:
- What is great: Good use of transaction.atomic around export generation
- BLOCKING: dashboardapp/views/v2/report.py - queryset missing organization scoping, risks cross-tenant leakage
- SHOULD_FIX: pulse_iq/tasks.py - potential N+1 when iterating over responses
- NIT: utils/date_helpers.py - docstring could clarify quarter definition, missing newline at EOF
- Tests: Missing coverage for multi-tenant edge cases
- Verdict: Request changes due to blocking multi-tenant issue
استخدام "monty-code-review". Check this migration for safety
النتيجة المتوقعة:
- What is great: Migration uses non-atomic operation for large table changes
- BLOCKING: Adding non-nullable column with default will lock the entire production table
- SHOULD_FIX: Consider adding column nullable first, then backfilling in batches
- Verdict: Request changes before merge
استخدام "monty-code-review". Review for SOLID violations
النتيجة المتوقعة:
- What is great: Clear separation between boundary and service layer
- BLOCKING: send_reminders directly instantiates ReminderService - violates DIP
- SHOULD_FIX: Accept injectable factory to enable clean testing without @patch
- Verdict: Request changes for testability
التدقيق الأمني
آمنPure prompt-based documentation skill containing only markdown files that instruct AI assistants how to perform code reviews. No executable code, no network calls, no credential access. All static findings are FALSE POSITIVES triggered by documentation patterns (backticks for code formatting, legitimate code review terminology, and standard metadata). The skill operates entirely within the AI assistant's context and does not interact with external systems.
عوامل الخطر
🌐 الوصول إلى الشبكة (1)
⚙️ الأوامر الخارجية (168)
درجة الجودة
ماذا يمكنك بناءه
Révision de PR pour applications Django
Réviser les PR pour les apps dashboard, survey ou optimo avec focus sur l'intégrité des données et la sécurité multi-locataire
Standards de qualité de code
Appliquer des standards de qualité de code cohérents dans l'équipe avec des retours structurés et tagués par sévérité
Vérifications de sécurité des migrations
Valider les migrations Django pour la sécurité des temps d'arrêt et les risques de changements de schéma sur les grandes tables
جرّب هذه الموجهات
Use your monty-code-review skill to review this Django PR like Monty would, and be fully pedantic with your usual severity tags
Use your monty-code-review skill to skim this PR and only flag blocking or should-fix issues; skip most nits
Use monty-code-review to check this Django migration for downtime risks and large-table column additions
Review this code for SOLID principle violations, specifically Dependency Inversion and Single Responsibility
أفضل الممارسات
- Fournir des descriptions de PR et du contexte de ticket pour des révisions plus ciblées
- Utiliser les tags de sévérité pour prioriser efficacement les corrections
- Appliquer les vérifications SOLID tôt lors de la conception de nouveaux services
- Exécuter la compétence sur les migrations avant le déploiement en production
تجنب
- Sauter les révisions pour les petites PR afin d'éviter les bugs de correction
- Fusionner du code avec des problèmes multi-locataires BLOCKING
- Ajouter le support de nouvelles plateformes via des chaînes if/elif au lieu de patterns de registre
- Utiliser un couplage fort qui nécessite des appels @patch extensifs dans les tests