Claude Code skills 機能をつかって、コードレビューをさせてみた！

---
paths:
- "src/**"
description: コーディング時の基本ルール
---

## コーディングルール
**アーキテクチャ**
- ベースは 増田亨氏が唱えるドメイン駆動設計に基づいた構成とする。

**サービス層**
- サービス層に複雑な処理は記載しないこと。複雑な処理はドメイン処理として切り出すこと。

**プレゼンテーション層**
- プレゼンテーション層（tsxファイルなどのフロント側コード）で、複雑な処理は書かないこと。複雑な処理はドメイン処理として切り出すこと。

**トランスレーター層**
- APIや画面のリクエスト情報をDB保存に変換処理、DB情報をレスポンスに格納する変換処理を、トランスレーター層に切り出すこと。
- サービス層やプレゼンテーション層にて、変換処理は書かないこと。

**データベースアクセス**
- データベースアクセスはクライアント側で行ってはならない
- すべてのデータベース操作はサーバー側でのみ実行する
- データベースアクセスにはService LayerまたはDomain Logic Layerを使用
- クライアントコンポーネントはAPIルートのみを呼び出し、APIルートがService/Domain Layerを使用する

my-skill/ 
├── SKILL.md # Main instructions (required) 
├── template.md # Template for Claude to fill in 
├── examples/ 
│ └── sample.md # Example output showing expected format 
└── scripts/  └── validate.sh # Script Claude can execute

---
name: review-code-skill
description: Next.jsのコードをレビューを実施する。ユーザからコードレビューを依頼されたときにこのスキルを使用する。
---

- 日本語で回答してください。
- ベストプラクティスの具体例を提示してください。

## レビュー対象の決定
引数（`$ARGUMENTS`）の有無に応じてレビュー対象ファイルを決定してください。

### 引数がない場合（デフォルト）
Gitで変更されたファイルをレビュー対象とします。以下の手順で対象ファイルを特定してください。
1. `git diff --name-only HEAD` を実行し、コミット済みでないステージング・未ステージングの変更ファイルを取得
2. `git diff --name-only --cached` を実行し、ステージング済みの変更ファイルを取得
3. 上記の結果をマージし、重複を除いた一覧をレビュー対象とする
4. 変更ファイルがない場合は「変更されたファイルがありません。レビュー対象のファイルまたは処理名を引数で指定してください。」と表示して終了

### 引数がある場合
- **ファイルパスが指定された場合**: そのファイルをレビュー対象とする
- **処理名・機能名が指定された場合**: 関連するファイルをコードベースから検索し、レビュー対象とする

### 対象外ファイル
以下のファイルはレビュー対象から除外してください。
- `src/lib/` 配下（自動生成コード）
- `.next/`, `node_modules/` 配下
- `*.env*` ファイル
- 画像・フォントなどのバイナリファイル

## コードレビュー専用指示
### 重点チェック項目
1. **セキュリティ**: SQLインジェクション、XSS、認証・認可の不備をチェック。機密情報が直接コードにかかれていないかチェック。
2. **パフォーマンス**: N+1問題、不要なループ、メモリリーク
3. **可読性**: 変数名、関数名、コメントの適切性。
4. **保守性**: DRY原則、SOLID原則の遵守
5. **アーキテクチャ**: ベースは 増田亨氏が唱えるドメイン駆動設計に基づいた構成となっているかの観点でチェック。
6. **言語固有のベストプラクティス**: 各言語の推奨パターンに準拠しているか、非推奨・廃止予定のAPIや構文を使用していないかチェック
7. **コーディングルール**: コーディングルールは、.claude/rules/に記載しているのを参照してチェック。

### ESLint & Prettier チェック
レビュー対象ファイルに対して、以下のコマンドを実行し、結果をレビューに含めてください。
1. **ESLint**: `npx eslint {対象ファイル}` を実行し、lint違反があれば指摘してください。
2. **Prettier**: `npx prettier --check {対象ファイル}` を実行し、フォーマット違反があれば指摘してください。
- 違反がない場合は「ESLint/Prettier チェック: 問題なし」と記載してください。
- 違反があり、コマンド実行後修正された場合は、その内容を提示してください。修正前に戻す必要はないです。

---
まとめ

重要度: 重大
件数: 5
主な指摘: テスト用デフォルト値、コード重複、サービス層の責務過多 
────────────────────────────────────────
重要度: 中
件数: 5
主な指摘: バリデーション不足、N+1問題、日付フォーマット
────────────────────────────────────────
重要度: 軽微
件数: 2
主な指摘: HTML id重複