@pmelab/gtd 55.0.0 → 56.0.0

package/src/workflows/feature/refactor-absorbs.feature

@@ -74,10 +74,6 @@ Feature: a refactor that carried the whole feature still gets a quality lap over
       """
     And step "refactor.health.check" changes nothing
     And step "refactor.closing" runs its script
-    And judge step "refactor.build.review.ranking" answers:
-      """
-      []
-      """
     And step "refactor.build.review.carrying" runs its script
     And step "refactor.build.review.reviewing" writes ".gtd/REVIEW.md" with:
       """
@@ -100,9 +96,7 @@ Feature: a refactor that carried the whole feature still gets a quality lap over
 
       <!-- base: 0000000 -->
 
-      ## Walkthrough
-
-      ### calc
+      ## calc
 
       - [x] ./src/calc.ts#1-2
         subtract next to add

package/src/workflows/feature/refactor-first.feature

@@ -77,10 +77,6 @@ Feature: a refactor-first judgment walks one fixed path to the review gate
       """
     And step "refactor.health.check" changes nothing
     And step "refactor.closing" runs its script
-    And judge step "refactor.build.review.ranking" answers:
-      """
-      []
-      """
     And step "refactor.build.review.carrying" runs its script
     And step "refactor.build.review.reviewing" writes ".gtd/REVIEW.md" with:
       """
@@ -104,9 +100,7 @@ Feature: a refactor-first judgment walks one fixed path to the review gate
 
       <!-- base: 0000000 -->
 
-      ## Walkthrough
-
-      ### calc
+      ## calc
 
       - [x] ./src/calc.ts#1-2
         add built on the shared op
@@ -147,9 +141,7 @@ Feature: a refactor-first judgment walks one fixed path to the review gate
       gtd(agent): refactor.building → refactor.health.check
       gtd(check): refactor.health.check → refactor.closing
       gtd(check): refactor.closing → refactor.build.review.reviewing
-      gtd(agent): refactor.build.review.reviewing → refactor.build.review.ranking
-      gtd(judge): refactor.build.review.ranking → refactor.build.review.carrying
-      gtd(check): refactor.build.review.carrying → refactor.build.review.await-review
+      gtd(agent): refactor.build.review.reviewing → refactor.build.review.await-review
       gtd(human): refactor.build.review.await-review → refactor.build.review.closing
       gtd(check): refactor.build.review.closing → implement.building
       gtd(agent): implement.building → implement.health.check

package/src/workflows/feature/refactor-rebuild.feature

@@ -98,10 +98,6 @@ Feature: refactor-first writes a refactor note, refactors on its own, then rebui
       """
     And step "refactor.health.check" changes nothing
     And step "refactor.closing" runs its script
-    And judge step "refactor.build.review.ranking" answers:
-      """
-      []
-      """
     And step "refactor.build.review.carrying" runs its script
     And step "refactor.build.review.reviewing" writes ".gtd/REVIEW.md" with:
       """
@@ -124,9 +120,7 @@ Feature: refactor-first writes a refactor note, refactors on its own, then rebui
 
       <!-- base: 0000000 -->
 
-      ## Walkthrough
-
-      ### calc
+      ## calc
 
       - [x] ./src/calc.ts#1-2
         add built on the shared op
@@ -170,10 +164,6 @@ Feature: refactor-first writes a refactor note, refactors on its own, then rebui
       """
     And step "refactor.health.check" changes nothing
     And step "refactor.closing" runs its script
-    And judge step "refactor.build.review.ranking" answers:
-      """
-      []
-      """
     And step "refactor.build.review.carrying" runs its script
     And step "refactor.build.review.reviewing" writes ".gtd/REVIEW.md" with:
       """
@@ -196,9 +186,7 @@ Feature: refactor-first writes a refactor note, refactors on its own, then rebui
 
       <!-- base: 0000000 -->
 
-      ## Walkthrough
-
-      ### calc
+      ## calc
 
       - [x] ./src/calc.ts#1-2
         add built on the shared op
@@ -222,10 +210,6 @@ Feature: refactor-first writes a refactor note, refactors on its own, then rebui
       """
     And step "refactor.health.check" changes nothing
     And step "refactor.closing" runs its script
-    And judge step "refactor.build.review.ranking" answers:
-      """
-      []
-      """
     And step "refactor.build.review.carrying" runs its script
     And step "refactor.build.review.reviewing" writes ".gtd/REVIEW.md" with:
       """
@@ -248,9 +232,7 @@ Feature: refactor-first writes a refactor note, refactors on its own, then rebui
 
       <!-- base: 0000000 -->
 
-      ## Walkthrough
-
-      ### calc
+      ## calc
 
       - [x] ./src/calc.ts#1-2
         add built on the shared op
@@ -279,9 +261,7 @@ Feature: refactor-first writes a refactor note, refactors on its own, then rebui
       gtd(agent): refactor.building → refactor.health.check
       gtd(check): refactor.health.check → refactor.closing
       gtd(check): refactor.closing → refactor.build.review.reviewing
-      gtd(agent): refactor.build.review.reviewing → refactor.build.review.ranking
-      gtd(judge): refactor.build.review.ranking → refactor.build.review.carrying
-      gtd(check): refactor.build.review.carrying → refactor.build.review.await-review
+      gtd(agent): refactor.build.review.reviewing → refactor.build.review.await-review
       gtd(human): refactor.build.review.await-review → refactor.build.review.closing
       gtd(check): refactor.build.review.closing → implement.building
       gtd(agent): implement.building → implement.health.check

package/src/workflows/feature/refactor-review.feature

@@ -83,10 +83,6 @@ Feature: the refactor is reviewed on its own before the feature is implemented
       """
     And step "refactor.health.check" changes nothing
     And step "refactor.closing" runs its script
-    And judge step "refactor.build.review.ranking" answers:
-      """
-      []
-      """
     And step "refactor.build.review.carrying" runs its script
     And step "refactor.build.review.reviewing" writes ".gtd/REVIEW.md" with:
       """
@@ -185,10 +181,6 @@ Feature: the refactor is reviewed on its own before the feature is implemented
       """
     And step "refactor.health.check" changes nothing
     And step "refactor.closing" runs its script
-    And judge step "refactor.build.review.ranking" answers:
-      """
-      []
-      """
     And step "refactor.build.review.carrying" runs its script
     And step "refactor.build.review.reviewing" writes ".gtd/REVIEW.md" with:
       """
@@ -211,9 +203,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
       """
       gtd(check): refactor.closing → refactor.build.quality.correctness.reviewing
       gtd(agent): refactor.build.quality.correctness.reviewing → refactor.build.review.reviewing
-      gtd(agent): refactor.build.review.reviewing → refactor.build.review.ranking
-      gtd(judge): refactor.build.review.ranking → refactor.build.review.carrying
-      gtd(check): refactor.build.review.carrying → refactor.build.review.await-review
+      gtd(agent): refactor.build.review.reviewing → refactor.build.review.await-review
       """
 
 
@@ -288,10 +278,6 @@ Feature: the refactor is reviewed on its own before the feature is implemented
         """
       And step "refactor.health.check" changes nothing
       And step "refactor.closing" runs its script
-      And judge step "refactor.build.review.ranking" answers:
-        """
-        []
-        """
       And step "refactor.build.review.carrying" runs its script
       And step "refactor.build.review.reviewing" writes ".gtd/REVIEW.md" with:
         """
@@ -315,9 +301,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2
           add built on the shared op[^why]
@@ -354,9 +338,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2
           add built on the shared op
@@ -380,9 +362,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
         gtd(agent): refactor.building → refactor.health.check
         gtd(check): refactor.health.check → refactor.closing
         gtd(check): refactor.closing → refactor.build.review.reviewing
-        gtd(agent): refactor.build.review.reviewing → refactor.build.review.ranking
-        gtd(judge): refactor.build.review.ranking → refactor.build.review.carrying
-        gtd(check): refactor.build.review.carrying → refactor.build.review.await-review
+        gtd(agent): refactor.build.review.reviewing → refactor.build.review.await-review
         """
 
     Scenario: a question answered at the refactor review reaches the re-derived package as ### Answered Questions
@@ -394,9 +374,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-1
           why a shared op?
@@ -473,9 +451,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
         gtd(agent): refactor.building → refactor.health.check
         gtd(check): refactor.health.check → refactor.closing
         gtd(check): refactor.closing → refactor.build.review.reviewing
-        gtd(agent): refactor.build.review.reviewing → refactor.build.review.ranking
-        gtd(judge): refactor.build.review.ranking → refactor.build.review.carrying
-        gtd(check): refactor.build.review.carrying → refactor.build.review.await-review
+        gtd(agent): refactor.build.review.reviewing → refactor.build.review.await-review
         """
 
     Scenario: an edit note the collecting turn leaves out refuses the landing and never signs off
@@ -487,9 +463,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2
           add built on the shared op — this should change behaviour, not just structure
@@ -533,9 +507,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2 rename the helper
           add built on the shared op
@@ -562,9 +534,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2
           add built on the shared op, lovely work
@@ -583,9 +553,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2
           add built on the shared op[^q]
@@ -635,9 +603,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2
           add built on the shared op[^q]
@@ -669,9 +635,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2
           add built on the shared op[^q]
@@ -718,9 +682,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2
           add built on the shared op[^n1]
@@ -746,9 +708,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2
           add built on the shared op[^n1]
@@ -775,9 +735,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2
           add built on the shared op[^q]
@@ -809,9 +767,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2
           add built on the shared op[^n1]
@@ -837,9 +793,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2
           add built on the shared op - this should change behaviour, not just structure[^q]
@@ -871,9 +825,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2
           add built on the shared op - this should change behaviour, not just structure[^q]
@@ -918,9 +870,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2
           add built on the shared op[^q]
@@ -952,9 +902,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2
           add built on the shared op[^q]
@@ -1000,9 +948,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2
           add built on the shared op, lovely work[^q]
@@ -1034,9 +980,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2
           add built on the shared op, lovely work[^q]
@@ -1087,9 +1031,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [x] ./src/calc.ts#1-2
           add built on the shared op
@@ -1112,9 +1054,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2
           add built on the shared op — rename `sign` to `direction`
@@ -1143,9 +1083,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2
           add built on the shared op — rename `sign` to `direction`
@@ -1166,9 +1104,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
       And the commit subjects end with:
         """
         gtd(check): refactor.closing → refactor.build.review.reviewing
-        gtd(agent): refactor.build.review.reviewing → refactor.build.review.ranking
-        gtd(judge): refactor.build.review.ranking → refactor.build.review.carrying
-        gtd(check): refactor.build.review.carrying → refactor.build.review.await-review
+        gtd(agent): refactor.build.review.reviewing → refactor.build.review.await-review
         """
 
     Scenario: a reviewer walks the human through the whole refactor, every chunk pointing at its code
@@ -1176,9 +1112,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
       Then the process rests at "refactor.build.review.await-review"
       And ".gtd/REVIEW.md" contains:
         """
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2
           add built on the shared op
@@ -1206,21 +1140,17 @@ Feature: the refactor is reviewed on its own before the feature is implemented
       Then the process rests at "refactor.build.review.await-review"
       And ".gtd/REVIEW.md" contains:
         """
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2 — Risk: still drops the sign
         """
-      And ".gtd/REVIEW.md" contains "## Walkthrough" exactly once
+      And ".gtd/REVIEW.md" contains "## calc" exactly once
       And the commit subjects end with:
         """
         gtd(agent): refactor.build.review.reviewing → refactor.build.review.fix.risks.fixing
         gtd(agent): refactor.build.review.fix.risks.fixing → refactor.build.health.check
         gtd(check): refactor.build.health.check → refactor.build.review.reviewing
-        gtd(agent): refactor.build.review.reviewing → refactor.build.review.ranking
-        gtd(judge): refactor.build.review.ranking → refactor.build.review.carrying
-        gtd(check): refactor.build.review.carrying → refactor.build.review.await-review
+        gtd(agent): refactor.build.review.reviewing → refactor.build.review.await-review
         """
 
   Rule: with a quality lens on both the refactor and the feature
@@ -1295,10 +1225,6 @@ Feature: the refactor is reviewed on its own before the feature is implemented
         """
       And step "refactor.health.check" changes nothing
       And step "refactor.closing" runs its script
-      And judge step "refactor.build.review.ranking" answers:
-        """
-        []
-        """
       And step "refactor.build.review.carrying" runs its script
       And step "refactor.build.review.reviewing" writes ".gtd/REVIEW.md" with:
         """
@@ -1338,9 +1264,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [x] ./src/calc.ts#1-2
           add built on the shared op
@@ -1509,10 +1433,6 @@ Feature: the refactor is reviewed on its own before the feature is implemented
         """
       And step "refactor.health.check" changes nothing
       And step "refactor.closing" runs its script
-      And judge step "refactor.build.review.ranking" answers:
-        """
-        []
-        """
       And step "refactor.build.review.carrying" runs its script
       And step "refactor.build.review.reviewing" writes ".gtd/REVIEW.md" with:
         """
@@ -1533,9 +1453,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2
           add built on the shared op[^why]
@@ -1575,9 +1493,7 @@ Feature: the refactor is reviewed on its own before the feature is implemented
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#1-2
           add built on the shared op

package/src/workflows/feature/refactor.test.ts

@@ -930,8 +930,6 @@ describe("reviewedRefactor", () => {
       "refactor.build.quality.ponytail-review.reviewing",
       "refactor.build.quality.conventions.reviewing",
       "refactor.build.review.reviewing",
-      "refactor.build.review.ranking",
-      "refactor.build.review.carrying",
       "refactor.build.review.await-review",
       "refactor.build.review.closing",
     ])

package/src/workflows/feature/refactor.ts

@@ -355,7 +355,6 @@ export const reviewedRefactor = async (
         fixAccess: strayAccess(p.strays),
         lensBase: entry,
         rules: t.refactorRules(p.strays, p.pattern),
-        walkthrough: true,
         lap: "refactor",
       })
       if (outcome.verdict === "signoff") return true

package/src/workflows/feature/review-gate.feature

@@ -71,10 +71,6 @@ Feature: a feature process ends at the human review
       """
     And step "build.review.closing" runs its script
     And step "build.review.carrying" runs its script
-    And judge step "build.review.ranking" answers:
-      """
-      []
-      """
 
   Scenario: a feature process ends at the review gate
     When the process is driven
@@ -85,9 +81,7 @@ Feature: a feature process ends at the human review
       """
       gtd(check): implement.health.check → implement.closing
       gtd(check): implement.closing → build.review.reviewing
-      gtd(agent): build.review.reviewing → build.review.ranking
-      gtd(judge): build.review.ranking → build.review.carrying
-      gtd(check): build.review.carrying → build.review.await-review
+      gtd(agent): build.review.reviewing → build.review.await-review
       """
 
   Scenario: a sign-off ends the process at idle
@@ -99,9 +93,7 @@ Feature: a feature process ends at the human review
 
       <!-- base: 0000000 -->
 
-      ## Walkthrough
-
-      ### calc
+      ## calc
 
       - [x] ./src/calc.ts#2-2
         new subtract function
@@ -112,9 +104,7 @@ Feature: a feature process ends at the human review
     And ".gtd/REVIEW.md" does not exist
     And the commit subjects end with:
       """
-      gtd(agent): build.review.reviewing → build.review.ranking
-      gtd(judge): build.review.ranking → build.review.carrying
-      gtd(check): build.review.carrying → build.review.await-review
+      gtd(agent): build.review.reviewing → build.review.await-review
       gtd(human): build.review.await-review → build.review.closing
       gtd(check): build.review.closing → idle
       """
@@ -156,9 +146,7 @@ Feature: a feature process ends at the human review
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#2-2
           new subtract function[^why]
@@ -176,9 +164,7 @@ Feature: a feature process ends at the human review
 
         <!-- base: 0000000 -->
 
-        ## Walkthrough
-
-        ### calc
+        ## calc
 
         - [ ] ./src/calc.ts#2-2
           new subtract function
@@ -229,9 +215,7 @@ Feature: a feature process ends at the human review
         gtd(agent): implement.building → implement.health.check
         gtd(check): implement.health.check → implement.closing
         gtd(check): implement.closing → build.review.reviewing
-        gtd(agent): build.review.reviewing → build.review.ranking
-        gtd(judge): build.review.ranking → build.review.carrying
-        gtd(check): build.review.carrying → build.review.await-review
+        gtd(agent): build.review.reviewing → build.review.await-review
         """
 
     Scenario: a design numbering new behaviour at or below an earlier ID is refused