sgl-project/sglang · 作者 merrymercy · 合并时间 2026-03-24 12:31
PR body中说明：'Updates .claude/skills/add-jit-kernel/SKILL.md to match the JIT kernel CI flow after the refactor to test/run_suite.py and register_cuda_ci (see #21239 / commit 260abe1).'，动机是保持文档与代码变更同步，避免因文档过时导致开发者错误配置JIT内核测试和基准。
建议开发者快速浏览此PR以了解JIT内核CI注册的新流程，关注register_cuda_ci的使用和套件名称，无需深入分析代码设计，仅作为文档更新参考。
无review讨论，变更由作者直接合并，表明文档更新已被接受无需额外审查。
实现集中在单个文件.claude/skills/add-jit-kernel/SKILL.md的更新，关键改动点包括：1) 文档化测试和基准通过test/run_suite.py的AST解析发现机制，取代直接运行pytest；2) 引入register_cuda_ci函数、套件名称（如stage-b-kernel-unit-1-gpu-large）和可选参数（如nightly=True、disabled=）；3) 提供测试文件和基准文件的示例代码；4) 添加故障排除指南，处理缺失CI注册或AST解析问题。
| 文件 | 模块 | 状态 | 重要度 |
|---|---|---|---|
.claude/skills/add-jit-kernel/SKILL.md |
documentation/skills | modified | 3.0 |
分析完成后，这里会展示 LLM 生成的相对完整源码片段和详细注释。
当前评论区没有形成足够清晰的争议点或结论，后续有更多讨论时会体现在这里。
风险较低，主要涉及文档准确性：1) 如果文档错误或过时，可能导致开发者错误注册测试，影响CI运行；2) 依赖AST解析的机制未被深入验证，但文档已提供故障排除指南。无代码或CI行为变更，不引入回归、性能或安全风险。
影响范围主要针对开发者团队：确保JIT内核测试和基准的正确注册，提升CI流程清晰度，尤其对新贡献者有帮助。对系统无直接影响，影响程度中等，仅限于JIT内核模块的CI配置和文档指导。
当前没有检测到明确关联的 Issue 链接，后续同步到相关引用后会出现在这里。
本PR更新了JIT内核技能文档以适配重构后的CI流程，通过test/run_suite.py和register_cuda_ci注册机制，确保开发者正确设置测试和基准。作为纯文档变更，风险低，但对团队CI配置有重要指导作用。
为解决CI重构（如引用#21239）后的文档不一致问题，本PR旨在更新.claude/skills/add-jit-kernel/SKILL.md，使其匹配新的JIT内核测试流程。动机直接来自PR body："Updates .claude/skills/add-jit-kernel/SKILL.md to match the JIT kernel CI flow after the refactor to test/run_suite.py and `register_cuda_ci"，强调保持文档同步以避免配置错误。
.claude/skills/add-jit-kernel/SKILL.md，增43行、删2行。test/run_suite.py的AST解析发现，而非直接运行pytest。register_cuda_ci函数，例如：python
from sglang.test.ci.ci_register import register_cuda_ci
register_cuda_ci(est_time=30, suite="stage-b-kernel-unit-1-gpu-large")stage-b-kernel-unit-1-gpu-large、stage-b-kernel-benchmark-1-gpu-large、nightly-kernel-1-gpu。无review讨论，变更由作者直接合并，表明文档更新已被快速接受，无需技术争议。
参与讨论