roblox-code-maintenance-refactor

Roblox コード保守リファクタリング

目標

プレイヤー向けの動作を意図せず変更することなく、コードの健全性を高めます。

リファクタリングのルール

• 編集する前に読みます。
• 変更の範囲を絞ります。
• 意図的に移行する場合を除き、リモートの契約を維持します。
• 動作の変更と整理を混在させません。
• 意味のあるリファクタリングの後にプレイテストを実施します。
• コンソール出力を確認します。

対象

• 重複した定数や報酬計算式。
• 複数の役割が混在する大規模なスクリプト。
• クライアントとサーバーの境界に関する混乱。
• 使用されていないリモート、古い UI パス、未使用のモジュール。
• 繰り返されるイベント接続パターン。
• 重要な状態を示す名前が不明瞭な箇所。
• バランステーブルにすべきマジックナンバー。

深化の機会を探す

明らかな整理にとどまらず、浅いモジュールも探して、より深いモジュールにしましょう。インターフェースが実装とほぼ同じくらい複雑で、隠すべき複雑さを呼び出し側が抱えているモジュールです。

• ある概念を理解するために、数多くの小さなスクリプトを行き来する必要がある箇所はどこですか？
• 純粋なヘルパーを整理のためだけに切り出したものの、実際のバグはその呼び出し方にある（局所性がない）箇所はどこですか？
• クライアントとサーバーが責任を共有し、その責任がリモートの境界を越えて漏れている箇所はどこですか？

浅いと思われるものには削除テストを適用します。削除すると複雑さが別の場所に移るだけならインライン化し、同じ複雑さが多数の呼び出し箇所に散らばるなら、そのモジュールを維持し、ゲーム内で使われている用語に基づいた、小さく安定したインターフェースを設けます。それぞれの機会について、次の形式で提示します。関連ファイル · 現在の問題点 · 提案する変更 · メリット（局所性＋テスト容易性）。単一の実装に対して、新たな間接化を提案しないでください。深いモジュールのモデルについては、roblox-luau-architecture を参照してください。

MCP ワークフロー

1. script_grep を使って、重複、古い名前、呼び出し箇所を探します。
2. script_read を使って、影響を受けるスクリプトをすべて読みます。
3. multi_edit で小さな編集を行います。
4. start_stop_play を使って、関連するループを検証します。
5. get_console_output を確認します。

出力

以下を返します。

• リファクタリングの目標。
• 維持された動作。
• 変更したスクリプト。
• 実施した検証。
• 今後対応する技術的負債。

次の手順

roblox-playtest-qa で動作が変わっていないことを確認します。大規模な構造変更については、roblox-luau-architecture で設計し、roblox-to-issues で安全な小さなステップごとに実施できるよう分割します。