@pmelab/gtd 27.9.0 → 28.0.0

package/dist/gtd.bundle.mjs

@@ -21520,8 +21520,11 @@ hands off to \`gtd-build\`, which halts at the blocked gate and prints it,
 rather than starting the review; \`gtd-door fix\` on a GREEN suite is a no-op
 straight back to \`idle\` — the exec'd \`gtd-build\` exits immediately on it.
 \`gtd doors\` lists the doors a repository offers (those of the workflows its
-config exports — the bundled \`fix\` and \`review [base]\` — plus any it
-declares). Refuses with
+config exports — the bundled \`fix\` and \`review [base]\`, plus the opt-in
+\`rollout <flag>\` (turn a release flag default-on) and \`cleanup <flag>\`
+(remove it), which a config adds by exporting \`rollout\` from
+\`@pmelab/gtd/workflows/rollout\` and \`cleanup\` from
+\`@pmelab/gtd/workflows/cleanup\` — plus any it declares). Refuses with
 \`usage: gtd-door <name> [args...]\` on stderr and exit \`2\` when the name is
 missing — nothing was even attempted. Default path: \`~/.local/bin/gtd-door\`.
 
@@ -21631,7 +21634,10 @@ const renderBriefing = () => HEADER() + BEAT_PROTOCOL + JSON_FIELD_REFERENCE + D
 const CONFIG_SCAFFOLD = `// The workflows gtd runs here: gtd's shipped feature, review and fix.
 // Every exported function is startable with \`gtd --workflow <name>\`; the
 // default export is what a change in the tree starts. Each workflow brings
-// its own doors: \`gtd door review [base]\` and \`gtd door fix\`.
+// its own doors: \`gtd door review [base]\` and \`gtd door fix\`. Export \`rollout\`
+// from \`@pmelab/gtd/workflows/rollout\` and \`cleanup\` from
+// \`@pmelab/gtd/workflows/cleanup\` to add \`gtd door rollout <flag>\` and
+// \`gtd door cleanup <flag>\`.
 export {
   feature as default,
   feature,
@@ -38993,10 +38999,10 @@ const doorsFrom = (exports) => {
 };
 /**
 * Each exported workflow's own doors, onto the name it is exported under. The
-* `doors` export adds to them and settles a name two workflows both declare.
+* `doors` export adds to them and declares a name several workflows clash on.
 */
 const derivedDoors = (catalogue, declared) => {
-	const doors = /* @__PURE__ */ new Map();
+	const builders = /* @__PURE__ */ new Map();
 	for (const [workflow, loaded] of catalogue) for (const [name, spec] of Object.entries(loaded.doors ?? {})) {
 		const door = {
 			...spec,
@@ -39004,12 +39010,22 @@ const derivedDoors = (catalogue, declared) => {
 		};
 		const problem = doorProblem(name, door);
 		if (problem !== void 0) throw new Error(`workflow "${workflow}" builds door "${name}", which ${problem}`);
-		const other = doors.get(name);
-		if (other !== void 0 && !declared.has(name)) throw new Error(`door "${name}" is built by workflows "${other.workflow}" and "${workflow}" — declare it in the "doors" export to choose one (a declared door replaces the derived one, so restate its args and vars)`);
-		doors.set(name, door);
+		builders.set(name, [...builders.get(name) ?? [], door]);
+	}
+	const doors = /* @__PURE__ */ new Map();
+	const clashes = /* @__PURE__ */ new Map();
+	for (const [name, built] of builders) {
+		const owner = ownerOf(name, built);
+		if (owner !== void 0) doors.set(name, owner);
+		else if (!declared.has(name)) clashes.set(name, built.map((door) => door.workflow));
 	}
-	return new Map([...doors, ...declared]);
+	return {
+		doors: new Map([...doors, ...declared]),
+		clashes
+	};
 };
+/** The export named after the door owns it, else the sole builder; a clash of several builds no door. */
+const ownerOf = (name, built) => built.find((door) => door.workflow === name) ?? (built.length === 1 ? built[0] : void 0);
 /** Sort the doors and check each names a startable workflow. */
 const checkDoors = (declared, catalogue) => {
 	const doors = new Map([...declared].sort(([a], [b]) => a.localeCompare(b)));
@@ -39040,16 +39056,28 @@ const loadWorkflows = (module, shipped) => {
 		return workflow === void 0 ? [] : [[name, workflow]];
 	}));
 	const defaultWorkflow = built.get(value);
+	const derived = rejections.length > 0 ? {
+		doors: /* @__PURE__ */ new Map(),
+		clashes: /* @__PURE__ */ new Map()
+	} : derivedDoors(catalogue, doorsFrom(exports));
 	return {
 		file,
 		catalogue,
-		doors: rejections.length > 0 ? /* @__PURE__ */ new Map() : checkDoors(derivedDoors(catalogue, doorsFrom(exports)), catalogue),
+		doors: checkDoors(derived.doors, catalogue),
 		default: defaultWorkflow && {
 			...defaultWorkflow,
 			name: defaultName
 		},
 		config: compileConfig(exports["config"], module.filepath),
-		diagnostics: nameRejections(rejections)
+		diagnostics: [...nameRejections(rejections), ...[...derived.clashes].map(([name, builders]) => {
+			const names = [...builders].sort().map((b) => JSON.stringify(b));
+			return {
+				severity: "warning",
+				message: `door "${name}" is built by workflows ${names.slice(0, -1).join(", ")} and ${names.at(-1)} and none is exported under that name — declare it in the "doors" export`,
+				path: ["doors", name],
+				origin: file.origin
+			};
+		})]
 	};
 };
 const evaluateFromDisk = (module, shipped) => {
@@ -42186,10 +42214,10 @@ const COMMAND_ROWS = [
 			"positional args to that workflow's process settings — the",
 			"same start `gtd --workflow` makes, printed the same way.",
 			"Each workflow the config exports brings its own doors (gtd's",
-			"`fix` brings `fix`, `review` brings `review [base]`), plus",
-			"the config's `doors` export. An unknown door, or too few or",
-			"too many args, is a usage error (exit 2) naming the doors or",
-			"the door's synopsis"
+			"`fix`, `review [base]`, `rollout <flag>` and",
+			"`cleanup <flag>`), plus the config's `doors` export. An",
+			"unknown door, or too few or too many args, is a usage error",
+			"(exit 2) naming the doors or the door's synopsis"
 		]
 	},
 	{

package/dist/testkit.mjs

@@ -6647,8 +6647,11 @@ hands off to \`gtd-build\`, which halts at the blocked gate and prints it,
 rather than starting the review; \`gtd-door fix\` on a GREEN suite is a no-op
 straight back to \`idle\` — the exec'd \`gtd-build\` exits immediately on it.
 \`gtd doors\` lists the doors a repository offers (those of the workflows its
-config exports — the bundled \`fix\` and \`review [base]\` — plus any it
-declares). Refuses with
+config exports — the bundled \`fix\` and \`review [base]\`, plus the opt-in
+\`rollout <flag>\` (turn a release flag default-on) and \`cleanup <flag>\`
+(remove it), which a config adds by exporting \`rollout\` from
+\`@pmelab/gtd/workflows/rollout\` and \`cleanup\` from
+\`@pmelab/gtd/workflows/cleanup\` — plus any it declares). Refuses with
 \`usage: gtd-door <name> [args...]\` on stderr and exit \`2\` when the name is
 missing — nothing was even attempted. Default path: \`~/.local/bin/gtd-door\`.
 
@@ -6758,7 +6761,10 @@ const renderBriefing = () => HEADER() + BEAT_PROTOCOL + JSON_FIELD_REFERENCE + D
 const CONFIG_SCAFFOLD = `// The workflows gtd runs here: gtd's shipped feature, review and fix.
 // Every exported function is startable with \`gtd --workflow <name>\`; the
 // default export is what a change in the tree starts. Each workflow brings
-// its own doors: \`gtd door review [base]\` and \`gtd door fix\`.
+// its own doors: \`gtd door review [base]\` and \`gtd door fix\`. Export \`rollout\`
+// from \`@pmelab/gtd/workflows/rollout\` and \`cleanup\` from
+// \`@pmelab/gtd/workflows/cleanup\` to add \`gtd door rollout <flag>\` and
+// \`gtd door cleanup <flag>\`.
 export {
   feature as default,
   feature,
@@ -8104,10 +8110,10 @@ const doorsFrom = (exports) => {
 };
 /**
 * Each exported workflow's own doors, onto the name it is exported under. The
-* `doors` export adds to them and settles a name two workflows both declare.
+* `doors` export adds to them and declares a name several workflows clash on.
 */
 const derivedDoors = (catalogue, declared) => {
-	const doors = /* @__PURE__ */ new Map();
+	const builders = /* @__PURE__ */ new Map();
 	for (const [workflow, loaded] of catalogue) for (const [name, spec] of Object.entries(loaded.doors ?? {})) {
 		const door = {
 			...spec,
@@ -8115,12 +8121,22 @@ const derivedDoors = (catalogue, declared) => {
 		};
 		const problem = doorProblem(name, door);
 		if (problem !== void 0) throw new Error(`workflow "${workflow}" builds door "${name}", which ${problem}`);
-		const other = doors.get(name);
-		if (other !== void 0 && !declared.has(name)) throw new Error(`door "${name}" is built by workflows "${other.workflow}" and "${workflow}" — declare it in the "doors" export to choose one (a declared door replaces the derived one, so restate its args and vars)`);
-		doors.set(name, door);
+		builders.set(name, [...builders.get(name) ?? [], door]);
+	}
+	const doors = /* @__PURE__ */ new Map();
+	const clashes = /* @__PURE__ */ new Map();
+	for (const [name, built] of builders) {
+		const owner = ownerOf(name, built);
+		if (owner !== void 0) doors.set(name, owner);
+		else if (!declared.has(name)) clashes.set(name, built.map((door) => door.workflow));
 	}
-	return new Map([...doors, ...declared]);
+	return {
+		doors: new Map([...doors, ...declared]),
+		clashes
+	};
 };
+/** The export named after the door owns it, else the sole builder; a clash of several builds no door. */
+const ownerOf = (name, built) => built.find((door) => door.workflow === name) ?? (built.length === 1 ? built[0] : void 0);
 /** Sort the doors and check each names a startable workflow. */
 const checkDoors = (declared, catalogue) => {
 	const doors = new Map([...declared].sort(([a], [b]) => a.localeCompare(b)));
@@ -8151,16 +8167,28 @@ const loadWorkflows = (module, shipped) => {
 		return workflow === void 0 ? [] : [[name, workflow]];
 	}));
 	const defaultWorkflow = built.get(value);
+	const derived = rejections.length > 0 ? {
+		doors: /* @__PURE__ */ new Map(),
+		clashes: /* @__PURE__ */ new Map()
+	} : derivedDoors(catalogue, doorsFrom(exports));
 	return {
 		file,
 		catalogue,
-		doors: rejections.length > 0 ? /* @__PURE__ */ new Map() : checkDoors(derivedDoors(catalogue, doorsFrom(exports)), catalogue),
+		doors: checkDoors(derived.doors, catalogue),
 		default: defaultWorkflow && {
 			...defaultWorkflow,
 			name: defaultName
 		},
 		config: compileConfig(exports["config"], module.filepath),
-		diagnostics: nameRejections(rejections)
+		diagnostics: [...nameRejections(rejections), ...[...derived.clashes].map(([name, builders]) => {
+			const names = [...builders].sort().map((b) => JSON.stringify(b));
+			return {
+				severity: "warning",
+				message: `door "${name}" is built by workflows ${names.slice(0, -1).join(", ")} and ${names.at(-1)} and none is exported under that name — declare it in the "doors" export`,
+				path: ["doors", name],
+				origin: file.origin
+			};
+		})]
 	};
 };
 const evaluateFromDisk = (module, shipped) => {
@@ -11284,10 +11312,10 @@ const COMMAND_ROWS = [
 			"positional args to that workflow's process settings — the",
 			"same start `gtd --workflow` makes, printed the same way.",
 			"Each workflow the config exports brings its own doors (gtd's",
-			"`fix` brings `fix`, `review` brings `review [base]`), plus",
-			"the config's `doors` export. An unknown door, or too few or",
-			"too many args, is a usage error (exit 2) naming the doors or",
-			"the door's synopsis"
+			"`fix`, `review [base]`, `rollout <flag>` and",
+			"`cleanup <flag>`), plus the config's `doors` export. An",
+			"unknown door, or too few or too many args, is a usage error",
+			"(exit 2) naming the doors or the door's synopsis"
 		]
 	},
 	{

package/dist/workflows/cleanup.mjs

@@ -1,7 +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-UdSuHDUJ.mjs";
-import { i as refusePendingHandOff, r as handOffTail } from "../chunks/tail-fj60rQ3y.mjs";
-import { i as requireFlag, r as registryEntry, t as flagReferences } from "../chunks/registry-CP3YuV9j.mjs";
+import { a as flagShipped, ct as defaults, dt as flagReferences, i as skills, lt as envDefaults, mt as requireFlag, n as summary, pt as registryEntry, r as access, t as steering, v as coder } from "../chunks/reserved-BIsngMGL.mjs";
+import { i as refusePendingHandOff, r as handOffTail } from "../chunks/tail-DHEaOHyp.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.`;
 const cleanupFlow = async () => {
@@ -29,7 +28,7 @@ const cleanupFlow = async () => {
 	});
 	await handOffTail(false, start(), guard);
 };
-const cleanup = flagShipped(cleanupFlow);
+const cleanup = flagShipped(cleanupFlow, "cleanup");
 //#endregion
 //#region src/workflows/cleanup/index.ts
 var cleanup_default = cleanup;

package/dist/workflows/feature.mjs

@@ -1,7 +1,7 @@
 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 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-UdSuHDUJ.mjs";
-import { a as tail, i as refusePendingHandOff, n as closeReview } from "../chunks/tail-fj60rQ3y.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-BIsngMGL.mjs";
+import { a as tail, i as refusePendingHandOff, n as closeReview } from "../chunks/tail-DHEaOHyp.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";

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-UdSuHDUJ.mjs";
-import { r as handOffTail, t as REVIEW_ORIG } from "../chunks/tail-fj60rQ3y.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-BIsngMGL.mjs";
+import { r as handOffTail, t as REVIEW_ORIG } from "../chunks/tail-DHEaOHyp.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 { 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-UdSuHDUJ.mjs";
-import { i as refusePendingHandOff, r as handOffTail } from "../chunks/tail-fj60rQ3y.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-BIsngMGL.mjs";
+import { i as refusePendingHandOff, r as handOffTail } from "../chunks/tail-DHEaOHyp.mjs";
 //#region src/workflows/review/review.ts
 /**
 * Pure review of everything since `reviewBase`: a green gate, the quality

package/dist/workflows/rollout.mjs

@@ -1,9 +1,23 @@
-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-UdSuHDUJ.mjs";
-import { i as requireFlag, n as flipProblems } from "../chunks/registry-CP3YuV9j.mjs";
+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-BIsngMGL.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.";
+const MARKED_MENTIONS = 50;
+const listMentions = async (registry, name, guard) => {
+	await run("mentions", removeScript([FEEDBACK]), { label: "Listing flag mentions" });
+	guard();
+	const mentions = flagMentions(glob("**").filter((path) => path !== registry && path !== ".gtd" && !path.startsWith(".gtd/")).sort().map((path) => ({
+		path,
+		text: read(path) ?? ""
+	})), name);
+	mark("flag-mentions", String(mentions.length));
+	for (const m of mentions.slice(0, MARKED_MENTIONS)) try {
+		mark("flag-mention", `${m.path}:${m.line}`);
+	} catch {
+		continue;
+	}
+};
 const rolloutFlow = async () => {
 	const registry = vars.flagRegistry ?? "";
 	const name = vars.flag ?? "";
@@ -23,9 +37,10 @@ const rolloutFlow = async () => {
 		});
 		guard();
 		await gate("suite", suiteBlockedMessage(), guard);
+		await listMentions(registry, name, guard);
 	});
 };
-const rollout = flagShipped(rolloutFlow);
+const rollout = flagShipped(rolloutFlow, "rollout");
 //#endregion
 //#region src/workflows/rollout/index.ts
 var rollout_default = rollout;