执行摘要

• 一句话：移除测试和注释中的 'smoke' 措辞
• 推荐动作：该 PR 属于纯代码清理，不涉及功能变更，适合在代码审查中快速过目。对于追求代码术语一致性的项目，这是一个可取的实践。建议关注 gemini-code-assist 提出的不一致问题，确保后续修复。

功能与动机

从PR body可见，目的是替换 'smoke' 措辞为更具体的标签（如sanity check、build-and-run check、validation settings），并重命名CUDA coredump debug测试文件/类避免 'smoke' 标签，同时保持benchmark/test prompt载荷不变。

实现拆解

1. 全局搜索与替换：使用 git grep 定位所有包含 'smoke' 的测试名、类名和注释。
2. 测试函数重命名：将 test_generate_smoke 改为 test_generate_basic，test_generate_smoke_tp_mismatch 改为 test_generate_with_tp_mismatch，test_text_output_smoke 改为 test_text_output_format。
3. 测试文件重命名：将 test_cuda_coredump_smoke.py 重命名为 test_cuda_coredump.py，类名 TestCudaCoredumpSmoke 改为 TestCudaCoredump。
4. 注释措辞调整：将 benchmark 中的 '# Smoke test' 改为 '# Validate output shapes before benchmarking.'；deepseek-v4-deployment.jsx 中的 'static smoke' 改为 'static validation'；diffusion trajectory 工具中的 'smoke settings' 改为 'deterministic settings'。
5. 配套修改：更新了多处的 Skill 文档、CI workflow 和测试配置文件中的 'smoke' 相关措辞。
6. 验证：通过 git diff --check 和 git grep -n -I -i 'smoke' 确认无残留。

关键文件：

• test/registered/debug_utils/test_cuda_coredump.py（模块 CUDA调试；类别 test；类型 rename-or-move；符号 TestCudaCoredumpSmoke, TestCudaCoredump）: 核心变更：测试文件重命名并移除类名中'Smoke'，体现PR主要意图。
• test/manual/test_mori_transfer_engine_e2e.py（模块 传输引擎；类别 test；类型 test-coverage；符号 test_generate_smoke, test_generate_basic, test_generate_smoke_tp_mismatch, test_generate_with_tp_mismatch）: 涉及两个测试函数重命名，展示'Smoke'替换为'Basic'、'With'等。
• benchmark/bench_rope/benchmark_rope_index.py（模块 基准测试；类别 source；类型 core-logic）: 注释清理示例：'# Smoke test' 改为 '# Validate output shapes before benchmarking.'
• docs_new/src/snippets/autoregressive/deepseek-v4-deployment.jsx（模块 文档片段；类别 source；类型 core-logic）: 文档示例中'static smoke'改为'static validation'
• python/sglang/multimodal_gen/tools/compare_diffusion_trajectory_similarity.py（模块 多模态生成；类别 source；类型 core-logic）: 注释清理：'smoke settings' 改为 'deterministic settings'
• test/registered/debug_utils/comparator/test_entrypoint.py（模块 入口测试；类别 test；类型 test-coverage；符号 test_text_output_smoke, test_text_output_format）: 测试方法重命名：test_text_output_smoke→test_text_output_format
• python/sglang/test/attention/test_trtllm_mla_backend.py（模块 MLA后端；类别 test；类型 test-coverage）: 文本清理，涉及'comprehensive'等措辞调整
• test/registered/hicache/test_hicache_storage_runtime_attach_detach.py（模块 HiCache存储；类别 test；类型 test-coverage）: 附带清理，重要性较低
• test/registered/moe/test_cutedsl_moe.py（模块 MoE测试；类别 test；类型 test-coverage）: 附带清理，重要性较低
• .github/workflows/pr-test-sgl-kernel.yml（模块 CI配置；类别 infra；类型 infrastructure）: CI工作流中'comprehensive'等措辞调整
• python/sglang/multimodal_gen/.claude/skills/sglang-diffusion-modelopt-quant/SKILL.md（模块 多模态文档；类别 docs；类型 documentation）: 文档清理，涉及控制流/异常路径术语调整
• .claude/skills/sglang-auto-benchmark/SKILL.md（模块 Claude技能；类别 docs；类型 documentation）: 文档清理，重要性最低

关键符号：TestCudaCoredump, test_generate_basic, test_generate_with_tp_mismatch, test_text_output_format

关键源码片段

test/manual/test_mori_transfer_engine_e2e.py

涉及两个测试函数重命名，展示'Smoke'替换为'Basic'、'With'等。

    def test_generate_basic(self): # 原名为 test_generate_smoke
        resp = requests.post(
            self.lb_url + "/generate",
            json={
                "text": "Hello",
                "sampling_params": {"temperature": 0, "max_new_tokens": 8},
            },
            timeout=120,
        )
        self.assertEqual(resp.status_code, 200, resp.text)
        out = resp.json()
        self.assertIn("text", out)
        self.assertIsInstance(out["text"], str)
        self.assertGreater(len(out["text"]), 0)
​
    # ... 另一处类似：
    def test_generate_with_tp_mismatch(self): # 原名为 test_generate_smoke_tp_mismatch
        # 测试体相同

评论区精华

gemini-code-assist[bot] 对 test/manual/layers/attention/nsa/test_act_quant_triton.py 提出两个 medium 优先级的评论：更新后的 docstring 提到 'with and without CUDA graphs'，但对应的 print 语句只提到 'with CUDA Graphs'，建议保持一致。PR 作者未公开回复，但从 PR 合并状态看该问题未在当前 PR 中修复（该文件未在变更集中）。

• test_act_quant_triton.py docstring 和 print 不一致 (testing): 建议确保一致性，但该文件未在当前 PR 的变更集中，状态未知。

风险与影响

• 风险：风险极低：所有修改均为注释、测试命名和类名，不涉及运行时逻辑。潜在风险是测试重命名可能导致外部脚本或 CI 配置引用旧的测试名称，但 PR 验证了 git grep 无 'smoke' 残留，且项目内无其他引用。
• 影响：
  • 用户：无感知，不影响任何功能。
  • 系统：无性能变化，无行为变化。
  • 团队：改善了代码可读性，减少模糊术语。但测试名称变更可能需要团队成员更新个人工作流中的测试选择器。
  • 风险标记：仅注释和命名变更, 无逻辑影响

关联脉络

• 暂无明显关联 PR