infinity-harness 2.2.0 → 2.2.1

package/CHANGELOG.md

@@ -4,6 +4,22 @@ All notable changes to this project are documented here.
 Format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/); versions follow
 [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [2.2.1] — 2026-08-23
+
+### Fixed
+
+- **Reviewing a goal before the pipeline finished threw an internal error.** `GoalLoopStateError:
+  Cannot update goal iteration 2 from status pending` — a phase name from inside the state machine,
+  thrown at whoever called the tool. Reviewing early is legitimate: you can see a pass will not meet
+  the goal well before the pipeline agrees, and waiting for a doomed pipeline to finish first is
+  theatre. The review now records the pass itself and answers.
+
+  Found by running the shipped package against a real project, in the first minute. Every test
+  recorded a pipeline pass before reviewing, so not one of them ever asked what happens when a
+  review arrives without one.
+
+---
+
 ## [2.2.0] — 2026-08-23
 
 Nine modules shipped in this package, typechecked, and passed their tests while no code path in the

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "infinity-harness",
-  "version": "2.2.0",
+  "version": "2.2.1",
   "description": "A pi agent extension that runs a gated build pipeline unattended \u2014 enforces phases, validates with deterministic gates, and keeps working for hours or days without losing the plan.",
   "type": "module",
   "keywords": [

package/src/goal.ts

@@ -262,6 +262,17 @@ export async function reviewGoal(
     return { state, terminal: true, message: `The goal loop already finished: ${state.status}.`, rewoundTo: null };
   }
 
+  // A review can legitimately arrive before the pipeline finishes: someone can
+  // already see that this pass will not meet the goal, and making them wait
+  // for a doomed pipeline to complete first is theatre. The state machine only
+  // accepts a verdict on an iteration that has recorded its work, so record it
+  // — otherwise the caller gets `Cannot update goal iteration 2 from status
+  // pending`, which names an internal phase and helps nobody.
+  if (state.phase === "goal_received" || state.phase === "todo_generated") {
+    const caught = await recordPipelinePass(targetDir, "reviewed before the pipeline finished", now);
+    if (caught) state = caught;
+  }
+
   const at = (now ?? new Date()).toISOString();
   const remainingWork = (input.remainingWork ?? []).map((s) => s.trim()).filter(Boolean);
   if (input.decision !== "complete" && remainingWork.length === 0) {