feat(dispatch): port review to harness CEL trigger

[ifireball]

What is missing

review routing logic lives in bash in reusable-dispatch.yml. It is not expressed as a CEL trigger: on the harness file, so routing cannot be customized without editing shared dispatch bash.

What should happen

One vertical PR ports review only:

Harness

• Add trigger: CEL expressing today's bash routing for: PR opened/sync, ready-for-review, /fs-review, bot review rules
• CEL lint/type-check in the same PR

Gating

• Decide review-specific gates (may differ from GCD): what moves into fullsend dispatch, what stays in workflow, whether new ADR or harness format extension is needed
• Decide whether roles: config check still applies for review in a custom-agents world

Workflow

• harness-dispatch handles review when CEL matches
• Bash route still runs in parallel for shadow/diff until cutover epic child
• Do not remove bash routing for review in this PR

Tests (same PR)

• Extend behaviour scenarios from test(behaviour): add review dispatch trigger scenarios (bash routing) #2892 to cover harness-dispatch path
• Unit parity golden: normative fixtures for review events match bash routing

Prerequisites

• feat(dispatch): add harness-dispatch job with ADR 54 auth, kill switch, and custom-agent behaviour test #2889 (harness-dispatch + custom agent)
• test(behaviour): add review dispatch trigger scenarios (bash routing) #2892 (review bash trigger behaviour baseline)

Merge criterion

review behaviour scenarios green on harness-dispatch; shadow diff shows parity with bash for its trigger set.

Related

Part of epic #2888

[fullsend-ai-triage]

🤖 Finished Triage · ✅ Success · Started 8:34 AM UTC · Completed 8:37 AM UTC
Commit: 9f4f2b6 · View workflow run →

[fullsend-ai-triage]

Triage: blocked on prerequisites

This issue is well-structured and ready to implement once its dependencies land. Two prerequisites remain open:

• feat(dispatch): add harness-dispatch job with ADR 54 auth, kill switch, and custom-agent behaviour test #2889 — harness-dispatch job with ADR 54 auth, kill switch, and custom-agent behaviour test. This provides the harness-dispatch infrastructure that review CEL triggers will plug into.
• test(behaviour): add review dispatch trigger scenarios (bash routing) #2892 — review dispatch trigger scenarios (bash routing baseline). This provides the normative behaviour fixtures that the new harness-dispatch path must match for parity.

Both must merge before work on this issue can begin meaningfully. The issue scope is clear: add trigger: CEL expressions to the review harness file, wire harness-dispatch to handle review when CEL matches, run bash routing in parallel for shadow/diff, and extend the behaviour scenarios from #2892 to cover the harness-dispatch path.

Once both prerequisites are resolved, this issue should be ready to code without further clarification.

Blocked by:

• feat(dispatch): add harness-dispatch job with ADR 54 auth, kill switch, and custom-agent behaviour test #2889
• test(behaviour): add review dispatch trigger scenarios (bash routing) #2892