@pmelab/gtd 51.0.1 → 52.1.0

package/src/workflows/feature/acceptance-criteria.feature

@@ -30,11 +30,15 @@ Feature: acceptance criteria carry stable IDs, and a sketch that states clear on
         greeting says "Helo"; make it "Hello".
         """
 
-    Scenario: triage writes ID'd criteria that reach the naive pass through the design gate
+    Scenario: [AC-6] triage writes ID'd criteria that reach the naive pass through the design gate
       Given step "design.triage" writes ".gtd/REQUIREMENTS.md" with:
         """
         # Requirements
 
+        ## Impact
+
+        - the greeting module
+
         ## Acceptance Criteria
 
         - AC-1: `greeting` reads "Hello, world".
@@ -45,6 +49,7 @@ Feature: acceptance criteria carry stable IDs, and a sketch that states clear on
       Then the process rests at "seam.naive"
       And ".gtd/REQUIREMENTS.md" contains "- AC-1: `greeting` reads \"Hello, world\"."
       And ".gtd/REQUIREMENTS.md" contains "- AC-2: no other export changes."
+      And the resting step's content contains "maps where the change lands"
       And the resting step's content contains "Acceptance Criteria"
       And the commit subjects end with:
         """
@@ -58,6 +63,10 @@ Feature: acceptance criteria carry stable IDs, and a sketch that states clear on
         """
         # Requirements
 
+        ## Impact
+
+        - the greeting module
+
         ## Acceptance Criteria
 
         - `greeting` reads "Hello, world".
@@ -69,6 +78,10 @@ Feature: acceptance criteria carry stable IDs, and a sketch that states clear on
         """
         # Requirements
 
+        ## Impact
+
+        - the greeting module
+
         ## Acceptance Criteria
 
         - AC-1: `greeting` reads "Hello, world".
@@ -81,6 +94,10 @@ Feature: acceptance criteria carry stable IDs, and a sketch that states clear on
         """
         # Requirements
 
+        ## Impact
+
+        - the greeting module
+
         ## Acceptance Criteria
 
         - AC-1: `greeting` reads "Hello, world".
@@ -110,15 +127,114 @@ Feature: acceptance criteria carry stable IDs, and a sketch that states clear on
         """
         # Requirements
 
+        ## Impact
+
+        - the greeting module
+
+        ## Acceptance Criteria
+
+        - AC-1: `greeting` reads "Hi".
+        """
+      Then driving the process is refused with "criteria-frozen"
+
+  Rule: impact
+
+    Scenario: [AC-1] a triage turn with no Impact section is refused, naming it
+      Given a file "TODO.md" with:
+        """
+        # Fix the typo
+
+        greeting says "Helo"; make it "Hello".
+        """
+      And step "design.triage" writes ".gtd/REQUIREMENTS.md" with:
+        """
+        # Requirements
+
+        ## Acceptance Criteria
+
+        - AC-1: `greeting` reads "Hello, world".
+        """
+      Then driving the process is refused with "impact: .gtd/REQUIREMENTS.md has no \"## Impact\" section"
+
+    Scenario: [AC-9] a design.impact turn that changes the criteria is refused
+      Given a file "TODO.md" with:
+        """
+        # Fix the typo
+
+        ## Acceptance criteria
+
+        - `greeting` reads "Hello, world".
+        """
+      And judge step "design.clear" answers:
+        """
+        [{"id":"clear","answer":"yes","p":0.9}]
+        """
+      And step "design.normalise" runs its script
+      And step "design.impact" writes ".gtd/REQUIREMENTS.md" with:
+        """
+        # Requirements
+
+        ## Impact
+
+        - the greeting module
+
         ## Acceptance Criteria
 
         - AC-1: `greeting` reads "Hi".
         """
       Then driving the process is refused with "criteria-frozen"
 
+    Scenario: [AC-9] a design.impact turn that adds no Impact section is refused
+      Given a file "TODO.md" with:
+        """
+        # Fix the typo
+
+        ## Acceptance criteria
+
+        - `greeting` reads "Hello, world".
+        """
+      And judge step "design.clear" answers:
+        """
+        [{"id":"clear","answer":"yes","p":0.9}]
+        """
+      And step "design.normalise" runs its script
+      And step "design.impact" appends to ".gtd/REQUIREMENTS.md":
+        """
+        Touches the greeting module.
+        """
+      Then driving the process is refused with "impact: .gtd/REQUIREMENTS.md has no \"## Impact\" section"
+
+    Scenario: [AC-9] a design.impact turn that drops the sketch pointer is refused
+      Given a file "TODO.md" with:
+        """
+        # Fix the typo
+
+        ## Acceptance criteria
+
+        - `greeting` reads "Hello, world".
+        """
+      And judge step "design.clear" answers:
+        """
+        [{"id":"clear","answer":"yes","p":0.9}]
+        """
+      And step "design.normalise" runs its script
+      And step "design.impact" writes ".gtd/REQUIREMENTS.md" with:
+        """
+        # Requirements
+
+        ## Impact
+
+        - the greeting module
+
+        ## Acceptance Criteria
+
+        - AC-1: `greeting` reads "Hello, world".
+        """
+      Then driving the process is refused with "impact-only"
+
   Rule: the skip path
 
-    Scenario: clear stated criteria are normalised into ID'd requirements with no triage and no design gate
+    Scenario: [AC-9] [AC-10] clear stated criteria are normalised into ID'd requirements with no triage and no design gate
       Given a file "TODO.md" with:
         """
         # Fix the typo
@@ -133,18 +249,52 @@ Feature: acceptance criteria carry stable IDs, and a sketch that states clear on
         [{"id":"clear","answer":"yes","p":0.9}]
         """
       And step "design.normalise" runs its script
+      And step "design.impact" appends to ".gtd/REQUIREMENTS.md":
+        """
+        ## Impact
+
+        - the greeting module
+        """
       When the process is driven
       Then the process rests at "seam.naive"
       And ".gtd/REQUIREMENTS.md" contains "- AC-1: `greeting` reads \"Hello, world\"."
       And ".gtd/REQUIREMENTS.md" contains "- AC-2: no other export changes."
+      And ".gtd/REQUIREMENTS.md" contains "Stated in `TODO.md` by the sketch"
+      And the resting step's content contains "maps where the change lands"
       And the commit subjects end with:
         """
         gtd(human): idle → unwind
         gtd(check): unwind → start-gate.check
         gtd(check): start-gate.check → design.clear
         gtd(judge): design.clear → design.normalise
-        gtd(check): design.normalise → seam.naive
+        gtd(check): design.normalise → design.impact
+        gtd(agent): design.impact → seam.naive
+        """
+
+    Scenario: a fast-path process recorded before design.impact existed fails replay
+      Given a file "TODO.md" with:
+        """
+        # Fix the typo
+
+        ## Acceptance criteria
+
+        - `greeting` reads "Hello, world".
+        """
+      And judge step "design.clear" answers:
+        """
+        [{"id":"clear","answer":"yes","p":0.9}]
+        """
+      And step "design.normalise" runs its script
+      When the process is driven
+      Then the process rests at "design.impact"
+      Given the working tree is committed with the message:
+        """
+        gtd(agent): seam.naive → seam.check
+
+        Gtd-Step: seam.naive#1
+        Gtd-Format: 1
         """
+      Then driving the process is refused with "the workflow changed under this process"
 
     Scenario: IDs the sketch already gave its criteria are kept
       Given a file "TODO.md" with:
@@ -159,6 +309,12 @@ Feature: acceptance criteria carry stable IDs, and a sketch that states clear on
         [{"id":"clear","answer":"yes","p":0.9}]
         """
       And step "design.normalise" runs its script
+      And step "design.impact" appends to ".gtd/REQUIREMENTS.md":
+        """
+        ## Impact
+
+        - the greeting module
+        """
       When the process is driven
       Then the process rests at "seam.naive"
       And ".gtd/REQUIREMENTS.md" contains "- AC-3: `greeting` reads \"Hello, world\"."

package/src/workflows/feature/continuation-cap.feature

@@ -26,6 +26,10 @@ Feature: past its continuation cap a phase rests for a person
       """
       # Requirements
 
+      ## Impact
+
+      - the greeting module
+
       ## Acceptance Criteria
 
       - AC-1: `subtract(5, 3)` returns 2.

package/src/workflows/feature/continuation-rebuild.feature

@@ -31,6 +31,10 @@ Feature: a rebuild stopped at its context limit is continued on the branch
       """
       # Requirements
 
+      ## Impact
+
+      - the greeting module
+
       ## Acceptance Criteria
 
       - AC-1: `subtract(5, 3)` returns 2.

package/src/workflows/feature/continuation-slices.feature

@@ -30,6 +30,10 @@ Feature: a continuation stays inside its coarse slice
       """
       # Requirements
 
+      ## Impact
+
+      - the greeting module
+
       ## Acceptance Criteria
 
       - AC-1: subtract(a, b) returns a - b.

package/src/workflows/feature/continuation.feature

@@ -29,6 +29,10 @@ Feature: a turn stopped at its context limit is snapshotted and continued
       """
       # Requirements
 
+      ## Impact
+
+      - the greeting module
+
       ## Acceptance Criteria
 
       - AC-1: `subtract(5, 3)` returns 2.

package/src/workflows/feature/criteria.test.ts

@@ -1,6 +1,13 @@
 import { describe, expect, it } from "vitest"
 import type { Change } from "../../flows/index.js"
-import { criteriaOf, criterionIds, statedCriteria, unstableIds } from "./criteria.js"
+import {
+  criteriaOf,
+  criterionIds,
+  handsOnIds,
+  hasImpact,
+  statedCriteria,
+  unstableIds,
+} from "./criteria.js"
 
 const added = (path: string, after: string): Change => ({
   path,
@@ -48,6 +55,17 @@ describe("criteriaOf", () => {
     ).toEqual([{ id: "AC-1", text: "exits 2 when config is bad - missing file" }])
   })
 
+  it("reads the (no hands-on) tag between an ID and its colon", () => {
+    expect(
+      criteriaOf(
+        "## Acceptance Criteria\n\n- AC-1: runs\n- AC-2 (no hands-on): the prompt says so\n",
+      ),
+    ).toEqual([
+      { id: "AC-1", text: "runs" },
+      { id: "AC-2", text: "the prompt says so", noHandsOn: true },
+    ])
+  })
+
   it("is undefined without the section", () => {
     expect(criteriaOf("# R\n\n- AC-1: x\n")).toBeUndefined()
   })
@@ -66,6 +84,19 @@ describe("statedCriteria", () => {
     })
   })
 
+  it("keeps a stated (no hands-on) tag, renumbered or not", () => {
+    const tagged = "## Acceptance criteria\n\n- AC-1: a\n- AC-2 (no hands-on): b\n"
+    expect(statedCriteria([added("TODO.md", tagged)], "abc")?.criteria).toEqual([
+      { id: "AC-1", text: "a" },
+      { id: "AC-2", text: "b", noHandsOn: true },
+    ])
+    const partial = "## Acceptance criteria\n\n- a\n- AC-7 (no hands-on): b\n"
+    expect(statedCriteria([added("TODO.md", partial)], "abc")?.criteria).toEqual([
+      { id: "AC-1", text: "a" },
+      { id: "AC-2", text: "b", noHandsOn: true },
+    ])
+  })
+
   it("renumbers when the stated IDs are partial or clash", () => {
     const sketch = [added("TODO.md", "## Acceptance criteria\n\n- AC-4: a\n- b\n")]
     expect(statedCriteria(sketch, "abc")?.criteria).toEqual([
@@ -121,6 +152,20 @@ describe("criterionIds", () => {
   })
 })
 
+describe("handsOnIds", () => {
+  const requirements =
+    "## Acceptance Criteria\n\n- AC-1: a\n- AC-2 (no hands-on): b\n- AC-3: c\n- AC-4 (no hands-on): d\n"
+
+  it("leaves out every criterion tagged (no hands-on)", () => {
+    expect(handsOnIds(requirements)).toEqual(["AC-1", "AC-3"])
+  })
+
+  it("keeps to the slice", () => {
+    expect(handsOnIds(requirements, { only: ["AC-2", "AC-3"] })).toEqual(["AC-3"])
+    expect(handsOnIds(requirements, { only: ["AC-2", "AC-4"] })).toEqual([])
+  })
+})
+
 describe("unstableIds", () => {
   const ids = { closed: ["AC-1", "AC-3"], used: ["AC-1", "AC-2", "AC-3"] }
 
@@ -139,3 +184,17 @@ describe("unstableIds", () => {
     expect(unstableIds(["AC-0", "AC-2"], ids)).toMatch(/^numbers AC-0, AC-2 at or below AC-3/)
   })
 })
+
+describe("hasImpact", () => {
+  it.each([
+    ["a level-2 heading", "# R\n\n## Impact\n\nx\n", true],
+    ["a mixed-case heading", "## impact\n", true],
+    ["a level-3 heading", "### Impact\n", false],
+    ["a level-1 heading", "# Impact\n", false],
+    ["a longer heading", "## Impactful\n", false],
+    ["a fenced heading", "```md\n## Impact\n```\n", false],
+    ["no heading", "# R\n\nImpact\n", false],
+  ])("%s", (_name, text, expected) => {
+    expect(hasImpact(text)).toBe(expected)
+  })
+})

package/src/workflows/feature/criteria.ts

@@ -1,20 +1,24 @@
 import {
   answered,
   changes,
+  head,
   judge,
   read,
   refuse,
+  requireProgress,
   run,
   scope,
   type Change,
 } from "../../flows/index.js"
 import { HEADING, ITEM, markFenced } from "../shared/fences.js"
-import { REQUIREMENTS } from "../shared/steps.js"
+import { planner, REQUIREMENTS } from "../shared/steps.js"
 import * as t from "../shared/text.js"
 
 export interface Criterion {
   readonly id: string
   readonly text: string
+  /** Tagged `(no hands-on)`: its tests prove it, so the hands-on test skips it. */
+  readonly noHandsOn?: true
 }
 
 /** Acceptance criteria the human's sketch stated outright, ready to normalise. */
@@ -29,7 +33,13 @@ export interface StatedCriteria {
 /** Below this the judge's `yes` is not trusted: a wrong skip builds against criteria nobody can check. */
 const CLEAR_MIN_P = 0.8
 
-const ID = /^(AC-\d+):\s*(.*)$/
+const ID = /^(AC-\d+)(\s*\(no hands-on\))?:\s*(.*)$/
+
+const criterion = (id: RegExpExecArray, as = id[1]!): Criterion => ({
+  id: as,
+  text: id[3]!,
+  ...(id[2] === undefined ? {} : { noHandsOn: true }),
+})
 
 /** The list items under the first heading `title` matches (at `level` when given), up to the next heading; wrapped lines joined, fenced lines ignored. */
 const listUnder = (text: string, title: RegExp, level?: number): string[] | undefined => {
@@ -66,7 +76,7 @@ const listUnder = (text: string, title: RegExp, level?: number): string[] | unde
 export const criteriaOf = (text: string): readonly Criterion[] | undefined =>
   listUnder(text, /^acceptance criteria$/i, 2)?.map((item) => {
     const id = ID.exec(item)
-    return id === null ? { id: "", text: item } : { id: id[1]!, text: id[2]! }
+    return id === null ? { id: "", text: item } : criterion(id)
   })
 
 /**
@@ -90,6 +100,12 @@ export const criterionIds = (text: string, slice?: CriteriaSlice): string[] =>
         !(slice?.except ?? []).includes(id),
     )
 
+/** The ID'd criteria of `text` (`slice`'s when given) a person can try by hand: none tagged `(no hands-on)`. */
+export const handsOnIds = (text: string, slice?: CriteriaSlice): string[] => {
+  const tagged = (criteriaOf(text) ?? []).filter((c) => c.noHandsOn).map((c) => c.id)
+  return criterionIds(text, slice).filter((id) => !tagged.includes(id))
+}
+
 /** `slice`'s IDs in `.gtd/REQUIREMENTS.md` as it is now; `undefined` for an unsliced change. */
 export const sliceIds = (slice: CriteriaSlice | undefined): string[] | undefined =>
   slice === undefined ? undefined : criterionIds(read(REQUIREMENTS) ?? "", slice)
@@ -98,11 +114,11 @@ export const sliceIds = (slice: CriteriaSlice | undefined): string[] | undefined
 const numbered = (items: readonly string[]): Criterion[] => {
   const ids = items.map((item) => ID.exec(item))
   const own = ids.every((id) => id !== null) && new Set(ids.map((id) => id![1])).size === ids.length
-  return items.map((item, i) =>
-    own
-      ? { id: ids[i]![1]!, text: ids[i]![2]! }
-      : { id: `AC-${i + 1}`, text: item.replace(ID, "$2") },
-  )
+  return items.map((item, i) => {
+    const id = ids[i]
+    if (own) return criterion(id!)
+    return id === null ? { id: `AC-${i + 1}`, text: item } : criterion(id!, `AC-${i + 1}`)
+  })
 }
 
 const STATED = /^acceptance criteria:?$/i
@@ -151,6 +167,54 @@ export const requireCriteria = (file: string): void => {
   )
 }
 
+const isImpactHeading = (h: RegExpExecArray | null): boolean =>
+  h !== null && h[1]!.length === 2 && /^impact$/i.test(h[2]!.trim())
+
+/** Whether `text` has an unfenced, level-2 `## Impact` heading. */
+export const hasImpact = (text: string): boolean =>
+  markFenced(text).some((l) => !l.fenced && isImpactHeading(HEADING.exec(l.line)))
+
+/** `text` without its `## Impact` section, whitespace collapsed. */
+const withoutImpact = (text: string): string => {
+  let inImpact = false
+  const kept = 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
+  })
+  return kept
+    .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.
+ */
+export const requireImpact = (file: string): void => {
+  if (head() !== "") return
+  const text = read(file)
+  if (text === undefined) 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. */
+export const requireImpactOnly = (file: string): void => {
+  const change = changes().get(file)
+  if (change === undefined) 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`,
+  )
+}
+
 /** The IDs a review's feedback lap must respect. */
 export interface StableIds {
   /** Listed when the build closed: each may stay, reworded or not, or go. */
@@ -197,6 +261,20 @@ const clear = async (stated: StatedCriteria): Promise<boolean> => {
   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 (): Promise<void> => {
+  await t.agentWithSkills("impact", t.designImpactPrompt(), {
+    label: "Mapping the change's impact",
+    file: REQUIREMENTS,
+    model: planner(),
+    system: t.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
@@ -211,6 +289,7 @@ export const skipDesign = (stated: StatedCriteria | undefined): Promise<boolean>
           label: "Recording the stated acceptance criteria",
           file: REQUIREMENTS,
         })
+        await impact()
         return true
       })
 

package/src/workflows/feature/fast-path.feature

@@ -27,10 +27,6 @@ Feature: a trivial change takes the same few beats at either risk tier
     And step "design.gate.answer" changes nothing
     And step "implement.health.check" changes nothing
     And step "implement.closing" runs its script
-    And judge step "implement.coverage" answers:
-      """
-      [{"id":"AC-1","answer":"yes","p":0.9}]
-      """
     And step "build.quality.correctness.reviewing" changes nothing
 
   Scenario Outline: a one-line fix runs from idle to the review gate with the same beats at either tier
@@ -42,6 +38,10 @@ Feature: a trivial change takes the same few beats at either risk tier
 
       <tier> — one-line copy fix
 
+      ## Impact
+
+      - the greeting module
+
       ## Acceptance Criteria
 
       - AC-1: the change behaves as described.
@@ -78,8 +78,7 @@ Feature: a trivial change takes the same few beats at either risk tier
       gtd(check): seam.check → seam.judge
       gtd(judge): seam.judge → implement.applying
       gtd(check): implement.applying → implement.health.check
-      gtd(check): implement.health.check → implement.coverage
-      gtd(judge): implement.coverage → implement.closing
+      gtd(check): implement.health.check → implement.closing
       gtd(check): implement.closing → build.quality.correctness.reviewing
       gtd(agent): build.quality.correctness.reviewing → build.review.reviewing
       """

package/src/workflows/feature/feature.test.ts

@@ -99,7 +99,7 @@ describe("afterTail feedback lap", () => {
       "build.review.collecting": (_, r) => {
         r.write(
           ".gtd/REQUIREMENTS.md",
-          "# Requirements\n\n- multiply\n\n## Acceptance Criteria\n\n- AC-1: done\n",
+          "# Requirements\n\n- multiply\n\n## Impact\n\n- the module\n\n## Acceptance Criteria\n\n- AC-1: done\n",
         )
         r.write(".gtd/REVIEW.md", `${review("also export multiply")}    - A: noted.\n`)
       },
@@ -115,7 +115,7 @@ describe("afterTail feedback lap", () => {
     "design.triage": (_: unknown, r: ReturnType<typeof testRepo>) =>
       r.write(
         ".gtd/REQUIREMENTS.md",
-        "# Requirements\n\n- multiply, designed\n\n## Acceptance Criteria\n\n- AC-1: done\n",
+        "# Requirements\n\n- multiply, designed\n\n## Impact\n\n- the module\n\n## Acceptance Criteria\n\n- AC-1: done\n",
       ),
     "design.gate.answer": () => undefined,
   }
@@ -214,7 +214,7 @@ describe("an in-flight 28.0.0 naive-pass process", () => {
       "design.triage": (_, r) =>
         r.write(
           ".gtd/REQUIREMENTS.md",
-          "# Requirements\n\n## Concerns\n\n### S\n\n- `s`\n\n## Acceptance Criteria\n\n- AC-1: done\n",
+          "# Requirements\n\n## Concerns\n\n### S\n\n- `s`\n\n## Impact\n\n- the module\n\n## Acceptance Criteria\n\n- AC-1: done\n",
         ),
       "design.gate.answer": () => undefined,
       "seam.naive": (_, r) => {
@@ -258,6 +258,7 @@ describe("an in-flight 28.0.0 naive-pass process", () => {
 })
 
 describe("an in-flight process from before the architecture path was removed", () => {
+  // No `## Impact`: no version before it wrote one, and replaying its triage must not refuse.
   const REQUIREMENTS =
     "# Requirements\n\n## Concerns\n\n### Subtract\n\n- `subtract`\n\n## Acceptance Criteria\n\n- AC-1: done\n"
 
@@ -321,7 +322,7 @@ describe("flagMode", () => {
       "design.triage": (_, r) =>
         r.write(
           ".gtd/REQUIREMENTS.md",
-          "# Requirements\n\n## Concerns\n\n### S\n\n- `s`\n\n## Acceptance Criteria\n\n- AC-1: done\n",
+          "# Requirements\n\n## Concerns\n\n### S\n\n- `s`\n\n## Impact\n\n- the module\n\n## Acceptance Criteria\n\n- AC-1: done\n",
         ),
       "design.gate.answer": () => undefined,
     })
@@ -342,7 +343,7 @@ describe("flagMode", () => {
       "design.triage": (_, r) =>
         r.write(
           ".gtd/REQUIREMENTS.md",
-          "# Requirements\n\n## Concerns\n\n### S\n\n- `s`\n\n## Acceptance Criteria\n\n- AC-1: done\n",
+          "# Requirements\n\n## Concerns\n\n### S\n\n- `s`\n\n## Impact\n\n- the module\n\n## Acceptance Criteria\n\n- AC-1: done\n",
         ),
       "design.gate.answer": () => undefined,
       "seam.naive": (_, r) => {

package/src/workflows/feature/feature.ts

@@ -123,7 +123,6 @@ const buildSlice = async (sliceBase: string, build: SliceBuild): Promise<void> =
       naive: seam.land === true ? seam.reference : undefined,
       rebuild: seam.rebuild,
       slice: build.criteria,
-      base: build.criteria === undefined ? undefined : sliceBase,
       more: build.more,
     })
     // Closing kept the criteria exactly when there is more to build, so this reads as it did.
@@ -179,7 +178,7 @@ const reviewedTail = (
   )
 
 // seam.naive / seam.judge never run on a feedback lap: its design → implement is built unjudged
-// and unrefactored, held to every criterion's coverage since the process start.
+// and unrefactored.
 const featureFeedback = async (
   first: ReviewOutcome,
   lensBase: string,

package/src/workflows/feature/flag-mode.feature

@@ -37,6 +37,10 @@ Feature: flag mode off lands the naive pass with no release flag, seam judge or
 
       - `subtract(a, b)` returns `a - b`.
 
+      ## Impact
+
+      - the greeting module
+
       ## Acceptance Criteria
 
       - AC-1: the change behaves as described.
@@ -61,10 +65,6 @@ Feature: flag mode off lands the naive pass with no release flag, seam judge or
     And step "implement.applying" runs its script
     And step "implement.health.check" changes nothing
     And step "implement.closing" runs its script
-    And judge step "implement.coverage" answers:
-      """
-      [{"id":"AC-1","answer":"yes","p":0.9}]
-      """
     And step "build.quality.correctness.reviewing" changes nothing
     When the process is driven
     Then the process rests at "build.review.reviewing"
@@ -82,8 +82,7 @@ Feature: flag mode off lands the naive pass with no release flag, seam judge or
       gtd(agent): seam.naive → seam.check
       gtd(check): seam.check → implement.applying
       gtd(check): implement.applying → implement.health.check
-      gtd(check): implement.health.check → implement.coverage
-      gtd(judge): implement.coverage → implement.closing
+      gtd(check): implement.health.check → implement.closing
       gtd(check): implement.closing → build.quality.correctness.reviewing
       gtd(agent): build.quality.correctness.reviewing → build.review.reviewing
       """