执行摘要

本PR新增了一个pre-commit钩子，用于自动验证test/registered/目录下的Python测试文件是否包含必要的CI注册调用（如register_cuda_ci）。此举旨在预防因测试文件缺失注册而导致的CI中断问题，通过将检查提前到提交阶段，提升开发效率和CI稳定性。变更涉及两个文件：新增验证脚本和更新pre-commit配置，风险较低，主要影响开发团队的工作流程。

功能与动机

根据PR描述，test/registered/目录下的文件由run_suite.py的collect_tests()收集，要求每个文件必须包含CI注册调用。缺失注册会导致CI失败并抛出ValueError: No CI registry found。此前#22298已修复了一个因文件放置在test/registered/但未注册而中断所有CI套件的案例。本PR的动机是通过pre-commit钩子在提交时捕获此类错误，避免等待CI运行时失败，从而提高开发效率和CI可靠性。

实现拆解

实现分为两个关键部分：

1. pre-commit配置更新：在.pre-commit-config.yaml中添加新的钩子check-registered-tests，配置如下：
   

   - id: check-registered-tests
     name: check registered tests have CI registry
     entry: python3 scripts/ci/check_registered_tests.py
     language: system
     files: ^test/registered/.*\.py$
     pass_filenames: false
   

   
   该钩子针对test/registered/目录下的所有.py文件运行验证脚本。

2. 验证脚本实现：新增scripts/ci/check_registered_tests.py，核心逻辑包括：

   • 导入ci_register.py中的ut_parse_one_file()函数，复用其AST解析逻辑以匹配run_suite.py的行为。
   • 遍历test/registered/目录下的Python文件，跳过conftest.py和__init__.py。
   • 检查每个文件是否包含CI注册调用，若无则报错并提示将手动测试移至test/manual/。
   • 脚本返回非零状态码以阻止提交，确保问题在早期被发现。

评论区精华

由于review评论为空，没有公开的讨论或争议点。但从提交历史可见，作者通过临时添加错误文件验证了钩子的有效性，并在确认后恢复，体现了测试的严谨性。

风险与影响

风险分析：

• 脚本依赖ci_register.py的ut_parse_one_file()函数，若该函数逻辑变更或存在bug，可能导致误报或漏报。
• 脚本仅检查文件是否包含注册调用，未验证注册调用的正确性（如参数有效性），潜在CI问题可能仍存在。
• 新增pre-commit钩子可能轻微增加提交开销，但检查范围有限，影响可忽略。

影响分析：

• 对用户无直接影响。
• 对开发团队：显著提高CI稳定性，减少因注册缺失导致的调试时间；增强开发流程，通过早期验证预防问题。
• 对系统性能无显著影响。

关联脉络

本PR与近期历史PR关联如下：

• #22298：PR body中提及该Issue修复了test/registered/目录下文件缺失注册导致CI中断的案例，是本PR的直接动机来源。
  • #22270（Refactor auto benchmark unit tests and fix CI bug）：涉及test/registered/目录的重构和CI修复，与本PR在测试基础设施和CI稳定性方面有共同关注点。

整体来看，本PR是sglang仓库持续优化CI和测试基础设施的一部分，通过自动化检查提升代码质量和开发效率，符合近期多个PR（如#22395、#22391）对CI流程的改进趋势。