Automatiser les revues de code avec Tactic Remote

La revue de code est essentielle pour la qualité logicielle, mais elle est aussi chronophage. Les développeurs passent environ 6 à 8 heures par semaine à examiner le code de leurs collègues — du temps qui pourrait être consacré à la construction de fonctionnalités. La revue de code assistée par l'IA ne remplace pas les réviseurs humains, mais elle gère les aspects mécaniques (détecter les bugs, les violations de style, la gestion d'erreurs manquante) pour que les réviseurs humains puissent se concentrer sur l'architecture, le design et la logique métier.

Ce tutoriel vous montre comment mettre en place un workflow de revue de code assisté par l'IA avec Tactic Remote.

Le workflow de revue

Voici le workflow que nous allons mettre en place :

1. Un développeur termine le travail sur une branche de fonctionnalité.
2. Avant de créer un pull request, il lance une session de revue Claude Code depuis son téléphone.
3. Claude Code analyse le diff par rapport à la branche principale et génère une revue complète.
4. Le développeur traite les résultats depuis son téléphone (approuve les corrections ou note les exceptions).
5. Le code nettoyé est poussé et un PR est créé avec le résumé de la revue IA inclus.

Cela capture 60 à 80 % des problèmes qui seraient autrement trouvés lors de la revue humaine, réduisant considérablement le temps du cycle de revue.

Étape 1 : Préparer votre prompt de revue

Créez un fichier nommé .claude-review-prompt.md à la racine de votre projet. Il sert de jeu d'instructions pour les revues IA :

# Code Review Instructions
 
Review all changes on the current branch compared to the `main` branch.
 
## What to Check
 
### Critical Issues (must fix before merge)
 
- **Logic errors**: Code that doesn't do what it claims
- **Security vulnerabilities**: SQL injection, XSS, auth bypasses, credential exposure
- **Data loss risks**: Missing transactions, incorrect cascade deletes, race conditions
- **Breaking changes**: API contract violations, backwards-incompatible modifications
 
### Warnings (should fix, but not blocking)
 
- **Missing error handling**: Uncaught exceptions, unhandled promise rejections
- **Performance concerns**: N+1 queries, unnecessary re-renders, missing memoization
- **Missing validation**: User input not validated before processing
- **Test coverage gaps**: New code paths without corresponding tests
 
### Suggestions (nice to have)
 
- **Readability improvements**: Unclear variable names, complex logic that could be simplified
- **Documentation gaps**: Public APIs without JSDoc/TSDoc comments
- **Potential refactoring**: Duplicated logic, overly long functions
 
## Output Format
 
For each finding, provide:
 
1. **Severity**: CRITICAL / WARNING / SUGGESTION
2. **File**: Exact file path and line numbers
3. **Description**: What the issue is, in one sentence
4. **Explanation**: Why it matters, with context
5. **Fix**: Concrete code suggestion to resolve the issue
 
At the end, provide:
 
- Summary: X critical, Y warnings, Z suggestions
- Overall assessment: Is this branch ready for human review?
 
## What NOT to Flag
 
- Style preferences covered by our linter (ESLint/Prettier handle this)
- Import ordering (automated)
- Whitespace or formatting (automated)
- Type annotations that TypeScript can infer

Ce prompt est délibérément spécifique. Des instructions de revue vagues produisent des revues vagues. Des instructions spécifiques produisent des résultats exploitables.

Étape 2 : Exécuter la revue depuis votre téléphone

Quand vous êtes prêt à réviser une branche, ouvrez Tactic Remote et lancez une session pointant vers le répertoire du projet. Envoyez :

Checkout branch feature/user-dashboard.
Follow the review instructions in .claude-review-prompt.md.

Claude Code va :

1. Faire un checkout de la branche
2. Calculer le diff par rapport à main
3. Analyser chaque fichier modifié selon vos critères de revue
4. Produire un rapport de revue structuré

Cela prend généralement 3 à 8 minutes selon la taille du diff. Vous recevrez une notification quand c'est terminé.

Étape 3 : Examiner les résultats

La sortie de la revue apparaît dans votre vue terminal. Voici à quoi ressemble un résultat typique :

🔴 CRITICAL — src/api/routes/users.ts:47-52

Description: SQL query constructed with string concatenation, vulnerable to injection.

Explanation: The search query parameter is interpolated directly into the SQL
string without parameterization. An attacker could craft a search input that
modifies the query logic, potentially extracting or modifying data.

Fix: Use parameterized query:
  // Before (vulnerable)
  const results = db.query(`SELECT * FROM users WHERE name LIKE '%${search}%'`)

  // After (safe)
  const results = db.query('SELECT * FROM users WHERE name LIKE ?', [`%${search}%`])

Pour chaque résultat, vous avez trois options depuis votre téléphone :

Appliquer la correction : Demandez à Claude Code d'implémenter la correction suggérée :

Apply the fix for the SQL injection issue in users.ts:47.

Ignorer : Si vous n'êtes pas d'accord avec le résultat ou s'il s'agit d'un faux positif, notez-le pour plus tard et continuez.

Discuter : Si vous n'êtes pas sûr, demandez à Claude Code plus de contexte :

Explain the SQL injection risk in more detail. Is this exploitable given
our authentication middleware that runs before this route?

Étape 4 : Application automatisée des corrections

Pour les revues avec de nombreux résultats, vous pouvez appliquer les corrections par lot :

Apply fixes for all CRITICAL and WARNING findings from the review.
For each fix, run the relevant tests to verify the fix doesn't break anything.
Report which fixes were applied and which tests passed.

Claude Code travaillera les résultats systématiquement, appliquant les corrections et exécutant les tests. C'est une autre tâche où vous pouvez poser votre téléphone et attendre la notification.

Important : Vérifiez les corrections appliquées avant de committer. Les corrections générées par l'IA sont généralement correctes, mais « généralement » ne suffit pas pour du code de production. Utilisez la vue terminal pour vérifier le diff :

Show me the complete git diff of all changes made during this review.

Étape 5 : Générer le résumé du PR

Une fois les corrections appliquées et vérifiées, générez un résumé prêt pour le PR :

Create a pull request description that includes:
1. What this branch does (based on the commit history)
2. Summary of the AI review findings and how they were addressed
3. Any remaining suggestions that were intentionally deferred
4. Test results

Incluez ceci dans la description de votre PR. Cela donne confiance aux réviseurs humains que le code a été pré-révisé et leur indique sur quoi se concentrer.

Avancé : Sessions de revue continue

Pour les branches de fonctionnalité de longue durée, exécutez des revues régulièrement plutôt qu'uniquement au moment du PR :

Review all commits since the last review marker.
After completing the review, create a git tag 'last-ai-review' at HEAD.

En effectuant des revues quotidiennes, vous détectez les problèmes plus près du moment où ils ont été introduits. C'est plus facile à corriger que de découvrir des problèmes après une semaine de développement.

Intégration avec CI/CD

Pour les équipes souhaitant formaliser la revue IA, vous pouvez déclencher automatiquement une session de revue quand une branche est poussée. Le workflow :

1. Le webhook CI déclenche l'application compagnon Tactic Remote sur le Mac.
2. L'application compagnon lance une session Claude Code sur la branche mise à jour.
3. Claude Code exécute le prompt de revue.
4. Les résultats sont publiés comme commentaire sur le pull request.

Cela nécessite du scripting côté Mac (l'application compagnon expose une API pour la gestion de sessions) et est détaillé dans notre documentation d'intégration avancée.

Mesure de l'impact

Après avoir adopté la revue de code IA, suivez ces métriques :

• Temps de revue : Devrait diminuer de 30 à 50 % car les réviseurs humains passent moins de temps sur les vérifications mécaniques.
• Problèmes détectés en revue vs. production : Devrait se décaler vers la gauche — plus de problèmes détectés pendant la revue IA, moins atteignent la production.
• Temps du cycle de revue : Le temps entre la création du PR et l'approbation devrait diminuer car les PR arrivent pré-révisés.
• Satisfaction des développeurs : Les réviseurs devraient rapporter moins de monotonie et plus de concentration sur le feedback architectural significatif.

La revue de code IA ne vise pas à remplacer le jugement humain. Elle vise à s'assurer que le jugement humain est consacré aux problèmes qui comptent le plus — architecture, design et logique métier — plutôt qu'à détecter des coquilles et des vérifications null manquantes.

Commencez par un seul projet, affinez votre prompt de revue en fonction des résultats, et élargissez à l'équipe une fois que vous avez calibré l'approche. L'investissement dans un bon prompt de revue porte ses fruits à chaque revue qui l'utilise.