@pmelab/gtd 59.0.1 → 60.0.0

package/src/twins/EmittedScriptRecognizer.ts

@@ -30,7 +30,7 @@ import {
   formatFinding,
   steeringFormatFor,
   type SteeringFormat,
-} from "../steering/index.js"
+} from "../steering/internal.js"
 import { refNameFault } from "../replay/index.js"
 import { renderFormat } from "../wire/index.js"
 import { applyFlowScript, rewritten } from "./FlowScripts.js"
@@ -518,12 +518,8 @@ const recognizeGtdCheck = (repo: InMemRepo, block: string): BlockOutcome | undef
     const known = BUILT_IN_MODE_NAMES.join(", ")
     return { kind: "failed", error: `gtd check: unknown mode "${mode}" — known modes: ${known}` }
   }
-  // An ABSENT file has nothing to check and exits 0 — `runCheckCommand`'s own
-  // documented behavior. Validating `""` instead would report every "missing
-  // header" finding the format has, turning "the reviewer deleted the file"
-  // (a case the review-doc guard owns) into a script failure.
   const content = repo.readFile(file)
-  if (content === undefined) return { kind: "noop" }
+  if (content === undefined) return { kind: "failed", error: `gtd check: no such file: ${file}` }
   const findings = format.validate(content).map((finding) => formatFinding(file, finding))
   if (findings.length === 0) return { kind: "noop" }
   return {

package/src/twins/EmittedScripts.test.ts

@@ -140,8 +140,10 @@ for (const makeTier of gitTiers) {
     })
 
     describe("gtd check", () => {
-      it("passes a valid file, and an absent one", () => {
-        expect(run(script(`gtd check qa '${FILE}'`)).ok).toBe(true)
+      it("refuses an absent file, and passes a valid one", () => {
+        const absent = run(script(`gtd check qa '${FILE}'`))
+        expect(absent).toMatchObject({ ok: false, status: 1 })
+        expect(absent.output).toContain(`gtd check: no such file: ${FILE}`)
         t.seed.writeFile(FILE, "## Open Questions\n")
         expect(run(script(`gtd check qa '${FILE}'`))).toEqual({ ok: true, status: 0, output: "" })
       })

package/src/twins/InMemRepo.ts

@@ -30,6 +30,7 @@ function makeHash(message: string, parent: string | null, tree: Map<string, stri
 export class InMemRepo {
   // Data-only marker read by no code path: embeds the sentinel in every
   // instance so a leaked object still carries it.
+  // fallow-ignore-next-line unused-class-member
   readonly testDouble = TEST_DOUBLE_SENTINEL
 
   private commits: Map<string, Commit> = new Map()

package/src/wire/Script.ts

@@ -0,0 +1,11 @@
+import { WIRE_SCHEMA } from "./constants.js"
+
+/** `--json` of every command that prints a script and nothing else to decide on: abandon, restore, start, door, validate, history fetch. */
+export interface ScriptFields {
+  readonly schema: number
+  readonly script: string
+}
+
+export const scriptFields = (script: string): ScriptFields => ({ schema: WIRE_SCHEMA, script })
+
+export const renderScriptJson = (fields: ScriptFields): string => JSON.stringify(fields) + "\n"

package/src/wire/WireSchema.test.ts

@@ -9,6 +9,8 @@ import {
   landFields,
   renderBeatJson,
   renderLandJson,
+  renderScriptJson,
+  scriptFields,
   statusOf,
   WIRE_SCHEMA,
   type BeatKind,
@@ -116,6 +118,7 @@ const documents: Readonly<Record<string, string>> = {
     }),
   ),
   "judge.json": JUDGE,
+  "script.json": renderScriptJson(scriptFields("git reset --mixed ...\n")),
 }
 
 describe(`wire schema ${WIRE_SCHEMA}`, () => {

package/src/wire/golden/1/script.json

@@ -0,0 +1,4 @@
+{
+  "schema": 1,
+  "script": "git reset --mixed ...\n"
+}

package/src/wire/index.ts

@@ -6,6 +6,7 @@ export * from "./BeatStatus.js"
 export * from "./BeatDocument.js"
 export * from "./BeatPlain.js"
 export * from "./Land.js"
+export * from "./Script.js"
 export * from "./Judge.js"
 export { selectPath } from "./Select.js"
 export { resolveSession } from "./Sessions.js"

package/src/workflow/SteeringFormats.test.ts

@@ -1,7 +1,7 @@
 import { describe, expect, it } from "vitest"
 import fc from "fast-check"
 import { isSeededValidateCommand, seededValidateCommand } from "./SteeringFormats.js"
-import { BUILT_IN_MODE_NAMES } from "../steering/index.js"
+import { BUILT_IN_MODE_NAMES } from "../steering/internal.js"
 
 describe("seededValidateCommand / isSeededValidateCommand", () => {
   it("recognizes its own seeded command for every built-in mode", () => {

package/src/workflow/compile.ts

@@ -1,7 +1,7 @@
 import { seededValidateCommand } from "./SteeringFormats.js"
-import { BUILT_IN_MODE_NAMES } from "../steering/index.js"
+import { BUILT_IN_MODE_NAMES } from "../steering/internal.js"
 import type { ModeDef } from "./Workflow.js"
-import { configProblems, type GtdConfig, type UiSettings } from "../flows/index.js"
+import { configProblems, type GtdConfig, type UiSettings } from "../flows/internal.js"
 import type { Diagnostic } from "./Diagnostic.js"
 
 export interface CompiledConfig {

package/src/workflow/discovery.test.ts

@@ -22,15 +22,15 @@ interface Tier {
 
 const config = (imports: string): string =>
   [
-    'import { agent, human } from "@pmelab/gtd/flows"',
+    'import { agent, human, workflow } from "@pmelab/gtd/flows"',
     imports,
     "export default async () => {",
     '  await human("idle", { message: "go" })',
     '  await agent("working", brief)',
     "}",
-    "export const named = async () => {",
+    "export const named = workflow(async () => {",
     '  await agent("working", brief)',
-    "}",
+    "})",
     "",
   ].join("\n")