コードレビュー
SwiftLintの自動チェックと包括的なコード品質分析を備えた、homete iOSプロジェクト用のSwiftコードレビュースキル。
ワークフロー
このスキルは以下の手順でコードレビューを実行します:
1. SwiftLintの実行
まず、SwiftLintを実行してコードスタイルと品質をチェックします:
bash
swift run --package-path ProjectTools swiftlint lint --config .swiftlint.yml
SwiftLint結果の報告:
- •警告やエラーが存在する場合、ファイルパスと行番号を明確にリスト表示
- •重要度でグループ化(エラーを最初に、次に警告)
- •エラーがある場合はコードレビューに進まず、ユーザーが修正する必要があることを伝える
2. 変更差分の取得
現在のブランチの派生元(分岐点)からの変更差分を取得:
bash
git diff $(git merge-base main HEAD)..HEAD -- '*.swift'
ファイルが見つからない場合は、origin/mainとの差分を試す:
bash
git diff $(git merge-base origin/main HEAD)..HEAD -- '*.swift'
差分取得のポイント:
- •差分が大きすぎる場合(10000行以上)は、主要な変更箇所を優先してレビュー
- •差分がない場合は、ユーザーに変更がないことを伝える
3. ios-code-reviewerエージェントを起動
Taskツールを使用してios-code-reviewerエージェントを起動し、取得した差分をレビューさせます:
code
Task tool with: - subagent_type: "ios-code-reviewer" - description: "Swiftコードの差分をレビュー" - prompt: "以下のSwift差分をレビューしてください:\n\n[差分内容を貼り付け]"
重要: エージェントには具体的な差分内容を渡すこと。「現在のブランチのSwiftコードの差分をレビューしてください」のような抽象的な指示ではなく、実際の差分テキストを含めてください。
4. レビュー結果のユーザーへの報告
エージェントから返されたレビュー結果を、以下の形式でユーザーに報告します:
- •SwiftLintの結果(エラー・警告のサマリー)
- •レビューの総合評価
- •主要なフィードバック(アーキテクチャ、並行性、コード品質)
- •改善提案と良い点
- •優先度の高い修正項目
使用タイミング
このスキルを使用する場合
- •ユーザーが
/code-reviewコマンドを実行した時 - •機能実装、リファクタリング、バグ修正が完了した時
- •ユーザーが「コードレビューをお願いします」と依頼した時
このスキルを使用しない場合
- •コード構造に関する一般的な質問(代わりに探索を使用)
- •Swiftファイルに変更がない場合
- •初期のコード探索や理解フェーズ中
注意事項
- •このスキルはエントリーポイントとして機能します
- •実際のレビュー処理は
ios-code-reviewerエージェントが実行します - •スキル自身でSwiftLintやgit diffを直接実行しないでください
リファレンス
プロジェクトの詳細なアーキテクチャと規約については、以下を参照:
- •
homete/CLAUDE.md- プロジェクト概要とアーキテクチャパターン - •
.swiftlint.yml- SwiftLint設定とルール - •
homete/Model/AppDependencies.swift- DIコンテナ実装