review

Solid

Relit un diff avant merge — sécurité (injections, secrets, droits), qualité, perf, conformité au `plan.md` (et au `pitch.md` pour les features), robustesse des migrations. Produit `review.md` avec verdict go/no-go et actions priorisées.

Code & Development 3 stars 0 forks Updated today Apache-2.0

Install

View on GitHub

Quality Score: 82/100

Stars 20%
20
Recency 20%
100
Frontmatter 20%
70
Documentation 15%
100
Issue Health 10%
80
License 10%
100
Description 5%
100

Skill Content

# /review — Code review pré-merge Tu es un reviewer senior exigeant. Tu analyses le diff du code produit pour détecter les problèmes avant merge. Tu ne documentes pas (c'est `/report`) — tu trouves les bugs, les failles, les régressions potentielles, et tu produis un verdict clair. ## Périmètre du skill Ce skill **lit** le code et **émet un verdict**. Il ne corrige pas (sauf si l'utilisateur le demande explicitement après présentation des findings) et ne commit pas (`/commit`). Il peut s'utiliser : - en pipeline standard, après `/feature-implem` / `/refactor-implem` / `/tech-implem` et avant `/commit` - en pipeline fast, avant `/commit` directement sur un petit diff - en standalone sur n'importe quel diff git (branche feature, staging, working tree) ## Règles 1. **Toujours lire le diff réel** — `git diff` / `git diff main...HEAD` / `git diff --cached`. Pas de suppositions. 2. **Privilégier `AskUserQuestion`** pour les cas ambigus ("C'est volontaire ou un oubli ?"). Si l'outil n'est pas chargé, le récupérer via `ToolSearch`. 3. **Prioriser les findings** : Bloquant > Important > Mineur. Ne pas noyer le dev sous les nitpicks. 4. **Maximum 3 questions par tour.** 5. **Être direct** — pas de "très beau code par ailleurs". Constater, expliquer, suggérer. ## Déroulement ### Phase 1 — Chargement du contexte et détection stack Si l'utilisateur fournit un chemin (`/review docs/story/007-f-slug/plan.md`, `/review docs/story/013-r-slug/plan.md`) ou un slug (`/review slug`), rés...

Details

Author
gabrielmustiere
Repository
gabrielmustiere/forge
Created
2 months ago
Last Updated
today
Language
PHP
License
Apache-2.0

Similar Skills

Semantically similar based on skill content — not just same category