#35847 refactor(disagg): collapse duplicated branches in get_kv_class
原始 PR · 作者 ShangmingCai · 合并时间 2026-08-21 22:11
收拢 get_kv_class 重复分支
该 PR 值得精读,因为它展示了如何在保持可分析性的前提下进行安全重构,特别是关于保留显式导入的决策值得借鉴。
标签列表
聚合结果
原始 PR · 作者 ShangmingCai · 合并时间 2026-08-21 22:11
收拢 get_kv_class 重复分支
该 PR 值得精读,因为它展示了如何在保持可分析性的前提下进行安全重构,特别是关于保留显式导入的决策值得借鉴。
原始 PR · 作者 ShangmingCai · 合并时间 2026-08-21 22:09
清理 disagg 模块 4 处无调用死代码,纯删除零行为变化
值得快速浏览,不是因为它改变了什么,而是因为它示范了“如何安全地删除代码”:每个删除项都给出来源 commit、存活替身、全仓库扫描证据、lint 验证,并明确区分死代码与扩展点。对维护者而言,真正可借鉴的是 page_indices_to_cp_rank_page_indices 被孤儿化的过程——重构时把调用方迁移到新实现后应顺手清理旧符号,避免正确性修复落在无人调用的代码上。若团队有 dead code 治理需求,本 PR 可作为模板。
清理 srt/managers 冗余注释,重写评论风格规则
值得精读 .claude/rules/comment-style.md 与提交序列。规则中“成本恢复”测试(删掉注释后,熟悉仓库的人能否从周围代码加 grep 恢复该事实)和“写作 vs 编辑”的决策不对称,是注释管理的实用框架;提交序列展示了“先删后补”的收敛过程,对评审者判断注释清理 PR 的边界有参考价值。若团队计划继续推广该风格,建议参照本 PR 的亮线规则,避免在后续清理中误删承载跨文件契约的注释。