/senior-review · Workflow

Senior Review

Une seconde paire d'yeux senior, qui prouve chaque bug qu'elle annonce.

FR EN

« /senior-review --deep avant le merge de ma branche feat/auth-refresh »

Le skill assemble d'abord le contexte : sites d'appel inter-fichiers, conventions du projet, historique git, ticket lié. Des relecteurs aveugles tournent en parallèle, chacun différent de l'auteur. Chaque finding critique exige une preuve concrète (grep, exécution, test rouge) et un panel adversarial tente de le réfuter. Le rapport final plafonne les broutilles et se tait si rien n'est matériel.

A.

Ce que ça fait

Pipeline ancré dans la recherche : assemblage du contexte d'abord (sites d'appel inter-fichiers, conventions, historique git, ticket), gate d'outils objectif, relecteurs aveugles en parallèle (relecteur différent de l'auteur), gate de vérification où chaque finding critique exige une preuve (grep, exécution, ou test rouge), puis un panel adversarial qui tente de réfuter les findings critiques ou incertains.

B.

Quand l'utiliser

Avant un merge, sur un working tree, du staged, une branche ou une PR. Optimise le rapport signal/bruit plutôt que le rappel : plafonne les broutilles, dit explicitement ce qu'il ne faut PAS signaler, se tait quand rien n'est matériel. Tiers --quick / standard / --deep. Pas pour construire une feature ni pour du lint pur.

Le parti pris

Le parti pris : une revue qui liste 40 remarques ne se lit pas. Celle-ci optimise le signal sur le bruit : chaque finding critique exige une preuve (grep, exécution, test rouge), un panel adversarial tente de le réfuter, et elle se tait quand rien n'est matériel.

Installer ce skill

claude code
$ /plugin marketplace add ohugonnot/claude-skills
$ /plugin install senior-review@web-developpeur-skills
#code-review#security#quality#fresh-eyes#pr#audit

Questions fréquentes

Quelle différence entre --quick, standard et --deep ?

Les trois tiers ajustent la profondeur d'analyse et le nombre de relecteurs aveugles. --quick convient à une PR courte avec peu de risque, --deep à du code critique ou une grosse surface de changement.

Pourquoi chaque finding critique exige-t-il une preuve ?

Pour éliminer les faux positifs. Un finding sans preuve (grep, test rouge, exécution) est rejeté par le gate de vérification, ce qui évite de noyer les vrais problèmes dans du bruit.

Puis-je l'utiliser sur du staged ou un working tree, pas seulement une PR ?

Oui. Le skill accepte un working tree, du staged, une branche ou une PR existante comme point d'entrée.

Voir le code du skill : SKILL.md sur GitHub. Le skill se déclenche aussi automatiquement quand ta demande correspond à sa description, et un agent IA le découvre via le serveur MCP. Pour comprendre comment ces skills sont conçus, lis les patterns des skills officiels.

Retours & discussion · 0

Aucun retour pour l'instant. Tu l'as essayé ? Dis ce que tu en penses.