Skip to content

移除 flow::review,并将 review 改为独立 PR/MR 自动检查任务 #1

Description

@nil4u

目标

移除 issue 状态机中的 flow::review,避免用户误以为给 source issue 贴上该 label 会触发 review;将 review 重新定位为独立的 PR/MR 自动检查任务,并提供单独开关控制是否自动运行。

背景与已查证现状

  • skills/issue-flow/scripts/apply.cjs 与项目级 .agentrix/plugins/issue-flow/.../apply.cjs 当前都允许 flow::review
  • skills/issue-flow/scripts/resolve.cjs 当前会把 flow::review 解析为 review action。
  • Agentrix runtime 的 SUPPORTED_ACTIONS 当前只有 triageplanbuildgeneral,不支持 review
  • test/dispatch.test.cjs 当前明确断言 flow::review 会因为 runtime 不支持而 unsupported_flow skip。
  • skills/issue-flow/references/labels.md 仍把 flow::review 记录为可自动化 flow。
  • docs/state-machine.md 的主状态机没有 review 节点,但路由/label 文档和脚本仍保留 review。

需求

  • 从 issue managed label 体系中移除 flow::review
  • 保持 flow:: 只表达 source issue 的下一步动作,例如 triage、plan、build、clarify、approve。
  • 将 review 设计为 PR/MR 级别的独立检查任务,可在 PR/MR opened、synchronize、ready_for_review 或人工触发时运行。
  • 增加独立配置开关,例如 ISSUE_FLOW_REVIEW_ENABLED=true,不要复用 automation::ISSUE_FLOW_AUTO_DEFAULT 表达 review 策略。
  • 更新文档、安装产物、运行时路由和测试,避免继续声明或接受 flow::review

验收标准

  • apply.cjs --flow flow::review 不再被接受。
  • resolve.cjs 和 dispatch/runtime 不再把 issue label flow::review 当作可执行 issue action。
  • label 参考文档、skill 文档、状态机文档和 README 不再把 flow::review 描述为 issue flow。
  • 新增或调整 PR/MR review job/action 的文档与安装产物,并说明 ISSUE_FLOW_REVIEW_ENABLED 的默认行为。
  • 相关测试覆盖旧 flow::review 行为被移除/拒绝,以及新 review 入口的启用和禁用行为。

非目标

  • 不新增 automation::review
  • 不把 review 加入 issue 的 flow:: 状态机。
  • 不把 source issue 的 flow::approve 语义替换为 review。

Triage 结论

这是一个功能/流程设计变更,影响 label schema、routing、runtime/workflow 安装产物、文档和测试。下一步应先进入 plan,确定 PR/MR review job 的触发入口、runtime action 形态和配置兼容策略;plan 通过后可自动进入 build。

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    automation::buildAutomation may execute the configured plan flow and create build PRs/MRspriority::p2Normal priority issuestatus::doneIssue is completetype::featureIssue is a feature or enhancement

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions