Prhub

#25012 [Spec] Drop Rule 5 (`_len` / `_lens` ban) from speculative naming rule

原始 PR 作者 hnyls2002 合并时间 2026-05-12 05:35 文件变更 1 提交数 1 评论 1 代码增减 +3 / -12

执行摘要

移除投机解码命名规则中的 _len/_lens 禁令

PR body 指出:num_X 仍是 Rule 4 规定的计数首选形式,但 _len / _lens 在投机解码标量名(如 accept_lenema_accept_len)中很常见,且与 PyTorch 生态中 Triton kernel 签名保持一致。原 Rule 5 过度限制,导致代码与社区习惯脱节。

值得快速合并,不需要深入精读。该 PR 体现了团队对代码规范与实践之间平衡的审慎调整,适合作为命名规范演进的参考。

讨论亮点

Review 讨论中仅有 gemini-code-assist[bot] 的自动消息提示达到每日限额,无实质性讨论。

实现拆解

仅修改一个 Markdown 文件,变更量为 +3/-12:

  1. 删除原 Rule 5(Avoid length / lens internally)及其例外说明。
  2. 将原 Rule 6(冗余 _token_id 后缀)重新编号为 Rule 5。
  3. 将原 Rule 7(单复数规则)重新编号为 Rule 6。
  4. 在文档末尾的「Out-of-scope」部分新增一条说明,明确 _len / _lens 可接受,但 Rule 1(去掉 -ed 后缀)仍然适用。
文件 模块 状态 重要度
.claude/rules/speculative-naming.md 规范文档 modified 2.79

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

评论区精华

没有提炼出高价值讨论线程

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

风险与影响

无技术风险。纯文档变更,不影响任何代码逻辑或运行时行为。

仅影响开发者遵循的命名规范文档。放松规则后,开发者可以更灵活地在投机解码相关代码中使用 _len / _lens,减少与现有代码和生态风格的冲突。

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论