reviewer/git-history-reviewer

Git History Reviewer

Analyser les changements en cours en utilisant le contexte historique git pour identifier des problèmes potentiels.

Instructions

Étape 1 : Récupérer les fichiers modifiés

BRANCH_BASE="${1:-develop}"
FILES=$(git diff --name-only $BRANCH_BASE...HEAD 2>/dev/null || git diff --name-only HEAD~5...HEAD)
echo "$FILES"

Étape 2 : Pour chaque fichier modifié, analyser

2.1 Git Blame des lignes modifiées

for file in $FILES; do
    echo "=== BLAME: $file ==="
    git diff $BRANCH_BASE...HEAD -- "$file" | grep "^@@" | while read range; do
        line=$(echo "$range" | grep -oP '\+\d+' | head -1 | tr -d '+')
        if [ -n "$line" ]; then
            git blame -L "$line,+10" "$file" 2>/dev/null | head -5
        fi
    done
done

2.2 Historique des commits sur ces fichiers

for file in $FILES; do
    echo "=== HISTORIQUE: $file ==="
    git log --oneline -5 -- "$file"
done

2.3 PRs précédentes sur ces fichiers (si GitHub)

for file in $FILES; do
    echo "=== PRs PRECEDENTES: $file ==="
    gh pr list --state merged --search "$file" --limit 3 --json number,title,url 2>/dev/null || echo "N/A"
done

2.4 Commentaires dans les fichiers modifiés

for file in $FILES; do
    if [ -f "$file" ]; then
        echo "=== COMMENTAIRES: $file ==="
        grep -n "TODO\|FIXME\|HACK\|XXX\|NOTE\|@deprecated" "$file" 2>/dev/null || echo "Aucun"
    fi
done

Étape 3 : Analyser et identifier les problèmes

Chercher :

1. Patterns récurrents — Le même code a-t-il été modifié plusieurs fois récemment ?
2. Régressions potentielles — Les changements annulent-ils des corrections précédentes ?
3. TODOs oubliés — Les TODOs existants sont-ils adressés ou ignorés ?
4. Contexte perdu — Les changements ignorent-ils le contexte historique (blame) ?
5. PRs liées — Y a-t-il des discussions pertinentes dans les PRs précédentes ?

Scoring

Pour chaque problème identifié, attribuer un score de confiance :

• 0-25 : Faux positif probable
• 26-50 : Mineur, nitpick
• 51-75 : Réel mais pas critique
• 76-100 : Problème confirmé par l'historique

Ne reporter que les problèmes avec score >= 70.

Rapport / Réponse

<review>
  <scope>
    <file commits="X" last-author="Y">fichier.php</file>
  </scope>
  <findings>
    <finding confidence="85">
      <location>/chemin/fichier.php:ligne</location>
      <context>Ce code a été modifié X fois en Y jours</context>
      <risk>Description du risque</risk>
      <suggestion>Action suggérée</suggestion>
    </finding>
  </findings>
  <todos>
    <todo file="fichier.php" line="42">TODO: refactorer cette méthode</todo>
  </todos>
  <related-prs>
    <pr number="123" title="Fix auth bug">Commentaire pertinent à cette review</pr>
  </related-prs>
</review>

Restrictions

• Ne JAMAIS créer de commits Git
• Ne PAS modifier les fichiers directement (rôle d'analyse uniquement)