Prhub

sgl-project/sglang · 标签视图

标签列表

聚合结果

cleanup 相关 PR

2026-08-21
重构 重要性 6.53 洞察度 5.00

清理 disagg 模块 4 处无调用死代码,纯删除零行为变化

值得快速浏览,不是因为它改变了什么,而是因为它示范了“如何安全地删除代码”:每个删除项都给出来源 commit、存活替身、全仓库扫描证据、lint 验证,并明确区分死代码与扩展点。对维护者而言,真正可借鉴的是 page_indices_to_cp_rank_page_indices 被孤儿化的过程——重构时把调用方迁移到新实现后应顺手清理旧符号,避免正确性修复落在无人调用的代码上。若团队有 dead code 治理需求,本 PR 可作为模板。

重构 重要性 6.46 洞察度 5.00

清理 srt/managers 冗余注释,重写评论风格规则

值得精读 .claude/rules/comment-style.md 与提交序列。规则中“成本恢复”测试(删掉注释后,熟悉仓库的人能否从周围代码加 grep 恢复该事实)和“写作 vs 编辑”的决策不对称,是注释管理的实用框架;提交序列展示了“先删后补”的收敛过程,对评审者判断注释清理 PR 的边界有参考价值。若团队计划继续推广该风格,建议参照本 PR 的亮线规则,避免在后续清理中误删承载跨文件契约的注释。