chore(ci): park the review swarm until it is a RelayFlow v2 flow

Closed
AG
2mo ago
+20-2

Stops review-swarm.yml running on pull_request. Tracked for the real fix in #470.

Why

The review check failed on every open PR — #451, #455, #441 all identical, in the swarm's own posting step rather than on a code verdict:

Fetching patch for run <uuid>...
No changes to sync — the workflow did not modify any files.
##[error]Process completed with exit code 1

Reproducible across reruns with fresh swarm ids, so not flaky infrastructure. A check that fails on every PR carries no signal — it can neither approve nor block, and in practice it gets merged over. Leaving it red only trains people to ignore a red check.

The failure is the v1 design showing through: the gate assumes a review's product is a patch to sync back, so a review that correctly proposes no code change reads as a failure. It also drives RelayFlow v1 (agent-relay cloud run ... --sync-code), the surface relay#1783 deprecated. Both argue for the v2 port rather than a repair here.

What this does not do

workflow_dispatch is a parking trigger, not a working manual path — nine steps read github.event.pull_request.*, empty off a pull_request event. Wiring that up is deliberately left undone: it is work spent on the surface #470 retires. The comment in the file says so, so nobody dispatches it expecting a run.

The wrapper guard (review-swarm-wrapper-guard.yml) is untouched and still runs on pull_request_target reading from base, so a candidate PR still cannot weaken the gate that will judge it once it returns.

🤖 Generated with Claude Code

[!NOTE]
Low Risk
CI-only change that stops a always-failing check; no application or security logic is modified.

Overview
Parks the Review swarm CI check so it no longer runs on every pull request.

The review-swarm.yml trigger moves from pull_request to workflow_dispatch only, with an in-file comment documenting why: the v1 gate failed on every PR when no patch was synced, drove deprecated RelayFlow v1, and a manual dispatch path is intentionally not wired (steps still expect pull_request context). The real fix is tracked in AgentWorkforce/flows#470; the wrapper guard workflow is unchanged.

Reviewed by Cursor Bugbot for commit 61116129c9d600a8e0672d57854123b3bde174f5. Bugbot is set up for automated code reviews on this repo. Configure here.

Summary by cubic

The review check was failing on every open PR in its own posting step (No changes to sync), so this parks review-swarm.yml instead of repairing the v1 design. It no longer runs on pull_request; the real fix is the RelayFlow v2 port tracked in #470.

  • The trigger is now workflow_dispatch, which is a parking trigger only: the steps still read pull_request event data, so a manual dispatch will not run the swarm.
  • The wrapper guard is unchanged and still runs on pull_request_target, so the base branch keeps its protection while the workflow is parked.