@linimin/pi-letscook 0.1.43 → 0.1.44

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## 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

package/extensions/completion/index.ts

@@ -563,7 +563,22 @@ function isWorkflowDone(snapshot: CompletionStateSnapshot | undefined): boolean
 }
 
 function shouldInjectCompletionWorkflowContext(snapshot: CompletionStateSnapshot | undefined): boolean {
-	return Boolean(snapshot) && !isWorkflowDone(snapshot);
+	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 {
@@ -3738,18 +3753,22 @@ export default function completionExtension(pi: ExtensionAPI) {
 			if (fingerprint) markQueuedDriverPromptInFlight(rootKey, fingerprint);
 		}
 		if (!loaded || !shouldInjectCompletionWorkflowContext(loaded.snapshot)) 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;
+		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;
@@ -3760,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 || !shouldInjectCompletionWorkflowContext(loaded.snapshot)) 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.43",
+  "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

@@ -431,9 +431,17 @@ 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
-}
+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"