执行摘要
修复 CLI 强制使用统计标志未启用连续使用统计的 bug。
根据PR body,动机是确保--enable-force-include-usage服务器级覆盖始终启用最终使用统计和连续使用统计,修复了原有逻辑中连续使用统计可能被stream options禁用的bug。PR body明确指出“make --enable-force-include-usage return (True, True) from should_include_usage()”以解决此问题。
建议快速浏览以理解修复逻辑,特别是should_include_usage()函数的变更。对于负责entrypoints模块或前端测试的工程师,值得关注回归测试的设计,以确保覆盖所有相关场景。
Review讨论较少,仅gemini-code-assist[bot]评论确认变更正确解决了bug,并指出逻辑更清晰。simon-mo批准了PR,无其他评论。无争议点或深度技术讨论,变更被快速接受。
参与讨论