# PR #24094 完整报告

- 仓库：`sgl-project/sglang`
- 标题：Add speculative decoding naming convention rule
- 合并时间：2026-05-09 05:52
- 原文链接：http://prhub.com.cn/sgl-project/sglang/pull/24094

---

# 执行摘要

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

# 功能与动机

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

# 实现拆解

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

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

关键符号：未识别


# 评论区精华

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

- 暂无高价值评论线程

# 风险与影响

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

# 关联脉络

- 暂无明显关联 PR