执行摘要

• 一句话：移除防御性 getattr，直接访问属性
• 推荐动作：值得精读：这是一个教科书级的机械重构案例，展示了如何安全地将防御性 getattr 替换为直接属性访问，提升代码健壮性。可作为团队代码清理的参考。

功能与动机

PR body 明确说明：这些属性在 __init__ 中被无条件设置，因此 getattr 的 fallback 默认值是死代码。直接访问能在未来重构时立即暴露 AttributeError，而非静默返回错误默认值，从而避免下游逻辑分支错误。

实现拆解

1. 在 tokenizer_manager_score_mixin.py：将 is_generation = getattr(self, "is_generation", True) 和 model_config = getattr(self, "model_config", None) 分别替换为 self.is_generation 和 self.model_config。
2. 在 scheduler_output_processor_mixin.py：将 if getattr(self, "enable_hicache_storage", False): 替换为 if self.enable_hicache_storage:。
3. 在 scheduler_pp_mixin.py：将 max_chunk_size = getattr(self, "max_prefill_tokens", None) 替换为 self.max_prefill_tokens。

关键文件：

• python/sglang/srt/managers/tokenizer_manager_score_mixin.py（模块 评分管理；类别 source；类型 core-logic）: 核心变更文件，修改了 3 个 getattr 调用，影响 score 请求处理流程中的分支判断和模型配置获取。
• python/sglang/srt/managers/scheduler_output_processor_mixin.py（模块 调度输出；类别 source；类型 core-logic）: 修改了 enable_hicache_storage 属性的访问方式，影响 L3 存储是否启用的判断。
• python/sglang/srt/managers/scheduler_pp_mixin.py（模块 调度PP；类别 source；类型 core-logic）: 修改了 max_prefill_tokens 属性的访问方式，影响动态分块大小的预测。

关键符号：_process_single_item_scoring_results, score_request, _get_cached_tokens_details, predict_next_chunk_size

关键源码片段

python/sglang/srt/managers/tokenizer_manager_score_mixin.py

核心变更文件，修改了 3 个 getattr 调用，影响 score 请求处理流程中的分支判断和模型配置获取。

# 变更前：使用 getattr 防御性访问，不存在的属性静默返回默认值
# is_generation = getattr(self, "is_generation", True)
# model_config = getattr(self, "model_config", None)
​
# 变更后：直接访问属性，若属性不存在则立即抛出 AttributeError
is_generation = self.is_generation
model_config = self.model_config

python/sglang/srt/managers/scheduler_pp_mixin.py

修改了 max_prefill_tokens 属性的访问方式，影响动态分块大小的预测。

# 变更前：getattr(self, "max_prefill_tokens", None)
# 变更后：直接访问 self.max_prefill_tokens
max_chunk_size = self.max_prefill_tokens
predicted_size = self.length_predictor.predict_next_chunk_size(
    history_len=history_len,
    base_chunk_size=self.chunked_prefill_size,
    page_size=self.page_size,
    context_len=self.model_config.context_len,
    max_chunk_size=max_chunk_size,
)

评论区精华

该 PR 没有 review 评论，无讨论亮点。

• 暂无高价值评论线程

风险与影响

• 风险：风险极低。变更的 4 个属性均在 __init__ 中无条件初始化，因此替换为直接访问是安全的。潜在风险：如果未来某个属性在特定条件下未被初始化（例如子类覆盖了 __init__），则原本静默返回默认值的行为会变为抛出 AttributeError。但这是改进而非风险，因为错误能被尽早发现。
• 影响：影响范围小，只涉及 3 个文件、4 行代码变更。无功能变化，仅改变错误暴露方式。用户无感知。
• 风险标记：低风险重构, 缺少测试覆盖

关联脉络

• PR #25442 Lift forward_ct/cur_batch and use direct access in watchdog: 同样属于移除防御性 getattr 的系列重构，由同一作者在同一重构链中完成。
• PR #25437 Drop dead hasattr guards (hisparse_coordinator, metrics_collector): 类似的重构，删除冗余的 hasattr 守卫。
• PR #25436 Cache _linear_attn_registry_cache with sentinel: 替换另一个 hasattr 惰性初始化模式。