@pmelab/gtd 29.3.1 → 30.1.0

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

@@ -0,0 +1,157 @@
+@inmem
+Feature: flag mode off builds a feature with no release flag, seam judge or refactor
+
+  `flagMode` is a factory option of the shipped workflows, on by default. Off,
+  the project ships without feature flags: no seam judge and no refactor run,
+  and the naive implementation goes straight through the health loop and the
+  quality lap.
+
+  Background:
+    Given a gtd config with:
+      """
+      import { feature } from "@pmelab/gtd/workflows/feature"
+      export { summary, steering, skills, access } from "@pmelab/gtd/workflows/feature"
+      export default feature({ flagMode: false })
+      """
+    And an environment variable "GTD_QUALITYREVIEWS" set to "correctness"
+    And a file "src/calc.ts" with:
+      """
+      export const add = (a: number, b: number) => a + b
+      """
+    And the working tree is committed as "feat: add calculator"
+    And a file "TODO.md" with:
+      """
+      # Add subtract
+
+      subtract(a, b) returns the difference.
+      """
+    And step "unwind" changes nothing
+    And step "start-gate.check" changes nothing
+    And step "design.triage" writes ".gtd/REQUIREMENTS.md" with:
+      """
+      # Requirements
+
+      ## Concerns
+
+      ### Subtract
+
+      - `subtract(a, b)` returns `a - b`.
+      """
+    And step "design.gate.answer" changes nothing
+
+  Scenario: flag mode off skips the seam judge and the refactor, and asks for no flag
+    When the process is driven
+    Then the process rests at "implement.building"
+    And the resting step's content contains "the quickest way that works"
+    And the resting step's content contains "Add no release flag — this project ships without release flags"
+    And the resting step's content does not contain "new release flag"
+    Given step "implement.building" writes "src/calc.ts" with:
+      """
+      export const add = (a: number, b: number) => a + b
+      export const subtract = (a: number, b: number) => a - b
+      """
+    And step "implement.health.check" changes nothing
+    And step "implement.closing" runs its script
+    And step "build.quality.correctness.reviewing" changes nothing
+    When the process is driven
+    Then the process rests at "build.review.reviewing"
+    And "src/calc.ts" contains "subtract"
+    And the commit subjects end with:
+      """
+      chore: initial commit
+      feat: add calculator
+      gtd(human): idle → unwind
+      gtd(check): unwind → start-gate.check
+      gtd(check): start-gate.check → design.triage
+      gtd(agent): design.triage → design.gate.answer
+      gtd(human): design.gate.answer → implement.building
+      gtd(agent): implement.building → implement.health.check
+      gtd(check): implement.health.check → implement.closing
+      gtd(check): implement.closing → build.quality.correctness.reviewing
+      gtd(agent): build.quality.correctness.reviewing → build.review.reviewing
+      """
+
+  Scenario: flag mode off asks for no flag on a feedback lap either
+    Given step "implement.building" writes "src/calc.ts" with:
+      """
+      export const add = (a: number, b: number) => a + b
+      export const subtract = (a: number, b: number) => a - b
+      """
+    And step "implement.health.check" changes nothing
+    And step "implement.closing" runs its script
+    And step "build.quality.correctness.reviewing" changes nothing
+    And step "build.review.reviewing" writes ".gtd/REVIEW.md" with:
+      """
+      # Review: abc1234
+
+      <!-- base: 0000000 -->
+
+      ## calc
+
+      - [ ] ./src/calc.ts#2-2
+        new subtract function
+      """
+    When the process is driven
+    Then the process rests at "build.review.await-review"
+    Given a file ".gtd/REVIEW.md" with:
+      """
+      # Review: abc1234
+
+      <!-- base: 0000000 -->
+
+      ## calc
+
+      - [ ] ./src/calc.ts#2-2
+        new subtract function[^why]
+
+      [^why]:
+          - H: also export multiply
+      """
+    And step "build.review.collecting" writes ".gtd/REQUIREMENTS.md" with:
+      """
+      # Requirements
+
+      ## Concerns
+
+      ### Multiply
+
+      - Export `multiply()`.
+      """
+    And step "build.review.collecting" writes ".gtd/REVIEW.md" with:
+      """
+      # Review: abc1234
+
+      <!-- base: 0000000 -->
+
+      ## calc
+
+      - [ ] ./src/calc.ts#2-2
+        new subtract function[^why]
+
+      [^why]:
+          - H: also export multiply
+          - A: noted in the requirements.
+      """
+    When gtd lands
+    And the process is driven
+    Then the process rests at "build.review.await-review"
+    Given a file ".gtd/REVIEW.md" with:
+      """
+      # Review: abc1234
+
+      <!-- base: 0000000 -->
+
+      ## calc
+
+      - [ ] ./src/calc.ts#2-2
+        new subtract function
+      """
+    And step "build.review.closing" runs its script
+    And step "re-unwind" runs its script
+    And step "implement.building" is no longer scripted
+    When gtd lands
+    And the process is driven
+    Then the process rests at "implement.building"
+    And the resting step's content contains "Add no release flag — this project ships without release flags"
+    And the resting step's content does not contain "release flag the earlier lap added"
+    And no commit body contains "Gtd-Kept:"

package/src/workflows/feature/index.ts

@@ -2,7 +2,6 @@ import { feature } from "./feature.js"
 
 export * from "./steps.js"
 export * from "./planning.js"
-export * from "./packages.js"
 export * from "./refactor.js"
 export * from "./seam.js"
 export {

package/src/workflows/feature/one-step-land.feature

@@ -2,16 +2,15 @@
 @inmem
 Feature: a naive pass the judge rules land goes straight to implement with no seam gate
 
-  With GTD_FLAG_NAIVE_PASS on and GTD_FLAG_RISK_TIER off, a naive pass that
+  With GTD_FLAG_RISK_TIER off, a naive pass that
   edits an existing source file is judged land: the process skips the approval
-  gate, any refactor and any architecture, and runs implement, quality and
-  review back to idle. The flags are pinned here so the per-flag test runs never
+  gate and any refactor, and runs implement, quality and
+  review back to idle. The flag is pinned here so the per-flag test runs never
   move it off this path.
 
   Background:
     Given the shipped workflows, as gtd install scaffolds them
     And an environment variable "GTD_QUALITYREVIEWS" set to "correctness"
-    And an environment variable "GTD_FLAG_NAIVE_PASS" set to "1"
     And an environment variable "GTD_FLAG_RISK_TIER" set to "0"
     And a file "src/greeting.ts" with:
       """
@@ -65,7 +64,7 @@ Feature: a naive pass the judge rules land goes straight to implement with no se
       """
     And step "build.review.closing" runs its script
 
-  Scenario: a land judgment runs idle to idle with no seam gate, refactor or architecture
+  Scenario: a land judgment runs idle to idle with no seam gate or refactor
     When the process is driven
     Then the process rests at "build.review.await-review"
     Given a file ".gtd/REVIEW.md" with:
@@ -100,7 +99,6 @@ Feature: a naive pass the judge rules land goes straight to implement with no se
       gtd(human): build.review.await-review → build.review.closing
       gtd(check): build.review.closing → idle
       """
-    And the commit "gtd(human): idle → unwind" has a body containing "Gtd-Var: flag_naive_pass=1"
     And the commit "gtd(human): idle → unwind" has a body containing "Gtd-Var: flag_risk_tier=0"
     And the commit "gtd(agent): seam.naive → seam.judge" has a body containing "Gtd-Kept:"
     And the commit "gtd(judge): seam.judge → implement.building" has a body containing "\"answer\":\"land\""

package/src/workflows/feature/planning.ts

@@ -1,21 +1,14 @@
 import {
   changes,
   hasThreadFor,
-  refuse,
   requireAnswers,
   requireReplies,
   requireThreadsClosed,
   requireProgress,
   scope,
 } from "../../flows/index.js"
-import { ARCHITECTURE, REQUIREMENTS } from "../shared/steps.js"
-import {
-  answerProductQuestions,
-  answerTechnicalQuestions,
-  author,
-  decompose,
-  triage,
-} from "./steps.js"
+import { REQUIREMENTS } from "../shared/steps.js"
+import { answerProductQuestions, triage } from "./steps.js"
 
 /**
  * Always stop for the human. Resolves `true` when the round changed anything
@@ -40,21 +33,3 @@ export const design = (base: string): Promise<void> =>
       requireReplies(REQUIREMENTS)
     } while (await noteGate(REQUIREMENTS, answerProductQuestions))
   })
-
-/** Work out the technical plan, then decompose it into package files. */
-export const architecture = (): Promise<void> =>
-  scope("architecture", async () => {
-    do {
-      await author()
-      requireReplies(ARCHITECTURE)
-    } while (await noteGate(ARCHITECTURE, answerTechnicalQuestions))
-    await decompose()
-    if (changes(".gtd/packages/**").length === 0) {
-      refuse("gtd land: decompose: write at least one package under .gtd/packages/")
-    }
-    if (changes(ARCHITECTURE).some((c) => c.status === "deleted")) {
-      refuse(
-        "gtd land: decompose: keep .gtd/ARCHITECTURE.md — the packages are built from it; it is removed with the last package",
-      )
-    }
-  })

package/src/workflows/feature/refactor-absorbs.feature

@@ -2,7 +2,7 @@
 @inmem
 Feature: a refactor that carried the whole feature still gets a feature review
 
-  With GTD_FLAG_NAIVE_PASS on and GTD_FLAG_RISK_TIER off, a refactor-first
+  With GTD_FLAG_RISK_TIER off, a refactor-first
   refactor can land everything the requirements ask for. The feature slice's
   implement turn then has nothing to author: its empty turn completes the
   step, and with no code changed since the refactor the feature review covers
@@ -12,7 +12,6 @@ Feature: a refactor that carried the whole feature still gets a feature review
     Given the shipped workflows, as gtd install scaffolds them
     And an environment variable "GTD_QUALITYREVIEWS" set to ""
     And an environment variable "GTD_REFACTORQUALITYREVIEWS" set to ""
-    And an environment variable "GTD_FLAG_NAIVE_PASS" set to "1"
     And an environment variable "GTD_FLAG_RISK_TIER" set to "0"
     And a file "src/calc.ts" with:
       """

package/src/workflows/feature/refactor-first.feature

@@ -2,7 +2,7 @@
 @inmem
 Feature: a refactor-first judgment walks one fixed path to idle
 
-  With GTD_FLAG_NAIVE_PASS on and GTD_FLAG_RISK_TIER off, a naive pass touching
+  With GTD_FLAG_RISK_TIER off, a naive pass touching
   an existing non-allowlisted file is ruled refactor-first, is refactored and
   that refactor reviewed, and only then implemented. The flags are
   pinned so the per-flag test runs never move it off its path.
@@ -11,7 +11,6 @@ Feature: a refactor-first judgment walks one fixed path to idle
     Given the shipped workflows, as gtd install scaffolds them
     And an environment variable "GTD_QUALITYREVIEWS" set to ""
     And an environment variable "GTD_REFACTORQUALITYREVIEWS" set to ""
-    And an environment variable "GTD_FLAG_NAIVE_PASS" set to "1"
     And an environment variable "GTD_FLAG_RISK_TIER" set to "0"
     And a file "src/calc.ts" with:
       """
@@ -154,7 +153,6 @@ Feature: a refactor-first judgment walks one fixed path to idle
       gtd(human): build.review.await-review → build.review.closing
       gtd(check): build.review.closing → idle
       """
-    And the commit "gtd(human): idle → unwind" has a body containing "Gtd-Var: flag_naive_pass=1"
     And the commit "gtd(human): idle → unwind" has a body containing "Gtd-Var: flag_risk_tier=0"
     And the commit "gtd(agent): seam.naive → seam.judge" has a body containing "Gtd-Kept:"
     And the commit "gtd(judge): seam.judge → refactor.deriving" has a body containing "\"answer\":\"refactor-first\""

package/src/workflows/feature/refactor-review.feature

@@ -10,7 +10,6 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
   Background:
     Given the shipped workflows, as gtd install scaffolds them
     And an environment variable "GTD_QUALITYREVIEWS" set to ""
-    And an environment variable "GTD_FLAG_NAIVE_PASS" set to "1"
     And an environment variable "GTD_FLAG_RISK_TIER" set to "0"
 
   Scenario: with a narrowed seamAllow a test file added by the refactor is refused

package/src/workflows/feature/risk-tier.feature

@@ -1,7 +1,7 @@
 @inmem
 Feature: the risk tier is proposed at triage and recorded behind GTD_FLAG_RISK_TIER
 
-  With GTD_FLAG_NAIVE_PASS and GTD_FLAG_RISK_TIER both on, design.triage asks
+  With GTD_FLAG_RISK_TIER on, design.triage asks
   for a risk tier and production signals, and the design gate's exit lands them
   as Gtd-Mark trailers: one `risk-tier=<low|high>`, one `production-signal`
   per signal. A missing or unreadable tier is high. With the flag off, nothing
@@ -10,7 +10,6 @@ Feature: the risk tier is proposed at triage and recorded behind GTD_FLAG_RISK_T
   Background:
     Given the shipped workflows, as gtd install scaffolds them
     And an environment variable "GTD_QUALITYREVIEWS" set to ""
-    And an environment variable "GTD_FLAG_NAIVE_PASS" set to "1"
 
   Rule: with GTD_FLAG_RISK_TIER on
 
@@ -35,7 +34,7 @@ Feature: the risk tier is proposed at triage and recorded behind GTD_FLAG_RISK_T
       And step "unwind" changes nothing
       And step "start-gate.check" changes nothing
 
-    Scenario: with both flags on the triage prompt asks for the tier and the signals
+    Scenario: with the flag on the triage prompt asks for the tier and the signals
       When the process is driven
       Then the process rests at "design.triage"
       And the resting step's content contains "## Risk Tier"

package/src/workflows/feature/seam-rejection.feature

@@ -2,7 +2,7 @@
 @inmem
 Feature: rejected naive passes walk one fixed path through the steer gate to idle
 
-  With GTD_FLAG_NAIVE_PASS on and GTD_FLAG_RISK_TIER off, three rejected naive
+  With GTD_FLAG_RISK_TIER off, three rejected naive
   passes rest at the steer gate, where a human edit restarts the seam and the
   steered rerun lands. The flags are pinned so the per-flag test runs never
   move it off its path.
@@ -10,7 +10,6 @@ Feature: rejected naive passes walk one fixed path through the steer gate to idl
   Background:
     Given the shipped workflows, as gtd install scaffolds them
     And an environment variable "GTD_QUALITYREVIEWS" set to ""
-    And an environment variable "GTD_FLAG_NAIVE_PASS" set to "1"
     And an environment variable "GTD_FLAG_RISK_TIER" set to "0"
     And a file "src/calc.ts" with:
       """
@@ -136,7 +135,6 @@ Feature: rejected naive passes walk one fixed path through the steer gate to idl
       gtd(human): build.review.await-review → build.review.closing
       gtd(check): build.review.closing → idle
       """
-    And the commit "gtd(human): idle → unwind" has a body containing "Gtd-Var: flag_naive_pass=1"
     And the commit "gtd(human): idle → unwind" has a body containing "Gtd-Var: flag_risk_tier=0"
     And the commit "gtd(agent): seam.naive → seam.judge" has a body containing "Gtd-Kept:"
     And the commit "gtd(judge): seam.judge → seam.naive" has a body containing "\"answer\":\"reject\""

package/src/workflows/feature/seam.feature

@@ -1,10 +1,9 @@
 @inmem
-Feature: the naive pass and seam judge replace architecture behind GTD_FLAG_NAIVE_PASS
+Feature: the naive pass and seam judge decide how a feature is built
 
-  With the flag on, after design a naive pass is set aside under a kept ref, the
-  judge reads its diff, and the process either implements straight away, refactors
-  the seam first behind a human approval, or retries the naive pass. With the flag
-  off nothing changes: architecture and packages run as before.
+  After design a naive pass is set aside under a kept ref, the judge reads its
+  diff, and the process either implements straight away, refactors the seam
+  first behind a human approval, or retries the naive pass.
 
   Background:
     Given the shipped workflows, as gtd install scaffolds them
@@ -17,7 +16,6 @@ Feature: the naive pass and seam judge replace architecture behind GTD_FLAG_NAIV
       }
       """
     And an environment variable "GTD_QUALITYREVIEWS" set to ""
-    And an environment variable "GTD_FLAG_NAIVE_PASS" set to "1"
     And a file "src/calc.ts" with:
       """
       export const add = (a: number, b: number) => a + b

package/src/workflows/feature/seam.ts

@@ -17,6 +17,7 @@ import { coder, FEEDBACK, REQUIREMENTS } from "../shared/steps.js"
 import * as t from "../shared/text.js"
 import { defaults } from "../shared/vars.js"
 import { refactorStrays, reviewedRefactor } from "./refactor.js"
+import { flagModeOn } from "../shared/options.js"
 import type { RiskTier } from "./tier.js"
 
 /** Naive-pass retries after a `reject`; the next `reject` rests at `seam.steer`. */
@@ -71,6 +72,7 @@ export function implement(
         options.tier,
         options.amend,
         options.afterRefactor,
+        flagModeOn(),
       ),
       {
         label: "Implementing",

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

@@ -21,26 +21,6 @@ describe("the feature workflow's own steps declare skills — a step's rendered
     expect(prompt).toContain("spec-driven-development, planning-and-task-breakdown")
   })
 
-  it("author carries architecture.author's bundled skills", async () => {
-    const prompt = agentPrompt(await capture(() => steps.author(), "architecture"))
-    expect(prompt).toContain("api-and-interface-design, documentation-and-adrs, ponytail")
-  })
-
-  it("decompose carries architecture.decompose's bundled skills", async () => {
-    const prompt = agentPrompt(await capture(() => steps.decompose(), "architecture"))
-    expect(prompt).toContain("incremental-implementation, planning-and-task-breakdown")
-  })
-
-  it("build carries packages.item.building's bundled skills", async () => {
-    const prompt = agentPrompt(await capture(() => steps.build("pkg"), "packages.item"))
-    expect(prompt).toContain("test-driven-development, incremental-implementation")
-  })
-
-  it("fixSuite carries packages.item.fix.suite's bundled skills", async () => {
-    const prompt = agentPrompt(await capture(() => steps.fixSuite(), "packages.item"))
-    expect(prompt).toContain("debugging-and-error-recovery")
-  })
-
   it("collecting carries build.review.collecting's bundled skills", async () => {
     const prompt = agentPrompt(await capture(() => steps.collecting("capture"), "build"))
     expect(prompt).toContain("code-review-and-quality")

package/src/workflows/feature/steps.ts

@@ -2,15 +2,7 @@ import { human, run, scope, type ScopeAccess } from "../../flows/index.js"
 import type { CollectRound } from "../../workflow-kit/index.js"
 import * as t from "../shared/text.js"
 import { riskTierOn } from "./tier.js"
-import {
-  ARCHITECTURE,
-  coder,
-  FEEDBACK,
-  planner,
-  REFACTOR,
-  REQUIREMENTS,
-  REVIEW,
-} from "../shared/steps.js"
+import { planner, REFACTOR, REQUIREMENTS, REVIEW } from "../shared/steps.js"
 
 /** Turn the sketch since `base` into `.gtd/REQUIREMENTS.md`. */
 export const triage = (base: string): Promise<void> =>
@@ -22,22 +14,6 @@ export const triage = (base: string): Promise<void> =>
     system: t.designSystem(),
   })
 
-export const author = (): Promise<void> =>
-  t.agentWithSkills("author", t.architectureAuthorPrompt(), {
-    label: "Refining the technical plan",
-    file: ARCHITECTURE,
-    mode: "qa",
-    model: planner(),
-    system: t.architectSystem(),
-  })
-
-export const decompose = (): Promise<void> =>
-  t.agentWithSkills("decompose.decomposing", t.architectureDecomposePrompt(), {
-    label: "Decomposing into packages",
-    model: planner(),
-    system: t.architectSystem(),
-  })
-
 export const answerProductQuestions = (): Promise<void> =>
   human("gate.answer", {
     message: t.designGateAnswerMessage(riskTierOn()),
@@ -47,30 +23,6 @@ export const answerProductQuestions = (): Promise<void> =>
     acceptClean: true,
   })
 
-export const answerTechnicalQuestions = (): Promise<void> =>
-  human("gate.answer", {
-    message: t.architectureGateAnswerMessage(),
-    label: "Awaiting your technical answers",
-    file: ARCHITECTURE,
-    mode: "qa",
-    acceptClean: true,
-  })
-
-export const build = (pkg: string): Promise<void> =>
-  t.agentWithSkills("building", t.packagesItemBuildingPrompt(pkg), {
-    label: "Building",
-    model: coder(),
-    system: t.builderSystem(),
-  })
-
-export const fixSuite = (): Promise<void> =>
-  t.agentWithSkills("fix.suite.fixing", t.packagesItemFixSuitePrompt(), {
-    label: "Fixing the check",
-    file: FEEDBACK,
-    model: coder(),
-    system: t.builderSystem(),
-  })
-
 /** Answer every `question` note inline in `.gtd/REVIEW.md`. */
 export const answerReviewQuestions = (notes: readonly t.NoteInput[]): Promise<void> =>
   t.agentWithSkills("review.answer-review-questions", t.buildReviewAnswerQuestionsPrompt(notes), {

package/src/workflows/feature/{seam-pin.feature → tier-pin.feature}

@@ -1,8 +1,8 @@
 @inmem
-Feature: a process keeps the path it started on when GTD_FLAG_NAIVE_PASS changes
+Feature: a process keeps the risk tier setting it started with when GTD_FLAG_RISK_TIER changes
 
   The flag is recorded in the opening commit, so flipping it mid-process never
-  moves a process between the architecture path and the naive-pass path.
+  changes whether the process asks for and records a risk tier.
 
   Background:
     Given the shipped workflows, as gtd install scaffolds them
@@ -29,24 +29,30 @@ Feature: a process keeps the path it started on when GTD_FLAG_NAIVE_PASS changes
       ### Subtract
 
       - `subtract(a, b)` returns `a - b`.
+
+      ## Risk Tier
+
+      low — a new pure function
       """
 
-  Scenario: a process started with the flag off stays on architecture after the flag turns on
-    Given the ambient environment variable "GTD_FLAG_NAIVE_PASS" is "0"
+  Scenario: a process started with the flag off records no tier after the flag turns on
+    Given the ambient environment variable "GTD_FLAG_RISK_TIER" is "0"
     When the process is driven
     Then the process rests at "design.gate.answer"
-    Given the ambient environment variable "GTD_FLAG_NAIVE_PASS" is "1"
+    Given the ambient environment variable "GTD_FLAG_RISK_TIER" is "1"
     And step "design.gate.answer" changes nothing
     When the process is driven
-    Then the process rests at "architecture.author"
-    And the commit "gtd(human): idle → unwind" has a body containing "Gtd-Var: flag_naive_pass=0"
+    Then the process rests at "seam.naive"
+    And the commit "gtd(human): idle → unwind" has a body containing "Gtd-Var: flag_risk_tier=0"
+    And no commit body contains "Gtd-Mark:"
 
-  Scenario: a process started with the flag on stays on the naive pass after the flag turns off
-    Given the ambient environment variable "GTD_FLAG_NAIVE_PASS" is "1"
+  Scenario: a process started with the flag on records its tier after the flag turns off
+    Given the ambient environment variable "GTD_FLAG_RISK_TIER" is "1"
     When the process is driven
     Then the process rests at "design.gate.answer"
-    Given the ambient environment variable "GTD_FLAG_NAIVE_PASS" is "0"
+    Given the ambient environment variable "GTD_FLAG_RISK_TIER" is "0"
     And step "design.gate.answer" changes nothing
     When the process is driven
     Then the process rests at "seam.naive"
-    And the commit "gtd(human): idle → unwind" has a body containing "Gtd-Var: flag_naive_pass=1"
+    And the commit "gtd(human): idle → unwind" has a body containing "Gtd-Var: flag_risk_tier=1"
+    And the last commit body contains "Gtd-Mark: risk-tier=low"

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

@@ -9,11 +9,9 @@ const withVars = (vars: Record<string, string>) => installContext(fixtureContext
 
 describe("riskTierOn", () => {
   it.each([
-    [{ flag_naive_pass: "1", flag_risk_tier: "1" }, true],
-    [{ flag_naive_pass: "1", flag_risk_tier: "0" }, false],
-    [{ flag_naive_pass: "0", flag_risk_tier: "1" }, false],
-    [{ flag_naive_pass: "0", flag_risk_tier: "0" }, false],
-    [{ flag_naive_pass: "1" }, false],
+    [{ flag_risk_tier: "1" }, true],
+    [{ flag_risk_tier: "0" }, false],
+    [{}, false],
   ])("%j is %s", (vars, expected) => {
     withVars(vars)
     expect(riskTierOn()).toBe(expected)

package/src/workflows/feature/tier.ts

@@ -4,8 +4,8 @@ import { REQUIREMENTS } from "../shared/steps.js"
 
 export type RiskTier = "low" | "high"
 
-/** GTD_FLAG_NAIVE_PASS, GTD_FLAG_RISK_TIER: both pinned at process start, so a mid-process flip never moves a process. */
-export const riskTierOn = (): boolean => vars.flag_naive_pass === "1" && vars.flag_risk_tier === "1"
+/** GTD_FLAG_RISK_TIER: pinned at process start, so a mid-process flip never moves a process. */
+export const riskTierOn = (): boolean => vars.flag_risk_tier === "1"
 
 const sectionLines = (
   text: string | undefined,

package/src/workflows/index.ts

@@ -22,7 +22,6 @@ export {
 export { lensesOf } from "../workflow-kit/index.js"
 export * from "./feature/steps.js"
 export * from "./feature/planning.js"
-export * from "./feature/packages.js"
 export * from "./feature/refactor.js"
 export * from "./feature/seam.js"
 export {

package/src/workflows/shared/access.test.ts

@@ -10,8 +10,6 @@ describe("the bundled workflow's access export", () => {
   it("restricts writes on planning, review and lens scopes only", () => {
     expect(bundled).toEqual({
       design: { write: [] },
-      architecture: { write: [".gtd/REQUIREMENTS.md"] },
-      "architecture.decompose": { write: [".gtd/packages/**", ".gtd/ARCHITECTURE.md"] },
       "build.review": { write: [".gtd/REVIEW.md"] },
       "build.review.defer": { write: [".gtd/TODO.md"] },
       "build.review.fix.nits": {},

package/src/workflows/shared/access.ts

@@ -12,8 +12,6 @@ export const access = (
   vars: Readonly<Record<string, string>>,
 ): Readonly<Record<string, ScopeAccess>> => ({
   design: { write: [] },
-  architecture: { write: [".gtd/REQUIREMENTS.md"] },
-  "architecture.decompose": { write: [".gtd/packages/**", ".gtd/ARCHITECTURE.md"] },
   "build.review": { write: [".gtd/REVIEW.md"] },
   "build.review.defer": { write: [".gtd/TODO.md"] },
   // `{}` reopens what `build.review` restricted: the fixes edit code.

package/src/workflows/shared/options.test.ts

@@ -23,11 +23,22 @@ describe("the shipped workflow factories", () => {
       expect(defaults).toHaveProperty(`flag_${name}`, defaultOn ? "1" : "0")
   })
 
-  it("declare seamAllow and flag_naive_pass defaults; flag_naive_pass is no option", () => {
-    expect(Object.keys(ShippedOptions.fields)).not.toContain("flag_naive_pass")
+  it("declare a seamAllow default a factory option overrides", () => {
     expect(configure(feature({ seamAllow: "docs/**" }))).toMatchObject({
       ok: true,
-      build: { defaults: { seamAllow: "docs/**", flag_naive_pass: "0" } },
+      build: { defaults: { seamAllow: "docs/**" } },
+    })
+  })
+
+  it("default flagMode on and lay a boolean factory option over it as a string", () => {
+    expect(defaults.flagMode).toBe("true")
+    expect(configure(feature({ flagMode: false }))).toMatchObject({
+      ok: true,
+      build: { defaults: { flagMode: "false" } },
+    })
+    expect(configure(review({ flagMode: true }))).toMatchObject({
+      ok: true,
+      build: { defaults: { flagMode: "true" } },
     })
   })