review-pr

Review PR Azure DevOps

Review complète d'une PR via une team de 2 agents : technique + fonctionnel.

Phase 1 — Identification de la PR

Déterminer l'ID de la PR :

1. Si l'utilisateur fournit un ID → l'utiliser directement
2. Sinon, tenter la détection automatique depuis la branche courante :

CURRENT_BRANCH=$(git rev-parse --abbrev-ref HEAD)
az repos pr list --source-branch "$CURRENT_BRANCH" -o json --query "[0].pullRequestId" -o tsv

3. Si aucune PR trouvée → AskUserQuestion : "Quel est l'ID de la Pull Request à reviewer ?"

Phase 2 — Extraction des données

Récupérer en parallèle (3 commandes Bash) :

2.1 Métadonnées de la PR

az repos pr show --id <PR_ID> -o json

Extraire : pullRequestId, title, description, createdBy.displayName, sourceRefName, targetRefName, status.

2.2 Diff de la PR

az repos pr diff --id <PR_ID>

Si le diff dépasse 5000 lignes, informer l'utilisateur et proposer de filtrer les fichiers (exclure tests auto-générés, migrations, lock files).

2.3 Work Items liés

az repos pr work-item list --id <PR_ID> -o json

Pour chaque work item trouvé, récupérer les détails :

az boards work-item show --id <WI_ID> -o json

Extraire par US : id, fields.System.Title, fields.System.Description, fields.Microsoft.VSTS.Common.AcceptanceCriteria, fields.System.State.

2.4 Détection du langage

Analyser les extensions des fichiers modifiés dans le diff pour déterminer le langage et framework. Chercher les fichiers marqueurs dans le repo (composer.json, package.json, pom.xml, go.mod, etc.).

Phase 3 — Présentation et confirmation

Avant de lancer la review, présenter un résumé à l'utilisateur :

PR #{id} — "{title}"
Auteur : {author}
Branche : {source} → {target}
Fichiers modifiés : {count}
US liées : {count} ({liste des IDs et titres})
Langage détecté : {langage}

AskUserQuestion : "Je lance la review technique + fonctionnelle en parallèle. On y va ?" (Oui / Je veux filtrer les fichiers)

Phase 4 — Lancement des agents

Lire references/review-team-composition.md pour les prompt templates détaillés et la configuration des agents.

Lancer les 2 agents en parallèle via Agent :

Agent(
  description: "Revue technique PR #{id}",
  subagent_type: "technical-reviewer",
  prompt: "<prompt template technique avec diff + contexte langage>"
)

Agent(
  description: "Revue fonctionnelle PR #{id}",
  subagent_type: "functional-reviewer",
  prompt: "<prompt template fonctionnel avec diff + US + critères d'acceptance>"
)

Quick reference — Flux

[Extraction données]  (séquentiel)
        │
        ▼
[Confirmation user]
        │
        ▼
┌───────┴───────┐
▼               ▼
technical    functional     (parallèle)
reviewer     reviewer
│               │
└───────┬───────┘
        ▼
[Agrégation rapport]
        │
        ▼
[Post sur PR ?]  (optionnel, avec confirmation)

Phase 5 — Agrégation du rapport

Quand les deux agents ont terminé, produire le rapport unifié :

# Review PR #{id} — "{title}"

## Verdict global

**[APPROVE / REQUEST_CHANGES / NEEDS_DISCUSSION]**

| Aspect | Score | Verdict |
|--------|-------|---------|
| Technique | X/100 | [verdict] |
| Fonctionnel | [Complète/Partielle] | [verdict] |

## Synthèse

[3-5 phrases résumant les conclusions des deux reviews]

---

[Rapport technique complet]

---

[Rapport fonctionnel complet]

Règles de verdict

• APPROVE : Score technique >= 70 ET toutes les US complètes ET aucun critique
• REQUEST_CHANGES : Score technique < 70 OU US incomplètes OU critiques détectés
• NEEDS_DISCUSSION : Ambiguïtés US OU scope creep significatif OU compromis technique nécessaire

Phase 6 — Posting sur la PR (optionnel)

AskUserQuestion : "Veux-tu que je poste ce rapport comme commentaire sur la PR ?" (Oui / Non / Je veux éditer d'abord)

Si oui :

az repos pr comment create --pr-id <PR_ID> --content "<rapport>"

Si "Je veux éditer d'abord" → afficher le rapport brut pour que l'utilisateur copie/modifie.

Invariants

• Read-only : Aucun agent ne modifie le code source — review uniquement
• Parallélisme : Les deux agents sont indépendants et tournent simultanément
• Confirmation avant post : JAMAIS poster sur la PR sans accord explicite de l'utilisateur
• Pas d'US = alerte : Si aucune US n'est liée, le signaler comme problème dans le rapport fonctionnel
• Diff trop gros : Si > 5000 lignes, proposer un filtrage plutôt que de tout envoyer