@pmelab/gtd 52.0.0 → 52.1.0

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "gtd",
-  "version": "52.0.0",
+  "version": "52.1.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

@@ -441,9 +441,12 @@ noted in steps 3 and 4 below.
    entry is the agent's, moving on is refused. Leave the file untouched and
    start the loop to accept the plan as-is, unanswered questions and all. The
    file lists numbered acceptance criteria (`AC-1`, `AC-2`, …) that bind the
-   build. Sketch your own under an `Acceptance criteria` heading and, when a
+   build, and an `## Impact` section mapping the units and public surfaces the
+   change touches — correct it here and the next lap re-explores from your edit.
+   Sketch your own criteria under an `Acceptance criteria` heading and, when a
    judge finds every one clear and checkable, they are recorded as they are and
-   this moment is skipped; one vague criterion brings it back. See
+   this moment is skipped: a `design.impact` agent turn writes Impact instead,
+   unreviewed. One vague criterion brings the moment back. See
    [Acceptance criteria](docs/configuration.md#acceptance-criteria).
 
 3. **You wait.** The agent first implements the requirements the quickest way

package/dist/chunks/{reserved-D0MKVFFa.mjs → reserved-DxB5CROj.mjs}

@@ -774,6 +774,19 @@ writing \`(no hands-on)\` between its ID and the colon:
 Leave the tag off whatever a person can see by running the change. A
 tag the human added or removed at the gate stays as they left it.
 `;
+const impactBlock = `
+## Impact
+
+\`.gtd/REQUIREMENTS.md\` carries a \`## Impact\` section: explore the
+codebase (not just the start diff) and name the units and boundaries
+this change touches, the public surfaces it affects (CLI commands and
+flags, config keys, driver protocol, exported API), and anything
+surprising. Name the project's own declared units first —
+dependency-cruiser rules, workspace packages, module boundaries named
+in its docs or agent instructions (\`AGENTS.md\`, \`CLAUDE.md\`). When
+none is declared, fall back to directories and packages. Landing is
+refused while the section is missing.
+`;
 const riskTierBlock = `
 ## Risk tier and production signals
 
@@ -813,7 +826,14 @@ ${footnoteFoldIn}${codeThreadReplies()}
   or both
 - Whichever lap this is, leave \`.gtd/REQUIREMENTS.md\`
   uncommitted and finish once it reflects this lap's own work
-${acceptanceCriteriaBlock}
+${acceptanceCriteriaBlock}${impactBlock}
+Every lap re-explores and leaves \`## Impact\` present. On a return
+lap or review loop-back the human may have edited \`## Impact\` (or
+footnoted it): read that edit from history (\`git log -1 -p --
+.gtd/REQUIREMENTS.md\`), keep the human's correction, and
+re-explore from it. A surprise the human must settle becomes an
+entry under \`## Open Questions\`.
+
 ## First lap
 
 \`.gtd/REQUIREMENTS.md\` does not exist yet (or holds whatever the
@@ -868,6 +888,21 @@ ${questionBarReturn}
   made). If red, make that breakage the first concern, ahead of
   everything else — the build assumes the suite started green
 ${riskTier ? riskTierBlock : ""}`;
+const designImpactPrompt = () => `${styleBlock}
+
+${styleFormatContract}
+
+${stateFileRules}
+- The only state file this turn touches is \`.gtd/REQUIREMENTS.md\`
+  — no other files for notes or output
+- \`.gtd/REQUIREMENTS.md\` already holds the settled acceptance
+  criteria. Add a \`## Impact\` section and change nothing else — a
+  turn that touches anything outside it is refused
+- No human reviews this before the build: put any surprise in the
+  Impact prose, never as a question
+${impactBlock}
+- Leave \`.gtd/REQUIREMENTS.md\` uncommitted and finish once it holds \`## Impact\`
+`;
 const clearCriteriaQuestion = {
 	id: "clear",
 	primitive: "noul",
@@ -900,9 +935,9 @@ const slicingPrompt = (cap) => `${stateFileRules}
 - Read \`.gtd/REQUIREMENTS.md\`: its \`## Acceptance Criteria\` are settled
   and more than ${cap}, too many to build in one slice
 - Group them into slices in \`.gtd/SLICES.md\`, one \`## <title>\` heading per
-  slice over a \`- AC-<n>\` list item per criterion it builds. Put criteria of
-  the same flow or subsystem together; every criterion in exactly one slice,
-  at most ${cap} per slice
+  slice over a \`- AC-<n>\` list item per criterion it builds. Read the
+  \`## Impact\` section of \`.gtd/REQUIREMENTS.md\` and slice along the units it
+  names; every criterion in exactly one slice, at most ${cap} per slice
 - Keep it coarse: as few slices as the cap allows, never one per criterion
 - Order the slices so each builds on the ones before: they run in that order,
   each from where the previous one landed
@@ -1568,6 +1603,9 @@ const listed = (paths, unreadable = []) => paths.map((path) => `  - \`${displayP
 const isRed = (retry) => "red" in retry;
 const isFault = (retry) => "fault" in retry;
 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 impactHints = `- Its \`## Impact\` section maps where the change lands — reviewed at the design
+  gate when design ran. A starting point and hints, not a list of allowed files
+  — change whatever the work needs`;
 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"}`;
@@ -1614,6 +1652,7 @@ the context limit first if the refactor is too big`}. \`gtd abandon\` gives the
 /** `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
+${impactHints}
 ${sliceLine(slice)}${criteriaBinding(false)}
 - Implement them the quickest way that works — the naive version. Do not
   restructure, abstract or refactor around the change; it may land exactly as
@@ -1681,6 +1720,7 @@ const flagLine = (flagMode, tier, amend, sliced) => !flagMode ? "- Add no releas
   repository's flag conventions`;
 const implementBuildingPrompt = (reference, tier, amend, afterRefactor, flagMode = true, rebuild = {}, slice, from) => `${stateFileRules}
 - Read \`.gtd/REQUIREMENTS.md\`: the settled requirements for this change
+${impactHints}
 ${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
   the re-derived requirements instead of rewriting it
@@ -1816,6 +1856,7 @@ ${retest ? `- This is a re-test: \`.gtd/PROTOTYPE.md\` holds the last round's re
 `;
 const prototypeFixingPrompt = (slice) => `${stateFileRules}
 - Read \`.gtd/REQUIREMENTS.md\`: the settled requirements for this change
+${impactHints}
 ${sliceLine(slice)}${criteriaBinding(true)}
 - Read \`.gtd/PROTOTYPE.md\`: a hands-on test of the change. Fix every
   \`fail\` chunk of this change's criteria${slice === void 0 ? "" : " (this slice's only)"} or a problem found
@@ -2214,4 +2255,4 @@ const steering = {
 	[REVIEW]: "review"
 };
 //#endregion
-export { normaliseCriteriaScript as $, reviewMissing as A, unwindFailure as At, composedReview as B, conflictTitle as Bt, coder as C, seamSteerMessage as Ct, fixQuality as D, titledNotes as Dt, fixNits as E, testerSystem as Et, buildReviewTriageMessage as F, envDefaults as Ft, finisherSystem as G, standardKey as Gt, designGateAnswerMessage as H, oneLine as Ht, builderSystem as I, testFileGlobs as It, idleMessage as J, flipProblems as Jt, fixReviewNotesPrompt as K, flagMentions as Kt, chunkTexts as L, STANDARDS as Lt, reviewing as M, withLapFindings as Mt, agentWithSkills as N, withWalkthrough as Nt, fixRisks as O, unranked as Ot, buildReviewCollectingPrompt as P, defaults as Pt, naivePassPrompt as Q, clearCriteriaMessage as R, conflictBody as Rt, awaitReview as S, seamByHandMessage as St, fixCheck as T, startGateBlockedMessage as Tt, designSystem as U, parseStandards as Ut, criterionLine as V, isConflictTitle as Vt, designTriagePrompt as W, renderStandards as Wt, implementFixPrompt as X, requireFlag as Xt, implementBuildingPrompt as Y, registryEntry as Yt, naiveApplyFailure as Z, REFACTOR_NOTE as _, reviewFeedbackFixPrompt as _t, flagModeOn as a, prototypeTestingPrompt as at, SLICES as b, reviewRankingMessage as bt, baseline as c, qualityTriageMessage as ct, healthy as d, refactorDroppedRefusal as dt, observableMessage as et, FEEDBACK as f, refactorFixPrompt as ft, REFACTOR as g, reviewEditsCapture as gt, QUALITY as h, reviewEditNotesCapture as ht, skills as i, prototypeOpenMessage as it, reviewQuality as j, walkthroughBlock as jt, planner as k, unwindFailedMessage as kt, gate as l, rankedReview as lt, PROTOTYPE as m, refactorRules as mt, summary as n, oversizedMessage as nt, flagShipped as o, prototypeUnguidedMessage as ot, HANDOFF as p, refactorNotePrompt as pt, handOffAwaitReviewMessage as q, flagReferences as qt, access as r, prototypeFixingPrompt as rt, shipped as s, qualityFindingsFile as st, steering as t, observableQuestion as tt, guarded as u, refactorBuildingPrompt as ut, REQUIREMENTS as v, reviewGateBlockedMessage as vt, collecting as w, slicingPrompt as wt, answerReviewQuestions as x, reviewerSystem as xt, REVIEW as y, reviewNotesCapture as yt, clearCriteriaQuestion as z, conflictFor as zt };
+export { naivePassPrompt as $, reviewMissing as A, unwindFailedMessage as At, composedReview as B, conflictFor as Bt, coder as C, seamByHandMessage as Ct, fixQuality as D, testerSystem as Dt, fixNits as E, startGateBlockedMessage as Et, buildReviewTriageMessage as F, defaults as Ft, designTriagePrompt as G, renderStandards as Gt, designGateAnswerMessage as H, isConflictTitle as Ht, builderSystem as I, envDefaults as It, handOffAwaitReviewMessage as J, flagReferences as Jt, finisherSystem as K, standardKey as Kt, chunkTexts as L, testFileGlobs as Lt, reviewing as M, walkthroughBlock as Mt, agentWithSkills as N, withLapFindings as Nt, fixRisks as O, titledNotes as Ot, buildReviewCollectingPrompt as P, withWalkthrough as Pt, naiveApplyFailure as Q, clearCriteriaMessage as R, STANDARDS as Rt, awaitReview as S, reviewerSystem as St, fixCheck as T, slicingPrompt as Tt, designImpactPrompt as U, oneLine as Ut, criterionLine as V, conflictTitle as Vt, designSystem as W, parseStandards as Wt, implementBuildingPrompt as X, registryEntry as Xt, idleMessage as Y, flipProblems as Yt, implementFixPrompt as Z, requireFlag as Zt, REFACTOR_NOTE as _, reviewEditsCapture as _t, flagModeOn as a, prototypeOpenMessage as at, SLICES as b, reviewNotesCapture as bt, baseline as c, qualityFindingsFile as ct, healthy as d, refactorBuildingPrompt as dt, normaliseCriteriaScript as et, FEEDBACK as f, refactorDroppedRefusal as ft, REFACTOR as g, reviewEditNotesCapture as gt, QUALITY as h, refactorRules as ht, skills as i, prototypeFixingPrompt as it, reviewQuality as j, unwindFailure as jt, planner as k, unranked as kt, gate as l, qualityTriageMessage as lt, PROTOTYPE as m, refactorNotePrompt as mt, summary as n, observableQuestion as nt, flagShipped as o, prototypeTestingPrompt as ot, HANDOFF as p, refactorFixPrompt as pt, fixReviewNotesPrompt as q, flagMentions as qt, access as r, oversizedMessage as rt, shipped as s, prototypeUnguidedMessage as st, steering as t, observableMessage as tt, guarded as u, rankedReview as ut, REQUIREMENTS as v, reviewFeedbackFixPrompt as vt, collecting as w, seamSteerMessage as wt, answerReviewQuestions as x, reviewRankingMessage as xt, REVIEW as y, reviewGateBlockedMessage as yt, clearCriteriaQuestion as z, conflictBody as zt };

package/dist/chunks/{review-ZzUQH1KZ.mjs → review-CKMSdvgW.mjs}

@@ -1,8 +1,8 @@
 import { answered, changesSince, head, judge, numeric, read, refuse, requireRevert, restoreScript, run, showScript, vars, writeScript } from "../flows.mjs";
 import { b as sectionedReview, h as reviewChunks, m as REVIEW_SECTIONS, v as reviewRisks, y as reviewSections } from "./steering-CLhu8C7q.mjs";
 import { removedAssertions, review } from "../workflow-kit.mjs";
-import { A as reviewMissing, B as composedReview, Bt as conflictTitle, Dt as titledNotes, E as fixNits, F as buildReviewTriageMessage, Gt as standardKey, Ht as oneLine, L as chunkTexts, Lt as STANDARDS, M as reviewing, Nt as withWalkthrough, O as fixRisks, Ot as unranked, Rt as conflictBody, S as awaitReview, T as fixCheck, Ut as parseStandards, Vt as isConflictTitle, Wt as renderStandards, bt as reviewRankingMessage, d as healthy, gt as reviewEditsCapture, ht as reviewEditNotesCapture, jt as walkthroughBlock, lt as rankedReview, u as guarded, w as collecting, x as answerReviewQuestions, y as REVIEW, yt as reviewNotesCapture, zt as conflictFor } from "./reserved-D0MKVFFa.mjs";
-import { i as isOptionalObject, n as closeReview, o as tail, s as NO_FINDINGS } from "./tail-I1-vVq0P.mjs";
+import { A as reviewMissing, B as composedReview, Bt as conflictFor, E as fixNits, F as buildReviewTriageMessage, Gt as renderStandards, Ht as isConflictTitle, Kt as standardKey, L as chunkTexts, M as reviewing, Mt as walkthroughBlock, O as fixRisks, Ot as titledNotes, Pt as withWalkthrough, Rt as STANDARDS, S as awaitReview, T as fixCheck, Ut as oneLine, Vt as conflictTitle, Wt as parseStandards, _t as reviewEditsCapture, bt as reviewNotesCapture, d as healthy, gt as reviewEditNotesCapture, kt as unranked, u as guarded, ut as rankedReview, w as collecting, x as answerReviewQuestions, xt as reviewRankingMessage, y as REVIEW, zt as conflictBody } from "./reserved-DxB5CROj.mjs";
+import { i as isOptionalObject, n as closeReview, o as tail, s as NO_FINDINGS } from "./tail-QZc7Uz2U.mjs";
 //#region src/workflows/shared/learn.ts
 const SUMMARY_MAX = 200;
 const BANNER = /^_(?:The fixer left this unresolved\.|A fix turn removed this while renaming another finding — check whether it was fixed\.|No detail given\.)_\s*/;

package/dist/chunks/{tail-I1-vVq0P.mjs → tail-QZc7Uz2U.mjs}

@@ -1,6 +1,6 @@
 import { answered, head, judge, mark, numeric, read, refuse, removeScript, run, scope, sectionBodies, showScript, vars, writeScript } from "../flows.mjs";
 import { lensEntriesOf, lensOf, qualityLap, reviewToHandOff } from "../workflow-kit.mjs";
-import { A as reviewMissing, D as fixQuality, M as reviewing, Mt as withLapFindings, O as fixRisks, S as awaitReview, T as fixCheck, ct as qualityTriageMessage, d as healthy, h as QUALITY, j as reviewQuality, q as handOffAwaitReviewMessage, st as qualityFindingsFile, u as guarded, y as REVIEW } from "./reserved-D0MKVFFa.mjs";
+import { A as reviewMissing, D as fixQuality, J as handOffAwaitReviewMessage, M as reviewing, Nt as withLapFindings, O as fixRisks, S as awaitReview, T as fixCheck, ct as qualityFindingsFile, d as healthy, h as QUALITY, j as reviewQuality, lt as qualityTriageMessage, u as guarded, y as REVIEW } from "./reserved-DxB5CROj.mjs";
 //#region src/workflows/shared/lenses.ts
 const NO_FINDINGS = {
 	notes: [],

package/dist/workflows/cleanup.mjs

@@ -1,6 +1,6 @@
 import { agent, changes, glob, read, refuse, scope, start, vars } from "../flows.mjs";
-import { C as coder, Ft as envDefaults, Pt as defaults, Xt as requireFlag, Yt as registryEntry, i as skills, n as summary, o as flagShipped, qt as flagReferences, r as access, t as steering } from "../chunks/reserved-D0MKVFFa.mjs";
-import { a as refusePendingHandOff, r as handOffTail } from "../chunks/tail-I1-vVq0P.mjs";
+import { C as coder, Ft as defaults, It as envDefaults, Jt as flagReferences, Xt as registryEntry, Zt as requireFlag, i as skills, n as summary, o as flagShipped, r as access, t as steering } from "../chunks/reserved-DxB5CROj.mjs";
+import { a as refusePendingHandOff, r as handOffTail } from "../chunks/tail-QZc7Uz2U.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 () => {

package/dist/workflows/feature.mjs

@@ -1,8 +1,8 @@
 import { agent, answered, applyScript, changes, changesSince, check, codeChanges, env, hasThreadFor, head, human, judge, judgeDiffShape, kept, mark, matches, quote, read, refuse, removeScript, requireAnswers, requireProgress, requireReplies, requireThreadsClosed, revertScript, run, scope, slice, start, vars, wrote } from "../flows.mjs";
 import { lensEntriesOf } from "../workflow-kit.mjs";
-import { $ as normaliseCriteriaScript, At as unwindFailure, C as coder, Ct as seamSteerMessage, E as fixNits, Et as testerSystem, Ft as envDefaults, H as designGateAnswerMessage, I as builderSystem, It as testFileGlobs, J as idleMessage, N as agentWithSkills, P as buildReviewCollectingPrompt, Pt as defaults, Q as naivePassPrompt, R as clearCriteriaMessage, St as seamByHandMessage, Tt as startGateBlockedMessage, U as designSystem, V as criterionLine, W as designTriagePrompt, X as implementFixPrompt, Y as implementBuildingPrompt, Z as naiveApplyFailure, _ as REFACTOR_NOTE, a as flagModeOn, at as prototypeTestingPrompt, b as SLICES, d as healthy, dt as refactorDroppedRefusal, et as observableMessage, f as FEEDBACK, ft as refactorFixPrompt, g as REFACTOR, i as skills, it as prototypeOpenMessage, k as planner, kt as unwindFailedMessage, l as gate, m as PROTOTYPE, mt as refactorRules, n as summary, nt as oversizedMessage, ot as prototypeUnguidedMessage, p as HANDOFF, pt as refactorNotePrompt, r as access, rt as prototypeFixingPrompt, s as shipped, t as steering, tt as observableQuestion, ut as refactorBuildingPrompt, v as REQUIREMENTS, w as collecting, wt as slicingPrompt, x as answerReviewQuestions, xt as reviewerSystem, z as clearCriteriaQuestion } from "../chunks/reserved-D0MKVFFa.mjs";
-import { a as refusePendingHandOff, c as joinReports, o as tail } from "../chunks/tail-I1-vVq0P.mjs";
-import { i as reUnwind, n as humanReview, r as isCode$1, t as buildTail } from "../chunks/review-ZzUQH1KZ.mjs";
+import { $ as naivePassPrompt, At as unwindFailedMessage, C as coder, Ct as seamByHandMessage, Dt as testerSystem, E as fixNits, Et as startGateBlockedMessage, Ft as defaults, G as designTriagePrompt, H as designGateAnswerMessage, I as builderSystem, It as envDefaults, Lt as testFileGlobs, N as agentWithSkills, P as buildReviewCollectingPrompt, Q as naiveApplyFailure, R as clearCriteriaMessage, St as reviewerSystem, Tt as slicingPrompt, U as designImpactPrompt, V as criterionLine, W as designSystem, X as implementBuildingPrompt, Y as idleMessage, Z as implementFixPrompt, _ as REFACTOR_NOTE, a as flagModeOn, at as prototypeOpenMessage, b as SLICES, d as healthy, dt as refactorBuildingPrompt, et as normaliseCriteriaScript, f as FEEDBACK, ft as refactorDroppedRefusal, g as REFACTOR, ht as refactorRules, i as skills, it as prototypeFixingPrompt, jt as unwindFailure, k as planner, l as gate, m as PROTOTYPE, mt as refactorNotePrompt, n as summary, nt as observableQuestion, ot as prototypeTestingPrompt, p as HANDOFF, pt as refactorFixPrompt, r as access, rt as oversizedMessage, s as shipped, st as prototypeUnguidedMessage, t as steering, tt as observableMessage, v as REQUIREMENTS, w as collecting, wt as seamSteerMessage, x as answerReviewQuestions, z as clearCriteriaQuestion } from "../chunks/reserved-DxB5CROj.mjs";
+import { a as refusePendingHandOff, c as joinReports, o as tail } from "../chunks/tail-QZc7Uz2U.mjs";
+import { i as reUnwind, n as humanReview, r as isCode$1, t as buildTail } from "../chunks/review-CKMSdvgW.mjs";
 import { posix } from "node:path";
 //#region src/workflows/shared/fences.ts
 const markFenced = (text) => {
@@ -121,6 +121,38 @@ const requireCriteria = (file) => {
 	if (criteria.length > 0 && unnamed.length === 0 && twice.length === 0) return;
 	refuse(`gtd land: acceptance-criteria: ${file} has ${criteria.length === 0 ? `no "## Acceptance Criteria" list` : unnamed.length > 0 ? `${unnamed.length} criterion without an ID:\n${unnamed.map((c) => `  - ${c.text}`).join("\n")}` : `duplicate IDs: ${[...new Set(twice)].join(", ")}`} — list every criterion under "## Acceptance Criteria" as "- AC-<n>: <checkable criterion>", each ID unique and never reused`);
 };
+const isImpactHeading = (h) => h !== null && h[1].length === 2 && /^impact$/i.test(h[2].trim());
+/** Whether `text` has an unfenced, level-2 `## Impact` heading. */
+const hasImpact = (text) => markFenced(text).some((l) => !l.fenced && isImpactHeading(HEADING.exec(l.line)));
+/** `text` without its `## Impact` section, whitespace collapsed. */
+const withoutImpact = (text) => {
+	let inImpact = false;
+	return markFenced(text).filter((l) => {
+		const h = l.fenced ? null : HEADING.exec(l.line);
+		if (h !== null && h[1].length <= 2) inImpact = isImpactHeading(h);
+		return !inImpact;
+	}).map((l) => l.line).join(" ").replace(/\s+/g, " ").trim();
+};
+/**
+* Refuse a turn that leaves `file` without a `## Impact` section. Only the
+* pending turn (`head()` is `""`): a triage lap recorded before Impact existed
+* replays without one, and refusing there would fail every in-flight process.
+* A human's edit at a later gate is not checked — the human owns that file then.
+*/
+const requireImpact = (file) => {
+	if (head() !== "") return;
+	const text = read(file);
+	if (text === void 0) return;
+	if (hasImpact(text)) return;
+	refuse(`gtd land: impact: ${file} has no "## Impact" section — explore the codebase and add "## Impact" naming the units, public surfaces and surprises this change touches`);
+};
+/** Refuse a turn that changed `file` beyond its `## Impact` section. */
+const requireImpactOnly = (file) => {
+	const change = changes().get(file);
+	if (change === void 0) return;
+	if (withoutImpact(change.before ?? "") === withoutImpact(change.after ?? "")) return;
+	refuse(`gtd land: impact-only: this turn may only add "## Impact" to ${file} — restore everything else`);
+};
 const idNumber = (id) => Number(id.slice(3));
 /** Why `listed` renumbers or reuses an earlier ID; `undefined` when it does neither. */
 const unstableIds = (listed, ids) => {
@@ -154,6 +186,19 @@ const clear = async (stated) => {
 	});
 	return truncated.length === 0 && answered(answers.clear, "yes", CLEAR_MIN_P);
 };
+/** The fast path's impact-only turn: add `## Impact`, leave the criteria alone. */
+const impact = async () => {
+	await agentWithSkills("impact", designImpactPrompt(), {
+		label: "Mapping the change's impact",
+		file: REQUIREMENTS,
+		model: planner(),
+		system: designSystem()
+	});
+	requireProgress(REQUIREMENTS);
+	requireCriteriaFrozen(REQUIREMENTS);
+	requireImpact(REQUIREMENTS);
+	requireImpactOnly(REQUIREMENTS);
+};
 /**
 * The skip path: when the sketch stated clear, checkable criteria, write them
 * to `.gtd/REQUIREMENTS.md` as they are and resolve `true` — no triage, no
@@ -165,6 +210,7 @@ const skipDesign = (stated) => stated === void 0 ? Promise.resolve(false) : scop
 		label: "Recording the stated acceptance criteria",
 		file: REQUIREMENTS
 	});
+	await impact();
 	return true;
 });
 /** After a gate that invites narrowing the requirements: an edit must leave valid criteria. */
@@ -261,6 +307,7 @@ const design = (base, guard) => scope("design", async () => {
 		requireProgress(REQUIREMENTS);
 		requireReplies(REQUIREMENTS);
 		requireCriteria(REQUIREMENTS);
+		requireImpact(REQUIREMENTS);
 		guard?.();
 	} while (await noteGate(REQUIREMENTS, answerProductQuestions));
 });
@@ -1345,4 +1392,4 @@ const feature = shipped(ordinaryStart);
 //#region src/workflows/feature/index.ts
 var feature_default = feature;
 //#endregion
-export { COMMENT, REFACTOR_PACKAGE, acceptanceFault, access, afterTail, answerProductQuestions, answerReviewQuestions, buildFeature, buildTail, closeOut, collecting, criteriaOf, criterionIds, feature_default as default, defaults, design, envDefaults, feature, fixNits, handsOnIds, holdsCallSites, implement, implementing, isTestFile, judgeSeam, naiveAcceptance, naivePass, ordinaryStart, planAndBuild, reUnwind, refactorCollecting, refactorFirst, refactorNote, refactorStrays, requireAcceptance, requireAcceptanceKept, requireCriteria, requireCriteriaFrozen, requireCriteriaIfEdited, requireNoNewFlags, requireNoTestFiles, requireSlices, requireStableIds, requireTestCallSitesOnly, requireWithin, reviewedRefactor, seamBuild, skills, skipDesign, sliceCap, sliceCriteria, sliceIds, slicesOf, slicing, statedCriteria, steering, summary, triage, unstableIds, unwind };
+export { COMMENT, REFACTOR_PACKAGE, acceptanceFault, access, afterTail, answerProductQuestions, answerReviewQuestions, buildFeature, buildTail, closeOut, collecting, criteriaOf, criterionIds, feature_default as default, defaults, design, envDefaults, feature, fixNits, handsOnIds, hasImpact, holdsCallSites, implement, implementing, isTestFile, judgeSeam, naiveAcceptance, naivePass, ordinaryStart, planAndBuild, reUnwind, refactorCollecting, refactorFirst, refactorNote, refactorStrays, requireAcceptance, requireAcceptanceKept, requireCriteria, requireCriteriaFrozen, requireCriteriaIfEdited, requireImpact, requireImpactOnly, requireNoNewFlags, requireNoTestFiles, requireSlices, requireStableIds, requireTestCallSitesOnly, requireWithin, reviewedRefactor, seamBuild, skills, skipDesign, sliceCap, sliceCriteria, sliceIds, slicesOf, slicing, statedCriteria, steering, summary, triage, unstableIds, unwind };

package/dist/workflows/fix.mjs

@@ -1,7 +1,7 @@
 import { read, refuse, removeScript, run, scope, start } from "../flows.mjs";
 import { _ as reviewNotes, v as reviewRisks } from "../chunks/steering-CLhu8C7q.mjs";
-import { C as coder, Ft as envDefaults, G as finisherSystem, K as fixReviewNotesPrompt, N as agentWithSkills, Pt as defaults, c as baseline, i as skills, n as summary, r as access, s as shipped, t as steering, y as REVIEW } from "../chunks/reserved-D0MKVFFa.mjs";
-import { r as handOffTail, t as REVIEW_ORIG } from "../chunks/tail-I1-vVq0P.mjs";
+import { C as coder, Ft as defaults, It as envDefaults, K as finisherSystem, N as agentWithSkills, c as baseline, i as skills, n as summary, q as fixReviewNotesPrompt, r as access, s as shipped, t as steering, y as REVIEW } from "../chunks/reserved-DxB5CROj.mjs";
+import { r as handOffTail, t as REVIEW_ORIG } from "../chunks/tail-QZc7Uz2U.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,7 +1,7 @@
 import { read, refuse, removeScript, run, start } from "../flows.mjs";
-import { C as coder, Ft as envDefaults, G as finisherSystem, N as agentWithSkills, Pt as defaults, _t as reviewFeedbackFixPrompt, i as skills, l as gate, n as summary, r as access, s as shipped, t as steering, v as REQUIREMENTS, vt as reviewGateBlockedMessage } from "../chunks/reserved-D0MKVFFa.mjs";
-import { a as refusePendingHandOff } from "../chunks/tail-I1-vVq0P.mjs";
-import { i as reUnwind, t as buildTail } from "../chunks/review-ZzUQH1KZ.mjs";
+import { C as coder, Ft as defaults, It as envDefaults, K as finisherSystem, N as agentWithSkills, i as skills, l as gate, n as summary, r as access, s as shipped, t as steering, v as REQUIREMENTS, vt as reviewFeedbackFixPrompt, yt as reviewGateBlockedMessage } from "../chunks/reserved-DxB5CROj.mjs";
+import { a as refusePendingHandOff } from "../chunks/tail-QZc7Uz2U.mjs";
+import { i as reUnwind, t as buildTail } from "../chunks/review-CKMSdvgW.mjs";
 //#region src/workflows/review/review.ts
 const fixFeedback = async () => {
 	await agentWithSkills("feedback.fixing", reviewFeedbackFixPrompt(), {

package/dist/workflows/rollout.mjs

@@ -1,5 +1,5 @@
 import { agent, changesSince, glob, head, mark, read, refuse, removeScript, run, scope, vars } from "../flows.mjs";
-import { C as coder, Ft as envDefaults, Jt as flipProblems, Kt as flagMentions, Pt as defaults, Xt as requireFlag, f as FEEDBACK, i as skills, l as gate, n as summary, o as flagShipped, r as access, t as steering } from "../chunks/reserved-D0MKVFFa.mjs";
+import { C as coder, Ft as defaults, It as envDefaults, Yt as flipProblems, Zt as requireFlag, f as FEEDBACK, i as skills, l as gate, n as summary, o as flagShipped, qt as flagMentions, r as access, t as steering } from "../chunks/reserved-DxB5CROj.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.";

package/dist/workflows.mjs

@@ -17982,6 +17982,19 @@ writing \`(no hands-on)\` between its ID and the colon:
 Leave the tag off whatever a person can see by running the change. A
 tag the human added or removed at the gate stays as they left it.
 `;
+const impactBlock = `
+## Impact
+
+\`.gtd/REQUIREMENTS.md\` carries a \`## Impact\` section: explore the
+codebase (not just the start diff) and name the units and boundaries
+this change touches, the public surfaces it affects (CLI commands and
+flags, config keys, driver protocol, exported API), and anything
+surprising. Name the project's own declared units first —
+dependency-cruiser rules, workspace packages, module boundaries named
+in its docs or agent instructions (\`AGENTS.md\`, \`CLAUDE.md\`). When
+none is declared, fall back to directories and packages. Landing is
+refused while the section is missing.
+`;
 const riskTierBlock = `
 ## Risk tier and production signals
 
@@ -18021,7 +18034,14 @@ ${footnoteFoldIn}${codeThreadReplies()}
   or both
 - Whichever lap this is, leave \`.gtd/REQUIREMENTS.md\`
   uncommitted and finish once it reflects this lap's own work
-${acceptanceCriteriaBlock}
+${acceptanceCriteriaBlock}${impactBlock}
+Every lap re-explores and leaves \`## Impact\` present. On a return
+lap or review loop-back the human may have edited \`## Impact\` (or
+footnoted it): read that edit from history (\`git log -1 -p --
+.gtd/REQUIREMENTS.md\`), keep the human's correction, and
+re-explore from it. A surprise the human must settle becomes an
+entry under \`## Open Questions\`.
+
 ## First lap
 
 \`.gtd/REQUIREMENTS.md\` does not exist yet (or holds whatever the
@@ -18076,6 +18096,21 @@ ${questionBarReturn}
   made). If red, make that breakage the first concern, ahead of
   everything else — the build assumes the suite started green
 ${riskTier ? riskTierBlock : ""}`;
+const designImpactPrompt = () => `${styleBlock}
+
+${styleFormatContract}
+
+${stateFileRules}
+- The only state file this turn touches is \`.gtd/REQUIREMENTS.md\`
+  — no other files for notes or output
+- \`.gtd/REQUIREMENTS.md\` already holds the settled acceptance
+  criteria. Add a \`## Impact\` section and change nothing else — a
+  turn that touches anything outside it is refused
+- No human reviews this before the build: put any surprise in the
+  Impact prose, never as a question
+${impactBlock}
+- Leave \`.gtd/REQUIREMENTS.md\` uncommitted and finish once it holds \`## Impact\`
+`;
 const clearCriteriaQuestion = {
 	id: "clear",
 	primitive: "noul",
@@ -18108,9 +18143,9 @@ const slicingPrompt = (cap) => `${stateFileRules}
 - Read \`.gtd/REQUIREMENTS.md\`: its \`## Acceptance Criteria\` are settled
   and more than ${cap}, too many to build in one slice
 - Group them into slices in \`.gtd/SLICES.md\`, one \`## <title>\` heading per
-  slice over a \`- AC-<n>\` list item per criterion it builds. Put criteria of
-  the same flow or subsystem together; every criterion in exactly one slice,
-  at most ${cap} per slice
+  slice over a \`- AC-<n>\` list item per criterion it builds. Read the
+  \`## Impact\` section of \`.gtd/REQUIREMENTS.md\` and slice along the units it
+  names; every criterion in exactly one slice, at most ${cap} per slice
 - Keep it coarse: as few slices as the cap allows, never one per criterion
 - Order the slices so each builds on the ones before: they run in that order,
   each from where the previous one landed
@@ -18776,6 +18811,9 @@ const listed = (paths, unreadable = []) => paths.map((path) => `  - \`${displayP
 const isRed = (retry) => "red" in retry;
 const isFault = (retry) => "fault" in retry;
 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 impactHints = `- Its \`## Impact\` section maps where the change lands — reviewed at the design
+  gate when design ran. A starting point and hints, not a list of allowed files
+  — change whatever the work needs`;
 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"}`;
@@ -18822,6 +18860,7 @@ the context limit first if the refactor is too big`}. \`gtd abandon\` gives the
 /** `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
+${impactHints}
 ${sliceLine(slice)}${criteriaBinding(false)}
 - Implement them the quickest way that works — the naive version. Do not
   restructure, abstract or refactor around the change; it may land exactly as
@@ -18889,6 +18928,7 @@ const flagLine = (flagMode, tier, amend, sliced) => !flagMode ? "- Add no releas
   repository's flag conventions`;
 const implementBuildingPrompt = (reference, tier, amend, afterRefactor, flagMode = true, rebuild = {}, slice, from) => `${stateFileRules}
 - Read \`.gtd/REQUIREMENTS.md\`: the settled requirements for this change
+${impactHints}
 ${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
   the re-derived requirements instead of rewriting it
@@ -19024,6 +19064,7 @@ ${retest ? `- This is a re-test: \`.gtd/PROTOTYPE.md\` holds the last round's re
 `;
 const prototypeFixingPrompt = (slice) => `${stateFileRules}
 - Read \`.gtd/REQUIREMENTS.md\`: the settled requirements for this change
+${impactHints}
 ${sliceLine(slice)}${criteriaBinding(true)}
 - Read \`.gtd/PROTOTYPE.md\`: a hands-on test of the change. Fix every
   \`fail\` chunk of this change's criteria${slice === void 0 ? "" : " (this slice's only)"} or a problem found
@@ -39090,6 +39131,38 @@ const requireCriteria = (file) => {
 	if (criteria.length > 0 && unnamed.length === 0 && twice.length === 0) return;
 	refuse(`gtd land: acceptance-criteria: ${file} has ${criteria.length === 0 ? `no "## Acceptance Criteria" list` : unnamed.length > 0 ? `${unnamed.length} criterion without an ID:\n${unnamed.map((c) => `  - ${c.text}`).join("\n")}` : `duplicate IDs: ${[...new Set(twice)].join(", ")}`} — list every criterion under "## Acceptance Criteria" as "- AC-<n>: <checkable criterion>", each ID unique and never reused`);
 };
+const isImpactHeading = (h) => h !== null && h[1].length === 2 && /^impact$/i.test(h[2].trim());
+/** Whether `text` has an unfenced, level-2 `## Impact` heading. */
+const hasImpact = (text) => markFenced(text).some((l) => !l.fenced && isImpactHeading(HEADING.exec(l.line)));
+/** `text` without its `## Impact` section, whitespace collapsed. */
+const withoutImpact = (text) => {
+	let inImpact = false;
+	return markFenced(text).filter((l) => {
+		const h = l.fenced ? null : HEADING.exec(l.line);
+		if (h !== null && h[1].length <= 2) inImpact = isImpactHeading(h);
+		return !inImpact;
+	}).map((l) => l.line).join(" ").replace(/\s+/g, " ").trim();
+};
+/**
+* Refuse a turn that leaves `file` without a `## Impact` section. Only the
+* pending turn (`head()` is `""`): a triage lap recorded before Impact existed
+* replays without one, and refusing there would fail every in-flight process.
+* A human's edit at a later gate is not checked — the human owns that file then.
+*/
+const requireImpact = (file) => {
+	if (head$1() !== "") return;
+	const text = read(file);
+	if (text === void 0) return;
+	if (hasImpact(text)) return;
+	refuse(`gtd land: impact: ${file} has no "## Impact" section — explore the codebase and add "## Impact" naming the units, public surfaces and surprises this change touches`);
+};
+/** Refuse a turn that changed `file` beyond its `## Impact` section. */
+const requireImpactOnly = (file) => {
+	const change = changes().get(file);
+	if (change === void 0) return;
+	if (withoutImpact(change.before ?? "") === withoutImpact(change.after ?? "")) return;
+	refuse(`gtd land: impact-only: this turn may only add "## Impact" to ${file} — restore everything else`);
+};
 const idNumber = (id) => Number(id.slice(3));
 /** Why `listed` renumbers or reuses an earlier ID; `undefined` when it does neither. */
 const unstableIds = (listed, ids) => {
@@ -39123,6 +39196,19 @@ const clear = async (stated) => {
 	});
 	return truncated.length === 0 && answered(answers.clear, "yes", CLEAR_MIN_P);
 };
+/** The fast path's impact-only turn: add `## Impact`, leave the criteria alone. */
+const impact = async () => {
+	await agentWithSkills("impact", designImpactPrompt(), {
+		label: "Mapping the change's impact",
+		file: REQUIREMENTS,
+		model: planner(),
+		system: designSystem()
+	});
+	requireProgress(REQUIREMENTS);
+	requireCriteriaFrozen(REQUIREMENTS);
+	requireImpact(REQUIREMENTS);
+	requireImpactOnly(REQUIREMENTS);
+};
 /**
 * The skip path: when the sketch stated clear, checkable criteria, write them
 * to `.gtd/REQUIREMENTS.md` as they are and resolve `true` — no triage, no
@@ -39134,6 +39220,7 @@ const skipDesign = (stated) => stated === void 0 ? Promise.resolve(false) : scop
 		label: "Recording the stated acceptance criteria",
 		file: REQUIREMENTS
 	});
+	await impact();
 	return true;
 });
 /** After a gate that invites narrowing the requirements: an edit must leave valid criteria. */
@@ -39230,6 +39317,7 @@ const design = (base, guard) => scope("design", async () => {
 		requireProgress(REQUIREMENTS);
 		requireReplies(REQUIREMENTS);
 		requireCriteria(REQUIREMENTS);
+		requireImpact(REQUIREMENTS);
 		guard?.();
 	} while (await noteGate(REQUIREMENTS, answerProductQuestions));
 });
@@ -41023,4 +41111,4 @@ var workflows_default = feature;
 */
 const doors = {};
 //#endregion
-export { COMMENT, ESCALATION, FEEDBACK, FIX_CAP, FlagOptions, HANDOFF, PROTOTYPE, 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, describeEscalation, design, doors, envDefaults, escalate, escalation, escalationExhausted, feature, fix, fixCheck, fixFlow, fixNits, fixQuality, fixReviewNotes, fixRisks, gate, guarded, handOffReview, handOffTail, handsOnIds, healthy, holdsCallSites, implement, implementing, isTestFile, judgeSeam, lensEntriesOf, lensOf, lensesOf, naiveAcceptance, naivePass, ordinaryStart, planAndBuild, planner, qualityLap, reUnwind, recheck, refactorCollecting, refactorFirst, refactorNote, refactorStrays, requireAcceptance, requireAcceptanceKept, requireCriteria, requireCriteriaFrozen, requireCriteriaIfEdited, requireNoNewFlags, requireNoTestFiles, requireSlices, requireStableIds, 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, unstableIds, unwind };
+export { COMMENT, ESCALATION, FEEDBACK, FIX_CAP, FlagOptions, HANDOFF, PROTOTYPE, 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, describeEscalation, design, doors, envDefaults, escalate, escalation, escalationExhausted, feature, fix, fixCheck, fixFlow, fixNits, fixQuality, fixReviewNotes, fixRisks, gate, guarded, handOffReview, handOffTail, handsOnIds, hasImpact, healthy, holdsCallSites, implement, implementing, isTestFile, judgeSeam, lensEntriesOf, lensOf, lensesOf, naiveAcceptance, naivePass, ordinaryStart, planAndBuild, planner, qualityLap, reUnwind, recheck, refactorCollecting, refactorFirst, refactorNote, refactorStrays, requireAcceptance, requireAcceptanceKept, requireCriteria, requireCriteriaFrozen, requireCriteriaIfEdited, requireImpact, requireImpactOnly, requireNoNewFlags, requireNoTestFiles, requireSlices, requireStableIds, 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, unstableIds, unwind };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pmelab/gtd",
-  "version": "52.0.0",
+  "version": "52.1.0",
   "private": false,
   "description": "Git-aware CLI that emits the next prompt for an autonomous coding agent based on the current repository state",
   "bin": {
@@ -165,8 +165,9 @@
     "@types/react": "^19.2.7",
     "@types/react-dom": "^19.2.3",
     "@vitejs/plugin-react": "^5.2.0",
-    "@vitest/browser": "^3.2.7",
-    "@vitest/coverage-v8": "^3.2.7",
+    "@vitest/browser": "^4.1.12",
+    "@vitest/browser-playwright": "^4.1.12",
+    "@vitest/coverage-v8": "^4.1.12",
     "dependency-cruiser": "^18.3.1",
     "fallow": "3.25.0",
     "fast-check": "^4.8.0",
@@ -186,7 +187,7 @@
     "tailwindcss": "^4.3.3",
     "tsdown": "^0.22.8",
     "turbo": "^2.10.9",
-    "vitest": "^3.2.0",
+    "vitest": "^4.1.12",
     "yaml": "^2.8.2",
     "zustand": "^5.0.15"
   },

package/src/testkit/steps.ts

@@ -149,6 +149,10 @@ Given("the working tree is committed as {string}", (world: TestkitWorld, message
   world.repo().commit(message)
 })
 
+Given("the working tree is committed with the message:", (world: TestkitWorld, message: string) => {
+  world.repo().commit(message)
+})
+
 Given("step {string} changes nothing", (world: TestkitWorld, step: string) => {
   world.act(step, () => undefined)
 })
@@ -180,6 +184,15 @@ Given(
   },
 )
 
+Given(
+  "step {string} appends to {string}:",
+  (world: TestkitWorld, step: string, path: string, text: string) => {
+    world.act(step, (_, repo) => {
+      repo.write(path, `${(repo.read(path) ?? "").trimEnd()}\n\n${withNewline(text)}`)
+    })
+  },
+)
+
 Given("step {string} deletes {string}", (world: TestkitWorld, step: string, path: string) => {
   world.act(step, (_, repo) => repo.remove(path))
 })