@windyroad/itil 0.61.1 → 0.61.2

package/skills/capture-story/test/capture-story-behavioural.bats

@@ -1,317 +0,0 @@
-#!/usr/bin/env bats
-# Behavioural fixtures for /wr-itil:capture-story (P170 Phase 2 Slice 7).
-#
-# Per ADR-052 (Behavioural-tests-default for skill testing), these tests
-# exercise the load-bearing primitives the skill dispatches and assert
-# observable state — NOT the prose contents of SKILL.md.
-#
-# Behavioural surfaces under test:
-#   1. Next-ID computation — capture-story uses the inline
-#      max(local, origin) + 1 formula scanning docs/stories/*/STORY-*.md
-#      + git ls-tree -r origin/main docs/stories/ per ADR-019 inline
-#      collision-guard approved at Slice 3 design review (option a).
-#   2. ID collision-on-origin renumber — when an origin RFC IS not yet
-#      pulled locally, the formula must compute the higher of the two
-#      and increment from there (no silent collision).
-#   3. Reverse-trace helper "Stories" section-name support — the three
-#      Slice 2a/2b helpers (update-problem / update-jtbd / update-rfc
-#      -references-section.sh) must accept "Stories" as a section name.
-#   4. Frontmatter shape — a captured story file conforms to ADR-060
-#      lines 220-228 (status: draft / story-id / problems / jtbd / etc.).
-#   5. NO update-story-references-section.sh "Stories" path —
-#      story-maps are HTML; the story-tier reverse-trace helper does NOT
-#      accept "Stories" (per architect amend finding 2 on Slice 7).
-#
-# @problem P170
-# @jtbd JTBD-008 (Decompose a Fix Into Coordinated Changes — capture-time
-#                  decomposition surface)
-# @jtbd JTBD-001 (extended scope — change-set-level governance)
-# @adr  ADR-060  (Problem-RFC-Story framework — story tier)
-# @adr  ADR-052  (Behavioural-tests-default)
-# @adr  ADR-019  (ID collision-on-origin renumber)
-# @adr  ADR-014  (single-commit grain)
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../../.." && pwd)"
-  SKILL_FILE="${REPO_ROOT}/packages/itil/skills/capture-story/SKILL.md"
-  HELPER_PROBLEM="${REPO_ROOT}/packages/itil/scripts/update-problem-references-section.sh"
-  HELPER_JTBD="${REPO_ROOT}/packages/itil/scripts/update-jtbd-references-section.sh"
-  HELPER_RFC="${REPO_ROOT}/packages/itil/scripts/update-rfc-references-section.sh"
-  HELPER_STORY="${REPO_ROOT}/packages/itil/scripts/update-story-references-section.sh"
-
-  TMPROOT=$(mktemp -d)
-  ORIG_DIR="$PWD"
-  cd "$TMPROOT"
-}
-
-teardown() {
-  cd "$ORIG_DIR"
-  rm -rf "$TMPROOT"
-}
-
-# ---------------------------------------------------------------------------
-# Surface 0: SKILL.md exists with correct name (minimum discoverability)
-# ---------------------------------------------------------------------------
-
-@test "capture-story: SKILL.md exists" {
-  [ -f "$SKILL_FILE" ]
-}
-
-@test "capture-story: SKILL.md frontmatter declares wr-itil:capture-story name" {
-  # Discoverable on / autocomplete depends on the canonical name.
-  run grep -E '^name: wr-itil:capture-story$' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-# ---------------------------------------------------------------------------
-# Surface 1: Next-ID computation — inline formula matches capture-rfc precedent
-# (verified against a fixture stories directory).
-# ---------------------------------------------------------------------------
-
-@test "capture-story: next-ID formula computes 001 for empty stories directory" {
-  mkdir -p docs/stories/draft
-  local_max=$(ls docs/stories/*/STORY-*.md 2>/dev/null | sed 's|.*/STORY-||;s|-.*||' | grep -oE '^[0-9]+' | sort -n | tail -1)
-  origin_max=""  # no git fixture; treat as empty
-  next=$(printf '%03d' $(( 10#$(echo -e "${local_max:-0}\n${origin_max:-0}" | sort -n | tail -1) + 1 )))
-  [ "$next" = "001" ]
-}
-
-@test "capture-story: next-ID formula computes 003 when STORY-002 exists locally" {
-  mkdir -p docs/stories/draft docs/stories/in-progress
-  touch docs/stories/draft/STORY-002-foo.md
-  touch docs/stories/in-progress/STORY-001-bar.md
-  local_max=$(ls docs/stories/*/STORY-*.md 2>/dev/null | sed 's|.*/STORY-||;s|-.*||' | grep -oE '^[0-9]+' | sort -n | tail -1)
-  next=$(printf '%03d' $(( 10#${local_max:-0} + 1 )))
-  [ "$next" = "003" ]
-}
-
-@test "capture-story: next-ID formula picks max(local, origin) when origin is higher (collision-on-origin renumber)" {
-  # Simulates the renumber: local has STORY-002, origin has STORY-005.
-  # Formula must pick 005 + 1 = 006, not 002 + 1 = 003 (silent collision).
-  local_max=2
-  origin_max=5
-  next=$(printf '%03d' $(( 10#$(echo -e "${local_max:-0}\n${origin_max:-0}" | sort -n | tail -1) + 1 )))
-  [ "$next" = "006" ]
-}
-
-@test "capture-story: next-ID formula picks max(local, origin) when local is higher" {
-  # Reverse case: local STORY-007, origin STORY-003. Local wins → 008.
-  local_max=7
-  origin_max=3
-  next=$(printf '%03d' $(( 10#$(echo -e "${local_max:-0}\n${origin_max:-0}" | sort -n | tail -1) + 1 )))
-  [ "$next" = "008" ]
-}
-
-# ---------------------------------------------------------------------------
-# Surface 2: Reverse-trace helpers accept "Stories" section name
-# (Slice 2a/2b shipped these — verify they're still wired correctly).
-# ---------------------------------------------------------------------------
-
-@test "capture-story: update-problem-references-section.sh accepts 'Stories' section name without rejecting" {
-  # Set up minimal fixture: a problem file with a Story Maps placeholder
-  # section. The helper should accept "Stories" as a valid lookup key
-  # (architect finding on Slice 7: the absence of Stories support here
-  # would block the inline reverse-trace refresh in capture-story Step 6).
-  mkdir -p docs/problems/known-error docs/stories/draft
-  cat > docs/problems/known-error/170-test-problem.md <<EOF
-# P170: Test problem
-
-## Story Maps
-
-(empty)
-
-## Stories
-
-(empty)
-EOF
-  run bash "$HELPER_PROBLEM" docs/problems/known-error/170-test-problem.md Stories
-  # Acceptable outcomes: success (0) or no-op-with-stderr; rejection (e.g.
-  # "unknown section-name") would fail this assertion.
-  [[ "$output" != *"unknown section-name"* ]]
-}
-
-@test "capture-story: update-jtbd-references-section.sh accepts 'Stories' section name" {
-  mkdir -p docs/jtbd/solo-developer docs/stories/draft
-  cat > docs/jtbd/solo-developer/JTBD-008-test.proposed.md <<EOF
-# JTBD-008: Test
-
-## Stories
-
-(empty)
-EOF
-  run bash "$HELPER_JTBD" docs/jtbd/solo-developer/JTBD-008-test.proposed.md Stories
-  [[ "$output" != *"unknown section-name"* ]]
-}
-
-@test "capture-story: update-rfc-references-section.sh accepts 'Stories' section name" {
-  mkdir -p docs/rfcs docs/stories/draft
-  cat > docs/rfcs/RFC-002-test.verifying.md <<EOF
-# RFC-002: Test
-
-## Stories
-
-(empty)
-EOF
-  run bash "$HELPER_RFC" docs/rfcs/RFC-002-test.verifying.md Stories
-  [[ "$output" != *"unknown section-name"* ]]
-}
-
-# ---------------------------------------------------------------------------
-# Surface 3: NO update-story-references-section.sh "Stories" path
-# (architect amend finding 2 on Slice 7 — story-maps are HTML, no markdown
-# reverse-trace section auto-maintained on the HTML map; capture-story does
-# NOT stage story-map files.)
-# ---------------------------------------------------------------------------
-
-@test "capture-story: update-story-references-section.sh does NOT accept 'Stories' section name (architect finding 2 — story-maps are HTML, manually authored)" {
-  mkdir -p docs/stories/draft
-  cat > docs/stories/draft/STORY-001-test.md <<EOF
-# STORY-001: Test
-EOF
-  run bash "$HELPER_STORY" docs/stories/draft/STORY-001-test.md Stories
-  # The helper's lookup table supports RFCs + Story Maps but NOT Stories
-  # (per architect finding 2). The helper MUST exit non-zero AND name
-  # the supported sections in its error stream.
-  [ "$status" -ne 0 ]
-  [[ "$output" == *"unknown section-name"* ]] || [[ "$output" == *"Supported"* ]]
-}
-
-# ---------------------------------------------------------------------------
-# Surface 4: Frontmatter shape — verify ADR-060 lines 220-228 fields
-# round-trip through a minimal capture sequence.
-# ---------------------------------------------------------------------------
-
-@test "capture-story: captured frontmatter contains required fields per ADR-060 lines 220-228" {
-  # Construct the frontmatter the way the skill prescribes (Step 5).
-  mkdir -p docs/stories/draft
-  next_id="001"
-  slug="test-capture"
-  description="Test capture"
-  reported=$(date -u +%Y-%m-%d)
-  cat > "docs/stories/draft/STORY-${next_id}-${slug}.md" <<EOF
----
-status: draft
-story-id: ${slug}
-reported: ${reported}
-decision-makers: [Test]
-problems: [P170]
-jtbd: [JTBD-008]
-rfcs: []
-story-maps: [STORY-MAP-001]
-estimated-effort: M
----
-
-# STORY-${next_id}: ${description}
-EOF
-  # Assert each required frontmatter field appears (grep is sufficient —
-  # ADR-052 prefers behavioural assertion on the observable file state).
-  story_file="docs/stories/draft/STORY-${next_id}-${slug}.md"
-  for field in status story-id reported decision-makers problems jtbd rfcs story-maps estimated-effort; do
-    run grep -E "^${field}:" "$story_file"
-    [ "$status" -eq 0 ]
-  done
-}
-
-@test "capture-story: I8-at-capture invariant — a valid captured story has a NON-EMPTY story-maps: (ADR-095)" {
-  mkdir -p docs/stories/draft
-  cat > docs/stories/draft/STORY-001-mapped.md <<EOF
----
-status: draft
-story-maps: [STORY-MAP-002]
----
-EOF
-  run grep -E '^story-maps: \[STORY-MAP-[0-9]' docs/stories/draft/STORY-001-mapped.md
-  [ "$status" -eq 0 ]
-  run grep -E '^story-maps: \[\]' docs/stories/draft/STORY-001-mapped.md
-  [ "$status" -ne 0 ]
-}
-
-@test "capture-story: SKILL prescribes refuse-and-route on a missing story-map trace (I8/ADR-095)" {
-  run grep -E 'I8 hard-block AT CAPTURE|refuse-and-route|missing-story-map-trace' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  run grep -E 'story-map traces are optional at capture' "$SKILL_FILE"
-  [ "$status" -ne 0 ]
-}
-
-@test "capture-story: captured story file lands at docs/stories/draft/ per ADR-060 lines 145-147" {
-  # Story-maps use HTML and lifecycle subdirs (draft/accepted/in-progress/
-  # completed/archived); stories use markdown and lifecycle subdirs
-  # (draft/accepted/in-progress/done/archived). Captured story lands at
-  # draft/ per the skill's Step 5 prescription.
-  mkdir -p docs/stories/draft
-  touch docs/stories/draft/STORY-001-test.md
-  [ -f docs/stories/draft/STORY-001-test.md ]
-  # Negative assertion: NOT in any other subdir at capture time.
-  [ ! -f docs/stories/accepted/STORY-001-test.md ]
-  [ ! -f docs/stories/in-progress/STORY-001-test.md ]
-  [ ! -f docs/stories/done/STORY-001-test.md ]
-}
-
-# --- P474 / STORY-054: the lifecycle mirror must stay gone ---------------------
-# A `**Status**:` body line duplicating the frontmatter `status:` used to sit in
-# this template. The oversight fingerprint excludes the frontmatter key but
-# hashed the body copy, so advancing a story un-ratified it and the no-implement
-# gate then denied its own implementing commit. This was the FOURTH such
-# lifecycle mirror in a family of three, so the removal needs a lock: without
-# one, the next template edit reintroduces it and nothing fails.
-
-@test "P474: the capture-story template emits no **Status** body mirror" {
-  # Behavioural on the artefact the skill prescribes, not on prose ABOUT it:
-  # extract the template's own story scaffold and assert the shape it produces.
-  # The SKILL may still MENTION the forbidden line in its do-not-reintroduce
-  # note, so match only a line-start emission inside a fenced block.
-  run bash -c "awk '/^# STORY-<NNN>/,/^## Dependencies/' '$SKILL_FILE' | grep -cE '^\*\*Status\*\*:'"
-  [ "$output" = "0" ]
-  # Anti-vacuity: if a heading rename ever empties the awk range, the grep -c
-  # above returns 0 and passes while the mirror is present. Assert the range
-  # actually contains the sibling field that DID survive.
-  run bash -c "awk '/^# STORY-<NNN>/,/^## Dependencies/' '$SKILL_FILE' | grep -cE '^\*\*Reported\*\*:'"
-  [ "$output" -ge 1 ]
-}
-
-@test "P474: the documented story body shape emits no **Status** body mirror" {
-  # The second carrier. docs/stories/README.md is the ADOPTER-facing one —
-  # reconcile-stories.sh reads the adopter's own copy — so a stale shape here
-  # has them hand-author the mirror straight back in.
-  local readme="${REPO_ROOT}/docs/stories/README.md"
-  [ -f "$readme" ]
-  run bash -c "awk '/^# STORY-<NNN>/,/^## /' '$readme' | grep -cE '^\*\*Status\*\*:'"
-  [ "$output" = "0" ]
-  # Same anti-vacuity guard as the template carrier above.
-  run bash -c "awk '/^# STORY-<NNN>/,/^## /' '$readme' | grep -cE '^\*\*Reported\*\*:'"
-  [ "$output" -ge 1 ]
-}
-
-@test "P474: a template-shaped story keeps its ratification across an accept transition" {
-  # The invariant the prose serves. Seeds a story in the shape the template now
-  # produces, ratifies it, advances status + ticks a criterion, and asserts the
-  # fingerprint still matches — the exact sequence that used to drift.
-  # shellcheck source=/dev/null
-  source "${REPO_ROOT}/packages/itil/lib/story-oversight.sh"
-  mkdir -p docs/stories/draft
-  local f=docs/stories/draft/STORY-902-shape.md
-  printf -- '---\nstatus: draft\nstory-id: shape\nhuman-oversight: confirmed\noversight-hash: P\n---\n\n# STORY-902: a story\n\n**Reported**: 2026-07-30\n\n## User value\n\nvalue.\n\n## Acceptance criteria\n\n- [ ] a criterion\n' > "$f"
-  local h; h="$(oversight_content_hash "$f")"
-  sed -i.bak "s/oversight-hash: P/oversight-hash: $h/" "$f" && rm -f "$f.bak"
-  run is_story_map_ratified "$f"
-  [ "$status" -eq 0 ]
-
-  sed -i.bak 's/^status: draft/status: accepted/; s/- \[ \] a criterion/- [x] a criterion/' "$f" && rm -f "$f.bak"
-  run is_story_map_ratified "$f"
-  [ "$status" -eq 0 ]
-}
-
-@test "P474: reintroducing the mirror DOES drift a ratified story (the lock has teeth)" {
-  # Proves the invariant test above is not vacuous: put the mirror back and the
-  # accept transition drifts again, which is the defect.
-  # shellcheck source=/dev/null
-  source "${REPO_ROOT}/packages/itil/lib/story-oversight.sh"
-  mkdir -p docs/stories/draft
-  local f=docs/stories/draft/STORY-903-regress.md
-  printf -- '---\nstatus: draft\nstory-id: regress\nhuman-oversight: confirmed\noversight-hash: P\n---\n\n# STORY-903: a story\n\n**Status**: draft\n**Reported**: 2026-07-30\n\n## User value\n\nvalue.\n' > "$f"
-  local h; h="$(oversight_content_hash "$f")"
-  sed -i.bak "s/oversight-hash: P/oversight-hash: $h/" "$f" && rm -f "$f.bak"
-  sed -i.bak 's/^status: draft/status: accepted/; s/^\*\*Status\*\*: draft/**Status**: accepted/' "$f" && rm -f "$f.bak"
-  run is_story_map_ratified "$f"
-  [ "$status" -ne 0 ]
-}

package/skills/capture-story-map/test/capture-story-map-behavioural.bats

@@ -1,98 +0,0 @@
-#!/usr/bin/env bats
-# Behavioural fixtures for /wr-itil:capture-story-map (P170 Phase 2 Slice 3 — ADR-060).
-#
-# Mirrors capture-story-behavioural.bats with story-map-tier adjustments
-# (HTML encoding, I3 + I4 invariants instead of I6 + I9, no optional
-# --rfc / --story-map flags).
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../../.." && pwd)"
-  SKILL_FILE="${REPO_ROOT}/packages/itil/skills/capture-story-map/SKILL.md"
-  HELPER_PROBLEM="${REPO_ROOT}/packages/itil/scripts/update-problem-references-section.sh"
-  HELPER_JTBD="${REPO_ROOT}/packages/itil/scripts/update-jtbd-references-section.sh"
-
-  TMPROOT=$(mktemp -d)
-  ORIG_DIR="$PWD"
-  cd "$TMPROOT"
-}
-
-teardown() {
-  cd "$ORIG_DIR"
-  rm -rf "$TMPROOT"
-}
-
-@test "capture-story-map: SKILL.md exists" {
-  [ -f "$SKILL_FILE" ]
-}
-
-@test "capture-story-map: SKILL.md frontmatter declares wr-itil:capture-story-map name" {
-  run grep -E '^name: wr-itil:capture-story-map$' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "capture-story-map: SKILL.md names I3 trace-to-problem invariant" {
-  run grep -E 'I3 hard-block|I3.*trace-to-problem' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "capture-story-map: SKILL.md names I4 trace-to-JTBD invariant" {
-  run grep -E 'I4 hard-block|I4.*trace-to-JTBD' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "capture-story-map: next-ID formula computes 001 for empty story-maps directory" {
-  mkdir -p docs/story-maps/draft
-  local_max=$(ls docs/story-maps/*/STORY-MAP-*.html 2>/dev/null | sed 's|.*/STORY-MAP-||;s|-.*||' | grep -oE '^[0-9]+' | sort -n | tail -1)
-  next=$(printf '%03d' $(( 10#${local_max:-0} + 1 )))
-  [ "$next" = "001" ]
-}
-
-@test "capture-story-map: next-ID formula computes 003 when STORY-MAP-002 exists locally" {
-  mkdir -p docs/story-maps/draft docs/story-maps/in-progress
-  touch docs/story-maps/draft/STORY-MAP-002-foo.html
-  touch docs/story-maps/in-progress/STORY-MAP-001-bar.html
-  local_max=$(ls docs/story-maps/*/STORY-MAP-*.html 2>/dev/null | sed 's|.*/STORY-MAP-||;s|-.*||' | grep -oE '^[0-9]+' | sort -n | tail -1)
-  next=$(printf '%03d' $(( 10#${local_max:-0} + 1 )))
-  [ "$next" = "003" ]
-}
-
-@test "capture-story-map: update-problem-references-section.sh accepts 'Story Maps' section name" {
-  mkdir -p docs/problems/known-error docs/story-maps/draft
-  cat > docs/problems/known-error/170-test.md <<'EOF'
-# P170: Test
-
-## Story Maps
-
-(empty)
-EOF
-  run bash "$HELPER_PROBLEM" docs/problems/known-error/170-test.md "Story Maps"
-  [[ "$output" != *"unknown section-name"* ]]
-}
-
-@test "capture-story-map: update-jtbd-references-section.sh accepts 'Story Maps' section name" {
-  mkdir -p docs/jtbd/solo-developer docs/story-maps/draft
-  cat > docs/jtbd/solo-developer/JTBD-008-test.proposed.md <<'EOF'
-# JTBD-008: Test
-
-## Story Maps
-
-(empty)
-EOF
-  run bash "$HELPER_JTBD" docs/jtbd/solo-developer/JTBD-008-test.proposed.md "Story Maps"
-  [[ "$output" != *"unknown section-name"* ]]
-}
-
-@test "capture-story-map: SKILL.md prescribes HTML encoding per ADR-060 amendment 2026-05-12" {
-  run grep -E 'HTML5|<!DOCTYPE html>|encoding amendment 2026-05-12' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "capture-story-map: SKILL.md prescribes draft/ landing subdir" {
-  run grep -E 'docs/story-maps/draft/STORY-MAP' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "capture-story-map: SKILL.md prohibits inline style on data-bearing elements per STYLE-GUIDE.md" {
-  run grep -iE 'NO inline.*style.*data-bearing|prohibited inline.*style' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}

package/skills/close-incident/test/close-incident-contract.bats

@@ -1,146 +0,0 @@
-#!/usr/bin/env bats
-# Contract assertions for /wr-itil:close-incident (P071 split slice 6c).
-#
-# This skill hosts the "close a restored incident" user intent previously
-# hidden behind /wr-itil:manage-incident <I> close. It checks the Linked
-# Problem's file suffix and, if the problem is in an acceptable terminal
-# state (.known-error.md / .verifying.md / .closed.md) OR the incident
-# documents a No Problem justification, renames the incident file from
-# .restored.md to .closed.md and updates the Status field.
-#
-# Structural assertion — Permitted Exception to the source-grep ban
-# (ADR-005 / P011 / ADR-037 contract-assertion pattern).
-#
-# @problem P071
-# @jtbd JTBD-001 (enforce governance without slowing down — discoverable surface)
-# @jtbd JTBD-101 (extend the suite with clear patterns — one skill per distinct user intent)
-# @jtbd JTBD-201 (restore service fast with an audit trail — close gate preserves the problem-handoff link)
-#
-# Cross-reference:
-#   P071: docs/problems/071-argument-based-skill-subcommands-are-not-discoverable.open.md
-#   ADR-010 amended (Skill Granularity section) — split naming + forwarder contract
-#   ADR-011 — manage-incident skill-wrapping precedent (close gate on linked problem)
-#   ADR-022 — problem lifecycle verification-pending status (.verifying.md accepted alongside .known-error.md / .closed.md)
-#   ADR-037 — contract-assertion bats pattern
-
-setup() {
-  SKILL_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  SKILL_FILE="${SKILL_DIR}/SKILL.md"
-}
-
-@test "SKILL.md exists and has frontmatter" {
-  [ -f "$SKILL_FILE" ]
-  run head -1 "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  [ "$output" = "---" ]
-}
-
-@test "SKILL.md frontmatter name is wr-itil:close-incident (P071 + ADR-010 amended)" {
-  # Split naming convention per ADR-010 amendment: <verb>-<object> pair.
-  # The new skill's name must match the phased-landing plan pinned in P071.
-  run grep -n "^name: wr-itil:close-incident$" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md frontmatter description names the close intent (P071)" {
-  # Description must name "close" and "incident" so Claude Code autocomplete
-  # surfaces the user intent rather than a generic name.
-  run grep -inE "^description:.*clos.*incident|^description:.*incident.*clos" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md frontmatter allowed-tools grants file-mutation surface (P071 — close requires rename + edit)" {
-  # close-incident renames .restored.md → .closed.md and updates the
-  # Status field. Unlike restore-incident, the linked-problem gate is a
-  # hard check (not a prompt), so no AskUserQuestion is required. No
-  # cross-skill invocation either, so no Skill tool is required.
-  run grep -nE "^allowed-tools:" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  run grep -nE "^allowed-tools:.*Write" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  run grep -nE "^allowed-tools:.*Edit" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  run grep -nE "^allowed-tools:.*Bash" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md documents the .restored.md → .closed.md rename (P071 + ADR-011)" {
-  # The close transition renames the incident file from .restored.md
-  # to .closed.md. The SKILL.md must name both suffixes explicitly so
-  # the file-suffix contract is legible.
-  run grep -inE "\.restored\.md" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  run grep -inE "\.closed\.md" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md documents the Linked-Problem gate with .known-error.md allowance (P071 + ADR-011)" {
-  # Per ADR-011, the close gate accepts a linked problem in
-  # .known-error.md state. Older form of the gate — must still be
-  # named explicitly post-ADR-022.
-  run grep -inE "known-error" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md documents the Linked-Problem gate .verifying.md allowance (P071 + ADR-022)" {
-  # Per ADR-022, .verifying.md (fix released, root cause confirmed,
-  # awaiting verification) also satisfies the incident-close gate.
-  # The SKILL.md must name this allowance explicitly so the ADR-022
-  # extension is preserved post-split.
-  run grep -inE "\.verifying\.md|verifying" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md documents the No Problem justification path (P071 + ADR-011)" {
-  # Per ADR-011, the incident may carry a ## No Problem section with a
-  # justification (e.g. "one-off cosmic-bit-flip"). In that case the
-  # linked-problem gate is bypassed. The SKILL.md must document the
-  # No Problem acceptance so the audit trail invariant is preserved.
-  run grep -inE "No Problem|no problem" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md documents the .open.md block condition (P071 + ADR-011)" {
-  # Per ADR-011, a linked problem in .open.md state blocks the close.
-  # The user must transition the problem to Known Error (or the
-  # verification pipeline must complete) before the incident can close.
-  # The SKILL.md must name this blocking condition explicitly.
-  run grep -inE "\.open\.md|Open" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md cites P071 and ADR-010 amended (P071 + ADR-025)" {
-  # ADR-025 inheritance per ADR-037: contract-assertion bats should reflect
-  # traceability cites on the skill spec document.
-  run grep -inE "P071|ADR-010" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md cites ADR-011 + ADR-022 for the close gate (P071 + ADR-011 + ADR-022)" {
-  # close-incident inherits the Linked-Problem gate from ADR-011 and the
-  # .verifying.md extension from ADR-022. Both must be cited so the
-  # precedent chain is legible.
-  run grep -inE "ADR-011" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  run grep -inE "ADR-022" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md does not carry a deprecated-arguments frontmatter flag (clean-split skill)" {
-  # close-incident is a clean-split skill with no argument-subcommands
-  # itself (its argument is a data parameter — incident ID only).
-  # ADR-010 amendment's `deprecated-arguments: true` flag is only valid
-  # on host skills with forwarder routes. close-incident is a
-  # forwarder TARGET, not a host. It must NOT carry the flag.
-  run grep -E "^deprecated-arguments:" "$SKILL_FILE"
-  [ "$status" -ne 0 ]
-}
-
-@test "SKILL.md does not use word-argument subcommand branching (P071 regression guard)" {
-  # The whole point of P071: Claude Code autocomplete does not surface
-  # word-argument subcommands. A clean-split skill must not reintroduce
-  # word-arg subcommand routing. The data parameter <I> is a string,
-  # not a verb keyword.
-  run grep -inE "If arguments start with \"(list|mitigate|restore|close|link)\"|If arguments contain \"(list|mitigate|restore|close|link)\"" "$SKILL_FILE"
-  [ "$status" -ne 0 ]
-}