@pmelab/gtd 15.4.0 → 15.6.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pmelab/gtd",
-  "version": "15.4.0",
+  "version": "15.6.0",
   "private": false,
   "description": "Git-aware CLI that emits the next prompt for an autonomous coding agent based on the current repository state",
   "bin": {
@@ -106,7 +106,8 @@
     "tailwindcss": "^4.3.3",
     "tsdown": "^0.22.8",
     "turbo": "^2.10.9",
-    "vitest": "^3.2.0"
+    "vitest": "^3.2.0",
+    "zustand": "^5.0.15"
   },
   "peerDependencies": {
     "typescript": "^5"

package/src/workflows/steps.test.ts

@@ -25,6 +25,7 @@ describe("the bundled workflow's steps declare skills", () => {
     expect(agentSkills(await capture(() => steps.author()))).toEqual([
       "api-and-interface-design",
       "documentation-and-adrs",
+      "ponytail",
     ])
   })
 

package/src/workflows/text.test.ts

@@ -2,6 +2,7 @@ import { describe, expect, it } from "vitest"
 import {
   agentWithSkills,
   architectureAuthorPrompt,
+  buildFixQualityPrompt,
   designTriagePrompt,
   splitSkills,
   summaryPrompt,
@@ -103,6 +104,21 @@ describe("the shared open-question instruction", () => {
   })
 })
 
+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")
+  })
+})
+
+describe("buildFixQualityPrompt", () => {
+  it("states the missing-test-signal-beats-line-count tiebreak", () => {
+    const prompt = renderText(() => buildFixQualityPrompt())
+    expect(prompt).toContain("missing test signal beats line count")
+    expect(prompt).toContain("a test is never deleted to satisfy a simplification finding")
+  })
+})
+
 describe("summaryPrompt", () => {
   it("puts the total and every model's cost on a line of its own", () => {
     const prompt = summaryPrompt({

package/src/workflows/text.ts

@@ -267,6 +267,9 @@ ${footnoteFoldIn}
   the product ones, one phase earlier. PERMISSIVE: answer it
   yourself unless you genuinely cannot defend a default; a wrong
   technical call is still caught at spec review
+- The narrow exception: a simplification that drops something
+  \`.gtd/REQUIREMENTS.md\` mentions is an open question, not a
+  silent default
 
 ${questionBar}
 ## Return lap
@@ -485,6 +488,8 @@ export const buildFixQualityPrompt = (): string =>
 - Read \`.gtd/QUALITY.md\` — one \`## \` chunk per quality dimension
   that found something blocking. Merge duplicate findings across
   dimensions FIRST, then fix every chunk
+- When findings conflict, missing test signal beats line count —
+  a test is never deleted to satisfy a simplification finding
 - Delete \`.gtd/QUALITY.md\` once every finding is resolved
 - Leave everything else uncommitted and finish your turn
 `

package/src/workflows/vars.ts

@@ -11,7 +11,7 @@ export const defaults: Readonly<Record<string, string>> = {
   reviewNoteActionable: "0.7",
   architectureSkipMinP: "0.85",
   triageSkills: "spec-driven-development, planning-and-task-breakdown",
-  architectureSkills: "api-and-interface-design, documentation-and-adrs",
+  architectureSkills: "api-and-interface-design, documentation-and-adrs, ponytail",
   decomposeSkills: "incremental-implementation, planning-and-task-breakdown",
   buildSkills: "test-driven-development, incremental-implementation",
   fixSkills: "debugging-and-error-recovery",
@@ -19,5 +19,5 @@ export const defaults: Readonly<Record<string, string>> = {
   reviewSkills: "code-review-and-quality",
   specReviewSkills: "code-review-and-quality, spec-driven-development",
   escalateSkills: "debugging-and-error-recovery",
-  qualityReviews: "owasp-security, code-simplification",
+  qualityReviews: "owasp-security, ponytail-review, test-audit",
 }