执行摘要
启用 Ruff UP037 全库清理冗余引号注解
启用 Ruff 的 UP037 规则,自动检测并移除冗余的引号注解,消除代码噪声,统一代码风格。详见 PR Body。
建议阅读 .pre-commit-config.yaml 的配置变更,了解如何加入新 lint 规则。变更本身无需深入 code review。
无 review 讨论。作者自行检查了字节码编译通过且 pre-commit 三件套(ruff, black, isort)达到不动点后合并。
启用 Ruff 的 UP037 规则,自动检测并移除冗余的引号注解,消除代码噪声,统一代码风格。详见 PR Body。
建议阅读 .pre-commit-config.yaml 的配置变更,了解如何加入新 lint 规则。变更本身无需深入 code review。
无 review 讨论。作者自行检查了字节码编译通过且 pre-commit 三件套(ruff, black, isort)达到不动点后合并。
.pre-commit-config.yaml:在 Ruff 钩子的 --select 中添加 UP037,使后续每次 pre-commit 都会自动检测并修复冗余引号。black 重新格式化为单行。无需测试或其他配套修改。| 文件 | 模块 | 状态 | 重要度 |
|---|---|---|---|
.pre-commit-config.yaml |
代码风格 | modified | 4.0 |
python/sglang/srt/disaggregation/decode_hicache_mixin.py |
解耦缓存 | modified | 6.1 |
python/sglang/srt/disaggregation/common/staging_handler.py |
分段传输 | modified | 6.05 |
python/sglang/srt/hardware_backend/mlx/scheduler_mixin.py |
MLX 调度 | modified | 6.02 |
python/sglang/srt/disaggregation/decode_hicache_mixin.py
lint-fix
自动修复的源文件之一,演示了类方法注解引号的去除。
def _build_decode_prefix_match(self, req: Req, result: Any) -> DecodePrefixMatch:
# 此函数原本签名为 req: "Req",因启用 UP037 后不再需要引号
# (文件顶部包含 from __future__ import annotations)
prefix_indices = result.device_indices
l1_prefix_len = len(prefix_indices)
l2_host_hit_length = result.host_hit_length
l3_storage_hit_length = 0
last_host_node = None
if self.scheduler.enable_decode_hicache:
last_host_node = result.last_host_node
if last_host_node.backuped or last_host_node is self.tree_cache.root_node:
matched_len = l1_prefix_len + l2_host_hit_length
suffix_tokens = req.origin_input_ids[matched_len:]
last_hash = last_host_node.get_last_hash_value()
prefix_keys = (
last_host_node.get_prefix_hash_values(last_host_node.parent)
if self.tree_cache.hicache_storage_pass_prefix_keys
else None
)
l3_storage_hit_length = self.tree_cache.query_storage_hit_length(
last_host_node, suffix_tokens, last_hash, prefix_keys,
)
return DecodePrefixMatch(
prefix_indices=prefix_indices,
l2_host_hit_length=l2_host_hit_length,
l3_storage_hit_length=l3_storage_hit_length,
last_device_node=result.last_device_node,
last_host_node=last_host_node if l3_storage_hit_length > 0 else None,
)
当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。
风险极低。UP037 仅在可证明为冗余时去除引号,且对 from __future__ import annotations 环境或函数作用域注解生效。但跨越 185 个文件,可能引入合并冲突,尤其当其他分支正修改同一文件的注解区域。
影响所有开发者:从此提交开始,新增代码若使用 from __future__ import annotations 则不需要再写注解引号;pre-commit 自动清理遗留引号。CI 时间无影响。
当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。
参与讨论