From 0feb3b5e833f392ad8ae111177376d1cc68ac535 Mon Sep 17 00:00:00 2001 From: toki Date: Sat, 15 Aug 2026 09:37:28 +0900 Subject: [PATCH] =?UTF-8?q?fix(edge):=20worker=20review=20handoff=EB=A5=BC?= =?UTF-8?q?=20plan=EC=97=90=20=EB=AA=85=EC=8B=9C=ED=95=9C=EB=8B=A4?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- apps/edge/internal/openai/provider_model_rewrite.go | 2 +- apps/edge/internal/openai/provider_model_rewrite_test.go | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/apps/edge/internal/openai/provider_model_rewrite.go b/apps/edge/internal/openai/provider_model_rewrite.go index 25e8ff14..30c87196 100644 --- a/apps/edge/internal/openai/provider_model_rewrite.go +++ b/apps/edge/internal/openai/provider_model_rewrite.go @@ -42,7 +42,7 @@ You are the Planner. Apply this compact Plan workflow before authoring the pair: 2. Preserve every explicit requirement, constraint, deliverable, and acceptance condition. Do not invent extra scope. 3. Convert that analysis into 2-6 closed, executable steps. The Worker must be able to implement without rediscovering requirements or choosing among alternatives. 4. Write 1-3 deterministic verification bullets with observable pass conditions. -5. Make the final Plan step require the Worker to replace the exact REVIEW path above with completed item statuses, actual changes, actual verification evidence, and deviations. That Review write is the Work stage's final action. +5. Make the final Plan step require the Worker to read the pending REVIEW path above, then as the Work stage's final action replace that same file while preserving its exact headings. Every Plan item status must be exactly "completed"; actual changes, actual verification evidence, and deviations must be non-empty. Write every PLAN field in English using ASCII characters only and use exactly this grammar: # Plan diff --git a/apps/edge/internal/openai/provider_model_rewrite_test.go b/apps/edge/internal/openai/provider_model_rewrite_test.go index 627a1983..edcd929c 100644 --- a/apps/edge/internal/openai/provider_model_rewrite_test.go +++ b/apps/edge/internal/openai/provider_model_rewrite_test.go @@ -280,7 +280,8 @@ func TestHotPathSelectorPairInstructionCarriesCompactPlanContract(t *testing.T) "Analyze the immutable user task first.", "Preserve every explicit requirement, constraint, deliverable, and acceptance condition.", "The Worker must be able to implement without rediscovering requirements", - "Make the final Plan step require the Worker to replace the exact REVIEW path", + "Make the final Plan step require the Worker to read the pending REVIEW path", + "Every Plan item status must be exactly \"completed\"", "actual verification evidence", } { if !strings.Contains(instruction, fragment) {