code-simplifier

コードの簡素化

動作が正しいことを確認した後に実行します。リポジトリの指示と、正確性または言語の専門家による指示を優先してください。出力スタイルに関するスキルは、提示方法にのみ影響します。

対象範囲

ベースとの差分にある現在の変更のみを調べます。この変更によって不要になったコード、または変更によって事実と異なる内容になったテキストを修正する場合に限り、差分の外にある参照先も確認してください。

リポジトリの指示と、languages/ にある該当するガイドを読みます。

• TypeScript：languages/typescript.md
• JavaScript：languages/javascript.md
• Go：languages/go.md
• Rust：languages/rust.md
• Python：languages/python.md
• CSS：languages/css.md

手順

• API、シグネチャ、出力、動作を維持します。
• 不要な間接化、1 か所でしか使わない抽象化、重複する分岐、コードの言い換えにすぎないコメントを削除します。
• 気の利いた式より、分かりやすい文を優先します。
• 抽出によって動作が分かりにくくなる場合は、意図的な小規模の重複を残します。
• 依存関係を追加する代わりに、標準ライブラリを使用します。
• 呼び出し側から必要とされない限り、可視性は非公開にします。
• 変更によって事実と異なる内容になったコメント、ドキュメント、エラー、名前を更新します。
• 何を削除、改名したか、以前と異なる方法で行ったことなど、変更内容を説明するコメントは追加しないでください。それらはコミットメッセージまたは PR の説明に記載する内容です。そのようなコメントがあれば削除してください。
• コメントは短く保ち、隠れた制約や自明でない理由など、コードから分からないことだけを記載します。見れば分かるコードの説明は避けてください。
• 一見使われていないコードを削除する前に、リポジトリ内を検索してください。テスト、登録、リフレクション、設定キー、公開されている利用者を考慮してください。
• 正当なリファクタリング後のテストカバレッジを維持するために必要な場合に限り、テストを更新します。アサーションを弱めたり、期待される動作を変えたりしないでください。

インポートの順序を変更したり、無関係なクリーンアップを差分に混ぜたり、動作している API を再設計したりしないでください。簡素化によって差分が大幅に拡大する場合は、コードに手を加えず、その理由を説明してください。

実装ですでに実行した検証を使用します。この手順で変更した内容が実施済みの検証でカバーされていない場合に限り、対象を絞った追加チェックを実行してください。ステージングやコミットは決して行わないでください。