Review

コードレビュースキル

ChordBook プロジェクトのコードレビューガイドラインです。

レビュー観点

1. コード品質

TypeScript/React (フロントエンド)

• 型定義が適切か（any の使用を避ける）
• コンポーネントが適切に分割されているか
• useEffect の依存配列が正しいか
• 不要な再レンダリングがないか
• エラーハンドリングが適切か

C# (バックエンド)

• Clean Architecture の原則に従っているか
• 非同期処理に CancellationToken が含まれているか
• 依存性注入が適切に使用されているか
• 例外処理が適切か
• nullチェックが適切か

2. 設計・アーキテクチャ

• 単一責任の原則に従っているか
• 適切な抽象化レベルか
• 重複コードがないか
• 拡張性を考慮しているか

3. パフォーマンス

• N+1 クエリ問題がないか
• 不要なデータベースアクセスがないか
• メモリリークの可能性がないか
• 適切なキャッシングがされているか

4. テスト

• テストカバレッジは十分か
• エッジケースがテストされているか
• テストが独立しているか

5. ドキュメント

• 複雑なロジックにコメントがあるか
• API の変更がドキュメント化されているか

レビューコメントの書き方

カテゴリプレフィックス

[必須] - 修正が必要
[提案] - 改善提案（任意）
[質問] - 確認したい点
[称賛] - 良い実装への賞賛

例

[必須] この `any` 型を適切な型に変更してください。

[提案] このロジックは `useMemo` でメモ化するとパフォーマンスが向上します。

[質問] このエラーハンドリングで全てのケースがカバーされていますか？

[称賛] Clean Architecture の原則に沿った素晴らしい実装です！

レビュー実施コマンド

GitHub PR のレビュー

# PR の差分を確認
gh pr diff <PR番号>

# PR の詳細を確認
gh pr view <PR番号>

# レビューコメントを投稿
gh pr review <PR番号> --comment --body "レビューコメント"

# 承認
gh pr review <PR番号> --approve

# 変更要求
gh pr review <PR番号> --request-changes --body "変更理由"

チェックリスト実行

レビュー時に以下を確認：

# フロントエンド
cd apps/frontend
pnpm lint
pnpm build

# バックエンド
cd apps/backend
dotnet build ChordBook.sln
dotnet test