Synthèse automatique
Ce skill exécute une revue de code complète des modifications locales ou d'une Pull Request GitHub, incluant la qualité, la performance, la sécurité et la conformité aux guidelines.
Points forts
- Interroge l'utilisateur pour déterminer précisément le périmètre de la revue
- Exécute en parallèle plusieurs sous-agents spécialisés pour une analyse multi-dimensionnelle
- Détecte les changements cassants (API, schéma, configuration, dépendances)
- Produit un rapport structuré avec scores et priorisation des problèmes
Limites
- Dépend de l'outil GitHub CLI (gh) pour les PRs, donc nécessite une authentification préalable
- Ne remplace pas une revue humaine approfondie, notamment pour les aspects métier
- Peut être long sur de gros changements en raison de l'exécution parallèle de multiples sous-agents
Avant de merger une branche, pour obtenir un feedback rapide et systématique sur la qualité du code.
Pour des modifications triviales (typo, commentaire) où le coût d'exécution dépasse le bénéfice.
Analyse de sécurité
SûrThe skill uses Bash only for safe, read-only git commands (git diff, gh pr view/diff). No destructive operations, data exfiltration, or safety bypasses are instructed. Although the PR number input from AskUserQuestion is inserted into shell commands without explicit sanitization, the user is the one providing the input in their own environment, so command injection risk is minimal and inherent to any interactive shell usage.
Aucun point d'attention détecté
Comment cette analyse est faiteExemples
コードレビューして現在のPRをレビューしてname: code-review description: 現在のブランチをコードレビューする。「コードレビューして」「レビューして」「PRの前にチェック」「破壊的変更をチェック」といった依頼、またはマージ前の品質確認が必要な場合に使用する。 allowed-tools:
- Read
- Grep
- Glob
- Bash
- Task
- AskUserQuestion
コードレビュー
概要
目的: ローカルブランチまたは GitHub PR の変更内容をレビューし、品質・一貫性・潜在的な問題を検出する。
AskUserQuestion ツールを使ってレビュー条件をインタビューし、適切なサブエージェントでレビューを実行する。
フェーズ 1: レビュー対象の確認
AskUserQuestion ツールを使って質問する。
質問内容:
- question: "レビュー対象は？"
- header: "対象"
- options:
- label: "ローカルブランチ", description: "現在のブランチと main の差分をレビュー"
- label: "GitHub PR", description: "現在のブランチPRの差分をレビュー"
- label: "ローカル差分", description: "ローカルにある差分をレビュー"
- label: "ステージ済みのローカル差分", description: "ローカルにあるステージ済みの差分をレビュー"
フェーズ 2: 破壊的変更チェックの確認
AskUserQuestion ツールを使って質問する。
質問内容:
- question: "破壊的変更チェックは？"
- header: "破壊的変更"
- options:
- label: "行う", description: "API/スキーマ変更確認"
- label: "行わない", description: "通常レビューのみ"
フェーズ 3: 変更内容の取得
フェーズ 1 の回答に基づいて変更内容を取得する。
ローカルブランチの場合:
git diff main...HEAD
git diff --name-only main...HEAD
GitHub PR の場合:
AskUserQuestion ツールを使って PR 番号を確認する。
<rules> - **AskUserQuestion ツールを必ず使用** - 会話形式の質問は不可 </rules>その後:
gh pr view <PR番号> --json title,body,files,additions,deletions
gh pr diff <PR番号>
フェーズ 4: レビューの実行
全てのサブエージェントを並列で実行する。
サブエージェント
| サブエージェント | 専門領域 |
|------------------|----------|
| review-quality | コード品質（命名、SRP、複雑度、エラー処理） |
| review-performance | パフォーマンス（計算効率、メモリ、レンダリング） |
| review-security | セキュリティ（XSS、SQLi、入力検証） |
| review-guideline | ガイドライン準拠（TypeScript/React/テストルール） |
フェーズ 6: 破壊的変更のチェック
「行う」を選択した場合のみ実行する。
| カテゴリ | 例 | |----------|-----| | API 変更 | 関数の引数追加/削除、戻り値の型変更 | | スキーマ変更 | DB カラムの削除、型変更、NOT NULL 追加 | | 設定変更 | 環境変数の追加/削除、デフォルト値の変更 | | 依存関係 | メジャーバージョンアップ、パッケージの削除 |
フェーズ 7: 結果の出力
レビュー観点（サブエージェント別）
| サブエージェント | チェック内容 | |------------------|-------------| | review-quality | 命名規則、単一責任原則、関数複雑度、エラーハンドリング | | review-performance | 計算効率、メモリ管理、レンダリング最適化、N+1問題 | | review-security | XSS、SQLi、入力検証、認証・認可、OWASP Top 10 | | review-guideline | TypeScript/React/テストルール準拠、コードスタイル |
出力形式
## コードレビュー結果
### サマリー
- レビュー対象: ローカルブランチ / PR #XXX
- 変更ファイル数: X
- 追加行数: +XXX
- 削除行数: -XXX
### スコア一覧
| 観点 | スコア |
|------|--------|
| 品質 | A/B/C/D |
| パフォーマンス | A/B/C/D |
| テスト | A/B/C/D |
| ドキュメント | A/B/C/D |
| セキュリティ | A/B/C/D |
| ガイドライン | A/B/C/D |
| Design Doc | A/B/C/D（設計書ありの場合） |
### 破壊的変更
🔴 あり / 🟢 なし
### 発見事項
#### 🔴 Critical（修正必須）
- [ファイル名:行番号] 問題の説明
#### 🟡 Warning（推奨）
- [ファイル名:行番号] 改善提案
#### 🟢 Good（良い点）
- 良かった実装のポイント
### 総評
全体的な評価とコメント
重要な注意事項
- AskUserQuestion ツールを必ず使用 - 会話形式の質問は不可
- 各フェーズで必ず AskUserQuestion を呼び出してからユーザーの回答を待つ
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 annexesCe 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.mdDepuis la racine d’un projet de test, enregistrez le fichier sous .skill-downloads/code-review-mma3jruy/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-mma3jruy' && curl --fail --show-error --location 'https://skills-guides.com/api/skills/code-review-mma3jruy/download' --output '.skill-downloads/code-review-mma3jruy/SKILL.md.part' && mv '.skill-downloads/code-review-mma3jruy/SKILL.md.part' '.skill-downloads/code-review-mma3jruy/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-mma3jruy/SKILL.md' && test -s '.skill-downloads/code-review-mma3jruy/SKILL.md' && mkdir -p '.claude/skills' && test ! -e '.claude/skills/code-review' && cp -R '.skill-downloads/code-review-mma3jruy' '.claude/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’OpenSkillsopenskills install './.skill-downloads/code-review-mma3jruy'openskills read 'code-review-mma3jruy'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 syncMode 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-mma3jruy/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.
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.
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.
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.