radixark/miles

提取 Ray actor 构造函数至 `server_cell.py`，分离引擎创建与槽位选择职责。

该 PR 是一个**设计良好、目标明确且执行干净的重构**。它属于一个更大的模块化重构栈（#1837），其核心价值在于**改善了代码的模块职责划分**，为系统架构的下一步演进（如更细粒度的引擎生命周期管理）奠定了坚实基础。建议**精读**此 PR，重点关注 `launch_sglang_ray_actor` 函数如何封装复杂的 actor 创建细节，以及它如何简化 `ServerGroup` 的逻辑。审核者关于“保持重构纯粹性、隔离 RDT 逻辑”的评论也体现了良好的工程实践，值得借鉴。

将引擎关闭逻辑从 ServerGroup 下沉至 ServerCell，提升模块封装性与职责清晰度。

这是一个清晰、有价值且风险可控的重构 PR。它很好地遵循了单一职责原则，将特定层次的逻辑封装到对应的模块中。PR 改动集中，有配套测试更新，且获得了审核者的积极反馈。对于维护 Rollout 模块代码质量的工程师而言，此 PR 值得精读，以理解模块化重构的具体手法。其将协调与执行分离、利用数据结构封装行为的设计模式具有参考价值。

将内存释放、恢复和权重检查操作从 ServerGroup 下沉到 ServerCell，实现职责更精确的封装。

此 PR 值得精读。它是一个典型的、执行良好的职责下沉重构，变更集中、意图清晰、测试充分。其设计决策（明确 `primary_engine` 的概念并将其操作封装）对于理解整个 rollout 模块的架构演进（尤其是与近期一系列如 #2048、#2006、#1999、#1997 等重构 PR 的脉络）至关重要。它展示了在复杂系统中通过渐进式重构改善代码结构的良好实践。

将 RolloutExecutor.generate 方法重命名为 get，以准确反映其职责。

对于关注系统架构演进或参与相关模块开发的工程师，建议精读此 PR，以理解团队在命名约定和职责划分上的意图。对于日常功能开发，此 PR 影响有限，但可作为未来相关重构的参考。

将 RolloutManager 拆分为 InferenceController 与 RolloutExecutor，并将控制器 API 异步化。

这是一个需要精读的架构级重构 PR。建议关注：1) `InferenceController` 异步 API 的设计及其对训练循环时序的影响；2) `RolloutComponents` 的构建顺序如何确保依赖（如路由器地址）正确传递；3) 测试如何验证新组件间的职责边界。鉴于其高风险和跨模块影响，合并后需密切监控所有训练作业。

将 RolloutManager 拆分为 InferenceController 与 RolloutExecutor，实现职责分离。

此 PR 是一个有意义的架构重构，值得精读，尤其关注两个新类之间的职责边界划分是否清晰合理。设计决策上，将推理基础设施管理与数据执行流分离，符合单一职责原则，是合理的架构演进。建议结合后续 PR#1843 一起阅读，以获得完整的重构图景。

将 CI 故障注入等待从同步 sleep 改为异步 await，避免阻塞主线程。

这是一个小规模、目标明确的重构 PR，值得精读。其核心价值不在于改动的复杂性，而在于它清晰展示了如何逐步、安全地将一个阻塞操作异步化。对于关注系统异步化重构或 CI 测试性能优化的工程师，此 PR 提供了一个具体的实施范例。

提取生成和评估过程中的未来推理引擎钩子，为异步场景做准备。

这个PR值得精读，因为它展示了如何通过提取钩子来解耦核心逻辑，为未来架构演进做准备。设计决策（如将准备逻辑移到单独方法）可借鉴，但注意RDT回滚是临时的，后续PR会重新集成。风险较低，适合快速评审。