name: code-review description: | コードレビューを実施する際に使用。プルリクエストのレビュー、コード品質チェック、 セキュリティ検査、パフォーマンス分析を行う。「レビューして」「コードをチェック」 「品質を確認」などの指示で起動。

Code Review Skill

Purpose

このスキルは、以下のガイドライン(~/.claude/CLAUDE.md)に基づいた體系的なコードレビューを提供します:

• コード品質の向上
• セキュリティ脆弱性の検出
• パフォーマンス問題の特定
• 保守性の評価
• ベストプラクティスの適用

When to Use

以下の場合にこのスキルを使用:

• プルリクエストのレビュー依頼時
• 「コードをレビューして」という明示的な指示
• コード品質チェックの要求
• セキュリティレビューの必要時
• リファクタリング前の評価

Instructions

1. Check CI/CD Status

プルリクエストの場合、最初にCI/CDステータスをチェックする:

# GitHubのPR番號が明示されている場合
gh pr view <PR番號> --json statusCheckRollup

# または、現在のブランチのPRを確認
gh pr status
gh pr checks

CI/CDチェックの結果に応じた対応:

• ✅ すべてパス: 通常のコードレビューに進む
• ❌ 失敗あり:
• 自分が作成したPR: 失敗原因を分析し、修正を実施
• 他人のPRレビュー: 失敗內容を確認し、レビューコメントに含める

CI失敗時の詳細確認:

# 失敗したチェックの詳細を確認
gh pr checks <PR番號>

# ログの取得（GitHub Actionsの場合）
gh run view <run-id> --log-failed

2. Confirm Review Scope

レビュー対象を明確にする:

# 変更ファイルの確認
git status
git diff --name-only

# PRの場合は差分を確認
gh pr diff <PR番號>

# または特定のファイルが指定されている場合はそれを対象とする

3. Understand Context

レビュー前に以下を確認:

• プロジェクトの種類(プロトタイプ/MVP/本番)
• 関連する既存コード
• テストの存在
• ドキュメント

4. Handle CI Failures

自分のPRでCI失敗の場合:

1. 失敗の原因を特定:
2. テスト失敗: どのテストがなぜ失敗したか
3. ビルドエラー: コンパイルエラー、型エラー等
4. Linter/Formatter: コーディングスタイル違反

5. セキュリティスキャン: 脆弱性検出

6. 修正を実施:

7. エラーログを読み、根本原因を修正
8. 関連するテストケースも更新

9. ローカルで同じチェックを実行して検証

10. 再プッシュ: bash git add . git commit -m "fix: resolve CI failures" git push

他人のPRレビューでCI失敗の場合:

レビューコメントの冒頭に以下を追加:

## ⚠️ CI/CD Status

### 失敗しているチェック
- [チェック名1]: [失敗理由の要約]
- [チェック名2]: [失敗理由の要約]

### 修正が必要なアクション
1. [具體的な修正內容]
2. [具體的な修正內容]

詳細は以下のコードレビューを參照してください。

---

5. Conduct Systematic Review

以下の観點でコードを評価(詳細はreference.md參照):

A. コード品質

• 可読性(変數名、関數名の適切性)
• DRY原則の遵守
• 適切な抽象化レベル
• コメントの質(「なぜ」を説明)

B. エラーハンドリング

• エラーケースの適切な処理
• エラーの抑制ではなく根本原因の解決
• 明確なエラーメッセージ
• 外部API呼び出しの失敗処理

C. セキュリティ

• 認証情報のハードコード確認
• 入力検証の実裝
• OWASP Top 10の脆弱性チェック
• 最小権限の原則

D. テスト

• テストカバレッジ
• エッジケースのテスト
• テストの獨立性
• テストの可読性

E. パフォーマンス

• N+1問題
• 不要なループや計算
• キャッシング戦略
• リソースリーク

F. 保守性

• 適切なモジュール分割
• 依存関係の明確性
• 技術的負債の記録
• 一貫したコーディングスタイル

6. Provide Feedback

レビュー結果を以下の形式で提供:

## コードレビュー結果

### ⚠️ CI/CD Status (該當する場合)
[CI失敗情報をここに記載]

### 🔴 Critical Issues (即座に修正が必要)
- [具體的な問題と場所]
- [修正案]

### 🟡 Warnings (修正を推奨)
- [改善提案]
- [理由]

### 🟢 Good Practices (良い點)
- [評価できる実裝]

### 💡 Suggestions (任意の改善案)
- [より良い実裝の提案]

### 📝 Notes
- [その他の気づき]

7. Constructive Communication

• 個人ではなくコードに焦點を當てる
• 問題だけでなく、良い點も指摘
• 具體的な改善案を提示
• 「なぜ」その変更が必要かを説明

Key Principles

1. プロジェクトコンテキストを考慮: プロトタイプと本番環境では異なる基準を適用
2. 優先順位付け: Critical > Warning > Suggestion の順で重要度を明示
3. 実用性: 完璧を求めすぎず、適切なバランスを取る
4. 學習機會: レビューを教育の機會として活用

Reference Documents

• 詳細なレビュー基準
• レビュー実例

Dependencies

• Git (変更差分の確認用)

  小蔥技能7w4.net持續更新中。

• GitHub CLI (gh) - PR情報とCI/CDステータスの取得用
• プロジェクト固有のlinter/formatter(存在する場合)

CI/CD Integration

このスキルは以下のCI/CDシステムに対応: - GitHub Actions - CircleCI (GitHub統合経由) - その他GitHub Checksに統合されているCI/CD

注意: PRのCI/CDステータス確認にはghコマンドが必要です。インストール方法:

# macOS
brew install gh

# 認証
gh auth login

這是一個質量較好的程式碼審查技能，文件結構完整、示例豐富，CI檢查整合做得實用，審查維度覆蓋全面。針對不同場景有明確的處理流程。不過全部文件都是日語，限制了使用範圍；缺少審查標準量化指標和常見問題解答；部分邊界場景的示例還不夠充分。總體適合需要系統性程式碼審查的團隊使用。