@pmelab/gtd 23.2.0 → 24.0.0

package/.claude-plugin/plugin.json

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

package/README.md

@@ -501,17 +501,24 @@ the agent may overturn the answer. See
 [Configuration](https://github.com/pmelab/gtd/blob/main/docs/configuration.md)
 for its `ui:` settings.
 
-The bundled workflows are named `feature` (the ordinary start above), `fix` and
-`review`. Two doors start the other two and skip step 1. `gtd door fix` repairs
-a red baseline as its own reviewed commit instead of starting a process. And
+The bundled workflows are named `feature` (the ordinary start above), `review`
+and `fix`. They are independent: `review` and `fix` never plan, and they meet
+only in `.gtd/REVIEW.md`. Each brings its own door. This command:
 
 ```bash
 gtd door review <commitish>
 ```
 
-starts a pure review of everything from `<commitish>` to HEAD — straight to step
-4, no planning and no building. `gtd doors` lists every door; a `gtd.config.ts`
-can add its own. `gtd --workflow <name>` starts any workflow by name.
+starts a pure review of everything from `<commitish>` to HEAD. It goes straight
+to step 4, with no planning and no building. Sign off and it closes. Leave notes
+and the process ends with `.gtd/REVIEW.md` (and the reviewer's original,
+`.gtd/REVIEW.orig.md`) left for `gtd door fix`. Until `fix` consumes them,
+starting `feature` or `review` is refused; remove both files to abandon the
+hand-off. `gtd door fix` fixes those notes and every `Risk:` in one turn, then
+asks for your review of the fix. With no review waiting, it repairs a red
+baseline as its own reviewed commit instead. `gtd doors` lists the doors of the
+workflows your `gtd.config.ts` exports, plus any it adds.
+`gtd --workflow <name>` starts any workflow by name.
 
 The workflow itself is a plain async TypeScript function, and the
 `gtd.config.ts` at the repository root names it: the one `gtd install` writes

package/dist/gtd.bundle.mjs

@@ -32061,8 +32061,9 @@ its next human gate, not just a starter: \`gtd-door review\` on a RED baseline
 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 (the bundled \`fix\` and
-\`review [base]\`, plus any its config declares). Refuses with
+\`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
 \`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\`.
 
@@ -32171,7 +32172,8 @@ const renderBriefing = () => HEADER() + BEAT_PROTOCOL + JSON_FIELD_REFERENCE + D
 //#region src/install/ConfigScaffold.ts
 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.
+// default export is what a change in the tree starts. Each workflow brings
+// its own doors: \`gtd door review [base]\` and \`gtd door fix\`.
 export {
   feature as default,
   feature,
@@ -32180,11 +32182,9 @@ export {
   defaults,
   envDefaults,
   summary,
-  base,
   steering,
   skills,
   access,
-  doors,
 } from "@pmelab/gtd/workflows"
 `;
 var FlowsProtocolError = class extends Error {
@@ -49820,7 +49820,7 @@ const load = gen(function* () {
 	const define = (entry) => ({
 		flow: entry.flow,
 		summary: entry.file.summary,
-		base: entry.file.base,
+		base: (w, v) => entry.file.base?.(w, v) ?? entry.base?.(w, v),
 		steering,
 		modes: compiled.modes,
 		skills: entry.file.skills,
@@ -50019,6 +50019,25 @@ const doorsFrom = (exports) => {
 	}
 	return new Map(Object.entries(value));
 };
+/**
+* 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.
+*/
+const derivedDoors = (catalogue, declared) => {
+	const doors = /* @__PURE__ */ new Map();
+	for (const [workflow, loaded] of catalogue) for (const [name, spec] of Object.entries(loaded.doors ?? {})) {
+		const door = {
+			...spec,
+			workflow
+		};
+		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);
+	}
+	return new Map([...doors, ...declared]);
+};
 /** 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)));
@@ -50055,7 +50074,7 @@ const loadWorkflows = (module, shipped) => {
 	return {
 		file,
 		catalogue,
-		doors: rejections.length > 0 ? /* @__PURE__ */ new Map() : checkDoors(doorsFrom(exports), catalogue),
+		doors: rejections.length > 0 ? /* @__PURE__ */ new Map() : checkDoors(derivedDoors(catalogue, doorsFrom(exports)), catalogue),
 		default: defaultWorkflow && {
 			...defaultWorkflow,
 			name: defaultName
@@ -50087,7 +50106,9 @@ const buildWorkflow = (name, value, file, rejections) => {
 	return {
 		flow: result.build.flow,
 		file,
-		...builtSettings(name, file, result.build)
+		...builtSettings(name, file, result.build),
+		doors: result.build.doors,
+		base: result.build.base
 	};
 };
 /** A factory's settings over its file's, held to the rules a file's own exports are. */
@@ -70081,10 +70102,11 @@ const COMMAND_ROWS = [
 			"Start a process through a named shortcut, mapping the",
 			"positional args to that workflow's process settings — the",
 			"same start `gtd --workflow` makes, printed the same way.",
-			"Doors come from the config's `doors` export; gtd's shipped",
-			"ones are `fix` and `review [base]`. An unknown door, or too",
-			"few or too many args, is a usage error (exit 2) naming the",
-			"doors or the door's synopsis"
+			"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"
 		]
 	},
 	{