@pmelab/gtd 64.0.0 → 64.1.1

package/src/twins/EmittedScripts.test.ts

@@ -16,6 +16,8 @@ import {
 import { Effect } from "effect"
 import { ScriptSurface } from "../emit/index.js"
 import { GitService } from "../platform/index.js"
+import { listFormat } from "../steering/index.js"
+import { declaredValidateCommand } from "../workflow/index.js"
 import { gitTiers, type GitTier, type ScriptedCommand } from "./index.js"
 
 // Every case runs through real `sh` — with this build's `gtd` on PATH — and
@@ -158,6 +160,23 @@ for (const makeTier of gitTiers) {
           ),
         )
       })
+
+      it("checks a declared format from the spec on its command line", () => {
+        const ids = listFormat({
+          name: "ids",
+          item: { id: /^A-\d+$/, separator: ":", example: "A-1" },
+          ids: { unique: true },
+        })
+        const check = declaredValidateCommand("ids", ids.spec).replace('"$GTD_FILE"', `'${FILE}'`)
+        t.seed.writeFile(FILE, "- A-1: it's\n")
+        expect(run(script(check))).toEqual({ ok: true, status: 0, output: "" })
+        t.seed.writeFile(FILE, "- A-1: a\n- A-1: b\n")
+        const result = run(script(check))
+        expect(result).toMatchObject({ ok: false, status: 1 })
+        expect(result.output).toBe(
+          `gtd check: ${FILE} is not valid under mode "ids" (1 finding(s)):\n${FILE}:2: duplicate ID A-1 — first on line 1\n`,
+        )
+      })
     })
 
     it("gtd uncheck clears a review file's ticks", () => {

package/src/workflow/SteeringFormats.ts

@@ -4,3 +4,7 @@ export const seededValidateCommand = (mode: string): string => `gtd check ${mode
 /** True when `command` is exactly `mode`'s seeded template (compared literally, not rendered) — distinguishes gtd's own seeding from a user override. */
 export const isSeededValidateCommand = (mode: string, command: string): boolean =>
   command === seededValidateCommand(mode)
+
+/** The `validate:` command for a declared format: its spec travels on the command line, so `gtd check` loads no workflow to find it. */
+export const declaredValidateCommand = (mode: string, spec: string): string =>
+  `gtd check ${mode} "$GTD_FILE" --spec '${spec.replace(/'/g, "'\\''")}'`

package/src/workflow/Workflow.ts

@@ -32,6 +32,8 @@ export interface StepDef {
   readonly label?: string
   readonly file?: string
   readonly mode?: StateMode
+  /** A declared format's spec, when `mode` names one rather than a built-in or `modes:` entry. */
+  readonly modeSpec?: string
   readonly model?: string
   readonly system?: string
   readonly judge?: string

package/src/workflow/index.ts

@@ -23,7 +23,11 @@ export {
   SETTING_NAME_RULE,
 } from "./vars.js"
 export { resolveScopeAccess, resolveScopeSkills } from "./skills.js"
-export { seededValidateCommand, isSeededValidateCommand } from "./SteeringFormats.js"
+export {
+  declaredValidateCommand,
+  seededValidateCommand,
+  isSeededValidateCommand,
+} from "./SteeringFormats.js"
 export {
   type ChangeStatus,
   type PendingChange,