@hallucination-studio/harness-engine 1.0.0-beta.17.412ec6e → 1.0.0-beta.18.ab4b55a

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hallucination-studio/harness-engine",
-  "version": "1.0.0-beta.17.412ec6e",
+  "version": "1.0.0-beta.18.ab4b55a",
   "description": "Install the harness-engine Codex skill for initializing and reconciling advanced repository harness docs.",
   "repository": {
     "type": "git",

package/skills/harness-engine/evals/harness_engine_evals/cases_lifecycle.py

@@ -1516,6 +1516,12 @@ def test_plan_close_moves_sidecar_and_rejects_stale_score(tmp_root):
     completed_check = run_manager("check", "--repo", str(repo))
     if completed_check["status"] != "pass":
         raise AssertionError("completed structured plan should satisfy check")
+    sidecar_before_check = completed_sidecar.read_text()
+    second_completed_check = run_manager("check", "--repo", str(repo))
+    if second_completed_check["status"] != "pass":
+        raise AssertionError("repeated completed-plan check should still pass")
+    if completed_sidecar.read_text() != sidecar_before_check:
+        raise AssertionError("check should not rewrite unchanged completed plan sidecars or bump updated_at")
 
 
 def test_evidence_prune_generated_artifacts(tmp_root):

package/skills/harness-engine/scripts/harness_engine/plans.py

@@ -172,19 +172,22 @@ def sync_plan_markdown_from_state(plan_path, state):
 def sync_state_from_markdown(plan_path, state):
     from .knowledge import extract_defect_items, extract_knowledge_items, parse_defect_item, parse_knowledge_item
     text = plan_path.read_text()
-    state["defects"] = []
+    defects = []
     for item in extract_defect_items(text):
         parsed = parse_defect_item(item)
         if parsed:
-            state["defects"].append(parsed)
-    state["knowledge_items"] = []
+            defects.append(parsed)
+    knowledge_items = []
     for item in extract_knowledge_items(text):
         if item == DEFAULT_KNOWLEDGE_PLACEHOLDER:
             continue
         parsed = parse_knowledge_item(item)
         if parsed:
-            state["knowledge_items"].append(parsed)
-    save_plan_state(plan_path, state)
+            knowledge_items.append(parsed)
+    if state.get("defects") != defects or state.get("knowledge_items") != knowledge_items:
+        state["defects"] = defects
+        state["knowledge_items"] = knowledge_items
+        save_plan_state(plan_path, state)
     return state
 
 
@@ -625,4 +628,3 @@ def close_plan(repo, plan_relative_path, summary, force):
     update_workstreams_after_plan_close(repo, active_relative_path, completed_relative_path)
     return destination, open_items
 
-