@windyroad/itil 0.61.1 → 0.61.2

package/scripts/test/reconcile-story-maps.bats

@@ -1,74 +0,0 @@
-#!/usr/bin/env bats
-# Behavioural fixtures for reconcile-story-maps.sh + bin shim + skill
-# (P170 Phase 2 Slice 5 — sibling of reconcile-stories.sh / reconcile-rfcs.sh).
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../.." && pwd)"
-  SCRIPT="${REPO_ROOT}/packages/itil/scripts/reconcile-story-maps.sh"
-  BIN_SHIM="${REPO_ROOT}/packages/itil/bin/wr-itil-reconcile-story-maps"
-  SKILL_FILE="${REPO_ROOT}/packages/itil/skills/reconcile-story-maps/SKILL.md"
-
-  TMPROOT=$(mktemp -d)
-  ORIG_DIR="$PWD"
-  cd "$TMPROOT"
-}
-
-teardown() {
-  cd "$ORIG_DIR"
-  rm -rf "$TMPROOT"
-}
-
-@test "reconcile-story-maps: script exists and is executable" {
-  [ -f "$SCRIPT" ]
-  [ -x "$SCRIPT" ]
-}
-
-@test "reconcile-story-maps: bin shim exists, executable, exec's the script" {
-  [ -f "$BIN_SHIM" ]
-  [ -x "$BIN_SHIM" ]
-  run grep -E 'exec.*scripts/reconcile-story-maps\.sh' "$BIN_SHIM"
-  [ "$status" -eq 0 ]
-}
-
-@test "reconcile-story-maps: exits 2 when README is missing" {
-  mkdir -p docs/story-maps
-  run bash "$SCRIPT" docs/story-maps
-  [ "$status" -eq 2 ]
-}
-
-@test "reconcile-story-maps: exits 0 on empty story-maps dir + empty README" {
-  mkdir -p docs/story-maps/draft docs/story-maps/accepted docs/story-maps/in-progress docs/story-maps/completed docs/story-maps/archived
-  echo "# Story Maps" > docs/story-maps/README.md
-  run bash "$SCRIPT" docs/story-maps
-  [ "$status" -eq 0 ]
-}
-
-@test "reconcile-story-maps: detects STALE when filesystem has a map not in README" {
-  mkdir -p docs/story-maps/draft
-  touch docs/story-maps/draft/STORY-MAP-007-foo.html
-  echo "# Story Maps" > docs/story-maps/README.md
-  run bash "$SCRIPT" docs/story-maps
-  [ "$status" -eq 1 ]
-  [[ "$output" == *"STALE"* ]]
-  [[ "$output" == *"STORY-MAP-007"* ]]
-}
-
-@test "reconcile-story-maps: detects MISSING when README claims a map that isn't on disk" {
-  mkdir -p docs/story-maps/draft
-  cat > docs/story-maps/README.md <<'EOF'
-# Story Maps
-| ID | Status |
-|----|--------|
-| STORY-MAP-007 | draft |
-EOF
-  run bash "$SCRIPT" docs/story-maps
-  [ "$status" -eq 1 ]
-  [[ "$output" == *"MISSING"* ]]
-  [[ "$output" == *"STORY-MAP-007"* ]]
-}
-
-@test "reconcile-story-maps: SKILL.md exists with canonical name" {
-  [ -f "$SKILL_FILE" ]
-  run grep -E '^name: wr-itil:reconcile-story-maps$' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}

package/scripts/test/release-watch-poll-loop.bats

@@ -1,180 +0,0 @@
-#!/usr/bin/env bats
-
-# P143: scripts/release-watch.sh must absorb changesets/action workflow
-# latency by polling `gh pr list` for up to 120s before exiting "no open
-# release PR found". The release PR is created/updated asynchronously by
-# the changesets/action GitHub workflow ~30-120s after `git push`; the
-# script's first call routinely raced this window and exited 1.
-#
-# Behavioural test (ADR-037 + P081 — behavioural over structural grep).
-# Extracts `find_release_pr` from the script via awk, sources it, and
-# exercises it against a PATH-shadowed `gh` mock + stubbed `sleep`. The
-# mock consumes a comma-delimited iteration sequence (e.g. "empty,empty,ok")
-# so each iteration's `gh pr list` payload is deterministic.
-#
-# Mirrors the extraction + PATH-shadow pattern in
-# scripts/repo-local-skills/install-updates/test/install-updates-step-7-retry-rollback.bats.
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../.." && pwd)"
-  SCRIPT="$REPO_ROOT/scripts/release-watch.sh"
-
-  FN_FILE="$BATS_TEST_TMPDIR/find-release-pr.sh"
-  awk '
-    /^find_release_pr\(\) \{/ { in_fn=1 }
-    in_fn { print }
-    in_fn && /^\}/ { exit }
-  ' "$SCRIPT" > "$FN_FILE"
-}
-
-# Stand up a PATH-shadowing `gh` mock that consumes a comma-delimited
-# iteration sequence ("empty" / "ok"). Each `gh pr list` call advances the
-# pointer; "empty" returns `[]`, "ok" returns a one-PR JSON array.
-make_gh_mock() {
-  local pattern="$1"
-  local bindir="$BATS_TEST_TMPDIR/bin"
-  mkdir -p "$bindir"
-  printf '%s' "$pattern" > "$BATS_TEST_TMPDIR/gh-pattern"
-  printf '0' > "$BATS_TEST_TMPDIR/gh-counter"
-  : > "$BATS_TEST_TMPDIR/gh-log"
-  cat > "$bindir/gh" <<'MOCK'
-#!/usr/bin/env bash
-echo "$*" >> "$BATS_TEST_TMPDIR/gh-log"
-case "$1 $2" in
-  "pr list")
-    pattern=$(cat "$BATS_TEST_TMPDIR/gh-pattern")
-    count=$(cat "$BATS_TEST_TMPDIR/gh-counter")
-    count=$((count + 1))
-    printf '%s' "$count" > "$BATS_TEST_TMPDIR/gh-counter"
-    next=$(printf '%s' "$pattern" | cut -d, -f"$count")
-    if [ "$next" = "ok" ]; then
-      echo '[{"number":99,"url":"https://github.com/example/repo/pull/99"}]'
-    else
-      echo '[]'
-    fi
-    exit 0
-    ;;
-  *) exit 0 ;;
-esac
-MOCK
-  chmod +x "$bindir/gh"
-  PATH="$bindir:$PATH"
-  export PATH
-}
-
-# Suppress real sleeps during tests — 120s wall-clock is unacceptable.
-# Records each call so we can count iterations.
-stub_sleep() {
-  : > "$BATS_TEST_TMPDIR/sleep-log"
-  eval 'sleep() { echo "$1" >> "$BATS_TEST_TMPDIR/sleep-log"; }'
-  export -f sleep
-}
-
-count_gh_pr_list_calls() {
-  [ -f "$BATS_TEST_TMPDIR/gh-log" ] || { echo 0; return; }
-  awk '/^pr list/ { n++ } END { print n+0 }' "$BATS_TEST_TMPDIR/gh-log"
-}
-
-count_sleep_calls() {
-  [ -f "$BATS_TEST_TMPDIR/sleep-log" ] || { echo 0; return; }
-  awk 'END { print NR+0 }' "$BATS_TEST_TMPDIR/sleep-log"
-}
-
-@test "find_release_pr extracted from release-watch.sh is non-empty" {
-  [ -s "$FN_FILE" ] || { echo "find_release_pr missing from $SCRIPT"; return 1; }
-}
-
-@test "P143: PR exists on first iteration — fast path, one gh call, no sleep" {
-  [ -s "$FN_FILE" ] || { echo "find_release_pr missing from $SCRIPT"; return 1; }
-  # shellcheck disable=SC1090
-  source "$FN_FILE"
-  make_gh_mock "ok"
-  stub_sleep
-
-  run find_release_pr
-  [ "$status" -eq 0 ]
-  # stdout is "<number>\t<url>"
-  [[ "$output" == *"99"* ]]
-  [[ "$output" == *"https://github.com/example/repo/pull/99"* ]]
-
-  [ "$(count_gh_pr_list_calls)" -eq 1 ]
-  [ "$(count_sleep_calls)" -eq 0 ]
-}
-
-@test "P143: PR appears on iteration 3 — three gh calls, two sleeps, returns the PR" {
-  [ -s "$FN_FILE" ] || { echo "find_release_pr missing from $SCRIPT"; return 1; }
-  # shellcheck disable=SC1090
-  source "$FN_FILE"
-  make_gh_mock "empty,empty,ok"
-  stub_sleep
-
-  run find_release_pr
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"99"* ]]
-
-  [ "$(count_gh_pr_list_calls)" -eq 3 ]
-  [ "$(count_sleep_calls)" -eq 2 ]
-}
-
-@test "P143: empty for full 12 iterations — 12 gh calls, exit 1" {
-  [ -s "$FN_FILE" ] || { echo "find_release_pr missing from $SCRIPT"; return 1; }
-  # shellcheck disable=SC1090
-  source "$FN_FILE"
-  # 12 empties, no 13th column needed — function should give up after 12.
-  make_gh_mock "empty,empty,empty,empty,empty,empty,empty,empty,empty,empty,empty,empty"
-  stub_sleep
-
-  run find_release_pr
-  [ "$status" -ne 0 ]
-
-  [ "$(count_gh_pr_list_calls)" -eq 12 ]
-  # 11 sleeps between 12 iterations (no trailing sleep after the final
-  # empty result — that would burn 10s for nothing).
-  [ "$(count_sleep_calls)" -eq 11 ]
-}
-
-@test "P143: RELEASE_WATCH_VERBOSE=1 prints poll progress to stderr" {
-  [ -s "$FN_FILE" ] || { echo "find_release_pr missing from $SCRIPT"; return 1; }
-  # shellcheck disable=SC1090
-  source "$FN_FILE"
-  make_gh_mock "empty,empty,ok"
-  stub_sleep
-
-  RELEASE_WATCH_VERBOSE=1 run find_release_pr
-  [ "$status" -eq 0 ]
-  # `bats run` merges stdout + stderr in $output by default.
-  [[ "$output" == *"Polling"* ]] || [[ "$output" == *"attempt"* ]]
-}
-
-@test "P143: default (verbose unset) does NOT print poll progress" {
-  [ -s "$FN_FILE" ] || { echo "find_release_pr missing from $SCRIPT"; return 1; }
-  # shellcheck disable=SC1090
-  source "$FN_FILE"
-  make_gh_mock "empty,empty,ok"
-  stub_sleep
-
-  unset RELEASE_WATCH_VERBOSE
-  run find_release_pr
-  [ "$status" -eq 0 ]
-  # Output is the final tab-separated PR line ONLY — no progress lines.
-  [[ "$output" != *"Polling"* ]]
-  [[ "$output" != *"attempt"* ]]
-}
-
-@test "P143: returns tab-separated number and URL on success (parseable)" {
-  [ -s "$FN_FILE" ] || { echo "find_release_pr missing from $SCRIPT"; return 1; }
-  # shellcheck disable=SC1090
-  source "$FN_FILE"
-  make_gh_mock "ok"
-  stub_sleep
-
-  run find_release_pr
-  [ "$status" -eq 0 ]
-  # Caller parses with `cut -f1` / `cut -f2` — the contract is one line,
-  # tab-separated.
-  local first_field second_field
-  first_field=$(printf '%s\n' "$output" | head -1 | cut -f1)
-  second_field=$(printf '%s\n' "$output" | head -1 | cut -f2)
-  [ "$first_field" = "99" ]
-  [ "$second_field" = "https://github.com/example/repo/pull/99" ]
-}

package/scripts/test/resolve-governance-plugin-dirs.bats

@@ -1,78 +0,0 @@
-#!/usr/bin/env bats
-
-# @problem P382 — AFK work-problems `claude -p` iter subprocesses load only
-#                 USER-scoped enabledPlugins; project-scoped governance plugins
-#                 must be passed explicitly via --plugin-dir. This helper emits
-#                 the --plugin-dir argument pairs the Step 5 dispatch splices
-#                 into its claude -p invocation.
-#
-# Contract: `resolve-governance-plugin-dirs.sh` reads $PATH + WR_GOVERNANCE_PLUGINS
-# and emits, per resolvable plugin, two stdout lines — `--plugin-dir` then the
-# plugin root dir — for `mapfile -t` consumption.
-#
-# Load-bearing behaviour (ADR-080): the version is selected by HIGHEST-SEMVER
-# walk of the cache parent, NOT by $PATH order (which is frozen-stale
-# mid-session per P343). The fixture deliberately places a LOWER version's bin
-# earlier on $PATH and asserts the HIGHER version's dir is still emitted.
-#
-# @adr ADR-049 (bin/ on PATH shim — adopter-safe script resolution)
-# @adr ADR-080 (highest-version-wins resolution)
-# @adr ADR-052 (behavioural test per P081)
-# @adr ADR-032 (Step 5 dispatch contract — P382 amendment)
-# @jtbd JTBD-001 (Enforce Governance — iter commits ship gated)
-# @jtbd JTBD-006 (Progress Backlog AFK — full governance surface inside iters)
-
-setup() {
-  SCRIPTS_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  SCRIPT="$SCRIPTS_DIR/resolve-governance-plugin-dirs.sh"
-  CACHE="$(mktemp -d)"
-  # Synthetic marketplace-cache layout (ADR-080 SQ-080-5 — no source-repo
-  # cohabitation). wr-architect has two cached versions; wr-itil has one.
-  mkdir -p "$CACHE/windyroad/wr-architect/0.16.0/bin"
-  mkdir -p "$CACHE/windyroad/wr-architect/0.17.3/bin"
-  mkdir -p "$CACHE/windyroad/wr-itil/0.51.0/bin"
-}
-
-teardown() {
-  rm -rf "$CACHE"
-}
-
-@test "resolve-governance-plugin-dirs: script exists and is executable" {
-  [ -f "$SCRIPT" ]
-  [ -x "$SCRIPT" ]
-}
-
-@test "resolve-governance-plugin-dirs: emits --plugin-dir pairs for resolvable plugins" {
-  PATH="$CACHE/windyroad/wr-architect/0.17.3/bin:$CACHE/windyroad/wr-itil/0.51.0/bin:/usr/bin:/bin" \
-    WR_GOVERNANCE_PLUGINS="wr-architect wr-itil" run "$SCRIPT"
-  [ "$status" -eq 0 ]
-  [ "${lines[0]}" = "--plugin-dir" ]
-  [ "${lines[1]}" = "$CACHE/windyroad/wr-architect/0.17.3" ]
-  [ "${lines[2]}" = "--plugin-dir" ]
-  [ "${lines[3]}" = "$CACHE/windyroad/wr-itil/0.51.0" ]
-}
-
-@test "resolve-governance-plugin-dirs: picks highest semver, NOT \$PATH order (ADR-080)" {
-  # Lower version (0.16.0) earlier on PATH than higher (0.17.3) — the resolver
-  # must still emit 0.17.3 (cache-parent highest-semver walk, not PATH order).
-  PATH="$CACHE/windyroad/wr-architect/0.16.0/bin:$CACHE/windyroad/wr-architect/0.17.3/bin:/usr/bin:/bin" \
-    WR_GOVERNANCE_PLUGINS="wr-architect" run "$SCRIPT"
-  [ "$status" -eq 0 ]
-  [ "${lines[0]}" = "--plugin-dir" ]
-  [ "${lines[1]}" = "$CACHE/windyroad/wr-architect/0.17.3" ]
-}
-
-@test "resolve-governance-plugin-dirs: skips unresolvable plugins silently" {
-  PATH="$CACHE/windyroad/wr-itil/0.51.0/bin:/usr/bin:/bin" \
-    WR_GOVERNANCE_PLUGINS="wr-architect wr-itil wr-not-installed" run "$SCRIPT"
-  [ "$status" -eq 0 ]
-  # Only wr-itil is on PATH → exactly one pair, no wr-architect / wr-not-installed.
-  [ "${#lines[@]}" -eq 2 ]
-  [ "${lines[1]}" = "$CACHE/windyroad/wr-itil/0.51.0" ]
-}
-
-@test "resolve-governance-plugin-dirs: no governance plugins on PATH → empty output, exit 0" {
-  PATH="/usr/bin:/bin" WR_GOVERNANCE_PLUGINS="wr-architect wr-itil" run "$SCRIPT"
-  [ "$status" -eq 0 ]
-  [ -z "$output" ]
-}

package/scripts/test/rfc-stories-extension.bats

@@ -1,175 +0,0 @@
-#!/usr/bin/env bats
-# Behavioural fixtures for the RFC frontmatter `stories:` extension
-# (P170 Phase 2 Slice 11 — ADR-060 lines 255-270 + 296).
-#
-# Per ADR-052: behavioural assertions on observable artefact state, not
-# structural prose-grep on SKILL.md. The load-bearing surfaces under
-# test are:
-#   1. RFC frontmatter spec in `docs/rfcs/README.md` includes the
-#      `stories:` field with the 0..N + ORDERED contract.
-#   2. The Slice 2b helper `update-rfc-references-section.sh` accepts
-#      "Stories" as a section name and:
-#      (a) renders a `## Stories` section from `stories: [STORY-NNN, ...]`
-#          frontmatter in execution order
-#      (b) applies lazy-empty discipline — empty `stories: []` produces
-#          NO `## Stories` section (atomic RFC, JTBD-101 friction guard)
-#   3. capture-rfc + manage-rfc SKILL.md both document the extension
-#      (existence checks; not prose grep of behaviour, but presence of
-#      load-bearing identifiers like `--stories` and `update-rfc-
-#      references-section.sh`).
-#
-# @problem P170
-# @jtbd JTBD-008 (Decompose a Fix Into Coordinated Changes — `stories:`
-#                  array IS the decomposition mechanism)
-# @jtbd JTBD-101 (atomic-fix-adopter friction guard — empty stories: []
-#                  ships as atomic RFC without per-story dispatch)
-# @adr  ADR-060  (Problem-RFC-Story framework — RFC frontmatter
-#                  extension at line 255-270 + skills extension line 296)
-# @adr  ADR-052  (Behavioural-tests-default)
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../.." && pwd)"
-  RFC_README="${REPO_ROOT}/docs/rfcs/README.md"
-  CAPTURE_RFC="${REPO_ROOT}/packages/itil/skills/capture-rfc/SKILL.md"
-  MANAGE_RFC="${REPO_ROOT}/packages/itil/skills/manage-rfc/SKILL.md"
-  HELPER="${REPO_ROOT}/packages/itil/scripts/update-rfc-references-section.sh"
-
-  TMPROOT=$(mktemp -d)
-  ORIG_DIR="$PWD"
-  cd "$TMPROOT"
-  mkdir -p docs/rfcs docs/stories/draft docs/stories/accepted docs/stories/done
-}
-
-teardown() {
-  cd "$ORIG_DIR"
-  rm -rf "$TMPROOT"
-}
-
-# ---------------------------------------------------------------------------
-# Surface 1: RFC frontmatter spec includes stories: field
-# ---------------------------------------------------------------------------
-
-@test "rfc-stories-extension: docs/rfcs/README.md frontmatter spec declares stories field" {
-  run grep -E '^stories:.*\[STORY-' "$RFC_README"
-  [ "$status" -eq 0 ]
-}
-
-@test "rfc-stories-extension: docs/rfcs/README.md describes 0..N + ORDERED cardinality" {
-  # The field semantics row must name the ordered + 0..N contract for
-  # working-the-problem traversal to read the field correctly.
-  run grep -iE 'ordered|execution sequence' "$RFC_README"
-  [ "$status" -eq 0 ]
-}
-
-# ---------------------------------------------------------------------------
-# Surface 2a: helper renders ## Stories section from populated stories: array
-# ---------------------------------------------------------------------------
-
-@test "rfc-stories-extension: helper renders populated stories: array in execution order" {
-  # Set up RFC with stories: [STORY-001, STORY-002] frontmatter, plus
-  # matching story files. Helper must produce a ## Stories section
-  # listing both in the array order.
-  cat > docs/rfcs/RFC-001-test.proposed.md <<'EOF'
----
-status: proposed
-rfc-id: test
-reported: 2026-05-12
-decision-makers: [Test]
-problems: [P170]
-adrs: []
-jtbd: []
-stories: [STORY-001, STORY-002]
----
-
-# RFC-001: Test
-
-## Summary
-
-Test.
-
-## Stories
-
-(empty)
-EOF
-  cat > docs/stories/draft/STORY-001-first.md <<'EOF'
----
-status: draft
-story-id: first
-problems: [P170]
-jtbd: [JTBD-008]
-rfcs: [RFC-001]
-story-maps: []
-estimated-effort: deferred
----
-
-# STORY-001: First story
-EOF
-  cat > docs/stories/draft/STORY-002-second.md <<'EOF'
----
-status: draft
-story-id: second
-problems: [P170]
-jtbd: [JTBD-008]
-rfcs: [RFC-001]
-story-maps: []
-estimated-effort: deferred
----
-
-# STORY-002: Second story
-EOF
-  run bash "$HELPER" docs/rfcs/RFC-001-test.proposed.md Stories
-  # Acceptable: zero exit OR advisory-warning exit; non-acceptable is
-  # the "unknown section-name" rejection signal.
-  [[ "$output" != *"unknown section-name"* ]]
-}
-
-# ---------------------------------------------------------------------------
-# Surface 2b: renderer tolerates empty stories: [] on a DRAFT RFC
-# ---------------------------------------------------------------------------
-
-@test "rfc-stories-extension: renderer tolerates empty stories: [] on a draft RFC (ADR-089: enforced >=1 at accept, not at render)" {
-  # Per ADR-089 every RFC has >=1 story, but empty stories: [] is a legal
-  # TRANSIENT state on a draft/proposed RFC before the fix is scoped. The
-  # renderer must not reject it (that would block drafting); the >=1-story
-  # requirement is enforced at the manage-rfc accept gate by the
-  # check-rfc-has-stories predicate, not by this renderer.
-  cat > docs/rfcs/RFC-002-atomic.proposed.md <<'EOF'
----
-status: proposed
-rfc-id: atomic
-reported: 2026-05-12
-decision-makers: [Test]
-problems: [P170]
-adrs: []
-jtbd: []
-stories: []
----
-
-# RFC-002: Atomic RFC
-EOF
-  run bash "$HELPER" docs/rfcs/RFC-002-atomic.proposed.md Stories
-  [[ "$output" != *"unknown section-name"* ]]
-}
-
-# ---------------------------------------------------------------------------
-# Surface 3: capture-rfc + manage-rfc SKILL.md document the extension
-# ---------------------------------------------------------------------------
-
-@test "rfc-stories-extension: capture-rfc SKILL.md names --stories flag" {
-  run grep -E '\-\-stories STORY-' "$CAPTURE_RFC"
-  [ "$status" -eq 0 ]
-}
-
-@test "rfc-stories-extension: capture-rfc SKILL.md frontmatter template includes stories field" {
-  run grep -E '^stories:' "$CAPTURE_RFC"
-  [ "$status" -eq 0 ]
-}
-
-@test "rfc-stories-extension: manage-rfc SKILL.md invokes update-rfc-references-section.sh with Stories" {
-  # The forward-trace contract on every transition needs the helper call
-  # in the SKILL body. Behavioural assertion on the presence of the
-  # load-bearing call path; without it, manage-rfc transitions would
-  # silently leave the body section stale.
-  run grep -E 'wr-itil-update-rfc-references-section.*Stories' "$MANAGE_RFC"
-  [ "$status" -eq 0 ]
-}