Skip to content

ci: add multi-review workflow#84

Merged
Svtter merged 1 commit into
mainfrom
feat/multi-review-workflow
May 19, 2026
Merged

ci: add multi-review workflow#84
Svtter merged 1 commit into
mainfrom
feat/multi-review-workflow

Conversation

@Svtter

@Svtter Svtter commented May 19, 2026

Copy link
Copy Markdown
Collaborator

Summary

Test plan

  • Merge this PR
  • Open a new test PR to verify multi-review triggers and posts synthesized comment

🤖 Generated with Claude Code

Add the GitHub Actions workflow that triggers multi-agent parallel code
review on pull requests. Uses quality + security reviewer team by default.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

架构合理

该 PR 仅新增 .github/workflows/multi-review.yml 一个文件,整体结构与现有的 architect-review.yml 高度一致(trigger 条件、permissions、checkout、git config、API key 传入等),遵循了项目已有的 CI 工作流惯例。新增的 default-team 参数是 multi-review 场景特有的合理扩展。

阻塞项:无

建议项:无

New%20session%20-%202026-05-19T10%3A39%3A49.637Z
opencode session  |  github run

@github-actions

Copy link
Copy Markdown

可有条件合并

该 PR 新增 .github/workflows/multi-review.yml,结构和用法与已有的 review.yml / architect-review.yml 一致,引用的本地 action ./multi-reviewsetup-opencode/run-opencode/ 等依赖均存在,无明显正确性问题。

阻塞项:无

建议项:

  1. 缺少 TODO 注释architect-review.yml:29# TODO: switch to Svtter/opencode-actions/architect-review@v2 after releasemulti-review.yml 没有,release 前需要补上。
  2. 缺少 concurrency 控制 — 若 PR 快速多次 push,会导致多个 review workflow 并发执行,浪费 API 配额且产生冗余评论。建议添加:
    concurrency:
      group: ${{ github.workflow }}-${{ github.event.pull_request.number }}
      cancel-in-progress: true
  3. 空行位置不一致architect-review.ymlmodelgithub-token 之间有一个空行做分隔,而 multi-review.ymldefault-team 处于分隔线之上(与 model 同组),建议统一分隔风格。

New%20session%20-%202026-05-19T10%3A39%3A11.263Z
opencode session  |  github run

@github-actions

Copy link
Copy Markdown

我已完成全面分析:

无遗漏

本 PR 正确添加了缺失的 .github/workflows/multi-review.yml CI 工作流文件,与 PR #81 构建的 multi-review action 完整集成。对比 spec(PR #81 描述 + 文档)和实际实现:

  • 触发条件(pull_request: opened/synchronize/reopened/ready_for_review)✅
  • 条件过滤(跳过 draft PR + fork PR)✅
  • 代码检出(actions/checkout@v6, PR head)✅
  • git identity 配置(opencode 内部操作需要)✅
  • 核心 inputs(model、default-team、github-token、3 个 API key)均已传入 ✅
  • action 引用方式./multi-review,与 architect-review.yml 模式一致)✅
  • 可选功能(自定义 personas、fallback models、coordinator prompt、retry/timeout 等)均通过 action 默认值可用,工作流无需显式配置 ✅

唯一可提及的是缺少 architect-review.yml 中那种 TODO: switch to Svtter/opencode-actions/multi-review@v2 after release 的注释,但这属于样式一致性而非功能遗漏。

New%20session%20-%202026-05-19T10%3A39%3A10.652Z
opencode session  |  github run

@Svtter Svtter merged commit 0040e7f into main May 19, 2026
4 of 5 checks passed
@Svtter Svtter mentioned this pull request May 19, 2026
2 tasks
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