opencode-empirical-plan 0.2.0 → 0.3.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-empirical-plan",
-  "version": "0.2.0",
+  "version": "0.3.1",
   "description": "OpenCode lifecycle plugin for empirical plan orchestration.",
   "author": "chuck",
   "license": "MIT",

package/src/state/lifecycle-state.ts

@@ -1,6 +1,25 @@
 import * as fs from "node:fs/promises";
 import * as path from "node:path";
-import { type LifecycleState, STATE_FILENAME, validateLifecycleState } from "../schema.ts";
+import { type LifecyclePhase, type LifecycleState, STATE_FILENAME, validateLifecycleState } from "../schema.ts";
+
+const PHASE_ORDER: LifecyclePhase[] = ["plan", "execute", "reflect", "done"];
+
+export async function assertPhaseAtLeast(
+  runDir: string,
+  required: LifecyclePhase,
+): Promise<{ ok: true } | { ok: false; reason: string }> {
+  const state = await readLifecycleState(runDir);
+  if (!state) return { ok: true };
+  const current = PHASE_ORDER.indexOf(state.phase);
+  const needed = PHASE_ORDER.indexOf(required);
+  if (current < needed) {
+    return {
+      ok: false,
+      reason: `lifecycle phase is "${state.phase}", expected at least "${required}"`,
+    };
+  }
+  return { ok: true };
+}
 
 export async function readLifecycleState(runDir: string): Promise<LifecycleState | null> {
   const stateFile = path.join(runDir, STATE_FILENAME);

package/src/tools/lifecycle-record-execution.ts

@@ -3,6 +3,7 @@ import * as fs from "node:fs/promises";
 import * as path from "node:path";
 import { atomicWrite } from "../state/frontmatter.ts";
 import { ensureExecutionMd } from "../hooks/tool-after.ts";
+import { assertPhaseAtLeast } from "../state/lifecycle-state.ts";
 
 const SUMMARY_MARKER = "<!-- EXECUTION:SUMMARY -->";
 
@@ -25,8 +26,9 @@ export const lifecycleRecordExecutionTool = tool({
 execution.md is the dedicated execution record file — separate from plan.md (which stays read-only after Phase 1-3).
 
 Guards (hard-block):
-- plan.md must exist and contain "## Plan Steps" (Phase 1-3 complete)
-- execution.md must already exist (initialized by the tool.execute.after hook when EXECUTE phase began)
+- lifecycle-state.json phase must be >= "execute" (fail-open if state file missing)
+- plan.md must exist
+- execution.md must exist (auto-created if missing)
 - <!-- EXECUTION:SUMMARY --> must NOT already exist in execution.md (idempotent guard)
 
 On success: appends the execution_summary section into execution.md and updates its Phase frontmatter field to "summarized".
@@ -47,33 +49,33 @@ The execution_summary should compare each Plan Step against what actually happen
     const planFile = path.join(runDir, "plan.md");
     const executionFile = path.join(runDir, "execution.md");
 
-    const result = await doRecord(planFile, executionFile, args.execution_summary);
+    const result = await doRecord(runDir, planFile, executionFile, args.execution_summary);
     return JSON.stringify(result);
   },
 });
 
 async function doRecord(
+  runDir: string,
   planFile: string,
   executionFile: string,
   executionSummary: string,
 ): Promise<RecordResult> {
-  // Guard 1: plan.md must exist with Phase 1-3 complete
-  if (!(await fileExists(planFile))) {
+  const phaseCheck = await assertPhaseAtLeast(runDir, "execute");
+  if (!phaseCheck.ok) {
     return {
       success: false,
       code: "PHASE_NOT_READY",
-      reason: `plan.md not found at ${planFile}`,
+      reason: phaseCheck.reason,
       next_action: "Complete the PLAN phase first — run empirical-plan skill to produce plan.md",
     };
   }
 
-  const planContent = await fs.readFile(planFile, "utf-8");
-  if (!planContent.includes("## Plan Steps")) {
+  if (!(await fileExists(planFile))) {
     return {
       success: false,
       code: "PHASE_NOT_READY",
-      reason: "plan.md does not contain '## Plan Steps' — Phase 1-3 incomplete",
-      next_action: "Complete the PLAN phase using the empirical-plan skill",
+      reason: `plan.md not found at ${planFile}`,
+      next_action: "Complete the PLAN phase first — run empirical-plan skill to produce plan.md",
     };
   }
 

package/src/tools/lifecycle-record-reflection.ts

@@ -2,8 +2,8 @@ import { tool } from "@opencode-ai/plugin";
 import * as fs from "node:fs/promises";
 import * as path from "node:path";
 import { applyFrontmatterLifecycle, atomicWrite } from "../state/frontmatter.ts";
+import { assertPhaseAtLeast } from "../state/lifecycle-state.ts";
 
-const EXECUTION_SUMMARY_MARKER = "<!-- EXECUTION:SUMMARY -->";
 const REFLECTION_MARKER = "<!-- LIFECYCLE:REFLECTION -->";
 
 async function fileExists(p: string): Promise<boolean> {
@@ -19,7 +19,8 @@ export const lifecycleRecordReflectionTool = tool({
   description: `Record Phase 5 (Reflection & Optimization) into plan.md.
 
 Guards (hard-block):
-- execution.md must exist and contain <!-- EXECUTION:SUMMARY --> (Phase 4 already recorded)
+- lifecycle-state.json phase must be >= "reflect" (fail-open if state file missing)
+- execution.md must exist
 - reflect.md must exist in run_dir (REFLECT phase complete)
 - <!-- LIFECYCLE:REFLECTION --> must NOT already exist in plan.md (idempotent guard)
 
@@ -40,7 +41,16 @@ The reflection_record should contain: execution quality assessment, plan design
     const executionFile = path.join(runDir, "execution.md");
     const reflectFile = path.join(runDir, "reflect.md");
 
-    // Guard 1: plan.md must exist
+    const phaseCheck = await assertPhaseAtLeast(runDir, "reflect");
+    if (!phaseCheck.ok) {
+      return JSON.stringify({
+        success: false,
+        code: "PHASE_NOT_READY",
+        reason: phaseCheck.reason,
+        next_action: "Complete EXECUTE phase and call lifecycle_record_execution first",
+      });
+    }
+
     if (!(await fileExists(planFile))) {
       return JSON.stringify({
         success: false,
@@ -50,7 +60,7 @@ The reflection_record should contain: execution quality assessment, plan design
       });
     }
 
-    // Guard 2: execution.md must exist and contain EXECUTION:SUMMARY
+    // Guard 2: execution.md must exist
     if (!(await fileExists(executionFile))) {
       return JSON.stringify({
         success: false,
@@ -60,16 +70,6 @@ The reflection_record should contain: execution quality assessment, plan design
       });
     }
 
-    const executionContent = await fs.readFile(executionFile, "utf-8");
-    if (!executionContent.includes(EXECUTION_SUMMARY_MARKER)) {
-      return JSON.stringify({
-        success: false,
-        code: "EXECUTION_NOT_RECORDED",
-        reason: "Phase 4 summary (EXECUTION:SUMMARY) not yet recorded in execution.md",
-        next_action: "Call lifecycle_record_execution first to record Phase 4",
-      });
-    }
-
     // Guard 3: reflect.md must exist
     if (!(await fileExists(reflectFile))) {
       return JSON.stringify({