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」列は preflight 通過後の話**で、lite でも Step 0 は飛ばさない。**Step 6 (dead-mock 削除)** は Ruby PR で `delegate :X` / `def X` 撤去を含む場合のみ実行 (tier 問わず)。**Step 9 (判断申し送りの集約)** は (Step 0 で中止した場合を除き) tier 問わず必ず実行 (フロー最終出力のため lite でも省略不可)。リスク領域 (auth / billing / payment / migration) は LoC によらず **deep**。
## Quick start
0. **preflight (全 tier 必須・最初に実行)**: `feature-dev` plugin (Step 8 の `feature-dev:code-reviewer` が依存) の導入を確認。未導入なら**インストール方法を提示して即終了**し、以降の Step を一切実行しない (下記 Workflow Step 0)。
1. 引数 `$ARGUMENTS` あり → そのファイルを対象。なし → `git diff --name-o