simplify

簡素化

完成した変更を保守しやすくするため、動作を保ったまま最小限のクリーンアップを行います。コードを減らし、動作はそのまま。それがすべてです。

1. 変更の範囲を定める

発端となった依頼、レビューのベースライン、変更されたパス、現在の差分またはコンテンツハッシュを記録します。提案するクリーンアップについて、差分全体と周辺のコードをすべて読みます。

クリーンアップの範囲は、現在のタスクの差分です。無関係な既存コードには触れず、その旨をゲートとして明示します。範囲を逸脱しません。

2. 削除可能な複雑さを見つける

次の項目だけを調べます。

• 変更によって生じた重複。
• 依頼に不要な抽象化、分岐、設定、エラー処理。
• 変更によって生じた不要なインポート、変数、関数、ファイル、または間接処理。
• 公開契約を変えずに、より明確にできる制御フローや命名。
• 以下の追加コメントチェックに違反するコメント。

新しいヘルパーを追加するより、削除して直接的なコードにすることを優先します。分岐を減らせる場合は、既存のローカルなパターンを再利用します。必要なガード、根拠、復旧動作、テスト、ユーザーが確認した決定は維持します。

追加コメントチェック

グローバルなコメントルールに従って、変更で追加されたすべてのコメントを調べます。コメントを残せるのは、エクスポートされたシンボルに付ける1行の /** */ 契約コメントか、URL、仕様書のセクション、ADR または docs/ のパス、あるいは issue 番号を含む1行の引用コメントだけです。それ以外はすべて、コードの動作やその形の理由、あるいは何が壊れるかを説明するコメントも含め、見つけ次第削除します。ルールの grep を差分に対して実行し、出力されたコメントをすべて削除します。プロジェクトのルールでチェックを厳しくすることはできますが、緩めることはできません。

提案する編集にはそれぞれ、何がよりシンプルになるのか、また観察可能な動作が同等のままなのはなぜかを明記します。変更によって追加された、引用のないコメントは一切残しません。このチェックは必須です。差分にコメントが1つでも残っている間は、呼び出し元は作業完了を報告できません。

3. 適用して検証する

コードと既存のテストによって同等性を裏付けられるクリーンアップだけを適用します。クリーンアップによって動作、要件、公開出力、永続化、認可、または失敗時の意味が変わる場合は適用せず、別の提案として返します。

クリーンアップによって無効になったチェックを再実行し、最終差分を発端となった依頼と比較します。完了済みのレビュー対象となったコンテンツをクリーンアップで変更した場合は、converge-reviews を通じて該当するレビューだけを無効化し、再実行します。古い収束結果では、簡素化した差分を検証できません。行った編集内容、または No simplification needed と、検証結果の詳細を報告します。

差分が依頼の範囲を超えず、残されたすべての行にタスク上の理由があり、該当するチェックに引き続き合格したとき、差分の作業完了と判断します。