reviewing-code について
reviewing-codeは、優先度を付けた実行可能なフィードバックを行うコードレビュー用スキルです。セキュリティ、正確性、パフォーマンス、保守性を確認し、統一された形式で結果をまとめます。
主な機能
- 01指摘事項を優先度別に分類
- 02セキュリティと正確性のリスクを確認
- 03パフォーマンスとロジックを検査
- 04保守性の問題を指摘
- 05レビュー結果の定型形式を提供
活用シーン
- 01マージ前の変更をレビュー
- 02よくあるセキュリティと正確性の問題を確認
- 03一貫した形式のレビューコメントを作成
インストール
コマンドをコピーしてターミナルで実行します。
$
npx skills add https:/インストール後、Claude Code で/reviewing-codeを入力して読み込まれたか確認します。
同梱ファイルREACT-PATTERNS.md
詳細
コードレビュー
優先順位を付けた、実行可能なフィードバックを行います。相手に悪意がないことを前提とします。
優先度
| レベル | 接頭辞 | 意味 |
|---|---|---|
| ブロッカー | 🚨 | マージ前に必ず修正 |
| 提案 | ⚠️ | 検討が必要 |
| 軽微 | 💭 | 任意の改善 |
| 質問 | ❓ | 説明が必要 |
| 賞賛 | ✨ | よくできています |
チェックリスト
ブロッカー（セキュリティと正確性）
- コードにシークレットや認証情報が含まれていない
- ユーザー入力を検証／サニタイズしている
- 必要な箇所に認証チェックがある
- エッジケース（null、空、エラー）に対応している
- 非同期コードに競合状態がない
優先度高（パフォーマンスとロジック）
- N+1 クエリがない
- 大きなリストをページネーションしている
- コストの高い処理をレンダーループ内で実行していない
- 適切なエラー処理を行っている
優先度中（保守性）
- 関数が単一の責務を担っている
- コピー＆ペーストによる重複がない
- 命名が明確である
出力形式
## Summary
[Approve / Request Changes / Discuss]
## 🚨 Blockers
- `file:line` - Issue and fix
## ⚠️ Suggestions
- `file:line` - Suggestion and why
## 💭 Nits
- `file:line` - Minor improvement
## ✨ Good Stuff
- What was done well
よくある問題
React：useEffect の依存配列の不足、導出すべき状態、プロップドリリング
TypeScript：any 型、null チェックの不足
API：ローディング／エラー状態の不足、上限のないクエリ
React／フックのパターンやアクセシビリティのチェックについて詳しくは、REACT-PATTERNS.md を参照してください。
コメント
Markdown 対応