Aller au contenu

Mise en place de l'agent IA Code Reviewer

Billet #267 : Mise en place de l'agent IA Code Reviewer
Type : Qualité / Gouvernance / Automatisation
Composants concernés : .github/agents/, .specify/scripts/powershell/, src/pr_reviewer/, tests/pr_reviewer/, specs/009-ai-code-reviewer/

1. Contexte

À la suite de ma réflexion sur un SDLC agentique, j'avais acté une décision structurante: conserver Speckit comme système agentique unique pour le développement structurel de cette application. Dans cette continuité, j'ai entrepris de créer un agent IA dédié à la revue de code, conçu selon l'approche Speckit, afin d'intégrer cette étape de qualité de façon naturelle et cohérente dans mon processus actuel de production.

2. Objectif

Mettre en place un agent IA de revue qui aide à décider clairement :

  • soit la Pull Request peut être validée,
  • soit elle doit être corrigée avant fusion,
  • avec une explication lisible et actionnable.

3. Ce qui a été livré

  • Un flux de revue structuré autour de l'agent /speckit.review.
  • Une évaluation systématique sur 5 axes :
    • respect de la constitution projet,
    • risques de sécurité,
    • alignement architecture,
    • couverture de tests,
    • respect du périmètre planifié.
  • Une publication automatique d'un avis GitHub en sortie :
    • validation (APPROVE) quand le niveau de risque est acceptable,
    • blocage motivé quand un écart critique est détecté.
  • Un mécanisme de remplacement de l'avis précédent sur la même PR pour garder une décision finale claire.

4. Impact business

  • Moins de risque de régression avant la mise en production.
  • Décision de merge plus fiable avec un standard unique.
  • Gain de temps de coordination grâce à un avis déjà structuré.
  • Traçabilité améliorée des décisions de qualité au niveau PR.

5. Validation et statut

  • Les modules de revue et de publication ont été testés localement.
  • Le comportement de base est validé (classification, format de rapport, publication d'avis).
  • Le billet de traçabilité associé est créé : #267.

6. Leçons apprises

  1. Un cadre de décision simple et stable accélère les arbitrages de fusion.
  2. La valeur principale de l'IA ici est la constance de l'analyse, pas la complexité.
  3. La traçabilité (issue + avis PR) est essentielle pour la gouvernance et l'audit.