@pmelab/gtd 49.0.0 → 51.0.0

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

@@ -9,6 +9,7 @@ Feature: a refactor-first judgment walks one fixed path to the review gate
 
   Background:
     Given the shipped workflows, as gtd install scaffolds them
+    And an environment variable "GTD_PROTOTYPEROUNDS" set to "0"
     And an environment variable "GTD_QUALITYREVIEWS" set to ""
     And an environment variable "GTD_REFACTORQUALITYREVIEWS" set to ""
     And an environment variable "GTD_FLAG_RISK_TIER" set to "0"
@@ -72,7 +73,12 @@ 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 step "refactor.build.review.composing" writes ".gtd/REVIEW.md" with:
+    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:
       """
       # Review: abc1234
 
@@ -96,6 +102,8 @@ Feature: a refactor-first judgment walks one fixed path to the review gate
 
       ## calc
 
+      _critical: judge unsure_
+
       - [x] ./src/calc.ts#1-2
         add built on the shared op
       """
@@ -138,8 +146,10 @@ Feature: a refactor-first judgment walks one fixed path to the review gate
       gtd(check): refactor.deriving → refactor.building
       gtd(agent): refactor.building → refactor.health.check
       gtd(check): refactor.health.check → refactor.closing
-      gtd(check): refactor.closing → refactor.build.review.composing
-      gtd(check): refactor.build.review.composing → refactor.build.review.await-review
+      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(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

@@ -10,6 +10,7 @@ Feature: refactor-first writes a refactor note, refactors on its own, then rebui
 
   Background:
     Given the shipped workflows, as gtd install scaffolds them
+    And an environment variable "GTD_PROTOTYPEROUNDS" set to "0"
     And an environment variable "GTD_QUALITYREVIEWS" set to ""
     And an environment variable "GTD_REFACTORQUALITYREVIEWS" set to ""
     And an environment variable "GTD_FLAG_RISK_TIER" set to "0"
@@ -83,7 +84,12 @@ 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 step "refactor.build.review.composing" writes ".gtd/REVIEW.md" with:
+    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:
       """
       # Review: abc1234
 
@@ -106,6 +112,8 @@ Feature: refactor-first writes a refactor note, refactors on its own, then rebui
 
       ## calc
 
+      _critical: judge unsure_
+
       - [x] ./src/calc.ts#1-2
         add built on the shared op
       """
@@ -147,7 +155,12 @@ 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 step "refactor.build.review.composing" writes ".gtd/REVIEW.md" with:
+    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:
       """
       # Review: abc1234
 
@@ -170,6 +183,8 @@ Feature: refactor-first writes a refactor note, refactors on its own, then rebui
 
       ## calc
 
+      _critical: judge unsure_
+
       - [x] ./src/calc.ts#1-2
         add built on the shared op
       """
@@ -193,7 +208,12 @@ 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 step "refactor.build.review.composing" writes ".gtd/REVIEW.md" with:
+    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:
       """
       # Review: abc1234
 
@@ -216,6 +236,8 @@ Feature: refactor-first writes a refactor note, refactors on its own, then rebui
 
       ## calc
 
+      _critical: judge unsure_
+
       - [x] ./src/calc.ts#1-2
         add built on the shared op
       """
@@ -253,8 +275,10 @@ Feature: refactor-first writes a refactor note, refactors on its own, then rebui
       gtd(check): refactor.deriving → refactor.building
       gtd(agent): refactor.building → refactor.health.check
       gtd(check): refactor.health.check → refactor.closing
-      gtd(check): refactor.closing → refactor.build.review.composing
-      gtd(check): refactor.build.review.composing → refactor.build.review.await-review
+      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(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

@@ -9,6 +9,7 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
   Background:
     Given the shipped workflows, as gtd install scaffolds them
+    And an environment variable "GTD_PROTOTYPEROUNDS" set to "0"
     And an environment variable "GTD_QUALITYREVIEWS" set to ""
     And an environment variable "GTD_FLAG_RISK_TIER" set to "0"
 
@@ -78,7 +79,12 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
       """
     And step "refactor.health.check" changes nothing
     And step "refactor.closing" runs its script
-    And step "refactor.build.review.composing" writes ".gtd/REVIEW.md" with:
+    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:
       """
       # Review: abc1234
 
@@ -171,7 +177,12 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
       """
     And step "refactor.health.check" changes nothing
     And step "refactor.closing" runs its script
-    And step "refactor.build.review.composing" writes ".gtd/REVIEW.md" with:
+    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:
       """
       # Review: abc1234
 
@@ -191,8 +202,10 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
     And the commit subjects end with:
       """
       gtd(check): refactor.closing → refactor.build.quality.correctness.reviewing
-      gtd(agent): refactor.build.quality.correctness.reviewing → refactor.build.review.composing
-      gtd(check): refactor.build.review.composing → refactor.build.review.await-review
+      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
       """
 
 
@@ -263,7 +276,12 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
         """
       And step "refactor.health.check" changes nothing
       And step "refactor.closing" runs its script
-      And step "refactor.build.review.composing" writes ".gtd/REVIEW.md" with:
+      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:
         """
         # Review: abc1234
 
@@ -287,6 +305,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-2
           add built on the shared op[^why]
 
@@ -324,6 +344,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-2
           add built on the shared op
         """
@@ -345,8 +367,10 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
         gtd(check): refactor.deriving → refactor.building
         gtd(agent): refactor.building → refactor.health.check
         gtd(check): refactor.health.check → refactor.closing
-        gtd(check): refactor.closing → refactor.build.review.composing
-        gtd(check): refactor.build.review.composing → refactor.build.review.await-review
+        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
         """
 
     Scenario: a question answered at the refactor review reaches the re-derived package as ### Answered Questions
@@ -360,6 +384,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-1
           why a shared op?
         - [ ] ./src/calc.ts#2-2
@@ -434,8 +460,10 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
         gtd(check): refactor.deriving → refactor.building
         gtd(agent): refactor.building → refactor.health.check
         gtd(check): refactor.health.check → refactor.closing
-        gtd(check): refactor.closing → refactor.build.review.composing
-        gtd(check): refactor.build.review.composing → refactor.build.review.await-review
+        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
         """
 
     Scenario: an edit note the collecting turn leaves out refuses the landing and never signs off
@@ -449,6 +477,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-2
           add built on the shared op — this should change behaviour, not just structure
         """
@@ -493,6 +523,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-2 rename the helper
           add built on the shared op
         """
@@ -520,6 +552,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-2
           add built on the shared op, lovely work
         """
@@ -539,6 +573,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-2
           add built on the shared op[^q]
 
@@ -593,6 +629,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-2
           add built on the shared op[^q]
 
@@ -625,6 +663,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-2
           add built on the shared op[^q]
 
@@ -676,6 +716,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-2
           add built on the shared op[^n1]
 
@@ -702,6 +744,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-2
           add built on the shared op[^n1]
 
@@ -729,6 +773,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-2
           add built on the shared op[^q]
 
@@ -761,6 +807,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-2
           add built on the shared op[^n1]
 
@@ -787,6 +835,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-2
           add built on the shared op - this should change behaviour, not just structure[^q]
 
@@ -819,6 +869,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-2
           add built on the shared op - this should change behaviour, not just structure[^q]
 
@@ -864,6 +916,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-2
           add built on the shared op[^q]
 
@@ -896,6 +950,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-2
           add built on the shared op[^q]
 
@@ -946,6 +1002,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-2
           add built on the shared op, lovely work[^q]
 
@@ -978,6 +1036,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-2
           add built on the shared op, lovely work[^q]
 
@@ -1033,6 +1093,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [x] ./src/calc.ts#1-2
           add built on the shared op
         """
@@ -1056,6 +1118,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-2
           add built on the shared op — rename `sign` to `direction`
         """
@@ -1085,6 +1149,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-2
           add built on the shared op — rename `sign` to `direction`
         """
@@ -1103,9 +1169,25 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
       Then the process rests at "refactor.build.review.await-review"
       And the commit subjects end with:
         """
-        gtd(check): refactor.closing → refactor.build.review.composing
-        gtd(check): refactor.build.review.composing → refactor.build.review.await-review
+        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
+        """
+
+    Scenario: a reviewer walks the human through the whole refactor, every chunk pointing at its code
+      When the process is driven
+      Then the process rests at "refactor.build.review.await-review"
+      And ".gtd/REVIEW.md" contains:
+        """
+        ## calc
+
+        _critical: judge unsure_
+
+        - [ ] ./src/calc.ts#1-2
+          add built on the shared op
         """
+      And no commit body contains "refactor.build.review.composing"
 
   Rule: with a quality lens on both slices
 
@@ -1175,7 +1257,12 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
         """
       And step "refactor.health.check" changes nothing
       And step "refactor.closing" runs its script
-      And step "refactor.build.review.composing" writes ".gtd/REVIEW.md" with:
+      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:
         """
         # Review: abc1234
 
@@ -1219,6 +1306,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [x] ./src/calc.ts#1-2
           add built on the shared op
         """
@@ -1382,7 +1471,12 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
         """
       And step "refactor.health.check" changes nothing
       And step "refactor.closing" runs its script
-      And step "refactor.build.review.composing" writes ".gtd/REVIEW.md" with:
+      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:
         """
         # Review: abc1234
 
@@ -1403,6 +1497,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-2
           add built on the shared op[^why]
 
@@ -1443,6 +1539,8 @@ Feature: the refactor slice is reviewed on its own before the feature is impleme
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#1-2
           add built on the shared op
         """

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

@@ -820,7 +820,7 @@ const driveReviewed = async (
     ...Object.fromEntries(lensEffects),
     deriving: () => void files.set(REFACTOR_PACKAGE, "package"),
     closing: () => void files.delete(REFACTOR_PACKAGE),
-    "review.composing": () => void files.set(REVIEW, reviewDoc("add")),
+    "review.reviewing": () => void files.set(REVIEW, reviewDoc("add")),
     "review.closing": () => void files.delete(REVIEW),
     "review.collecting": () => {
       if (!script.silentCollect)
@@ -929,7 +929,9 @@ describe("reviewedRefactor", () => {
       "refactor.build.quality.correctness.reviewing",
       "refactor.build.quality.ponytail-review.reviewing",
       "refactor.build.quality.conventions.reviewing",
-      "refactor.build.review.composing",
+      "refactor.build.review.reviewing",
+      "refactor.build.review.ranking",
+      "refactor.build.review.carrying",
       "refactor.build.review.await-review",
       "refactor.build.review.closing",
     ])
@@ -937,8 +939,16 @@ describe("reviewedRefactor", () => {
 
   it("reviews from HEAD at scope entry", async () => {
     const h = await driveReviewed(shape())
-    const composing = h.steps.find((s) => s.kind === "run" && s.name === "review.composing")
-    expect(composing?.kind === "run" ? composing.body : "").toContain("<!-- base: c0 -->")
+    const reviewing = h.steps.find((s) => s.kind === "agent" && s.name === "review.reviewing")
+    expect(reviewing?.kind === "agent" ? reviewing.prompt : "").toContain("<!-- base: c0 -->")
+  })
+
+  it("has the reviewer walk the whole refactor on every round", async () => {
+    const h = await driveReviewed(shape())
+    const reviewing = h.steps.find((s) => s.kind === "agent" && s.name === "review.reviewing")
+    expect(reviewing?.kind === "agent" ? reviewing.prompt : "").toContain(
+      "it covers the whole refactor since it started, on every round",
+    )
   })
 
   it("names the entry commit in every lens prompt, on the first lap and after a feedback lap", async () => {

package/src/workflows/feature/refactor.ts

@@ -389,6 +389,8 @@ 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
       await reUnwind(outcome)

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

@@ -10,6 +10,7 @@ Feature: every note in the review is answered with one tick
 
   Background:
     Given the shipped workflows, as gtd install scaffolds them
+    And an environment variable "GTD_PROTOTYPEROUNDS" set to "0"
     And an environment variable "GTD_QUALITYREVIEWS" set to "correctness"
     And an environment variable "GTD_FLAGMODE" set to "false"
     And a file "src/calc.ts" with:
@@ -109,6 +110,10 @@ Feature: every note in the review is answered with one tick
       - [ ] ./src/calc.ts#2-2 — now rounds the difference
       """
     And step "build.review.carrying" runs its script
+    And judge step "build.review.ranking" answers:
+      """
+      []
+      """
     And judge step "build.quality.triage" answers:
       """
       [{"id":"Q1","answer":"noise","p":0.9},{"id":"Q2","answer":"noise","p":0.9},{"id":"Q3","answer":"design","p":0.9}]
@@ -122,6 +127,8 @@ Feature: every note in the review is answered with one tick
       """
       ## AC-1 ignores floating point
 
+      _critical: judge unsure_
+
       Should subtract round? The criterion does not say.
 
       In the code:
@@ -215,7 +222,8 @@ Feature: every note in the review is answered with one tick
       gtd(agent): build.review.fix.nits.fixing → build.health.check
       gtd(check): build.health.check → build.review.closing
       gtd(check): build.review.closing → build.review.reviewing
-      gtd(agent): build.review.reviewing → build.review.carrying
+      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
       """
     And ".gtd/REVIEW.md" does not contain "AC-1 ignores floating point"

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

@@ -10,6 +10,7 @@ Feature: a feature process ends at the human review
 
   Background:
     Given the shipped workflows, as gtd install scaffolds them
+    And an environment variable "GTD_PROTOTYPEROUNDS" set to "0"
     And an environment variable "GTD_QUALITYREVIEWS" set to ""
     And an environment variable "GTD_FLAGMODE" set to "false"
     And a file "src/calc.ts" with:
@@ -69,6 +70,11 @@ Feature: a feature process ends at the human review
         new subtract function
       """
     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
@@ -79,7 +85,9 @@ Feature: a feature process ends at the human review
       """
       gtd(judge): implement.coverage → implement.closing
       gtd(check): implement.closing → build.review.reviewing
-      gtd(agent): build.review.reviewing → build.review.await-review
+      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
       """
 
   Scenario: a sign-off ends the process at idle
@@ -93,6 +101,8 @@ Feature: a feature process ends at the human review
 
       ## calc
 
+      _critical: judge unsure_
+
       - [x] ./src/calc.ts#2-2
         new subtract function
       """
@@ -102,7 +112,9 @@ 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.await-review
+      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(human): build.review.await-review → build.review.closing
       gtd(check): build.review.closing → idle
       """
@@ -146,6 +158,8 @@ Feature: a feature process ends at the human review
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#2-2
           new subtract function[^why]
 
@@ -164,6 +178,8 @@ Feature: a feature process ends at the human review
 
         ## calc
 
+        _critical: judge unsure_
+
         - [ ] ./src/calc.ts#2-2
           new subtract function
         """
@@ -215,7 +231,9 @@ Feature: a feature process ends at the human review
         gtd(check): implement.health.check → implement.coverage
         gtd(judge): implement.coverage → implement.closing
         gtd(check): implement.closing → build.review.reviewing
-        gtd(agent): build.review.reviewing → build.review.await-review
+        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
         """
 
     Scenario: a design numbering new behaviour at or below an earlier ID is refused