This Claude skill serves as a comprehensive framework for mastering the code review process, focusing on both technical excellence and team synergy. It provides actionable checklists for functionality, security, and performance, while offering specialized guidance on constructive feedback, self-review strategies, and the integration of automation tools like Danger and GitHub Actions to streamline development workflows.

When should I use code-review?

code-review is useful in the following scenarios: • Establishing a standardized team code review culture using multi-dimensional checklists (Design, Readability, Testing, etc.). • Training developers to provide constructive, prefix-based feedback (e.g., [Must], [Suggestion], [Praise]) to improve communication quality. • Automating Pull Request audits by integrating AI agents and static analysis tools into CI/CD pipelines. • Performing rigorous self-reviews to catch common anti-patterns and bugs before submitting code for peer evaluation. • Resolving technical conflicts and improving review efficiency through proven best practices and communication templates.

name	code-review
description	効果的なコードレビューの実施方法、レビュー観点チェックリスト、建設的なフィードバック技術、セルフレビュー手法、自動化ツール活用まで、品質とチームコラボレーションを向上させる包括的ガイド。

Code Review Skill

概要

このSkillは、効果的なコードレビューの全てをカバーします：

✅ レビュー観点チェックリスト
✅ セルフレビュー手法
✅ 建設的なフィードバック技術
✅ レビュープロセス設計
✅ 自動レビューツール活用
✅ チームコラボレーション向上
✅ よくある指摘事項パターン
✅ レビュー効率化テクニック

📚 公式ドキュメント・参考リソース

このガイドで学べること: レビュープロセス設計、効果的なフィードバック方法、自動化ツール活用、チーム文化構築 公式で確認すべきこと: 最新のレビューツール機能、GitHub/GitLab新機能、ベストプラクティス

主要な公式ドキュメント

GitHub Pull Request Documentation - PR運用ガイド
- Reviewing Changes
- Code Review
Google Engineering Practices - Googleコードレビューガイド
- Reviewer Guide
- Author Guide
GitLab Code Review Guidelines - GitLabレビューガイド

いつ使うか

自動的に参照されるケース

PRを作成する時（セルフレビュー）
PRをレビューする時
レビューコメントに返信する時

手動で参照すべきケース

チームのレビュー基準を決定する時
レビュープロセスを改善する時
新メンバーへのレビュー教育時
レビューで議論が発生した時

レビュー観点

7つの主要観点

観点	確認内容	詳細ガイド
1. 機能性	要件を満たしているか	guides/01-functionality.md
2. 設計	アーキテクチャに従っているか	guides/02-design.md
3. 可読性	理解しやすいか	guides/03-readability.md
4. テスト	十分なテストがあるか	guides/04-testing.md
5. セキュリティ	脆弱性はないか	guides/05-security.md
6. パフォーマンス	効率的か	guides/06-performance.md
7. 保守性	将来の変更が容易か	guides/07-maintainability.md

完全チェックリスト: checklists/review-checklist.md

レビュープロセス

1. セルフレビュー（PR作成前）

必須ステップ:

1. 変更をdiffで確認
2. セルフレビューチェックリスト実行
3. デバッグコード・コメント削除
4. テスト実行
5. ビルド確認

詳細: guides/08-self-review.md チェックリスト: checklists/self-review.md

2. レビュー実施

レビュワーのステップ:

1. PRの目的・背景を理解
2. テスト・ビルドの通過確認
3. コード全体の構造把握
4. 詳細レビュー（7つの観点）
5. コメント作成
6. 総合判断（Approve/Request Changes）

詳細: guides/09-reviewing.md

3. フィードバック対応

作成者のステップ:

1. コメントを全て読む
2. 質問に回答
3. 修正が必要なものを修正
4. 議論が必要なものを議論
5. Re-review依頼

詳細: guides/10-feedback-response.md

コミュニケーション

建設的なフィードバック

✅ Good

「ここでnil checkを追加すると、クラッシュを防げます。
Optional bindingを使うのはどうでしょうか？

if let user = user {
    // ...
}
」

❌ Bad

「これだとクラッシュするよ」

コメントの種類

プレフィックス	意味	例
[必須]	修正必須	`[必須] メモリリークの可能性`
[推奨]	修正推奨	`[推奨] この関数は分割できます`
[質問]	質問・確認	`[質問] この仕様で良いですか？`
[提案]	代替案提示	`[提案] Combineを使うと簡潔に`
[学習]	学習機会	`[学習] こういう書き方もあります`
[賞賛]	良い点	`[賞賛] いいリファクタですね！`

詳細: guides/11-communication.md

自動化

自動レビューツール

ツール	用途	設定ガイド
SwiftLint	コーディング規約	scripts/swiftlint-setup.sh
SwiftFormat	フォーマット	scripts/swiftformat-setup.sh
Danger	PR自動チェック	scripts/danger-setup.sh
SonarQube	静的解析	scripts/sonarqube-setup.sh

詳細: guides/12-automation.md

GitHub Actions統合

→ references/github-actions-integration.md

よくある問題

よくある指摘事項

1. 強制アンラップ

❌ let name = user!.name
✅ guard let user = user else { return }
   let name = user.name

2. Magic Number

❌ if items.count > 10 { ... }
✅ private let maxItemsCount = 10
   if items.count > maxItemsCount { ... }

3. 長すぎる関数

❌ func processUser() { // 100行 }
✅ func processUser() {
      validateUser()
      saveUser()
      notifyUser()
   }

全パターン: references/common-issues.md

レビューが遅い

→ references/review-efficiency.md

レビューで対立

→ references/conflict-resolution.md

Agent連携

このSkillを使用するAgents

code-review-agent
- PR自動レビュー
- 7つの観点でチェック
- Thoroughness: thorough
style-checker-agent
- コーディングスタイル確認
- Thoroughness: quick
security-scanner-agent
- セキュリティ脆弱性スキャン
- Thoroughness: thorough
test-coverage-agent
- テストカバレッジ確認
- Thoroughness: medium

推奨Agentワークフロー

PR作成時（並行実行）

code-review-agent (包括的レビュー) +
style-checker-agent (スタイルチェック) +
security-scanner-agent (セキュリティ) +
test-coverage-agent (カバレッジ)
→ 結果統合 → PRに自動コメント

セルフレビュー時（順次実行）

self-review-agent (セルフレビュー支援)
→ 指摘事項を確認・修正
→ 再度確認

クイックリファレンス

レビューコメントテンプレート

機能性の問題

[必須] エッジケースの考慮不足

現在の実装だと、空配列の場合にクラッシュします。

提案:
guard !items.isEmpty else { return }

テストケース:
- 空配列
- 要素1つ
- 大量の要素

設計の改善提案

[推奨] 責務の分離

現在ViewControllerに全てのロジックがありますが、
ViewModelに移動することで、テスタビリティが向上します。

メリット:
- テストが容易
- 再利用性向上
- 関心の分離

賞賛

[賞賛] エラーハンドリング

きちんとエラーケースを考慮されていて素晴らしいです！
ユーザーにわかりやすいメッセージも表示されていますね。

詳細ドキュメント

Guides（詳細ガイド）

包括的ガイド（新規追加）

ベストプラクティス完全ガイド 🌟
- レビューの原則と目的
- レビュワー/作成者/メンテナーの視点
- 建設的なフィードバック技術
- セルフレビュー戦略
- レビューツール活用
- 言語別ベストプラクティス（TypeScript, Python, Swift, Go）
- チーム文化の構築
- ケーススタディ
チェックリスト完全ガイド ✅
- 総合チェックリスト
- TypeScript/JavaScriptチェックリスト
- Pythonチェックリスト
- Swiftチェックリスト
- Goチェックリスト
- セキュリティチェックリスト（OWASP Top 10）
- パフォーマンスチェックリスト
- テストチェックリスト
- アーキテクチャチェックリスト
- ドキュメントチェックリスト
レビュープロセス完全ガイド 📋
- コードレビューの基礎
- レビュープロセス詳細
- レビュー観点とチェックリスト
- 効果的なフィードバック
- レビュー時間管理
- チーム文化
- メトリクス測定
- ベストプラクティス
自動化完全ガイド 🤖
- 自動化の基礎
- Danger.js実装
- ReviewDog設定
- 自動ラベリング
- 自動レビュワー割り当て
- AI支援レビュー
- メトリクス自動収集
- 統合ワークフロー
レビューテクニック完全ガイド 🔍
- レビューテクニック基礎
- 静的解析活用
- セキュリティレビュー
- パフォーマンスレビュー
- アーキテクチャレビュー
- テストレビュー
- ドキュメントレビュー
- ペアレビュー

Checklists（チェックリスト）

Templates（テンプレート）

PRテンプレート - 包括的なPull Requestテンプレート
- 概要、変更内容、テスト
- Breaking Changes、マイグレーション
- セキュリティ、アクセシビリティ
- セルフレビューチェックリスト
Dangerfile - 自動PRチェック
- PRサイズチェック
- Conventional Commits検証
- カバレッジチェック
- デバッグコード検出
- セキュリティチェック
- 影響範囲分析

Workflows（ワークフロー）

完全レビューワークフロー - GitHub Actions完全自動化
- 基本チェック（PRサイズ、デバッグコード）
- Linting & Formatting（ESLint, Prettier, TypeScript）
- テスト & カバレッジ
- セキュリティスキャン（npm audit, Snyk, CodeQL）
- 依存関係分析
- コード品質分析
- Danger.js自動レビュー
- ReviewDog統合
- パフォーマンステスト
- 自動ラベリング
- レビュワー自動割り当て
- メトリクス収集
- 通知（Slack）

When & Why to Use This Skill

Use Cases