#24094 Add speculative decoding naming convention rule

原始 PR 作者 hnyls2002 合并时间 2026-05-09 05:52 文件变更 1 提交数 9 评论 1 代码增减 +115 / -0

执行摘要

speculative decoding 代码中存在命名不一致现象，如 accepted_drafts 不符合规则（应为 accept 而非 accepted）。PR 作者在 body 中列出了 meta_info、IPC dataclass 等处的违规项，表明需要显式约定来指导后续开发与重构。

建议 speculative decoding 相关模块的开发和 reviewer 阅读该规则，并将其作为代码审查的标准之一。PR 本身设计简洁，规则定义清晰（尤其是 accept vs correct 的语义区分），值得借鉴。

讨论亮点

该 PR 没有任何 review 评论，仅有一条 gemini-code-assistant 的自动配额提示，故无实质讨论。

建立命名规则：定义了 7 条规则覆盖动词形式（accept 而非 accepted）、bonus 令牌命名、accept/correct 语义区分、计数/计数器/速率的后缀规范等。
外部 key 审计：对照规则逐一检查 Prometheus 指标、HTTP meta_info、IPC SpeculativeDecodingMetricsMixin 字段和 SpeculativeMetrics 数据类，记录违规项。
提交为 Claude 规则文件，以便 IDE 辅助检查新代码。

文件	模块	状态	重要度
`.claude/rules/speculative-naming.md`	规则配置	added	4.57

分析完成后，这里会展示 LLM 生成的相对完整源码片段和详细注释。

当前评论区没有形成足够清晰的争议点或结论，后续有更多讨论时会体现在这里。

该 PR 仅新增文档文件，不修改任何代码，因此无直接回归或性能风险。但规则可能用于指导后续代码重构，如果未来基于此规则进行重命名，需注意兼容性（如 Prometheus 指标名变更可能导致 dashboards 失效）。PR 的审计部分已识别出潜在违规项，可作为重命名参考。

对用户无直接功能影响；对开发者，尤其是参与 speculative decoding 模块的贡献者，提供了一个清晰的命名指南，有助于统一代码风格。长期看可能触发一系列重命名 PR，需协调团队步调。

仅文档变更低风险

当前没有检测到明确关联的 Issue 链接，后续同步到相关引用后会出现在这里。

一句话：新增 speculative decoding 命名规则文件
推荐动作：建议 speculative decoding 相关模块的开发和 reviewer 阅读该规则，并将其作为代码审查的标准之一。PR 本身设计简洁，规则定义清晰（尤其是 accept vs correct 的语义区分），值得借鉴。

建立命名规则：定义了 7 条规则覆盖动词形式（accept 而非 accepted）、bonus 令牌命名、accept/correct 语义区分、计数/计数器/速率的后缀规范等。
外部 key 审计：对照规则逐一检查 Prometheus 指标、HTTP meta_info、IPC SpeculativeDecodingMetricsMixin 字段和 SpeculativeMetrics 数据类，记录违规项。
提交为 Claude 规则文件，以便 IDE 辅助检查新代码。

关键文件：

.claude/rules/speculative-naming.md（模块规则配置；类别 docs；类型 documentation）: 唯一的变更文件，定义了 speculative decoding 命名规范，并附有外部 key 审计。

关键符号：未识别

该 PR 没有任何 review 评论，仅有一条 gemini-code-assistant 的自动配额提示，故无实质讨论。

风险：该 PR 仅新增文档文件，不修改任何代码，因此无直接回归或性能风险。但规则可能用于指导后续代码重构，如果未来基于此规则进行重命名，需注意兼容性（如 Prometheus 指标名变更可能导致 dashboards 失效）。PR 的审计部分已识别出潜在违规项，可作为重命名参考。
影响：对用户无直接功能影响；对开发者，尤其是参与 speculative decoding 模块的贡献者，提供了一个清晰的命名指南，有助于统一代码风格。长期看可能触发一系列重命名 PR，需协调团队步调。
风险标记：仅文档变更, 低风险