# PR番号を指定してレビュー
/pr-review 123

# 現在のブランチの変更をレビュー
/pr-review

# GitHub CLIでPR情報を取得
gh pr view $PR_NUMBER --json title,body,author,commits,files,additions,deletions

# 現在のブランチから関連PRを検索
gh pr list --head $(git branch --show-current) --json number,title

## プルリクエスト情報

- **PR**: #123
- **タイトル**: feat: Add product domain with DDD pattern
- **作成者**: @username
- **コミット数**: 5
- **変更ファイル数**: 12
- **追加**: +450 lines
- **削除**: -20 lines

# PRの変更ファイルとdiffを取得
gh pr diff $PR_NUMBER > /tmp/pr-$PR_NUMBER.diff

# 変更ファイル一覧
gh pr view $PR_NUMBER --json files --jq '.files[].path'

## 変更ファイル

### ドメインロジック（7ファイル）
- src/application/domain/product/usecase.ts
- src/application/domain/product/service.ts
- src/application/domain/product/data/repository.ts
- src/application/domain/product/data/converter.ts
- src/application/domain/product/data/type.ts
- src/application/domain/product/presenter.ts
- src/application/domain/product/controller/resolver.ts

### GraphQLスキーマ（3ファイル）
- src/application/domain/product/schema/query.graphql
- src/application/domain/product/schema/mutation.graphql
- src/application/domain/product/schema/type.graphql

### 設定ファイル（2ファイル）
- src/application/provider.ts
- src/presentation/graphql/resolver.ts

# 変更されたドメインのアーキテクチャを検証
/validate-architecture product

## アーキテクチャレビュー

### ✅ 準拠項目
- Resolverは UseCaseのみを呼び出している
- UseCaseで適切にトランザクションを管理
- Serviceがビジネスロジックを実装
- Repositoryで RLS を使用
- GraphQL型が正しい命名規則

### ⚠️  改善推奨
- [ ] `controller/dataloader.ts` の追加を推奨（N+1問題防止）
- [ ] `service.ts:45` でのenum値バリデーションの追加を検討

# 変更されたファイルのセキュリティチェック
/security-review product

## セキュリティレビュー

### ✅ セキュリティ準拠
- 適切な認証チェック
- RLS が全てのクエリで使用されている
- 入力バリデーションが実装されている

### ⚠️  セキュリティ警告
- なし

# TypeScriptコンパイルエラーをチェック
pnpm build 2>&1 | grep "error TS"

# Lintエラーをチェック
pnpm lint 2>&1 | grep "error"

## コード品質

### ✅ 品質基準
- TypeScriptコンパイルエラー: 0件
- Lintエラー: 0件
- 適切なコメント
- 命名規則が一貫している

### 📝 推奨事項
- [ ] `service.ts` の複雑な関数を分割することを検討
- [ ] `converter.ts` にJSDocコメントの追加を推奨

# 新規ドメインに対応するテストファイルを検索
find src/application/domain/product -name "*.test.ts"
find __tests__ -path "*/product/*" -name "*.test.ts"

# ドメインのテストを実行
pnpm test src/application/domain/product/**/*.test.ts --coverage --runInBand

## テストカバレッジ

### 📊 カバレッジ統計
- Statements: 92.5%
- Branches: 85.3%
- Functions: 90.0%
- Lines: 91.8%

### ✅ テスト実装
- ユニットテスト: 8ファイル、42テスト
- 統合テスト: 2ファイル、12テスト

### ⚠️  カバレッジ不足
- [ ] `service.ts` のエラーハンドリングをテスト（現在: 78%）
- [ ] `converter.ts` のエッジケースをテスト（現在: 82%）

### 💡 推奨
- 目標カバレッジ90%に対して現在92.5% ✅

# GraphQLスキーマファイルの変更を確認
git diff origin/main -- '**/*.graphql'

## GraphQLスキーマ

### ✅ スキーマ準拠
- 型命名が `Gql*` 規則に従っている
- 破壊的変更なし
- 全フィールドに説明あり

### 📝 提案
- [ ] `GqlProduct` に `imageUrl` フィールドの追加を検討
- [ ] `productCreate` ミューテーションのエラーハンドリングを明記

# Prismaスキーマファイルの変更を確認
git diff origin/main -- prisma/schema.prisma

# 新しいマイグレーションファイルを確認
ls -t prisma/migrations/ | head -1

## データベーススキーマ

### ✅ スキーマ準拠
- 命名規則に従っている
- 適切なインデックスが設定されている
- 外部キー制約が適切

### 📝 確認事項
- マイグレーション: `add_products_table`
- マイグレーションSQLが安全であることを確認済み ✅

# ドキュメントファイルの変更を確認
git diff origin/main -- 'docs/**/*.md' 'README.md' 'CHANGELOG.md'

## ドキュメント

### ✅ ドキュメント更新
- CHANGELOG.md に機能追加を記載

### 📝 推奨更新
- [ ] README.md に新規ドメインの説明を追加
- [ ] `docs/handbook/FEATURES.md` に機能詳細を追加

# プルリクエストレビュー総合評価

## PR情報
- **PR**: #123
- **タイトル**: feat: Add product domain with DDD pattern
- **作成者**: @username
- **変更**: +450 / -20 lines (12ファイル)

## 総合評価: ✅ 承認（条件付き）

このPRは全体的に高品質で、DDD/Clean Architectureの原則に従っています。いくつかの軽微な改善提案がありますが、マージ可能な状態です。

---

## レビュー詳細

### ✅ 優れている点

1. **アーキテクチャ準拠**
   - レイヤー責任が明確
   - トランザクション管理が適切
   - RLSが全てのクエリで使用されている

2. **セキュリティ**
   - 認証・認可が適切に実装
   - 入力バリデーションが実装されている
   - センシティブデータの取り扱いが安全

3. **テストカバレッジ**
   - カバレッジ92.5%（目標90%を達成）
   - ユニットテストと統合テストの両方を実装

4. **GraphQLスキーマ**
   - 命名規則に従っている
   - 破壊的変更なし
   - 全フィールドに説明あり

### 📝 改善提案（マージ前）

- [ ] **High**: `controller/dataloader.ts` を追加してN+1問題を防止
- [ ] **Medium**: `service.ts` の複雑な関数を分割
- [ ] **Low**: README.md に新規ドメインの説明を追加

### 💡 将来の改善提案（マージ後）

- [ ] `service.ts` のエラーハンドリングのテストカバレッジ向上（現在78%）
- [ ] `GqlProduct` に画像URLフィールドの追加を検討
- [ ] パフォーマンスベンチマークの実施

---

## チェックリスト

- ✅ アーキテクチャレビュー完了
- ✅ セキュリティレビュー完了
- ✅ コード品質チェック完了
- ✅ テストカバレッジ確認完了
- ✅ GraphQLスキーマレビュー完了
- ✅ データベーススキーマレビュー完了
- ⚠️  ドキュメント更新推奨

---

## 次のステップ

1. **DataLoaderの追加**（必須）
   - `controller/dataloader.ts` を作成
   - N+1問題を防止

2. **ドキュメント更新**（推奨）
   - README.md に新規ドメインの説明を追加

3. **マージ後**
   - `/test-domain product` で最終確認
   - 本番環境へのデプロイ前にステージング環境でテスト

---

## コメント

素晴らしいPRです！DDD/Clean Architectureの原則に忠実で、テストカバレッジも十分です。DataLoaderの追加後、安心してマージできます。

お疲れ様でした！🎉

# レビューコメントを投稿
gh pr review $PR_NUMBER --approve --body "$(cat review-report.md)"

# または変更要求の場合
gh pr review $PR_NUMBER --request-changes --body "$(cat review-report.md)"

# レビュー履歴をファイルに保存
echo "$(date): PR #$PR_NUMBER reviewed" >> .claude/review-history.log