roblox-code-maintenance-refactor

Roblox 代码维护重构

目标

改善代码健康度，同时避免意外改变玩家可见行为。

重构规则

• 编辑前先阅读。
• 控制改动范围。
• 除非有意迁移，否则保留远程调用契约。
• 避免将行为变更与清理混在一起。
• 进行有意义的重构后进行试玩测试。
• 检查控制台输出。

目标

• 重复的常量和奖励公式。
• 职责混杂的大型脚本。
• 客户端／服务端边界不清。
• 无用的远程调用、过时的 UI 路径、未使用的模块。
• 重复的事件连接模式。
• 重要状态的命名不清晰。
• 应改用平衡表的魔法数字。

寻找深化机会

除了显而易见的清理工作，还要寻找可以深化的浅层模块——这类模块的接口几乎和实现一样复杂，导致调用方承担了本应隐藏的复杂性。

• 要理解一个概念，是否需要在许多小脚本之间来回切换？
• 是否只是为了整洁而提取了纯辅助函数，但真正的 bug 出在调用方式上（缺乏内聚性）？
• 客户端和服务端是否承担了相同的职责，并导致职责跨越远程调用边界？

对所有怀疑属于浅层模块的对象应用删除测试：如果删除它只是转移了复杂性，就将其内联；如果删除它会导致相同的复杂性散落在许多调用点，就保留它，并为其提供一个以游戏术语命名的小型稳定接口。逐项说明每个机会：涉及的文件 · 当前的摩擦 · 拟议的变更 · 带来的好处（内聚性 + 可测试性）。不要为单一实现引入新的间接层。深层模块模型请参阅 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 将其拆分，确保每次只落地一个安全步骤。