radixark/miles

重构 ServerCell 状态转换，通过单元级辅助方法统一管理所有引擎的状态变更。

此 PR 是 **#1837 重构计划**中一个逻辑自洽、目标明确的步骤。虽然改动不大，但它展示了通过封装和提取方法来显式化系统不变量的优秀实践，其设计思路和重构模式值得学习。对于关心 `rollout` 模块架构演进或 Ray Actor 生命周期管理的工程师，推荐精读。需要特别注意其与后续 PR（如 #2112, #2115）的关联，以理解对潜在风险（如启动失败）的完整解决方案。

提取 Rollout 状态机到独立模块，提升代码组织和可维护性。

本 PR 是一个有价值的重构，值得精读以理解状态机的设计和模块化决策。作为系列重构的一部分，建议结合 Issue #1837 的其他 PR（如 #2048、#2049）一起查看，以把握整体演进方向。

将路由器注册逻辑从 ServerGroup 下沉到 ServerCell，统一使用单元格索引管理。

**值得精读**。这个 PR 是一次高质量、目标明确的内部重构，虽然改动文件不多，但设计决策和实现方式值得学习。它清晰地展示了如何通过将逻辑下沉到数据模型（`ServerCell`）来简化高层协调者（`ServerGroup`）的复杂性。PR 的提交信息清晰解释了动机，测试覆盖充分，审核意见确认了重构的一致性和正确性。对于理解该仓库的 `rollout` 模块架构演进方向很有价值。

将引擎启动配置从 ServerGroup 移入 ServerCell，简化 start_engines 接口并增强数据局部性。

建议精读此 PR，因为它展示了通过数据局部性简化接口的实用重构模式。关键设计决策是将配置下沉到更细粒度的细胞单元，这增强了封装性并减少了参数传递。测试覆盖充分，风险可控。

重构测试 Mock 引擎，使其在自身分配的端口上提供 HTTP 服务，提升测试保真度。

此 PR 是有价值的测试基础设施改进，它遵循了“测试应尽可能接近生产环境”的最佳实践。**建议精读**，特别是 MockSGLangHttpServer 中连接跟踪和端口绑定的实现细节，以及测试夹具简化后的模式。设计决策上，将 HTTP 服务器生命周期与 Actor 绑定是正确的方向，但 **必须关注并跟进 review 中提出的 IP 地址硬编码问题**，这是一个在后续测试扩展中必须解决的技术债。

重构 `PortAllocator`，引入 `alloc` 方法，简化端口分配与状态管理。

建议精读此 PR。这是一个高质量的重构，展示了如何通过封装状态操作来简化复杂的分配逻辑。值得学习的设计点包括：1) 将分散的状态管理（`node_port_cursor` 和返回新 `PortAllocator`）内聚到一个明确的 `alloc` 方法中；2) 使用 `functools.partial` 清晰地创建特定于上下文（`engine` 和 `node_ip`）的端口分配回调；3) 配套的测试策略从测试辅助方法（`assign`/`next_base_port`）转向测试核心行为（`alloc` 的各种场景）。 此 PR 是 Issue #1837 这一宏大重构目标中的一个坚实步骤，其编码质量和测试覆盖度值得借鉴。

将端口分配器内部状态键从节点索引改为节点 IP，提升身份标识的准确性。

这是一个清晰、内聚的重构 PR，值得精读。其核心决策——将端口分配状态的键从代理的节点索引改为真实的节点 IP——体现了对系统健壮性的主动改进。虽然逻辑简单，但它是整个 Rollout 模块规范化、健壮化进程中的必要步骤，可以作为代码重构中“改善内部表示”的范例学习。

将 cell 管理接口从 engine 索引切换到 cell 索引，简化启动/停止/恢复操作的接口。

此 PR 值得精读。它展示了如何通过接口简化（从引擎索引到单元格索引）来提升代码的内聚性和可理解性，是模块化重构的良好实践。核心设计决策（`_engine_indices_of_cell` 的引入、`CellIndexer` 的简化、`recover` 断言的移除）值得借鉴。作为 #1837 重构链的一环，它清晰地展示了该功能线的演进方向。