缺陷修复
重要性 4.85
洞察度 4.00
CLI 显式空 prompt 与 --max-tokens 0 不再被丢弃
值得快速精读而非深读:改动只有 3 行,逻辑直白,但它是“真值判断 vs `is not None`”这一经典 CLI/配置解析陷阱的干净样例,适合作为团队 review checklist 的案例。 阅读时建议关注三点:一是为什么只在 `cmd` 的 guard 上动手而不改 `add_cli_args`(因为 argparse 未设 default,`None` 即“未提供”);二是为什么必须把 `0` 透传而不是在客户端提前校验(Issue 明确要求不放宽服务端校验,保持单一权威校验点);三是 `tests/entrypoints/test_cli_openai.py` 被删除这一决策——它比代码本身更有组织价值,说明“写了测试”与“测试会被执行”是两件事,后续若再改动该 CLI,需要在本地用真实服务端跑端到端用例而不是依赖 CI。