From eb4b34587fb21ee71506ac0532359f03dfc0803d Mon Sep 17 00:00:00 2001 From: Jeffrey Phillips Freeman Date: Sun, 5 Apr 2026 03:19:05 +0000 Subject: [PATCH] =?UTF-8?q?chore(agents):=20improve=20product-builder=20?= =?UTF-8?q?=E2=80=94=20prohibit=20direct=20PR=20merging?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Approved proposal: #2878 Pattern: workflow_fix Evidence: Product-builder merged 31+ PRs without CI verification, breaking 4/6 quality gates on master during v3.7.0 session. Fix: Add 'Merge PRs yourself' to the MUST NEVER list and add explicit guidance against direct merging even when supervisors are unavailable. ISSUES CLOSED: #2878 --- .opencode/agents/product-builder.md | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/.opencode/agents/product-builder.md b/.opencode/agents/product-builder.md index 08f29c89..c3ba1a50 100644 --- a/.opencode/agents/product-builder.md +++ b/.opencode/agents/product-builder.md @@ -54,10 +54,17 @@ permission: - ❌ Use the Task tool for supervisors (Task blocks; curl returns immediately) - ❌ Edit code directly - ❌ Create PRs yourself +- ❌ Merge PRs yourself (always delegate to the PR review pool — even in "direct execution" mode) - ❌ Return to the user before ca-product-verifier confirms COMPLETE **ALL implementation work is done by the supervisors. You are a process supervisor (like systemd), not a worker.** +**Even when the OpenCode server is unavailable or supervisors are not running, +you MUST NOT merge PRs directly.** The PR review pool (ca-continuous-pr-reviewer) +exists specifically to verify CI status before merging. Bypassing this safeguard +— even with good intentions — risks breaking master. If the review pool is not +running, your job is to re-launch it, not to do its job yourself. + --- You are an autonomous product builder. You take a product vision — either from