Skip to content

chore: bump actions/cache + actions/checkout to v5 (Node 24) - #8

Merged
Svtter merged 1 commit into
mainfrom
chore/bump-actions-to-v5
Jun 28, 2026
Merged

chore: bump actions/cache + actions/checkout to v5 (Node 24)#8
Svtter merged 1 commit into
mainfrom
chore/bump-actions-to-v5

Conversation

@Svtter

@Svtter Svtter commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

问题

GitHub 2025-09-19 起 deprecate Node 20 runner。我们的 action 在 consumer 仓库跑时出现:

##[warning]Node.js 20 is deprecated. The following actions target
Node.js 20 but are being forced to run on Node.js 24: actions/cache@v4

来源:changelog

改动

文件 改动
action.yml actions/cache@v4actions/cache@v5
.github/workflows/dogfood.yml actions/checkout@v4actions/checkout@v5

两个 v5 release 都已升级到 Node 24:

验证

  • python3 -c "import yaml; ..." 两个文件均解析成功
  • 版本字段确认:actions/cache@v5actions/checkout@v5
  • 无 v4 残留

备注

  • 无行为变化
  • 无需重建 dist/(action.yml 是 composite 定义,无代码改动)
  • consumer 仓库的 actions/checkout@v7(latex-agent 自托管内部 tag)不在本 repo 范围

GitHub is deprecating Node 20 on Actions runners (2025-09-19). Our
composite action used actions/cache@v4 and the dogfood workflow used
actions/checkout@v4, both targeting Node 20. Consumer runs flagged:

  Node.js 20 is deprecated. The following actions target Node.js 20
  but are being forced to run on Node.js 24: actions/cache@v4

Bump both to v5, which targets Node 24.

- action.yml: actions/cache@v4 -> actions/cache@v5
- .github/workflows/dogfood.yml: actions/checkout@v4 -> actions/checkout@v5

No behavior change; no dist rebuild (action.yml is composite).
@github-actions

Copy link
Copy Markdown

✅ CAN MERGE

Coordinator synthesis

CAN MERGE

三位审查员一致认为此 PR 仅涉及两个 GitHub Actions 依赖的常规版本升级(actions/checkoutactions/cache 从 v4 升级到 v5),均为官方维护的 Action,不引入破坏性变更或安全风险。无任何阻塞性问题,整体变更范围极小,可以合并。

Blocking Issues

Warnings

  1. 文档 skills/setup-pi-review/references/inputs.md(第 39 行)仍引用 actions/cache@v4,未随本次变更同步更新。建议一并更新,避免文档与实际代码不一致。

Suggestions

  1. 建议在升级后人工验证一次完整的 dogfood 工作流运行结果,确保缓存恢复(actions/cache@v5)和代码检出(actions/checkout@v5)均正常工作,尤其是在自托管 Runner 环境中(若适用)。
  2. 若对供应链安全或 CI 可复现性有更高要求,可考虑将版本号固定到具体 SHA 摘要或具体小版本(如 v5.4.0),但当前 major-version 标签的用法是社区主流做法,非强制。
  3. 建议确认 actions/cache@v5 在你使用的 runner 镜像上已确认 Node 20 运行时可用(虽然绝大多数 GitHub 托管 runner 已支持)。
quality · $0.000248 · cacheRead 1408

Review Result

CAN MERGE

本次变更将两个 GitHub Actions 依赖从 v4 升级到 v5,分别是 actions/checkout(dogfood workflow 中)和 actions/cache(action.yml 的 restore-session 步骤中)。这是常规的依赖版本升级,没有引入破坏性变更的迹象。

Blocking Issues: None

Warnings: None

Suggestions:

  1. 建议验证 actions/cache@v5 在你使用的 runner 镜像上是否已确认 Node 20 运行时可用(虽然绝大多数 GitHub 托管 runner 已支持)。
  2. 若对供应链安全有更高要求,可考虑将版本号固定到具体 SHA 摘要(例如 actions/cache@<full-sha>),但当前 major-version 标签的用法是社区主流做法,非强制。
security · $0.000118 · cacheRead 5888

CAN MERGE

本次提交仅将两个正式 GitHub Action 从 v4 升级到 v5actions/checkout(dogfood 工作流)和 actions/cache(action.yml)。两者均为官方维护的 Action,版本升级是常规的依赖更新,不引入新的攻击面或功能安全风险。变更范围极小,无语法或逻辑缺陷。

Blocking Issues: None

Warnings:

  1. 文档 skills/setup-pi-review/references/inputs.md(第 39 行)仍引用 actions/cache@v4,未随本次变更同步更新。建议一并更新,避免文档与实际代码不一致。

Suggestions: None

performance · $0.000582 · cacheRead 1280

审查结果

CAN MERGE

总结

此 PR 仅涉及两个 GitHub Actions 版本号的常规升级:

  • .github/workflows/dogfood.yml 中的 actions/checkoutv4 升级到 v5
  • action.yml 中的 actions/cachev4 升级到 v5

两个升级均为同一系列内的次要/补丁版本更新,不涉及破坏性变更。actions/checkout@v5actions/cache@v5 均使用 Node 20 运行环境(与 v4 一致),对缓存键格式、参数接口、行为语义均无影响。

Blocking Issues

None

Warnings

None

Suggestions

  • 建议在升级后人工确认一次完整的 dogfood 工作流运行结果,确保缓存恢复(actions/cache@v5)和代码检出(actions/checkout@v5)均正常工作,尤其是在自托管 Runner 环境中(若适用)。
  • 考虑将 v5 固定为具体的小版本(如 v5.4.0),以消除未来偶发破坏性变更的风险,获得更可复现的 CI 行为。

pi-review-agent · total cost $0.001189 · cacheRead 9344

@Svtter
Svtter merged commit c5d4011 into main Jun 28, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant