执行摘要

• 一句话：拒绝负值配置参数（max_logprobs 除 -1 外）
• 推荐动作：建议合并。该 PR 修复了配置验证漏洞，采用社区推荐的标准模式，无副作用。值得作为配置健壮性参考。

功能与动机

Issue #43985 报告 max_logprobs 和 long_prefill_token_threshold 接受负值，下游代码未预期处理：max_logprobs 除 -1 外的负值导致混淆错误或无效果；long_prefill_token_threshold 负值因 0 < threshold 条件不成立而静默失效。这属于配置验证缺口，需要收紧约束。

实现拆解

1. vllm/config/model.py：将 max_logprobs: int = 20 改为 max_logprobs: int = Field(default=20, ge=-1)，利用 pydantic 的 ge=-1 约束拒绝小于 -1 的值，保留 -1 作为 sentinel。
2. vllm/config/scheduler.py：将 long_prefill_token_threshold: int = 0 改为 long_prefill_token_threshold: int = Field(default=0, ge=0)，约束为非负整数，0 表示关闭功能。
3. 精简实现：根据审核意见，去除初始的 field_validator 实现，直接使用 Field(ge=...) 声明，并删除对应的测试用例，因为约束自身提供了足够验证。

关键文件：

• vllm/config/model.py（模块 模型配置；类别 source；类型 data-contract；符号 max_logprobs）: 核心变更文件之一，添加 max_logprobs 的 Field 约束，拒绝负值（除 -1 外）。
• vllm/config/scheduler.py（模块 调度配置；类别 source；类型 core-logic；符号 long_prefill_token_threshold）: 核心变更文件之一，添加 long_prefill_token_threshold 的 Field 约束，拒绝负值。

关键符号：未识别

评论区精华

• 改用 Field(ge=...) 简化：审核者 yewentao256 建议直接使用 pydantic Field 约束而非 field_validator，避免冗余校验逻辑。作者采纳并重构，使代码更简洁。
• 删除测试用例：审核者认为改动极小，无需单独编写单元测试。作者同意并移除测试。

• 默认值选择：在 long_prefill_token_threshold 上，原本考虑用 None 表示关闭，但最终保留 0 作为默认值，与现有 __post_init__ 逻辑兼容。

  • 使用 Field(ge=...) 代替 field_validator (design): 作者接受并改为直接使用 Field 约束，移除了 validator。
  • 删除专门测试用例 (testing): 作者删除测试文件中的新增测试。
  • long_prefill_token_threshold 默认值选择 (question): 维持原默认值 0，与 __post_init__ 中 == 0 的检查一致。

风险与影响

• 风险：风险极低。仅添加 pydantic 字段校验，不影响已有业务逻辑。若现有代码内部误传负数（如批量赋值），会在构造配置对象时立即抛出 ValidationError，但经排查无内部路径满足此条件。对于依赖静默接受负值的极端外部配置，需要调整参数。
• 影响：用户现在会在启动时收到清晰的校验错误，而非运行时迷惑行为。受影响参数：--max-logprobs（除 -1 外不能为负）和 --long-prefill-token-threshold（不能为负）。对正确定义配置的用户无感知。
• 风险标记：配置验证增强, 低风险

关联脉络

• 暂无明显关联 PR