@majiayu000/fix-review-code
BCorriger automatiquement les problemes identifies par /code-review en se basant sur les regles et conventions evan-workflow.
Install
agr install @majiayu000/fix-review-code --target claudeWrites 2 files into .claude/skills/, pinned to git-8f911cb7.
- .claude/skills/fix-review-code/SKILL.md
- .claude/skills/fix-review-code/metadata.json
Document
name: fix-review-code description: Corriger automatiquement les problemes identifies par /code-review en se basant sur les regles et conventions evan-workflow.
Fix Review Code
Objectif
Corriger automatiquement les problemes identifies par /code-review. Peut travailler a partir d'un rapport existant ou lancer sa propre review en interne avant de corriger.
Input
- Rapport existant :
/fix-review-code path/to/review-report.mdou le rapport est dans le contexte de conversation - Memes parametres que /code-review :
/fix-review-code --commit abc123,/fix-review-code --branch feature/xxx, etc. (lance une review interne puis corrige)
Parametres
--auto: Mode B — corriger tout ce qui est non-ambigu, skip les ambigus--dry-run: Mode C — montrer ce qui serait corrige sans modifier aucun fichier
Sans parametre, le mode par defaut est le Mode A (correction interactive).
Workflow
Etape 1 — Obtenir le rapport
- Si un rapport de code-review est fourni (fichier ou contexte) : le lire et parser les problemes
- Sinon : lancer
/code-reviewen interne avec les parametres fournis pour obtenir le rapport
Etape 2 — Trier les problemes
Trier les problemes par severite, du plus critique au moins critique :
- CRITIQUE
- HAUTE
- MOYENNE
- BASSE
Etape 3 — Corriger chaque probleme
Pour chaque probleme, dans l'ordre de severite :
- Lire le fichier concerne a la ligne indiquee
- Lire la regle evan-workflow applicable pour comprendre la correction attendue
- Determiner la strategie de fix selon
references/fix-strategies.md - Evaluer l'ambiguite :
- Non-ambigu : la correction est evidente et sans risque → appliquer directement
- Ambigu : plusieurs corrections possibles ou risque d'effet de bord → selon le mode :
- Mode A : demander confirmation au user
- Mode B : skip le probleme
- Mode C : mentionner l'ambiguite dans le rapport dry-run
- Appliquer la correction avec l'outil Edit
- Verifier que la correction ne casse pas les imports, les types ou les tests
Etape 4 — Resume des corrections
Produire un resume :
- Nombre de corrections appliquees
- Nombre de problemes skipes (ambigus)
- Liste des fichiers modifies
- Problemes restants a traiter manuellement
Etape 5 — Proposer le commit
- Mode A : proposer un ou plusieurs commits au user
- Mode B : creer le(s) commit(s) automatiquement
- Mode C : aucun commit
Le message de commit doit suivre le scope commit du manifeste evan-workflow (~/evan-workflow/common/git-conventions.md). Si les corrections couvrent plusieurs categories, proposer un commit par categorie (ex: un commit pour le style, un pour les fixes de bugs).
Securite
- Ne corriger que les violations claires. Si une correction est ambigue, demander au user (Mode A) ou skipper (Mode B).
- Ne jamais supprimer de code fonctionnel. Un fix ne doit pas changer le comportement metier.
- Verifier apres chaque correction que les imports et les types restent coherents.
- En cas de doute, ne pas corriger. Mieux vaut laisser un probleme signale que d'introduire une regression.
Modes
| Mode | Comportement | Ambiguite | Commit |
|---|---|---|---|
| A (defaut) | Correction interactive | Demande confirmation | Propose au user |
B (--auto) | Correction automatique | Skip | Commit auto |
C (--dry-run) | Aucune modification | Signale | Aucun |
Repository README
Describes majiayu000/claude-skill-registry-data as a whole, which may contain artifacts other than this one. Where this artifact had no useful description of its own, its summary was taken from here.
Claude Skill Registry (Data)
This repo contains the archived skill contents (the heavy, browsable skill files).
Canonical layout
- Category folders at repo root (e.g.
development/,documents/,data/, ...) - Each skill lives under a category:
<category>/<skill>/SKILL.md+<category>/<skill>/metadata.json - Case conflicts are resolved with
{name}-{owner}-{repo}suffixes (fallback:-{short-hash}).
Archive status
- Live badges above are sourced from
claude-skill-registry-corestats.json. - Counts in this README are intentionally dynamic, not hardcoded.
- If the badges look stale, refresh the
corebuild/index pipeline rather than editing numbers here.
Where the index + site live
- Core repo: https://github.com/majiayu000/claude-skill-registry-core
- Main repo (merged publish artifact): https://github.com/majiayu000/claude-skill-registry
Trustgrade B
- passBody integrity
Whether the stored document is plausibly the kind of file the artifact declares, rather than something fetched by mistake.
- passType matchnot applicable to this artifact type
Whether the artifact is really the kind of thing its metadata claims it is.
- passFreshness
How long since the source repository was last pushed to.
- passPrompt injection
Scans the artifact's own text for instructions aimed at your agent rather than at you.
- warnLicenseno SPDX license detected
Whether the source repository declares an SPDX license permissive enough to redistribute.
How the grade is calculated
Each check contributes 0 points when it passes, 1 when it warns, and 2 when it fails. The total maps to a letter:
- Aevery check passed
- Bone warning
- Ctwo warnings
- Dprompt injection or body integrity failed, or three warnings
- Fone of those failed, and something else is wrong
These are automated hygiene checks, not a security audit, and not a dependency or vulnerability scan. A grade of A means nothing was flagged — not that the artifact is safe.
Versions
git-8f911cb787f72026-07-31