@linimin/pi-letscook 0.1.68 → 0.1.69

package/extensions/completion/prompt-surfaces.ts

@@ -24,26 +24,6 @@ export type AdvisoryStartupBrief = {
 	evaluation_profile?: string;
 };
 
-export type ApprovedStartupPlan = {
-	artifact_type: "completion-startup-plan";
-	schema_version: 1;
-	status: "approved";
-	source: AdvisoryStartupBrief["source"];
-	captured_at: string;
-	mission_anchor: string;
-	goal_text: string;
-	task_type?: string;
-	evaluation_profile?: string;
-	scope: string[];
-	constraints: string[];
-	acceptance: string[];
-	risks: string[];
-	notes: string[];
-	planned_surfaces: string[];
-	verification_intent: string[];
-	sequencing_hints: string[];
-};
-
 export function buildCookHandoffBoundaryReminder(): string {
 	return [
 		"You are in ordinary main chat unless the user explicitly runs /cook.",
@@ -53,11 +33,10 @@ export function buildCookHandoffBoundaryReminder(): string {
 		"In ordinary chat, do not load or follow completion-protocol, and do not call completion_role.",
 		"If the user wants direct implementation now, stay in ordinary chat and help directly instead of blocking on /cook.",
 		"If the user asks follow-up questions or wants to keep refining scope, continue helping naturally in ordinary chat.",
-		"If the user explicitly runs /cook, the extension should call a primary-agent startup-plan synthesis step from the current task context, show Start/Cancel confirmation in the same /cook entry, and only write the approved plan into .agent after Start.",
-		"Do not expect /cook to infer or guess startup intent from recent discussion alone, and do not expect /cook to directly reuse an old preview capsule; /cook should always synthesize the startup plan fresh in the same entry from current task context.",
-		"Only provide a preview startup plan or ```cook_handoff``` capsule in ordinary chat when the user explicitly asks for that preview behavior.",
-		"Any preview capsule is advisory only until /cook reruns same-entry primary-agent startup-plan synthesis: do not present it as canonical .agent state, an active slice, or a persistent repo contract.",
-		"When /cook starts, the approved startup plan should be written into .agent and then handed to completion-regrounder so canonical slices can be derived from repo truth.",
+		"If the user explicitly runs /cook, the extension should call a primary-agent handoff synthesis step from the current task context, show Start/Cancel confirmation, and persist the confirmed startup brief into .agent/** without making the user rerun /cook.",
+		"Do not expect /cook to infer or guess startup intent from recent discussion alone; /cook should use explicit primary-agent handoff data, whether it already exists or is synthesized in the same /cook entry.",
+		"Only provide a preview startup brief or ```cook_handoff``` capsule in ordinary chat when the user explicitly asks for that preview behavior.",
+		"Any preview capsule is startup intake for /cook only: do not present it as canonical .agent state, an active slice, or a persistent repo contract.",
 		"When you continue in ordinary chat, do not pretend /cook already started and do not silently rewrite discussion into canonical workflow state.",
 	].join(" ");
 }
@@ -106,28 +85,7 @@ function buildAdvisoryStartupBriefNotes(analysis: ContextProposalAnalysis): stri
 		...analysis.critique,
 		...analysis.possibleNoise.map((item) => `Possible noise: ${item}`),
 	];
-	return notes.length > 0 ? notes : ["No additional operator notes were captured for the approved startup plan."];
-}
-
-function startupPlanSourceForProposal(source: ContextProposal["source"]): AdvisoryStartupBrief["source"] {
-	return source === "handoff_capsule"
-		? "primary_agent_handoff"
-		: source === "deferred_primary_agent_handoff"
-			? "deferred_primary_agent_handoff"
-			: "recent_discussion";
-}
-
-function extractDelimitedNoteValues(notes: string[], prefix: string): string[] {
-	return notes.flatMap((note) => {
-		if (!note.startsWith(prefix)) return [];
-		return note.slice(prefix.length).split("|").map((item) => item.trim()).filter(Boolean);
-	});
-}
-
-function extractSequencingHints(notes: string[]): string[] {
-	return notes.filter((note) =>
-		note.startsWith("First slice goal:") || note.startsWith("First slice non-goals:") || note.startsWith("Why this slice first:"),
-	);
+	return notes.length > 0 ? notes : ["No additional operator notes were derived from recent discussion."];
 }
 
 export function buildAdvisoryStartupBrief(args: {
@@ -137,7 +95,12 @@ export function buildAdvisoryStartupBrief(args: {
 }): AdvisoryStartupBrief {
 	return {
 		kind: "startup_brief",
-		source: startupPlanSourceForProposal(args.proposal.source),
+		source:
+			args.proposal.source === "handoff_capsule"
+				? "primary_agent_handoff"
+				: args.proposal.source === "deferred_primary_agent_handoff"
+					? "deferred_primary_agent_handoff"
+					: "recent_discussion",
 		confirmed: true,
 		captured_at: args.capturedAt ?? new Date().toISOString(),
 		goal_text: args.proposal.goalText,
@@ -152,81 +115,6 @@ export function buildAdvisoryStartupBrief(args: {
 	};
 }
 
-export function buildApprovedStartupPlan(args: {
-	proposal: Pick<ContextProposal, "goalText" | "mission" | "scope" | "constraints" | "acceptance" | "source">;
-	analysis: ContextProposalAnalysis;
-	missionAnchor?: string;
-	capturedAt?: string;
-}): ApprovedStartupPlan {
-	const capturedAt = args.capturedAt ?? new Date().toISOString();
-	const notes = buildAdvisoryStartupBriefNotes(args.analysis);
-	return {
-		artifact_type: "completion-startup-plan",
-		schema_version: 1,
-		status: "approved",
-		source: startupPlanSourceForProposal(args.proposal.source),
-		captured_at: capturedAt,
-		mission_anchor: args.missionAnchor ?? args.proposal.mission,
-		goal_text: args.proposal.goalText,
-		task_type: args.analysis.taskType,
-		evaluation_profile: args.analysis.evaluationProfile,
-		scope: [...args.proposal.scope],
-		constraints: [...args.proposal.constraints],
-		acceptance: [...args.proposal.acceptance],
-		risks: [...args.analysis.risks],
-		notes: [...notes],
-		planned_surfaces: extractDelimitedNoteValues(notes, "Implementation surfaces:"),
-		verification_intent: extractDelimitedNoteValues(notes, "Verification commands:"),
-		sequencing_hints: extractSequencingHints(notes),
-	};
-}
-
-export function buildApprovedStartupPlanMarkdown(plan: ApprovedStartupPlan): string {
-	const lines = [
-		"# Approved Startup Plan",
-		"",
-		`Mission anchor: ${plan.mission_anchor}`,
-		`Source: ${plan.source}`,
-		`Captured at: ${plan.captured_at}`,
-	];
-	if (plan.task_type) lines.push(`Task type: ${plan.task_type}`);
-	if (plan.evaluation_profile) lines.push(`Evaluation profile: ${plan.evaluation_profile}`);
-	lines.push("", "## Goal", "", plan.goal_text);
-	if (plan.scope.length > 0) {
-		lines.push("", "## Scope", "");
-		for (const item of plan.scope) lines.push(`- ${item}`);
-	}
-	if (plan.constraints.length > 0) {
-		lines.push("", "## Constraints", "");
-		for (const item of plan.constraints) lines.push(`- ${item}`);
-	}
-	if (plan.acceptance.length > 0) {
-		lines.push("", "## Acceptance", "");
-		for (const item of plan.acceptance) lines.push(`- ${item}`);
-	}
-	if (plan.risks.length > 0) {
-		lines.push("", "## Risks", "");
-		for (const item of plan.risks) lines.push(`- ${item}`);
-	}
-	if (plan.planned_surfaces.length > 0) {
-		lines.push("", "## Planned surfaces", "");
-		for (const item of plan.planned_surfaces) lines.push(`- ${item}`);
-	}
-	if (plan.verification_intent.length > 0) {
-		lines.push("", "## Verification intent", "");
-		for (const item of plan.verification_intent) lines.push(`- ${item}`);
-	}
-	if (plan.sequencing_hints.length > 0) {
-		lines.push("", "## Sequencing hints", "");
-		for (const item of plan.sequencing_hints) lines.push(`- ${item}`);
-	}
-	if (plan.notes.length > 0) {
-		lines.push("", "## Notes", "");
-		for (const item of plan.notes) lines.push(`- ${item}`);
-	}
-	return `${lines.join("\n")}\n`;
-}
-
 export function buildContextProposalCritiqueText(analysis: ContextProposalAnalysis): string {
 	const lines: string[] = [];
 	if (analysis.critique.length > 0) {
@@ -259,7 +147,7 @@ export function buildContextProposalCritiqueText(analysis: ContextProposalAnalys
 		for (const item of analysis.suppressedNegatedTopics) lines.push(`- ${item}`);
 	}
 	if (lines.length === 0) {
-		return "No additional operator notes or risks were derived for this startup plan.";
+		return "No additional operator notes or risks were derived for this startup brief.";
 	}
 	return lines.join("\n");
 }
@@ -308,7 +196,7 @@ export function buildContextProposalConfirmationActions(mainChatRerunGuidance: s
 		{
 			id: "start",
 			label: "Start",
-			description: "Accept this startup plan and let /cook write or refocus canonical workflow state.",
+			description: "Accept this startup brief and let /cook write or refocus canonical workflow state.",
 		},
 		{
 			id: "cancel",
@@ -328,8 +216,8 @@ export function buildContextProposalConfirmationLayout(args: {
 }): ContextProposalConfirmationLayout {
 	return {
 		title: args.title,
-		intro: "Review the startup plan (mission, scope, constraints, acceptance, and notes/risks) plus the routing details before /cook writes canonical workflow state. This gate is approval-only: either Start it as-is or Cancel, discuss changes in the main chat, and rerun /cook.",
-		proposalHeading: "Startup plan",
+		intro: "Review the startup brief (mission, scope, constraints, acceptance, and notes/risks) plus the routing details before /cook writes canonical workflow state. This gate is approval-only: either Start it as-is or Cancel, discuss changes in the main chat, and rerun /cook.",
+		proposalHeading: "Startup brief",
 		proposalBody: buildContextProposalDisplayText(args.proposal),
 		critiqueHeading: "Notes and risks",
 		critiqueBody: buildContextProposalCritiqueText(args.analysis),
@@ -529,15 +417,6 @@ type CompletionVerificationEvidenceSummary = {
 	summary: string;
 };
 
-type CompletionStartupPlanSummary = {
-	path: string;
-	status: string;
-	source?: string;
-	plannedSurfaces: string[];
-	verificationIntent: string[];
-	summary: string;
-};
-
 export function buildSystemReminder(args: {
 	missionAnchor?: string;
 	taskType?: string;
@@ -561,12 +440,11 @@ export function buildSystemReminder(args: {
 	implementationSurfacesLine?: string;
 	verificationCommandsLine?: string;
 	evidence: CompletionVerificationEvidenceSummary;
-	startupPlan: CompletionStartupPlanSummary;
 	evaluationRoleReminderText?: string;
 }): string {
 	const lines = [
 		"Completion workflow detected.",
-		"Canonical truth lives in .agent/state.json, .agent/startup-plan.json, .agent/plan.json, .agent/active-slice.json, .agent/slice-history.jsonl, .agent/stop-check-history.jsonl, and .agent/verification-evidence.json.",
+		"Canonical truth lives in .agent/state.json, .agent/plan.json, .agent/active-slice.json, .agent/slice-history.jsonl, .agent/stop-check-history.jsonl, and .agent/verification-evidence.json.",
 		`Mission anchor: ${args.missionAnchor ?? "(unknown)"}`,
 		`Task type: ${args.taskType ?? "(missing)"}`,
 		`Evaluation profile: ${args.evaluationProfile ?? "(missing)"}`,
@@ -583,7 +461,6 @@ export function buildSystemReminder(args: {
 		"Only stop for the user when continuation_policy is await_user_input, blocked, paused, or done.",
 		"If canonical state is stale, invalid, ambiguous, or missing, route to completion-regrounder.",
 		"When recovering from compaction, prefer a deterministic restart from canonical files over conversational inference.",
-		`Approved startup plan: ${args.startupPlan.path} (${args.startupPlan.status})`,
 	];
 	if (args.exactActiveContract) {
 		lines.push("Selected/in-progress/committed/done .agent/active-slice.json is the canonical implementation contract.");
@@ -597,14 +474,6 @@ export function buildSystemReminder(args: {
 	else if (args.implementationSurfaces.length > 0) lines.push(`Active implementation surfaces: ${args.implementationSurfaces.join(", ")}`);
 	if (args.verificationCommandsLine) lines.push(args.verificationCommandsLine);
 	else if (args.verificationCommands.length > 0) lines.push(`Active verification commands: ${args.verificationCommands.join(" | ")}`);
-	if (args.startupPlan.source) lines.push(`Approved startup plan source: ${args.startupPlan.source}`);
-	if (args.startupPlan.plannedSurfaces.length > 0) {
-		lines.push(`Approved startup plan surfaces: ${args.startupPlan.plannedSurfaces.join(" | ")}`);
-	}
-	if (args.startupPlan.verificationIntent.length > 0) {
-		lines.push(`Approved startup plan verification intent: ${args.startupPlan.verificationIntent.join(" | ")}`);
-	}
-	lines.push(`Approved startup plan summary: ${args.startupPlan.summary}`);
 	lines.push(`Verification evidence artifact: ${args.evidence.path} (${args.evidence.status})`);
 	if (args.evidence.subjectType) lines.push(`Verification evidence subject: ${args.evidence.subjectType}`);
 	if (args.evidence.outcome) lines.push(`Verification evidence outcome: ${args.evidence.outcome}`);
@@ -633,7 +502,6 @@ export function buildResumeCapsule(args: {
 	activeSliceMatchesPlan: "yes" | "no" | "unknown";
 	activeSliceContractDrift: string;
 	implementerHandoffSnapshot: "present" | "missing_or_unclear";
-	startupPlan: CompletionStartupPlanSummary;
 	evidence: CompletionVerificationEvidenceSummary;
 	activeSlice: {
 		sliceId?: string;
@@ -676,14 +544,6 @@ export function buildResumeCapsule(args: {
 		`implementer_handoff_snapshot: ${args.implementerHandoffSnapshot}`,
 		`history_counts: reviewed=${args.history.reviewed}, audited=${args.history.audited}, accepted=${args.history.accepted}, reopened=${args.history.reopened}, judgments=${args.history.judgments}`,
 		"",
-		"startup_plan:",
-		`- path: ${args.startupPlan.path}`,
-		`- status: ${args.startupPlan.status}`,
-		`- source: ${args.startupPlan.source ?? "(missing)"}`,
-		`- planned_surfaces: ${args.startupPlan.plannedSurfaces.length > 0 ? args.startupPlan.plannedSurfaces.join(" | ") : "(none)"}`,
-		`- verification_intent: ${args.startupPlan.verificationIntent.length > 0 ? args.startupPlan.verificationIntent.join(" | ") : "(none)"}`,
-		`- summary: ${args.startupPlan.summary}`,
-		"",
 		"verification_evidence:",
 		`- path: ${args.evidence.path}`,
 		`- status: ${args.evidence.status}`,
@@ -731,9 +591,8 @@ export function buildResumeCapsule(args: {
 		"- Treat this block as continuity support derived from canonical .agent state.",
 		"- For selected/in-progress/committed/done slices, .agent/active-slice.json is the canonical implementation contract and the selected plan slice must mirror it exactly.",
 		"- Preserve exact slice_id, goal, contract_ids, acceptance criteria, blocked_on, priority, why_now, implementation surfaces, verification commands, locked notes, must-fix findings, basis_commit, and before-slice counters where still true.",
-		"- .agent/startup-plan.json is the approved workflow plan captured at /cook entry. completion-regrounder must treat it as planning input, then reconcile canonical slices against repo truth instead of copying it blindly into plan.json.",
 		"- When populated, .agent/verification-evidence.json is the durable canonical verification record for the selected slice or current HEAD and should be consumed instead of temp-only artifacts or conversational summaries.",
-		"- After compaction, re-read .agent/state.json, .agent/startup-plan.json, .agent/plan.json, .agent/active-slice.json, .agent/slice-history.jsonl, .agent/stop-check-history.jsonl, and .agent/verification-evidence.json before resuming long-running completion work.",
+		"- After compaction, re-read .agent/state.json, .agent/plan.json, .agent/active-slice.json, .agent/slice-history.jsonl, .agent/stop-check-history.jsonl, and .agent/verification-evidence.json before resuming long-running completion work.",
 		"- Invoke completion-regrounder before continuing when requires_reground is true or unknown.",
 		"- Invoke completion-regrounder before continuing when next_mandatory_role or next_mandatory_action is unknown or ambiguous.",
 		"- Invoke completion-regrounder before continuing when active_slice_matches_plan is no, active_slice_contract_drift_fields is not none, or implementer_handoff_snapshot is missing_or_unclear.",

package/extensions/completion/proposal.ts

@@ -60,11 +60,11 @@ export type CookHandoffCapsule = {
 	risks: string[];
 	notes: string[];
 	handoff_kind: "implementation_workflow_handoff";
-	first_slice_goal?: string;
+	first_slice_goal: string;
 	first_slice_non_goals: string[];
 	implementation_surfaces: string[];
 	verification_commands: string[];
-	why_this_slice_first?: string;
+	why_this_slice_first: string;
 	task_type?: string;
 	evaluation_profile?: string;
 	why_cook_now?: string;
@@ -1279,7 +1279,7 @@ function parseCookHandoffCapsulesFromText(
 		const mission = deps.asString(parsed.mission);
 		const firstSliceGoal = deps.asString(parsed.first_slice_goal ?? parsed.firstSliceGoal);
 		const whyThisSliceFirst = deps.asString(parsed.why_this_slice_first ?? parsed.whyThisSliceFirst);
-		if (!mission) continue;
+		if (!mission || !firstSliceGoal || !whyThisSliceFirst) continue;
 		const scope = deps.asStringArray(parsed.scope);
 		const constraints = deps.asStringArray(parsed.constraints);
 		const nonGoals = deps.asStringArray(parsed.non_goals ?? parsed.nonGoals);
@@ -1325,12 +1325,12 @@ function buildCookHandoffBasisPreview(capsule: CookHandoffCapsule): string {
 		...capsule.constraints,
 		...capsule.non_goals,
 		...capsule.acceptance,
+		`first_slice_goal: ${capsule.first_slice_goal}`,
+		...capsule.first_slice_non_goals.map((item) => `first_slice_non_goals: ${item}`),
+		...capsule.implementation_surfaces.map((item) => `implementation_surfaces: ${item}`),
+		...capsule.verification_commands.map((item) => `verification_commands: ${item}`),
+		`why_this_slice_first: ${capsule.why_this_slice_first}`,
 	];
-	if (capsule.first_slice_goal) parts.push(`first_slice_goal: ${capsule.first_slice_goal}`);
-	parts.push(...capsule.first_slice_non_goals.map((item) => `first_slice_non_goals: ${item}`));
-	parts.push(...capsule.implementation_surfaces.map((item) => `implementation_surfaces: ${item}`));
-	parts.push(...capsule.verification_commands.map((item) => `verification_commands: ${item}`));
-	if (capsule.why_this_slice_first) parts.push(`why_this_slice_first: ${capsule.why_this_slice_first}`);
 	if (capsule.why_cook_now) parts.push(`why_cook_now: ${capsule.why_cook_now}`);
 	return parts.join("\n").trim();
 }
@@ -1363,32 +1363,23 @@ function cookHandoffStartabilityFailures(
 	else if (!cookHandoffAcceptanceIsRepoChangeOriented(capsule)) {
 		failures.push("acceptance is not anchored to concrete repo changes or verification");
 	}
-	if (capsule.first_slice_goal) {
-		const firstSliceGoal = deps.normalizeMissionAnchorText(capsule.first_slice_goal);
-		if (!firstSliceGoal || deps.isWeakMissionAnchor(firstSliceGoal) || COOK_HANDOFF_NEGATIVE_MISSION_REGEX.test(firstSliceGoal)) {
-			failures.push("first_slice_goal is not a useful sequencing hint");
-		} else if (hasExplicitPlanningOnlyDeliverable([capsule.first_slice_goal]) || hasClearNoImplementationSignal([capsule.first_slice_goal])) {
-			failures.push("first_slice_goal is planning-only instead of a repo-change sequencing hint");
-		}
+	const firstSliceGoal = deps.normalizeMissionAnchorText(capsule.first_slice_goal);
+	if (!firstSliceGoal || deps.isWeakMissionAnchor(firstSliceGoal) || COOK_HANDOFF_NEGATIVE_MISSION_REGEX.test(firstSliceGoal)) {
+		failures.push("first_slice_goal is not a bounded implementation slice");
+	} else if (hasExplicitPlanningOnlyDeliverable([capsule.first_slice_goal]) || hasClearNoImplementationSignal([capsule.first_slice_goal])) {
+		failures.push("first_slice_goal is planning-only instead of a repo-change slice");
 	}
+	if (capsule.implementation_surfaces.length === 0) failures.push("implementation_surfaces is empty");
+	if (capsule.verification_commands.length === 0) failures.push("verification_commands is empty");
 	return failures;
 }
 
-function buildNonStartableCookHandoffMessage(
-	failures: string[],
-	context: "explicit_preview" | "same_entry_synthesis" = "explicit_preview",
-): string {
-	return context === "same_entry_synthesis"
-		? [
-			"/cook failed closed because the same-entry primary-agent startup-plan synthesis step returned a startup plan that is still not concrete enough to seed workflow planning yet.",
-			"Clarify the mission, scope, acceptance, or verification intent in the main chat, then rerun /cook so the primary agent can synthesize a tighter startup plan.",
-			`Blocking details: ${failures.join("; ")}.`,
-		].join(" ")
-		: [
-			"/cook failed closed because a fresh explicit primary-agent startup plan exists, but it is not concrete enough to seed workflow planning yet.",
-			"Tighten the startup plan in the main chat so it captures a concrete mission, repo-change-oriented acceptance, and truthful verification intent, then rerun /cook.",
-			`Blocking details: ${failures.join("; ")}.`,
-		].join(" ");
+function buildNonStartableCookHandoffMessage(failures: string[]): string {
+	return [
+		"/cook failed closed because a fresh explicit primary-agent handoff exists, but it is not concrete enough to start implementation workflow yet.",
+		"Tighten the handoff in the main chat so it names a bounded first implementation slice, repo-change-oriented acceptance, implementation_surfaces, and verification_commands, then rerun /cook.",
+		`Blocking details: ${failures.join("; ")}.`,
+	].join(" ");
 }
 
 function isStartableCookHandoffCapsule(
@@ -1444,11 +1435,11 @@ function buildContextProposalFromCookHandoffCapsule(
 				evaluationProfile: capsule.evaluation_profile,
 				critique: [
 					...capsule.notes,
-					...(capsule.first_slice_goal ? [`First slice goal: ${capsule.first_slice_goal}`] : []),
+					`First slice goal: ${capsule.first_slice_goal}`,
 					...(capsule.first_slice_non_goals.length > 0 ? [`First slice non-goals: ${capsule.first_slice_non_goals.join(" | ")}`] : []),
 					...(capsule.implementation_surfaces.length > 0 ? [`Implementation surfaces: ${capsule.implementation_surfaces.join(" | ")}`] : []),
 					...(capsule.verification_commands.length > 0 ? [`Verification commands: ${capsule.verification_commands.join(" | ")}`] : []),
-					...(capsule.why_this_slice_first ? [`Why this slice first: ${capsule.why_this_slice_first}`] : []),
+					`Why this slice first: ${capsule.why_this_slice_first}`,
 					...(capsule.why_cook_now ? [`Primary-agent /cook handoff rationale: ${capsule.why_cook_now}`] : []),
 				],
 				risks: capsule.risks,
@@ -1464,11 +1455,11 @@ function buildContextProposalFromCookHandoffCapsule(
 				...capsule.scope,
 				...constraints,
 				...capsule.acceptance,
-				...(capsule.first_slice_goal ? [capsule.first_slice_goal] : []),
+				capsule.first_slice_goal,
 				...capsule.first_slice_non_goals,
 				...capsule.implementation_surfaces,
 				...capsule.verification_commands,
-				...(capsule.why_this_slice_first ? [capsule.why_this_slice_first] : []),
+				capsule.why_this_slice_first,
 			],
 		),
 		goalText,
@@ -1479,32 +1470,6 @@ function buildContextProposalFromCookHandoffCapsule(
 	return finalizeContextProposal(proposal, projectName, deps);
 }
 
-export function assessCookHandoffText(
-	text: string,
-	projectName: string,
-	deps: ProposalParseDeps,
-	options?: {
-		messageId?: string;
-		timestampMs?: number;
-		context?: "explicit_preview" | "same_entry_synthesis";
-	},
-): CookHandoffProposalAssessment {
-	const capsules = parseCookHandoffCapsulesFromText(text, options?.messageId, options?.timestampMs, deps);
-	for (let capsuleIndex = capsules.length - 1; capsuleIndex >= 0; capsuleIndex -= 1) {
-		const capsule = capsules[capsuleIndex];
-		const failures = cookHandoffStartabilityFailures(capsule, deps);
-		if (failures.length > 0) {
-			return {
-				status: "fresh_but_not_startable",
-				message: buildNonStartableCookHandoffMessage(failures, options?.context ?? "explicit_preview"),
-			};
-		}
-		const proposal = buildContextProposalFromCookHandoffCapsule(capsule, projectName, deps);
-		if (proposal) return { status: "startable", proposal };
-	}
-	return { status: "none" };
-}
-
 export function assessLatestCookHandoffProposal(
 	recentMessages: RecentSessionMessage[],
 	projectName: string,
@@ -1524,7 +1489,7 @@ export function assessLatestCookHandoffProposal(
 			if (failures.length > 0) {
 				return {
 					status: "fresh_but_not_startable",
-					message: buildNonStartableCookHandoffMessage(failures, "explicit_preview"),
+					message: buildNonStartableCookHandoffMessage(failures),
 				};
 			}
 			const proposal = buildContextProposalFromCookHandoffCapsule(capsule, projectName, deps);

package/extensions/completion/role-runner.ts

@@ -97,18 +97,14 @@ const STARTUP_ANALYST_ROLE = "cook-proposal-analyst";
 const ANALYST_HEARTBEAT_MS = 5_000;
 
 const PRIMARY_AGENT_HANDOFF_SYSTEM_PROMPT = [
-	"You are the primary agent preparing an explicit /cook startup plan after the user already chose workflow mode.",
-	"Return either exactly one fenced ```cook_handoff JSON block or one brief plain sentence explaining why no concrete startup plan can be prepared.",
-	"If you can prepare a plan, the JSON must use kind cook_handoff, source primary_agent, and handoff_kind implementation_workflow_handoff.",
-	"Author the approved workflow startup plan now from the primary-agent view of the task so /cook can persist it under .agent before completion-regrounder derives canonical slices.",
-	"Capture the agreed mission, scope, constraints or non_goals, acceptance, risks, notes, and any concrete planning hints that will help completion-regrounder split slices later.",
-	"If a bounded first slice, likely implementation surfaces, or likely verification commands are already obvious, include first_slice_goal, first_slice_non_goals, implementation_surfaces, verification_commands, and why_this_slice_first as optional hints only. They are not required when the overall startup plan is already concrete enough to begin workflow planning.",
-	"Prefer the latest user-authored task context plus canonical workflow context over older assistant-authored previews or stale planning text.",
-	"Do not directly reuse an old preview capsule as-is; either synthesize a fresh startup plan from the current task context or return a brief plain sentence saying no concrete startup plan should replace canonical state yet.",
-	"If canonical workflow context already exists and the latest discussion does not clearly ask to replace the mission or start the next round, return a brief plain sentence instead of inventing a replacement startup plan.",
-	"Do not make /cook infer or rediscover the mission later; author the startup plan now from the primary-agent view of the task.",
+	"You are the primary agent preparing an explicit /cook handoff after the user already chose workflow mode.",
+	"Return either exactly one fenced ```cook_handoff JSON block or one brief plain sentence explaining why no concrete handoff can be prepared.",
+	"If you can prepare a handoff, the JSON must use kind cook_handoff, source primary_agent, and handoff_kind implementation_workflow_handoff.",
+	"When the user has clearly accepted a concrete assistant-proposed slice, carry that slice forward into the handoff instead of broadening or re-guessing the mission.",
+	"Do not make /cook infer or rediscover the mission from recent discussion later; author the handoff now from the primary-agent view of the task.",
 	"Do not emit markdown commentary before or after the capsule.",
-	"If the task is not concrete enough for workflow startup, do not invent missing detail.",
+	"If the task is not concrete enough for implementation workflow, do not invent the slice.",
+	"A valid implementation-ready handoff must include mission, scope, constraints or non_goals, acceptance, risks, notes, first_slice_goal, first_slice_non_goals, implementation_surfaces, verification_commands, and why_this_slice_first.",
 ].join(" ");
 const PRIMARY_AGENT_HANDOFF_ROLE = "cook-primary-agent-handoff";
 
@@ -346,8 +342,7 @@ function buildPrimaryAgentHandoffPrompt(projectName: string, recentEntries: Rece
 	lines.push(
 		"",
 		"Task:",
-		"The user explicitly invoked /cook. Prepare the primary-agent startup plan that /cook should synthesize immediately for Start/Cancel confirmation, persistence under .agent, and later slice derivation by completion-regrounder.",
-		"If the latest discussion does not justify a concrete new startup plan, return a brief plain sentence instead of speculative JSON.",
+		"The user explicitly invoked /cook. Prepare the primary-agent handoff that /cook should consume immediately for Start/Cancel confirmation.",
 	);
 	return lines.join("\n");
 }
@@ -364,7 +359,7 @@ async function runPrimaryAgentHandoffSubprocess(params: GenerateCookHandoffWithA
 	const invocation = getPiInvocation(args);
 	const liveActivity = createLiveRoleActivity(PRIMARY_AGENT_HANDOFF_ROLE);
 	liveActivity.progress = "Preparing primary-agent /cook handoff";
-	liveActivity.currentAction = "Authoring explicit startup plan from current task context";
+	liveActivity.currentAction = "Authoring explicit startup handoff from current task context";
 	liveActivity.assistantSummary = liveActivity.progress;
 	try {
 		const output = await new Promise<string | undefined>((resolve) => {
@@ -414,7 +409,7 @@ export async function generateCookHandoffWithAgent(params: GenerateCookHandoffWi
 	try {
 		return await runPrimaryAgentHandoffSubprocess(params);
 	} catch (error) {
-		console.warn("[completion] primary-agent startup-plan generation failed", error);
+		console.warn("[completion] primary-agent handoff generation failed", error);
 		return undefined;
 	}
 }