code-review

コードレビュー

機能を実装するのではなく、コード変更をレビューするタスクでこのスキルを使用します。

まず確認すること

• リポジトリの正式なレビュー規則については、references/review-checklist.md を読んでください。
• ルートの AGENTS.md と、レビュー対象のファイルに最も近いパッケージの AGENTS.md を読んでください。
• ClickHouse に関わるレビューでは、共有の clickhouse-best-practices スキルも使用してください。
• バックエンドコードに関わるレビューでは、該当する場合、共有の backend-dev-guidelines スキルも使用してください。
• 変更によって、ユーザー指定の URL を受け入れる場合、外部への HTTP 通信を追加する場合、新しいインテグレーションを導入する場合、シークレット、RBAC、リダイレクト処理、プロダクト分析、ブラウザーモニタリング、またはセッションリプレイに関わる場合は、共有の security-review スキルも使用してください。承認前に、その references/checklist.md を実行してください。

レビューの優先事項

次の点に注目してください。

• 正確性に関するバグ
• 動作のリグレッション
• セキュリティおよびテナント分離のリスク
• 実際に影響のあるパフォーマンスの問題
• リスクの高い変更に対するテストの不足や不備
• テストカバレッジが不足していると指摘する前に、提案する各テストが検出する固有のリグレッションを特定してください。差分をそのまま言い換えるだけのアサーション（間隔の値やラベルなど）しかないテストは、ファイルを増やすだけで何も証明しないため、求めないでください。最も近い既存のテストスイートへの統合を優先してください。各テストが異なるトランスポート、投影、または実行境界を検証するのでなければ、同じ述語をレイヤーごとに繰り返しテストしている点を指摘してください。

出力に関する期待事項

• 指摘事項を重大度の高い順に最初に示す
• 各指摘事項にファイルと行番号を記載する
• 指摘事項の後にのみ、短い概要を示す
• 指摘事項がない場合は、その旨を明記し、残存するリスクやカバレッジの不足についても触れる

対象範囲のガイダンス

Langfuse 固有の次の確認項目には、references/review-checklist.md を使用してください。

• ClickHouse および Postgres のマイグレーションに関する要件
• プロジェクト単位のテナント分離の確認
• Prisma のレコード未検出チェック（インラインでの P2025 チェックではなく、isPrismaRecordNotFoundError / isRecordNotFoundError を使用）
• API と Fern の整合性
• バナーのオフセットに応じた UI の配置
• 環境変数のアクセスパターン

これらの規則を、場当たり的なプロンプトやツール固有のコマンドファイルで重複させないでください。