@windyroad/jtbd 0.8.4 → 0.9.0-preview.446

package/.claude-plugin/plugin.json

@@ -90,5 +90,5 @@
     }
   },
   "name": "wr-jtbd",
-  "version": "0.8.4"
+  "version": "0.9.0"
 }

package/agents/agent.md

@@ -52,8 +52,8 @@ When the change or plan under review **explicitly cites, implements, or serves**
 wr-jtbd-is-job-or-persona-unconfirmed <persona-name | JTBD-NNN>
 ```
 
-- **Exit 0** (frontmatter lacks the marker AND the artifact is not superseded) → the artifact is **unratified**. Emit **ISSUES FOUND / [Unratified Dependency]** with action: "ratify `<persona | JTBD-NNN>` via `/wr-jtbd:confirm-jobs-and-personas` before this lands." (The predicate prints the resolved path on stdout.)
-- **Exit 1** (ratified, or superseded) → do NOT flag.
+- **Exit 0** (frontmatter lacks the marker AND the artifact is not superseded AND it does not carry the rejected-pending-supersede + supersede-ticket pair) → the artifact is **unratified**. Emit **ISSUES FOUND / [Unratified Dependency]** with action: "ratify `<persona | JTBD-NNN>` via `/wr-jtbd:confirm-jobs-and-personas` before this lands." (The predicate prints the resolved path on stdout.)
+- **Exit 1** (ratified, superseded, or rejected-pending-supersede with a tracked `supersede-ticket: P<NNN>` — ADR-068 amendment per P316) → do NOT flag. A user-rejected artifact with a tracked supersede ticket is ratified-equivalent for the build-upon guard.
 - **Exit 2** (ref not found) → the change cites a persona/job that does not exist; that is a separate Job Gap / Persona Mismatch, not an Unratified Dependency.
 
 **Key the flag on the oversight marker, NEVER on `status:`.** `status: proposed`/`accepted` and `human-oversight:` are orthogonal axes (ADR-066). Building on a **ratified** job whose `status` is still `proposed` is fine — do NOT flag it; only the *unratified* (marker-absent, non-superseded) case flags.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@windyroad/jtbd",
-  "version": "0.8.4",
+  "version": "0.9.0-preview.446",
   "description": "Jobs-to-be-done enforcement for UI changes",
   "bin": {
     "windyroad-jtbd": "./bin/install.mjs"

package/scripts/detect-unoversighted.sh

@@ -41,7 +41,18 @@ for f in "$JTBD_DIR"/*/*.md "$JTBD_DIR"/*.md; do
     { print }
   ' "$f")"
 
-  if ! printf '%s\n' "$fm" | grep -qiE '^human-oversight:[[:space:]]*confirmed[[:space:]]*$'; then
-    echo "$f"
+  if printf '%s\n' "$fm" | grep -qiE '^human-oversight:[[:space:]]*confirmed[[:space:]]*$'; then
+    continue
   fi
+
+  # ADR-068 amendment (P316): mirror the architect detector's
+  # rejected-pending-supersede exclusion. Both the marker AND a
+  # supersede-ticket: P<NNN> scalar must be present; an un-ticketed marker
+  # still surfaces (defensive — preserves JTBD-201/202 audit-trail guard).
+  if printf '%s\n' "$fm" | grep -qiE '^human-oversight:[[:space:]]*rejected-pending-supersede[[:space:]]*$' \
+     && printf '%s\n' "$fm" | grep -qiE '^supersede-ticket:[[:space:]]*P[0-9]+[[:space:]]*$'; then
+    continue
+  fi
+
+  echo "$f"
 done | sort

package/scripts/is-job-or-persona-unconfirmed.sh

@@ -84,6 +84,16 @@ if printf '%s\n' "$fm" | grep -qiE '^human-oversight:[[:space:]]*confirmed[[:spa
   exit 1   # confirmed — OK to build on
 fi
 
+# ADR-068 amendment (P316): mirror the architect predicate's
+# rejected-pending-supersede exclusion. A persona/job the user explicitly
+# rejected with a tracked supersede ticket is ratified-equivalent for the
+# build-upon guard — the [Unratified Dependency] verdict must NOT re-fire on
+# it. Marker without ticket still fires (defensive).
+if printf '%s\n' "$fm" | grep -qiE '^human-oversight:[[:space:]]*rejected-pending-supersede[[:space:]]*$' \
+   && printf '%s\n' "$fm" | grep -qiE '^supersede-ticket:[[:space:]]*P[0-9]+[[:space:]]*$'; then
+  exit 1
+fi
+
 # Unconfirmed — the build-upon guard SHOULD fire. Name the file for the guard.
 echo "$file"
 exit 0

package/scripts/test/detect-unoversighted.bats

@@ -83,3 +83,31 @@ mk() { # mk <relpath under docs/jtbd> <extra frontmatter lines...>
   [ "$status" -eq 0 ]
   [ -z "$output" ]
 }
+
+# ADR-068 amendment (P316): mirror the architect `rejected-pending-supersede`
+# exclusion onto the JTBD detector. Exclusion requires BOTH the marker AND
+# a supersede-ticket: P<NNN> scalar; a marker without the ticket is
+# malformed and still surfaces (defensive).
+
+@test "JTBD with rejected-pending-supersede + supersede-ticket is excluded" {
+  mk "solo-developer/JTBD-020-rejected.proposed.md" \
+    "human-oversight: rejected-pending-supersede" \
+    "supersede-ticket: P297"
+  run bash "$SCRIPT" "$DIR/docs/jtbd"
+  [[ "$output" != *"JTBD-020-rejected.proposed.md"* ]]
+}
+
+@test "JTBD with rejected-pending-supersede WITHOUT supersede-ticket still surfaces (defensive)" {
+  mk "solo-developer/JTBD-021-untracked.proposed.md" \
+    "human-oversight: rejected-pending-supersede"
+  run bash "$SCRIPT" "$DIR/docs/jtbd"
+  [[ "$output" == *"JTBD-021-untracked.proposed.md"* ]]
+}
+
+@test "persona with rejected-pending-supersede + ticket is also excluded" {
+  mk "rejected-persona/persona.md" \
+    "human-oversight: rejected-pending-supersede" \
+    "supersede-ticket: P299"
+  run bash "$SCRIPT" "$DIR/docs/jtbd"
+  [[ "$output" != *"rejected-persona/persona.md"* ]]
+}

package/scripts/test/is-job-or-persona-unconfirmed.bats

@@ -124,3 +124,44 @@ mk_job() {
   [[ "$detect_out" != *"tech-lead/persona.md"* ]]
   run bash "$SCRIPT" "tech-lead" "$DIR/docs/jtbd"; [ "$status" -eq 1 ]
 }
+
+# ADR-068 amendment (P316): mirror the architect `rejected-pending-supersede`
+# exclusion onto the JTBD predicate.
+
+# mk_rejected_persona <name> [supersede-ticket-or-empty]
+mk_rejected_persona() {
+  local name="$1"; local ticket="${2:-}"
+  mkdir -p "$DIR/docs/jtbd/$name"
+  {
+    echo "---"
+    echo "name: $name"
+    echo "human-oversight: rejected-pending-supersede"
+    [ -n "$ticket" ] && echo "supersede-ticket: $ticket"
+    echo "---"
+    echo "# $name"
+  } > "$DIR/docs/jtbd/$name/persona.md"
+}
+
+@test "persona with rejected-pending-supersede + supersede-ticket does NOT fire the guard (exit 1)" {
+  mk_rejected_persona "scrapped-persona" "P299"
+  run bash "$SCRIPT" "scrapped-persona" "$DIR/docs/jtbd"
+  [ "$status" -eq 1 ]
+  [ -z "$output" ]
+}
+
+@test "persona with rejected-pending-supersede WITHOUT supersede-ticket DOES fire the guard (exit 0)" {
+  mk_rejected_persona "untracked-persona" ""
+  run bash "$SCRIPT" "untracked-persona" "$DIR/docs/jtbd"
+  [ "$status" -eq 0 ]
+  [[ "$output" == *"untracked-persona/persona.md"* ]]
+}
+
+@test "agrees with detect-unoversighted on rejected-pending-supersede (sync guard)" {
+  mk_rejected_persona "alpha-tracked" "P297"
+  mk_rejected_persona "beta-untracked" ""
+  detect_out="$(bash "$DETECT" "$DIR/docs/jtbd")"
+  [[ "$detect_out" != *"alpha-tracked/persona.md"* ]]
+  run bash "$SCRIPT" "alpha-tracked" "$DIR/docs/jtbd"; [ "$status" -eq 1 ]
+  [[ "$detect_out" == *"beta-untracked/persona.md"* ]]
+  run bash "$SCRIPT" "beta-untracked" "$DIR/docs/jtbd"; [ "$status" -eq 0 ]
+}

package/skills/confirm-jobs-and-personas/SKILL.md

@@ -42,7 +42,7 @@ For each job/persona in the ordered queue, surface it as an `AskUserQuestion` (c
 - **Options** per artifact:
   - **Confirm** — the job/persona accurately reflects real need; write the marker.
   - **Amend** — mostly right but needs a change; capture it, apply it to the file, then write the marker.
-  - **Reject** — the auto-derived job/persona does not reflect real need; do NOT write the marker. Note the rework (a `wr-jtbd:update-guide` rewrite, or retirement).
+  - **Reject** — the auto-derived job/persona does not reflect real need; capture the supersede ticket (see Step 4) and write the **rejected-pending-supersede** marker so the drain stops re-asking.
   - **Defer** — skip this sitting; leave unoversighted for later.
 
 This is a genuine human-decision surface (the point of P288/ADR-068) — `AskUserQuestion` is correct here, not over-asking. Do not auto-confirm; do not prose-ask.
@@ -50,7 +50,10 @@ This is a genuine human-decision surface (the point of P288/ADR-068) — `AskUse
 ### Step 4: Apply the outcome
 
 - **Confirm / Amend**: write `human-oversight: confirmed` + `oversight-date: <today, YYYY-MM-DD>` into the file's frontmatter (insert after the `status:`/`date-created:` line if absent; never duplicate). For Amend, apply the directed change first. Edits go through the standard JTBD / architect edit gate per ADR-014.
-- **Reject**: leave the marker absent; record the rework.
+- **Reject** (ADR-068 amendment per P316, mirroring ADR-066):
+  1. Capture the supersede ticket via a follow-up `AskUserQuestion`: "Which problem ticket tracks the rework?" — options: existing `P<NNN>` IDs surfaced from `docs/problems/`, **Capture a new ticket** (delegate to `/wr-itil:capture-problem`), or **Defer (leave un-tracked for now)**.
+  2. If a ticket ID is captured, write `human-oversight: rejected-pending-supersede` + `supersede-ticket: P<NNN>` into the file's frontmatter. The detector excludes artifacts carrying both, so the drain stops re-asking until either the rework lands (file renamed to `*.superseded.md`) or the rejection is revisited.
+  3. If the user defers ticket capture, leave the marker absent — the artifact re-surfaces next drain (the un-tracked case is intentionally re-asked so it doesn't silently rot).
 - **Defer**: no write.
 
 **Unoversighted ≠ unusable** (ADR-068): an unconfirmed job/persona stays fully readable and review-anchorable. The marker records provenance; it never quarantines the doc or blocks reviews from reading it.
@@ -61,7 +64,7 @@ Commit the confirmed/amended files per ADR-014 (one commit per drain sitting is
 
 ## Notes
 
-- **Never re-ask** — a confirmed job/persona carries the marker permanently and is excluded from future runs (ADR-009). Write-once **except** when the job statement / persona definition is materially rewritten — a material amend clears the marker for re-confirmation (ADR-068 Reassessment).
+- **Never re-ask** — a confirmed job/persona carries the marker permanently and is excluded from future runs (ADR-009). The same write-once-permanence applies to the `rejected-pending-supersede` value (P316 amendment): once the user rejects with a tracked ticket, the drain stops asking. Write-once **except** when the job statement / persona definition is materially rewritten — a material amend clears the marker for re-confirmation (ADR-068 Reassessment). When the rework lands and the file is renamed to `*.superseded.md`, the existing superseded-name skip takes over; the `rejected-pending-supersede` lines become historical residue.
 - **AFK** — interactive by construction (the confirm IS the human decision); not dispatched in AFK iteration subprocesses. The session-start nudge self-suppresses there (`WR_SUPPRESS_OVERSIGHT_NUDGE=1`).
 - **Born-confirmed going forward** — `/wr-jtbd:update-guide` writes the marker when the user confirms a new/edited job or persona, so new artifacts enter the set already oversighted and the unoversighted count only shrinks.