Satoru314 Review Fix Loop
変更に対してレビューと修正を反復し、問題がなくなるまで進めるためのSkill。
ゴール
High/Mediumの指摘を 0 件にする。- 可能な範囲で
Lowも解消する。 - 毎ループで検証結果を残す。
実行ルール
- ユーザー確認は最小限にし、通常の編集・コミットは継続して実施する。
- レビューは常に Findings 先出し、重大度順で扱う。
- 1ループで止めず、修正後に必ず再レビューする。
- 推測で断定せず、必ず根拠ファイルを示す。
- 開始時にレビュー対象を必ず確認する。
事前確認
- ユーザーにレビュー対象を確認する。
未コミット全体（staged + unstaged）stagedのみ（git diff --cached）unstagedのみ（git diff）
docs/README.mdを確認する。- 変更対象に応じて
docs/dev_rules/を確認する。 - 1で選ばれた範囲の差分を取得して対象範囲を確定する。
ループ手順
レビュー実行
- 差分中心でレビューし、次を優先して確認する。
- 正しさ、仕様逸脱、セキュリティ、データ整合性、回帰リスク、テスト不足
Findings を整理
High/Medium/Lowで分類する。- 各指摘に
LocationIssueWhyFixを付ける。
修正実施
High→Medium→Lowの順で対応する。- 1つの修正で他の観点を壊していないか、関連箇所も確認する。
検証
- 可能な限り次を実行する。
npm run typecheck
npm run build
npm run lint
npm run test
- 再レビュー
- 修正差分を再レビューし、新規問題と未解決問題を更新する。
High/Mediumが残る限り手順3に戻る。
完了条件
- レビュー結果が
重大な指摘はありませんになる。 - 具体的には
High=0かつMedium=0。 Lowが残る場合は理由と影響を明記してクローズ可。
出力フォーマット
- ループごとに以下を報告する。
Loop: 回数Findings: 重大度順の一覧Fix Applied: 実施した修正Validation: 実行コマンドと成功/失敗Next: 次ループで対応する内容
失敗時の扱い
- コマンド失敗時は代替手段を試す。
- 代替不能なら失敗理由、試した手段、必要な入力を明確に報告する。