requesting-code-review
Demander une revue de code pour le travail terminé
Também disponível em: DMJGilbert,DYAI2025,Cygnusfear,obra,Cycleaddict,CodingCossack
Les problèmes de qualité de code s'accumulent au fil du temps. Cette compétence fournit un flux de travail structuré pour demander des revues de code après avoir terminé les tâches de développement, attrapant les bugs et les améliorations avant qu'ils ne se transforment en problèmes plus importants.
Baixar o ZIP da skill
Upload no Claude
Vá em Configurações → Capacidades → Skills → Upload skill
Ative e comece a usar
Testar
A utilizar "requesting-code-review". Request code review for my authentication module. Base SHA: a7981ec. Head SHA: 3df7661. What was implemented: User login and session management with JWT tokens. Requirements: auth-spec.md in docs folder.
Resultado esperado:
- ### Strengths
- - Clean separation of concerns in auth module (auth.ts:15-45)
- - Proper JWT token handling with expiration (token.ts:22-31)
- - Good test coverage for login flow (auth.test.ts:8-67)
- ### Issues
- #### Important
- 1. Missing rate limiting on login endpoint (auth.ts:89)
- Fix: Add express-rate-limit middleware
- #### Minor
- 1. Error messages reveal whether user exists (auth.ts:95)
- Impact: User enumeration risk in login form
- ### Assessment
- **Ready to merge: With fixes**
- **Reasoning:** Solid architecture with good security practices. Rate limiting should be added before production deployment.
Auditoria de Segurança
SeguroThis skill consists purely of documentation and prompt templates with no executable code. The static scanner misinterpreted shell command examples (shown as documentation to users) and git SHA references as actual security risks. No network calls, file system access, or environment variable reads occur. All 39 flagged findings are false positives.
Fatores de risco
⚙️ Comandos externos (18)
🌐 Acesso à rede (1)
Pontuação de qualidade
O Que Você Pode Construir
Revue après chaque tâche
Attrapez les problèmes immédiatement après avoir terminé les tâches de développement plutôt que d'attendre les cycles de revue ultérieurs
Contrôle qualité avant fusion
Assurez-vous que tout le code répond aux normes de qualité avant de fusionner dans la branche principale via un processus de revue structuré
Valider le code généré par IA
Revoir les implémentations générées par IA pour vérifier qu'elles répondent aux exigences et suivent les bonnes pratiques
Tente Estes Prompts
Request code review for my recent work. Base SHA: [git commit]. Head SHA: [current HEAD]. What was implemented: [brief description]. Requirements reference: [link to plan or requirements].
I just completed a major feature. Request comprehensive code review. Base SHA: [previous main]. Head SHA: [current]. Feature description: [what was built]. Full requirements: [spec document].
Request final code review before merging to main. Base SHA: [origin/main]. Head SHA: [current]. Description: [summary of all changes]. Assessment needed: production readiness, security concerns, and migration requirements.
Request code review before refactoring. Base SHA: [current HEAD]. Head SHA: [same as base - this is baseline]. Document current implementation strengths and weaknesses to establish refactoring baseline.
Melhores Práticas
- Revue tôt et souvent plutôt que d'attendre des grandes PR difficiles à modifier
- Corrigez les problèmes Critiques et Important avant de passer à la tâche suivante
- Utilisez des SHA git spécifiques pour assurer une revue précise du diff
Evitar
- Sauter la revue parce que le code semble simple
- Ignorer les problèmes Critiques juste pour gagner du temps
- Procéder avec des problèmes Important non corrigés qui pourraient causer des problèmes