polish-before-commitlisted
Install: claude install-skill YasuakiOmokawa/skills
# polish-before-commit
**提案だけでなく、自動修正まで行う。** プロジェクト規約・パターン一貫性・impl/spec 整合 (現状 Ruby/RSpec の delegate/def 撤去後 dead-mock 削除のみ、TS/JS/Python は範囲外で skip) を点検し、Step 4 → 5 → 6 → 7 は順序固定で再評価ループ禁止。
**フロー最終段の役割**: この skill は `/simplify` → `/vercel-react-best-practices` → `/review-code-quality` → 本 skill というフローの最後に置かれる。Step 9 で `/review-code-quality` からの申し送り (`.git/quality-review-handoff.md`) と本 skill の Manual Review Items を集約し、**末尾でユーザー判断が必要な項目を一覧提示してから止まる** (連続スキル実行で個別レポートが transcript に埋もれ握りつぶされるのを防ぐため)。
## Task complexity tier
| Tier | 判定 | 実行 Step |
|---|---|---|
| **lite** | 1 ファイル <30 LoC, 規約 hit 0, Ruby delegate/def 撤去なし | Step 5 (lint) + Step 8 (final review) + Step 9 (集約) |
| **standard** (default) | 2-5 ファイル, 規約 hit 1-3 | Step 1-5 + Step 8 + Step 9 (Step 6/7 は条件 hit 時のみ) |
| **deep** | 6+ ファイル / 規約 hit 4+ / Ruby delegate or def 撤去あり / multi-language | 全 Step (1-9) |
**Step 0 (preflight)** は全 tier で最初に必ず実行 (`feature-dev` 未導入なら他の Step に入る前に中止)。**Step 1 (規約の収集) も tier 判定の前に全 tier で必ず実行**する — tier 判定基準の「規約 hit 数」は Step 1 の収集結果からしか得られないため (読み取りのみで安価)。上表の「実行 Step」列は **Step 0/1 通過後にどの検査・修正 Step (4 以降) を実行するか**を指し、lite でも Step 0/1 は飛ばさない。**Step 6 (dead-mock 削除)** は Ruby PR で `delegate :X` / `def X` 撤去を含む場合のみ実行 (tier 問わず)。**Step 9 (判断申し送りの集約)** は (Step 0 で中止した場合を除き) tier 問わず必ず実行 (フロー最終出力のため lite でも省略不可)。リスク領域 (auth / billing / payment / migration) は LoC によらず **deep**。
## Manual Review Items (自動修正せず提案のみ → Step 9 で集約)
以下は本 skill が検出しても自動修正せず、Step 9 の「ユーザー判断が必要な項目」に集約する (auto-fix に