执行摘要

• 一句话：将指标报告从SchedulerMetricsMixin迁移到SchedulerMetricsReporter
• 推荐动作：值得精读。该 PR 展示了如何通过系统化的机械操作安全地将职责从大型 Mixin 中剥离，同时保持行为不变。重点关注：
  • @staticmethod 转为实例方法时的签名简化
  • 调用点替换模式（self.method_A(reporter, ...) → reporter.method_A(...)）
  • 测试从继承 mixin 的桩类迁移为直接构造目标组件

功能与动机

作为Scheduler类去Mixin化重构的一部分，将指标报告逻辑从多重继承的Mixin中剥离，降低Scheduler类的复杂度和MRO深度，使指标报告的职责边界更清晰、更易独立测试。PR描述为'纯机械的复制粘贴操作'（Mechanical cut + paste），确保无逻辑变更。

实现拆解

1. 迁移数据与逻辑：将 scheduler_metrics_mixin.py 中的 PrefillStats dataclass、模块级常量（RECORD_STEP_TIME、LOG_FORWARD_ITERS、ENABLE_METRICS_DEVICE_TIMER）以及 SchedulerMetricsMixin 类内所有 @staticmethod 方法（_init_metrics、_install_device_timer_on_runners、_init_fpm 等）整体复制到 metrics_reporter.py 的 SchedulerMetricsReporter 类中，去除 @staticmethod 装饰器，简化 self 类型标注。
2. 更新调用点：在 scheduler.py、scheduler_output_processor_mixin.py、dllm/mixin/scheduler.py、disaggregation/prefill.py 中，将所有 self.<method>(self.metrics_reporter, ...) 形式的调用替换为 self.metrics_reporter.<method>(...)。在 __post_init__ 中，原 SchedulerMetricsMixin._init_metrics(self, ...) 改为 self._init_metrics(...)。
3. 删除旧 mixin 并调整导入：删除 observability/scheduler_metrics_mixin.py 文件。将 scheduler.py、schedule_batch.py、dllm/mixin/scheduler.py 中对旧模块的导入全部更新为从 scheduler_components/metrics_reporter 导入相应符号。从 Scheduler 类的基类列表中移除 SchedulerMetricsMixin。
4. 同步测试文件：在 test_forward_pass_metrics.py 中，将 _DummyScheduler(SchedulerMetricsMixin) 替换为 _make_reporter 工厂函数，直接构造 SchedulerMetricsReporter 实例，并更新所有 mock 路径至新模块。

关键文件：

• python/sglang/srt/observability/scheduler_metrics_mixin.py（模块 可观测性；类别 source；类型 deletion；符号 PrefillStats, from_adder, SchedulerMetricsMixin, _init_metrics）: 被删除的源Mixin文件，所有指标报告逻辑从此迁出。
• python/sglang/srt/managers/scheduler_components/metrics_reporter.py（模块 可观测性；类别 source；类型 dependency-wiring；符号 PrefillStats, from_adder, _init_metrics, _wrap_execution_reporter）: 迁移后的目标文件，承担了所有指标报告职责，是变更的核心。
• python/sglang/srt/managers/scheduler.py（模块 调度器；类别 source；类型 dependency-wiring）: 修改了类继承关系和调用点，是迁移的主要消费者。
• test/registered/unit/observability/test_forward_pass_metrics.py（模块 测试；类别 test；类型 test-coverage；符号 _DummyScheduler, _make_reporter）: 测试文件同步更新，展示了组件化的测试模式。

关键符号：_init_metrics, _install_device_timer_on_runners, _init_fpm, _fpm_device_timer_reporter, _build_scheduled_request_metrics, report_prefill_stats, update_spec_metrics, report_decode_stats, log_batch_result_stats, reset_metrics, update_device_timer, reset_device_timer_window, _shutdown_fpm

评论区精华

无 review 评论。该 PR 被标记为纯机械迁移，团队未产生设计争议，变更被直接合并。

• 暂无高价值评论线程

风险与影响

• 风险：主要风险在于调用点重写可能遗漏。虽然操作是系统化的前缀替换（self.<method>(self.metrics_reporter, ...) → self.metrics_reporter.<method>(...)），但若存在条件分支或动态 dispatch 未覆盖，则可能导致 TypeError 或 AttributeError。此外，scheduler_metrics_mixin.py 被删除后，任何外部依赖该模块的代码（如插件或自定义 runner）将立刻失效。不过该模块并非公开接口，内部影响可控。
• 影响：
  • 用户：零影响，指标报告的行为与输出完全不变。
  • 系统：Scheduler 类的 MRO 缩短，初始化流程略微变化（不再通过 Mixin 的 __init__ 间接初始化），但逻辑等价。CPU/GPU 指标收集与上报路径不变。
  • 团队：后续在指标模块添加新功能只需修改 SchedulerMetricsReporter，无需改动 Scheduler 核心类。测试可以独立构造 SchedulerMetricsReporter 实例，不再需要 mixin 的继承或猴子 patch，可测试性显著提升。
  • 风险标记：核心路径变更, 删除旧接口

关联脉络

• PR #25631 Move idle-metrics logging to SchedulerMetricsReporter: 后续步骤，将空闲指标日志移动至已创建的 SchedulerMetricsReporter
• PR #25638 Move module-level helpers out of scheduler.py: 同一重构链中的模块级辅助函数移动，进一步简化 scheduler.py
• PR #25639 Delete the now-unused is_work_request from scheduler.py: 同一重构链中的清理步骤，删除无用函数