执行摘要

• 一句话：统一 server_args 中的环境变量读取方式，从工具函数改为 environ 描述符。
• 推荐动作：该 PR 是一次典型的代码风格重构，值得快速浏览以了解环境变量管理的最佳实践。重点关注 server_args.py 中环境变量读取点的统一模式，以及 environ.py 中新增描述符的同步添加。对于涉及类型转换的逻辑（如 NPU fused MOE mode）应仔细验证，但整体风险可控。

功能与动机

根据 PR 标题和 body 描述，动机是替换 server_args.py 中的 get_bool_env_var/get_int_env_var/os.environ 等魔法字符串，改用 sglang.srt.environ 描述符。这符合代码库中环境变量管理向集中式描述符演进的趋势，旨在提高代码的可读性、可维护性和类型安全性。

实现拆解

1. 移除旧工具函数导入并统一访问模式：在 server_args.py 中，从 sglang.srt.utils.common 导入列表中移除 get_bool_env_var 和 get_int_env_var，因为它们不再被使用。同时，将文件中多处直接使用这些函数或 os.environ.get 的环境变量读取，统一替换为 envs.<ENV_NAME>.get() 或 envs.<ENV_NAME>.is_set() 的调用方式。
2. 新增环境变量描述符：在 environ.py 的 Envs 类中，新增两个环境变量描述符：SGLANG_USE_AITER_UNIFIED_ATTN = EnvBool(False) 和 SGLANG_MORI_NUM_MAX_DISPATCH_TOKENS_PER_RANK = EnvInt(4096)，以支持 server_args 中新增的统一访问需求。
3. 类型安全与逻辑调整：在 _handle_a2a_moe 方法中，将 SGLANG_NPU_FUSED_MOE_MODE 的读取从 os.environ.get（返回字符串）改为 envs.SGLANG_NPU_FUSED_MOE_MODE.get()（返回整数），并相应地将比较值从字符串 "1"/"2" 改为整数 1/2，确保类型一致性。
4. 无测试或配置配套改动：本次变更仅涉及源码重构，未发现对应的测试文件、配置或部署脚本的配套改动。

关键文件：

• python/sglang/srt/server_args.py（模块 服务器参数；类别 source；类型 core-logic；符号 _handle_missing_default_values, _handle_model_specific_adjustments, _handle_moe_kernel_config, _handle_a2a_moe）: 这是服务器参数解析的核心文件，本次重构统一了多处环境变量读取逻辑，影响服务器启动和配置处理。
• python/sglang/srt/environ.py（模块 环境变量；类别 source；类型 configuration；符号 Envs）: 环境变量描述符的定义文件，本次新增了两个描述符以支持 server_args 中的统一访问。

关键符号：_handle_missing_default_values, _handle_model_specific_adjustments, _handle_moe_kernel_config, _handle_a2a_moe

关键源码片段

python/sglang/srt/server_args.py

这是服务器参数解析的核心文件，本次重构统一了多处环境变量读取逻辑，影响服务器启动和配置处理。

# 在 _handle_missing_default_values 方法中，统一环境变量读取方式
if envs.SGLANG_USE_MODELSCOPE.get(): # 原为 get_bool_env_var("SGLANG_USE_MODELSCOPE")
    self._handle_modelscope_paths()
​
# 在 _handle_model_specific_adjustments 方法中，调整 AMD ROCm 相关逻辑
elif is_hip() and envs.SGLANG_USE_AITER.get(): # 原为 get_bool_env_var("SGLANG_USE_AITER")
    # 对于 GPT-OSS bf16 on ROCm with aiter，使用 triton 后端
    self.moe_runner_backend = "triton"
​
# 在 _handle_a2a_moe 方法中，处理 NPU fused MOE 模式，并确保类型安全
fuse_mode = envs.SGLANG_NPU_FUSED_MOE_MODE.get() # 原为 os.environ.get("SGLANG_NPU_FUSED_MOE_MODE", None)，返回字符串
if fuse_mode not in [1, 2]: # 现在比较整数，而非字符串 "1" 或 "2"
    raise ValueError(f"Wrong value of {fuse_mode=}, the NPU only support 1 or 2.")
elif fuse_mode == 2: # 原为 "2"
    assert self.quantization == "modelslim", "When fuse_mode is set to 2, the NPU supports only ModelSlim quantization."

python/sglang/srt/environ.py

环境变量描述符的定义文件，本次新增了两个描述符以支持 server_args 中的统一访问。

# 在 Envs 类中，新增 AMD & ROCm 相关的环境变量描述符
class Envs:
    # ... 其他定义 ...
​
    # AMD & ROCm
    SGLANG_USE_AITER = EnvBool(False)
    SGLANG_USE_AITER_UNIFIED_ATTN = EnvBool(False) # 新增：用于统一注意力机制的环境变量
    SGLANG_ROCM_FUSED_DECODE_MLA = EnvBool(False)
    SGLANG_ROCM_DISABLE_LINEARQUANT = EnvBool(False)
    SGLANG_MORI_NUM_MAX_DISPATCH_TOKENS_PER_RANK = EnvInt(4096) # 新增：MoRI 每 rank 最大分发 token 数
​
    # MPS (Apple Silicon)
    SGLANG_USE_MLX = EnvBool(False)
    # ... 后续定义 ...

评论区精华

本次 PR 没有 review 评论，仅有的两条评论是自动化 bot 的配额警告和作者触发 CI 的指令。因此，没有实质性的技术讨论或争议点。

• 暂无高价值评论线程

风险与影响

• 风险：
  1. 回归风险：变更涉及多个环境变量读取点（如 SGLANG_USE_MODELSCOPE、SGLANG_USE_AITER、SGLANG_NPU_FUSED_MOE_MODE 等），如果描述符的默认值或行为与原有工具函数不一致，可能导致服务器启动或运行时行为变化。例如，envs.SGLANG_NPU_FUSED_MOE_MODE.get() 返回整数，而原 os.environ.get 返回字符串，比较逻辑已调整，但需确保所有使用场景都正确转换。
  2. 兼容性风险：新增的 SGLANG_USE_AITER_UNIFIED_ATTN 和 SGLANG_MORI_NUM_MAX_DISPATCH_TOKENS_PER_RANK 描述符，如果已有环境变量设置但未在代码中定义，可能引发 AttributeError。不过，本次 PR 已同步添加，风险较低。
  3. 性能影响：环境变量描述符的访问可能涉及额外的方法调用，但属于微开销，不影响核心路径性能。
• 影响：
  1. 对用户影响：无直接影响，环境变量名称和默认值保持不变，用户无需调整配置。
  2. 对系统影响：统一了环境变量管理方式，提升了代码可维护性，但需确保所有读取点逻辑等价。
  3. 对团队影响：为后续环境变量相关开发树立了规范，减少了魔法字符串的使用，但团队成员需熟悉新的 envs 描述符模式。
     - 风险标记：环境变量读取逻辑变更, 类型转换风险

关联脉络

• PR #22926 [misc] Configure logging before ServerArgs.post_init: 同样修改了 server_args.py，涉及服务器初始化和环境配置，属于同一模块的近期调整。
• PR #22959 fix(loads): preserve include filtering after watching mode switch: 涉及 sglang/srt 模块的环境变量或配置处理，但具体关联较弱；主要体现同一模块的活跃维护。