@pmelab/gtd 38.0.0 → 38.2.0

package/dist/workflows.mjs

@@ -2784,7 +2784,7 @@ const characterEntities = {
 };
 //#endregion
 //#region node_modules/decode-named-character-reference/index.js
-const own$1 = {}.hasOwnProperty;
+const own$2 = {}.hasOwnProperty;
 /**
 * Decode a single character reference (without the `&` or `;`).
 * You probably only need this when you’re building parsers yourself that follow
@@ -2797,7 +2797,7 @@ const own$1 = {}.hasOwnProperty;
 *   Decoded reference.
 */
 function decodeNamedCharacterReference(value) {
-	return own$1.call(characterEntities, value) ? characterEntities[value] : false;
+	return own$2.call(characterEntities, value) ? characterEntities[value] : false;
 }
 //#endregion
 //#region node_modules/micromark-util-chunked/index.js
@@ -10261,7 +10261,7 @@ function index(value) {
 *   Options
 * } from './types.js'
 */
-const own = {}.hasOwnProperty;
+const own$1 = {}.hasOwnProperty;
 /**
 * Turn markdown into a syntax tree.
 *
@@ -10440,7 +10440,7 @@ function compiler(options) {
 		index = -1;
 		while (++index < events.length) {
 			const handler = config[events[index][0]];
-			if (own.call(handler, events[index][1].type)) handler[events[index][1].type].call(Object.assign({ sliceSerialize: events[index][2].sliceSerialize }, context), events[index][1]);
+			if (own$1.call(handler, events[index][1].type)) handler[events[index][1].type].call(Object.assign({ sliceSerialize: events[index][2].sliceSerialize }, context), events[index][1]);
 		}
 		if (context.tokenStack.length > 0) {
 			const tail = context.tokenStack[context.tokenStack.length - 1];
@@ -11152,7 +11152,7 @@ function configure(combined, extensions) {
 function extension(combined, extension) {
 	/** @type {keyof Extension} */
 	let key;
-	for (key in extension) if (own.call(extension, key)) switch (key) {
+	for (key in extension) if (own$1.call(extension, key)) switch (key) {
 		case "canContainEols": {
 			const right = extension[key];
 			if (right) combined[key].push(...right);
@@ -16937,7 +16937,9 @@ const defaults = {
 	sliceMaxCriteria: "8",
 	flag_risk_tier: "0",
 	qualityReviews: "correctness, owasp-security, ponytail-review, test-audit, conventions, spec-challenge",
-	refactorQualityReviews: "correctness, ponytail-review, conventions"
+	refactorQualityReviews: "correctness, ponytail-review, conventions",
+	continuations: "3",
+	continuationStall: "2"
 };
 /** Environment settings. */
 const envDefaults = {
@@ -17455,7 +17457,7 @@ ${strays.map((path) => `  - \`${displayPath(path)}\``).join("\n")}
 - Introduce no new flag read and add no second read of an existing one:
   every match of \`${pattern}\` may move, never increase in number`;
 const withRules = (rules) => rules === void 0 ? "" : `${rules}\n`;
-const refactorBuildingPrompt = ({ pkg, strays, pattern, reference, feedback = false, note = false }) => `${stateFileRules}
+const refactorBuildingPrompt = ({ pkg, strays, pattern, reference, feedback = false, note = false, from }) => `${stateFileRules}
 - The package to implement is \`${pkg}\`; never delete it
 ${note ? `- The package's \`## Refactor note\` section names the seams to open and the
   shape each takes, written from the whole naive pass: build exactly those
@@ -17470,8 +17472,9 @@ ${reference === void 0 ? "" : `- Reference only: \`git show ${reference} -- ${st
   restructuring afresh here, minus the feature's own behaviour. A
   refactor that touches none of it is wrong, however clean — never
   swap in an unrelated tidy-up
-`}- Leave everything uncommitted and finish your turn — do not commit
-`;
+`}${handoffRule()}
+- Leave everything uncommitted and finish your turn — do not commit
+${from === void 0 ? "" : continuationLines(from)}`;
 const refactorFixPrompt = (strays, pattern) => `${stateFileRules}
 ${fixFeedbackPrompt}
 ${refactorRules(strays, pattern)}
@@ -18039,7 +18042,7 @@ Print the closing message and stop — this writes nothing itself.
 const listed = (paths, unreadable = []) => paths.map((path) => `  - \`${displayPath(path)}\`${unreadable.includes(path) ? " (unreadable)" : ""}`).join("\n");
 const isRed = (retry) => "red" in retry;
 const isFault = (retry) => "fault" in retry;
-const pickUp = (hash) => `Pick it up — \`git diff --binary ${hash}^ ${hash} | git apply\``;
+const pickUp = (hash, exclude) => `Pick it up — \`git diff --binary --no-color --no-ext-diff --no-textconv ${hash}^ ${hash}${exclude === void 0 ? "" : ` -- ${quote(":(top)")} ${quote(`:(top,exclude)${exclude}`)}`} | git apply\``;
 const criteriaBinding = (refused) => `- Its \`## Acceptance Criteria\` (\`AC-<n>\`) bind this turn: re-read them from
   the file, never from memory, and stop only when each one holds. They are
   settled — ${refused ? "a turn that changes them is refused" : "leave them as they are"}`;
@@ -18058,7 +18061,33 @@ re-implement. It changed these files outside the expected set:
 ${listed(retry.strays)}
 Start over and choose a different approach; do not rebuild the same diff.
 `;
-const naivePassPrompt = (retry, slice) => `${stateFileRules}
+const quoted = (text) => text.trimEnd().split("\n").map((line) => line === "" ? "" : `  > ${line}`).join("\n");
+/** Every turn a continuation may pick up tells its agent where the note goes. */
+const handoffRule = () => `- Should you be told to stop at a context limit, write the handoff note to
+  \`.gtd/HANDOFF.md\` — what you touched, what was next — and finish your turn:
+  a fresh turn picks up from what you leave, broken or not. That file is the one
+  exception to any path rule here; write it at no other time`;
+/** `pickUp`: how the snapshot gets into the tree — absent when it already is. */
+const continuationLines = (from, pickUp) => `
+Your previous turn stopped at its context limit; this fresh turn picks up from
+its snapshot (\`${from.snapshot}\`). ${pickUp === void 0 ? "Its work is already in the tree." : `${pickUp}.`} Its handoff note:
+
+${quoted(from.note === "" ? "(empty)" : from.note)}
+
+Still open: ${from.open.length === 0 ? "nothing the workflow can count — finish what the note says is next" : from.open.join(", ")}.
+${pickUp === void 0 ? "Delete `.gtd/HANDOFF.md` once you are done; rewrite it should you be stopped again.\n" : ""}`;
+const oversizedMessage = (phase, from, why, narrowable = true) => `The ${phase} keeps outgrowing its context: ${"cap" in why ? why.cap === 0 ? "continuations are off (`continuations` is 0) and the turn stopped at its context limit" : `${why.cap} continuation${why.cap === 1 ? "" : "s"} ran and the last still stopped at its context limit` : `the last ${why.stall} continuation${why.stall === 1 ? "" : "s"} closed nothing still open`}. The last snapshot is \`${from.snapshot}\`; still open: ${from.open.length === 0 ? "nothing countable" : from.open.join(", ")}.
+
+Its handoff note:
+
+${quoted(from.note === "" ? "(empty)" : from.note)}
+
+Land to go on: a fresh round of continuations picks up from that snapshot${narrowable ? ` — narrow
+\`.gtd/REQUIREMENTS.md\` first if the change is too big, or raise the context limit` : ` — raise
+the context limit first if the refactor is too big`}. \`gtd abandon\` gives the change up.
+`;
+/** `from`: a continuation of a pass that stopped at its context limit, in place of `retry`'s pick-up. */
+const naivePassPrompt = (retry, slice, from) => `${stateFileRules}
 - Read \`.gtd/REQUIREMENTS.md\`: the settled requirements for this change
 ${sliceLine(slice)}${criteriaBinding(false)}
 - Implement them the quickest way that works — the naive version. Do not
@@ -18075,9 +18104,20 @@ ${sliceLine(slice)}${criteriaBinding(false)}
 - Run \`${env.testCommand}\` and finish only once it is green with every
   criterion's test passing: the next step runs it on your diff and sends a red
   pass back
-- Leave \`.gtd/\` untouched
+- Leave \`.gtd/\` untouched, a handoff note excepted
+${handoffRule()}
 - Leave everything uncommitted and finish your turn — do not commit
-${retry === void 0 ? "" : naiveRetry(retry)}`;
+${from !== void 0 ? `${continuationLines(from, pickUp(from.snapshot, ".gtd/HANDOFF.md"))}${retry === void 0 ? "" : isFault(retry) ? `
+The pass this continues was a retry: its tests fell short, and still must not:
+${retry.fault}
+` : isRed(retry) ? `
+The pass this continues was a retry after a red suite (\`.gtd/FEEDBACK.md\`
+holds the output): finish only once the suite is green.
+` : `
+The pass this continues was a retry after a \`reject\`: keep away from the
+approach that changed these files outside the expected set:
+${listed(retry.strays)}
+`}` : retry === void 0 ? "" : naiveRetry(retry)}`;
 const carries = (rebuild) => (rebuild.acceptance?.ids.length ?? 0) > 0;
 const rebuildLines = ({ note, acceptance }) => `${note === void 0 ? "" : `- The refactor note (\`.gtd/REFACTOR-NOTE.md\`) named the seams the refactor
   opened; plug the change into them:
@@ -18114,7 +18154,7 @@ const flagLine = (flagMode, tier, amend, sliced) => !flagMode ? "- Add no releas
   change added; only when none did, behind a new one, following the
   repository's flag conventions` : `- Put the new behaviour behind a new release flag, following the
   repository's flag conventions`;
-const implementBuildingPrompt = (reference, tier, amend, afterRefactor, flagMode = true, rebuild = {}, slice) => `${stateFileRules}
+const implementBuildingPrompt = (reference, tier, amend, afterRefactor, flagMode = true, rebuild = {}, slice, from) => `${stateFileRules}
 - Read \`.gtd/REQUIREMENTS.md\`: the settled requirements for this change
 ${sliceLine(slice)}${criteriaBinding(true)}
 ${afterRefactor === true ? afterRefactorLine(rebuild) : ""}${amend === true ? `- The change already exists in the tree from an earlier lap; amend it to meet
@@ -18129,9 +18169,10 @@ ${afterRefactor === true ? afterRefactorLine(rebuild) : ""}${amend === true ? `-
 - Write the tests first, then the code that makes them pass. Name each
   criterion's ID in the test that covers it (\`[AC-3] …\`): once the suite is
   green a judge checks every ID has a passing test, and sends uncovered ones back
+${handoffRule()}
 - Leave \`.gtd/REQUIREMENTS.md\` in place and everything uncommitted, then
   finish your turn
-`;
+${from === void 0 ? "" : continuationLines(from)}`;
 const implementFixPrompt = () => `${stateFileRules}
 ${fixFeedbackPrompt}
 - Leave everything uncommitted and finish your turn — do not commit
@@ -18236,6 +18277,7 @@ const REFACTOR = ".gtd/REFACTOR.md";
 const REFACTOR_NOTE = ".gtd/REFACTOR-NOTE.md";
 const QUALITY = ".gtd/QUALITY.md";
 const SLICES = ".gtd/SLICES.md";
+const HANDOFF = ".gtd/HANDOFF.md";
 const planner = () => env.plannerModel ?? "";
 const coder = () => env.coderModel ?? "";
 const fixCheck = (rules) => agentWithSkills("fix", buildFixPrompt(rules), {
@@ -37738,21 +37780,23 @@ Struct({
 });
 //#endregion
 //#region src/workflows/shared/options.ts
-const setting = optionalWith(String$, { exact: true });
+const setting$1 = optionalWith(String$, { exact: true });
 const ShippedOptions = Struct({
-	reviewBase: setting,
-	judgeBudgetBytes: setting,
-	judgeIdenticalMinP: setting,
-	reviewNoteActionable: setting,
-	reviewSeverityMinP: setting,
-	refactorFlagPattern: setting,
-	seamAllow: setting,
-	sliceMaxCriteria: setting,
-	qualityReviews: setting,
-	refactorQualityReviews: setting,
-	testCommand: setting,
-	plannerModel: setting,
-	coderModel: setting,
+	reviewBase: setting$1,
+	judgeBudgetBytes: setting$1,
+	judgeIdenticalMinP: setting$1,
+	reviewNoteActionable: setting$1,
+	reviewSeverityMinP: setting$1,
+	refactorFlagPattern: setting$1,
+	seamAllow: setting$1,
+	sliceMaxCriteria: setting$1,
+	qualityReviews: setting$1,
+	refactorQualityReviews: setting$1,
+	continuations: setting$1,
+	continuationStall: setting$1,
+	testCommand: setting$1,
+	plannerModel: setting$1,
+	coderModel: setting$1,
 	flagMode: optionalWith(Boolean$, { exact: true }),
 	contextLimit: optionalWith(Struct({
 		soft: optionalWith(Number$, { exact: true }),
@@ -37783,7 +37827,7 @@ const shipped = (flow, doors, base) => defineWorkflow(standardSchemaV1(ShippedOp
 }));
 const FlagOptions = Struct({
 	...ShippedOptions.fields,
-	flagRegistry: setting
+	flagRegistry: setting$1
 });
 const flagShipped = (flow, name) => defineWorkflow(standardSchemaV1(FlagOptions, {
 	onExcessProperty: "error",
@@ -38056,6 +38100,13 @@ const skipDesign = (stated) => stated === void 0 ? Promise.resolve(false) : scop
 	});
 	return true;
 });
+/** After a gate that invites narrowing the requirements: an edit must leave valid criteria. */
+const requireCriteriaIfEdited = () => {
+	const change = changes().get(REQUIREMENTS);
+	if (change === void 0) return;
+	if (change.status === "deleted") refuse(`gtd land: criteria-missing: ${REQUIREMENTS} must stay — narrow its criteria, never delete it`);
+	requireCriteria(REQUIREMENTS);
+};
 //#endregion
 //#region src/workflows/feature/tier.ts
 /** GTD_FLAG_RISK_TIER: pinned at process start, so a mid-process flip never moves a process. */
@@ -38404,6 +38455,53 @@ const reUnwind = async (feedback) => {
 	requireRevert(edited, base);
 };
 //#endregion
+//#region src/workflows/shared/continuation.ts
+const phase = (name, steered) => ({
+	name,
+	spent: false,
+	count: 0,
+	stalled: 0,
+	steered
+});
+const own = (fn) => fn();
+const continued = (fn) => scope("continue", fn);
+const afresh = (fn) => scope("fresh", fn);
+const setting = (name, least) => {
+	const raw = vars[name] ?? defaults[name];
+	const n = Number(raw);
+	if (raw.trim() === "" || !Number.isInteger(n) || n < least) refuse(`gtd: ${name} must be a whole number of at least ${least}, got ${JSON.stringify(raw)}`);
+	return n;
+};
+/**
+* Run `turn` until one ends without a handoff note: each stopped turn is picked
+* up by a continuation in a fresh conversation. Past `continuations` of them in
+* the phase, or after `continuationStall` in a row that close nothing still
+* open, the phase rests at `oversized` for a person; landing there grants a new round.
+*/
+const continuing = async (phase, turn) => {
+	const cap = setting("continuations", 0);
+	const stall = setting("continuationStall", 1);
+	let last = await turn(void 0, phase.spent ? afresh : own);
+	while (last.handoff !== void 0) {
+		phase.spent = true;
+		const from = last.handoff;
+		if (phase.count >= cap || phase.stalled >= stall) {
+			await human("oversized", {
+				message: oversizedMessage(phase.name, from, phase.count >= cap ? { cap } : { stall }, phase.steered !== void 0),
+				label: "Too big for its context",
+				acceptClean: true
+			});
+			phase.steered?.();
+			phase.count = 0;
+			phase.stalled = 0;
+		}
+		last = await turn(from, continued);
+		phase.count++;
+		phase.stalled = from.open.length === 0 || from.open.some((item) => !(last.handoff?.open ?? []).includes(item)) ? 0 : phase.stalled + 1;
+	}
+	return last.result;
+};
+//#endregion
 //#region src/workflows/feature/refactor.ts
 const REFACTOR_PACKAGE = REFACTOR;
 const isCode = (path) => path !== ".gtd" && !path.startsWith(".gtd/");
@@ -38549,7 +38647,7 @@ const guardOf = ({ pattern, strays }, entry) => () => {
 	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 refactorLap = async (p, guard, refactor, feedback) => {
 	const { pattern, strays, reference, note } = p;
 	await run("deriving", deriveScript([
 		...packageLines(strays, pattern, reference),
@@ -38569,32 +38667,48 @@ const refactorLap = async (p, guard, feedback) => {
 		label: "Deriving the refactor package",
 		file: REFACTOR_PACKAGE
 	});
-	await agentWithSkills("building", refactorBuildingPrompt({
-		pkg: REFACTOR_PACKAGE,
-		strays,
-		pattern,
-		reference,
-		feedback: feedback !== void 0,
-		note: note !== 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,
+	const reached = /* @__PURE__ */ new Set();
+	await continuing(refactor, async (from, fresh) => {
+		await fresh(() => agentWithSkills("building", refactorBuildingPrompt({
+			pkg: REFACTOR_PACKAGE,
+			strays,
+			pattern,
+			reference,
+			feedback: feedback !== void 0,
+			note: note !== void 0,
+			from
+		}), {
+			label: from === void 0 ? "Refactoring" : "Continuing the refactor",
 			model: coder(),
 			system: builderSystem()
-		});
+		}));
+		const left = read(HANDOFF);
 		guard();
-	}, {
-		describeGuard: guard,
-		rules: refactorRules(strays, pattern)
+		const snapshot = head$1();
+		for (const c of changes()) reached.add(c.path);
+		await healthy(async () => {
+			await agentWithSkills("fix.suite.fixing", refactorFixPrompt(strays, pattern), {
+				label: "Fixing the check",
+				file: FEEDBACK,
+				model: coder(),
+				system: builderSystem()
+			});
+			guard();
+		}, {
+			describeGuard: guard,
+			rules: refactorRules(strays, pattern)
+		});
+		if (left === void 0) return { result: void 0 };
+		return {
+			result: void 0,
+			handoff: {
+				snapshot,
+				note: left.trim(),
+				open: strays.filter((path) => !reached.has(path))
+			}
+		};
 	});
-	await run("closing", removeScript([REFACTOR_PACKAGE]), { label: "Closing out the refactor" });
+	await run("closing", removeScript([REFACTOR_PACKAGE, HANDOFF]), { label: "Closing out the refactor" });
 };
 const strayAccess = (strays) => ({ write: [
 	...strays,
@@ -38608,7 +38722,7 @@ const refactorFirst = async (shape, options = {}) => {
 		name: "refactor",
 		access: strayAccess(p.strays)
 	}, async () => {
-		await refactorLap(p, guardOf(p, head$1()));
+		await refactorLap(p, guardOf(p, head$1()), phase("refactor"));
 		return true;
 	});
 };
@@ -38633,9 +38747,10 @@ const reviewedRefactor = async (shape, options = {}) => {
 			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
 		};
+		const refactor = phase("refactor");
 		let collected;
 		for (;;) {
-			await refactorLap(p, guard, collected);
+			await refactorLap(p, guard, refactor, collected);
 			const outcome = await buildTail(entry, {
 				green: true,
 				guard,
@@ -38746,12 +38861,32 @@ const judgeSeam = (tip) => judgeDiffShape("judge", {
 	allow: (vars.seamAllow ?? defaults.seamAllow).split(",").map((glob) => glob.trim()).filter((glob) => glob !== ""),
 	exempt: tip.changes.filter((c) => c.status === "added").map((c) => c.path)
 });
-const naivePass = (retry, slice) => agentWithSkills("naive", naivePassPrompt(retry, sliceIds(slice)), {
-	label: retry === void 0 ? "Writing the naive pass" : "Rewriting the naive pass",
+const naivePass = (retry, slice, from) => agentWithSkills("naive", naivePassPrompt(retry, sliceIds(slice), from), {
+	label: from !== void 0 ? "Continuing the naive pass" : retry === void 0 ? "Writing the naive pass" : "Rewriting the naive pass",
 	keep: true,
 	model: coder(),
 	system: builderSystem()
 });
+const handoffOf = (tip, slice) => {
+	const note = tip.changes.find((c) => c.path === ".gtd/HANDOFF.md" && c.after !== void 0);
+	if (note === void 0) return void 0;
+	return {
+		snapshot: tip.hash,
+		note: note.after.trim(),
+		open: unnamed(idsOf(REQUIREMENTS, slice), acceptanceTests(tip.changes))
+	};
+};
+/** A naive pass, continued until a pass ends without a handoff note: the snapshot is its kept tip. */
+const naiveContinued = (retry, slice, naive) => continuing(naive, async (from, fresh) => {
+	const tip = await fresh(async () => {
+		await naivePass(retry, slice, from);
+		return kept("naive");
+	});
+	return {
+		result: tip,
+		handoff: handoffOf(tip, slice)
+	};
+});
 const IMPORTED = /(?:\bfrom|\bimport|\brequire\s*\(|\bmock\s*\()\s*\(?\s*(["'`])([^"'`\n]+)\1/g;
 const EXTENSION = /\.[\w-]+$/;
 const ALIAS = /^(?:[@~]\/|#)/;
@@ -38879,6 +39014,32 @@ const applied = async (tip) => {
 	await run("applying", applyScript(tip, FEEDBACK, naiveApplyFailure(tip), [".gtd"]), { label: "Applying the naive pass" });
 	return !wrote(".gtd/FEEDBACK.md") && codeChanges().length > 0;
 };
+/** `carried` limited to the criteria `.gtd/REQUIREMENTS.md` lists now; a file listing none narrows nothing. */
+const settled = (carried, slice) => {
+	const listed = idsOf(REQUIREMENTS, slice);
+	if (carried === void 0) return listed;
+	return listed.length === 0 ? carried : carried.filter((id) => listed.includes(id));
+};
+const build = (options) => continuing(phase("build", requireCriteriaIfEdited), async (from, fresh) => {
+	await fresh(() => agentWithSkills("building", implementBuildingPrompt(options.reference, options.tier, options.amend, options.afterRefactor, flagModeOn(), options.rebuild, sliceIds(options.slice), from), {
+		label: from === void 0 ? "Implementing" : "Continuing the implementation",
+		model: coder(),
+		system: builderSystem(),
+		...options.afterRefactor === true ? { allowEmpty: true } : {}
+	}));
+	requireCriteriaFrozen(REQUIREMENTS);
+	const note = read(HANDOFF);
+	if (note === void 0) return { result: void 0 };
+	const ids = settled(options.rebuild?.acceptance?.ids, options.slice);
+	return {
+		result: void 0,
+		handoff: {
+			snapshot: head$1(),
+			note: note.trim(),
+			open: unnamed(ids, acceptanceTests(changesSince(start())))
+		}
+	};
+});
 function implement(first, tier, amend) {
 	const options = typeof first === "object" && first !== null ? first : {
 		reference: first,
@@ -38889,14 +39050,12 @@ function implement(first, tier, amend) {
 		const since = head$1();
 		const acceptance = options.rebuild?.acceptance;
 		const built = options.naive === void 0 || !await applied(options.naive);
-		if (built) await agentWithSkills("building", implementBuildingPrompt(options.reference, options.tier, options.amend, options.afterRefactor, flagModeOn(), options.rebuild, sliceIds(options.slice)), {
-			label: "Implementing",
-			model: coder(),
-			system: builderSystem(),
-			...options.afterRefactor === true ? { allowEmpty: true } : {}
-		});
-		requireCriteriaFrozen(REQUIREMENTS);
-		if (built && acceptance !== void 0) requireAcceptance(acceptance, since);
+		if (built) await build(options);
+		else requireCriteriaFrozen(REQUIREMENTS);
+		if (built && acceptance !== void 0) requireAcceptance({
+			...acceptance,
+			ids: settled(acceptance.ids, options.slice)
+		}, since);
 		await green(built && acceptance !== void 0 ? () => requireAcceptanceKept(acceptance) : void 0);
 		await requireCoverage(options.base, options.slice);
 		await closing(options.more === true);
@@ -38905,10 +39064,11 @@ function implement(first, tier, amend) {
 /** The last slice's closing, for a last slice a steer left with no criterion to build. */
 const closeOut = () => scope("implement", () => closing(false));
 /** A slice with `more` after it keeps the criteria and the slicing for the next one. */
-const closing = (more) => run("closing", removeScript(more ? [REFACTOR_NOTE] : [
+const closing = (more) => run("closing", removeScript(more ? [REFACTOR_NOTE, HANDOFF] : [
 	REQUIREMENTS,
 	REFACTOR_NOTE,
-	SLICES
+	SLICES,
+	HANDOFF
 ]), { label: "Closing out the implementation" });
 /** Resolves `true` when the human steered (edited), `false` on a clean overrule. */
 const steer = async (retry) => {
@@ -38980,10 +39140,10 @@ const decide = (slice) => scope("seam", async () => {
 		again: budget(),
 		waived: false
 	};
+	const naive = phase("naive pass", requireCriteriaIfEdited);
 	let retry;
 	for (;;) {
-		await naivePass(retry, slice);
-		const tip = kept("naive");
+		const tip = await naiveContinued(retry, slice, naive);
 		const outcome = await vetted(tip, vetting, slice) ?? (flagModeOn() ? await ruled(tip, vetting.again) : {
 			reference: tip.hash,
 			tip
@@ -39418,4 +39578,4 @@ var workflows_default = feature;
 */
 const doors = {};
 //#endregion
-export { COMMENT, ESCALATION, FEEDBACK, FIX_CAP, FlagOptions, QUALITY, REFACTOR, REFACTOR_NOTE, REFACTOR_PACKAGE, REQUIREMENTS, REVIEW, REVIEW_ORIG, SLICES, ShippedOptions, acceptanceFault, access, afterTail, agentWithSkills, answerProductQuestions, answerReviewQuestions, awaitReview, base, baseline, buildFeature, buildTail, builtInLenses, cleanup, cleanupFlow, closeOut, closeReview, coder, collecting, criteriaOf, criterionIds, workflows_default as default, defaults, deferNotes, describeEscalation, design, doors, envDefaults, escalate, escalation, escalationExhausted, feature, fix, fixCheck, fixFlow, fixNits, fixQuality, fixQualityFindings, fixReviewNotes, fixRisks, gate, guarded, handOffReview, handOffTail, healthy, holdsCallSites, implement, isTestFile, judgeSeam, lensEntriesOf, lensOf, lensesOf, naiveAcceptance, naivePass, ordinaryStart, planAndBuild, planner, qualityLap, reUnwind, refactorCollecting, refactorFirst, refactorNote, refactorStrays, requireAcceptance, requireAcceptanceKept, requireCriteria, requireCriteriaFrozen, requireNoNewFlags, requireNoTestFiles, requireSlices, requireTestCallSitesOnly, requireWithin, retryQuestion, review, reviewFlow, reviewMissing, reviewQuality, reviewedRefactor, reviewing, rollout, rolloutFlow, seamBuild, skills, skipDesign, sliceCap, sliceCriteria, sliceIds, slicesOf, slicing, statedCriteria, steering, summary, tail, threads, triage, unwind };
+export { COMMENT, ESCALATION, FEEDBACK, FIX_CAP, FlagOptions, HANDOFF, QUALITY, REFACTOR, REFACTOR_NOTE, REFACTOR_PACKAGE, REQUIREMENTS, REVIEW, REVIEW_ORIG, SLICES, ShippedOptions, acceptanceFault, access, afterTail, agentWithSkills, answerProductQuestions, answerReviewQuestions, awaitReview, base, baseline, buildFeature, buildTail, builtInLenses, cleanup, cleanupFlow, closeOut, closeReview, coder, collecting, criteriaOf, criterionIds, workflows_default as default, defaults, deferNotes, describeEscalation, design, doors, envDefaults, escalate, escalation, escalationExhausted, feature, fix, fixCheck, fixFlow, fixNits, fixQuality, fixQualityFindings, fixReviewNotes, fixRisks, gate, guarded, handOffReview, handOffTail, healthy, holdsCallSites, implement, isTestFile, judgeSeam, lensEntriesOf, lensOf, lensesOf, naiveAcceptance, naivePass, ordinaryStart, planAndBuild, planner, qualityLap, reUnwind, refactorCollecting, refactorFirst, refactorNote, refactorStrays, requireAcceptance, requireAcceptanceKept, requireCriteria, requireCriteriaFrozen, requireCriteriaIfEdited, requireNoNewFlags, requireNoTestFiles, requireSlices, requireTestCallSitesOnly, requireWithin, retryQuestion, review, reviewFlow, reviewMissing, reviewQuality, reviewedRefactor, reviewing, rollout, rolloutFlow, seamBuild, skills, skipDesign, sliceCap, sliceCriteria, sliceIds, slicesOf, slicing, statedCriteria, steering, summary, tail, threads, triage, unwind };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pmelab/gtd",
-  "version": "38.0.0",
+  "version": "38.2.0",
   "private": false,
   "description": "Git-aware CLI that emits the next prompt for an autonomous coding agent based on the current repository state",
   "bin": {

package/src/cli/Cli.test.ts

@@ -1021,14 +1021,21 @@ describe("parseArgv — gtd history", () => {
     expect(parse("history", "fetch", "upstream")).toMatchObject({
       command: { kind: "historyFetch", remote: "upstream" },
     })
+    expect(parse("history", "metrics")).toMatchObject({
+      command: { kind: "historyMetrics", refs: [] },
+    })
+    expect(parse("history", "metrics", "a", "feat/b")).toMatchObject({
+      command: { kind: "historyMetrics", refs: ["a", "feat/b"] },
+    })
   })
 
-  it("bare history and an unknown subcommand are usage errors naming list, show, fetch", () => {
+  it("bare history and an unknown subcommand are usage errors naming list, show, metrics, fetch", () => {
     for (const args of [["history"], ["history", "nope"]]) {
       const message = usage(parse(...args))
       expect(message).toContain("list")
       expect(message).toContain("show")
       expect(message).toContain("fetch")
+      expect(message).toContain("metrics")
     }
   })
 
@@ -1039,19 +1046,23 @@ describe("parseArgv — gtd history", () => {
     expect(usage(parse("history", "fetch", "a", "b"))).toContain("too many arguments")
   })
 
-  it("--json is accepted for list and show, rejected for fetch", () => {
+  it("--json is accepted for list, show and metrics, rejected for fetch", () => {
     expect(parse("history", "list", "--json")).toMatchObject({ json: { kind: "document" } })
     expect(parse("history", "show", "x", "--json=ended")).toMatchObject({
       json: { kind: "select", path: "ended" },
     })
+    expect(parse("history", "metrics", "--json=total.refactorFirstRate")).toMatchObject({
+      json: { kind: "select", path: "total.refactorFirstRate" },
+    })
     expect(parse("history", "fetch", "--json").kind).toBe("usage")
   })
 
-  it("help lists the three history rows", () => {
+  it("help lists the four history rows", () => {
     const help = renderHelp()
     expect(help).toMatch(/^ {2}history list\b/m)
     expect(help).toContain("history show <ref>")
     expect(help).toContain("history fetch [remote]")
+    expect(help).toContain("history metrics [ref...]")
   })
 })
 
@@ -1136,6 +1147,7 @@ describe("standaloneKinds / needsOf", () => {
       "slices",
       "historyList",
       "historyShow",
+      "historyMetrics",
       "historyFetch",
     ] as const) {
       expect(needsOf(kind)).toBe("state")