Skip to content

fix: update action refs from v1 to v2#42

Merged
Svtter merged 1 commit into
mainfrom
fix/update-action-version-to-v2
Apr 29, 2026
Merged

fix: update action refs from v1 to v2#42
Svtter merged 1 commit into
mainfrom
fix/update-action-version-to-v2

Conversation

@Svtter

@Svtter Svtter commented Apr 29, 2026

Copy link
Copy Markdown
Collaborator

Summary

Update action references from v1 to v2 to fix missing input parameter errors.

Changes

  • feature-missing.yml: Svtter/opencode-actions/feature-missing@v1 → @v2
  • review.yml: Svtter/opencode-actions/review@v1 → @v2

Root Cause

Both workflows pass deepseek-api-key input, but v1 tag does not define this input parameter. v2 includes deepseek-api-key support.

This fixes the 'Unexpected input(s) deepseek-api-key' warning and ProviderModelNotFoundError failure.

@Svtter Svtter force-pushed the fix/update-action-version-to-v2 branch from e0b4230 to 898f506 Compare April 29, 2026 04:10
@github-actions

Copy link
Copy Markdown

发现遗漏

概要

PR 声称将 action 引用从 v1 升级到 v2 并添加 deepseek-api-key 支持,但实际有多个引用未更新、关键测试会失败、新输入参数未写入文档。

CRITICAL

1. 测试 test_uses_review_v1 会因 workflow 已改为 v2 而失败

  • tests/test_all.py:671 断言 self.assertIn("uses: Svtter/opencode-actions/review@v1", content),但 .github/workflows/review.yml 已更新为 @v2。该测试是本次 PR 新加的,却与 PR 自身的变更冲突,直接导致测试套件失败。
  • 需要在 tests/test_all.py 中将该断言改为 @v2

MEDIUM

2. README 文档中的代码示例仍使用 @v1

  • README.md:144-149 列出 5 个 public consumer 示例(review, feature-missing, github-run-opencode, setup-opencode, run-opencode)全部为 @v1,未更新到 @v2
  • README.md:154-158 的 review 示例同样为 @v1
  • 应将这些文档引用更新为 @v2

3. 示例文件仍使用 @v1

  • examples/opencode-feature-missing.yml:23uses: Svtter/opencode-actions/feature-missing@v1
  • examples/opencode-review.yml:23uses: Svtter/opencode-actions/review@v1
  • 两个示例文件均未更新为 @v2

4. deepseek-api-key 输入参数未写入 README 文档

  • PR body 明确指出添加 deepseek-api-key 是本次修复的核心目的(v1 缺少此输入导致 "Unexpected input(s) 'deepseek-api-key'" 错误),三个 action YML 文件均已添加该输入,但 README.mdgithub-run-opencode 输入参数表(第 53 行附近)缺少 deepseek-api-key 一行,用户无法从文档得知此参数的存在。
  • 应在 README 的 github-run-opencode 输入参数表中新增 deepseek-api-key 行,与 zhipu-api-keyopencode-go-api-key 并列。

LOW

5. runner 类型变更未在 PR 描述中说明

  • .github/workflows/feature-missing.ymlruns-on: self-hosted 改为 ubuntu-latest
  • .github/workflows/update-major-tag.ymlruns-on: ubuntu-latest 改为 self-hosted
  • 这两项基础设施变更是 PR 的隐式改动,但与 PR 标题/正文声明的"更新 action 版本号"无关,且未在 PR 描述中说明。虽然不是功能缺失,但属于文档/沟通遗漏。

New%20session%20-%202026-04-29T04%3A09%3A24.829Z
opencode session  |  github run

@github-actions

Copy link
Copy Markdown

可合并

PR 将 Svtter/opencode-actions/*@v1 升级为 @v2,并在 feature-missing.yml 中补充了缺失的 modelopencode-go-api-key 参数。远程已存在 v2 标签,action.yml 均正确定义了相关 inputs,修复了 "Unexpected input 'deepseek-api-key'" 的问题。

阻塞项:无

建议项:无

New%20session%20-%202026-04-29T04%3A10%3A55.119Z
opencode session  |  github run

@github-actions

Copy link
Copy Markdown

无遗漏

PR 的 spec(从标题和正文提取)只有两点:(1) 将 action 引用从 @v1 更新为 @v2;(2) 修复 deepseek-api-key 输入参数缺失的问题。经检查:

  • 工作流文件中的 @v1 引用已全部更新为 @v2
  • deepseek-api-key 输入在 v2 对应的 action.yml 中均已定义 ✅
  • feature-missing.yml 额外补充了 model 输入传递,属于正向改进
  • reasoning-effort / enable-thinking 等新增输入从 action.yml → 环境变量 → Python 处理 → README 文档 → 示例文件,链路完整,无遗漏
  • runner 类型变更(ubuntu-latestself-hosted)不在 spec 范围内,不属于遗漏

所有 spec 需求均已覆盖,不存在实现缺漏。

New%20session%20-%202026-04-29T04%3A09%3A11.741Z
opencode session  |  github run

@github-actions

Copy link
Copy Markdown

有条件合并

该 PR 的主要目的是将 action 引用从 v1 升级到 v2,并添加 reasoning-effort / enable-thinking 输入支持及 deepseek-api-key 支持。除了以下阻塞项外,整体代码质量良好。

阻塞项:

  1. smoke-test.yml 和 update-major-tag.yml 的 runs-on 被错误回退smoke-test.yml:8update-major-tag.yml:13ubuntu-latest 被改回 self-hosted。这是合并冲突解决时错误保留了分支旧版本的 self-hosted,回退了 PR ci: switch from self-hosted to ubuntu-latest #41 的修复。修复提交 e0b4230 只修正了 review.yml,遗漏了这两个文件。若不修复,这两个 workflow 在没有 self-hosted runner 的仓库中会直接失败。

  2. 示例文件和 README 中的 action 引用仍为 @v1examples/opencode-review.yml:23examples/opencode-feature-missing.yml:23 及 README 中多处用法示例仍使用 @v1。既然正式将 action 升级到 v2,示例也应当同步更新,避免误导用户继续使用旧版本。

建议项:

  1. run-github-opencode.py:169enable_thinking == "true" 区分大小写,建议使用 .lower() == "true" 增强健壮性,防止用户输入 True/TRUE 时判断失败。

New%20session%20-%202026-04-29T04%3A09%3A22.935Z
opencode session  |  github run

@github-actions

Copy link
Copy Markdown

We couldn't respond to your request in time. Sorry about that. Please try resubmitting your request and contact us if the problem persists.

opencode session  |  github run

@github-actions

Copy link
Copy Markdown

发现遗漏

概要

PR 将 .github/workflows/review.yml.github/workflows/feature-missing.yml 中的 action 引用从 @v1 更新为 @v2,并给 feature-missing 工作流补充了 model 输入。但以下项目未同步更新:

MEDIUM

  1. 测试断言未更新tests/test_all.py:687
    test_uses_review_v1 直接断言 uses: Svtter/opencode-actions/review@v1。合并后该测试必定失败。应将行 687 改为 @v2

LOW

  1. 示例文件仍引用 v1examples/opencode-review.yml:23examples/opencode-feature-missing.yml:23
    用户复制示例时会得到 @v1,无法受益于 v2 的输入定义修复。应将这两处 @v1 更新为 @v2

  2. README 文档仍推荐 v1README.md:13,86,141-145,150,182,192
    多处说明和代码块中展示 @v1,用户按文档指引会用到过时版本。应更新为 @v2

  3. examples/opencode-command.yml:68 仍引用 v1 — 虽然 github-run-opencode 的 v1 可能已定义所有输入,但为保持仓库内所有引用一致,建议一并更新为 @v2

New%20session%20-%202026-04-29T04%3A10%3A54.650Z
opencode session  |  github run

- feature-missing: v1 → v2, add model parameter
- review: v1 → v2

v2 includes deepseek-api-key support. Using MODEL_NAME var
for model selection instead of default zhipu model.
@Svtter Svtter force-pushed the fix/update-action-version-to-v2 branch from 898f506 to 28d7160 Compare April 29, 2026 04:13
@github-actions

Copy link
Copy Markdown

可合并

PR 将 7 个文件中的所有 Svtter/opencode-actions/*@v1 引用正确更新为 @v2,并在 feature-missing.yml 中补充了 deepseek-api-keymodel 输入。测试断言、示例文件、README 均已同步更新,无语法错误或功能缺失。

阻塞项:无

建议项:

  • README.md:183consumers can pin to @v1 for compatible updates 建议同步更新为 @v2,以反映当前主版本号

New%20session%20-%202026-04-29T04%3A13%3A45.620Z
opencode session  |  github run

@github-actions

Copy link
Copy Markdown

无遗漏

PR spec(从标题和正文提取)明确两点:(1) feature-missing.yml 和 review.yml 的 action 引用从 @v1 升级为 @v2;(2) 修复 deepseek-api-key 输入缺失问题(v1 未定义该参数,v2 已定义)。

经逐项验证:

  • ✅ 两个工作流文件的 action 引用已更新为 @v2
  • ✅ review.yml 新增了 deepseek-api-key 输入传递
  • ✅ feature-missing.yml 新增了 modeldeepseek-api-keyopencode-go-api-key 输入传递
  • ✅ README 中所有 action 引用已更新为 @v2deepseek-api-key 已写入输入参数表
  • ✅ 3 个示例文件全部更新为 @v2
  • ✅ 测试断言已从 @v1 改为 @v2
  • ✅ v2 对应 action.yml 均已定义 deepseek-api-key 输入并正确传递到 Python 脚本

仅剩的 @v1 引用位于 docs/releases/v1.0.0.md(历史发布说明,不应修改)。所有 spec 需求均已覆盖,无实现缺漏。

New%20session%20-%202026-04-29T04%3A13%3A48.942Z
opencode session  |  github run

@Svtter Svtter merged commit 958b0b2 into main Apr 29, 2026
3 checks 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