@windyroad/itil 0.61.1 → 0.61.2

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

@@ -1,125 +0,0 @@
-#!/usr/bin/env bats
-# Contract assertions for /wr-itil:link-incident (P071 split slice 6d).
-#
-# This skill hosts the "link an incident to a problem" user intent
-# previously hidden behind /wr-itil:manage-incident <I> link P<M>. It
-# verifies the target problem file exists, then writes (or updates) the
-# ## Linked Problem section of the incident file with the problem ID,
-# title, and current status.
-#
-# 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 — Linked Problem traceability)
-#
-# 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 (Linked Problem section)
-#   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:link-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:link-incident$" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md frontmatter description names the link intent (P071)" {
-  # Description must name "link" and "incident" / "problem" so Claude Code
-  # autocomplete surfaces the user intent rather than a generic name.
-  run grep -inE "^description:.*link.*(incident|problem)|^description:.*(incident|problem).*link" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md frontmatter allowed-tools grants file-mutation surface (P071 — link requires read + edit)" {
-  # link-incident verifies the problem file exists (Read + Glob/Bash),
-  # reads the incident file (Read), and writes the Linked Problem
-  # section (Edit / Write). No Skill tool (no cross-skill invocation),
-  # no AskUserQuestion (both args are data parameters).
-  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 P<MMM> problem-file lookup (P071 + ADR-011)" {
-  # The link operation verifies docs/problems/P<MMM>-*.md exists before
-  # writing the Linked Problem section. The SKILL.md must name the
-  # problem-file lookup explicitly so the cross-directory dependency is
-  # legible.
-  run grep -inE "docs/problems|P<MMM>|P<NNN>|problem file" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md documents the ## Linked Problem section write (P071 + ADR-011)" {
-  # The link operation writes (or updates) the ## Linked Problem section
-  # with `P<MMM> (<title>) — <status>`. The SKILL.md must name the
-  # section explicitly so the audit-trail invariant (JTBD-201) is
-  # preserved post-split.
-  run grep -inE "Linked Problem" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md documents both data-parameter arguments (P071 — incident ID + problem ID)" {
-  # link-incident takes two positional data parameters: the incident ID
-  # (I###) and the problem ID (P###). The SKILL.md must document both
-  # so the argument shape is legible.
-  run grep -inE "I<NNN>|I<###>|incident ID|I007|<I###>" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  run grep -inE "P<MMM>|P<###>|P<NNN>|problem ID" "$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 for the Linked Problem section convention (P071 + ADR-011)" {
-  # link-incident inherits the Linked Problem section convention from
-  # ADR-011. The SKILL.md must cite ADR-011 so the precedent chain is
-  # legible.
-  run grep -inE "ADR-011" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md does not carry a deprecated-arguments frontmatter flag (clean-split skill)" {
-  # link-incident is a clean-split skill with no argument-subcommands
-  # itself (its arguments are data parameters — incident ID + problem ID).
-  # ADR-010 amendment's `deprecated-arguments: true` flag is only valid
-  # on host skills with forwarder routes. link-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 parameters <I> and <P> are
-  # strings, not verb keywords.
-  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 ]
-}

package/skills/list-incidents/test/list-incidents-contract.bats

@@ -1,115 +0,0 @@
-#!/usr/bin/env bats
-# Contract assertions for /wr-itil:list-incidents (P071 split slice 5).
-#
-# This skill hosts the "list active incidents" user intent previously
-# hidden behind /wr-itil:manage-incident list. It is a pure read-only
-# display skill — no branching, no interaction, no file edits.
-#
-# 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 — incident status visibility)
-#
-# 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 (list-incidents mirrors its file conventions)
-#   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:list-incidents (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:list-incidents$" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md frontmatter description names the list intent (P071)" {
-  # Description must name "list" and "incident" so Claude Code autocomplete
-  # surfaces the user intent rather than a generic name.
-  run grep -inE "^description:.*list.*incident|^description:.*incident.*list" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md frontmatter allowed-tools is read-only (P071 — read-only list display)" {
-  # Pure read-only list display (mirrors list-problems slice 1). No Write,
-  # no Edit, no AskUserQuestion. This assertion is the enforceable contract —
-  # future maintainers cannot accidentally grow the tool surface past the
-  # intent.
-  run grep -nE "^allowed-tools:" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  run grep -E "^allowed-tools:.*(Write|Edit|AskUserQuestion)" "$SKILL_FILE"
-  [ "$status" -ne 0 ]
-}
-
-@test "SKILL.md documents the read-only scan scope (P071 + ADR-011)" {
-  # The skill reads the three active incident statuses per ADR-011:
-  # .investigating.md, .mitigating.md, and .restored.md. Closed incidents
-  # are omitted (the view is active backlog, not archive). The SKILL.md
-  # must name all three glob patterns explicitly so the contract is
-  # legible.
-  run grep -inE "\.investigating\.md" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  run grep -inE "\.mitigating\.md" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  run grep -inE "\.restored\.md" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md documents severity-sorted output (P071 + ADR-011)" {
-  # Unlike list-problems (WSJF-sorted), list-incidents is severity-sorted
-  # per ADR-011 "Severity, not WSJF" — incidents are time-bound events
-  # where effort divisor is meaningless. The SKILL.md must name severity
-  # as the sort key.
-  run grep -inE "severity" "$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 does not carry a deprecated-arguments frontmatter flag (clean-split skill)" {
-  # list-incidents is a clean-split skill with no argument-subcommands
-  # itself. ADR-010 amendment's `deprecated-arguments: true` flag is only
-  # valid on host skills with forwarder routes — list-incidents is the
-  # forwarder TARGET, not the 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
-  # arguments. A clean-split skill must not reintroduce word-arg subcommand
-  # routing. The SKILL.md must not contain `If arguments start with "list"`
-  # / `If arguments contain "mitigate"` / etc. patterns.
-  run grep -inE "If arguments start with|If arguments contain" "$SKILL_FILE"
-  [ "$status" -ne 0 ]
-}
-
-@test "SKILL.md documents the ownership boundary — no README rewrite (P071)" {
-  # list-incidents displays active incidents but does NOT maintain a README
-  # cache (unlike list-problems which reads a README written by review-problems).
-  # For incidents, there is no README cache — the skill always runs a live
-  # scan. The SKILL.md must explicitly name the "read-only / no-edit" stance
-  # so the ownership boundary is legible to future maintainers.
-  run grep -inE "read-only|does not (edit|modify|rewrite|commit)" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}

package/skills/list-problems/test/list-problems-contract.bats

@@ -1,100 +0,0 @@
-#!/usr/bin/env bats
-# Contract assertions for /wr-itil:list-problems (P071 split slice 1).
-#
-# This skill hosts the "list open and known-error problems" user intent
-# previously hidden behind /wr-itil:manage-problem list. It is a pure
-# read-only display skill — no branching, no interaction, no file edits.
-#
-# 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)
-#
-# 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-022 — Verification Pending status conventions (list-problems respects `.verifying.md` exclusion)
-#   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:list-problems (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:list-problems$" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md frontmatter description names the list intent (P071)" {
-  # Description must name "list" and "problem" so Claude Code autocomplete
-  # surfaces the user intent rather than a generic name.
-  run grep -inE "^description:.*list.*problem|^description:.*problem.*list" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md frontmatter allowed-tools is read-only (P071 + architect Q3)" {
-  # Architect Q3: pure read-only list display. No Write, no Edit, no
-  # AskUserQuestion. This assertion is the enforceable contract — future
-  # maintainers cannot accidentally grow the tool surface past the intent.
-  run grep -nE "^allowed-tools:" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  run grep -E "^allowed-tools:.*(Write|Edit|AskUserQuestion)" "$SKILL_FILE"
-  [ "$status" -ne 0 ]
-}
-
-@test "SKILL.md documents the read-only scan scope (P071)" {
-  # The skill reads .open.md and .known-error.md files from docs/problems/.
-  # .verifying.md and .parked.md are excluded from the dev-work ranking per
-  # ADR-022 (Verification Pending) and the Parked-policy; but they MAY be
-  # shown in dedicated sections. The SKILL.md must name both `.open.md` and
-  # `.known-error.md` glob patterns explicitly so the contract is legible.
-  run grep -inE "\.open\.md" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  run grep -inE "\.known-error\.md" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md reuses the git-log freshness check from manage-problem review (P071 + architect Q4)" {
-  # Architect Q4: reuse the git-log freshness check from manage-problem
-  # review (same skill suite, same cache semantics per P031).
-  run grep -inE "git log.*README\.md|readme_commit|git log -1 --format=%H" "$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 does not carry a deprecated-arguments frontmatter flag (clean-split skill)" {
-  # Architect advisory: list-problems is a clean-split skill with no
-  # argument-subcommands itself. ADR-010 amendment's
-  # `deprecated-arguments: true` flag is only valid on host skills with
-  # forwarder routes — list-problems is the forwarder TARGET, not the
-  # 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
-  # arguments. A clean-split skill must not reintroduce word-arg subcommand
-  # routing. The SKILL.md must not contain `If arguments start with "list"`
-  # / `If arguments contain "work"` / etc. patterns.
-  run grep -inE "If arguments start with|If arguments contain" "$SKILL_FILE"
-  [ "$status" -ne 0 ]
-}

package/skills/list-stories/test/list-stories-contract.bats

@@ -1,127 +0,0 @@
-#!/usr/bin/env bats
-# Behavioural contract fixtures for /wr-itil:list-stories (P170 Phase 2 Slice 10).
-#
-# Per ADR-037 + ADR-052: behavioural tests over structural prose-grep.
-# These tests exercise the read-only display surface contract — read
-# story files, render markdown tables, no edits.
-#
-# Behavioural surfaces under test:
-#   1. SKILL.md presence + canonical name (minimum discoverability).
-#   2. Read-only contract — no Write/Edit tools in frontmatter.
-#   3. Lifecycle enumeration — list-stories enumerates all five state
-#      subdirectories (draft / accepted / in-progress / done / archived).
-#   4. RFC-filter ordering — when --rfc filter is provided, ordering
-#      follows the RFC frontmatter `stories:` array, not lexical /
-#      filesystem order.
-#
-# @problem P170
-# @jtbd JTBD-008 (Decompose a Fix Into Coordinated Changes — list view
-#                  supports the working-the-problem flow's per-RFC iter
-#                  dispatch in Slice 13)
-# @jtbd JTBD-006 (Progress the Backlog While I'm Away — filtered mode
-#                  feeds the AFK orchestrator)
-# @adr  ADR-060  (Problem-RFC-Story framework — story tier line 294
-#                  list-stories description)
-# @adr  ADR-037  (Skill testing strategy — contract bats)
-# @adr  ADR-052  (Behavioural-tests-default)
-# @adr  ADR-010  (Amended skill granularity — phased-landing split
-#                  precedent from list-problems / P071)
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../../.." && pwd)"
-  SKILL_FILE="${REPO_ROOT}/packages/itil/skills/list-stories/SKILL.md"
-
-  TMPROOT=$(mktemp -d)
-  ORIG_DIR="$PWD"
-  cd "$TMPROOT"
-  # Set up minimal story corpus fixture
-  mkdir -p docs/stories/draft docs/stories/accepted docs/stories/in-progress docs/stories/done docs/stories/archived
-  mkdir -p docs/rfcs
-}
-
-teardown() {
-  cd "$ORIG_DIR"
-  rm -rf "$TMPROOT"
-}
-
-# ---------------------------------------------------------------------------
-# Surface 0: SKILL.md exists with correct name (minimum discoverability)
-# ---------------------------------------------------------------------------
-
-@test "list-stories: SKILL.md exists" {
-  [ -f "$SKILL_FILE" ]
-}
-
-@test "list-stories: SKILL.md frontmatter declares wr-itil:list-stories name" {
-  run grep -E '^name: wr-itil:list-stories$' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-# ---------------------------------------------------------------------------
-# Surface 1: Read-only contract — no Write/Edit in allowed-tools
-# (ADR-010 phased-landing split rule: list-* skills are pure read views)
-# ---------------------------------------------------------------------------
-
-@test "list-stories: SKILL.md allowed-tools does NOT include Write or Edit" {
-  # Behavioural: extract the allowed-tools line and check it omits the
-  # write tools. The list-* family is read-only by contract per ADR-010.
-  run grep '^allowed-tools:' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  [[ "$output" != *"Write"* ]]
-  [[ "$output" != *"Edit"* ]]
-}
-
-# ---------------------------------------------------------------------------
-# Surface 2: Lifecycle enumeration — all 5 state subdirectories
-# (story lifecycle per ADR-060 mirrors problem lifecycle ADR-022)
-# ---------------------------------------------------------------------------
-
-@test "list-stories: SKILL.md names all 5 lifecycle state subdirectories" {
-  # The Scope section enumerates the lifecycle states. Without all five,
-  # the list view would miss stories in some lifecycle state silently —
-  # a behavioural defect not a prose preference.
-  for state in draft accepted in-progress done archived; do
-    run grep -E "docs/stories/${state}" "$SKILL_FILE"
-    [ "$status" -eq 0 ]
-  done
-}
-
-# ---------------------------------------------------------------------------
-# Surface 3: --rfc filter ordering follows RFC frontmatter `stories:` array
-# (load-bearing for the working-the-problem flow per ADR-060 line 314)
-# ---------------------------------------------------------------------------
-
-@test "list-stories: --rfc filter mode enumerates stories via RFC frontmatter stories array" {
-  # The SKILL.md filter-mode pseudocode reads the RFC's frontmatter and
-  # extracts STORY-NNN tokens in order. This is the load-bearing
-  # behaviour for Slice 13's working-the-problem traversal — verify
-  # the SKILL prescribes RFC-frontmatter-driven order, not filesystem
-  # / lexical order.
-  run grep -E 'stories_list.*RFC|stories: array|RFC frontmatter' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-# ---------------------------------------------------------------------------
-# Surface 4: Cache-freshness check pattern — same git log shape as list-problems
-# (P031: filesystem mtime unreliable in worktrees; git history is authoritative)
-# ---------------------------------------------------------------------------
-
-@test "list-stories: SKILL.md uses git log cache-freshness pattern per P031" {
-  run grep -E 'git log -1 --format=%H -- docs/stories/README\.md' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-# ---------------------------------------------------------------------------
-# Surface 5: No-WSJF-leak — I11 invariant
-# (ADR-060 line 253: stories MUST NOT carry a WSJF field in Phase 2)
-# ---------------------------------------------------------------------------
-
-@test "list-stories: SKILL.md does NOT render a WSJF column (I11 invariant)" {
-  # Phase 2 invariant: no story-level WSJF. The display tables must
-  # not include a WSJF column header. This is a behavioural contract
-  # at the SKILL output surface, not a prose-grep.
-  # The output tables specified in SKILL.md should have NO 'WSJF' header.
-  # Find any markdown table header line and verify none include WSJF.
-  run grep -E '^\| WSJF\b|\| WSJF \|' "$SKILL_FILE"
-  [ "$status" -ne 0 ]
-}

package/skills/list-story-maps/test/list-story-maps-contract.bats

@@ -1,46 +0,0 @@
-#!/usr/bin/env bats
-# Behavioural contract fixtures for /wr-itil:list-story-maps (P170 Phase 2 Slice 6).
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../../.." && pwd)"
-  SKILL_FILE="${REPO_ROOT}/packages/itil/skills/list-story-maps/SKILL.md"
-}
-
-@test "list-story-maps: SKILL.md exists" {
-  [ -f "$SKILL_FILE" ]
-}
-
-@test "list-story-maps: SKILL.md frontmatter declares wr-itil:list-story-maps name" {
-  run grep -E '^name: wr-itil:list-story-maps$' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "list-story-maps: SKILL.md allowed-tools does NOT include Write or Edit (read-only contract)" {
-  run grep '^allowed-tools:' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  [[ "$output" != *"Write"* ]]
-  [[ "$output" != *"Edit"* ]]
-}
-
-@test "list-story-maps: SKILL.md names all 5 lifecycle state subdirectories" {
-  for state in draft accepted in-progress completed archived; do
-    run grep -E "docs/story-maps/${state}|${state}/" "$SKILL_FILE"
-    [ "$status" -eq 0 ]
-  done
-}
-
-@test "list-story-maps: SKILL.md does NOT render a WSJF column (I5 invariant)" {
-  # Story-maps are planning artefacts, not work items — no WSJF per ADR-060 line 145.
-  run grep -E '^\| WSJF\b|\| WSJF \|' "$SKILL_FILE"
-  [ "$status" -ne 0 ]
-}
-
-@test "list-story-maps: SKILL.md names <meta> block parse target per ADR-060 lines 381-435" {
-  run grep -E '<meta name=|xmllint.*meta' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "list-story-maps: SKILL.md uses git log cache-freshness pattern per P031" {
-  run grep -E 'git log -1 --format=%H -- docs/story-maps/README\.md' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}