Simplify: コードレビューとクリーンアップ
変更されたすべてのファイルを、再利用性、品質、効率性の観点からレビューします。見つかった問題を修正します。
フェーズ1: 変更の特定
git diff(またはステージングされた変更がある場合は git diff HEAD)を実行して、変更内容を確認します。
フェーズ2: レビュー
コードの再利用
- 新しいコードを置き換えられる既存のユーティリティを検索します。
- 重複する機能をフラグ立てし、既存の関数を提案します。
- 既存のユーティリティを使用できるインラインロジックをフラグ立てします。
コード品質
- 導出可能な冗長な状態またはキャッシュ値。
- わずかなバリエーションでのコピーペースト — 共有抽象化で統一します。
- コードが「何を」行うかを説明する不要なコメント。
- 定数やenumが既に存在する生の文字列。
効率性
- 冗長な計算、繰り返されるファイル読み込み。
- 並列実行可能なのに逐次実行されている独立した操作。
- 操作前の不要な存在確認(TOCTOU)。
フェーズ3: 修正
各問題を直接修正します。誤検出の場合は、それを記録して次に進みます。 完了したら、修正した内容を簡潔に要約してください(またはコードが既にクリーンであることを確認してください)。
動作を維持してください。小さな機械的なリファクタリングを優先します。無関係な書き換えは避けてください。