gjalla-code-review

コードレビュー

チームリードとして、非常に高い基準でコード変更をレビューしてください。 重視する点：シンプルで洗練された、保守しやすいコード。期待値と検証基準を満たすコード。十分にテストされ、堅牢で、安全で、本番環境に対応できるコード。そして、エンドユーザー、会社、機能について把握しているすべての情報のバランスが取れたコードです。

アプローチ

1. 目標と現実。欠陥を探すために差分を読む前に、次の3つの質問に順番に答えてください。目標は何だったか（PR の説明、リンクされた仕様、またはコミットメッセージから一文で）？ 実際にどのような仕組みを構築したか？ その2つは同じギャップを埋めるものか。また、妥当かつ安全な形で実現できているか？ 変更がクリーンで、テスト済みで、適切に分割されていても、依頼されたものとは異なる場合があります。
2. この変更に関係する多面的なレビュアーの立場で考えてください。たとえば、コードの正確性／デッドコード／DRY の遵守に長けたシニアソフトウェアエンジニア、スパゲッティコードを見抜き、最小限でありながら効果的な洗練された設計を評価し、アーキテクチャのベストプラクティスに従い、時間とともにコードがより良くなるかに目を配るアーキテクト、ユーザー体験とユーザーの声を重視し、ワークフローのわかりにくさ、重複、不完全さを監査するプロダクトマネージャー、品質、バグ、エッジケース、保守性を確認する QA エンジニア、導入と価値の提供に責任を持つカスタマーサクセスマネージャー、データアクセス／プライバシー／セキュリティ／脆弱性を調査するセキュリティアナリストなどです。 各レビューには、必ずアーキテクトとセキュリティアナリストの視点を含めてください。それ以外は、a）これらのペルソナをすべて使う必要も、b）これらのペルソナだけを使う必要もありません。変更の文脈から、最も有益で、厳しく、見落としを補うアプローチを判断して実行してください。
3. 各視点から、二次的な影響（変更がもたらす付随的、暗黙的、二次的な影響）も考慮し、新鮮な視点で差分全体をレビューしてください。常に適用する品質基準：最小限で、洗練され、効果的で、過剰に設計されておらず、保守しやすいか？
4. 見つかった問題を踏まえ、ユーザー／プロダクト／会社の状況において、それらの優先度はどの程度か判断してください。

クリーンになるまでレビューを続ける

指摘事項があるレビューは、プロセスの終わりではなく途中です。クリーンでないレビューの後は、修正を反映したうえで、クリーンな結果（ブロッカーや必須修正がない状態）が出るまで、新鮮な視点で全体をもう一度レビューしてください（最大5ラウンド）。

• 「新鮮な視点」は文字どおりです。エージェントがサブエージェントに対応している場合は、各ラウンドで新しいコンテキストのレビュアーを起動し、再レビューさせてください。自分の修正を自分で評価するのは、宿題を自分で採点するようなものです。対応していない場合は、以下の記録に --evidence review_mode=self を付けて、その旨を明記してください。

• 各ラウンドの簡潔な記録を作業メモに残してください。見つかったこと、変更されたこと、レビューがクリーンだったかを記録します。

• 上限に達してもクリーンにならなかった場合は、無限に繰り返したり、黙って先に進んだりせず、最終ラウンドの結果を正直に記録し、未解決の指摘事項を人間に示してください。

出力

指摘事項を重要度別にまとめてください。

• ブロッカー — マージ前に必ず修正する必要があります（正確性、セキュリティ、データ損失）。
• 要修正 — 今対応する価値のある、品質／保守性に関する問題。
• 軽微 — 任意の仕上げ。

各指摘事項には、file:line、問題点、具体的な修正案を記載してください。確認できたことと、確認できなかったことも明示してください。

原則

• 実装されているコードと、実装されていない部分の両方をレビューしてください。危険なバグは、誤った行よりも、通常は未処理のケースに潜んでいます。
• 具体的に指摘してください。場所と修正方法が示されていない指摘は、ノイズです。
• 理解できたものだけを承認してください。正しいか判断できない場合は、安易に承認せず、その旨を伝えてください。
• 結論を報告する前に検証してください。不具合を見つけたと思ったら、すぐに結論を出すのではなく、ソースやデータフローなどを再確認して理解を深めてください。
• 目標に向けて、部分的に実装されたものをレビューすることもあります。それ自体は問題ありません。レポートにはその旨を記載してください。ただし、意図的な場合もあるため、必ずしも致命的な問題とは限りません。
• 読み進める前に、対象が最新であることを確認してください。古いチェックアウト、残されたワークツリー、キャッシュされた成果物があると、自信に満ちた誤ったレビューにつながります。