@pmelab/gtd 58.0.0 → 58.1.0

package/dist/workflows/cleanup.mjs

@@ -1,6 +1,6 @@
 import { agent, changes, glob, read, refuse, scope, start, vars } from "../flows.mjs";
-import { It as registryEntry, Lt as requireFlag, Pt as flagReferences, St as envDefaults, a as flagShipped, i as skills, n as summary, r as access, t as steering, x as coder, xt as defaults } from "../chunks/reserved-BzAFXuz_.mjs";
-import { a as refusePendingHandOff, r as handOffTail } from "../chunks/tail-JQzB21Q3.mjs";
+import { It as registryEntry, Lt as requireFlag, Pt as flagReferences, St as envDefaults, a as flagShipped, i as skills, n as summary, r as access, t as steering, x as coder, xt as defaults } from "../chunks/reserved-lMYCaapY.mjs";
+import { a as refusePendingHandOff, r as handOffTail } from "../chunks/tail-Bd7L940p.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, restoreScript, revertScript, run, scope, start, vars, wrote } from "../flows.mjs";
 import { lensEntriesOf } from "../workflow-kit.mjs";
-import { B as designSystem, Ct as testFileGlobs, D as planner, F as clearCriteriaMessage, I as clearCriteriaQuestion, J as implementBuildingPrompt, L as criterionLine, M as buildReviewCollectingPrompt, P as builderSystem, Q as naiveTryMessage, R as designGateAnswerMessage, S as collecting, St as envDefaults, V as designTriagePrompt, X as naiveApplyFailure, Y as implementFixPrompt, Z as naivePassPrompt, _ as REQUIREMENTS, _t as unwindFailedMessage, at as refactorFixPrompt, d as healthy, et as normaliseCriteriaScript, f as FEEDBACK, g as REFACTOR_NOTE, gt as startGateBlockedMessage, h as REFACTOR, ht as seamSteerMessage, i as skills, it as refactorDroppedRefusal, j as agentWithSkills, l as gate, mt as seamByHandMessage, n as summary, o as naiveTestOn, ot as refactorNotePrompt, p as NAIVE_TEST, pt as reviewerSystem, q as idleMessage, r as access, rt as refactorBuildingPrompt, s as shipped, st as refactorRules, t as steering, vt as unwindFailure, w as fixNits, x as coder, xt as defaults, y as answerReviewQuestions, z as designImpactPrompt } from "../chunks/reserved-BzAFXuz_.mjs";
-import { a as refusePendingHandOff } from "../chunks/tail-JQzB21Q3.mjs";
-import { n as isCode$1, r as reUnwind, t as buildTail } from "../chunks/review-CVqRS0mc.mjs";
+import { B as designSystem, Ct as testFileGlobs, D as planner, F as clearCriteriaMessage, I as clearCriteriaQuestion, J as implementBuildingPrompt, L as criterionLine, M as buildReviewCollectingPrompt, P as builderSystem, Q as naiveTryMessage, R as designGateAnswerMessage, S as collecting, St as envDefaults, V as designTriagePrompt, X as naiveApplyFailure, Y as implementFixPrompt, Z as naivePassPrompt, _ as REQUIREMENTS, _t as unwindFailedMessage, at as refactorFixPrompt, d as healthy, et as normaliseCriteriaScript, f as FEEDBACK, g as REFACTOR_NOTE, gt as startGateBlockedMessage, h as REFACTOR, ht as seamSteerMessage, i as skills, it as refactorDroppedRefusal, j as agentWithSkills, l as gate, mt as seamByHandMessage, n as summary, o as naiveTestOn, ot as refactorNotePrompt, p as NAIVE_TEST, pt as reviewerSystem, q as idleMessage, r as access, rt as refactorBuildingPrompt, s as shipped, st as refactorRules, t as steering, vt as unwindFailure, w as fixNits, x as coder, xt as defaults, y as answerReviewQuestions, z as designImpactPrompt } from "../chunks/reserved-lMYCaapY.mjs";
+import { a as refusePendingHandOff } from "../chunks/tail-Bd7L940p.mjs";
+import { n as isCode$1, r as reUnwind, t as buildTail } from "../chunks/review-Bfaror38.mjs";
 import { posix } from "node:path";
 //#region src/workflows/shared/fences.ts
 const markFenced = (text) => {

package/dist/workflows/fix.mjs

@@ -1,7 +1,7 @@
 import { read, refuse, removeScript, run, scope, start } from "../flows.mjs";
 import { b as reviewNotes, x as reviewRisks } from "../chunks/steering-CCNvngSH.mjs";
-import { G as fixReviewNotesPrompt, St as envDefaults, W as finisherSystem, c as baseline, i as skills, j as agentWithSkills, n as summary, r as access, s as shipped, t as steering, v as REVIEW, x as coder, xt as defaults } from "../chunks/reserved-BzAFXuz_.mjs";
-import { r as handOffTail, t as REVIEW_ORIG } from "../chunks/tail-JQzB21Q3.mjs";
+import { G as fixReviewNotesPrompt, St as envDefaults, W as finisherSystem, c as baseline, i as skills, j as agentWithSkills, n as summary, r as access, s as shipped, t as steering, v as REVIEW, x as coder, xt as defaults } from "../chunks/reserved-lMYCaapY.mjs";
+import { r as handOffTail, t as REVIEW_ORIG } from "../chunks/tail-Bd7L940p.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 { St as envDefaults, W as finisherSystem, _ as REQUIREMENTS, dt as reviewGateBlockedMessage, i as skills, j as agentWithSkills, l as gate, n as summary, r as access, s as shipped, t as steering, ut as reviewFeedbackFixPrompt, x as coder, xt as defaults } from "../chunks/reserved-BzAFXuz_.mjs";
-import { a as refusePendingHandOff } from "../chunks/tail-JQzB21Q3.mjs";
-import { r as reUnwind, t as buildTail } from "../chunks/review-CVqRS0mc.mjs";
+import { St as envDefaults, W as finisherSystem, _ as REQUIREMENTS, dt as reviewGateBlockedMessage, i as skills, j as agentWithSkills, l as gate, n as summary, r as access, s as shipped, t as steering, ut as reviewFeedbackFixPrompt, x as coder, xt as defaults } from "../chunks/reserved-lMYCaapY.mjs";
+import { a as refusePendingHandOff } from "../chunks/tail-Bd7L940p.mjs";
+import { r as reUnwind, t as buildTail } from "../chunks/review-Bfaror38.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 { Ft as flipProblems, Lt as requireFlag, Nt as flagMentions, St as envDefaults, a as flagShipped, f as FEEDBACK, i as skills, l as gate, n as summary, r as access, t as steering, x as coder, xt as defaults } from "../chunks/reserved-BzAFXuz_.mjs";
+import { Ft as flipProblems, Lt as requireFlag, Nt as flagMentions, St as envDefaults, a as flagShipped, f as FEEDBACK, i as skills, l as gate, n as summary, r as access, t as steering, x as coder, xt as defaults } from "../chunks/reserved-lMYCaapY.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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pmelab/gtd",
-  "version": "58.0.0",
+  "version": "58.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": {
@@ -66,6 +66,11 @@
       "types": "./src/testkit/index.ts",
       "source": "./src/testkit/index.ts",
       "default": "./dist/testkit.mjs"
+    },
+    "./testkit/gherkin": {
+      "types": "./src/gherkin/index.ts",
+      "source": "./src/gherkin/index.ts",
+      "default": "./dist/testkit/gherkin.mjs"
     }
   },
   "files": [
@@ -76,6 +81,8 @@
     "src/wire/",
     "src/steering/",
     "src/testkit/",
+    "src/twins/",
+    "src/gherkin/",
     "src/cli/",
     "src/edge/",
     "src/emit/",

package/src/cli/Cli.test.ts

@@ -25,7 +25,7 @@ import {
 import { EXIT_CODES, EXIT_RUNTIME_ERROR, EXIT_USAGE_ERROR } from "../wire/index.js"
 import { Host } from "../platform/index.js"
 import { type CommandRequirements } from "./Cli.js"
-import { InMemRepo, makeCapturingCliIo, testLayers } from "../testkit/index.js"
+import { InMemRepo, makeCapturingCliIo, testLayers } from "../twins/index.js"
 import { CONFIG_SCAFFOLD } from "../install/index.js"
 
 const FLAG_NAMES = [

package/src/cli/History.test.ts

@@ -1,6 +1,6 @@
 import { Effect, Exit } from "effect"
 import { describe, expect, it } from "vitest"
-import { applyEmittedScript, InMemRepo, makeCapturingCliIo, testLayers } from "../testkit/index.js"
+import { applyEmittedScript, InMemRepo, makeCapturingCliIo, testLayers } from "../twins/index.js"
 import { runCli } from "./index.js"
 import { historyEntries, resolveHistoryRef, type HistoryEntry } from "./History.js"
 

package/src/cli/Metrics.test.ts

@@ -1,6 +1,6 @@
 import { Effect } from "effect"
 import { describe, expect, it } from "vitest"
-import { applyEmittedScript, InMemRepo, makeCapturingCliIo, testLayers } from "../testkit/index.js"
+import { applyEmittedScript, InMemRepo, makeCapturingCliIo, testLayers } from "../twins/index.js"
 import { runCli } from "./index.js"
 import { processMetrics, readProcessMetrics, totalMetrics, type MetricCommit } from "./Metrics.js"
 

package/src/cli/Program.test.ts

@@ -18,7 +18,7 @@ vi.mock("node:os", async (importOriginal) => ({
 import { runCli, type Command } from "./index.js"
 import { EXIT_USAGE_ERROR, stallDiagnosis, noopText, WIRE_SCHEMA } from "../wire/index.js"
 import { needsOf, runCommand, SelectorUsageError } from "./Program.js"
-import { InMemRepo, makeCapturingCliIo, testLayers, applyEmittedScript } from "../testkit/index.js"
+import { InMemRepo, makeCapturingCliIo, testLayers, applyEmittedScript } from "../twins/index.js"
 import {
   commitAll,
   DID_NOT_RUN_COMMENT,

package/src/cli/Strays.test.ts

@@ -1,7 +1,7 @@
 import { Effect } from "effect"
 import { describe, expect, it } from "vitest"
 import { EXIT_OK, EXIT_RUNTIME_ERROR, EXIT_STRAYS_FOUND } from "../wire/index.js"
-import { InMemRepo, makeCapturingCliIo } from "../testkit/index.js"
+import { InMemRepo, makeCapturingCliIo } from "../twins/index.js"
 import { runCli } from "./index.js"
 
 const seed = (): InMemRepo => {

package/src/edge/Edge.test.ts

@@ -31,7 +31,7 @@ import {
   makeCapturingCliIo,
   makeInMemoryWorkspaceOps,
   testLayers,
-} from "../testkit/index.js"
+} from "../twins/index.js"
 
 type Env = Readonly<Record<string, string | undefined>>
 type Files = Readonly<Record<string, string>>

package/src/emit/Emit.ts

@@ -110,7 +110,7 @@ export const emitScripts = (
   optional: assembleScript(optional),
 })
 
-/** Exported so `src/testkit/EmittedScriptRecognizer.ts` can recognize these by exact string comparison. `printf`, not `echo`, avoids shell-specific backslash-escape quirks. */
+/** Exported so `src/twins/EmittedScriptRecognizer.ts` can recognize these by exact string comparison. `printf`, not `echo`, avoids shell-specific backslash-escape quirks. */
 export const DID_NOT_RUN_COMMENT =
   "# gtd emitted this and did NOT run it — pipe it into `sh` to land the turn"
 

package/src/emit/GitScript.ts

@@ -130,7 +130,7 @@ export const fetchHistoryRefs = (remote: string): string =>
 // No test asserts the two stay byte-identical; a drift between them is a
 // review-time risk this boundary accepts.
 
-/** Marks an emitted block as print-only — mirrors `OutcomeScript.ts`'s `OUTCOME_MARKER` verbatim, so `src/testkit/EmittedScriptRecognizer.ts` recognizes either path's output identically. */
+/** Marks an emitted block as print-only — mirrors `OutcomeScript.ts`'s `OUTCOME_MARKER` verbatim, so `src/twins/EmittedScriptRecognizer.ts` recognizes either path's output identically. */
 const OUTCOME_MARKER = "# gtd: outcome (print-only)"
 
 const printfLine = (fmt: string, args: readonly string[]): string =>

package/src/emit/OutcomeScript.ts

@@ -19,7 +19,7 @@ const printfLine = (fmt: string, args: readonly string[]): string =>
 
 /**
  * Marks an emitted block as print-only. Load-bearing beyond documentation:
- * `src/testkit/EmittedScriptRecognizer.ts` recognizes an outcome block by
+ * `src/twins/EmittedScriptRecognizer.ts` recognizes an outcome block by
  * this line, and a bare `printf` would be ambiguous with a workflow
  * `script:` command that happens to start the same way.
  */

package/src/{testkit/steps.ts → gherkin/index.ts}

@@ -11,7 +11,15 @@ import {
 } from "quickpickle"
 import assert from "node:assert"
 import type { TestContext } from "vitest"
-import { runsItsScript, testRepo, type Actor, type TestRepo } from "./TestRepo.js"
+import { resolve } from "node:path"
+import { pathToFileURL } from "node:url"
+import {
+  runsItsScript,
+  testRepo,
+  type Actor,
+  type TestRepo,
+  type WorkflowModule,
+} from "../testkit/index.js"
 
 export class TestkitWorld extends QuickPickleWorld {
   // Re-declared: under `nodenext` a subclass loses the base's members from TS's view.
@@ -21,7 +29,7 @@ export class TestkitWorld extends QuickPickleWorld {
     super(context, info)
   }
 
-  private gtdConfig: string | undefined = undefined
+  private gtdConfig: WorkflowModule | undefined = undefined
   private env: Record<string, string> = {}
   /** Per step, what its scripted actor does, in the order the scenario declared it. */
   actions = new Map<string, Actor[]>()
@@ -33,7 +41,7 @@ export class TestkitWorld extends QuickPickleWorld {
   // The repository is created on first use and keeps the config and
   // environment it was created with, so a later change to either is refused
   // rather than silently ignored.
-  configure(config: string): void {
+  configure(config: WorkflowModule): void {
     this.assertNotCreated("a gtd config")
     this.gtdConfig = config
   }
@@ -41,6 +49,8 @@ export class TestkitWorld extends QuickPickleWorld {
   extendConfig(tail: string): void {
     this.assertNotCreated("a gtd config")
     if (this.gtdConfig === undefined) throw new Error("no gtd config to extend")
+    if (typeof this.gtdConfig !== "string")
+      throw new Error("a gtd config from a module cannot be extended")
     this.gtdConfig += `\n${tail}`
   }
 
@@ -108,6 +118,13 @@ Given("a gtd config with:", (world: TestkitWorld, source: string) => {
   world.configure(withNewline(source))
 })
 
+// Imported by the test runner itself, from the working directory: the module and
+// the engine share one `@pmelab/gtd/flows`, and a config source string could
+// import only files written into the in-memory repository.
+Given("a gtd config from the module {string}", async (world: TestkitWorld, path: string) => {
+  world.configure({ ...(await import(pathToFileURL(resolve(path)).href)) })
+})
+
 Given("the gtd config ends with:", (world: TestkitWorld, source: string) => {
   world.extendConfig(withNewline(source))
 })

package/src/gherkin/module-config.feature

@@ -0,0 +1,24 @@
+@inmem
+Feature: a scenario runs a workflow module from disk
+
+  The test runner imports the module, so it shares the engine's runtime and
+  may import anything on disk. A config written as source can import only
+  files inside the in-memory repository.
+
+  Scenario: the module's default export starts on a change in the tree
+    Given a gtd config from the module "src/gherkin/tiny.fixture.ts"
+    And a file "IDEA.md" with:
+      """
+      a calculator
+      """
+    And step "plan" writes "PLAN.md" with:
+      """
+      add two numbers
+      """
+    When the process is driven
+    Then the process rests at "sign-off"
+    And the commit subjects end with:
+      """
+      gtd(agent): plan → sign-off
+      """
+

package/src/gherkin/tiny.fixture.ts

@@ -0,0 +1,8 @@
+import { agent, human } from "../flows/index.js"
+
+export const tiny = async (): Promise<void> => {
+  await agent("plan", "Write PLAN.md.")
+  await human("sign-off")
+}
+
+export default tiny

package/src/platform/Git.test.ts

@@ -7,11 +7,11 @@ import { Cause, Effect, Exit, Option } from "effect"
 import { describe, expect, it } from "vitest"
 import { GitService, InvalidRefError, resolvedRefOrCorrupted } from "./Git.js"
 import { Host } from "./index.js"
-import { gitTiers, runGitServiceContract, runGitScriptContract } from "../testkit/index.js"
+import { gitTiers, runGitServiceContract, runGitScriptContract } from "../twins/index.js"
 
 /**
  * The `GitOperations` contract, run identically against both the Live and
- * InMemory tiers (`src/testkit/GitTiers.ts`) — 20 operations × 2 tiers.
+ * InMemory tiers (`src/twins/GitTiers.ts`) — 20 operations × 2 tiers.
  */
 for (const makeTier of gitTiers) {
   const name = makeTier().name

package/src/platform/RetainedHistory.test.ts

@@ -1,6 +1,6 @@
 import { Effect, Option } from "effect"
 import { describe, expect, it, vi } from "vitest"
-import { strictGitOperations as stubGit } from "../testkit/index.js"
+import { strictGitOperations as stubGit } from "../twins/index.js"
 import {
   clearRetainedHistory,
   HISTORY_REF,
@@ -13,7 +13,7 @@ import {
 
 /**
  * Unit coverage for the retention edge module (`src/platform/RetainedHistory.ts`),
- * using `strictGitOperations` (`src/testkit/StrictGitOperations.ts`) as its
+ * using `strictGitOperations` (`src/twins/StrictGitOperations.ts`) as its
  * `GitOperations` double — a Proxy failing on any method a test didn't
  * override, so an unexpected call fails loudly instead of silently succeeding.
  */

package/src/platform/Workspace.test.ts

@@ -15,7 +15,7 @@ import { afterEach, describe, expect, it, vi } from "vitest"
 import { Effect, Layer } from "effect"
 import { NodeContext } from "@effect/platform-node"
 import { GitService, Host, Workspace, type WorkspaceOps } from "./index.js"
-import { InMemRepo, makeInMemoryWorkspaceOps } from "../testkit/index.js"
+import { InMemRepo, makeInMemoryWorkspaceOps } from "../twins/index.js"
 
 // `execFileSync` is a named export of the built-in `node:child_process` ESM
 // module — not configurable, so `vi.spyOn` can't wrap it directly. `vi.mock`
@@ -31,7 +31,7 @@ vi.mock("node:child_process", async (importOriginal) => {
 })
 
 /**
- * One `Workspace` tier, in the shape `src/testkit/GitTiers.ts` establishes
+ * One `Workspace` tier, in the shape `src/twins/GitTiers.ts` establishes
  * for `GitOperations`: a `root`, a way to seed working-tree/committed
  * content, and `provide` to run an Effect against this tier's `Workspace`.
  */

package/src/platform/WorktreeState.test.ts

@@ -16,7 +16,7 @@ import { NodeContext } from "@effect/platform-node"
 import { describe, expect, it } from "vitest"
 import { loopLogPath, worktreeGitDir } from "./WorktreeState.js"
 import { GitService, Host, Workspace } from "./index.js"
-import { InMemRepo, testLayers } from "../testkit/index.js"
+import { InMemRepo, testLayers } from "../twins/index.js"
 
 const provide = <A>(
   eff: Effect.Effect<A, never, Host | Workspace>,

package/src/step/planStart.test.ts

@@ -1,7 +1,7 @@
 import { Effect } from "effect"
 import { describe, expect, it } from "vitest"
 import { planStart, type StartOutcome } from "./planStart.js"
-import { InMemRepo, testLayers } from "../testkit/index.js"
+import { InMemRepo, testLayers } from "../twins/index.js"
 import { ConfigService } from "../workflow/index.js"
 
 const WORKFLOW = `import { agent, human } from "@pmelab/gtd/flows"

package/src/testkit/TestRepo.ts

@@ -2,10 +2,13 @@ import { Effect } from "effect"
 import { PassThrough } from "node:stream"
 import { runCli } from "../cli/index.js"
 import { EXIT_OK, type BeatDocument } from "../wire/index.js"
-import { makeCapturingCliIo, type CapturedCliResult } from "./cliIo.js"
-import { applyEmittedScript } from "./EmittedScriptRecognizer.js"
-import { applyFlowScript } from "./FlowScripts.js"
-import { InMemRepo } from "./InMemRepo.js"
+import {
+  applyEmittedScript,
+  applyFlowScript,
+  InMemRepo,
+  makeCapturingCliIo,
+  type CapturedCliResult,
+} from "../twins/index.js"
 
 /**
  * The repository's `gtd.config.ts`: its source, or the module itself. A module

package/src/testkit/index.ts

@@ -1,9 +1,15 @@
-export { runsItsScript, testRepo, type Actor, type TestRepo } from "./TestRepo.js"
-export { InMemRepo } from "./InMemRepo.js"
-export { makeInMemoryWorkspaceOps, testLayers, type ScriptedCommand } from "./Layers.js"
-export { makeCapturingCliIo } from "./cliIo.js"
-export { applyEmittedScript } from "./EmittedScriptRecognizer.js"
-export { gitTiers, runGitServiceContract, runGitScriptContract, type GitTier } from "./GitTiers.js"
-export { strictGitOperations } from "./StrictGitOperations.js"
-export { fakeGitOperations } from "./FakeGitOperations.js"
-export type { TestkitWorld } from "./steps.js"
+// The public testkit: runs any workflow against the real engine in an in-memory
+// repository. It imports no test runner, so any runner can use it; the
+// gherkin steps are the opt-in \`@pmelab/gtd/testkit/gherkin\`.
+export {
+  runsItsScript,
+  testRepo,
+  type Actor,
+  type Actors,
+  type DriveOptions,
+  type TestRepo,
+  type TestRepoOptions,
+  type WorkflowModule,
+} from "./TestRepo.js"
+export { InMemRepo, type CapturedCliResult } from "../twins/index.js"
+export type { TestkitWorld } from "../gherkin/index.js"

package/src/{testkit → twins}/Layers.ts

@@ -4,7 +4,7 @@
 // gtd itself spawns no subprocess at all any more (`gtd renders; the driver
 // executes`) — a mode's `format:`/`validate:` command is emitted text, never
 // run in-process. `ScriptedCommand` below still exists for the world's own
-// fake-shell interpreter (`src/testkit/EmittedScriptRecognizer.ts`), which
+// fake-shell interpreter (`src/twins/EmittedScriptRecognizer.ts`), which
 // simulates a driver running gtd's emitted scripts against the in-memory
 // worktree — a wholly separate mechanism from any gtd-internal layer.
 
@@ -193,7 +193,7 @@ export interface TestWorldOptions {
    * Effect test has no `--verbose` flag of its own to gate on.
    */
   readonly narrate?: (line: string) => void
-  /** Gates `narrate` above — defaults to `true` (fires whenever a sink is given) since these tests aren't exercising `Cli.ts`'s own `--verbose` gate. `src/testkit/cliIo.ts` overrides this with the real decoded flag. */
+  /** Gates `narrate` above — defaults to `true` (fires whenever a sink is given) since these tests aren't exercising `Cli.ts`'s own `--verbose` gate. `src/twins/cliIo.ts` overrides this with the real decoded flag. */
   readonly verbose?: boolean
   /** Real or fake driver and tool commands — each one left out fails when called. */
   readonly tools?: Partial<ToolCommands["Type"]>

package/src/twins/index.ts

@@ -0,0 +1,8 @@
+export { InMemRepo } from "./InMemRepo.js"
+export { makeInMemoryWorkspaceOps, testLayers, type ScriptedCommand } from "./Layers.js"
+export { makeCapturingCliIo, type CapturedCliResult } from "./cliIo.js"
+export { applyEmittedScript } from "./EmittedScriptRecognizer.js"
+export { applyFlowScript } from "./FlowScripts.js"
+export { gitTiers, runGitServiceContract, runGitScriptContract, type GitTier } from "./GitTiers.js"
+export { strictGitOperations } from "./StrictGitOperations.js"
+export { fakeGitOperations } from "./FakeGitOperations.js"

package/src/workflow/discovery.test.ts

@@ -6,7 +6,7 @@ import { NodeContext } from "@effect/platform-node"
 import { Effect, Layer } from "effect"
 import { afterEach, describe, expect, it } from "vitest"
 import { GitService, Host, Narrator, Workspace } from "../platform/index.js"
-import { InMemRepo, testLayers } from "../testkit/index.js"
+import { InMemRepo, testLayers } from "../twins/index.js"
 import { ConfigDiscovery, ConfigService } from "./index.js"
 
 // A config and the local modules it imports load the same from disk, where

package/src/workflow/discovery.ts

@@ -38,7 +38,7 @@ export const WORKFLOW_MODULE = "gtd.config.ts"
  * The file names gtd read repository settings from before `gtd.config.ts` took
  * them over. Still looked for, only to refuse them: a leftover one would
  * otherwise be silently ignored. Shared with the in-memory counterpart in
- * `src/testkit/Layers.ts`, so both tiers refuse the same names.
+ * `src/twins/Layers.ts`, so both tiers refuse the same names.
  */
 export const LEGACY_RC_FILES = [
   ".gtdrc",
@@ -71,7 +71,7 @@ const legacyRcFileAt = (path: string): LegacyRcFile | undefined => {
  * the user's home dir (inclusive, when it is an ancestor) or after reaching the
  * filesystem root — whichever comes first. Returned innermost→outermost.
  * Exported (via the barrel) for the same reason `SEARCH_PLACES` is — one
- * shared implementation, not a second copy in `src/testkit/Layers.ts` that
+ * shared implementation, not a second copy in `src/twins/Layers.ts` that
  * could silently diverge.
  */
 export const walkUp = (from: string, home: string): ReadonlyArray<string> => {
@@ -115,7 +115,7 @@ const workflowModule = (
  * reads; discovery is effectful and environment-dependent: `.Live` reads
  * REAL disk, so an `@inmem` scenario's fake
  * `Workspace` — a pure in-memory `Map`, no real files anywhere on disk —
- * cannot back it. `src/testkit/Layers.ts` provides the in-memory
+ * cannot back it. `src/twins/Layers.ts` provides the in-memory
  * counterpart instead, mirroring how `Workspace`/`Host`/`GitService` already
  * each have a production and a fake layer behind one Tag.
  */

package/src/workflow/load.ts

@@ -357,14 +357,14 @@ const shippedModules = (): Promise<ShippedModules> =>
     return { modules, missing }
   })())
 
-/** The file each subpath a config may import serves: all but `flows`, the engine's own, and `testkit`. */
+/** The file each subpath a config may import serves: all but `flows`, the engine's own, and the testkit's. */
 const subpathFiles = (condition: string): ReadonlyMap<string, string> => {
   const { root, pkg } = ownPackage()
   const files = new Map<string, string>()
   for (const [subpath, target] of Object.entries(pkg.exports)) {
     const file = typeof target === "string" ? target : target[condition]
-    // testkit imports the consumer's test runner, and no config runs one.
-    if (subpath === "./flows" || subpath === "./testkit" || file === undefined) continue
+    // A config runs no tests, and the gherkin steps import the consumer's test runner.
+    if (subpath === "./flows" || subpath.startsWith("./testkit") || file === undefined) continue
     files.set(`${PACKAGE}/${subpath.replace(/^\.\//, "")}`, join(root, file))
   }
   return files