simplify

Simplify

概要

レビュー前のクリーンアップゲートとして使用します。正式な承認を行うことが目的ではありません。変更されたコードを調べて、再利用の機会、コード品質の問題、効率上の問題を見つけ、動作を変更せずに価値のある指摘を修正します。

使用するタイミング

• reviewer または code-reviewer によるレビューの前に、/simplify のようなクリーンアップを行いたい場合
• 機能が完成し、差分に重複や粗い部分、回避可能な非効率が含まれている可能性がある場合
• 正式なレビュー前に、回避可能なレビュアーからの指摘を減らしたい場合
• 変更が十分な数のファイルやロジックに及び、簡素化を一度行うだけでは不十分な場合

次の場合、このスキルは使用しないでください。

• 差分がごく小さく、局所的なクリーンアップだけで十分な場合
• 要件を変更する動作変更、機能追加、バグ修正を行う場合
• 変更していないモジュールを含む大規模なアーキテクチャの再設計を行う場合
• 仕様の正式な検証やセキュリティレビューを行う場合

レビューフロー内での位置

推奨される配置：

implement -> simplify -> code-reviewer and/or reviewer -> receiving-code-review

次の場合は、3 エージェントによる一連のレビューを省略してください。

• 変更がごく小さく、明白な場合
• 変更された行が 1、2 行だけで、内容も些細な場合
• クリーンアップではなく、すぐに正式なレビューを行いたい場合

ワークフロー

フェーズ 1：変更点を特定する

• ユーザーから base/head の範囲が指定されている場合は、それを使用します
• ステージ済みの変更がある場合は、ステージ済みと未ステージの変更を両方含めるために git diff HEAD を調べます
• それ以外の場合は、git diff を使用します
• git diff がない場合は、ユーザーが指定した、またはセッション中に以前編集したファイルのうち、最も最近変更されたものをレビューします
• 変更したファイルの簡潔な概要と、レビュー担当者向けの差分全文を保持します

フェーズ 2：ローカルの規約を確認する

• まず AGENTS.md、リポジトリの規約、関連するスキルなどのローカル指示を確認します
• 変更したファイルでは、既存の命名、書式、構成に従います
• ローカルの規約が一般的なクリーンアップの好みと矛盾する場合は、ローカルの規約を優先します

フェーズ 3：3 つのレビューエージェントを並行して起動する

中規模または大規模な差分では、1 回のラウンドで 3 つの読み取り専用サブエージェントを並行して起動します。実装ではなくコードベースの分析を行うため、可能であれば explorer エージェントを使用します。

各エージェントには次の内容を渡す必要があります。

• 差分全文
• 変更したファイルの一覧
• クリーンアップの制約：動作変更や編集は行わず、指摘のみを行う

3 つのレビュー領域は次のとおりです。

• コード再利用のレビュー
• コード品質のレビュー
• 効率性のレビュー

次のプロンプトテンプレートを使用します。

• references/reuse-reviewer.md
• references/quality-reviewer.md
• references/efficiency-reviewer.md

重要：

• レビューエージェントは読み取り専用で、ファイルを編集しません
• メインエージェントが指摘を集約し、修正を行います
• 同じ変更ファイルを複数の作業エージェントが並行して編集することは避けてください

フェーズ 4：価値のある問題を修正する

• すべてのレビュアーが完了するまで待ちます
• 指摘を集約し、重複を取り除きます
• 動作を変更せずにコードを改善できる場合は、問題を直接修正します
• 指摘が誤検出であるか、変更に見合わない場合は、その旨を記録して、長々と議論せずに見送ります

フェーズ 5：検証する

• 変更した範囲に対して、意味のある最小限の検証を実行します
• まず対象を絞ったテストを優先し、必要に応じて lint/build を実行します
• 自動検証がない場合は、そのことを明記し、残るリスクにも触れます
• 修正内容を簡潔にまとめるか、コードがすでに十分に整理されていたことを伝えます

クイックリファレンス

目標	推奨	避けること
可読性	明確な名前、明示的な分岐、小さくまとまりのあるヘルパー	密度の高いワンライナー、巧妙すぎるテクニック
範囲の管理	変更したファイル、最近の差分、ユーザーが指定したパス	依頼なしでリポジトリ全体をクリーンアップすること
安全性	編集後に対象を絞って検証すること	リファクタリングが安全だと思い込むこと
コメント	意図と制約	明らかなコードの説明
エージェントの使用	3 つの読み取り専用レビュアーを並行して起動し、1 つのコントローラーが修正を適用すること	同じファイルに複数の編集エージェントが変更を加えること

よくある間違い

• 簡素化を再設計に変えてしまう
• 抽象化を早く導入しすぎる
• 無関係なファイルまで広範囲に名前を変更する
• 保守しやすさより行数の少なさを優先する
• 変更が「機械的に見える」という理由で検証を省略する
• 3つすべてのエージェントにコードを編集させ、調査結果の報告だけにとどめない
• このスキルを正式なレビューや仕様の検証の代わりに使う

上流の参照先

この Codex スキルは、Claude ベースのワークフローで使われる /simplify フローを、Codex ネイティブのスキルに適応したものです。出典のメモと適応の詳細については、references/upstream-agent.md を参照してください。