@pmelab/gtd 28.2.0 → 29.1.0

package/dist/workflows/feature.mjs

@@ -1,10 +1,8 @@
-import { answered, changes, changesSince, glob, hasThreadFor, head, human, judge, judgeDiffShape, kept, mark, numeric, quote, read, refuse, removeScript, requireAnswers, requireProgress, requireReplies, requireRevert, requireThreadsClosed, restoreScript, revertScript, run, scope, seamGate, start, vars } from "../flows.mjs";
-import { review as review$1 } from "../workflow-kit.mjs";
-import { $ as reviewEditsCapture, A as architectureGateAnswerMessage, B as designTriagePrompt, C as reviewMissing, D as architectSystem, E as agentWithSkills, F as buildReviewTriageMessage, G as implementBuildingPrompt, I as builderSystem, J as packagesItemBuildingPrompt, K as implementFixPrompt, L as deferNotesScript, M as buildReviewAwaitAfterDeferMessage, N as buildReviewCollectingPrompt, O as architectureAuthorPrompt, P as buildReviewFixNitsPrompt, Q as reviewEditNotesCapture, R as designGateAnswerMessage, S as planner, T as reviewing, W as idleMessage, X as refactorBuildingPrompt, Y as packagesItemFixSuitePrompt, Z as refactorFixPrompt, _ as awaitReview, at as startGateBlockedMessage, ct as defaults, d as healthy, f as ARCHITECTURE, g as REVIEW, h as REQUIREMENTS, i as skills, it as seamSteerMessage, j as buildReviewAnswerQuestionsPrompt, k as architectureDecomposePrompt, l as gate, lt as envDefaults, n as summary, nt as reviewerSystem, o as shipped, ot as unwindFailedMessage, p as FEEDBACK, q as naivePassPrompt, r as access, rt as seamByHandMessage, st as unwindFailure, t as steering, tt as reviewNotesCapture, v as coder, x as fixRisks, y as fixCheck, z as designSystem } from "../chunks/reserved-DeBS4Ymh.mjs";
-import { a as tail, i as refusePendingHandOff, n as closeReview } from "../chunks/tail-B2SER2Ih.mjs";
-//#region src/workflows/feature/tier.ts
-/** GTD_FLAG_NAIVE_PASS, GTD_FLAG_RISK_TIER: both pinned at process start, so a mid-process flip never moves a process. */
-const riskTierOn = () => vars.flag_naive_pass === "1" && vars.flag_risk_tier === "1";
+import { answered, changes, changesSince, glob, hasThreadFor, head, human, judge, judgeDiffShape, kept, mark, matches, numeric, quote, read, refuse, removeScript, requireAnswers, requireProgress, requireReplies, requireRevert, requireThreadsClosed, restoreScript, revertScript, run, scope, slice, start, vars } from "../flows.mjs";
+import { lensesOf, review as review$1 } from "../workflow-kit.mjs";
+import { $ as refactorFixPrompt, A as architectureDecomposePrompt, B as designSystem, C as planner, D as agentWithSkills, E as reviewing, F as buildReviewFixNitsPrompt, G as idleMessage, I as buildReviewTriageMessage, J as naivePassPrompt, K as implementBuildingPrompt, L as builderSystem, M as buildReviewAnswerQuestionsPrompt, N as buildReviewAwaitAfterDeferMessage, O as architectSystem, P as buildReviewCollectingPrompt, Q as refactorDroppedRefusal, R as deferNotesScript, S as fixRisks, V as designTriagePrompt, X as packagesItemFixSuitePrompt, Y as packagesItemBuildingPrompt, Z as refactorBuildingPrompt, _ as REVIEW, at as reviewerSystem, b as fixCheck, ct as startGateBlockedMessage, d as healthy, dt as defaults, et as refactorRules, f as ARCHITECTURE, ft as envDefaults, g as REQUIREMENTS, h as REFACTOR, i as skills, it as reviewNotesCapture, j as architectureGateAnswerMessage, k as architectureAuthorPrompt, l as gate, lt as unwindFailedMessage, n as summary, nt as reviewEditsCapture, o as shipped, ot as seamByHandMessage, p as FEEDBACK, pt as testFileGlobs, q as implementFixPrompt, r as access, st as seamSteerMessage, t as steering, tt as reviewEditNotesCapture, u as guarded, ut as unwindFailure, v as awaitReview, w as reviewMissing, y as coder, z as designGateAnswerMessage } from "../chunks/reserved-DoHUII2D.mjs";
+import { a as tail, i as refusePendingHandOff, n as closeReview } from "../chunks/tail-C6YBG5zh.mjs";
+//#region src/workflows/shared/fences.ts
 const markFenced = (text) => {
 	let fence;
 	return text.split(/\r?\n/).map((line) => {
@@ -18,6 +16,10 @@ const markFenced = (text) => {
 		};
 	});
 };
+//#endregion
+//#region src/workflows/feature/tier.ts
+/** GTD_FLAG_NAIVE_PASS, GTD_FLAG_RISK_TIER: both pinned at process start, so a mid-process flip never moves a process. */
+const riskTierOn = () => vars.flag_naive_pass === "1" && vars.flag_risk_tier === "1";
 const sectionLines = (text, title) => {
 	if (text === void 0) return [];
 	const heading = new RegExp(`^##[ \\t]+${title}[ \\t]*$`, "i");
@@ -107,12 +109,18 @@ const answerReviewQuestions = (notes) => agentWithSkills("review.answer-review-q
 	system: reviewerSystem()
 });
 /** Fix every `nit` note in one turn. */
-const fixNits = (notes) => agentWithSkills("review.fix.nits.fixing", buildReviewFixNitsPrompt(notes), {
-	label: "Fixing your nits",
-	file: REVIEW,
-	model: planner(),
-	system: reviewerSystem()
-});
+const fixNits = (notes, access, rules) => {
+	const fixing = (name) => agentWithSkills(name, buildReviewFixNitsPrompt(notes, rules), {
+		label: "Fixing your nits",
+		file: REVIEW,
+		model: planner(),
+		system: reviewerSystem()
+	});
+	return access === void 0 ? fixing("review.fix.nits.fixing") : scope({
+		name: "review.fix.nits",
+		access
+	}, () => fixing("fixing"));
+};
 /** Turn the review round `capture` describes into `.gtd/REQUIREMENTS.md`, or change nothing. */
 const collecting = (capture) => agentWithSkills("review.collecting", buildReviewCollectingPrompt(capture), {
 	label: "Collecting your feedback",
@@ -122,6 +130,17 @@ const collecting = (capture) => agentWithSkills("review.collecting", buildReview
 	system: reviewerSystem(),
 	allowEmpty: true
 });
+/** Turn the refactor review round `capture` describes into `.gtd/REFACTOR.md`, or change nothing. */
+const refactorCollecting = (capture, round, strays) => agentWithSkills("review.collecting", buildReviewCollectingPrompt(capture, {
+	strays,
+	round
+}), {
+	label: "Collecting your refactor feedback",
+	file: REFACTOR,
+	model: planner(),
+	system: reviewerSystem(),
+	allowEmpty: true
+});
 /** `build.review.defer`: append `notes` to `.gtd/TODO.md` as one section; the step's landing is the commit. */
 const deferNotes = (notes, round) => run("review.defer", deferNotesScript(notes, round), {
 	label: "Deferring could-notes to TODO.md",
@@ -339,26 +358,37 @@ const isCode$1 = (path) => path !== ".gtd" && !path.startsWith(".gtd/");
 * another planning lap, questions are answered in place, nits fixed in one
 * batch, praise dropped.
 */
-const review = (base, escalations = { rounds: 0 }) => {
+const review = (base, escalations = { rounds: 0 }, options = {}) => {
+	const { guard, guardRests, fixAccess, rules } = options;
+	const feedback = options.feedback ?? {
+		file: ".gtd/REQUIREMENTS.md",
+		collecting,
+		unmatched: "gtd land: no declared pattern matches the pending changes — write .gtd/REQUIREMENTS.md from the feedback, or change nothing when it asks for nothing"
+	};
 	const severities = /* @__PURE__ */ new Map();
 	let deferred = [];
 	return review$1(base, {
 		file: REVIEW,
-		feedback: REQUIREMENTS,
+		feedback: feedback.file,
 		isCode: isCode$1,
 		reviewing: (b, carry) => {
 			deferred = [];
-			return reviewing(b, carry);
+			return reviewing(b, carry, options.lap);
 		},
-		fixRisks,
-		keepGreen: () => healthy(fixCheck, { escalations }),
+		fixRisks: guarded((notes) => fixRisks(notes, fixAccess, rules), guard),
+		keepGreen: () => healthy(() => fixCheck(rules), {
+			escalations,
+			guard,
+			guardRests,
+			rules
+		}),
 		awaitReview: (b) => {
 			const notes = deferred;
 			deferred = [];
 			return notes.length > 0 ? awaitReview(b, buildReviewAwaitAfterDeferMessage(b, notes)) : awaitReview(b);
 		},
 		reviewMissing,
-		collecting,
+		collecting: feedback.collecting,
 		triage: triage$1(severities),
 		defer: (notes) => {
 			deferred = notes.map((n) => ({
@@ -370,7 +400,7 @@ const review = (base, escalations = { rounds: 0 }) => {
 			return deferNotes(deferred, head());
 		},
 		answerQuestions: answerReviewQuestions,
-		fixNits,
+		fixNits: guarded((notes) => fixNits(notes, fixAccess, rules), guard),
 		close: closeReview,
 		captures: {
 			edits: reviewEditsCapture,
@@ -388,21 +418,44 @@ const review = (base, escalations = { rounds: 0 }) => {
 		},
 		refusals: {
 			deleted: "gtd land: review-doc: .gtd/REVIEW.md was deleted — restore it, or leave a note (or edit code) to request changes.",
-			unmatched: "gtd land: no declared pattern matches the pending changes — write .gtd/REQUIREMENTS.md from the feedback, or change nothing when it asks for nothing"
+			unmatched: feedback.unmatched,
+			...feedback.dropped === void 0 ? {} : { dropped: feedback.dropped }
 		}
 	});
 };
 /** The build tail: fix (when entered red), keep green unless `green`, the quality lap, then human review since `base`. */
-const buildTail = (fixFirst, base, green = false) => tail(fixFirst, (escalations) => review(base, escalations), green);
+const buildTail = (fixFirst, base, green = false, options = {}) => tail(fixFirst, (escalations) => review(base, escalations, options), green, options.guard, options.lenses, options.guardRests, options.lensBase, options.rules);
+/**
+* Undo the human's review-round code edits. A path changed since the human
+* left it is not overwritten; requireRevert names it instead.
+*/
+const reUnwind = async (feedback) => {
+	const { base, restoreFrom, edited } = feedback;
+	await run("re-unwind", restoreScript(base, {
+		restore: edited.filter((c) => c.status !== "added").map((c) => c.path),
+		remove: edited.filter((c) => c.status === "added").map((c) => c.path)
+	}, restoreFrom), {
+		label: "Re-unwinding your review edit",
+		file: REVIEW,
+		base
+	});
+	requireRevert(edited, base);
+};
 //#endregion
 //#region src/workflows/feature/refactor.ts
-const REFACTOR_PACKAGE = ".gtd/REFACTOR.md";
+const REFACTOR_PACKAGE = REFACTOR;
 const isCode = (path) => path !== ".gtd" && !path.startsWith(".gtd/");
 const requireWithin = (allowed) => {
 	const outside = changes().map((c) => c.path).filter((path) => isCode(path) && !allowed.includes(path));
 	if (outside.length === 0) return;
 	refuse(`gtd land: refactor-scope: the refactor may change only its stray paths — revert these:\n${outside.map((path) => `  - ${path}`).join("\n")}`);
 };
+const isTestFile = (path) => testFileGlobs.some((glob) => matches(path, glob));
+const requireNoTestFiles = () => {
+	const tests = changes().map((c) => c.path).filter((path) => isCode(path) && isTestFile(path));
+	if (tests.length === 0) return;
+	refuse(`gtd land: refactor-tests: the refactor may not change a test file — revert these:\n${tests.map((path) => `  - ${path}`).join("\n")}`);
+};
 const flagExpression = (pattern) => {
 	if (pattern.trim() === "") throw new Error("gtd: refactorFlagPattern must not be blank");
 	let expression;
@@ -444,6 +497,8 @@ const requireNoNewFlags = (pattern, since) => {
 	if (raised.length === 0) return;
 	refuse(`gtd land: refactor-flags: the refactor may not introduce or add reads of a flag (${pattern}):\n${raised.map(([name, count]) => `  - ${name}: ${before.get(name) ?? 0} → ${count}`).join("\n")}`);
 };
+/** A test-file stray is never refactored: the implement turn still reads it in the naive tip. */
+const refactorStrays = (shape) => shape.strays.filter((path) => !isTestFile(path));
 const packageLines = (strays, pattern, reference) => [
 	"# Behaviour-neutral refactor",
 	"",
@@ -456,7 +511,7 @@ const packageLines = (strays, pattern, reference) => [
 	"## Rules",
 	"",
 	"- The suite's outcome must not change",
-	"- Change only the paths above; a test file only when it is one of them",
+	"- Change only the paths above, and never a test file",
 	`- Introduce no flag read and add no second read of an existing one (\`${pattern}\`)`,
 	"",
 	"## Reference",
@@ -464,49 +519,125 @@ const packageLines = (strays, pattern, reference) => [
 	reference === void 0 ? "None." : `Reading material only, never applied: \`git show ${reference} -- ${strays.map(quote).join(" ")}\``,
 	""
 ];
+const demoteHeadings = (text) => {
+	const marked = markFenced(text.trimEnd());
+	const lines = marked.map((m) => m.line);
+	const headings = marked.map((m) => m.fenced ? void 0 : /^(#{1,6}) /.exec(m.line)?.[1]?.length);
+	const shift = 3 - Math.min(...headings.filter((n) => n !== void 0));
+	return lines.map((line, i) => {
+		const depth = headings[i];
+		if (depth === void 0) return line;
+		return "#".repeat(Math.min(6, depth + shift)) + line.slice(depth);
+	});
+};
 const deriveScript = (lines) => `mkdir -p .gtd
 printf '%s\\n' ${lines.map(quote).join(" ")} > ${quote(REFACTOR_PACKAGE)}
 `;
-const refactorFirst = async (shape, options = {}) => {
-	if (shape.choice !== "refactor-first" || shape.unreadable.length > 0) return false;
+/** Undefined for every shape that is not a readable `refactor-first`; throws on a bad flag pattern. */
+const plan = (shape, options) => {
+	if (shape.choice !== "refactor-first" || shape.unreadable.length > 0) return void 0;
 	const pattern = vars.refactorFlagPattern ?? defaults.refactorFlagPattern;
 	const expression = flagExpression(pattern);
-	const { strays } = shape;
+	const strays = refactorStrays(shape);
+	if (strays.length === 0) return void 0;
 	const zeroLength = strays.filter((path) => countFlags(read(path) ?? "", expression, /* @__PURE__ */ new Map()));
 	if (zeroLength.length > 0) throw new Error(`gtd: refactorFlagPattern (${pattern}) produces a zero-length match in:\n${listPaths(zeroLength)}`);
-	const reference = options.reference?.trim() ? options.reference : void 0;
-	return scope({
-		name: "refactor",
-		access: { write: [...strays, ".gtd/**"] }
-	}, async () => {
-		await run("deriving", deriveScript(packageLines(strays, pattern, reference)), {
-			label: "Deriving the refactor package",
-			file: REFACTOR_PACKAGE
-		});
-		const from = head();
-		const guard = () => {
-			requireWithin(strays);
-			requireNoNewFlags(pattern, from);
-		};
-		await agentWithSkills("building", refactorBuildingPrompt(REFACTOR_PACKAGE, strays, pattern, reference), {
-			label: "Refactoring",
+	return {
+		pattern,
+		strays,
+		reference: options.reference?.trim() ? options.reference : void 0
+	};
+};
+const guardOf = ({ pattern, strays }, entry) => () => {
+	requireNoTestFiles();
+	requireWithin(strays);
+	requireNoNewFlags(pattern, entry);
+};
+/** Derive, build, check and close one refactor lap; `feedback` is the collected review text of the lap before. */
+const refactorLap = async (p, guard, feedback) => {
+	const { pattern, strays, reference } = p;
+	await run("deriving", deriveScript([...packageLines(strays, pattern, reference), ...feedback === void 0 ? [] : [
+		"## Review feedback",
+		"",
+		...demoteHeadings(feedback),
+		""
+	]]), {
+		label: "Deriving the refactor package",
+		file: REFACTOR_PACKAGE
+	});
+	await agentWithSkills("building", refactorBuildingPrompt(REFACTOR_PACKAGE, strays, pattern, reference, feedback !== void 0), {
+		label: "Refactoring",
+		model: coder(),
+		system: builderSystem()
+	});
+	guard();
+	await healthy(async () => {
+		await agentWithSkills("fix.suite.fixing", refactorFixPrompt(strays, pattern), {
+			label: "Fixing the check",
+			file: FEEDBACK,
 			model: coder(),
 			system: builderSystem()
 		});
 		guard();
-		await healthy(async () => {
-			await agentWithSkills("fix.suite.fixing", refactorFixPrompt(strays, pattern), {
-				label: "Fixing the check",
-				file: FEEDBACK,
-				model: coder(),
-				system: builderSystem()
-			});
-			guard();
-		}, { describeGuard: guard });
-		await run("closing", removeScript([REFACTOR_PACKAGE]), { label: "Closing out the refactor" });
+	}, {
+		describeGuard: guard,
+		rules: refactorRules(strays, pattern)
+	});
+	await run("closing", removeScript([REFACTOR_PACKAGE]), { label: "Closing out the refactor" });
+};
+const strayAccess = (strays) => ({ write: [...strays, ".gtd/**"] });
+const refactorFirst = async (shape, options = {}) => {
+	const p = plan(shape, options);
+	if (p === void 0) return false;
+	return scope({
+		name: "refactor",
+		access: strayAccess(p.strays)
+	}, async () => {
+		await refactorLap(p, guardOf(p, head()));
 		return true;
 	});
 };
+/**
+* `refactorFirst` plus its review, in the one `refactor` scope so the tail's
+* steps are `refactor.build.*`. A review edit re-runs the refactor, never a
+* re-plan: the collected notes ride the re-derived package. Every round
+* reviews the whole refactor since `entry`, since each lap rebuilds over it.
+*/
+const reviewedRefactor = async (shape, options = {}) => {
+	const p = plan(shape, options);
+	if (p === void 0) return false;
+	return scope({
+		name: "refactor",
+		access: strayAccess(p.strays)
+	}, async () => {
+		const entry = head();
+		const guard = guardOf(p, entry);
+		const feedback = {
+			file: REFACTOR_PACKAGE,
+			collecting: (capture, round) => refactorCollecting(capture, round, p.strays),
+			unmatched: "gtd land: refactor-feedback: the review feedback changed files but wrote no .gtd/REFACTOR.md — a refactor cannot act on it; revert the change, or `gtd abandon` and restart",
+			dropped: refactorDroppedRefusal
+		};
+		let collected;
+		for (;;) {
+			await refactorLap(p, guard, collected);
+			const outcome = await buildTail(false, entry, true, {
+				guard,
+				guardRests: false,
+				lenses: lensesOf(vars.refactorQualityReviews ?? ""),
+				feedback,
+				fixAccess: strayAccess(p.strays),
+				lensBase: entry,
+				rules: refactorRules(p.strays, p.pattern),
+				lap: "refactor"
+			});
+			if (outcome.verdict === "signoff") return true;
+			await reUnwind(outcome);
+			collected = read(REFACTOR_PACKAGE);
+			if (collected === void 0) throw new Error(`gtd: review feedback left no ${REFACTOR_PACKAGE}`);
+		}
+	});
+};
 //#endregion
 //#region src/workflows/feature/seam.ts
 /** Naive-pass retries after a `reject`; the next `reject` rests at `seam.steer`. */
@@ -523,8 +654,8 @@ const naivePass = (rejection) => agentWithSkills("naive", naivePassPrompt(reject
 	model: coder(),
 	system: builderSystem()
 });
-const implement = (reference, tier) => scope("implement", async () => {
-	await agentWithSkills("building", implementBuildingPrompt(reference, tier), {
+const implement = (reference, tier, amend) => scope("implement", async () => {
+	await agentWithSkills("building", implementBuildingPrompt(reference, tier, amend), {
 		label: "Implementing",
 		model: coder(),
 		system: builderSystem()
@@ -545,13 +676,10 @@ const decide = () => scope("seam", async () => {
 		const tip = kept("naive");
 		const shape = await judgeSeam(tip);
 		if (shape.choice === "land") return { reference: tip.hash };
-		if (shape.choice === "refactor-first") {
-			await seamGate("approve", shape);
-			return {
-				reference: tip.hash,
-				shape
-			};
-		}
+		if (shape.choice === "refactor-first") return {
+			reference: tip.hash,
+			shape
+		};
 		rejection = {
 			strays: shape.strays,
 			p: shape.judgment?.answers.shape?.p ?? 0
@@ -571,14 +699,32 @@ const decide = () => scope("seam", async () => {
 		retries = 0;
 	}
 });
-const seamBuild = async (tier) => {
+const seamBuild = async (base) => {
 	const { reference, shape } = await decide();
-	if (shape !== void 0 && !await refactorFirst(shape, { reference })) await scope("seam", () => human("by-hand", {
-		message: seamByHandMessage(shape.strays, shape.unreadable, reference),
-		label: "Reviewing unreadable files by hand",
-		acceptClean: true
-	}));
-	await implement(reference, tier);
+	if (shape === void 0) return {
+		reference,
+		base
+	};
+	if (shape.unreadable.length > 0) {
+		await scope("seam", () => human("by-hand", {
+			message: seamByHandMessage(shape.strays, shape.unreadable, reference),
+			label: "Reviewing unreadable files by hand",
+			acceptClean: true
+		}));
+		return {
+			reference,
+			base
+		};
+	}
+	if (refactorStrays(shape).length === 0) return {
+		reference,
+		base
+	};
+	await slice("refactor", () => reviewedRefactor(shape, { reference }));
+	return {
+		reference,
+		base: head()
+	};
 };
 //#endregion
 //#region src/workflows/feature/feature.ts
@@ -587,44 +733,49 @@ const unwind = () => {
 	const commit = head();
 	return run("unwind", revertScript(commit, FEEDBACK, unwindFailure(commit)), { label: "Unwinding your input" });
 };
-/**
-* Undo the human's review-round code edits. A path changed since the human
-* left it is not overwritten; requireRevert names it instead.
-*/
-const reUnwind = async (feedback) => {
-	const { base, restoreFrom, edited } = feedback;
-	await run("re-unwind", restoreScript(base, {
-		restore: edited.filter((c) => c.status !== "added").map((c) => c.path),
-		remove: edited.filter((c) => c.status === "added").map((c) => c.path)
-	}, restoreFrom), {
-		label: "Re-unwinding your review edit",
-		file: REVIEW,
-		base
-	});
-	requireRevert(edited, base);
-};
 /** Plan, build and review until a review round signs off; feedback re-plans from scratch. */
-const planAndBuild = async (firstBase) => {
+const planAndBuild = async (firstBase, firstLap = "first") => {
+	if (vars.flag_naive_pass === "1") return naivePlanAndBuild(firstBase);
 	let base = firstBase;
+	let lap = firstLap;
 	for (;;) {
 		await design(base);
-		const tier = riskTierOn() ? recordRisk() : void 0;
-		if (vars.flag_naive_pass === "1") await seamBuild(tier);
-		else {
-			await architecture();
-			await packages();
-		}
-		const outcome = await buildTail(false, base, true);
+		await architecture();
+		await packages();
+		const outcome = await buildTail(false, base, true, lap === "first" ? {} : { lap });
 		if (outcome.verdict === "signoff") return;
 		await reUnwind(outcome);
 		base = outcome.base;
+		lap = "feedback";
+	}
+};
+const naivePlanAndBuild = async (firstBase) => {
+	await design(firstBase);
+	const tier = riskTierOn() ? recordRisk() : void 0;
+	const seam = await seamBuild(firstBase);
+	await slice("feature", async () => {
+		await implement(seam.reference, tier);
+		await featureFeedback(await buildTail(false, seam.base, true, { lensBase: seam.base }), seam.base);
+	});
+};
+const featureFeedback = async (first, lensBase) => {
+	let outcome = first;
+	while (outcome.verdict === "feedback") {
+		await reUnwind(outcome);
+		await design(outcome.base);
+		await implement(void 0, riskTierOn() ? recordRisk() : void 0, true);
+		outcome = await buildTail(false, outcome.base, true, {
+			lap: "feedback",
+			...lensBase === void 0 ? {} : { lensBase }
+		});
 	}
 };
-/** After a tail that skipped planning: stop on sign-off, re-plan on feedback. */
+/** After a tail that skipped planning: stop on sign-off, re-plan on feedback. Opens no slice: slices cannot nest. */
 const afterTail = async (outcome) => {
 	if (outcome.verdict === "signoff") return;
+	if (vars.flag_naive_pass === "1") return featureFeedback(outcome);
 	await reUnwind(outcome);
-	await planAndBuild(outcome.base);
+	await planAndBuild(outcome.base, "feedback");
 };
 /** An ordinary start: wait at `idle`, unwind the sketch, check the baseline, then plan and build. */
 const ordinaryStart = async () => {
@@ -648,4 +799,4 @@ const feature = shipped(ordinaryStart);
 //#region src/workflows/feature/index.ts
 var feature_default = feature;
 //#endregion
-export { REFACTOR_PACKAGE, access, afterTail, answerProductQuestions, answerReviewQuestions, answerTechnicalQuestions, architecture, author, build, buildTail, collecting, declaredTests, decompose, feature_default as default, defaults, deferNotes, design, envDefaults, feature, fixNits, fixSuite, implement, judgeSeam, naivePass, nextPackage, ordinaryStart, packageItem, packages, planAndBuild, reUnwind, refactorFirst, requireDeclaredTests, requireNoNewFlags, requireWithin, seamBuild, skills, steering, summary, triage, unwind };
+export { REFACTOR_PACKAGE, access, afterTail, answerProductQuestions, answerReviewQuestions, answerTechnicalQuestions, architecture, author, build, buildTail, collecting, declaredTests, decompose, feature_default as default, defaults, deferNotes, design, envDefaults, feature, fixNits, fixSuite, implement, judgeSeam, naivePass, nextPackage, ordinaryStart, packageItem, packages, planAndBuild, reUnwind, refactorCollecting, refactorFirst, refactorStrays, requireDeclaredTests, requireNoNewFlags, requireNoTestFiles, requireWithin, reviewedRefactor, seamBuild, skills, steering, summary, triage, unwind };

package/dist/workflows/fix.mjs

@@ -1,7 +1,7 @@
 import { read, refuse, removeScript, run, scope, start } from "../flows.mjs";
 import { d as reviewNotes, f as reviewRisks } from "../chunks/steering-DVILanVf.mjs";
-import { E as agentWithSkills, H as fixReviewNotesPrompt, V as finisherSystem, ct as defaults, g as REVIEW, i as skills, lt as envDefaults, n as summary, o as shipped, r as access, s as baseline, t as steering, v as coder } from "../chunks/reserved-DeBS4Ymh.mjs";
-import { r as handOffTail, t as REVIEW_ORIG } from "../chunks/tail-B2SER2Ih.mjs";
+import { D as agentWithSkills, H as finisherSystem, U as fixReviewNotesPrompt, _ as REVIEW, dt as defaults, ft as envDefaults, i as skills, n as summary, o as shipped, r as access, s as baseline, t as steering, y as coder } from "../chunks/reserved-DoHUII2D.mjs";
+import { r as handOffTail, t as REVIEW_ORIG } from "../chunks/tail-C6YBG5zh.mjs";
 //#region src/workflows/fix/fix.ts
 /**
 * Fix every note a human left in a handed-off `.gtd/REVIEW.md` and every

package/dist/workflows/review.mjs

@@ -1,6 +1,6 @@
 import { start } from "../flows.mjs";
-import { ct as defaults, et as reviewGateBlockedMessage, i as skills, l as gate, lt as envDefaults, n as summary, o as shipped, r as access, t as steering } from "../chunks/reserved-DeBS4Ymh.mjs";
-import { i as refusePendingHandOff, r as handOffTail } from "../chunks/tail-B2SER2Ih.mjs";
+import { dt as defaults, ft as envDefaults, i as skills, l as gate, n as summary, o as shipped, r as access, rt as reviewGateBlockedMessage, t as steering } from "../chunks/reserved-DoHUII2D.mjs";
+import { i as refusePendingHandOff, r as handOffTail } from "../chunks/tail-C6YBG5zh.mjs";
 //#region src/workflows/review/review.ts
 /**
 * Pure review of everything since `reviewBase`: a green gate, the quality

package/dist/workflows/rollout.mjs

@@ -1,5 +1,5 @@
 import { agent, changesSince, glob, head, mark, read, refuse, removeScript, run, scope, vars } from "../flows.mjs";
-import { a as flagShipped, ct as defaults, ft as flipProblems, i as skills, l as gate, lt as envDefaults, mt as requireFlag, n as summary, p as FEEDBACK, r as access, t as steering, ut as flagMentions, v as coder } from "../chunks/reserved-DeBS4Ymh.mjs";
+import { a as flagShipped, dt as defaults, ft as envDefaults, gt as flipProblems, i as skills, l as gate, mt as flagMentions, n as summary, p as FEEDBACK, r as access, t as steering, vt as requireFlag, y as coder } from "../chunks/reserved-DoHUII2D.mjs";
 //#region src/workflows/rollout/rollout.ts
 const flippingPrompt = (name, registry) => `Roll out the release flag "${name}": in ${registry}, set \`defaultOn: true\` on its entry. Change nothing else — no other entry, no other field, no other file, no comments.`;
 const suiteBlockedMessage = () => "The suite is red after the rollout. Fix it by hand — a rollout may only touch the flag's registry entry — then land.";