@windyroad/itil 0.61.1 → 0.61.2

package/scripts/test/classify-readme-drift.bats

@@ -1,392 +0,0 @@
-#!/usr/bin/env bats
-
-# @problem P149 — manage-problem Step 0 reconcile halt-on-drift directive
-# doesn't distinguish uncommitted-rename-rooted drift (same-session pending,
-# in-flow P094/P062 refresh will land it in the upcoming commit per ADR-014)
-# from committed cross-session drift (must halt and route to
-# /wr-itil:reconcile-readme).
-#
-# Contract: `classify-readme-drift.sh <drift-stdout-file> [<problems-dir>]`
-# reads the captured stdout of reconcile-readme.sh exit-1 output AND the
-# working-tree state via `git status --porcelain`, then classifies the drift.
-#
-# Output (stdout, one classification line):
-#   INLINE_REFRESH covered=<N>            — every drift ID is the destination
-#                                            of a staged rename in the working
-#                                            tree; defer to in-flow P094/P062
-#                                            refresh per ADR-014.
-#   HALT_ROUTE_RECONCILE uncovered=<N>    — at least one drift ID is NOT
-#                                            covered by a working-tree rename;
-#                                            committed cross-session drift OR
-#                                            mixed; route to
-#                                            /wr-itil:reconcile-readme.
-#
-# Exit codes:
-#   0 = INLINE_REFRESH
-#   1 = HALT_ROUTE_RECONCILE
-#   2 = parse error (drift-stdout-file missing or empty)
-#
-# @adr ADR-014 (single-commit grain — the carve-out preserves it for the
-#               in-flow path while keeping cross-session drift safety)
-# @adr ADR-005 (Plugin testing strategy — behavioural bats per P081)
-# @jtbd JTBD-006 (Progress the Backlog While I'm Away — AFK loop continuity)
-# @jtbd JTBD-001 (Enforce Governance Without Slowing Down — single-commit grain)
-
-setup() {
-  SCRIPTS_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  SCRIPT="$SCRIPTS_DIR/classify-readme-drift.sh"
-  FIXTURE_DIR="$(mktemp -d)"
-  cd "$FIXTURE_DIR"
-  git init -q
-  git config user.email test@example.com
-  git config user.name "Test"
-  mkdir -p docs/problems
-}
-
-teardown() {
-  cd /
-  rm -rf "$FIXTURE_DIR"
-}
-
-# ── Existence + executable ──────────────────────────────────────────────────
-
-@test "classify-readme-drift: script exists" {
-  [ -f "$SCRIPT" ]
-}
-
-@test "classify-readme-drift: script is executable" {
-  [ -x "$SCRIPT" ]
-}
-
-# ── Exit 0 (INLINE_REFRESH): every drift ID covered by staged rename ────────
-
-@test "classify-readme-drift: single drift ID covered by staged rename → INLINE_REFRESH" {
-  # Simulate the P148 scenario: prior session staged a rename for a ticket
-  # transition; reconcile-readme observes the README is stale (still claims
-  # the old status) and emits a DRIFT line. The in-flow P094/P062 refresh
-  # at Step 5/Step 7 will reconcile the README in the same commit per
-  # ADR-014.
-  echo "stub" > docs/problems/148-foo.open.md
-  git add docs/problems/148-foo.open.md
-  git commit -q -m "init"
-  git mv docs/problems/148-foo.open.md docs/problems/148-foo.verifying.md
-
-  cat > drift.txt <<'EOF'
-DRIFT    P148 wsjf-rankings: claims=open actual=verifying
-MISSING  P148 verification-queue: actual=verifying
-EOF
-
-  run "$SCRIPT" drift.txt docs/problems
-  [ "$status" -eq 0 ]
-  echo "$output" | grep -q "INLINE_REFRESH"
-  echo "$output" | grep -q "covered=1"
-}
-
-@test "classify-readme-drift: multiple drift IDs all covered by staged renames → INLINE_REFRESH" {
-  echo "stub" > docs/problems/100-aaa.open.md
-  echo "stub" > docs/problems/101-bbb.open.md
-  git add docs/problems/
-  git commit -q -m "init"
-
-  git mv docs/problems/100-aaa.open.md docs/problems/100-aaa.known-error.md
-  git mv docs/problems/101-bbb.open.md docs/problems/101-bbb.verifying.md
-
-  cat > drift.txt <<'EOF'
-DRIFT    P100 wsjf-rankings: claims=open actual=known-error
-DRIFT    P101 wsjf-rankings: claims=open actual=verifying
-MISSING  P101 verification-queue: actual=verifying
-EOF
-
-  run "$SCRIPT" drift.txt docs/problems
-  [ "$status" -eq 0 ]
-  echo "$output" | grep -q "INLINE_REFRESH"
-  echo "$output" | grep -q "covered=2"
-}
-
-@test "classify-readme-drift: rename-with-modification (RM) is recognised as covered" {
-  # `git mv` followed by an `Edit` to the renamed file's body shows up as
-  # `RM` in `git status --porcelain` (rename + unstaged modification). The
-  # carve-out must treat both `R ` and `RM` as covered.
-  cat > docs/problems/200-xyz.open.md <<'EOF'
-# Problem 200: XYZ
-**Status**: Open
-EOF
-  git add docs/problems/200-xyz.open.md
-  git commit -q -m "init"
-
-  git mv docs/problems/200-xyz.open.md docs/problems/200-xyz.verifying.md
-  # Modify the renamed file's content (without re-staging) — emits RM.
-  cat >> docs/problems/200-xyz.verifying.md <<'EOF'
-
-## Fix Released
-Deployed in v0.99.0.
-EOF
-
-  cat > drift.txt <<'EOF'
-DRIFT    P200 wsjf-rankings: claims=open actual=verifying
-EOF
-
-  run "$SCRIPT" drift.txt docs/problems
-  [ "$status" -eq 0 ]
-  echo "$output" | grep -q "INLINE_REFRESH"
-  echo "$output" | grep -q "covered=1"
-}
-
-# ── P306: same-ID D+A pair coverage (substantial-body in-flight rename) ─────
-# When `git mv` is followed by a substantial body edit, git's rename-detection
-# may not match the old/new paths and emits a delete+add pair instead of an R
-# entry. The classifier must recognise a same-ID D+A pair as same-session
-# coverage — equivalent to an R/RM entry — and return INLINE_REFRESH.
-
-@test "classify-readme-drift: same-ID D+A pair (substantial-body rename) → INLINE_REFRESH" {
-  # Seed an open ticket, commit, then `git mv` + substantial-body edit. With
-  # rename-detection thresholds, git may emit `D ` + `A ` (or `??`) rather
-  # than `R `. We force the D+A shape by writing wholly different content to
-  # the destination path.
-  cat > docs/problems/306-foo.open.md <<'EOF'
-# Problem 306: Foo
-
-**Status**: Open
-
-Original body — short.
-EOF
-  git add docs/problems/306-foo.open.md
-  git commit -q -m "init"
-
-  # Create the new file at the verifying path with substantially different
-  # body BEFORE removing the old path (avoid empty-dir prune). git's
-  # rename-detection then sees delete + add as distinct entries rather than
-  # an R-rename, because the bodies are wholly different.
-  cat > docs/problems/306-foo.verifying.md <<'EOF'
-# Problem 306: Foo
-
-**Status**: Verification Pending
-
-Wholly rewritten body so git rename-detection does not match the source.
-This is a multi-paragraph substantive rewrite that exercises the D+A path.
-
-## Fix Released
-
-Deployed in vX.Y.Z. Adds a behavioural fixture that exercises the
-classifier across the D+A coverage gap that P306 captured.
-EOF
-  git rm -q docs/problems/306-foo.open.md
-  git add docs/problems/306-foo.verifying.md
-
-  cat > drift.txt <<'EOF'
-DRIFT    P306 wsjf-rankings: claims=open actual=verifying
-MISSING  P306 verification-queue: actual=verifying
-EOF
-
-  run "$SCRIPT" drift.txt docs/problems
-  [ "$status" -eq 0 ]
-  echo "$output" | grep -q "INLINE_REFRESH"
-  echo "$output" | grep -q "covered=1"
-}
-
-@test "classify-readme-drift: same-ID D+A pair with untracked add (D + ??) → INLINE_REFRESH" {
-  # Variant: the new path is untracked (not yet `git add`-ed). git status
-  # emits `D ` for the old path and `??` for the new path. The classifier
-  # must still recognise the same-ID pair as same-session coverage.
-  cat > docs/problems/307-bar.open.md <<'EOF'
-# Problem 307: Bar
-**Status**: Open
-EOF
-  git add docs/problems/307-bar.open.md
-  git commit -q -m "init"
-
-  cat > docs/problems/307-bar.verifying.md <<'EOF'
-# Problem 307: Bar
-
-**Status**: Verification Pending
-
-Untracked add side of the D+?? pair.
-EOF
-  git rm -q docs/problems/307-bar.open.md
-
-  cat > drift.txt <<'EOF'
-DRIFT    P307 wsjf-rankings: claims=open actual=verifying
-EOF
-
-  run "$SCRIPT" drift.txt docs/problems
-  [ "$status" -eq 0 ]
-  echo "$output" | grep -q "INLINE_REFRESH"
-  echo "$output" | grep -q "covered=1"
-}
-
-@test "classify-readme-drift: D-only (no matching A for same ID) → HALT_ROUTE_RECONCILE" {
-  # Negative case: a delete without a corresponding add for the same ID is
-  # NOT a rename — it is a genuine deletion. Must HALT.
-  cat > docs/problems/308-baz.open.md <<'EOF'
-# Problem 308: Baz
-**Status**: Open
-EOF
-  git add docs/problems/308-baz.open.md
-  git commit -q -m "init"
-
-  git rm -q docs/problems/308-baz.open.md
-
-  cat > drift.txt <<'EOF'
-DRIFT    P308 wsjf-rankings: claims=open actual=verifying
-EOF
-
-  run "$SCRIPT" drift.txt docs/problems
-  [ "$status" -eq 1 ]
-  echo "$output" | grep -q "HALT_ROUTE_RECONCILE"
-  echo "$output" | grep -q "uncovered=1"
-}
-
-@test "classify-readme-drift: mismatched D + A (different IDs) → HALT_ROUTE_RECONCILE" {
-  # Negative case: a delete for one ID + an add for a different ID is NOT a
-  # rename of either ticket — both are uncovered.
-  cat > docs/problems/309-alpha.open.md <<'EOF'
-# Problem 309: Alpha
-**Status**: Open
-EOF
-  git add docs/problems/309-alpha.open.md
-  git commit -q -m "init"
-
-  cat > docs/problems/310-beta.open.md <<'EOF'
-# Problem 310: Beta
-**Status**: Open
-EOF
-  git rm -q docs/problems/309-alpha.open.md
-  git add docs/problems/310-beta.open.md
-
-  cat > drift.txt <<'EOF'
-DRIFT    P309 wsjf-rankings: claims=open actual=verifying
-EOF
-
-  run "$SCRIPT" drift.txt docs/problems
-  [ "$status" -eq 1 ]
-  echo "$output" | grep -q "HALT_ROUTE_RECONCILE"
-}
-
-# ── Exit 1 (HALT_ROUTE_RECONCILE): committed cross-session drift ────────────
-
-@test "classify-readme-drift: single drift ID not covered by any rename → HALT_ROUTE_RECONCILE" {
-  # Simulate the P118 originating scenario: a prior session committed a
-  # ticket transition without staging the README refresh. The README is
-  # stale; the working tree is clean (no pending rename). Halt and route
-  # to /wr-itil:reconcile-readme.
-  cat > docs/problems/074-foo.closed.md <<'EOF'
-# Problem 074: Foo
-**Status**: Closed
-EOF
-  git add docs/problems/074-foo.closed.md
-  git commit -q -m "init"
-
-  cat > drift.txt <<'EOF'
-DRIFT    P074 wsjf-rankings: claims=open actual=closed
-EOF
-
-  run "$SCRIPT" drift.txt docs/problems
-  [ "$status" -eq 1 ]
-  echo "$output" | grep -q "HALT_ROUTE_RECONCILE"
-  echo "$output" | grep -q "uncovered=1"
-}
-
-@test "classify-readme-drift: mixed (some covered, some uncovered) → HALT_ROUTE_RECONCILE" {
-  # Mixed case: one ID has a staged rename, another is committed drift.
-  # The architect's confirmation: mixed routes to halt (the safe path),
-  # because reconcile-readme will resolve both, and the in-flow refresh
-  # only handles the rename'd one.
-  cat > docs/problems/074-foo.closed.md <<'EOF'
-# Problem 074: Foo
-**Status**: Closed
-EOF
-  cat > docs/problems/148-bar.open.md <<'EOF'
-# Problem 148: Bar
-**Status**: Open
-EOF
-  git add docs/problems/
-  git commit -q -m "init"
-
-  git mv docs/problems/148-bar.open.md docs/problems/148-bar.verifying.md
-
-  cat > drift.txt <<'EOF'
-DRIFT    P074 wsjf-rankings: claims=open actual=closed
-DRIFT    P148 wsjf-rankings: claims=open actual=verifying
-EOF
-
-  run "$SCRIPT" drift.txt docs/problems
-  [ "$status" -eq 1 ]
-  echo "$output" | grep -q "HALT_ROUTE_RECONCILE"
-  # Only P074 is uncovered.
-  echo "$output" | grep -q "uncovered=1"
-}
-
-@test "classify-readme-drift: clean working tree (no renames) → HALT_ROUTE_RECONCILE for any drift" {
-  cat > docs/problems/050-zzz.verifying.md <<'EOF'
-# Problem 050: Zzz
-**Status**: Verification Pending
-EOF
-  git add docs/problems/050-zzz.verifying.md
-  git commit -q -m "init"
-
-  cat > drift.txt <<'EOF'
-MISSING  P050 verification-queue: actual=verifying
-EOF
-
-  run "$SCRIPT" drift.txt docs/problems
-  [ "$status" -eq 1 ]
-  echo "$output" | grep -q "HALT_ROUTE_RECONCILE"
-  echo "$output" | grep -q "uncovered=1"
-}
-
-# ── Exit 2: parse errors ────────────────────────────────────────────────────
-
-@test "classify-readme-drift: missing drift-stdout-file → exit 2" {
-  run "$SCRIPT" /nonexistent/drift.txt docs/problems
-  [ "$status" -eq 2 ]
-  echo "$output" | grep -qi "PARSE_ERROR"
-}
-
-@test "classify-readme-drift: empty drift-stdout-file → exit 2" {
-  : > drift.txt
-  run "$SCRIPT" drift.txt docs/problems
-  [ "$status" -eq 2 ]
-  echo "$output" | grep -qi "PARSE_ERROR"
-}
-
-@test "classify-readme-drift: no arguments → exit 2 with usage" {
-  run "$SCRIPT"
-  [ "$status" -eq 2 ]
-  echo "$output" | grep -qi "USAGE"
-}
-
-# ── Default problems-dir resolution ─────────────────────────────────────────
-
-@test "classify-readme-drift: defaults to ./docs/problems when problems-dir omitted" {
-  echo "stub" > docs/problems/300-default.open.md
-  git add docs/problems/300-default.open.md
-  git commit -q -m "init"
-  git mv docs/problems/300-default.open.md docs/problems/300-default.verifying.md
-
-  cat > drift.txt <<'EOF'
-DRIFT    P300 wsjf-rankings: claims=open actual=verifying
-EOF
-
-  run "$SCRIPT" drift.txt
-  [ "$status" -eq 0 ]
-  echo "$output" | grep -q "INLINE_REFRESH"
-}
-
-# ── Non-git working tree (defensive — outside a repo) ───────────────────────
-
-@test "classify-readme-drift: outside git repo with drift → HALT_ROUTE_RECONCILE (no rename evidence)" {
-  # If the script is invoked from outside a git repo (defensive — should not
-  # happen in the SKILL.md flow but safest fail-closed), there is no rename
-  # evidence, so all drift is treated as uncovered.
-  NON_GIT="$(mktemp -d)"
-  cd "$NON_GIT"
-  mkdir -p docs/problems
-  cat > drift.txt <<'EOF'
-DRIFT    P074 wsjf-rankings: claims=open actual=closed
-EOF
-  run "$SCRIPT" drift.txt docs/problems
-  [ "$status" -eq 1 ]
-  echo "$output" | grep -q "HALT_ROUTE_RECONCILE"
-  cd /
-  rm -rf "$NON_GIT"
-}

package/scripts/test/derive-first-dispatch.bats

@@ -1,274 +0,0 @@
-#!/usr/bin/env bats
-
-bats_require_minimum_version 1.5.0
-
-# Behavioural assertions for packages/itil/lib/derive-first-dispatch.sh —
-# the shared derive-first dispatch helper extracted in P132 Phase 2a-iii-A.
-#
-# The helper centralises the dispatch mechanism shipped across three
-# declaration-skill surfaces (manage-incident Step 4, manage-problem
-# Step 4, create-adr Step 2 — Phase 2a-iii-B 4th adopter). Each caller
-# passes surface-specific signal definitions; the helper owns:
-#
-#   - Slug derivation (Title) from prose
-#   - RISK-POLICY matrix lookup (Severity / Priority)
-#   - I2-isomorphic stderr advisory format
-#
-# P287 (2026-06-02) retired the two-sided lexical classifier (formerly
-# used by capture-problem Step 1.5 Type classification — technical vs
-# user-business). The type axis was removed as redundant with RFC/Story
-# persona-anchoring per ADR-060 Phase 4; the function had no remaining
-# consumer.
-#
-# @problem P132 (agents over-ask in interactive sessions — Phase 2a-iii-A
-#   shared helper extraction)
-# @problem P185 (capture-problem Step 1.5 worked-example precedent — retired by P287)
-# @problem P287 (type-classification retirement — lexical_classify_two_sided removed)
-# @adr ADR-044 (Decision-Delegation Contract — derive-first framework
-#   resolution boundary)
-# @adr ADR-026 (cost-source grounding — stderr advisory shape)
-# @adr ADR-052 (behavioural-by-default — these are runtime behaviour
-#   assertions on the helper functions, NOT structural greps)
-# @jtbd JTBD-001 (enforce governance without slowing down — primary)
-# @jtbd JTBD-101 (extend the suite with consistent patterns)
-
-setup() {
-  LIB_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../lib" && pwd)"
-  HELPER="${LIB_DIR}/derive-first-dispatch.sh"
-  [ -f "$HELPER" ]
-  # shellcheck disable=SC1090
-  source "$HELPER"
-  PKG_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../.." && pwd)"
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../.." && pwd)"
-  ARCHITECT_PKG_ROOT="$REPO_ROOT/packages/architect"
-}
-
-# ----------------------------------------------------------------------
-# Stderr advisory contract (I2-isomorphic format across all 3 surfaces).
-# Format: <skill>: derived <field>=<value> from <source>; <reversibility>
-# ----------------------------------------------------------------------
-
-@test "emit_stderr_advisory writes single canonical line to stderr" {
-  run -0 bash -c '
-    source "'"$HELPER"'"
-    emit_stderr_advisory manage-problem title my-slug "description tokens" \
-      "re-invoke with the desired title or rename the file if the slug is wrong"
-  '
-  # stderr captured in $output via run; assert structure
-  [[ "$output" == *"manage-problem: derived title=my-slug from description tokens; re-invoke with the desired title"* ]]
-}
-
-@test "emit_stderr_advisory uses default reversibility clause when omitted" {
-  run -0 bash -c '
-    source "'"$HELPER"'"
-    emit_stderr_advisory manage-incident severity "9 (Medium)" "RISK-POLICY matrix"
-  '
-  [[ "$output" == *"manage-incident: derived severity=9 (Medium) from RISK-POLICY matrix;"* ]]
-  [[ "$output" == *"re-invoke"* ]] || [[ "$output" == *"update"* ]]
-}
-
-@test "emit_stderr_advisory shape is I2-isomorphic across surfaces (same sentence structure)" {
-  run -0 bash -c '
-    source "'"$HELPER"'"
-    emit_stderr_advisory capture-problem type technical "description signals" "re-invoke with --type=user-business to override"
-    emit_stderr_advisory manage-incident title incident-slug "description" "re-invoke or rename"
-    emit_stderr_advisory manage-problem priority "9 (Medium)" "RISK-POLICY matrix" "re-invoke or update if mis-rated"
-  '
-  # Each surface emits the same sentence shape: <skill>: derived <field>=<value> from <source>; <clause>
-  line_count=$(printf '%s\n' "$output" | grep -c "^[a-z-]*: derived ")
-  [ "$line_count" -eq 3 ]
-}
-
-# ----------------------------------------------------------------------
-# Kebab-case slug derivation from prose.
-# ----------------------------------------------------------------------
-
-@test "derive_kebab_slug produces kebab-case from prose" {
-  run -0 bash -c '
-    source "'"$HELPER"'"
-    derive_kebab_slug "Agent over-asks during interactive sessions"
-  '
-  [[ "$output" == *"agent"* ]]
-  [[ "$output" == *"over"* ]] || [[ "$output" == *"asks"* ]]
-  [[ "$output" != *" "* ]]
-  [[ "$output" != *"_"* ]]
-}
-
-@test "derive_kebab_slug drops stopwords" {
-  run -0 bash -c '
-    source "'"$HELPER"'"
-    derive_kebab_slug "The agent is asking the user a question"
-  '
-  # stopwords like "the", "a", "is" must NOT appear as standalone tokens
-  [[ "$output" != *"-the-"* ]]
-  [[ "$output" != "the-"* ]]
-  [[ "$output" == *"agent"* ]]
-}
-
-@test "derive_kebab_slug caps token count (default 8)" {
-  run -0 bash -c '
-    source "'"$HELPER"'"
-    derive_kebab_slug "one two three four five six seven eight nine ten eleven twelve"
-  '
-  token_count=$(printf '%s\n' "$output" | tr '-' '\n' | wc -l | tr -d ' ')
-  [ "$token_count" -le 8 ]
-}
-
-@test "derive_kebab_slug accepts custom token count" {
-  run -0 bash -c '
-    source "'"$HELPER"'"
-    derive_kebab_slug "alpha beta gamma delta epsilon zeta" 3
-  '
-  token_count=$(printf '%s\n' "$output" | tr '-' '\n' | wc -l | tr -d ' ')
-  [ "$token_count" -le 3 ]
-}
-
-# ----------------------------------------------------------------------
-# P287 (2026-06-02) retirement: lexical_classify_two_sided removed.
-# Regression guard — the function must NOT exist in the helper.
-# ----------------------------------------------------------------------
-
-@test "P287: lexical_classify_two_sided is removed from the helper (regression guard)" {
-  # The function name must not appear as a function definition.
-  ! grep -E '^lexical_classify_two_sided\(\)' "$HELPER"
-}
-
-@test "P287: helper still exports the three surviving functions" {
-  # emit_stderr_advisory, derive_kebab_slug, risk_policy_matrix_lookup
-  # serve manage-incident severity, manage-problem priority, and
-  # create-adr title derivation — they must stay.
-  grep -E '^emit_stderr_advisory\(\)' "$HELPER"
-  grep -E '^derive_kebab_slug\(\)' "$HELPER"
-  grep -E '^risk_policy_matrix_lookup\(\)' "$HELPER"
-}
-
-# ----------------------------------------------------------------------
-# RISK-POLICY matrix lookup (manage-incident / manage-problem mechanism).
-# Returns:
-#   <score>|<label>|impact=<L>+likelihood=<L>  — clear single-cell match
-#   AMBIGUOUS|<reason>                        — multi-band or zero match
-# ----------------------------------------------------------------------
-
-@test "risk_policy_matrix_lookup returns clear cell on unambiguous impact + likelihood signals" {
-  run -0 bash -c '
-    source "'"$HELPER"'"
-    impact_high=("\\b(down|outage|data loss|unavailable)\\b")
-    impact_mod=("\\b(slow|latency|degraded)\\b")
-    impact_low=("\\b(typo|cosmetic)\\b")
-    likelihood_high=("\\b(every request|reproducible|always)\\b")
-    likelihood_med=("\\b(intermittent|flaky)\\b")
-    likelihood_low=("\\b(one-off|single)\\b")
-    risk_policy_matrix_lookup "service is down on every request" impact_high impact_mod impact_low likelihood_high likelihood_med likelihood_low
-  '
-  # Expect impact=high(5) + likelihood=high(5) -> score=25, label=Very High
-  [[ "$output" == "25|"* ]] || [[ "$output" == "20|"* ]] || [[ "$output" == "15|"* ]]
-  [[ "$output" == *"High"* ]] || [[ "$output" == *"Very High"* ]]
-}
-
-@test "risk_policy_matrix_lookup returns AMBIGUOUS on multi-band impact" {
-  run -0 bash -c '
-    source "'"$HELPER"'"
-    impact_high=("\\b(down)\\b")
-    impact_mod=("\\b(slow)\\b")
-    impact_low=("\\b(typo)\\b")
-    likelihood_high=("\\b(every request)\\b")
-    likelihood_med=("\\b(intermittent)\\b")
-    likelihood_low=("\\b(one-off)\\b")
-    risk_policy_matrix_lookup "service is down and slow with typo" impact_high impact_mod impact_low likelihood_high likelihood_med likelihood_low
-  '
-  [[ "$output" == "AMBIGUOUS|"* ]]
-}
-
-@test "risk_policy_matrix_lookup returns AMBIGUOUS when no signals match" {
-  run -0 bash -c '
-    source "'"$HELPER"'"
-    impact_high=("\\b(down)\\b")
-    impact_mod=("\\b(slow)\\b")
-    impact_low=("\\b(typo)\\b")
-    likelihood_high=("\\b(every request)\\b")
-    likelihood_med=("\\b(intermittent)\\b")
-    likelihood_low=("\\b(one-off)\\b")
-    risk_policy_matrix_lookup "totally bland text" impact_high impact_mod impact_low likelihood_high likelihood_med likelihood_low
-  '
-  [[ "$output" == "AMBIGUOUS|"* ]]
-}
-
-@test "risk_policy_matrix_lookup label band aligns with RISK-POLICY.md (Medium = 5-9)" {
-  # Verify a specific clear-cell mapping produces the RISK-POLICY-canonical label.
-  # impact=mod (3) * likelihood=high (5) = 15 -> "High" band (10-16)
-  run -0 bash -c '
-    source "'"$HELPER"'"
-    impact_high=("\\b(down)\\b")
-    impact_mod=("\\b(slow)\\b")
-    impact_low=("\\b(typo)\\b")
-    likelihood_high=("\\b(every request)\\b")
-    likelihood_med=("\\b(intermittent)\\b")
-    likelihood_low=("\\b(one-off)\\b")
-    risk_policy_matrix_lookup "the service is slow on every request" impact_high impact_mod impact_low likelihood_high likelihood_med likelihood_low
-  '
-  [[ "$output" == "15|High|"* ]]
-}
-
-# ----------------------------------------------------------------------
-# Cross-skill consistency: all 4 SKILL.md surfaces reference the helper
-# as the shared dispatch mechanism. The I2-isomorphic stderr advisory
-# format is locked-in by reference to derive-first-dispatch.sh.
-#
-# Phase 2a-iii-B (2026-05-16): 4th adopter wr-architect:create-adr added.
-# Helper canonical source moved to packages/shared/ per ADR-017 sync
-# pattern; per-package lib/ copies in packages/itil/lib/ and
-# packages/architect/lib/ stay byte-identical via scripts/sync-derive-first-dispatch.sh.
-# ----------------------------------------------------------------------
-
-@test "manage-incident Step 4 cross-references derive-first-dispatch.sh helper" {
-  run grep -c "derive-first-dispatch\\.sh\\|packages/itil/lib/derive-first-dispatch" \
-    "${PKG_ROOT}/skills/manage-incident/SKILL.md"
-  [ "$status" -eq 0 ]
-  [ "$output" -ge 1 ]
-}
-
-@test "manage-problem Step 4 cross-references derive-first-dispatch.sh helper" {
-  run grep -c "derive-first-dispatch\\.sh\\|packages/itil/lib/derive-first-dispatch" \
-    "${PKG_ROOT}/skills/manage-problem/SKILL.md"
-  [ "$status" -eq 0 ]
-  [ "$output" -ge 1 ]
-}
-
-@test "create-adr Step 2 cross-references derive-first-dispatch.sh helper (Phase 2a-iii-B 4th adopter)" {
-  # The 4th adopter (architect package) sources from its own per-package
-  # lib/ copy (NOT cross-package from itil) per ADR-017.
-  run grep -c "derive-first-dispatch\\.sh\\|packages/architect/lib/derive-first-dispatch" \
-    "${ARCHITECT_PKG_ROOT}/skills/create-adr/SKILL.md"
-  [ "$status" -eq 0 ]
-  [ "$output" -ge 1 ]
-}
-
-@test "helper file documents its four caller surfaces (audit trail)" {
-  # The helper's header comment must name the four SKILL.md surfaces it
-  # serves so the audit trail is recoverable from the helper itself.
-  # Phase 2a-iii-B adds create-adr as the 4th adopter.
-  run grep -E "capture-problem" "$HELPER"
-  [ "$status" -eq 0 ]
-  run grep -E "manage-incident" "$HELPER"
-  [ "$status" -eq 0 ]
-  run grep -E "manage-problem" "$HELPER"
-  [ "$status" -eq 0 ]
-  run grep -E "create-adr" "$HELPER"
-  [ "$status" -eq 0 ]
-}
-
-@test "per-package lib/ copies are byte-identical to canonical packages/shared/ source (ADR-017)" {
-  # Phase 2a-iii-B + ADR-017: canonical at packages/shared/, synced copies
-  # in per-package lib/. The sync script (scripts/sync-derive-first-dispatch.sh)
-  # in --check mode is the CI guard; this test asserts the post-condition.
-  local shared_src="${REPO_ROOT}/packages/shared/derive-first-dispatch.sh"
-  local itil_copy="${REPO_ROOT}/packages/itil/lib/derive-first-dispatch.sh"
-  local architect_copy="${REPO_ROOT}/packages/architect/lib/derive-first-dispatch.sh"
-  [ -f "$shared_src" ]
-  [ -f "$itil_copy" ]
-  [ -f "$architect_copy" ]
-  run diff -q "$shared_src" "$itil_copy"
-  [ "$status" -eq 0 ]
-  run diff -q "$shared_src" "$architect_copy"
-  [ "$status" -eq 0 ]
-}