@pmelab/gtd 27.2.0 → 27.3.0

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "gtd",
-  "version": "27.2.0",
+  "version": "27.3.0",
   "description": "Drive the gtd loop inside Claude Code: agent turns run as subagents, gtd decides every beat",
   "author": {
     "name": "Philipp Melab"

package/dist/chunks/{reserved-B2QaImnN.mjs → reserved-BPdVZkgv.mjs}

@@ -591,6 +591,26 @@ ${fixFeedbackPrompt}
   then legitimately report itself already satisfied later
 - Leave everything uncommitted and finish your turn — do not commit
 `;
+const refactorRules = (strays, pattern) => `- Behaviour-neutral: the suite's outcome must not change
+- Change only these paths, nothing else — a test file only when it is
+  one of them:
+${strays.map((path) => `  - \`${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 refactorBuildingPrompt = (pkg, strays, pattern, reference) => `${stateFileRules}
+- The package to implement is \`${pkg}\`; never delete it
+${refactorRules(strays, pattern)}
+${reference === void 0 ? "" : `- Reference only: \`git show ${reference} -- ${strays.map(quote).join(" ")}\` shows an
+  earlier attempt at this change. Read it for ideas; never apply,
+  cherry-pick or check it out — write a fresh refactor
+`}- Leave everything uncommitted and finish your turn — do not commit
+`;
+const refactorFixPrompt = (strays, pattern) => `${stateFileRules}
+${fixFeedbackPrompt}
+${refactorRules(strays, pattern)}
+- Never do another package's work, even when it would green the suite
+- Leave everything uncommitted and finish your turn — do not commit
+`;
 const healthJudgeMessage = () => `The check is still red, and this isn't the first round —
 \`.gtd/FEEDBACK.md\` holds this round's output, and the previous round's
 is in history. Run \`gtd judge answer\` and pipe a
@@ -1161,9 +1181,9 @@ const guarded = (turn, guard) => async (...a) => {
 	await turn(...a);
 	guard?.();
 };
-const guardedSteps = (guard) => ({
+const guardedSteps = (guard, describeGuard) => ({
 	limit: escalationSteps.limit,
-	describe: guarded(escalationSteps.describe, guard),
+	describe: guarded(guarded(escalationSteps.describe, guard), describeGuard),
 	stop: guarded(escalationSteps.stop, guard),
 	exhausted: guarded(escalationSteps.exhausted, guard)
 });
@@ -1215,7 +1235,7 @@ const healthy = (fix, options = {}) => healthy$1(guarded(fix, options.guard), {
 	fixesSoFar: options.fixesSoFar ?? 0,
 	sameFailure,
 	escalations: options.escalations ?? { rounds: 0 },
-	escalation: guardedSteps(options.guard)
+	escalation: guardedSteps(options.guard, options.describeGuard)
 });
 /**
 * Hold the process at `<name>.blocked` until the suite is green. `guard` runs
@@ -1236,6 +1256,7 @@ const defaults = {
 	judgeIdenticalMinP: "0.7",
 	reviewNoteActionable: "0.7",
 	reviewSeverityMinP: "0.7",
+	refactorFlagPattern: "\\bGTD_FLAG_\\w+",
 	qualityReviews: "correctness, owasp-security, ponytail-review, test-audit, conventions, spec-challenge"
 };
 /** Environment settings. */
@@ -1253,6 +1274,7 @@ const ShippedOptions = Struct({
 	judgeIdenticalMinP: setting,
 	reviewNoteActionable: setting,
 	reviewSeverityMinP: setting,
+	refactorFlagPattern: setting,
 	qualityReviews: setting,
 	testCommand: setting,
 	plannerModel: setting,
@@ -1314,6 +1336,9 @@ const skills = (vars) => ({
 	"build.review.fix.nits": ["incremental-implementation", "code-simplification"],
 	"build.review.fix.risks": ["debugging-and-error-recovery", "incremental-implementation"],
 	notes: ["incremental-implementation", "code-simplification"],
+	refactor: ["ponytail"],
+	"refactor.fix.suite": ["ponytail", "debugging-and-error-recovery"],
+	"refactor.health": ["ponytail", "debugging-and-error-recovery"],
 	...lensScopes("build.quality", lensesOf(vars.qualityReviews ?? ""), (lens) => builtInLenses[lens]?.skills ?? [lens])
 });
 //#endregion
@@ -1337,4 +1362,4 @@ const steering = {
 	[REVIEW]: "review"
 };
 //#endregion
-export { reviewerSystem as $, architectureAuthorPrompt as A, designGateAnswerMessage as B, fixRisks as C, reviewing as D, reviewQuality as E, buildReviewCollectingPrompt as F, handOffAwaitReviewMessage as G, designTriagePrompt as H, buildReviewFixNitsPrompt as I, packagesItemFixSuitePrompt as J, idleMessage as K, buildReviewTriageMessage as L, architectureGateAnswerMessage as M, buildReviewAnswerQuestionsPrompt as N, agentWithSkills as O, buildReviewAwaitAfterDeferMessage as P, reviewNotesCapture as Q, builderSystem as R, fixQuality as S, reviewMissing as T, finisherSystem as U, designSystem as V, fixReviewNotesPrompt as W, reviewEditsCapture as X, reviewEditNotesCapture as Y, reviewGateBlockedMessage as Z, REQUIREMENTS as _, flagShipped as a, coder as b, envDefaults as c, gate as d, startGateBlockedMessage as et, guarded as f, QUALITY as g, FEEDBACK as h, skills as i, architectureDecomposePrompt as j, architectSystem as k, baseline as l, ARCHITECTURE as m, summary as n, unwindFailure as nt, shipped as o, healthy as p, packagesItemBuildingPrompt as q, access as r, defaults as s, steering as t, unwindFailedMessage as tt, escalation as u, REVIEW as v, planner as w, fixCheck as x, awaitReview as y, deferNotesScript as z };
+export { reviewGateBlockedMessage as $, architectureAuthorPrompt as A, designGateAnswerMessage as B, fixRisks as C, reviewing as D, reviewQuality as E, buildReviewCollectingPrompt as F, handOffAwaitReviewMessage as G, designTriagePrompt as H, buildReviewFixNitsPrompt as I, packagesItemFixSuitePrompt as J, idleMessage as K, buildReviewTriageMessage as L, architectureGateAnswerMessage as M, buildReviewAnswerQuestionsPrompt as N, agentWithSkills as O, buildReviewAwaitAfterDeferMessage as P, reviewEditsCapture as Q, builderSystem as R, fixQuality as S, reviewMissing as T, finisherSystem as U, designSystem as V, fixReviewNotesPrompt as W, refactorFixPrompt as X, refactorBuildingPrompt as Y, reviewEditNotesCapture as Z, REQUIREMENTS as _, flagShipped as a, coder as b, envDefaults as c, gate as d, reviewNotesCapture as et, guarded as f, QUALITY as g, FEEDBACK as h, skills as i, unwindFailure as it, architectureDecomposePrompt as j, architectSystem as k, baseline as l, ARCHITECTURE as m, summary as n, startGateBlockedMessage as nt, shipped as o, healthy as p, packagesItemBuildingPrompt as q, access as r, unwindFailedMessage as rt, defaults as s, steering as t, reviewerSystem as tt, escalation as u, REVIEW as v, planner as w, fixCheck as x, awaitReview as y, deferNotesScript as z };

package/dist/chunks/{tail-BnhIxCYv.mjs → tail-iLAMoigl.mjs}

@@ -1,6 +1,6 @@
 import { read, refuse, removeScript, run, scope, showScript, vars } from "../flows.mjs";
 import { fixFindings, lensesOf, qualityLap as qualityLap$1, reviewToHandOff } from "../workflow-kit.mjs";
-import { C as fixRisks, D as reviewing, E as reviewQuality, G as handOffAwaitReviewMessage, S as fixQuality, T as reviewMissing, f as guarded, g as QUALITY, p as healthy, u as escalation, v as REVIEW, x as fixCheck, y as awaitReview } from "./reserved-B2QaImnN.mjs";
+import { C as fixRisks, D as reviewing, E as reviewQuality, G as handOffAwaitReviewMessage, S as fixQuality, T as reviewMissing, f as guarded, g as QUALITY, p as healthy, u as escalation, v as REVIEW, x as fixCheck, y as awaitReview } from "./reserved-BPdVZkgv.mjs";
 //#region src/workflows/shared/tail.ts
 /** The reviewer's own `.gtd/REVIEW.md`, kept beside the human's notes so `fix` can tell them apart. */
 const REVIEW_ORIG = ".gtd/REVIEW.orig.md";

package/dist/testkit.mjs

@@ -12961,6 +12961,17 @@ const recognizeDeferNotes = (repo, block) => {
 	repo.writeFile(".gtd/TODO.md", `${old}${sep}${lines.join("\n")}\n`);
 	return { kind: "applied" };
 };
+/** `mkdir -p <dir>` then `printf '%s\n' <quoted lines> > <path>`: a flow's one-file write. Re-rendered from what was extracted, so any other shape stays unrecognized. */
+const recognizeWriteLines = (repo, block) => {
+	const [mkdir, printf, ...rest] = block.split("\n");
+	if (rest.length > 0 || !mkdir?.startsWith("mkdir -p ") || !printf?.startsWith("printf '%s\\n' ")) return;
+	const tokens = extractQuotedTokens(printf.slice(14));
+	const path = tokens.pop();
+	if (path === void 0 || !path.includes("/")) return void 0;
+	if (`mkdir -p ${path.slice(0, path.lastIndexOf("/"))}\nprintf '%s\\n' ${tokens.map(shellQuote).join(" ")} > ${shellQuote(path)}` !== block) return void 0;
+	repo.writeFile(path, `${tokens.join("\n")}\n`);
+	return { kind: "applied" };
+};
 /** Every recognizer `applyEmittedScript` tries, first match wins — `recognizeScriptedCommand` stays last since it's the exact-hit fallback for anything the rest don't claim. */
 const recognizersFor = (repo, commands) => [
 	(block) => recognizeSetFlags(block),
@@ -12970,6 +12981,7 @@ const recognizersFor = (repo, commands) => [
 	(block) => recognizeBinaryGuard(commands, block),
 	(block) => recognizeGitBuilders(repo, block),
 	(block) => recognizeDeferNotes(repo, block),
+	(block) => recognizeWriteLines(repo, block),
 	(block) => recognizeFailurePromptWrapper(repo, commands, block),
 	(block) => recognizePresentationSubshell(repo, commands, block),
 	(block) => recognizeModeContradictionSkipNotice(block),

package/dist/workflows/cleanup.mjs

@@ -1,6 +1,6 @@
 import { agent, changes, glob, read, refuse, scope, start, vars } from "../flows.mjs";
-import { a as flagShipped, b as coder, c as envDefaults, i as skills, n as summary, r as access, s as defaults, t as steering } from "../chunks/reserved-B2QaImnN.mjs";
-import { i as refusePendingHandOff, r as handOffTail } from "../chunks/tail-BnhIxCYv.mjs";
+import { a as flagShipped, b as coder, c as envDefaults, i as skills, n as summary, 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 { i as requireFlag, r as registryEntry, t as flagReferences } from "../chunks/registry-CP3YuV9j.mjs";
 //#region src/workflows/cleanup/cleanup.ts
 const removingPrompt = (name, registry) => `Clean up the release flag "${name}", which is default-on: delete its entry in ${registry}, and remove every \`GTD_FLAG_${name.toUpperCase()}\` check from the shipped sources. Keep the flag-on path; delete code only the off path reached, including tests that pin the off path.`;

package/dist/workflows/feature.mjs

@@ -1,7 +1,7 @@
-import { answered, changes, changesSince, glob, hasThreadFor, head, human, judge, numeric, 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, numeric, quote, read, refuse, removeScript, requireAnswers, requireProgress, requireReplies, requireRevert, requireThreadsClosed, restoreScript, revertScript, run, scope, start, vars } from "../flows.mjs";
 import { review as review$1 } from "../workflow-kit.mjs";
-import { $ as reviewerSystem, 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 reviewNotesCapture, R as builderSystem, T as reviewMissing, V as designSystem, X as reviewEditsCapture, Y as reviewEditNotesCapture, _ as REQUIREMENTS, b as coder, c as envDefaults, d as gate, et as startGateBlockedMessage, h as FEEDBACK, i as skills, j as architectureDecomposePrompt, k as architectSystem, m as ARCHITECTURE, n as summary, nt as unwindFailure, o as shipped, p as healthy, q as packagesItemBuildingPrompt, r as access, s as defaults, t as steering, tt as unwindFailedMessage, v as REVIEW, w as planner, x as fixCheck, y as awaitReview, z as deferNotesScript } from "../chunks/reserved-B2QaImnN.mjs";
-import { a as tail, i as refusePendingHandOff, n as closeReview } from "../chunks/tail-BnhIxCYv.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";
 //#region src/workflows/feature/steps.ts
 /** Turn the sketch since `base` into `.gtd/REQUIREMENTS.md`. */
 const triage = (base) => agentWithSkills("triage", designTriagePrompt(base), {
@@ -282,7 +282,7 @@ const triage$1 = (severities) => async (notes) => {
 		return [n.id, severity.tier === "could" ? "defer" : "edit"];
 	}));
 };
-const isCode = (path) => path !== ".gtd" && !path.startsWith(".gtd/");
+const isCode$1 = (path) => path !== ".gtd" && !path.startsWith(".gtd/");
 /**
 * A reviewer writes `.gtd/REVIEW.md` over everything since `base`, a human
 * reviews and signs off or comments, and each note is judged: edits go to
@@ -295,7 +295,7 @@ const review = (base, escalations = { rounds: 0 }) => {
 	return review$1(base, {
 		file: REVIEW,
 		feedback: REQUIREMENTS,
-		isCode,
+		isCode: isCode$1,
 		reviewing: (b, carry) => {
 			deferred = [];
 			return reviewing(b, carry);
@@ -405,7 +405,120 @@ const ordinaryStart = async () => {
 };
 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/");
+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 flagExpression = (pattern) => {
+	if (pattern.trim() === "") throw new Error("gtd: refactorFlagPattern must not be blank");
+	let expression;
+	try {
+		expression = new RegExp(pattern, "g");
+	} catch (error) {
+		throw new Error(`gtd: refactorFlagPattern is not a valid regular expression: ${String(error)}`);
+	}
+	if (expression.test("")) throw new Error("gtd: refactorFlagPattern must not match the empty string");
+	return expression;
+};
+const countFlags = (text, expression, into) => {
+	let zeroLength = false;
+	for (const [name] of text.matchAll(expression)) if (name === "") zeroLength = true;
+	else into.set(name, (into.get(name) ?? 0) + 1);
+	return zeroLength;
+};
+const listPaths = (paths) => paths.map((path) => `  - ${path}`).join("\n");
+const requireNoNewFlags = (pattern, since) => {
+	const expression = flagExpression(pattern);
+	const before = /* @__PURE__ */ new Map();
+	const after = /* @__PURE__ */ new Map();
+	const unreadable = [];
+	const zeroLength = [];
+	for (const { path, status, before: was, after: now } of changesSince(since)) {
+		if (!isCode(path)) continue;
+		const oldSide = status === "added" ? "" : was;
+		const newSide = status === "deleted" ? "" : now;
+		if (oldSide === void 0 || newSide === void 0) {
+			unreadable.push(path);
+			continue;
+		}
+		countFlags(oldSide, expression, before);
+		if (countFlags(newSide, expression, after)) zeroLength.push(path);
+	}
+	if (unreadable.length > 0) refuse(`gtd land: refactor-flags: cannot compare flag reads in:\n${listPaths(unreadable)}`);
+	if (zeroLength.length > 0) refuse(`gtd land: refactor-flags: refactorFlagPattern (${pattern}) produced a zero-length match in:\n${listPaths(zeroLength)}`);
+	const raised = [...after].filter(([name, count]) => count > (before.get(name) ?? 0));
+	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")}`);
+};
+const packageLines = (strays, pattern, reference) => [
+	"# Behaviour-neutral refactor",
+	"",
+	"## Scope (read-only)",
+	"",
+	...strays.map((path) => `- \`${path}\``),
+	"",
+	"This list is fixed when the package is derived: editing it here widens or narrows nothing.",
+	"",
+	"## Rules",
+	"",
+	"- The suite's outcome must not change",
+	"- Change only the paths above; a test file only when it is one of them",
+	`- Introduce no flag read and add no second read of an existing one (\`${pattern}\`)`,
+	"",
+	"## Reference",
+	"",
+	reference === void 0 ? "None." : `Reading material only, never applied: \`git show ${reference} -- ${strays.map(quote).join(" ")}\``,
+	""
+];
+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;
+	const pattern = vars.refactorFlagPattern ?? defaults.refactorFlagPattern;
+	const expression = flagExpression(pattern);
+	const { strays } = shape;
+	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",
+			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" });
+		return true;
+	});
+};
+//#endregion
 //#region src/workflows/feature/index.ts
 var feature_default = feature;
 //#endregion
-export { 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, requireDeclaredTests, 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, nextPackage, ordinaryStart, packageItem, packages, planAndBuild, reUnwind, refactorFirst, requireDeclaredTests, requireNoNewFlags, requireWithin, 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-B2QaImnN.mjs";
-import { r as handOffTail, t as REVIEW_ORIG } from "../chunks/tail-BnhIxCYv.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";
 //#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 { Z 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-B2QaImnN.mjs";
-import { i as refusePendingHandOff, r as handOffTail } from "../chunks/tail-BnhIxCYv.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";
 //#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-B2QaImnN.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 { 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.`;

package/dist/workflows.mjs

@@ -250,9 +250,9 @@ const check = async (name, command, options) => {
 	}), { label });
 	return !wrote(report);
 };
-const isCode$1 = (path) => path !== ".gtd" && !path.startsWith(".gtd/");
+const isCode$2 = (path) => path !== ".gtd" && !path.startsWith(".gtd/");
 /** The last step's changes outside `.gtd/`. */
-const codeChanges = () => changes().filter((c) => isCode$1(c.path));
+const codeChanges = () => changes().filter((c) => isCode$2(c.path));
 /** Refuse a turn whose only change deletes `file` — unless it says it found nothing to do. */
 const requireProgress = (file) => {
 	const change = changes().get(file);
@@ -17235,6 +17235,26 @@ ${fixFeedbackPrompt}
   then legitimately report itself already satisfied later
 - Leave everything uncommitted and finish your turn — do not commit
 `;
+const refactorRules = (strays, pattern) => `- Behaviour-neutral: the suite's outcome must not change
+- Change only these paths, nothing else — a test file only when it is
+  one of them:
+${strays.map((path) => `  - \`${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 refactorBuildingPrompt = (pkg, strays, pattern, reference) => `${stateFileRules}
+- The package to implement is \`${pkg}\`; never delete it
+${refactorRules(strays, pattern)}
+${reference === void 0 ? "" : `- Reference only: \`git show ${reference} -- ${strays.map(quote).join(" ")}\` shows an
+  earlier attempt at this change. Read it for ideas; never apply,
+  cherry-pick or check it out — write a fresh refactor
+`}- Leave everything uncommitted and finish your turn — do not commit
+`;
+const refactorFixPrompt = (strays, pattern) => `${stateFileRules}
+${fixFeedbackPrompt}
+${refactorRules(strays, pattern)}
+- Never do another package's work, even when it would green the suite
+- Leave everything uncommitted and finish your turn — do not commit
+`;
 const healthJudgeMessage = () => `The check is still red, and this isn't the first round —
 \`.gtd/FEEDBACK.md\` holds this round's output, and the previous round's
 is in history. Run \`gtd judge answer\` and pipe a
@@ -17809,9 +17829,9 @@ const guarded = (turn, guard) => async (...a) => {
 	await turn(...a);
 	guard?.();
 };
-const guardedSteps = (guard) => ({
+const guardedSteps = (guard, describeGuard) => ({
 	limit: escalationSteps.limit,
-	describe: guarded(escalationSteps.describe, guard),
+	describe: guarded(guarded(escalationSteps.describe, guard), describeGuard),
 	stop: guarded(escalationSteps.stop, guard),
 	exhausted: guarded(escalationSteps.exhausted, guard)
 });
@@ -17863,7 +17883,7 @@ const healthy = (fix, options = {}) => healthy$1(guarded(fix, options.guard), {
 	fixesSoFar: options.fixesSoFar ?? 0,
 	sameFailure,
 	escalations: options.escalations ?? { rounds: 0 },
-	escalation: guardedSteps(options.guard)
+	escalation: guardedSteps(options.guard, options.describeGuard)
 });
 /**
 * Hold the process at `<name>.blocked` until the suite is green. `guard` runs
@@ -37198,6 +37218,7 @@ const defaults = {
 	judgeIdenticalMinP: "0.7",
 	reviewNoteActionable: "0.7",
 	reviewSeverityMinP: "0.7",
+	refactorFlagPattern: "\\bGTD_FLAG_\\w+",
 	qualityReviews: "correctness, owasp-security, ponytail-review, test-audit, conventions, spec-challenge"
 };
 /** Environment settings. */
@@ -37215,6 +37236,7 @@ const ShippedOptions = Struct({
 	judgeIdenticalMinP: setting,
 	reviewNoteActionable: setting,
 	reviewSeverityMinP: setting,
+	refactorFlagPattern: setting,
 	qualityReviews: setting,
 	testCommand: setting,
 	plannerModel: setting,
@@ -37614,7 +37636,7 @@ const triage$1 = (severities) => async (notes) => {
 		return [n.id, severity.tier === "could" ? "defer" : "edit"];
 	}));
 };
-const isCode = (path) => path !== ".gtd" && !path.startsWith(".gtd/");
+const isCode$1 = (path) => path !== ".gtd" && !path.startsWith(".gtd/");
 /**
 * A reviewer writes `.gtd/REVIEW.md` over everything since `base`, a human
 * reviews and signs off or comments, and each note is judged: edits go to
@@ -37627,7 +37649,7 @@ const review$1 = (base, escalations = { rounds: 0 }) => {
 	return review$2(base, {
 		file: REVIEW,
 		feedback: REQUIREMENTS,
-		isCode,
+		isCode: isCode$1,
 		reviewing: (b, carry) => {
 			deferred = [];
 			return reviewing(b, carry);
@@ -37756,6 +37778,9 @@ const skills = (vars) => ({
 	"build.review.fix.nits": ["incremental-implementation", "code-simplification"],
 	"build.review.fix.risks": ["debugging-and-error-recovery", "incremental-implementation"],
 	notes: ["incremental-implementation", "code-simplification"],
+	refactor: ["ponytail"],
+	"refactor.fix.suite": ["ponytail", "debugging-and-error-recovery"],
+	"refactor.health": ["ponytail", "debugging-and-error-recovery"],
 	...lensScopes("build.quality", lensesOf(vars.qualityReviews ?? ""), (lens) => builtInLenses[lens]?.skills ?? [lens])
 });
 //#endregion
@@ -37779,6 +37804,119 @@ const steering = {
 	[REVIEW]: "review"
 };
 //#endregion
+//#region src/workflows/feature/refactor.ts
+const REFACTOR_PACKAGE = ".gtd/REFACTOR.md";
+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 flagExpression = (pattern) => {
+	if (pattern.trim() === "") throw new Error("gtd: refactorFlagPattern must not be blank");
+	let expression;
+	try {
+		expression = new RegExp(pattern, "g");
+	} catch (error) {
+		throw new Error(`gtd: refactorFlagPattern is not a valid regular expression: ${String(error)}`);
+	}
+	if (expression.test("")) throw new Error("gtd: refactorFlagPattern must not match the empty string");
+	return expression;
+};
+const countFlags = (text, expression, into) => {
+	let zeroLength = false;
+	for (const [name] of text.matchAll(expression)) if (name === "") zeroLength = true;
+	else into.set(name, (into.get(name) ?? 0) + 1);
+	return zeroLength;
+};
+const listPaths = (paths) => paths.map((path) => `  - ${path}`).join("\n");
+const requireNoNewFlags = (pattern, since) => {
+	const expression = flagExpression(pattern);
+	const before = /* @__PURE__ */ new Map();
+	const after = /* @__PURE__ */ new Map();
+	const unreadable = [];
+	const zeroLength = [];
+	for (const { path, status, before: was, after: now } of changesSince(since)) {
+		if (!isCode(path)) continue;
+		const oldSide = status === "added" ? "" : was;
+		const newSide = status === "deleted" ? "" : now;
+		if (oldSide === void 0 || newSide === void 0) {
+			unreadable.push(path);
+			continue;
+		}
+		countFlags(oldSide, expression, before);
+		if (countFlags(newSide, expression, after)) zeroLength.push(path);
+	}
+	if (unreadable.length > 0) refuse(`gtd land: refactor-flags: cannot compare flag reads in:\n${listPaths(unreadable)}`);
+	if (zeroLength.length > 0) refuse(`gtd land: refactor-flags: refactorFlagPattern (${pattern}) produced a zero-length match in:\n${listPaths(zeroLength)}`);
+	const raised = [...after].filter(([name, count]) => count > (before.get(name) ?? 0));
+	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")}`);
+};
+const packageLines = (strays, pattern, reference) => [
+	"# Behaviour-neutral refactor",
+	"",
+	"## Scope (read-only)",
+	"",
+	...strays.map((path) => `- \`${path}\``),
+	"",
+	"This list is fixed when the package is derived: editing it here widens or narrows nothing.",
+	"",
+	"## Rules",
+	"",
+	"- The suite's outcome must not change",
+	"- Change only the paths above; a test file only when it is one of them",
+	`- Introduce no flag read and add no second read of an existing one (\`${pattern}\`)`,
+	"",
+	"## Reference",
+	"",
+	reference === void 0 ? "None." : `Reading material only, never applied: \`git show ${reference} -- ${strays.map(quote).join(" ")}\``,
+	""
+];
+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;
+	const pattern = vars.refactorFlagPattern ?? defaults.refactorFlagPattern;
+	const expression = flagExpression(pattern);
+	const { strays } = shape;
+	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$1();
+		const guard = () => {
+			requireWithin(strays);
+			requireNoNewFlags(pattern, from);
+		};
+		await agentWithSkills("building", refactorBuildingPrompt(REFACTOR_PACKAGE, strays, pattern, reference), {
+			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();
+		}, { describeGuard: guard });
+		await run("closing", removeScript([REFACTOR_PACKAGE]), { label: "Closing out the refactor" });
+		return true;
+	});
+};
+//#endregion
 //#region src/workflows/review/review.ts
 /**
 * Pure review of everything since `reviewBase`: a green gate, the quality
@@ -37996,4 +38134,4 @@ var workflows_default = feature;
 */
 const doors = {};
 //#endregion
-export { ARCHITECTURE, ESCALATION, FEEDBACK, FIX_CAP, FlagOptions, QUALITY, REQUIREMENTS, REVIEW, REVIEW_ORIG, ShippedOptions, access, afterTail, agentWithSkills, answerProductQuestions, answerReviewQuestions, answerTechnicalQuestions, architecture, author, awaitReview, base, baseline, build, buildTail, builtInLenses, cleanup, cleanupFlow, closeReview, coder, collecting, declaredTests, decompose, workflows_default as default, defaults, deferNotes, describeEscalation, design, doors, envDefaults, escalate, escalation, escalationExhausted, feature, fix, fixCheck, fixFlow, fixNits, fixQuality, fixQualityFindings, fixReviewNotes, fixRisks, fixSuite, gate, guarded, handOffReview, handOffTail, healthy, lensesOf, nextPackage, ordinaryStart, packageItem, packages, planAndBuild, planner, qualityLap, reUnwind, requireDeclaredTests, retryQuestion, review, reviewFlow, reviewMissing, reviewQuality, reviewing, rollout, rolloutFlow, skills, steering, summary, tail, threads, triage, unwind };
+export { ARCHITECTURE, ESCALATION, FEEDBACK, FIX_CAP, FlagOptions, QUALITY, REFACTOR_PACKAGE, REQUIREMENTS, REVIEW, REVIEW_ORIG, ShippedOptions, access, afterTail, agentWithSkills, answerProductQuestions, answerReviewQuestions, answerTechnicalQuestions, architecture, author, awaitReview, base, baseline, build, buildTail, builtInLenses, cleanup, cleanupFlow, closeReview, coder, collecting, declaredTests, decompose, workflows_default as default, defaults, deferNotes, describeEscalation, design, doors, envDefaults, escalate, escalation, escalationExhausted, feature, fix, fixCheck, fixFlow, fixNits, fixQuality, fixQualityFindings, fixReviewNotes, fixRisks, fixSuite, gate, guarded, handOffReview, handOffTail, healthy, lensesOf, nextPackage, ordinaryStart, packageItem, packages, planAndBuild, planner, qualityLap, reUnwind, refactorFirst, requireDeclaredTests, requireNoNewFlags, requireWithin, retryQuestion, review, reviewFlow, reviewMissing, reviewQuality, reviewing, rollout, rolloutFlow, skills, steering, summary, tail, threads, triage, unwind };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pmelab/gtd",
-  "version": "27.2.0",
+  "version": "27.3.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/testkit/EmittedScriptRecognizer.ts

@@ -667,6 +667,26 @@ const recognizeDeferNotes = (repo: InMemRepo, block: string): BlockOutcome | und
   return { kind: "applied" }
 }
 
+/** `mkdir -p <dir>` then `printf '%s\n' <quoted lines> > <path>`: a flow's one-file write. Re-rendered from what was extracted, so any other shape stays unrecognized. */
+const recognizeWriteLines = (repo: InMemRepo, block: string): BlockOutcome | undefined => {
+  const [mkdir, printf, ...rest] = block.split("\n")
+  if (
+    rest.length > 0 ||
+    !mkdir?.startsWith("mkdir -p ") ||
+    !printf?.startsWith("printf '%s\\n' ")
+  ) {
+    return undefined
+  }
+  const tokens = extractQuotedTokens(printf.slice("printf '%s\\n' ".length))
+  const path = tokens.pop()
+  if (path === undefined || !path.includes("/")) return undefined
+  const dir = path.slice(0, path.lastIndexOf("/"))
+  const rendered = `mkdir -p ${dir}\nprintf '%s\\n' ${tokens.map(shellQuote).join(" ")} > ${shellQuote(path)}`
+  if (rendered !== block) return undefined
+  repo.writeFile(path, `${tokens.join("\n")}\n`)
+  return { kind: "applied" }
+}
+
 /** Every recognizer `applyEmittedScript` tries, first match wins — `recognizeScriptedCommand` stays last since it's the exact-hit fallback for anything the rest don't claim. */
 const recognizersFor = (
   repo: InMemRepo,
@@ -679,6 +699,7 @@ const recognizersFor = (
   (block) => recognizeBinaryGuard(commands, block),
   (block) => recognizeGitBuilders(repo, block),
   (block) => recognizeDeferNotes(repo, block),
+  (block) => recognizeWriteLines(repo, block),
   (block) => recognizeFailurePromptWrapper(repo, commands, block),
   (block) => recognizePresentationSubshell(repo, commands, block),
   (block) => recognizeModeContradictionSkipNotice(block),

package/src/testkit/EmittedScripts.test.ts

@@ -275,6 +275,15 @@ for (const makeTier of gitTiers) {
       })
     })
 
+    it("a mkdir + printf write lands one line per token, whatever the token holds", () => {
+      const lines = String.raw`mkdir -p .gtd/packages
+printf '%s\n' 'a' '' 'it'\''s' '100%' '-x' 'a\b' '.gtd/packages/00-x.md' > '.gtd/packages/00-x.md'`
+      expect(run(lines).ok).toBe(true)
+      expect(read(".gtd/packages/00-x.md")).toBe(
+        "a\n\nit's\n100%\n-x\na\\b\n.gtd/packages/00-x.md\n",
+      )
+    })
+
     describe("commitAll", () => {
       it("a staging hook's edit lands in the commit", () => {
         t.seed.stagingHook("readme.txt", "hooked")

package/src/workflows/feature/index.ts

@@ -3,6 +3,7 @@ import { feature } from "./feature.js"
 export * from "./steps.js"
 export * from "./planning.js"
 export * from "./packages.js"
+export * from "./refactor.js"
 export { buildTail, type ReviewOutcome } from "./review.js"
 export * from "./feature.js"
 export * from "../shared/reserved.js"