执行摘要

• 一句话：清理 array-like msgspec 结构体配置
• 推荐动作：该 PR 属于低风险的清理工作，建议合并。它提高了代码一致性，但无需深度审查。

功能与动机

Array-like msgspec 结构体按位置序列化字段，启用 array_like=True 后 omit_defaults=True 是冗余的。PR 旨在使代码库中的此类配置保持一致，减少混淆。

实现拆解

1. SamplingParams 类配置修改：在 python/sglang/srt/sampling/sampling_params.py 中，将类定义从 msgspec.Struct, kw_only=True, omit_defaults=True 更改为 msgspec.Struct, kw_only=True, array_like=True。同时将 custom_params 字段从 stream_interval 之前移动到 sampling_seed 之后，以匹配 array_like 的预期顺序（先 API 参数，后内部字段）。
2. 移除 EventBatch 和 KVCacheEvent 的冗余 omit_defaults：在 python/sglang/srt/disaggregation/kv_events.py 中，删除了这两个类的 omit_defaults=True 选项，因为 array_like=True 已足够。
3. 更新 Rust 路由器注释：在 experimental/sgl-router/src/policies/kv_events/wire.rs 中，更新了关于序列化的文档注释，移除了 omit_defaults=True 的提及，并调整了字段缺失时的描述。
4. 删除过时测试：在 test/registered/unit/sampling/test_sampling_params.py 中，删除了 test_msgpack_omits_default_fields 测试，该测试验证了 omit_defaults=True 的行为，现因使用 array_like=True 而不再适用。

关键文件：

• python/sglang/srt/sampling/sampling_params.py（模块 采样参数；类别 source；类型 core-logic；符号 SamplingParams）: 核心修改，改变了 SamplingParams 的序列化方式。
• experimental/sgl-router/src/policies/kv_events/wire.rs（模块 路由器；类别 source；类型 entrypoint）: 更新了 Rust 端的线缆协议注释，反映序列化行为变化。
• python/sglang/srt/disaggregation/kv_events.py（模块 解体化；类别 source；类型 core-logic）: 移除了 EventBatch 和 KVCacheEvent 上的冗余 omit_defaults=True。
• test/registered/unit/sampling/test_sampling_params.py（模块 采样测试；类别 test；类型 test-coverage；符号 test_msgpack_omits_default_fields）: 删除了过时的测试用例。

关键符号：SamplingParams

关键源码片段

python/sglang/srt/sampling/sampling_params.py

核心修改，改变了 SamplingParams 的序列化方式。

class SamplingParams(msgspec.Struct, kw_only=True, array_like=True):
    """
    采样参数。设置为 array_like 以使用 msgpack 数组序列化，
    由于数组按位置，omit_defaults 是冗余的，因此移除。
    """
    # ... 字段定义 ...
    stream_interval: Optional[int] = None
    logit_bias: Optional[Dict[str, float]] = None
    sampling_seed: Optional[int] = None
    custom_params: Optional[Dict[str, CustomParamValue]] = None # 移到 sampling_seed 之后

python/sglang/srt/disaggregation/kv_events.py

移除了 EventBatch 和 KVCacheEvent 上的冗余 omit_defaults=True。

class EventBatch(
    msgspec.Struct,
    array_like=True, # type: ignore[call-arg]
    gc=False, # type: ignore[call-arg]
):
    ts: float
    events: list[Any]
    attn_dp_rank: Optional[int] = None
​
class KVCacheEvent(
    msgspec.Struct,
    array_like=True, # type: ignore[call-arg]
    gc=False, # type: ignore[call-arg]
    tag=True,
):
    """所有 KV 缓存相关事件的基类"""

评论区精华

PR 没有 review 评论或讨论线程；所有变更由作者自己审查合并。

• 暂无高价值评论线程

风险与影响

• 风险：变更风险低。
• 回归风险：SamplingParams 序列化从 map 变为 array，但 msgspec 的 array 编码与现有 wire 格式兼容？需要确认反向兼容性。然而 PR body 提到测试通过，且 Rust 解码器已调整注释以接受缺失字段。
• 性能影响：无预期性能变化。
• 兼容性：EventBatch 和 KVCacheEvent 的序列化行为未变（因为 array_like=True 时 omit_defaults 无效果），但线缆协议注释更新表明字段缺失处理已调整。潜在风险是 Rust 解码器可能对缺失字段处理不一致，但测试已覆盖。
• 测试覆盖：移除了一个测试，但保留的 test_msgpack_round_trip_preserves_normalized_state 提供了基本覆盖。
• 影响：
  • 用户影响：无直接影响。
  • 系统影响：仅影响序列化配置，不会改变运行时行为。
  • 团队影响：简化了配置，减少未来混淆。
  • 风险标记：低风险

关联脉络

• 暂无明显关联 PR