@linimin/pi-letscook 0.1.42 → 0.1.43

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## Unreleased
 
+## 0.1.43
+
+### Fixed
+
+- stopped injecting completion-workflow reminder and compaction-resume context into ordinary primary-agent turns after canonical `continuation_policy` reaches `done`, so users must rerun `/cook` before the workflow protocol reactivates
+
 ## 0.1.42
 
 ### Changed

package/extensions/completion/index.ts

@@ -562,6 +562,10 @@ function isWorkflowDone(snapshot: CompletionStateSnapshot | undefined): boolean
 	return asString(snapshot?.state?.continuation_policy) === "done";
 }
 
+function shouldInjectCompletionWorkflowContext(snapshot: CompletionStateSnapshot | undefined): boolean {
+	return Boolean(snapshot) && !isWorkflowDone(snapshot);
+}
+
 function extractTextFromMessageContent(content: unknown): string {
 	if (typeof content === "string") return content.trim();
 	if (!Array.isArray(content)) return "";
@@ -3733,7 +3737,7 @@ export default function completionExtension(pi: ExtensionAPI) {
 			const fingerprint = completionContinuationFingerprint(loaded.snapshot);
 			if (fingerprint) markQueuedDriverPromptInFlight(rootKey, fingerprint);
 		}
-		if (!loaded) return;
+		if (!loaded || !shouldInjectCompletionWorkflowContext(loaded.snapshot)) return;
 		const markerText = await readText(loaded.snapshot.files.compactionMarkerPath);
 		let marker: JsonRecord | undefined;
 		if (markerText) {
@@ -3756,7 +3760,7 @@ export default function completionExtension(pi: ExtensionAPI) {
 
 	pi.on("session_before_compact", async (event, ctx) => {
 		const loaded = await loadCompletionDataForReminder(getCtxCwd(ctx));
-		if (!loaded) return;
+		if (!loaded || !shouldInjectCompletionWorkflowContext(loaded.snapshot)) return;
 		const { preparation } = event;
 		const summary = buildResumeCapsule(loaded.snapshot, loaded.sliceHistory, loaded.stopHistory);
 		await fsp.mkdir(loaded.snapshot.files.tmpDir, { recursive: true });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@linimin/pi-letscook",
-  "version": "0.1.42",
+  "version": "0.1.43",
   "description": "Pi package for long-running completion workflows with canonical .agent state, role-based subagents, continuity, and verification helpers.",
   "license": "MIT",
   "private": false,

package/scripts/canonical-evidence-artifact-test.sh

@@ -392,4 +392,48 @@ assert 'Verification evidence summary:' in text, text
 assert 'selected_slice' in text, text
 PY
 
+python3 - <<'PY'
+import json
+from pathlib import Path
+
+state_path = Path('.agent/state.json')
+plan_path = Path('.agent/plan.json')
+active_path = Path('.agent/active-slice.json')
+
+state = json.loads(state_path.read_text())
+state.update({
+    'current_phase': 'done',
+    'continuation_policy': 'done',
+    'continuation_reason': 'Fixture is complete; ordinary primary-agent turns should stay outside completion until /cook runs again.',
+    'project_done': True,
+    'remaining_high_value_gaps': 0,
+    'unsatisfied_contract_ids': [],
+    'next_mandatory_action': None,
+    'next_mandatory_role': None,
+    'remaining_stop_judges': 0,
+    'contract_status': 'done',
+})
+state_path.write_text(json.dumps(state, indent=2) + '\n')
+
+plan = json.loads(plan_path.read_text())
+for slice_data in plan.get('candidate_slices', []):
+    if slice_data.get('slice_id') == 'evidence-fixture':
+        slice_data['status'] = 'done'
+plan_path.write_text(json.dumps(plan, indent=2) + '\n')
+
+active = json.loads(active_path.read_text())
+active['status'] = 'done'
+active_path.write_text(json.dumps(active, indent=2) + '\n')
+PY
+
+rm -f "$SYSTEM_REMINDER"
+PI_COMPLETION_TEST_SYSTEM_REMINDER_PATH="$SYSTEM_REMINDER" \
+pi -e "$PKG_ROOT" -p "Summarize the latest release briefly." \
+  >"$TMPDIR/pi-canonical-evidence-done-reminder.out" 2>"$TMPDIR/pi-canonical-evidence-done-reminder.err"
+
+[[ ! -f "$SYSTEM_REMINDER" ]] || {
+  echo "expected no completion reminder snapshot when continuation_policy is done" >&2
+  exit 1
+}
+
 echo "canonical evidence artifact test passed: $TMPDIR"