@linimin/pi-letscook 0.1.31 → 0.1.33

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## 0.1.33
+
+### Changed
+
+- kept full mission text in `/cook` confirmation instead of truncating mission anchors during derivation
+- refined `/cook` activity and completion-role text contrast by reducing overuse of `dim` styling in high-value status surfaces
+
+## 0.1.32
+
+### Changed
+
+- made `/cook` auto-continue workflows from canonical state when `continuation_policy == continue`, so the primary driver re-queues the canonical resume prompt after intermediate role turns instead of parking silently on known mandatory steps
+- added smoke coverage for the new auto-resume driver prompt behavior and a guarded parked-state warning path to avoid infinite requeue loops on an unchanged mandatory state
+
 ## 0.1.31
 
 ### Changed

package/extensions/completion/index.ts

@@ -203,7 +203,7 @@ class StartupAnalystOverlay extends Container {
 
 	private updateDisplay(): void {
 		this.title.setText(this.theme.fg("accent", this.theme.bold("/cook proposal analyst")));
-		this.body.setText(this.theme.fg("dim", this.lines.join("\n")));
+		this.body.setText(formatInlineRunningText(this.theme, this.lines, { primaryAssistant: true }));
 		this.footer.setText(this.theme.fg("muted", "Esc cancel • This analysis runs before /cook writes canonical workflow state"));
 	}
 
@@ -225,6 +225,16 @@ const liveRoleActivityByRoot = new Map<string, LiveRoleActivity>();
 const LIVE_ROLE_WAITING_MS = 15_000;
 const LIVE_ROLE_STALLED_MS = 45_000;
 const LIVE_ROLE_HEARTBEAT_MS = 5_000;
+const DRIVER_AUTO_CONTINUE_MAX_ATTEMPTS = 2;
+
+type DriverContinuationTracker = {
+	fingerprint: string;
+	attempts: number;
+	inFlight: boolean;
+	warned: boolean;
+};
+
+const driverContinuationByRoot = new Map<string, DriverContinuationTracker>();
 
 function isRecord(value: unknown): value is JsonRecord {
 	return typeof value === "object" && value !== null && !Array.isArray(value);
@@ -529,6 +539,14 @@ function completionTestDriverPromptPath(): string | undefined {
 	return asString(process.env.PI_COMPLETION_TEST_DRIVER_PROMPT_PATH);
 }
 
+function completionTestAutoContinuePromptPath(): string | undefined {
+	return asString(process.env.PI_COMPLETION_TEST_AUTO_CONTINUE_PROMPT_PATH);
+}
+
+function shouldTestAutoContinueOnSessionStart(): boolean {
+	return process.env.PI_COMPLETION_TEST_AUTO_CONTINUE_ON_SESSION_START === "1";
+}
+
 function completionTestSystemReminderPath(): string | undefined {
 	return asString(process.env.PI_COMPLETION_TEST_SYSTEM_REMINDER_PATH);
 }
@@ -1641,6 +1659,128 @@ function currentEvaluationProfile(snapshot: CompletionStateSnapshot): string | u
 	);
 }
 
+function completionContinuationFingerprint(snapshot: CompletionStateSnapshot): string | undefined {
+	if (asString(snapshot.state?.continuation_policy) !== "continue") return undefined;
+	const nextMandatoryRole = asString(snapshot.state?.next_mandatory_role);
+	if (!nextMandatoryRole) return undefined;
+	return JSON.stringify({
+		mission_anchor: asString(snapshot.state?.mission_anchor) ?? asString(snapshot.plan?.mission_anchor) ?? null,
+		task_type: currentTaskType(snapshot) ?? null,
+		evaluation_profile: currentEvaluationProfile(snapshot) ?? null,
+		current_phase: asString(snapshot.state?.current_phase) ?? null,
+		next_mandatory_role: nextMandatoryRole,
+		next_mandatory_action: asString(snapshot.state?.next_mandatory_action) ?? null,
+		active_status: asString(snapshot.active?.status) ?? null,
+		active_slice_id: asString(snapshot.active?.slice_id) ?? asString(snapshot.activeSlice?.slice_id) ?? null,
+		latest_completed_slice: asString(snapshot.state?.latest_completed_slice) ?? null,
+		latest_verified_slice: asString(snapshot.state?.latest_verified_slice) ?? null,
+	});
+}
+
+function noteQueuedDriverPrompt(rootKey: string, fingerprint: string): void {
+	const tracker = driverContinuationByRoot.get(rootKey);
+	if (tracker && tracker.fingerprint === fingerprint) {
+		tracker.attempts += 1;
+		tracker.inFlight = false;
+		tracker.warned = false;
+		return;
+	}
+	driverContinuationByRoot.set(rootKey, {
+		fingerprint,
+		attempts: 1,
+		inFlight: false,
+		warned: false,
+	});
+}
+
+function markQueuedDriverPromptInFlight(rootKey: string, fingerprint: string): void {
+	const tracker = driverContinuationByRoot.get(rootKey);
+	if (!tracker || tracker.fingerprint !== fingerprint) return;
+	tracker.inFlight = true;
+}
+
+function clearDriverContinuationTracker(rootKey: string): void {
+	driverContinuationByRoot.delete(rootKey);
+}
+
+function hasRunningCompletionRole(rootKey: string): boolean {
+	return liveRoleActivityByRoot.get(rootKey)?.status === "running";
+}
+
+function isWorkflowDriverActive(snapshot: CompletionStateSnapshot | undefined): boolean {
+	return Boolean(snapshot) && asString(snapshot?.state?.continuation_policy) === "continue";
+}
+
+function isDriverContinuationStateParked(rootKey: string, fingerprint: string): boolean {
+	const tracker = driverContinuationByRoot.get(rootKey);
+	if (!tracker || tracker.fingerprint !== fingerprint) return false;
+	return tracker.warned;
+}
+
+function rememberParkedDriverContinuation(rootKey: string, fingerprint: string): void {
+	const tracker = driverContinuationByRoot.get(rootKey);
+	if (!tracker || tracker.fingerprint !== fingerprint) return;
+	tracker.warned = true;
+	tracker.inFlight = false;
+}
+
+async function queueCompletionDriverPrompt(
+	pi: ExtensionAPI,
+	ctx: { cwd: string; hasUI: boolean; ui: any },
+	rootKey: string,
+	fingerprint: string,
+	prompt: string,
+	kind: "kickoff" | "resume" | "auto-resume",
+): Promise<boolean> {
+	const snapshotPath = kind === "auto-resume" ? completionTestAutoContinuePromptPath() : completionTestDriverPromptPath();
+	maybeWriteTestSnapshot(snapshotPath, `${prompt}\n`);
+	noteQueuedDriverPrompt(rootKey, fingerprint);
+	if (shouldSkipDriverKickoffForTests()) {
+		emitCommandText(ctx, `Skipped completion workflow ${kind} prompt (test mode)`, "info");
+		return false;
+	}
+	pi.sendUserMessage(prompt);
+	emitCommandText(ctx, `Queued completion workflow ${kind}`, "info");
+	return true;
+}
+
+async function autoContinueWorkflowIfNeeded(pi: ExtensionAPI, ctx: { cwd: string; hasUI: boolean; ui: any }): Promise<void> {
+	if (roleFromEnv()) return;
+	const snapshot = await loadCompletionSnapshot(getCtxCwd(ctx));
+	const rootKey = completionRootKey(snapshot, getCtxCwd(ctx));
+	if (!snapshot) {
+		clearDriverContinuationTracker(rootKey);
+		return;
+	}
+	const fingerprint = completionContinuationFingerprint(snapshot);
+	if (!fingerprint) {
+		clearDriverContinuationTracker(rootKey);
+		return;
+	}
+	if (!isWorkflowDriverActive(snapshot) || hasRunningCompletionRole(rootKey)) return;
+	const tracker = driverContinuationByRoot.get(rootKey);
+	if (tracker && tracker.fingerprint === fingerprint) {
+		if (tracker.inFlight) {
+			tracker.inFlight = false;
+			if (tracker.attempts >= DRIVER_AUTO_CONTINUE_MAX_ATTEMPTS) {
+				if (!isDriverContinuationStateParked(rootKey, fingerprint)) {
+					rememberParkedDriverContinuation(rootKey, fingerprint);
+					emitCommandText(
+						ctx,
+						`Completion workflow is parked before mandatory role dispatch: ${asString(snapshot.state?.next_mandatory_role) ?? "(unknown)"}. Rerun /cook to continue from canonical state.`,
+						"warning",
+					);
+				}
+				return;
+			}
+		} else {
+			return;
+		}
+	}
+	const resumePrompt = completionResumePrompt(currentTaskType(snapshot) ?? "(missing)", currentEvaluationProfile(snapshot) ?? "(missing)");
+	await queueCompletionDriverPrompt(pi, ctx, rootKey, fingerprint, resumePrompt, "auto-resume");
+}
+
 function isRubricEvaluationRole(role: string | undefined): role is RubricEvaluationRole {
 	return RUBRIC_EVALUATION_ROLES.includes(role as RubricEvaluationRole);
 }
@@ -1806,10 +1946,6 @@ function deriveMissionAnchor(rawGoal: string, projectName: string): string {
 		.replace(/\bwith docs\b/gi, "with docs parity")
 		.trim();
 
-	if (mission.length > 120) {
-		mission = `${mission.slice(0, 117).trimEnd()}...`;
-	}
-
 	if (!/[.!?。！？]$/u.test(mission)) mission += ".";
 	return mission;
 }
@@ -2798,6 +2934,53 @@ function collapseRecentActivity(items: string[], maxItems = 4): string[] {
 	return collapsed.slice(-maxItems);
 }
 
+function formatInlineRunningText(theme: any, lines: string[], options?: { primaryAssistant?: boolean }): string {
+	let text = "";
+	for (const [index, line] of lines.entries()) {
+		if (index > 0) text += "\n";
+		if (index === 0) {
+			const [prefix, ...rest] = line.split(" ");
+			text += theme.fg("warning", prefix);
+			if (rest.length > 0) text += ` ${theme.fg("accent", rest.join(" "))}`;
+			continue;
+		}
+		if (line.startsWith("tool:") || line.startsWith("progress:")) {
+			text += theme.fg("toolOutput", line);
+			continue;
+		}
+		if (line.startsWith("activity:")) {
+			text += theme.fg(line.includes("stalled") ? "warning" : "dim", line);
+			continue;
+		}
+		if (line === "recent tools:") {
+			text += theme.fg("muted", line);
+			continue;
+		}
+		if (line.startsWith("- ")) {
+			text += `${theme.fg("muted", "- ")}${theme.fg("muted", line.slice(2))}`;
+			continue;
+		}
+		if (line.startsWith("elapsed:")) {
+			text += theme.fg("dim", line);
+			continue;
+		}
+		if (line.startsWith("assistant:")) {
+			text += options?.primaryAssistant ? line : theme.fg("muted", line);
+			continue;
+		}
+		if (line.startsWith("next:") || line.startsWith("verifying:")) {
+			text += theme.fg("muted", line);
+			continue;
+		}
+		if (line.startsWith("rationale:") || line.startsWith("state-delta:")) {
+			text += theme.fg("dim", line);
+			continue;
+		}
+		text += theme.fg("muted", line);
+	}
+	return text;
+}
+
 function buildInlineRunningLines(details: {
 	role?: string;
 	startedAt?: number;
@@ -3020,6 +3203,9 @@ function completionResumePrompt(taskType: string, evaluationProfile: string): st
 export default function completionExtension(pi: ExtensionAPI) {
 	pi.on("session_start", async (_event, ctx) => {
 		await refreshStatus(ctx);
+		if (shouldTestAutoContinueOnSessionStart()) {
+			await autoContinueWorkflowIfNeeded(pi, ctx);
+		}
 	});
 
 	pi.on("turn_end", async (_event, ctx) => {
@@ -3032,10 +3218,16 @@ export default function completionExtension(pi: ExtensionAPI) {
 			await fsp.rm(snapshot.files.compactionMarkerPath, { force: true });
 		}
 		await refreshStatus(ctx);
+		await autoContinueWorkflowIfNeeded(pi, ctx);
 	});
 
 	pi.on("before_agent_start", async (_event, ctx) => {
 		const loaded = await loadCompletionDataForReminder(getCtxCwd(ctx));
+		if (loaded) {
+			const rootKey = completionRootKey(loaded.snapshot, getCtxCwd(ctx));
+			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;
@@ -3324,7 +3516,7 @@ export default function completionExtension(pi: ExtensionAPI) {
 			const task = typeof args.task === "string" ? args.task.trim() : "";
 			let text = theme.fg("toolTitle", theme.bold("completion_role ")) + theme.fg("accent", role);
 			if (task) {
-				text += `\n${theme.fg("dim", task)}`;
+				text += `\n${theme.fg("muted", task)}`;
 			}
 			return new Text(text, 0, 0);
 		},
@@ -3352,53 +3544,26 @@ export default function completionExtension(pi: ExtensionAPI) {
 			};
 			if (isPartial) {
 				const lines = buildInlineRunningLines(details);
-				let text = "";
-				for (const [index, line] of lines.entries()) {
-					if (index > 0) text += "\n";
-					if (index === 0) {
-						const [prefix, ...rest] = line.split(" ");
-						text += theme.fg("warning", prefix);
-						if (rest.length > 0) text += ` ${theme.fg("accent", rest.join(" "))}`;
-						continue;
-					}
-					if (line.startsWith("tool:") || line.startsWith("progress:")) {
-						text += theme.fg("toolOutput", line);
-						continue;
-					}
-					if (line.startsWith("activity:")) {
-						text += theme.fg(line.includes("stalled") ? "warning" : "dim", line);
-						continue;
-					}
-					if (line === "recent tools:") {
-						text += theme.fg("dim", line);
-						continue;
-					}
-					if (line.startsWith("- ")) {
-						text += `${theme.fg("muted", "- ")}${theme.fg("dim", line.slice(2))}`;
-						continue;
-					}
-					text += theme.fg("dim", line);
-				}
-				return new Text(text, 0, 0);
+				return new Text(formatInlineRunningText(theme, lines), 0, 0);
 			}
 			const role = details.role ?? "completion-role";
 			const ok = details.status === "ok" && !result.isError;
 			let text = `${theme.fg(ok ? "success" : "error", ok ? "done" : "error")} ${theme.fg("toolTitle", theme.bold(role))}`;
-			if (details.startedAt !== undefined) text += `\n${theme.fg("dim", `elapsed: ${formatElapsed(nowMs() - details.startedAt)}`)}`;
+			if (details.startedAt !== undefined) text += `\n${theme.fg("muted", `elapsed: ${formatElapsed(nowMs() - details.startedAt)}`)}`;
 			if (details.toolActivity) text += `\n${theme.fg("toolOutput", `tool: ${details.toolActivity}`)}`;
 			if (details.progress) text += `\n${theme.fg("toolOutput", `progress: ${details.progress}`)}`;
-			else if (details.assistantSummary) text += `\n${theme.fg("dim", `assistant: ${details.assistantSummary}`)}`;
-			if (details.rationale) text += `\n${theme.fg("dim", `rationale: ${details.rationale}`)}`;
-			if (details.nextStep) text += `\n${theme.fg("dim", `next: ${details.nextStep}`)}`;
-			if (details.verifying) text += `\n${theme.fg("dim", `verifying: ${details.verifying}`)}`;
+			else if (details.assistantSummary) text += `\nassistant: ${details.assistantSummary}`;
+			if (details.rationale) text += `\n${theme.fg("muted", `rationale: ${details.rationale}`)}`;
+			if (details.nextStep) text += `\n${theme.fg("muted", `next: ${details.nextStep}`)}`;
+			if (details.verifying) text += `\n${theme.fg("muted", `verifying: ${details.verifying}`)}`;
 			if (details.stateDeltas?.length) {
-				for (const delta of details.stateDeltas.slice(-4)) text += `\n${theme.fg("dim", `state-delta: ${delta}`)}`;
+				for (const delta of details.stateDeltas.slice(-4)) text += `\n${theme.fg("muted", `state-delta: ${delta}`)}`;
 			}
 			if (details.transcription?.appended?.length) {
 				text += `\n${theme.fg("success", `transcribed: ${details.transcription.appended.join(", ")}`)}`;
 			}
 			if (details.transcription?.skipped?.length && expanded) {
-				text += `\n${theme.fg("dim", `skipped: ${details.transcription.skipped.join(" | ")}`)}`;
+				text += `\n${theme.fg("muted", `skipped: ${details.transcription.skipped.join(" | ")}`)}`;
 			}
 			if (details.transcription?.errors?.length) {
 				text += `\n${theme.fg("warning", `warnings: ${details.transcription.errors.join(" | ")}`)}`;
@@ -3416,14 +3581,14 @@ export default function completionExtension(pi: ExtensionAPI) {
 			for (const key of summaryKeys) {
 				const value = reportFields[key];
 				if (!value) continue;
-				text += `\n${theme.fg("dim", `${key}: `)}${value}`;
+				text += `\n${theme.fg("muted", `${key}: `)}${value}`;
 			}
 			const body = result.content.find((item) => item.type === "text");
 			if (expanded && body?.type === "text") {
 				text += `\n\n${body.text}`;
 			} else if (!expanded && body?.type === "text") {
 				const preview = body.text.split("\n").slice(0, 4).join("\n");
-				text += `\n${theme.fg("dim", preview)}`;
+				text += `\n${theme.fg("muted", preview)}`;
 			}
 			if (details.stderr && expanded) text += `\n${theme.fg("error", details.stderr)}`;
 			return new Text(text, 0, 0);
@@ -3538,13 +3703,14 @@ export default function completionExtension(pi: ExtensionAPI) {
 						currentTaskType(snapshot) ?? "(missing)",
 						currentEvaluationProfile(snapshot) ?? "(missing)",
 					);
-					maybeWriteTestSnapshot(completionTestDriverPromptPath(), `${resumePrompt}\n`);
-					if (shouldSkipDriverKickoffForTests()) {
-						emitCommandText(ctx, "Skipped completion workflow resume kickoff (test mode)", "info");
-						return;
-					}
-					pi.sendUserMessage(resumePrompt);
-					emitCommandText(ctx, "Queued completion workflow resume", "info");
+					const rootKey = completionRootKey(snapshot, getCtxCwd(ctx));
+					const fingerprint = completionContinuationFingerprint(snapshot) ?? JSON.stringify({
+						kind: "resume",
+						mission_anchor: currentMissionAnchor(snapshot),
+						current_phase: asString(snapshot.state?.current_phase) ?? null,
+						next_mandatory_role: asString(snapshot.state?.next_mandatory_role) ?? null,
+					});
+					await queueCompletionDriverPrompt(pi, ctx, rootKey, fingerprint, resumePrompt, "resume");
 					return;
 				}
 			}
@@ -3613,13 +3779,16 @@ export default function completionExtension(pi: ExtensionAPI) {
 				kickoffIntent,
 				kickoffMissionAnchor,
 			);
-			maybeWriteTestSnapshot(completionTestDriverPromptPath(), `${kickoffPrompt}\n`);
-			if (shouldSkipDriverKickoffForTests()) {
-				emitCommandText(ctx, "Skipped completion workflow kickoff (test mode)", "info");
-				return;
-			}
-			pi.sendUserMessage(kickoffPrompt);
-			emitCommandText(ctx, "Queued completion workflow kickoff", "info");
+			const rootKey = completionRootKey(snapshot, getCtxCwd(ctx));
+			const fingerprint = completionContinuationFingerprint(snapshot) ?? JSON.stringify({
+				kind: "kickoff",
+				mission_anchor: kickoffMissionAnchor,
+				goal,
+				intent: kickoffIntent,
+				task_type: currentTaskType(snapshot) ?? "(missing)",
+				evaluation_profile: currentEvaluationProfile(snapshot) ?? "(missing)",
+			});
+			await queueCompletionDriverPrompt(pi, ctx, rootKey, fingerprint, kickoffPrompt, "kickoff");
 		},
 	});
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@linimin/pi-letscook",
-  "version": "0.1.31",
+  "version": "0.1.33",
   "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/smoke-test.sh

@@ -8,6 +8,7 @@ trap 'rm -rf "$TMPDIR"' EXIT
 ROOT="$TMPDIR/repo"
 KICKOFF_PROMPT="$TMPDIR/kickoff-prompt.txt"
 RESUME_PROMPT="$TMPDIR/resume-prompt.txt"
+AUTO_RESUME_PROMPT="$TMPDIR/auto-resume-prompt.txt"
 
 mkdir -p "$ROOT"
 cd "$ROOT"
@@ -72,6 +73,26 @@ assert f'- task_type: {expected_task_type}' in resume, 'resume prompt missing ca
 assert f'- evaluation_profile: {expected_eval_profile}' in resume, 'resume prompt missing canonical evaluation_profile'
 PY
 
+PI_COMPLETION_SKIP_DRIVER_KICKOFF=1 \
+PI_COMPLETION_TEST_AUTO_CONTINUE_ON_SESSION_START=1 \
+PI_COMPLETION_TEST_AUTO_CONTINUE_PROMPT_PATH="$AUTO_RESUME_PROMPT" \
+pi -e "$PKG_ROOT" -p "/cook" \
+  >"$TMPDIR/pi-completion-smoke-auto-resume.out" 2>"$TMPDIR/pi-completion-smoke-auto-resume.err"
+
+python3 - "$AUTO_RESUME_PROMPT" <<'PY'
+import sys
+from pathlib import Path
+
+expected_task_type = 'completion-workflow'
+expected_eval_profile = 'completion-rubric-v1'
+auto_resume = Path(sys.argv[1]).read_text()
+
+assert 'Resume the completion workflow from canonical state.' in auto_resume, 'auto-resume prompt should use the canonical resume workflow prompt'
+assert 'Canonical routing profile:' in auto_resume, 'auto-resume prompt should expose canonical routing profile'
+assert f'- task_type: {expected_task_type}' in auto_resume, 'auto-resume prompt missing canonical task_type'
+assert f'- evaluation_profile: {expected_eval_profile}' in auto_resume, 'auto-resume prompt missing canonical evaluation_profile'
+PY
+
 python3 - <<'PY'
 import json
 from pathlib import Path