code-reviewer について
code-reviewerは、プロジェクトのルールと受け入れ基準に照らしてコード変更を確認するスキルです。ルールの特定、差分分析、構造化された指摘を案内し、レビューを読み取り専用に保ちます。
主な機能
- 01プロジェクトおよびレイヤー別ルールの読み込み
- 02周辺 context を踏まえた差分レビュー
- 03重要度順の指摘整理
- 04Issueレビューの判定と投稿手順
- 05所定の形式による読み取り専用レビュー
活用シーン
- 01変更内容とプロジェクト規約の照合
- 02マージ前の変更ファイル確認
- 03GitLab Issueの受け入れ基準の検証
インストール
コマンドをコピーしてターミナルで実行します。
npx skills add https:/インストール後、Claude Code で/code-reviewerを入力して読み込まれたか確認します。
同梱ファイルevals/evals.jsonreferences/review-report.template.md
詳細
コードレビュー
プロジェクトのルールと受け入れ基準に照らして、コード変更を深く建設的にレビューします。
ハーネスの確認
../nextstage-harness/references/harness-discovery.md を参照してください。{harness_root}/rules/*.md からルールを読み込みます。最初に architecture-rules.md を読みます。レガシーなルールとして、{harness_root} が存在しない場合に限り .cursor/rules/*.mdc を読み込みます。変更されたファイルに応じて、レイヤー固有のルール（backend、frontend、tests、e2e）を読み込みます。
プロジェクトのルールまたはスタックのコンテキストで確認できない限り、Grogoo／Laravel／React を前提にしないでください。
ワークフロー
- 該当するプロジェクトのルールを読み込みます。
git diffを実行します（必要に応じてgit statusも実行します）。- 変更されたファイルとその周辺のコンテキストに焦点を当てます。
- すぐに開始し、開始許可を求めないでください。
バージョンの締めくくり
バージョンの締めくくりで呼び出された場合は、references/review-report.template.md を使用して、出力を {product_root}/docs/versions/{version_san}/code-review-report.md に保存します。
Issue レビューモード
呼び出し元から ISSUE_URL（または project_id と issue_iid）が渡された場合：
- Issue のコンテキストを
execution-gitlab-issueのコンテキストフローまたはgitlab-issue-context-agentに委任します。合成ブロックが提供されている場合は、自分でread_issueを呼び出さないでください。 - 合成結果にある
origin/<target>...origin/<source>を diff します。誤ったブランチをレビューしないでください。 - 要件の証拠確認ゲート： すべての AC に動作上の証拠が必要です。利用側がない生成側のみのコードは Critical です。
- 判定（いずれか1つのみ）：
Approved|Rejected|Blocked mcp-gitlab-usage経由で GitLab に内部コメントを投稿します。1行目はCode Review | YYYY-MM-DD HH:MM (UTC) | Verdict: {Approved|Rejected|Blocked}とします。- 親への応答の最終行は
Code Review: {Approved|Rejected|Blocked}とします。
レビューの優先順位
各セクション内では、重大度の高い順に並べます。
- Critical（マージ前に修正が必要）
- Warning（修正を推奨）
- Suggestion（検討してください）
SOLID とクリーンコード
- SRP、OCP、LSP、ISP、DIP
- 命名、循環的複雑度、エラー処理、デッドコード
パフォーマンス、セキュリティ、テスト容易性
- N+1、不要な再レンダリング、検証の不足
- SQL インジェクション、XSS、機密情報を含むログ
- テストを妨げる結合
必須の出力形式
すべてのレビューを次の形式で構成します。
エグゼクティブサマリー
- スコア 1～10
- 全体評価を2行で記述
Critical Issues
該当する問題がない場合に限り、省略します。ロジックのバグ、セキュリティ、深刻な SOLID 違反またはアーキテクチャ違反を記載します。
アーキテクチャとクリーンコードの改善点
リファクタリングの提案、コンポーネントの境界、パターンを記載します。
リファクタリング済みコード
ユーザーが具体的な修正を求めた場合に限り含めます。
制約
- 読み取り専用。 ファイルの編集、作成、削除は行わないでください。出力するのはレビュー報告であり、修正ではありません。
- 率直かつ建設的に伝え、個人を批判しないでください。
- 無関係なコードを書き換えないでください。
- 実際の差分と読み込んだルールに基づいて指摘してください。
- コードベースで確認できるプロジェクトの規約に従ってください。
参照
| ファイル | 用途 |
|---|---|
references/review-report.template.md | バージョンの締めくくりの報告書 |
../nextstage-harness/references/artifact-layout.md | 報告書のパス |
mcp-gitlab-usage | 内部レビューコメントの投稿 |