@windyroad/jtbd 0.13.1 → 0.13.2-preview.1058

package/.claude-plugin/plugin.json

@@ -90,5 +90,5 @@
     }
   },
   "name": "wr-jtbd",
-  "version": "0.13.1"
+  "version": "0.13.2"
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@windyroad/jtbd",
-  "version": "0.13.1",
+  "version": "0.13.2-preview.1058",
   "description": "Jobs-to-be-done enforcement for UI changes",
   "bin": {
     "windyroad-jtbd": "./bin/install.mjs"
@@ -26,6 +26,9 @@
     "scripts/",
     ".claude-plugin/",
     "lib/",
-    "!agents/eval/"
+    "!agents/eval/",
+    "!agents/test/",
+    "!hooks/test/",
+    "!scripts/test/"
   ]
 }

package/agents/test/jtbd-output-formatting.bats

@@ -1,26 +0,0 @@
-#!/usr/bin/env bats
-# Doc-lint guard: jtbd agent.md must include the output formatting rule
-# requiring human-readable titles alongside bare IDs (P032).
-#
-# Structural assertion — Permitted Exception to the source-grep ban (ADR-005 / P011).
-#
-# Cross-reference:
-#   P032 (agent output uses opaque IDs without titles)
-#   @jtbd JTBD-001 (enforce governance without slowing down)
-
-setup() {
-  AGENT_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  AGENT_FILE="${AGENT_DIR}/agent.md"
-}
-
-@test "agent.md contains output formatting section" {
-  run grep -n "## Output Formatting" "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "agent.md output formatting rule requires titles with IDs (P032)" {
-  run grep -n "title" "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-  run grep -n "Output Formatting" "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-}

package/agents/test/jtbd-pre-edit-review-mode.bats

@@ -1,48 +0,0 @@
-#!/usr/bin/env bats
-# Doc-lint guard: jtbd agent.md must carry an explicit pre-edit /
-# proposed-change review-mode carve-out so the agent classifies alignment
-# of the PROPOSAL when the calling prompt describes a not-yet-applied
-# change, instead of mis-classifying not-yet-applied state as ISSUES FOUND
-# (P313 catch-22: gate blocks edits; reviewer wants edits done first).
-#
-# tdd-review: structural-permitted (justification: P176 — agent behaviour is
-# prompt-driven with no skill-invocation harness to exercise the verdict
-# behaviourally; ADR-052 Surface 2 structural-justified case, NOT an ADR-005
-# Permitted Exception — ADR-052 narrows ADR-005 to exclude prose-doc greps).
-# When P176 lands, upgrade to a behavioural test that feeds the agent a
-# pre-edit proposal and asserts the PASS verdict.
-#
-# Cross-reference:
-#   P313 (Pre-edit governance-gate catch-22 — pass withheld pending edits)
-#   ADR-052 Surface 2 (structural-justified verdict) + P176 (harness gap)
-#   @jtbd JTBD-001 (Enforce Governance Without Slowing Down)
-
-setup() {
-  AGENT_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  AGENT_FILE="${AGENT_DIR}/agent.md"
-}
-
-@test "agent.md carries a Review Mode section distinguishing pre-edit from post-edit (P313)" {
-  run grep -nE "^## Review Mode: Pre-edit / proposed-change vs\. Post-edit / applied" "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "agent.md classifies alignment of the PROPOSAL in pre-edit mode (P313 verbatim core sentence)" {
-  run grep -nE "classify alignment of the PROPOSAL itself" "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "agent.md treats not-yet-applied state as the EXPECTED baseline of a pre-edit gate (P313)" {
-  run grep -nE "EXPECTED baseline of a pre-edit gate" "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "agent.md explicitly rejects 'edits aren't applied yet' as a valid ISSUES FOUND substance (P313)" {
-  run grep -nE "Do NOT treat .edits aren't applied yet" "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "agent.md cites P313 as the catch-22 closure (audit-trail)" {
-  run grep -nE "P313" "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-}

package/agents/test/jtbd-unratified-dependency-verdict.bats

@@ -1,62 +0,0 @@
-#!/usr/bin/env bats
-# Doc-lint guard: jtbd agent.md must carry the [Unratified Dependency] verdict
-# (ADR-068 enforcement surface 3 / RFC-011 / P323) — flag a change or plan that
-# explicitly cites/implements/serves a persona or job lacking `human-oversight:
-# confirmed` (unratified, non-superseded), keyed on the oversight marker NOT
-# `status:`. The JTBD twin of the architect side's surface 3 (RFC-010 / P318).
-#
-# tdd-review: structural-permitted (justification: P176 — agent behaviour is
-# prompt-driven with no skill-invocation harness to exercise the verdict
-# behaviourally; ADR-052 Surface 2 structural-justified case, NOT an ADR-005
-# Permitted Exception). When P176 lands, upgrade to a behavioural test that
-# feeds the agent a change citing an unratified persona/job and asserts the
-# verdict. The single-artifact predicate IS behaviourally tested today — see
-# packages/jtbd/scripts/test/is-job-or-persona-unconfirmed.bats.
-#
-# Cross-reference:
-#   ADR-068 (JTBD oversight marker + drain — surface 3 amendment 2026-05-27)
-#   ADR-074 (Confirm a decision's substance before building dependent work)
-#   ADR-066 (oversight marker; orthogonal status/oversight axes)
-#   RFC-011 / P323 (this enforcement surface); RFC-010 / P318 (the ADR-side twin)
-#   ADR-052 Surface 2 (structural-justified verdict) + P176 (harness gap)
-#   @jtbd JTBD-202 (pre-flight governance checks) / JTBD-101 (extend the suite)
-
-setup() {
-  AGENT_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  AGENT_FILE="${AGENT_DIR}/agent.md"
-}
-
-@test "agent.md lists [Unratified Dependency] as a verdict/issue type (ADR-068 surface 3)" {
-  run grep -n '\[Unratified Dependency\]' "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "agent.md has an 'Unratified Dependency (build-upon guard' section citing ADR-068" {
-  run grep -niE "Unratified Dependency \(build-upon guard" "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-  run grep -n "ADR-068" "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "agent.md keys the flag on the oversight marker, NOT on status (orthogonal axes)" {
-  # ADR-066 / user correction 2026-05-27: building on a ratified-but-proposed job is fine.
-  run grep -niE "NEVER on .?status|not .?\`?status|orthogonal" "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "agent.md invokes the predicate by exit code (the jtbd agent has Bash; not a prose-grep mirror)" {
-  run grep -n "wr-jtbd-is-job-or-persona-unconfirmed" "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-  run grep -niE "exit code|exit 0|exit 1" "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "agent.md guards against over-firing on ambient alignment (inverse-P078 / explicit cite)" {
-  run grep -niE "explicit(ly)? cite|ambient alignment|over-?fire|NOT fire on that mere match" "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "agent.md routes the fix to /wr-jtbd:confirm-jobs-and-personas (the surface-2 drain)" {
-  run grep -n "/wr-jtbd:confirm-jobs-and-personas" "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-}

package/agents/test/jtbd-verdict-contract.bats

@@ -1,57 +0,0 @@
-#!/usr/bin/env bats
-# Doc-lint guard: wr-jtbd:agent output contract — the agent MUST emit a
-# structured inline verdict in every response, regardless of the
-# /tmp/jtbd-verdict file write. Closes P037 (JTBD reviewer sometimes
-# returns a bare verdict without remediation reason).
-#
-# Structural assertion — Permitted Exception to the source-grep ban
-# (ADR-005 / P011).
-#
-# Cross-reference:
-#   P037 (JTBD reviewer bare-verdict bug)
-#   @jtbd JTBD-001 (enforce governance without slowing down)
-
-setup() {
-  AGENT_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  AGENT_FILE="${AGENT_DIR}/agent.md"
-}
-
-@test "agent.md contains inline verdict templates (How to Report section)" {
-  run grep -n "How to Report" "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "agent.md declares inline output is REQUIRED in every response (P037)" {
-  # Must say inline output is required / must be emitted every time.
-  run grep -niE "(always|every|must|required).*inline|inline.*(always|every|must|required)" "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "agent.md declares the primary communication channel is inline, not the file (P037)" {
-  # Must clarify the verdict file is an internal signal, not a replacement for inline output
-  run grep -niE "(primary|authoritative).*(inline|user-facing|response)|inline.*(primary|authoritative)" "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "agent.md requires a specific verdict line on every response (PASS or ISSUES FOUND)" {
-  # The verdict line is explicit - bold JTBD Review: PASS / ISSUES FOUND
-  run grep -nE "JTBD Review: (PASS|ISSUES FOUND|JOB UPDATE NEEDED|PERSONA UPDATE NEEDED)" "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "agent.md requires remediation guidance on every FAIL verdict (P037)" {
-  # On ISSUES FOUND the agent must include actionable guidance (what to change / what job / what fix)
-  run grep -niE "remediation|actionable|what (should|would need)|fix:|issue:" "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "agent.md cross-references P037 so readers can trace the contract's origin" {
-  run grep -n "P037" "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "agent.md explicitly forbids bare verdict without reason (P037)" {
-  # Directly addresses the observed bug: verdict without body
-  run grep -niE "(must not|MUST NOT|do not|DO NOT|never) (emit|return|produce) (a )?bare|without (reason|remediation|detail|actionable|explanation)" "$AGENT_FILE"
-  [ "$status" -eq 0 ]
-}

package/hooks/test/jtbd-enforce-scope.bats

@@ -1,199 +0,0 @@
-#!/usr/bin/env bats
-
-# Tests for jtbd-enforce-edit.sh — verifies broadened scope with exclusions.
-# All tests are functional: they execute the hook with mock JSON input
-# and assert on exit status and BLOCKED output. Source-grep assertions
-# were removed (P011) — they over-specified the implementation and
-# false-positived on legitimate refactors.
-
-setup() {
-  SCRIPT_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  HOOK="$SCRIPT_DIR/jtbd-enforce-edit.sh"
-  ORIG_DIR="$PWD"
-  TEST_DIR=$(mktemp -d)
-  cd "$TEST_DIR"
-}
-
-teardown() {
-  cd "$ORIG_DIR"
-  rm -rf "$TEST_DIR"
-}
-
-# Helper: run the hook with a mock JSON input for a given file path
-run_hook_with_file() {
-  local file_path="$1"
-  local json="{\"tool_input\":{\"file_path\":\"${file_path}\"},\"session_id\":\"test-session-$$\"}"
-  echo "$json" | bash "$HOOK"
-}
-
-# Helper: assert the hook exits 0 and does NOT emit BLOCKED for the given path
-assert_path_allowed() {
-  local file_path="$1"
-  run run_hook_with_file "$file_path"
-  [ "$status" -eq 0 ]
-  [[ "$output" != *"BLOCKED"* ]]
-}
-
-# Helper: assert the hook BLOCKS the given path
-assert_path_blocked() {
-  local file_path="$1"
-  run run_hook_with_file "$file_path"
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"BLOCKED"* ]]
-}
-
-# --- Exclusion tests (functional) ---
-
-# Claude Code passes absolute file paths in tool_input.file_path, so tests
-# use $PWD-prefixed paths to match the real shape (after the P004 root check).
-
-@test "enforce: excludes VCS-internal .git/ plumbing files (P458)" {
-  assert_path_allowed "$PWD/.git/cruise-brake-commit-msg.txt"
-  assert_path_allowed "$PWD/.git/COMMIT_EDITMSG"
-}
-
-@test "enforce: excludes CSS files" {
-  assert_path_allowed "$PWD/src/styles.css"
-}
-
-@test "enforce: excludes image files" {
-  assert_path_allowed "$PWD/public/logo.png"
-}
-
-@test "enforce: excludes font files" {
-  assert_path_allowed "$PWD/public/fonts/regular.woff"
-}
-
-@test "enforce: excludes lockfiles" {
-  assert_path_allowed "$PWD/package-lock.json"
-}
-
-@test "enforce: excludes changeset files" {
-  assert_path_allowed "$PWD/.changeset/some-change.md"
-}
-
-@test "enforce: excludes memory files" {
-  assert_path_allowed "$PWD/MEMORY.md"
-}
-
-@test "enforce: excludes plan files" {
-  assert_path_allowed "$PWD/.claude/plans/2026-01-01-plan.md"
-}
-
-@test "enforce: excludes risk reports" {
-  assert_path_allowed "$PWD/.risk-reports/2026-01-01.md"
-}
-
-@test "enforce: excludes RISK-POLICY.md" {
-  assert_path_allowed "$PWD/RISK-POLICY.md"
-}
-
-@test "enforce: does NOT exempt by UI-only extension (ADR-007/008)" {
-  # ADR-007/008 broadened scope: the hook must gate UI files like any
-  # other source file, not silently allow them.
-  assert_path_blocked "$PWD/src/Component.tsx"
-}
-
-@test "enforce: does NOT exempt .html files (ADR-007/008)" {
-  assert_path_blocked "$PWD/public/index.html"
-}
-
-@test "enforce: does NOT exempt .vue files (ADR-007/008)" {
-  assert_path_blocked "$PWD/src/App.vue"
-}
-
-# --- Functional tests (execute hook with mock JSON) ---
-
-@test "functional: exempts docs/jtbd/ files" {
-  run run_hook_with_file "docs/jtbd/solo-developer/persona.md"
-  [ "$status" -eq 0 ]
-  [[ "$output" != *"BLOCKED"* ]]
-}
-
-@test "functional: exempts docs/jtbd/ job files" {
-  run run_hook_with_file "docs/jtbd/solo-developer/JTBD-001-governance.proposed.md"
-  [ "$status" -eq 0 ]
-  [[ "$output" != *"BLOCKED"* ]]
-}
-
-@test "functional: exempts docs/jtbd/README.md" {
-  run run_hook_with_file "docs/jtbd/README.md"
-  [ "$status" -eq 0 ]
-  [[ "$output" != *"BLOCKED"* ]]
-}
-
-@test "functional: docs/JOBS_TO_BE_DONE.md is NOT an exempt governance artefact (ADR-008 Option 3, P019)" {
-  # Legacy single-file path is no longer a recognised governance artefact.
-  # When docs/jtbd/ exists the gate should fire against edits to the
-  # legacy file (it is not exempt). When docs/jtbd/ does not exist the
-  # gate blocks with an update-guide suggestion (covered separately).
-  mkdir -p docs/jtbd
-  echo "# Index" > docs/jtbd/README.md
-  run run_hook_with_file "docs/JOBS_TO_BE_DONE.md"
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"BLOCKED"* ]]
-}
-
-@test "functional: blocks src/index.ts when no JTBD docs exist" {
-  run run_hook_with_file "src/index.ts"
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"BLOCKED"* ]]
-  [[ "$output" == *"wr-jtbd:update-guide"* ]]
-}
-
-@test "functional: blocks src/index.ts when docs/jtbd exists (needs review)" {
-  mkdir -p docs/jtbd
-  echo "# Index" > docs/jtbd/README.md
-  run run_hook_with_file "src/index.ts"
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"BLOCKED"* ]]
-  [[ "$output" == *"wr-jtbd:agent"* ]]
-}
-
-# --- Story maps + stories exemptions (P170 Phase 2 Slice 2.5 / ADR-060 amendment 2026-05-12) ---
-# ADR-060 § Phase 2 encoding amendment lines 481-496 mandates exempting
-# `docs/story-maps/**/*.html` and `docs/stories/**/*.md` from the 5
-# enforce-edit gates. Mirrors the docs/problems and docs/jtbd exemptions —
-# these are governance-managed surfaces with their own capture/manage skills.
-
-@test "functional: exempts docs/story-maps/ HTML in per-state subdir (P170 / ADR-060)" {
-  assert_path_allowed "$PWD/docs/story-maps/draft/STORY-MAP-001-rfc-framework.html"
-}
-
-@test "functional: exempts docs/story-maps/ HTML in completed subdir (P170 / ADR-060)" {
-  assert_path_allowed "$PWD/docs/story-maps/completed/STORY-MAP-002-foo.html"
-}
-
-@test "functional: exempts docs/story-maps/README.md (P170 / ADR-060)" {
-  assert_path_allowed "$PWD/docs/story-maps/README.md"
-}
-
-@test "functional: exempts docs/stories/ markdown in per-state subdir (P170 / ADR-060)" {
-  assert_path_allowed "$PWD/docs/stories/draft/STORY-001-some-slice.md"
-}
-
-@test "functional: exempts docs/stories/ markdown in done subdir (P170 / ADR-060)" {
-  assert_path_allowed "$PWD/docs/stories/done/STORY-002-other.md"
-}
-
-@test "functional: exempts docs/stories/README.md (P170 / ADR-060)" {
-  assert_path_allowed "$PWD/docs/stories/README.md"
-}
-
-# --- Retros exemption (P203) ---
-# `docs/retros/*` is the ask-hygiene + run-retro narrative trail written
-# routinely by `/wr-retrospective:run-retro` Step 2d + Step 5. Routine
-# appends should not fire the JTBD gate — they are not user-job content.
-# Mirrors the docs/problems and docs/jtbd peer-plugin-policy exemptions.
-
-@test "functional: exempts docs/retros/ ask-hygiene file (P203)" {
-  assert_path_allowed "$PWD/docs/retros/2026-06-06-some-iter-ask-hygiene.md"
-}
-
-@test "functional: exempts docs/retros/ analyze-context file (P203)" {
-  assert_path_allowed "$PWD/docs/retros/2026-06-06-context-analysis.md"
-}
-
-@test "functional: exempts docs/retros/ retro narrative file (P203)" {
-  assert_path_allowed "$PWD/docs/retros/2026-06-06-session-wrap.md"
-}

package/hooks/test/jtbd-eval-once-per-session.bats

@@ -1,87 +0,0 @@
-#!/usr/bin/env bats
-
-# P095 / ADR-038: jtbd-eval.sh UserPromptSubmit hook must emit the full
-# MANDATORY block only on the first prompt of a session; subsequent
-# prompts emit a terse reminder.
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../.." && pwd)"
-  HOOK="$REPO_ROOT/packages/jtbd/hooks/jtbd-eval.sh"
-
-  WORKDIR="$(mktemp -d)"
-  mkdir -p "$WORKDIR/docs/jtbd"
-  : > "$WORKDIR/docs/jtbd/README.md"
-
-  SID="jtbd-eval-test-$$-$RANDOM"
-}
-
-teardown() {
-  rm -f "/tmp/jtbd-announced-${SID}"
-  rm -f "/tmp/jtbd-announced-${SID}-alt"
-  rm -rf "$WORKDIR"
-}
-
-run_hook() {
-  local sid="$1"
-  (cd "$WORKDIR" && echo "{\"session_id\":\"$sid\"}" | bash "$HOOK")
-}
-
-@test "jtbd-eval: first invocation emits the full MANDATORY block" {
-  run run_hook "$SID"
-  [ "$status" -eq 0 ]
-  [ "${#output}" -gt 400 ]
-  [[ "$output" == *"MANDATORY JTBD CHECK"* ]]
-  [[ "$output" == *"wr-jtbd:agent"* ]]
-  [[ "$output" == *"SCOPE:"* ]]
-}
-
-@test "jtbd-eval: first invocation writes the announcement marker" {
-  run run_hook "$SID"
-  [ -f "/tmp/jtbd-announced-${SID}" ]
-}
-
-@test "jtbd-eval: second invocation emits only a terse reminder" {
-  run_hook "$SID" >/dev/null
-  run run_hook "$SID"
-  [ "$status" -eq 0 ]
-  [ "${#output}" -lt 250 ]
-  [[ "$output" == *"jtbd"* ]] || [[ "$output" == *"JTBD"* ]]
-  [[ "$output" == *"wr-jtbd:agent"* ]]
-  [[ "$output" != *"Does NOT apply to"* ]]
-}
-
-@test "jtbd-eval: terse reminder preserves the MANDATORY signal word" {
-  run_hook "$SID" >/dev/null
-  run run_hook "$SID"
-  [[ "$output" == *"MANDATORY"* ]] || [[ "$output" == *"REQUIRED"* ]] || [[ "$output" == *"NON-OPTIONAL"* ]]
-}
-
-@test "jtbd-eval: terse reminder names the trigger artifact" {
-  run_hook "$SID" >/dev/null
-  run run_hook "$SID"
-  [[ "$output" == *"docs/jtbd"* ]]
-}
-
-@test "jtbd-eval: different session_id re-emits the full block" {
-  run_hook "$SID" >/dev/null
-  local SID2="${SID}-alt"
-  run run_hook "$SID2"
-  [ "${#output}" -gt 400 ]
-  [[ "$output" == *"MANDATORY JTBD CHECK"* ]]
-  rm -f "/tmp/jtbd-announced-${SID2}"
-}
-
-@test "jtbd-eval: empty session_id emits the full block and writes no marker" {
-  run run_hook ""
-  [ "${#output}" -gt 400 ]
-  [[ "$output" == *"MANDATORY JTBD CHECK"* ]]
-  [ ! -f "/tmp/jtbd-announced-" ]
-}
-
-@test "jtbd-eval: absent docs/jtbd/ emits the no-docs NOTE (unchanged by this ADR)" {
-  local NO_JTBD="$(mktemp -d)"
-  run bash -c "cd '$NO_JTBD' && echo '{\"session_id\":\"$SID\"}' | bash '$HOOK'"
-  [[ "$output" == *"no docs/jtbd/ directory"* ]]
-  [[ "$output" != *"MANDATORY JTBD CHECK"* ]]
-  rm -rf "$NO_JTBD"
-}

package/hooks/test/jtbd-eval-scope.bats

@@ -1,45 +0,0 @@
-#!/usr/bin/env bats
-
-# Tests for jtbd-eval.sh (UserPromptSubmit) — verifies the injected scope
-# exclusion text lists governance docs that the PreToolUse gate already
-# exempts (P029). Without this, the LLM wastes time delegating to the
-# jtbd-lead for files the edit gate would allow anyway.
-
-setup() {
-  SCRIPT_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  HOOK="$SCRIPT_DIR/jtbd-eval.sh"
-  ORIG_DIR="$PWD"
-  TEST_DIR=$(mktemp -d)
-  cd "$TEST_DIR"
-  mkdir -p docs/jtbd
-  echo "# Index" > docs/jtbd/README.md
-}
-
-teardown() {
-  cd "$ORIG_DIR"
-  rm -rf "$TEST_DIR"
-}
-
-@test "eval: scope text mentions problem files exemption (P029)" {
-  run bash "$HOOK" </dev/null
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"docs/problems/"* ]] || [[ "$output" == *"problem tickets"* ]]
-}
-
-@test "eval: scope text mentions BRIEFING.md exemption (P029)" {
-  run bash "$HOOK" </dev/null
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"BRIEFING"* ]]
-}
-
-@test "eval: scope text mentions docs/briefing/ exemption (P100)" {
-  run bash "$HOOK" </dev/null
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"docs/briefing/"* ]]
-}
-
-@test "eval: scope text mentions RISK-POLICY exemption (P029)" {
-  run bash "$HOOK" </dev/null
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"RISK-POLICY"* ]]
-}

package/hooks/test/jtbd-eval.bats

@@ -1,69 +0,0 @@
-#!/usr/bin/env bats
-
-# Tests for jtbd-eval.sh — verifies JTBD suggestion fires for any project.
-# Canonical layout is docs/jtbd/ only (ADR-008 Option 3 chosen 2026-04-20).
-# Legacy docs/JOBS_TO_BE_DONE.md is NOT consulted at runtime (P019).
-
-setup() {
-  SCRIPT_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  HOOK="$SCRIPT_DIR/jtbd-eval.sh"
-  ORIG_DIR="$PWD"
-  TEST_DIR=$(mktemp -d)
-  cd "$TEST_DIR"
-}
-
-teardown() {
-  cd "$ORIG_DIR"
-  rm -rf "$TEST_DIR"
-}
-
-@test "eval: suggests update-guide when docs/jtbd/ missing (no UI files)" {
-  # No UI files, no docs — should still suggest
-  run bash "$HOOK" </dev/null
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"wr-jtbd:update-guide"* ]]
-}
-
-@test "eval: suggests update-guide when docs/jtbd/ missing (with UI files)" {
-  mkdir -p src
-  touch src/App.tsx
-  run bash "$HOOK" </dev/null
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"wr-jtbd:update-guide"* ]]
-}
-
-@test "eval: injects enforcement when docs/jtbd/README.md exists" {
-  mkdir -p docs/jtbd
-  echo "# Index" > docs/jtbd/README.md
-  run bash "$HOOK" </dev/null
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"MANDATORY JTBD CHECK"* ]]
-}
-
-@test "eval: does NOT consult legacy docs/JOBS_TO_BE_DONE.md (ADR-008 Option 3, P019)" {
-  # Legacy single-file layout — gate must NOT inject the enforcement
-  # instruction; instead it should suggest update-guide so the project
-  # migrates into the directory layout.
-  mkdir -p docs
-  echo "# Jobs" > docs/JOBS_TO_BE_DONE.md
-  run bash "$HOOK" </dev/null
-  [ "$status" -eq 0 ]
-  [[ "$output" != *"MANDATORY JTBD CHECK"* ]]
-  [[ "$output" == *"wr-jtbd:update-guide"* ]]
-}
-
-@test "eval: uses docs/jtbd/ when both layouts coexist (ADR-008 Option 3)" {
-  mkdir -p docs/jtbd
-  echo "# Index" > docs/jtbd/README.md
-  echo "# Jobs" > docs/JOBS_TO_BE_DONE.md
-  run bash "$HOOK" </dev/null
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"docs/jtbd"* ]]
-  [[ "$output" == *"MANDATORY JTBD CHECK"* ]]
-}
-
-@test "eval: does not reference UI-only scoping in output" {
-  run bash "$HOOK" </dev/null
-  [ "$status" -eq 0 ]
-  [[ "$output" != *"UI files"* ]]
-}