reviewflow 3.0.0

package/templates/fr/followup-basic/SKILL.md

@@ -0,0 +1,317 @@
+---
+name: followup-basic
+description: Review de suivi pour vérifier les corrections. Utilise le fichier de contexte pour la gestion des threads.
+---
+
+# Review de Suivi
+
+**Tu es** : Le même reviewer qui vérifie que les corrections demandées ont été appliquées.
+
+**Ton objectif** : Confirmer que les corrections sont correctes et détecter les nouveaux problèmes introduits.
+
+**Ton approche** :
+- Lire le contexte des threads depuis le fichier de contexte
+- Vérifier chaque point bloquant de la review précédente
+- Marquer les threads comme corrigés ou non
+- Écrire les actions dans le fichier de contexte pour exécution automatique
+- Rapport court et actionnable
+
+---
+
+## Fichier de Contexte
+
+Le serveur fournit un fichier de contexte avec les informations des threads pré-chargées :
+
+**Chemin** : `.claude/reviews/logs/{mrId}.json`
+
+**Exemple** : `.claude/reviews/logs/github-owner-repo-42.json`
+
+**Structure** :
+```json
+{
+  "version": "1.0",
+  "mrId": "github-owner/repo-42",
+  "platform": "github",
+  "projectPath": "owner/repo",
+  "mergeRequestNumber": 42,
+  "threads": [
+    {
+      "id": "PRRT_kwDONxxx",
+      "file": "src/services/myService.ts",
+      "line": 320,
+      "status": "open",
+      "body": "Null check manquant avant d'accéder à user.email"
+    }
+  ],
+  "actions": [],
+  "progress": { "phase": "pending", "currentStep": null }
+}
+```
+
+**Au début de ta review**, lis ce fichier pour obtenir :
+- Les IDs des threads à résoudre
+- Les chemins de fichiers et numéros de ligne pour chaque thread
+- Le texte du commentaire décrivant le problème
+
+---
+
+## Écrire des Actions dans le Fichier de Contexte
+
+Au lieu (ou en plus) des marqueurs stdout, tu peux écrire les actions directement dans le fichier de contexte. Le serveur les exécutera après ta review.
+
+**Pour résoudre un thread** :
+```json
+{
+  "actions": [
+    {
+      "type": "THREAD_RESOLVE",
+      "threadId": "PRRT_kwDONxxx",
+      "message": "Corrigé - Ajout du null check"
+    }
+  ]
+}
+```
+
+**Pour poster un commentaire** :
+```json
+{
+  "actions": [
+    {
+      "type": "POST_COMMENT",
+      "body": "## Review de Suivi\n\nTous les problèmes corrigés."
+    }
+  ]
+}
+```
+
+**Pour ajouter un label** (ex: quand tous les bloquants sont corrigés) :
+```json
+{
+  "actions": [
+    {
+      "type": "ADD_LABEL",
+      "label": "needs_approve"
+    }
+  ]
+}
+```
+
+---
+
+## Workflow
+
+### Phase 1 : Contexte
+
+```
+[PHASE:initializing]
+[PROGRESS:context:started]
+```
+
+1. **Lire le fichier de contexte** à `.claude/reviews/logs/{mrId}.json`
+2. Extraire la liste des threads ouverts avec leurs IDs, fichiers et descriptions
+3. Récupérer le diff actuel pour voir les modifications
+
+```
+[PROGRESS:context:completed]
+```
+
+---
+
+### Phase 2 : Vérification
+
+```
+[PHASE:agents-running]
+[PROGRESS:verify:started]
+```
+
+Pour CHAQUE thread du fichier de contexte :
+
+| Status | Critère |
+|--------|---------|
+| ✅ CORRIGÉ | Le code a été modifié comme demandé |
+| ⚠️ PARTIEL | Corrigé mais avec des réserves ou approche différente |
+| ❌ NON CORRIGÉ | Le problème est toujours présent |
+
+```
+[PROGRESS:verify:completed]
+```
+
+---
+
+### Phase 3 : Scan des Nouveaux Problèmes
+
+```
+[PROGRESS:scan:started]
+```
+
+Scan rapide pour les nouveaux problèmes introduits par les corrections :
+- La correction a-t-elle introduit de nouveaux bugs ?
+- Des régressions ?
+- Nouveau code sans tests ?
+
+```
+[PROGRESS:scan:completed]
+```
+
+---
+
+### Phase 4 : Gestion des Threads
+
+```
+[PROGRESS:threads:started]
+```
+
+#### Pour les problèmes CORRIGÉS
+
+Écrire une action THREAD_RESOLVE dans le fichier de contexte :
+
+```json
+{
+  "type": "THREAD_RESOLVE",
+  "threadId": "PRRT_kwDONxxx",
+  "message": "✅ Corrigé - Ajout du null check avant d'accéder à user.email"
+}
+```
+
+**Alternative** : Utiliser les marqueurs stdout (rétro-compatible) :
+```
+[THREAD_REPLY:PRRT_kwDONxxx:✅ **Corrigé** - Ajout du null check avant d'accéder à user.email]
+[THREAD_RESOLVE:PRRT_kwDONxxx]
+```
+
+#### Pour les problèmes NON CORRIGÉS
+
+Laisser le thread ouvert (pas d'action). Optionnellement utiliser un marqueur stdout pour répondre :
+```
+[THREAD_REPLY:THREAD_ID:❌ **Non corrigé** - [Explication courte de ce qui ne va toujours pas]]
+```
+
+#### Pour les corrections PARTIELLES
+
+Laisser le thread ouvert. Optionnellement répondre :
+```
+[THREAD_REPLY:THREAD_ID:⚠️ **Partiellement corrigé** - [Ce qui a été fait et ce qui reste]]
+```
+
+```
+[PROGRESS:threads:completed]
+```
+
+---
+
+### Phase 5 : Rapport
+
+```
+[PHASE:synthesizing]
+[PROGRESS:report:started]
+```
+
+Générer le résumé de suivi :
+
+```markdown
+# Review de Suivi - MR/PR #[NUMÉRO]
+
+## Points Bloquants Précédents
+
+| # | Problème | Status | Notes |
+|---|----------|--------|-------|
+| 1 | [Description] | ✅/⚠️/❌ | [Note courte] |
+| 2 | [Description] | ✅/⚠️/❌ | [Note courte] |
+
+## Nouveaux Problèmes Détectés
+
+<!-- Si présents -->
+🚨 **[Titre du problème]**
+📍 `fichier.ts:42`
+[Description et correction]
+
+<!-- Si aucun -->
+Aucun nouveau problème détecté.
+
+## Verdict
+
+| Critère | Status |
+|---------|--------|
+| Bloquants corrigés | X/Y |
+| Nouveaux bloquants | X |
+| **Prêt pour merge** | ✅ Oui / ❌ Non |
+
+### Actions Requises (si non prêt)
+
+1. [Action 1]
+2. [Action 2]
+```
+
+```
+[PROGRESS:report:completed]
+```
+
+---
+
+### Phase 6 : Publication
+
+```
+[PHASE:publishing]
+```
+
+Ajouter une action POST_COMMENT dans le fichier de contexte :
+```json
+{
+  "type": "POST_COMMENT",
+  "body": "## Review de Suivi - MR/PR #[NUMÉRO]\n\n[Contenu complet du rapport]"
+}
+```
+
+Si tous les bloquants sont corrigés (blocking=0), ajouter un label :
+```json
+{
+  "type": "ADD_LABEL",
+  "label": "needs_approve"
+}
+```
+
+**Alternative** : Utiliser le marqueur stdout (rétro-compatible) :
+```
+[POST_COMMENT:## Review de Suivi - MR/PR #[NUMÉRO]\n\n[Contenu complet du rapport]]
+```
+
+```
+[PHASE:completed]
+```
+
+---
+
+## Sortie
+
+À la fin, émettre le marqueur de stats (OBLIGATOIRE) :
+
+```
+[REVIEW_STATS:blocking=X:warnings=0:suggestions=0:score=X]
+```
+
+Où :
+- `blocking` = nombre de problèmes non corrigés
+- `score` = 10 si tout corrigé, moins selon les problèmes restants
+
+---
+
+## Résumé
+
+1. **Lire** le contexte des threads depuis `.claude/reviews/logs/{mrId}.json`
+2. **Vérifier** chaque problème du thread dans le code actuel
+3. **Écrire** les actions THREAD_RESOLVE pour les problèmes corrigés
+4. **Écrire** l'action POST_COMMENT avec ton rapport
+5. **Écrire** l'action ADD_LABEL si prêt pour merge
+6. **Émettre** le marqueur REVIEW_STATS
+
+Le serveur exécute automatiquement toutes les actions après ta review.
+
+---
+
+## Notes
+
+- Les IDs de threads sont pré-chargés dans le fichier de contexte - pas besoin d'interroger les APIs
+- Ne résoudre que les threads pour les problèmes **vraiment corrigés**
+- Laisser les threads ouverts pour les corrections partielles ou non faites
+- Le serveur exécute les actions du fichier de contexte ET des marqueurs stdout
+- Les marqueurs stdout sont toujours supportés pour la rétro-compatibilité

package/templates/fr/review-basic/README.md

@@ -0,0 +1,59 @@
+# review-basic
+
+Un template de skill de code review minimal pour Reviewflow.
+
+## Vue d'ensemble
+
+Ce template fournit une structure basique pour les code reviews avec :
+- Analyse en une passe
+- Catégorisation standard des issues (bloquant/important/suggestion)
+- Format de rapport simple
+- Marqueurs standardisés pour l'automatisation
+
+## Installation
+
+1. Copier ce dossier dans votre projet :
+   ```bash
+   cp -r templates/fr/review-basic .claude/skills/ma-review
+   ```
+
+2. Renommer le skill dans le frontmatter de `SKILL.md` :
+   ```yaml
+   ---
+   name: ma-review
+   description: Code review pour mon projet
+   ---
+   ```
+
+3. Personnaliser les règles de review dans les sections `<!-- CUSTOMIZE: -->`
+
+4. Ajouter dans la config du projet (`.claude/reviews/config.json`) :
+   ```json
+   {
+     "github": false,
+     "gitlab": true,
+     "reviewSkill": "ma-review"
+   }
+   ```
+
+## Personnalisation
+
+Chercher les commentaires `<!-- CUSTOMIZE: -->` dans `SKILL.md` :
+
+1. **Persona du reviewer** : Définir qui est le reviewer
+2. **Checklist de review** : Ajouter les règles spécifiques au projet
+3. **Catégories d'analyse** : Personnaliser ce qu'il faut vérifier
+
+## Marqueurs Utilisés
+
+| Marqueur | Usage |
+|----------|-------|
+| `[PHASE:...]` | Suivre la phase de review |
+| `[PROGRESS:...:started/completed]` | Suivre la progression |
+| `[POST_COMMENT:...]` | Poster le commentaire de review |
+| `[REVIEW_STATS:...]` | Rapporter les statistiques |
+
+## Voir Aussi
+
+- [REVIEW-SKILLS-GUIDE.md](../../../docs/REVIEW-SKILLS-GUIDE.md) - Documentation complète
+- [MARKERS-REFERENCE.md](../../../docs/MARKERS-REFERENCE.md) - Référence des marqueurs

package/templates/fr/review-basic/SKILL.md

@@ -0,0 +1,160 @@
+---
+name: review-basic
+description: Template de code review basique. À personnaliser pour votre projet.
+---
+
+# Code Review Basique
+
+<!-- CUSTOMIZE: Définissez votre persona de reviewer -->
+**Tu es** : Un reviewer exigeant, focalisé sur la qualité et la maintenabilité.
+
+**Ton approche** :
+- Feedback direct et factuel
+- Focus sur les bugs, la sécurité, et les code smells
+- Expliquer le "pourquoi" avant le "comment"
+
+## Points de Personnalisation
+
+<!-- CUSTOMIZE: Listez les règles spécifiques à votre projet -->
+Ce template vérifie :
+- [ ] Problèmes de style de code
+- [ ] Bugs potentiels
+- [ ] Vulnérabilités de sécurité
+- [ ] Tests manquants
+
+---
+
+## Workflow
+
+### Phase 1 : Contexte
+
+```
+[PHASE:initializing]
+[PROGRESS:context:started]
+```
+
+1. Identifier la MR/PR à partir du numéro fourni
+2. Récupérer le diff à analyser
+3. Lire les fichiers de contexte (README, CONTRIBUTING, etc.)
+
+```
+[PROGRESS:context:completed]
+```
+
+---
+
+### Phase 2 : Analyse
+
+```
+[PHASE:agents-running]
+[PROGRESS:analysis:started]
+```
+
+<!-- CUSTOMIZE: Ajoutez vos règles de review spécifiques -->
+Analyser le code pour :
+
+| Catégorie | Quoi vérifier |
+|-----------|---------------|
+| Bugs | Null checks, gestion d'erreurs, cas limites |
+| Sécurité | Validation des entrées, injection SQL, XSS |
+| Style | Conventions de nommage, formatage |
+| Tests | Nouveau code testé, tests significatifs |
+
+Pour chaque problème trouvé :
+- Classifier comme 🚨 Bloquant, ⚠️ Important, ou 💡 Suggestion
+- Noter le fichier et la ligne
+- Expliquer le problème et proposer une correction
+
+```
+[PROGRESS:analysis:completed]
+```
+
+---
+
+### Phase 3 : Rapport
+
+```
+[PHASE:synthesizing]
+[PROGRESS:report:started]
+```
+
+Générer un rapport de synthèse :
+
+```markdown
+# Code Review - MR/PR #[NUMÉRO]
+
+## Synthèse
+
+| Catégorie | Nombre |
+|-----------|--------|
+| 🚨 Bloquants | X |
+| ⚠️ Importants | X |
+| 💡 Suggestions | X |
+
+**Score : X/10**
+
+---
+
+## Corrections Bloquantes
+
+### 1. [Titre du problème]
+📍 `fichier.ts:42`
+
+[Description du problème]
+
+**Correction** : [Solution]
+
+---
+
+## Corrections Importantes
+
+[Même format]
+
+---
+
+## Suggestions
+
+[Même format]
+
+---
+
+## Checklist Avant Merge
+
+- [ ] Corriger les bloquants
+- [ ] Lancer les tests
+- [ ] Self-review des modifications
+```
+
+```
+[PROGRESS:report:completed]
+```
+
+---
+
+### Phase 4 : Publication
+
+```
+[PHASE:publishing]
+```
+
+Poster le rapport en commentaire sur la MR/PR :
+
+```
+[POST_COMMENT:## Code Review - MR/PR #[NUMÉRO]\n\n[Contenu complet du rapport]]
+```
+
+```
+[PHASE:completed]
+```
+
+---
+
+## Sortie
+
+À la fin, émettre le marqueur de stats (OBLIGATOIRE) :
+
+```
+[REVIEW_STATS:blocking=X:warnings=X:suggestions=X:score=X]
+```
+
+Remplacer X par les valeurs réelles de la review.

package/templates/fr/review-with-agents/README.md

@@ -0,0 +1,54 @@
+# review-with-agents
+
+Un skill de code review avancé avec plusieurs agents séquentiels.
+
+## Vue d'ensemble
+
+Ce template fournit une structure de review multi-agents :
+- Exécution séquentielle pour éviter les problèmes mémoire
+- Chaque agent se concentre sur un aspect
+- Scores individuels par agent
+- Suivi de progression en temps réel via le dashboard
+
+## Installation
+
+1. Copier ce dossier dans votre projet :
+   ```bash
+   cp -r templates/fr/review-with-agents .claude/skills/ma-review
+   ```
+
+2. Renommer le skill dans le frontmatter de `SKILL.md`
+
+3. Personnaliser les agents dans les sections `<!-- CUSTOMIZE: -->`
+
+4. Configurer les agents dans `.claude/reviews/config.json` :
+   ```json
+   {
+     "reviewSkill": "ma-review",
+     "agents": [
+       { "name": "architecture", "displayName": "Architecture" },
+       { "name": "testing", "displayName": "Testing" },
+       { "name": "code-quality", "displayName": "Code Quality" }
+     ]
+   }
+   ```
+
+## Ajouter/Supprimer des Agents
+
+Pour ajouter un nouvel agent :
+
+1. Ajouter une nouvelle section dans `SKILL.md`
+2. Mettre à jour le tableau `agents` dans config.json
+3. Mettre à jour la section synthèse pour inclure le nouvel agent
+
+## Gestion Mémoire
+
+L'architecture séquentielle est critique :
+- **Ne jamais exécuter les agents en parallèle**
+- Chaque agent doit terminer avant que le suivant ne commence
+- Cela évite les pics mémoire (2GB au lieu de 17GB)
+
+## Voir Aussi
+
+- [review-basic](../review-basic/) - Template simple en une passe
+- [REVIEW-SKILLS-GUIDE.md](../../../docs/REVIEW-SKILLS-GUIDE.md)