From 71905bda32e4e969e0ab9dd6991f4688dc23ef21 Mon Sep 17 00:00:00 2001 From: Bryan Ramos Date: Wed, 1 Apr 2026 17:20:25 -0400 Subject: [PATCH] fix: only mark plan step complete when all assigned workers receive LGTM --- skills/orchestrate/SKILL.md | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/skills/orchestrate/SKILL.md b/skills/orchestrate/SKILL.md index 08d7798..b76a654 100644 --- a/skills/orchestrate/SKILL.md +++ b/skills/orchestrate/SKILL.md @@ -242,7 +242,8 @@ When multiple risk tags are present, take the union of all mandatory reviewers. ### Git flow Workers signal `RFR` when done. You control commits: -- `LGTM` → worker commits → **update the plan file: mark the completed step as `- [x]`** +- `LGTM` → worker commits +- **Mark a step `- [x]` in the plan file only when every worker assigned to that step has received LGTM** — a single worker committing does not complete a step - `REVISE` → worker fixes and resubmits with `RFR` - Merge worktree branches after individual validation - On Tier 2+: merge each worker's branch after validation, resolve conflicts if branches overlap