@pmelab/gtd 29.3.1 → 30.0.0

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/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,10 @@ 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/**" } },
     })
   })
 

package/src/workflows/shared/prose.ts

@@ -73,12 +73,6 @@ classified list of concerns, and hold the running product conversation
 with the human it depends on. You are read by that human — write
 plainly — not by a parser, except where a state says otherwise.`
 
-export const architectPersona = `You are the technical planning voice in gtd's build pipeline: take over
-once product concerns are settled, reading them cold, no carried
-conversation. Work out the *how* per concern — structure, data models,
-tech-stack choices, error handling — raise the technical open
-questions, then pin interfaces and tests in the architecture document.`
-
 export const reviewerPersona = `You are the independent reviewing mind in gtd's build pipeline —
 deliberately separate from whoever wrote the code, with no attachment
 to it. Write a structured review document grouping a diff into
@@ -88,11 +82,10 @@ and, when asked, fix the small nits the human flagged and the risks
 you marked yourself, each in one batch.
 Beyond that you never fix or build anything yourself.`
 
-export const builderPersona = `You are the TDD implementer in gtd's build pipeline: build one
-package's declared scope end to end, tests first, and come back to fix
+export const builderPersona = `You are the TDD implementer in gtd's build pipeline: build the
+task in front of you end to end, tests first, and come back to fix
 things when redirected — a failing check, or reviewer feedback. Stay
-strictly inside the package in front of you; never touch another
-package's files or code outside the task. Treat feedback as the
+strictly inside that task; never touch code outside it. Treat feedback as the
 diagnosis to act on; discard it, with reason, only when simply wrong.`
 
 export const finisherPersona = `You are the closing identity in gtd's build pipeline — the last coder
@@ -136,7 +129,7 @@ export const questionBar = `- The goal is shared understanding, not a quota or a
   settle it (treat an explicit \`.gtd/TODO.md\` directive, a committed
   hand-edit, or — on a loop-back lap — the human's own review-round
   edit/note as settled); or it would be expensive to undo once
-  packages are written
+  code is written
 - Where no gap in shared understanding exists, decide it yourself:
   record it under \`## Answered Questions\` as
   \`### <the point phrased as a question>\`

package/src/workflows/shared/reserved.ts

@@ -1,6 +1,6 @@
 import type { Summary } from "../../flows/index.js"
 import * as t from "./text.js"
-import { ARCHITECTURE, REQUIREMENTS, REVIEW } from "./steps.js"
+import { REQUIREMENTS, REVIEW } from "./steps.js"
 
 export { defaults, envDefaults } from "./vars.js"
 export { skills } from "./skills.js"
@@ -8,4 +8,4 @@ export { access } from "./access.js"
 
 export const summary: Summary = t.summaryPrompt
 
-export const steering = { [REQUIREMENTS]: "qa", [ARCHITECTURE]: "qa", [REVIEW]: "review" }
+export const steering = { [REQUIREMENTS]: "qa", [REVIEW]: "review" }

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

@@ -14,11 +14,6 @@ describe("the bundled workflow's skills export", () => {
     expect(Object.keys(bundled).sort()).toEqual(
       [
         "design",
-        "architecture",
-        "architecture.decompose",
-        "packages.item",
-        "packages.item.fix.suite",
-        "packages.item.health",
         "build",
         "build.health",
         "build.fix.quality",
@@ -54,7 +49,6 @@ describe("the bundled workflow's skills export", () => {
 
   it("keys no group scope or non-agent step", () => {
     for (const group of [
-      "packages.item.fix",
       "build.fix",
       "build.review.fix",
       "build.quality",
@@ -70,11 +64,6 @@ describe("the bundled workflow's skills export", () => {
   it("pins each scope's list", () => {
     expect(bundled).toMatchObject({
       design: ["spec-driven-development", "planning-and-task-breakdown"],
-      architecture: ["api-and-interface-design", "documentation-and-adrs", "ponytail"],
-      "architecture.decompose": ["incremental-implementation", "planning-and-task-breakdown"],
-      "packages.item": ["test-driven-development", "incremental-implementation"],
-      "packages.item.fix.suite": ["debugging-and-error-recovery"],
-      "packages.item.health": ["debugging-and-error-recovery"],
       build: ["debugging-and-error-recovery"],
       "build.health": ["debugging-and-error-recovery"],
       "build.fix.quality": ["incremental-implementation", "code-simplification"],

package/src/workflows/shared/skills.ts

@@ -4,7 +4,7 @@ import { lensesOf, lensScopes } from "../../workflow-kit/index.js"
 // The bundled workflow's default skills, one list per SCOPE, keyed by the
 // scope's full name — the same name a `config.skills` entry addresses. A
 // nested scope with no entry inherits its parent's; group scopes
-// (`packages.item.fix`, `build.fix`, `build.review.fix`, `build.quality`, `refactor.fix`) run
+// (`implement.fix`, `build.fix`, `build.review.fix`, `build.quality`, `refactor.fix`) run
 // no turn and carry no key. `build.quality.<lens>` is one key per
 // `qualityReviews` entry, so the export is a function of the vars.
 export const skills = (
@@ -20,12 +20,6 @@ export const skills = (
   }
   return {
     design: ["spec-driven-development", "planning-and-task-breakdown"],
-    architecture: ["api-and-interface-design", "documentation-and-adrs", "ponytail"],
-    "architecture.decompose": ["incremental-implementation", "planning-and-task-breakdown"],
-    "packages.item": ["test-driven-development", "incremental-implementation"],
-    "packages.item.fix.suite": ["debugging-and-error-recovery"],
-    // `health` is shared code reached from both `packages.item` and `build`.
-    "packages.item.health": ["debugging-and-error-recovery"],
     // `fix`'s turn over the notes a review handed on.
     notes: ["incremental-implementation", "code-simplification"],
     // The refactor's own turns load `ponytail`; `refactor.build.*` carry the `build.*` lists.
@@ -34,11 +28,10 @@ export const skills = (
     refactor: ["ponytail"],
     "refactor.fix.suite": ["ponytail", "debugging-and-error-recovery"],
     "refactor.health": ["ponytail", "debugging-and-error-recovery"],
-    // Declared whether or not `naive_pass` is on, so a config keyed on them keeps
-    // loading when the flag flips off.
     seam: ["incremental-implementation"],
     implement: ["test-driven-development", "incremental-implementation"],
     "implement.fix.suite": ["debugging-and-error-recovery"],
+    // `health` is shared code reached from both `implement` and `build`.
     "implement.health": ["debugging-and-error-recovery"],
     ...lensScopes(
       "build.quality",

package/src/workflows/shared/steps.ts

@@ -10,7 +10,6 @@ import * as t from "./text.js"
 export const FEEDBACK = ".gtd/FEEDBACK.md"
 export const ESCALATION = ".gtd/ESCALATION.md"
 export const REQUIREMENTS = ".gtd/REQUIREMENTS.md"
-export const ARCHITECTURE = ".gtd/ARCHITECTURE.md"
 export const REVIEW = ".gtd/REVIEW.md"
 export const REFACTOR = ".gtd/REFACTOR.md"
 export const QUALITY = ".gtd/QUALITY.md"

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

@@ -8,10 +8,6 @@ import {
   deferNotesScript,
   buildReviewReviewingPrompt,
   agentWithSkills,
-  architectureAuthorPrompt,
-  architectureDecomposePrompt,
-  packagesItemBuildingPrompt,
-  architectureGateAnswerMessage,
   buildFixPrompt,
   buildFixQualityPrompt,
   buildReviewFixNitsPrompt,
@@ -167,23 +163,6 @@ describe("the shared open-question instruction", () => {
     expect(prompt).toContain("one to four bullets, free-form")
     expect(prompt).toContain("labels, no fixed bullet count")
   })
-
-  it("is the identical block, full text, in both the design gate and the architecture gate", () => {
-    // Anchored at questionBar's own first and last lines (not a bullet
-    // partway in) so the whole ~3.4k-char block is compared — a divergence
-    // anywhere inside, including its opening bullets, fails this test the
-    // moment either gate's copy drifts from the shared source.
-    const start = "The goal is shared understanding, not a quota or an empty section"
-    const end = "Never tick a box yourself — the human ticks exactly one per question"
-    const sharedBlock = (prompt: string): string =>
-      prompt.slice(prompt.indexOf(start), prompt.indexOf(end) + end.length)
-
-    const designBlock = sharedBlock(renderText(() => designTriagePrompt("base")))
-    const architectureBlock = sharedBlock(renderText(() => architectureAuthorPrompt()))
-
-    expect(designBlock.length).toBeGreaterThan(3000)
-    expect(designBlock).toBe(architectureBlock)
-  })
 })
 
 describe("answered-entry footnote exception", () => {
@@ -196,32 +175,6 @@ describe("answered-entry footnote exception", () => {
     expect(prompt).not.toContain("never re-open a settled")
     expect(prompt).not.toContain("never re-open one")
   })
-
-  it("is in the architecture return lap and keeps the first-lap settled line", () => {
-    const prompt = renderText(() => architectureAuthorPrompt())
-    expect(prompt).toContain(`entry stays settled, except one ${exception}`)
-    expect(prompt).not.toContain("never re-open a settled")
-    expect(prompt).toMatch(/treat every decision[\s\S]*as settled — never re-open it/)
-  })
-})
-
-describe("architectureAuthorPrompt", () => {
-  it("qualifies the PERMISSIVE default with a settled-requirements exception", () => {
-    const prompt = renderText(() => architectureAuthorPrompt())
-    expect(prompt).toContain("is an open question, not a")
-  })
-  it("names the four content sections in order and the entry forms", () => {
-    const prompt = renderText(() => architectureAuthorPrompt())
-    const at = ["## Interfaces", "## Call Stacks", "## E2E Scenarios", "## Unit Tests"].map((h) =>
-      prompt.indexOf(h),
-    )
-    expect(at.every((i) => i >= 0)).toBe(true)
-    expect([...at].sort((a, b) => a - b)).toEqual(at)
-    expect(prompt).toContain("- unit: <path>")
-    expect(prompt).toContain("- e2e: <path>")
-    expect(prompt).toContain("- chore: <path>")
-    expect(prompt).toContain("No e2e change.")
-  })
 })
 
 describe("buildFixPrompt", () => {
@@ -688,16 +641,11 @@ Print the closing message and stop — this writes nothing itself.
   it("treats a flag-on process with no tier mark as high", () => {
     const prompt = summaryPrompt({
       ...base,
-      vars: { flag_risk_tier: "1", flag_naive_pass: "1" },
+      vars: { flag_risk_tier: "1" },
     })
     expect(prompt).toContain("Risk tier: high")
   })
 
-  it("marks no tier when the risk-tier flag is on without naive-pass", () => {
-    const prompt = summaryPrompt({ ...base, vars: { flag_risk_tier: "1", flag_naive_pass: "0" } })
-    expect(prompt).not.toContain("Risk tier")
-  })
-
   it("says the human left no comment or edit this process when humanCommits is empty", () => {
     expect(summaryPrompt(base)).toContain(
       "The human left no comment or edit this process —\nevery commit is machine-authored.",
@@ -812,10 +760,9 @@ describe("buildReviewCollectingPrompt without a refactor", () => {
 })
 
 describe("the thread rules", () => {
-  it("are in all three answering prompts and no longer say a footnote is human input only", () => {
+  it("are in both answering prompts and no longer say a footnote is human input only", () => {
     const prompts = renderText(() => [
       designTriagePrompt("base"),
-      architectureAuthorPrompt(),
       buildReviewCollectingPrompt("capture"),
     ])
     for (const prompt of prompts) {
@@ -837,13 +784,9 @@ describe("code threads in prompts", () => {
     { path: "src/b.ts", line: 9, waitingOn: "human", first: "done", faults: [] },
   ] as const
 
-  it("lists the code threads waiting on the agent in the three gates", () => {
+  it("lists the code threads waiting on the agent in both gates", () => {
     const prompts = renderText(
-      () => [
-        designTriagePrompt("base"),
-        architectureAuthorPrompt(),
-        buildReviewCollectingPrompt("capture"),
-      ],
+      () => [designTriagePrompt("base"), buildReviewCollectingPrompt("capture")],
       { codeThreads: waiting },
     )
     for (const prompt of prompts) {
@@ -866,7 +809,6 @@ describe("code threads in prompts", () => {
   it("teaches the syntax once in the footnote rules", () => {
     for (const message of renderText(() => [
       designGateAnswerMessage(),
-      architectureGateAnswerMessage(),
       buildReviewAwaitReviewMessage("base"),
     ])) {
       expect(message).toContain("gtd check --open-threads")
@@ -1306,45 +1248,3 @@ it("asks the refactor review, and only it, to flag a test-file hunk that is more
   expect(refactor).toContain("give any test-file hunk its own chunk")
   expect(first).not.toContain("give any test-file hunk its own chunk")
 })
-
-describe("architectureDecomposePrompt tests rule", () => {
-  it("states one rule: write-or-change, relied-on marked (unchanged), no other untouched", () => {
-    const prompt = renderText(() => architectureDecomposePrompt()).replace(/\s+/g, " ")
-    expect(prompt).toContain("List every test the package writes or changes")
-    expect(prompt).toContain("also list an existing test a criterion relies on")
-    expect(prompt).toContain("list no other untouched test")
-    expect(prompt).toContain("- unit: \\`<path>\\` (unchanged)")
-    expect(prompt).toContain("- e2e: \\`<path>\\` (unchanged)")
-    expect(prompt).toContain("only decompose places the marker")
-  })
-})
-
-describe("packagesItemBuildingPrompt", () => {
-  const prompt = renderText(() => packagesItemBuildingPrompt(".gtd/packages/01-a.md")).replace(
-    /\s+/g,
-    " ",
-  )
-  it("carries the evidence kinds and the already-done check", () => {
-    for (const w of [
-      "evidence per criterion",
-      "commit, file, symbol, issue URL, or command output — and change nothing else",
-      "it was not already done",
-      "via `gh`",
-    ])
-      expect(prompt).toContain(w)
-  })
-  it("states the already-met, no-tests out-of-tree and implement-normally rules flat", () => {
-    expect(prompt).toContain("Every criterion already met in the tree")
-    expect(prompt).toContain("No `## Tests` and work outside the tree")
-    expect(prompt).toContain(
-      "Any other package**: implement normally, write no `.gtd/SATISFIED.md`",
-    )
-  })
-  it("says an (unchanged) line must exist and stay untouched", () => {
-    expect(prompt).toContain("(unchanged)")
-    expect(prompt).toContain("stay untouched")
-    expect(prompt).toContain("creating, editing or deleting it refuses the turn")
-    expect(prompt).toContain("must exist before the turn")
-    expect(prompt).toContain("stop without creating it")
-  })
-})