review

レビュースキル

すべてのレビュアーのペルソナを1つのスキルに統合します。コードレビューが必要なときはいつでも Claude がこのスキルを有効にし、以下にリンクされたリファレンスドキュメントを開くか、ペルソナ固有のサブエージェントを起動して、必要な観点だけを「遅延読み込み」します。

重要なワークフロー

必須：コードレビューを行う前に、必ず Read ツールを使って関連するペルソナのリファレンスファイルを読み込んでください。これらのリファレンスには、各レビュアーのペルソナに固有のレビュー優先事項、視点、評価基準が記載されています。

1. コードや差分のコンテキストと、ユーザーの目標（バグ、アーキテクチャ、パフォーマンスなど）を把握します。
2. 必須：レビュアーに関するヒント（例：「perf, react, typescript」）を解析し、レビューを始める前に Read ツールを使って該当するリファレンスファイルを直接読み込みます。
   • AI／ML に関する懸念 → 最初に references/ai-reviewer.md を読み込みます
   • 型システムに関する懸念 → 最初に references/anders-reviewer.md を読み込みます
   • テスト／TDD に関する懸念 → 最初に references/beck-reviewer.md を読み込みます
   • パフォーマンス／抽象化に関する懸念 → 最初に references/bjarne-reviewer.md を読み込みます
   • 革新性／実用性に関する懸念 → 最初に references/brendan-reviewer.md を読み込みます
   • 低レベルのパフォーマンスに関する懸念 → 最初に references/carmack-reviewer.md を読み込みます
   • 分散システムに関する懸念 → 最初に references/dean-reviewer.md を読み込みます
   • 慣習／シンプルさに関する懸念 → 最初に references/dhh-reviewer.md を読み込みます
   • リファクタリング／アーキテクチャに関する懸念 → 最初に references/fowler-reviewer.md を読み込みます
   • コラボレーション／CI/CD に関する懸念 → 最初に references/github-reviewer.md を読み込みます
   • 抽象化／モジュール性に関する懸念 → 最初に references/grace-reviewer.md を読み込みます
   • 可読性／Python に関する懸念 → 最初に references/guido-reviewer.md を読み込みます
   • 移植性／Java に関する懸念 → 最初に references/james-reviewer.md を読み込みます
   • コンパイラー／ツールに関する懸念 → 最初に references/lattner-reviewer.md を読み込みます
   • システム／厳密性に関する懸念 → 最初に references/linus-reviewer.md を読み込みます
   • 開発者の喜び／Ruby に関する懸念 → 最初に references/matz-reviewer.md を読み込みます
   • 可観測性／トレーシングに関する懸念 → 最初に references/perf-reviewer.md を読み込みます
   • React のパターンに関する懸念 → 最初に references/react-reviewer.md を読み込みます
   • Go／並行処理に関する懸念 → 最初に references/rob-reviewer.md を読み込みます
   • Unix の哲学に関する懸念 → 最初に references/unix-reviewer.md を読み込みます
3. 読み込んだリファレンスに記載された具体的な指針と優先事項に従い、レビュアーのペルソナの視点を適用します。
4. 具体的なファイルや行を示し、問題点を指摘して、実行可能な推奨事項を提示します。

適切なペルソナのリファレンスファイルを先に読み込まずに、コードレビューを行ってはなりません。

一般的なチェックリスト

• 変更を評価する前に、入力／出力、依存関係、期待される動作を理解します。
• 許可されたツール（Read、Grep、Glob、Bash）を使って、実装、履歴、テストを調査します。
• 正確性、安全性、パフォーマンス、保守性、ユーザーへの影響を評価します。
• テストの不足、ドキュメントの不備、リグレッション、アーキテクチャの乖離を指摘し、具体的な修正案を提示します。
• 指摘事項を重大度の高い順にまとめ、その後にリスク、質問、検証手順を記載します。

複数の視点によるレビュー

複数のペルソナが指定された場合（例：「Anders と React の視点でレビューしてください」）：

• 以下のリストから、該当するリファレンスファイルをそれぞれ読み込みます
• 各視点の優先事項と懸念に基づいてコードを評価します
• 指摘事項を統合し、視点間で一致する点や対立する点を明らかにします
• すべての視点を通じて、問題を重大度順に優先付けします

ペルソナの参照（必要に応じて読み込み）

• AI Visionaries - 適応型システム、創発的な振る舞い、データ駆動型設計。手順を開く
• Anders Hejlsberg - 強い型付け、言語やツールの使いやすさ、構造化された API。手順を開く
• Kent Beck - TDD の規律、迅速なフィードバックループ、適応型設計。手順を開く
• Bjarne Stroustrup - 抽象化によるパフォーマンス、型安全性、規律あるエンジニアリング。手順を開く
• Brendan Eich - 迅速なイノベーション、創造的な問題解決、実践的な実験。手順を開く
• John Carmack - 低レベルの卓越性、グラフィックスやパフォーマンスの調整、正確な思考。手順を開く
• Jeff Dean - 地球規模のシステム、効率性、分散環境での信頼性。手順を開く
• DHH - 独自の規約、開発者の自律性、形式よりもシンプルさ。手順を開く
• Martin Fowler - リファクタリングへの備え、進化型アーキテクチャ、意図的な設計。手順を開く
• GitHub Generation - コラボレーションのベストプラクティス、ドキュメント、CI/CD の自動化。手順を開く
• Grace Hopper & Barbara Liskov - 抽象化の整合性、置換可能性、モジュール設計。手順を開く
• Guido van Rossum - 読みやすさ、Python らしいシンプルさ、実践的な明快さ。手順を開く
• James Gosling - JVM の移植性、API の安定性、後方互換性。手順を開く
• Chris Lattner - コンパイラーやツールチェーンのイノベーション、言語間の相互運用性、パフォーマンス。手順を開く
• Linus Torvalds - カーネルレベルの厳密さ、パッチの規律、率直なフィードバック。手順を開く
• Yukihiro "Matz" Matsumoto - Ruby の美しさ、人間中心の設計、コードを書く喜び。手順を開く
• Brendan Gregg & Liz Rice - 可観測性、トレーシング、データを重視したパフォーマンス分析。手順を開く
• React Core Maintainer - フック、並行レンダリング、開発者体験を重視したコンポーネントパターン。手順を開く
• Rob Pike - Go／Unix のミニマリズム、並行処理のプリミティブ、組み合わせ可能なツール。手順を開く
• Unix Traditionalist - 小さく鋭いツール、組み合わせやすさ、テキストを中心とした自動化。手順を開く

各参照は明示的に開くまでコンテキストに読み込まれないため、Claude のコンテキストを軽く保ちながら、元の詳細なレビュー指針にすばやくアクセスできます。