ci: 增加 actionlint 工作流校验 - #286
Conversation
There was a problem hiding this comment.
本 PR 为 CI 增加 actionlint 工作流校验、统一 CODEX/APP 配置名称并移除旧的 OPENAI_API_KEY/OPENAI_API_ENDPOINT 回退,整体方向清晰,actionlint 安装步骤也做了 sha256 校验。但 review-pr.yml 把 allow-bot-users: github-actions[bot] 改成了 allow-bots: github-actions[bot]:openai/codex-action@v1 的 allow-bots 是布尔输入(默认 false,见 action.yml/README),传入非布尔字符串会导致 action 内 check-write-access 的 parseBoolean 抛出 Invalid boolean value,使 Run AI review 这一步直接失败,整个 review 流程不可用。修复时需同时更新 .github/aicodingflow-tests/test_review_workflow_dispatch.py:152 中的断言。
|
@kratos-ci-bot /review |
There was a problem hiding this comment.
审查范围:23 个文件,主要包含新增 actionlint CI 校验、CODEX_*/APP_ID 配置重命名及文档同步。已在 pr-worktree 中运行 .github/tests(13 项通过)以及相关 .github/aicodingflow-tests(test_codex_model_provider_config.py、test_review_workflow_dispatch.py 全部通过),并逐一核对 11 个 Codex workflow 与测试断言一致。
结论:未发现阻塞性问题,建议合并。历史讨论中已解决的 review-pr.yml 的 allow-bots 布尔输入问题,本 diff 已按建议改为 allow-bots: true,不再重复提出。
非阻塞项:
- 文档措辞不一致:
README.md:57与docs/github-collaboration-flow.md:25将CODEX_MODEL描述为“必须配置”,但同文档表格(README.md:41、docs/github-collaboration-flow.md:17)写的是留空时回退 Codex 默认模型,workflow 也没有对CODEX_MODEL做非空校验。详见行内评论。 - 配置迁移为破坏性变更:移除
OPENAI_API_KEY/OPENAI_API_ENDPOINT回退并将 GitHub App variable 从APP_CLIENT_ID改为APP_ID后,已有目标仓库需手动迁移;建议在发布说明或升级指引中明确提示。
|
Applied requested changes on Summary: 摘要
验证
|
摘要
.github/workflows/下的 YAML 工作流。验证
PYTHONDONTWRITEBYTECODE=1 .venv/bin/python -m unittest discover -s .github/testsPYTHONDONTWRITEBYTECODE=1 .venv/bin/python -m unittest discover -s .github/aicodingflow-testsgit diff --checkRefs #285