review-code-qualitylisted
Install: claude install-skill YasuakiOmokawa/skills
# Review Code Quality
**🔴 Critical / 🟠 Major のうち機械的に安全なものは自動適用 + 適用後に lint/test で検証する。設計判断を要するものは `/polish-before-commit` への申し送りに回す (連続スキル実行で提案が握りつぶされるのを防ぐため)。🟡 Minor 以下は提案のみ。** 振り分け基準・検証・申し送り contract は [references/auto-apply.md](references/auto-apply.md) を SSOT とする。
4 観点を専用 agent で分析し統合レポートを出力する。Tier 1 (常時) = 凝集度 / 結合度 / 可読性 の設計レベル問題 (RuboCop/ESLint で漏れるもの)、Tier 2 (条件付き) = 業務副作用 chain (feature-flag revival / auth bypass 等) で、対象 diff に domain model attribute (`plan_code` / `role` / `status` 等) の更新が含まれる場合のみ実行し、無ければ `skip` 報告。
**重大度 (全 step 共通):** 🔴 Critical (即修正 / auto-apply 対象) / 🟠 Major (この PR で修正 / auto-apply 対象) / 🟡 Minor (次 PR / 提案のみ) / 🔵 Info (認識のみ) / ✅ Good (維持)。詳細・出力ルールは [references/integration-output.md](references/integration-output.md) を SSOT とする。
## Task complexity tier
| Tier | 判定 | 実行範囲 |
|---|---|---|
| **lite (skip)** | 1 ファイル <50 LoC かつ pure typo / copy / comment / lint-only / config 値変更のみ | **skip** (本 skill 不要) |
| **standard** (default) | 2 ファイル以下 (≤2) | main thread 順次 4 観点 |
| **deep** | 3 ファイル以上 (>2) | 4 agent 並列 |
**business-impact-analyzer (Tier 2)** は domain model attribute (`plan_code` / `role` / `status` 等) の更新を含む diff のみ実行。それ以外は skip 報告で完了。リスク領域 (auth / billing / payment / migration) は LoC によらず **deep** + business-impact-analyzer 必須。
## Quick start
1. `$ARGUMENTS` 指定があればそのファイル、なければ `git diff --name-only origin/develop...HEAD` で対象を確定。0 件なら終了
2. 処理方式を選ぶ (詳細: [references/execution.md](references/execution.md)):
- ファイル ≤ 2 → **main th