Pr Review

---
description: "Revue systématique de PR. Se déclenche avec « réviser cette PR », « code review »."
allowed-tools:
  - Bash(gh *)
  - Read
  - Grep
  - Glob
---

# Revue systématique de PR

Assure la qualité du code et la solidité architecturale grâce à des revues systématiques de Pull Requests.

## Utilisation

```bash
# Revue complète de PR
gh pr view 123 --comments
"Systematically review this PR and provide feedback from code quality, security, and architecture perspectives"

# Revue axée sur la sécurité
gh pr diff 123
"Focus on reviewing security risks and vulnerabilities"

# Revue de perspective architecturale
gh pr checkout 123 && find . -name "*.js" | head -10
"Evaluate the architecture from the perspectives of layer separation, dependencies, and SOLID principles"
```

## Exemples de base

```bash
# Évaluation quantitative de la qualité du code
find . -name "*.js" -exec wc -l {} + | sort -rn | head -5
"Evaluate code complexity, function size, and duplication, and point out improvements"

# Vérification des vulnérabilités de sécurité
grep -r "password\|secret\|token" . --include="*.js" | head -10
"Check for risks of sensitive information leakage, hardcoding, and authentication bypass"

# Détection de violations architecturales
grep -r "import.*from.*\.\./\.\." . --include="*.js"
"Evaluate layer violations, circular dependencies, and coupling issues"
```

## Système de classification des commentaires

```text
🔴 critical.must : Questions critiques
├─ Vulnérabilités de sécurité
├─ Problèmes d'intégrité des données
└─ Risques de panne système

🟡 high.imo : Améliorations haute priorité
├─ Risque de dysfonctionnement
├─ Problèmes de performance
└─ Diminution significative de la maintenabilité

🟢 medium.imo : Améliorations priorité moyenne
├─ Amélioration de la lisibilité
├─ Amélioration de la structure du code
└─ Amélioration de la qualité des tests

🟢 low.nits : Points mineurs
├─ Unification du style
├─ Corrections de typos
└─ Ajout de commentaires

🔵 info.q : Questions/informations
├─ Confirmation d'intention d'implémentation
├─ Contexte des décisions de conception
└─ Partage de bonnes pratiques
```

## Perspectives de revue

### 1. Correction du code

- **Erreurs logiques** : Valeurs limites, vérifications null, gestion des exceptions
- **Intégrité des données** : Sécurité de type, validation
- **Gestion d'erreurs** : Complétude, traitement approprié

### 2. Sécurité

- **Authentification/autorisation** : Vérifications appropriées, gestion des permissions
- **Validation d'entrée** : Contremesures injection SQL, XSS
- **Informations sensibles** : Restrictions de journalisation, chiffrement

### 3. Performance

- **Algorithmes** : Complexité temporelle, efficacité mémoire
- **Base de données** : Requêtes N+1, optimisation d'index
- **Ressources** : Fuites mémoire, utilisation du cache

### 4. Architecture

- **Séparation des couches** : Direction des dépendances, séparation appropriée
- **Couplage** : Couplage serré, utilisation d'interfaces
- **Principes SOLID** : Responsabilité unique, ouvert-fermé, inversion de dépendances

## Flux de revue

1. **Pré-vérification** : Informations PR, diff des changements, issues liées
2. **Vérifications systématiques** : Sécurité → Correction → Performance → Architecture
3. **Feedback constructif** : Suggestions d'amélioration spécifiques et exemples de code
4. **Suivi** : Confirmation des corrections, statut CI, approbation finale

## Exemples de commentaires efficaces

### Problèmes de sécurité

**Format :**

```text
**critical.must.** [Description du problème de sécurité]

[Code ou solution proposée]

[Explication de la nécessité]
```

**Exemple :**

```text
**critical.must.** Le mot de passe est stocké en texte brut

// Correction proposée
const bcrypt = require('bcrypt');
const hashedPassword = await bcrypt.hash(password, 12);

Le hachage est requis pour prévenir les risques de sécurité.
```

### Amélioration des performances

**Format :**

```text
**high.imo.** [Description du problème de performance]

[Code d'amélioration proposé]

[Explication de l'impact]
```

**Exemple :**

```text
**high.imo.** Problème de requête N+1 survient

// Amélioration : Chargement anticipé
const users = await User.findAll({ include: [Post] });

Ceci peut réduire significativement le nombre de requêtes.
```

### Violation architecturale

**Format :**

```text
**high.must.** [Description de la violation]

[Explication et solution recommandée]
```

**Exemple :**

```text
**high.must.** Violation de couche survenue

La couche domaine dépend directement de la couche infrastructure.
Veuillez introduire une interface suivant le principe d'inversion de dépendances.
```

## Notes

- **Ton constructif** : Communication collaborative plutôt qu'agressive
- **Suggestions spécifiques** : Fournir des solutions en plus de signaler les problèmes
- **Priorisation** : Traiter dans l'ordre Critique → Haute → Moyenne → Basse
- **Amélioration continue** : Documenter les résultats de revue dans une base de connaissances