执行摘要

• 一句话：修复CI语法错误，重构异步策略工作流
• 推荐动作：该 PR 是典型的 CI 基础设施修复，值得关注其将内嵌脚本迁移至声明式 Workflow 的做法，提高了可读性和 CI 兼容性。建议在后续类似重构中增加 Workflow 语法校验（如 action-validator 或 pre-commit 钩子），从源头避免语法错误。同时，考虑是否保留 tests/special_e2e/run_fully_async_policy.sh 作为独立测试入口，避免重复维护。

功能与动机

PR body 明确指出 'fix: several CI syntax errors as introduced in https://github.com/verl-project/verl/commit/e8aff345d15e9f87f3bbabeab6fc3a49ca8bf227'，即该 commit 引入了多个 CI 语法错误，导致工作流无法正常解析或运行，亟需修正。

实现拆解

1. 修复 cpu_unit_tests.yml 语法错误：在 Check final pip list 步骤中，补全了缺失的 run 关键字，使得 pip3 list 命令可以被正确执行。
2. 重写 e2e_fully_async_policy.yml 为标准工作流：原本的文件是 Bash 脚本（包含 #!/usr/bin/env bash 和 set -xeuo pipefail），无法被 GitHub Actions 解析。将其整体替换为 YAML 格式，定义 name、on（push、schedule、workflow_dispatch）、env（保留原超参）和 jobs（checkout、环境搭建、下载模型、运行训练等步骤），使 CI 能自动触发并正确执行。
3. 回退同步脚本的尝试：在 review 讨论中，作者尝试将 tests/special_e2e/run_fully_async_policy.sh 的内容同步为工作流的副本，但导致 CI 失败。最终决定回退该文件的改动，保留原有的独立 Shell 脚本，避免破坏现有测试流程。

关键文件：

• .github/workflows/e2e_fully_async_policy.yml（模块 CI工作流；类别 infra；类型 infrastructure）: 主要变更：从内嵌 Bash 脚本完全重写为 GitHub Actions YAML 工作流，修复语法错误，是 PR 的核心改动。
• .github/workflows/cpu_unit_tests.yml（模块 CI工作流；类别 infra；类型 infrastructure）: 辅助修复：在 Check final pip list 步骤中补充缺失的 run 关键字，修正 YAML 语法错误。

关键符号：未识别

评论区精华

Review 中唯一有意义的讨论发生在 tests/special_e2e/run_fully_async_policy.sh 第 68 行。审核者 wuxibin89 评论 '@tardis-key I think this is needed.' 指出 SKIP_ENABLE=True 行需要保留。作者 HollowMan6 回复解释该文件内容是从工作流复制而来但导致 CI 持续失败，因此已将其回退。结论是保持该文件原样，不与工作流同步。

• 是否同步 tests/special_e2e/run_fully_async_policy.sh 与 workflow (design): 回退 tests/special_e2e/run_fully_async_policy.sh 的改动，保持原有 shell 脚本独立。

风险与影响

• 风险：
  1. 工作流重构风险：e2e_fully_async_policy.yml 从 Bash 脚本完全改写为 YAML 工作流，原脚本中隐式的环境变量传递、步骤间状态依赖等可能缺失，需要 CI 实际运行来验证正确性。
  2. 同步脚本回归风险：回退 tests/special_e2e/run_fully_async_policy.sh 后，该脚本内容与工作流不再同步，未来若工作流逻辑变更，该脚本可能过时或产生误导。
  3. 语法修复覆盖面小：cpu_unit_tests.yml 的改动仅一行，风险极低，但未检查其他工作流是否存在类似遗漏。
     - 影响：直接影响 CI 系统的可用性和正确性：修复后，e2e_fully_async_policy 和 cpu_unit_tests 工作流能被 GitHub Actions 正常解析，避免因语法错误被跳过。不涉及训练、推理等用户功能，仅为基础设施维护。团队需注意 worklow 与对应特殊脚本的分离维护成本。
     - 风险标记：CI工作流重构, 脚本同步风险

关联脉络

• 暂无明显关联 PR