Skills Guide

Revue de code et de pull requests

VérifiéSûr

Compétence pour effectuer des revues de code et de pull requests, en se concentrant sur les problèmes substantiels et en ignorant les détails stylistiques.

Télécharger et utiliser
Spar Skills Guide Bot
DeveloppementIntermédiaire
06/08/2026
Claude CodeCursorCodex
#code-review#pull-request#github#code-quality#review-methodology

Recommandé pour

Développeur backendClaude CodeRevue de codeDéveloppeur frontendCursorDéveloppeur fullstack

Synthèse automatique

Fournit une méthodologie structurée pour relire une pull request GitHub en priorisant les problèmes de fond et en produisant des constats classés par sévérité.

Points forts

  • Ordre de priorité explicite (correction, sécurité, fiabilité, performance, ruptures d'API, couverture de tests) qui évite la dispersion
  • Format de sortie imposé avec sévérité bloquante ou non bloquante, emplacement fichier:ligne et correction suggérée
  • Incite explicitement à lire le code environnant avant de commenter, pour éviter les faux positifs liés aux conventions existantes
  • Écarte volontairement le style et le formatage au profit de ce qui compte

Limites

  • Dépend de la CLI GitHub (`gh`) et suppose un dépôt distant accessible avec les droits adéquats
  • Ne décrit aucun outillage d'exécution des tests ni de vérification automatique : l'analyse reste statique
  • La frontière entre bloquant et non bloquant est laissée à l'appréciation du modèle, sans grille de critères chiffrée
Quand l'utiliser

À utiliser lorsqu'une pull request doit être évaluée avant fusion et que l'on veut un avis hiérarchisé plutôt qu'une liste de remarques de style.

Quand l'éviter

À éviter pour une revue de code local non hébergée sur GitHub ou pour une simple demande de reformatage, que la compétence ignore délibérément.

Analyse de sécurité

Sûr
Score qualité88/100

The skill instructs running GitHub CLI commands (gh pr view, gh pr diff) to fetch pull request data and diffs for review purposes. This is a legitimate use of network access and does not involve destructive actions, exfiltration, obfuscation, or disabling safety. No sensitive data is transmitted outside the normal GitHub context.

Aucun point d'attention détecté

Comment cette analyse est faite

Exemples

Pre-merge PR review
Review PR #482 in acme/api before I merge it. Flag anything blocking and give me a suggested fix for each issue.
Security-focused review
Do a code review of the open pull request that touches the payment retry logic — I care most about correctness and security.
Triage a large diff
Use gh to pull the diff for PR #91 and tell me whether it's ready to merge, separating blocking issues from optional suggestions.

name: code-review description: Use this skill for code review and PR review tasks. Covers review methodology, what to flag, what to skip, and output structure.

Code Review

Review a GitHub pull request for substantive issues. Focus on what matters, skip what doesn't.

Steps

  1. Run gh pr view {pr_number} --json title,body,files to understand the PR's purpose and scope.
  2. Run gh pr diff {pr_number} to read the full diff.
  3. Before commenting, read surrounding code to understand full context — a pattern that looks wrong in isolation may be an established convention.

Focus Areas (priority order)

  1. Correctness — Logic bugs, edge cases, error handling
  2. Security — Input validation, injection risks, secrets exposure
  3. Reliability — Race conditions, failure modes, resource leaks
  4. Performance — Inefficient algorithms, N+1 queries, unnecessary work
  5. Breaking changes — API/behavior changes affecting callers
  6. Test coverage — Are new code paths tested?

Skip style/formatting unless it materially impacts readability. Do not nitpick naming, style preferences, or issues linters catch.

Severity Levels

  • Blocking: Must fix before merge (bugs, security, significant reliability/performance issues, breaking changes)
  • Non-blocking: Suggestions for improvement (optional)

Output Format

Findings

For each issue found:

[Issue Title]

  • Severity: blocking | non-blocking
  • Location: file_path:line_number
  • Description: What the issue is and why it matters
  • Suggested Fix: Concrete code change or approach

If no issues found, state that the changes look correct and explain why.

Summary

Overall assessment: is the PR ready to merge, or does it need changes? List the most important issues if any.

Du fichier au premier essai

Le téléchargement contient exactement le contenu affiché sur cette fiche. Cette copie peut différer de la dernière version du dépôt source.

Consulter la source et ses fichiers annexes

Ce téléchargement contient uniquement SKILL.md. Les scripts, références, modèles et dépendances éventuels ne sont pas inclus. Récupérez les fichiers nécessaires depuis la source en conservant leur arborescence avant de poursuivre.

1. Télécharger le fichier

Télécharger SKILL.md

Depuis la racine d’un projet de test, enregistrez le fichier sous .skill-downloads/code-review-and-pr-review-msh3uscu/SKILL.md, ou utilisez la commande suivante. Ce dossier sert à examiner la skill avant son installation.

Commande pour macOS/Linux ou WSL : un shell POSIX et curl sont nécessaires. Sous Windows sans WSL, utilisez le bouton de téléchargement et créez le dossier manuellement.

mkdir -p '.skill-downloads/code-review-and-pr-review-msh3uscu' && curl --fail --show-error --location 'https://skills-guides.com/api/skills/code-review-and-pr-review-msh3uscu/download' --output '.skill-downloads/code-review-and-pr-review-msh3uscu/SKILL.md.part' && mv '.skill-downloads/code-review-and-pr-review-msh3uscu/SKILL.md.part' '.skill-downloads/code-review-and-pr-review-msh3uscu/SKILL.md'

2. Vérifier les prérequis

Lisez SKILL.md avant de l’utiliser. Si le fichier cite des scripts ou services externes, examinez-les et préparez les accès nécessaires dans votre environnement de test.

Aucune section de prérequis reconnue dans ce fichier. Cela ne signifie pas qu’il est autonome : les bibliothèques, outils et accès peuvent être cités ailleurs dans le contenu ou dans le dépôt source.

Les plateformes de la fiche sont des indications du catalogue. Les chemins ci-dessous suivent la documentation des outils ; ils ne prouvent pas que cette skill y a été exécutée. Aucune version minimale de ces outils n’est attestée pour cette fiche.

3. Charger la skill

Après vérification, copiez le dossier complet dans le projet avec l’une des commandes ci-dessous, puis ouvrez une nouvelle session de l’outil. La commande s’arrête si un dossier du même nom existe déjà : comparez les fichiers avant de le remplacer.

Claude Code
test -f '.skill-downloads/code-review-and-pr-review-msh3uscu/SKILL.md' && test -s '.skill-downloads/code-review-and-pr-review-msh3uscu/SKILL.md' && mkdir -p '.claude/skills' && test ! -e '.claude/skills/code-review' && cp -R '.skill-downloads/code-review-and-pr-review-msh3uscu' '.claude/skills/code-review'

Cursor
test -f '.skill-downloads/code-review-and-pr-review-msh3uscu/SKILL.md' && test -s '.skill-downloads/code-review-and-pr-review-msh3uscu/SKILL.md' && mkdir -p '.cursor/skills' && test ! -e '.cursor/skills/code-review' && cp -R '.skill-downloads/code-review-and-pr-review-msh3uscu' '.cursor/skills/code-review'

Codex
test -f '.skill-downloads/code-review-and-pr-review-msh3uscu/SKILL.md' && test -s '.skill-downloads/code-review-and-pr-review-msh3uscu/SKILL.md' && mkdir -p '.agents/skills' && test ! -e '.agents/skills/code-review' && cp -R '.skill-downloads/code-review-and-pr-review-msh3uscu' '.agents/skills/code-review'

OpenSkills

OpenSkills doit déjà être installé. La procédure locale install/read est contrôlée avec OpenSkills 1.5.0, qui exige Node.js ≥ 20.6.0. Aucun slug du catalogue n’est résolu automatiquement.

Installation officielle d’OpenSkills
openskills install './.skill-downloads/code-review-and-pr-review-msh3uscu'

openskills read 'code-review-and-pr-review-msh3uscu'

La lecture doit afficher le contenu du fichier installé et son dossier de base. Elle ne lance ni la skill ni ses dépendances.

Pour un agent qui lit AGENTS.md, synchronisez ensuite l’index. Cette commande modifie AGENTS.md : relisez le diff avant de l’utiliser.

openskills sync

Mode manuel ou autre outil : joignez SKILL.md à une conversation, ou ouvrez-le dans un agent capable de lire les fichiers. Ajoutez les ressources annexes nécessaires. Il n’existe pas de dossier universel reconnu par tous les outils.

4. Faire un premier essai

Utilisez une entrée courte et fictive : un endpoint d’API, un petit document ou un brief d’interface. Dans une nouvelle conversation, joignez le fichier ou donnez son chemin à l’agent, puis adaptez cette demande :

Lis .skill-downloads/code-review-and-pr-review-msh3uscu/SKILL.md et applique ses instructions à cet exemple : [décrire une entrée de test]. Indique les fichiers, dépendances ou accès qui manquent. Produis le résultat demandé par la skill sans inventer de données ni exécuter de script non examiné.

Résultat attendu : une sortie correspondant aux instructions de la skill, ou une liste précise des prérequis manquants. Une simple confirmation de lecture ne valide pas le premier usage. Si une ressource externe est indisponible, ce parcours reste à tester.

Skills similaires

Bonnes pratiques React Hook Form

Developpement

Maîtrisez React Hook Form : évitez les anti-patterns, gérez les abonnements, et traitez correctement reset, dirty state et entrées contrôlées.

Claude CodeCursorWindsurf+2intermediateSûr 108,623
Skills Guide Bot

Migration Emotion vers Mantine

Developpement

Migrer les styled-components Emotion vers des composants Mantine avec style props et modules CSS. Pour convertir .styled.tsx ou supprimer les imports @emotion.

Claude CodeCursorWindsurf+1intermediateSûr 48,981
Skills Guide Bot

Détecteur de dérive de périmètre

Developpement

Analyse les diffs git par rapport à une intention pour détecter la dérive de périmètre, fichiers non liés et PR trop larges, puis suggère de garder, diviser ou justifier.

Claude CodeCursorCodexintermediateSûr 135,015
Skills Guide Bot