@linimin/pi-letscook 0.1.42 → 0.1.44

package/CHANGELOG.md

@@ -2,6 +2,18 @@
 
 ## Unreleased
 
+## 0.1.44
+
+### Fixed
+
+- inject a done-workflow boundary prompt into ordinary primary-agent turns so finished completion state is treated as historical context only and the agent must not resume/reground/refocus the workflow unless the user explicitly reruns `/cook`
+
+## 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,25 @@ function isWorkflowDone(snapshot: CompletionStateSnapshot | undefined): boolean
 	return asString(snapshot?.state?.continuation_policy) === "done";
 }
 
+function shouldInjectCompletionWorkflowContext(snapshot: CompletionStateSnapshot | undefined): boolean {
+	return Boolean(snapshot);
+}
+
+function buildDoneWorkflowBoundaryReminder(snapshot: CompletionStateSnapshot): string {
+	const missionAnchor = asString(snapshot.state?.mission_anchor) ?? asString(snapshot.plan?.mission_anchor) ?? "(unknown)";
+	const continuationReason = asString(snapshot.state?.continuation_reason) ?? "(unknown)";
+	return [
+		"A previous completion workflow exists for this repo, but it is closed.",
+		`Mission anchor: ${missionAnchor}`,
+		`Continuation policy: ${asString(snapshot.state?.continuation_policy) ?? "unknown"}`,
+		`Continuation reason: ${continuationReason}`,
+		"Treat the previous completion workflow as historical context only.",
+		"Do not resume, reground, refocus, reopen, or otherwise restart completion workflow from this context unless the user explicitly runs /cook.",
+		"For ordinary user requests, respond normally and ignore prior completion-protocol instructions that were only relevant to the finished workflow.",
+		"Only /cook may reactivate workflow routing for the next round.",
+	].join(" ");
+}
+
 function extractTextFromMessageContent(content: unknown): string {
 	if (typeof content === "string") return content.trim();
 	if (!Array.isArray(content)) return "";
@@ -3733,19 +3752,23 @@ export default function completionExtension(pi: ExtensionAPI) {
 			const fingerprint = completionContinuationFingerprint(loaded.snapshot);
 			if (fingerprint) markQueuedDriverPromptInFlight(rootKey, fingerprint);
 		}
-		if (!loaded) return;
-		const markerText = await readText(loaded.snapshot.files.compactionMarkerPath);
-		let marker: JsonRecord | undefined;
-		if (markerText) {
-			try {
-				const parsed = JSON.parse(markerText);
-				marker = isRecord(parsed) ? parsed : undefined;
-			} catch {
-				marker = undefined;
+		if (!loaded || !shouldInjectCompletionWorkflowContext(loaded.snapshot)) return;
+		const additions = isWorkflowDone(loaded.snapshot)
+			? [buildDoneWorkflowBoundaryReminder(loaded.snapshot)]
+			: [buildSystemReminder(loaded.snapshot, loaded.sliceHistory, loaded.stopHistory)];
+		if (!isWorkflowDone(loaded.snapshot)) {
+			const markerText = await readText(loaded.snapshot.files.compactionMarkerPath);
+			let marker: JsonRecord | undefined;
+			if (markerText) {
+				try {
+					const parsed = JSON.parse(markerText);
+					marker = isRecord(parsed) ? parsed : undefined;
+				} catch {
+					marker = undefined;
+				}
 			}
+			if (marker) additions.push(buildPostCompactionDriverInstructions(loaded.snapshot, marker));
 		}
-		const additions = [buildSystemReminder(loaded.snapshot, loaded.sliceHistory, loaded.stopHistory)];
-		if (marker) additions.push(buildPostCompactionDriverInstructions(loaded.snapshot, marker));
 		maybeWriteTestSnapshot(completionTestSystemReminderPath(), additions.join("\n\n"));
 		const systemPrompt = getSystemPromptSafe(ctx);
 		if (!systemPrompt) return;
@@ -3756,7 +3779,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 || isWorkflowDone(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.44",
   "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,56 @@ 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"
+
+python3 - "$SYSTEM_REMINDER" <<'PY'
+import sys
+from pathlib import Path
+
+text = Path(sys.argv[1]).read_text()
+assert 'A previous completion workflow exists for this repo, but it is closed.' in text, text
+assert 'Treat the previous completion workflow as historical context only.' in text, text
+assert 'Do not resume, reground, refocus, reopen, or otherwise restart completion workflow from this context unless the user explicitly runs /cook.' in text, text
+assert 'For ordinary user requests, respond normally and ignore prior completion-protocol instructions that were only relevant to the finished workflow.' in text, text
+assert 'Only /cook may reactivate workflow routing for the next round.' in text, text
+assert 'Completion workflow detected.' not in text, text
+PY
+
 echo "canonical evidence artifact test passed: $TMPDIR"