@pmelab/gtd 15.4.0 → 15.5.0

package/README.md

@@ -23,12 +23,10 @@ Or run without installing (prefix every `gtd` below with `npx`) — see
 [Configuration](https://github.com/pmelab/gtd/blob/main/docs/configuration.md)
 for the settings most projects tune.
 
-Also install
-[`addyosmani/agent-skills`](https://github.com/addyosmani/agent-skills) — the
-bundled workflow names skills from it at several states instead of spelling out
-technique in the prompt itself — the same names also ride `gtd next --json`'s
-`skills` key at those states, for a driver that wants to preload them; see
-[Setup](https://github.com/pmelab/gtd/blob/main/docs/setup.md) for the details.
+Also install all three skill sources the bundled workflow names in its prompts —
+none is optional. See
+[Setup](https://github.com/pmelab/gtd/blob/main/docs/setup.md) for the list, the
+install flags, and the never-install-the-plugin caveat.
 
 > **A repository's `gtd.config.ts` is code, and gtd runs it.** A custom workflow
 > is a TypeScript module, and every gtd command that looks at workflow state —

package/dist/gtd.bundle.mjs

@@ -47920,6 +47920,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
@@ -48095,6 +48098,8 @@ const buildFixQualityPrompt = () => `${stateFileRules}
 - 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
 `;
@@ -49005,7 +49010,7 @@ const defaults = {
 	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",
@@ -49013,7 +49018,7 @@ const defaults = {
 	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"
 };
 //#endregion
 //#region src/workflows/unified.ts

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pmelab/gtd",
-  "version": "15.4.0",
+  "version": "15.5.0",
   "private": false,
   "description": "Git-aware CLI that emits the next prompt for an autonomous coding agent based on the current repository state",
   "bin": {

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",
 }