# Code Review Git

> gitリモートとの差分を対象にコード品質チェックとドキュメント乖離検出を行う。

- Skill: `ousiass/code-review-git` (Agent Skill, multi-file: 3 files)
- Install (CLI): `npx skillmds@latest add ousiass/code-review-git`
- Raw SKILL.md: https://api.skillmd.com/api/skills/ousiass/code-review-git/raw
- Safety review: pending (external: skill-scanner PASS, skillspector PASS)
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Coding & Dev Tools
- Author: ousiass (https://skillmd.com/u/ousiass)
- Updated: 2026-09-22
- Page: https://skillmd.com/skills/ousiass/code-review-git

---


# code-review-git

PR 作成前のセルフレビューやマージ前の最終確認に最適。

## 前提条件

- Claude Code 環境
- `git`, `gh` CLI

## 引数

- 引数なし: 現在のブランチと `origin/develop` との差分を対象
- ブランチ指定: 比較先のリモートブランチを指定（例: `origin/main`）

## フェーズ1: 差分の取得と把握

1. `git fetch origin`
2. 比較元ブランチを決定（引数あり → 引数使用 / なし → `origin/develop`）
3. マージベースを取得（`git merge-base <比較元> HEAD`）
4. 差分ファイル一覧（`git diff --name-status <merge-base>...HEAD`）
5. 差分統計（`git diff --stat <merge-base>...HEAD`）
6. コミット一覧（`git log --oneline <merge-base>..HEAD`）
7. TaskCreate でチェック対象をタスク化

## フェーズ2: コード品質チェック（変更差分のみ）

変更ファイルの差分を取得し、**変更箇所を中心に**チェック。周辺コンテキスト（関数全体等）も必要に応じて読み取る。

チェック観点は `references/check-criteria.md` を参照。

#### 手順
1. 変更ファイルを分類（ソースコード、テスト、設定、ドキュメント）
2. 各ソースコードの差分を取得しチェック
3. 周辺コンテキストが必要ならファイル全体を読む
4. テストファイルの変更があればテストの妥当性もチェック

## フェーズ3: ドキュメント乖離チェック（差分内のドキュメント vs コード）

**差分内の変更同士を突き合わせる。** プロジェクト全体のドキュメント走査はしない。

1. 差分ファイルを分類（ドキュメント: `.md`/`.rst`/`.txt` / コード: ソース・設定）
2. コード差分からドキュメント反映が必要な変更を特定（API変更、設定変更、構成変更等）
3. ドキュメント差分から対応するコード変更があるべきものを特定
4. 突き合わせて乖離を検出:
   - コード変更あり・ドキュメント更新なし → 更新漏れ
   - ドキュメント変更あり・コード変更なし → 先行更新 or 不要変更
   - 両方変更あり・矛盾 → 不一致

## フェーズ4: テスト実行

テスト実行可能なら実行し、結果をレポートに含める。

## フェーズ5: レポート生成

1. `AskUserQuestion` で出力先確認:
   - **GitHub Issue**（推奨）: `review: Git差分レビュー(<ブランチ名> → <比較先>, <YYYY-MM-DD>)`
   - **ローカル MD**: `code-review-git-report.md`
   - **コンソール出力**: レポートをそのまま会話に出力
2. レポート形式は `templates/report.md` を参照
3. 要約をユーザーに報告

## 重大度の基準

| 重大度 | 基準 |
|-------|------|
| 🔴 重大 | セキュリティリスク、データ損失、本番障害、仕様と明確に矛盾するドキュメント |
| 🟠 重要 | バグリスク、アーキテクチャ違反、テスト不足、更新漏れのドキュメント |
| 🟡 提案 | 可読性・保守性改善、リファクタリング候補、ドキュメント補足 |
| 🟢 軽微 | 命名改善、コードスタイル、表記ゆれ |

## ルール

- **差分に集中。** 変更されていないコードは対象外（明らかなセキュリティリスクを除く）
- 推測で指摘しない。実際のコードと差分で裏付ける
- 指摘にはファイルパスと行番号を示す
- 改善案は具体的に示す
- 周辺コンテキスト（関数全体、呼び出し元等）も確認して正確にレビュー
- 🔴🟠 は必ずレポートに含める。🟡🟢 は明確なメリットがある場合のみ
- プロジェクトの既存パターン・規約を尊重する
- ドキュメント乖離は差分内の変更同士の突き合わせのみ
- TaskCreate/TaskUpdate で進捗を管理する