@pmelab/gtd 27.4.0 → 27.5.0

package/dist/workflows/feature.mjs

@@ -1,7 +1,7 @@
-import { answered, changes, changesSince, glob, hasThreadFor, head, human, judge, numeric, quote, read, refuse, removeScript, requireAnswers, requireProgress, requireReplies, requireRevert, requireThreadsClosed, restoreScript, revertScript, run, scope, start, vars } from "../flows.mjs";
+import { answered, changes, changesSince, glob, hasThreadFor, head, human, judge, judgeDiffShape, kept, 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 { A as architectureAuthorPrompt, B as designGateAnswerMessage, C as fixRisks, D as reviewing, F as buildReviewCollectingPrompt, H as designTriagePrompt, I as buildReviewFixNitsPrompt, J as packagesItemFixSuitePrompt, K as idleMessage, L as buildReviewTriageMessage, M as architectureGateAnswerMessage, N as buildReviewAnswerQuestionsPrompt, O as agentWithSkills, P as buildReviewAwaitAfterDeferMessage, Q as reviewEditsCapture, R as builderSystem, T as reviewMissing, V as designSystem, X as refactorFixPrompt, Y as refactorBuildingPrompt, Z as reviewEditNotesCapture, _ as REQUIREMENTS, b as coder, c as envDefaults, d as gate, et as reviewNotesCapture, h as FEEDBACK, i as skills, it as unwindFailure, j as architectureDecomposePrompt, k as architectSystem, m as ARCHITECTURE, n as summary, nt as startGateBlockedMessage, o as shipped, p as healthy, q as packagesItemBuildingPrompt, r as access, rt as unwindFailedMessage, s as defaults, t as steering, tt as reviewerSystem, v as REVIEW, w as planner, x as fixCheck, y as awaitReview, z as deferNotesScript } from "../chunks/reserved-BPdVZkgv.mjs";
-import { a as tail, i as refusePendingHandOff, n as closeReview } from "../chunks/tail-iLAMoigl.mjs";
+import { $ as refactorFixPrompt, A as architectureAuthorPrompt, B as designGateAnswerMessage, C as fixRisks, D as reviewing, F as buildReviewCollectingPrompt, H as designTriagePrompt, I as buildReviewFixNitsPrompt, J as implementFixPrompt, K as idleMessage, L as buildReviewTriageMessage, M as architectureGateAnswerMessage, N as buildReviewAnswerQuestionsPrompt, O as agentWithSkills, P as buildReviewAwaitAfterDeferMessage, Q as refactorBuildingPrompt, R as builderSystem, T as reviewMissing, V as designSystem, X as packagesItemBuildingPrompt, Y as naivePassPrompt, Z as packagesItemFixSuitePrompt, _ as REQUIREMENTS, at as seamByHandMessage, b as coder, c as envDefaults, ct as unwindFailedMessage, d as gate, et as reviewEditNotesCapture, h as FEEDBACK, i as skills, it as reviewerSystem, j as architectureDecomposePrompt, k as architectSystem, lt as unwindFailure, m as ARCHITECTURE, n as summary, o as shipped, ot as seamSteerMessage, p as healthy, q as implementBuildingPrompt, r as access, rt as reviewNotesCapture, s as defaults, st as startGateBlockedMessage, t as steering, tt as reviewEditsCapture, v as REVIEW, w as planner, x as fixCheck, y as awaitReview, z as deferNotesScript } from "../chunks/reserved-DM5W_LAt.mjs";
+import { a as tail, i as refusePendingHandOff, n as closeReview } from "../chunks/tail-6gvLXXcg.mjs";
 //#region src/workflows/feature/steps.ts
 /** Turn the sketch since `base` into `.gtd/REQUIREMENTS.md`. */
 const triage = (base) => agentWithSkills("triage", designTriagePrompt(base), {
@@ -345,66 +345,6 @@ const review = (base, escalations = { rounds: 0 }) => {
 /** 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);
 //#endregion
-//#region src/workflows/feature/feature.ts
-/** Revert the sketch that started the process out of the working tree; its intent survives in history. */
-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) => {
-	let base = firstBase;
-	for (;;) {
-		await design(base);
-		await architecture();
-		await packages();
-		const outcome = await buildTail(false, base, true);
-		if (outcome.verdict === "signoff") return;
-		await reUnwind(outcome);
-		base = outcome.base;
-	}
-};
-/** After a tail that skipped planning: stop on sign-off, re-plan on feedback. */
-const afterTail = async (outcome) => {
-	if (outcome.verdict === "signoff") return;
-	await reUnwind(outcome);
-	await planAndBuild(outcome.base);
-};
-/** An ordinary start: wait at `idle`, unwind the sketch, check the baseline, then plan and build. */
-const ordinaryStart = async () => {
-	await human("idle", {
-		message: idleMessage(),
-		label: "Idle",
-		file: ".gtd/TODO.md"
-	});
-	refusePendingHandOff();
-	await unwind();
-	if (changes(".gtd/FEEDBACK.md").some((c) => c.status !== "deleted")) await human("unwind-failed", {
-		message: unwindFailedMessage(),
-		label: "Could not unwind your input",
-		file: FEEDBACK
-	});
-	await gate("start-gate", startGateBlockedMessage());
-	await planAndBuild(start());
-};
-const feature = shipped(ordinaryStart);
-//#endregion
 //#region src/workflows/feature/refactor.ts
 const REFACTOR_PACKAGE = ".gtd/REFACTOR.md";
 const isCode = (path) => path !== ".gtd" && !path.startsWith(".gtd/");
@@ -518,7 +458,143 @@ const refactorFirst = async (shape, options = {}) => {
 	});
 };
 //#endregion
+//#region src/workflows/feature/seam.ts
+/** Naive-pass retries after a `reject`; the next `reject` rests at `seam.steer`. */
+const NAIVE_RETRIES = 2;
+/** A change to an existing file outside the allowlist is a stray even when the task needs it: the judge decides. */
+const judgeSeam = (tip) => judgeDiffShape("judge", {
+	changes: tip.changes,
+	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 = (rejection) => agentWithSkills("naive", naivePassPrompt(rejection), {
+	label: rejection === void 0 ? "Writing the naive pass" : "Rewriting the naive pass",
+	keep: true,
+	model: coder(),
+	system: builderSystem()
+});
+const implement = (reference) => scope("implement", async () => {
+	await agentWithSkills("building", implementBuildingPrompt(reference), {
+		label: "Implementing",
+		model: coder(),
+		system: builderSystem()
+	});
+	await healthy(() => agentWithSkills("fix.suite.fixing", implementFixPrompt(), {
+		label: "Fixing the check",
+		file: FEEDBACK,
+		model: coder(),
+		system: builderSystem()
+	}));
+	await run("closing", removeScript([REQUIREMENTS]), { label: "Closing out the implementation" });
+});
+const decide = () => scope("seam", async () => {
+	let rejection;
+	let retries = 0;
+	for (;;) {
+		await naivePass(rejection);
+		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
+			};
+		}
+		rejection = {
+			strays: shape.strays,
+			p: shape.judgment?.answers.shape?.p ?? 0
+		};
+		if (retries < NAIVE_RETRIES) {
+			retries++;
+			continue;
+		}
+		await human("steer", {
+			message: seamSteerMessage(rejection),
+			label: "Steer or overrule the rejected naive pass",
+			file: REQUIREMENTS,
+			acceptClean: true
+		});
+		if (changes().length === 0) return { reference: tip.hash };
+		rejection = void 0;
+		retries = 0;
+	}
+});
+const seamBuild = async () => {
+	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);
+};
+//#endregion
+//#region src/workflows/feature/feature.ts
+/** Revert the sketch that started the process out of the working tree; its intent survives in history. */
+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) => {
+	let base = firstBase;
+	for (;;) {
+		await design(base);
+		if (vars.flag_naive_pass === "1") await seamBuild();
+		else {
+			await architecture();
+			await packages();
+		}
+		const outcome = await buildTail(false, base, true);
+		if (outcome.verdict === "signoff") return;
+		await reUnwind(outcome);
+		base = outcome.base;
+	}
+};
+/** After a tail that skipped planning: stop on sign-off, re-plan on feedback. */
+const afterTail = async (outcome) => {
+	if (outcome.verdict === "signoff") return;
+	await reUnwind(outcome);
+	await planAndBuild(outcome.base);
+};
+/** An ordinary start: wait at `idle`, unwind the sketch, check the baseline, then plan and build. */
+const ordinaryStart = async () => {
+	await human("idle", {
+		message: idleMessage(),
+		label: "Idle",
+		file: ".gtd/TODO.md"
+	});
+	refusePendingHandOff();
+	await unwind();
+	if (changes(".gtd/FEEDBACK.md").some((c) => c.status !== "deleted")) await human("unwind-failed", {
+		message: unwindFailedMessage(),
+		label: "Could not unwind your input",
+		file: FEEDBACK
+	});
+	await gate("start-gate", startGateBlockedMessage());
+	await planAndBuild(start());
+};
+const feature = shipped(ordinaryStart);
+//#endregion
 //#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, nextPackage, ordinaryStart, packageItem, packages, planAndBuild, reUnwind, refactorFirst, requireDeclaredTests, requireNoNewFlags, requireWithin, 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, refactorFirst, requireDeclaredTests, requireNoNewFlags, requireWithin, 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 { O as agentWithSkills, U as finisherSystem, W as fixReviewNotesPrompt, b as coder, c as envDefaults, i as skills, l as baseline, n as summary, o as shipped, r as access, s as defaults, t as steering, v as REVIEW } from "../chunks/reserved-BPdVZkgv.mjs";
-import { r as handOffTail, t as REVIEW_ORIG } from "../chunks/tail-iLAMoigl.mjs";
+import { O as agentWithSkills, U as finisherSystem, W as fixReviewNotesPrompt, b as coder, c as envDefaults, i as skills, l as baseline, n as summary, o as shipped, r as access, s as defaults, t as steering, v as REVIEW } from "../chunks/reserved-DM5W_LAt.mjs";
+import { r as handOffTail, t as REVIEW_ORIG } from "../chunks/tail-6gvLXXcg.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 { $ as reviewGateBlockedMessage, c as envDefaults, d as gate, i as skills, n as summary, o as shipped, r as access, s as defaults, t as steering } from "../chunks/reserved-BPdVZkgv.mjs";
-import { i as refusePendingHandOff, r as handOffTail } from "../chunks/tail-iLAMoigl.mjs";
+import { c as envDefaults, d as gate, i as skills, n as summary, nt as reviewGateBlockedMessage, o as shipped, r as access, s as defaults, t as steering } from "../chunks/reserved-DM5W_LAt.mjs";
+import { i as refusePendingHandOff, r as handOffTail } from "../chunks/tail-6gvLXXcg.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, head, read, refuse, scope, vars } from "../flows.mjs";
-import { a as flagShipped, b as coder, c as envDefaults, d as gate, i as skills, n as summary, r as access, s as defaults, t as steering } from "../chunks/reserved-BPdVZkgv.mjs";
+import { a as flagShipped, b as coder, c as envDefaults, d as gate, i as skills, n as summary, r as access, s as defaults, t as steering } from "../chunks/reserved-DM5W_LAt.mjs";
 import { i as requireFlag, n as flipProblems } from "../chunks/registry-CP3YuV9j.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.`;