# Codex Review Fix Loop

> Codexのコードレビューと修正を繰り返す。Findingsを重大度順で扱い、毎ループで検証を行う。

- Skill: `satoru314/codex-review-fix-loop` (Agent Skill)
- Install (CLI): `npx skillmds@latest add satoru314/codex-review-fix-loop`
- Raw SKILL.md: https://api.skillmd.com/api/skills/satoru314/codex-review-fix-loop/raw
- Safety review: pending
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Coding & Dev Tools
- Author: Satoru314 (https://skillmd.com/u/satoru314)
- Updated: 2026-09-22
- Page: https://skillmd.com/skills/satoru314/codex-review-fix-loop

---


# Review Fix Loop

変更に対してレビューと修正を反復し、問題がなくなるまで進めるためのSkill。

## ゴール

- `High` / `Medium` の指摘を 0 件にする。
- 可能な範囲で `Low` も解消する。
- 毎ループで検証結果を残す。

## 実行ルール

- ユーザー確認は最小限にし、通常の編集・コミットは継続して実施する。
- レビューは常に Findings 先出し、重大度順で扱う。
- 1ループで止めず、修正後に必ず再レビューする。
- 推測で断定せず、必ず根拠ファイルを示す。
- 開始時にレビュー対象を必ず確認する。

## 事前確認

1. ユーザーにレビュー対象を確認する。
   - `未コミット全体`（staged + unstaged）
   - `stagedのみ`（`git diff --cached`）
   - `unstagedのみ`（`git diff`）
2. `docs/README.md` を確認する。
3. 変更対象に応じて `docs/dev_rules/` を確認する。
4. 1で選ばれた範囲の差分を取得して対象範囲を確定する。

## ループ手順

1. レビュー実行
   - 差分中心でレビューし、次を優先して確認する。
   - 正しさ、仕様逸脱、セキュリティ、データ整合性、回帰リスク、テスト不足

2. Findings を整理
   - `High` / `Medium` / `Low` で分類する。
   - 各指摘に `Location` `Issue` `Why` `Fix` を付ける。

3. 修正実施
   - `High` → `Medium` → `Low` の順で対応する。
   - 1つの修正で他の観点を壊していないか、関連箇所も確認する。

4. 検証
   - 可能な限り次を実行する。

```sh
npm run typecheck
npm run build
npm run lint
npm run test
```

5. 再レビュー
   - 修正差分を再レビューし、新規問題と未解決問題を更新する。
   - `High` / `Medium` が残る限り手順3に戻る。

## 完了条件

- レビュー結果が `重大な指摘はありません` になる。
- 具体的には `High=0` かつ `Medium=0`。
- `Low` が残る場合は理由と影響を明記してクローズ可。

## 出力フォーマット

- ループごとに以下を報告する。
  - `Loop`: 回数
  - `Findings`: 重大度順の一覧
  - `Fix Applied`: 実施した修正
  - `Validation`: 実行コマンドと成功/失敗
  - `Next`: 次ループで対応する内容

## 失敗時の扱い

- コマンド失敗時は代替手段を試す。
- 代替不能なら失敗理由、試した手段、必要な入力を明確に報告する。