@windyroad/itil 0.61.1 → 0.61.2

package/skills/manage-problem/test/manage-problem-verification-detection.bats

@@ -1,64 +0,0 @@
-#!/usr/bin/env bats
-# Doc-lint guard: manage-problem SKILL.md must surface Verification Pending
-# tickets whose fix path has been exercised in practice — the detection-layer
-# fix for P048. Minimal-scope candidates per P048:
-#   - Fix 1: step 9d always fires even on the fast-path cache hit.
-#   - Fix 4: step 9c / Verification Queue highlights tickets whose release
-#     age is >= 14 days as "likely verified" candidates.
-#
-# Candidates 2 (standalone verify-fixes op), 3 (exercise observations with a
-# new file-level state dimension), and 5 (AFK-mode hook) are deferred —
-# candidate 3 needs an ADR-scope decision per P048 investigation tasks.
-#
-# Structural assertion — Permitted Exception to the source-grep ban
-# (ADR-005 / P011).
-#
-# Cross-reference:
-#   P048: docs/problems/048-manage-problem-does-not-detect-verification-candidates.open.md
-#   ADR-022 (Verification Pending status): docs/decisions/022-problem-lifecycle-verification-pending-status.proposed.md
-#   ADR-013 Rule 1: docs/decisions/013-structured-user-interaction-for-governance-decisions.proposed.md
-#   @jtbd JTBD-001 (enforce governance without slowing down)
-#   @jtbd JTBD-006 (progress the backlog while I'm away)
-#   @jtbd JTBD-101 (extend the suite with clear patterns)
-
-setup() {
-  SKILL_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  SKILL_FILE="${SKILL_DIR}/SKILL.md"
-}
-
-@test "SKILL.md exists (P048 precondition)" {
-  [ -f "$SKILL_FILE" ]
-}
-
-@test "SKILL.md fast-path explicitly fires step 9d on cache hit (P048 Candidate 1)" {
-  # The fast-path currently reads: "Skip steps 9a-9b entirely / Proceed
-  # directly to step 9c". P048 reports step 9d being treated as skipped
-  # alongside 9a-9b. Fix: add explicit wording that step 9d fires even on
-  # cache hit, so verification candidates are always offered on review.
-  run grep -inE "step 9d.*(always fires?|still fires?|fires? (even|on) (cache|fast-path))|fast-path.*step 9d fires|cache hit.*step 9d" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md step 9c Verification Queue highlights release-age candidates (P048 Candidate 4)" {
-  # Review output must surface Verification Pending tickets whose release
-  # age crosses a threshold as "likely verified" candidates. Accept either
-  # a literal "likely verified" callout OR an explicit age-based criterion
-  # (>= 14 days) in the Verification Queue presentation.
-  run grep -inE "likely verified|release age|age.*(>=|greater than|over|older than).*(days?|d)|stale verification" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md step 9c Verification Queue documents the 14-day default threshold (P048 Candidate 4)" {
-  # The 14-day threshold is a within-skill default per architect review;
-  # the default must be named in the SKILL.md so implementations stay
-  # consistent until a policy-level decision moves it.
-  run grep -inE "14[ -]day|14 days|fourteen days" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md cites P048 for the detection-layer extension" {
-  # Traceability: the SKILL.md must name the problem ticket this
-  # detection extension addresses so reviewers can chase history.
-  run grep -n "P048" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}

package/skills/manage-problem/test/manage-problem-verification-pending.bats

@@ -1,114 +0,0 @@
-#!/usr/bin/env bats
-# Doc-lint guard: manage-problem / work-problems / manage-incident SKILL.md
-# files must document the Verification Pending lifecycle status per ADR-022.
-# README.md template must present a Verification Queue section driven off the
-# `.verifying.md` glob.
-#
-# Structural assertion — Permitted Exception to the source-grep ban
-# (ADR-005 / P011). These tests assert that the three skill specification
-# documents and the README template contain the status contract introduced
-# by ADR-022 (Problem lifecycle — Verification Pending).
-#
-# Migration of existing `.known-error.md` files to `.verifying.md` is a
-# separate follow-up commit per ADR-022 Scope; these tests cover only the
-# documentation/contract changes that land in this iteration.
-#
-# Cross-reference:
-#   ADR-022: docs/decisions/022-problem-lifecycle-verification-pending-status.proposed.md
-#   P049: docs/problems/049-known-error-status-overloaded-with-fix-released-substate.*.md
-#   P048: docs/problems/048-manage-problem-does-not-detect-verification-candidates.open.md
-#   ADR-011: docs/decisions/011-manage-incident-skill.proposed.md
-#   ADR-014: docs/decisions/014-governance-skills-commit-their-own-work.proposed.md
-#   @jtbd JTBD-001 (enforce governance without slowing down)
-#   @jtbd JTBD-006 (progress the backlog while I'm away)
-#   @jtbd JTBD-101 (extend the suite with clear patterns)
-#   @jtbd JTBD-201 (restore service fast with an audit trail)
-
-setup() {
-  TEST_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")" && pwd)"
-  # Test file lives at packages/itil/skills/manage-problem/test/ — walk up to the
-  # repo root and then into the three SKILL.md locations.
-  REPO_ROOT="$(cd "${TEST_DIR}/../../../../.." && pwd)"
-  MANAGE_PROBLEM="${REPO_ROOT}/packages/itil/skills/manage-problem/SKILL.md"
-  WORK_PROBLEMS="${REPO_ROOT}/packages/itil/skills/work-problems/SKILL.md"
-  MANAGE_INCIDENT="${REPO_ROOT}/packages/itil/skills/manage-incident/SKILL.md"
-  README_FILE="${REPO_ROOT}/docs/problems/README.md"
-}
-
-@test "ADR-022 exists (P049 precondition)" {
-  [ -f "${REPO_ROOT}/docs/decisions/022-problem-lifecycle-verification-pending-status.proposed.md" ] || \
-    [ -f "${REPO_ROOT}/docs/decisions/022-problem-lifecycle-verification-pending-status.accepted.md" ]
-}
-
-@test "manage-problem SKILL.md exists (P049 precondition)" {
-  [ -f "$MANAGE_PROBLEM" ]
-}
-
-@test "manage-problem SKILL.md lifecycle table includes Verification Pending status (P049 + ADR-022)" {
-  # ADR-022 Confirmation item 1: the lifecycle table must carry
-  # "Verification Pending | .verifying.md" as a first-class row.
-  run grep -inE "Verification Pending" "$MANAGE_PROBLEM"
-  [ "$status" -eq 0 ]
-  run grep -inE "\.verifying\.md" "$MANAGE_PROBLEM"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-problem SKILL.md WSJF multiplier table documents Verification Pending exclusion (P049 + ADR-022)" {
-  # ADR-022 Confirmation item 1: status multiplier table must carry a row
-  # with the new status and either multiplier 0 OR explicit "excluded"
-  # wording. Accept either so future tuning within the ADR's "reassessment
-  # criteria" envelope doesn't require this test to change.
-  run grep -inE "Verification Pending.*(0|excluded)|excluded.*Verification Pending" "$MANAGE_PROBLEM"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-problem SKILL.md documents the Known Error to Verification Pending transition (P049 + ADR-022)" {
-  # ADR-022 Scope line 61: the skill must describe the Known Error →
-  # Verification Pending transition explicitly (git mv + Status field
-  # update + Fix Released section in the same commit).
-  run grep -inE "(Known Error|\.known-error\.md).*(→|->|to).*(Verification Pending|\.verifying\.md)|git mv.*\.verifying\.md" "$MANAGE_PROBLEM"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-problem SKILL.md review step 9 targets .verifying.md via glob or suffix (P049 + ADR-022)" {
-  # ADR-022 Confirmation: step 9d targets *.verifying.md via glob rather
-  # than scanning .known-error.md bodies.
-  run grep -inE "\*\.verifying\.md|\.verifying\.md.*glob|glob.*\.verifying\.md" "$MANAGE_PROBLEM"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-problem SKILL.md review step 9 has a Verification Queue section (P049 + ADR-022)" {
-  # ADR-022 Scope line 63: dedicated Verification Queue section parallel to
-  # the main ranked table.
-  run grep -inE "Verification Queue" "$MANAGE_PROBLEM"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-problem SKILL.md cites ADR-022 (P049)" {
-  # Traceability: the skill must cite the ADR that governs the status
-  # contract so reviewers can chase the decision from the implementation.
-  run grep -n "ADR-022" "$MANAGE_PROBLEM"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems SKILL.md classifier uses the .verifying.md suffix (P049 + ADR-022)" {
-  # ADR-022 Scope line 67: work-problems classifier becomes suffix-based
-  # rather than file-body-scan-based.
-  run grep -inE "\.verifying\.md" "$WORK_PROBLEMS"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-incident SKILL.md linked-problem close gating accepts .verifying.md (P049 + ADR-022)" {
-  # ADR-022 Scope line 68: incident close gating accepts .verifying.md
-  # alongside .known-error.md and .closed.md; .open.md still blocks.
-  run grep -inE "\.verifying\.md" "$MANAGE_INCIDENT"
-  [ "$status" -eq 0 ]
-}
-
-@test "docs/problems/README.md template has a Verification Queue section (P049 + ADR-022)" {
-  # ADR-022 Scope line 69: README replaces the hand-maintained "Known Errors
-  # (Fix Released — pending verification)" shadow table with a Verification
-  # Queue driven off the glob.
-  run grep -inE "Verification Queue" "$README_FILE"
-  [ "$status" -eq 0 ]
-}

package/skills/manage-problem/test/manage-problem-work-forwarder.bats

@@ -1,88 +0,0 @@
-#!/usr/bin/env bats
-# Contract assertions for manage-problem's `work` subcommand forwarder (P071 split slice 3).
-#
-# Per ADR-010 amended (Skill Granularity section) + P071 phased plan:
-# `/wr-itil:manage-problem work` delegates to the new
-# `/wr-itil:work-problem` (singular) skill via a thin-router forwarder.
-# Original skill already carries `deprecated-arguments: true` frontmatter
-# from slice 1; this file asserts the work-forwarder-specific contract.
-#
-# CRITICAL naming distinction: the forwarder target is singular
-# `/wr-itil:work-problem` — NOT the plural `/wr-itil:work-problems`
-# (the AFK orchestrator that already exists). Name coexistence is a
-# P071-acknowledged trade-off.
-#
-# 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)
-# @jtbd JTBD-101 (extend the suite with clear patterns)
-#
-# Cross-reference:
-#   P071: docs/problems/071-argument-based-skill-subcommands-are-not-discoverable.open.md
-#   ADR-010 amended — split naming + forwarder contract + deprecated-arguments flag
-#   ADR-013 Rule 1 — structured user interaction (forwarder emits systemMessage, not AskUserQuestion)
-#   ADR-037 — contract-assertion bats pattern
-
-setup() {
-  SKILL_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  SKILL_FILE="${SKILL_DIR}/SKILL.md"
-}
-
-@test "manage-problem Step 1 forwards 'work' argument to /wr-itil:work-problem singular (P071 slice 3)" {
-  # The forwarder names the target skill explicitly so the router is
-  # legible at the contract level. ADR-010's canonical shape: "invokes
-  # the new named skill via the Skill tool, not via re-prompting the
-  # user". Target is the SINGULAR name.
-  run grep -inE "/wr-itil:work-problem\b" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-problem Step 1 work-forwarder target is singular, not plural (name collision guard)" {
-  # The forwarder target MUST be /wr-itil:work-problem (singular).
-  # The plural /wr-itil:work-problems already exists as the AFK
-  # orchestrator and has a completely different semantics (batch loop
-  # vs single ticket). A forwarder that routed 'work' to the plural
-  # orchestrator would subtly break interactive invocations — the user
-  # would get an AFK-mode iteration loop when they wanted one ticket.
-  #
-  # Assert the forwarder block explicitly names 'singular' so the
-  # distinction is legible at the contract level.
-  run grep -inE "/wr-itil:work-problem.*singular|singular.*work-problem" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-problem Step 1 emits the canonical work deprecation notice (ADR-010 amended)" {
-  # ADR-010's canonical deprecation-notice template:
-  # "/wr-<plugin>:<old> <arg> is deprecated; use /wr-<plugin>:<new>
-  #  directly. This forwarder will be removed in <plugin>'s next major
-  #  version."
-  # The notice MUST be emitted as a systemMessage (not AskUserQuestion)
-  # because deprecation is informational, not decisional (ADR-013 Rule 1
-  # scope). Work-forwarder notice is distinct from list + review
-  # forwarders; all three must co-exist verbatim.
-  run grep -inE "is deprecated.*use /wr-itil:work-problem|deprecated.*work-problem|removed in .* next major version" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-problem Step 1 work-forwarder does not re-implement selection logic (P071 regression guard)" {
-  # The forwarder must not duplicate the selection logic — it must
-  # delegate. Per ADR-010: "thin-router forwarder re-invokes the new
-  # named skill via the Skill tool". If the forwarder grows its own
-  # freshness-check + AskUserQuestion + delegate stack, the
-  # deprecation window will harden into a permanent fork.
-  run grep -inE "delegate.*work-problem\b|Skill tool.*work-problem\b|work-problem\b.*Skill tool" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-problem work-forwarder Step 1 parser delegates, does not run inline (P071 slice 3)" {
-  # Slice 3's defining behaviour: Step 1's `work` branch must delegate,
-  # not run the review+select flow inline. The parser line must say
-  # "delegate to /wr-itil:work-problem", matching the shape the list
-  # and review forwarders use. A stale "run a review (step 9) first"
-  # line would indicate the forwarder was added without updating the
-  # parser (common slip per slice 2's lesson).
-  run grep -nE "^- If arguments contain \"work\", \*\*delegate to \`/wr-itil:work-problem\`\*\*" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}

package/skills/manage-story/test/manage-story-contract.bats

@@ -1,174 +0,0 @@
-#!/usr/bin/env bats
-# Behavioural contract fixtures for /wr-itil:manage-story (P170 Phase 2
-# Slice 8 — ADR-060 amendment 2026-05-10 lines 200-253 + 270 + 292).
-#
-# Per ADR-052: behavioural assertions on observable SKILL contract
-# surfaces. Skill-prose behaviour for prompt-driven agents isn't
-# directly testable in bats; these tests assert presence of the
-# load-bearing identifiers in the SKILL contract (the closest in-
-# session-reachable approximation per P081 + P012).
-#
-# Behavioural surfaces under test:
-#   1. SKILL.md presence + canonical name.
-#   2. I6-I11 invariant table presence (load-bearing).
-#   3. I7 + I8 hard-block fires at accepted transition (not earlier).
-#   4. I10 INVEST shape gate names all 4 INVEST checks (Testable /
-#      Valuable / Independent / Estimable) + L/XL decomposition-
-#      candidate advisory per ADR-060 line 252 nitpick N3.
-#   5. Auto-transition triggers named: draft→in-progress on first
-#      non-capture commit; in-progress→done on criteria-ticked + RFC
-#      closed.
-#   6. Bootstrap-exemption marker contract named per ADR-060 line 339.
-#   7. Reverse-trace refresh on all 4 parent tiers (problem / JTBD /
-#      RFC / story-map) named, with story-map manual placement noted
-#      per architect amend finding 2 on Slice 7.
-#   8. No-WSJF-leak (I11) — no WSJF field in argument grammar or
-#      frontmatter handling.
-#
-# @problem P170
-# @jtbd JTBD-008 (Decompose a Fix Into Coordinated Changes)
-# @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-032  (governance-skill aside-invocation pattern)
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../../.." && pwd)"
-  SKILL_FILE="${REPO_ROOT}/packages/itil/skills/manage-story/SKILL.md"
-}
-
-# ---------------------------------------------------------------------------
-# Surface 0: SKILL.md presence + canonical name
-# ---------------------------------------------------------------------------
-
-@test "manage-story: SKILL.md exists" {
-  [ -f "$SKILL_FILE" ]
-}
-
-@test "manage-story: SKILL.md frontmatter declares wr-itil:manage-story name" {
-  run grep -E '^name: wr-itil:manage-story$' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-# ---------------------------------------------------------------------------
-# Surface 1: I6-I11 invariant table presence
-# ---------------------------------------------------------------------------
-
-@test "manage-story: SKILL.md declares I6 trace-to-problem invariant" {
-  run grep -E 'I6.*trace-to-problem|trace-to-problem.*I6' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-story: SKILL.md declares I7 trace-to-RFC invariant" {
-  run grep -E 'I7.*trace-to-RFC|trace-to-RFC.*I7' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-story: SKILL.md declares I8 trace-to-story-map invariant" {
-  run grep -E 'I8.*trace-to-story-map|trace-to-story-map.*I8' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-story: SKILL.md declares I9 trace-to-JTBD invariant" {
-  run grep -E 'I9.*trace-to-JTBD|trace-to-JTBD.*I9' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-story: SKILL.md declares I10 INVEST shape invariant" {
-  run grep -E 'I10.*INVEST|INVEST.*I10' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-story: SKILL.md declares I11 no-WSJF-leak invariant" {
-  run grep -E 'I11.*WSJF|WSJF.*I11|no-WSJF-leak' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-# ---------------------------------------------------------------------------
-# Surface 2: I7 + I8 fire at accepted transition (not earlier)
-# ---------------------------------------------------------------------------
-
-@test "manage-story: SKILL.md states I7 + I8 hard-block fires at accepted transition" {
-  # The deferred-from-capture contract is load-bearing per ADR-060
-  # line 291 — capture-story permits absent traces; manage-story
-  # accepted gate enforces them.
-  run grep -iE 'I7.*accepted|hard-block at.*accepted|fires at.*accepted.*I7' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-# ---------------------------------------------------------------------------
-# Surface 3: I10 INVEST 4-axis check
-# ---------------------------------------------------------------------------
-
-@test "manage-story: SKILL.md names all 4 INVEST checks (Testable/Valuable/Independent/Estimable)" {
-  for axis in Testable Valuable Independent Estimable; do
-    run grep -i "$axis" "$SKILL_FILE"
-    [ "$status" -eq 0 ]
-  done
-}
-
-@test "manage-story: SKILL.md names L/XL decomposition-candidate advisory per ADR-060 nitpick N3" {
-  run grep -iE 'L/XL.*decomposition|decomposition.candidate|N3' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-# ---------------------------------------------------------------------------
-# Surface 4: Auto-transition triggers
-# ---------------------------------------------------------------------------
-
-@test "manage-story: SKILL.md names accepted → in-progress on the first implementing commit (ADR-096: draft blocked, not auto-advanced)" {
-  run grep -iE 'accepted → in-progress' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  # ADR-096: the old draft → in-progress auto-transition is removed.
-  run grep -iE 'NO .draft → in-progress. transition|never implementable' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-story: SKILL.md names in-progress → done auto-transition on criteria-ticked + RFC closed" {
-  run grep -iE 'in-progress.*done.*criteria.*ticked|all.*criteria.*ticked.*RFC.*closed|RFC.*closed.*all-criteria' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-# ---------------------------------------------------------------------------
-# Surface 5: Bootstrap-exemption marker
-# ---------------------------------------------------------------------------
-
-@test "manage-story: SKILL.md names bootstrap-exemption marker per ADR-060 line 339" {
-  run grep -iE 'bootstrap-exempt|bootstrap.exemption|ADR-053.*Bootstrapping' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-# ---------------------------------------------------------------------------
-# Surface 6: Reverse-trace refresh on 4 parent tiers
-# ---------------------------------------------------------------------------
-
-@test "manage-story: SKILL.md names reverse-trace refresh on problem parents via update-problem-references-section.sh" {
-  run grep -E 'update-problem-references-section\.sh.*Stories' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-story: SKILL.md names reverse-trace refresh on JTBD parents via update-jtbd-references-section.sh" {
-  run grep -E 'update-jtbd-references-section\.sh.*Stories' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-story: SKILL.md names reverse-trace refresh on RFC parents via update-rfc-references-section.sh" {
-  run grep -E 'update-rfc-references-section\.sh.*Stories' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-story: SKILL.md names story-map parents as MANUAL placement (no automatic refresh per Slice 7 architect amend finding 2)" {
-  run grep -iE 'manually-authored|manual placement|no automatic refresh' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-# ---------------------------------------------------------------------------
-# Surface 7: I11 no-WSJF-leak — no WSJF field in argument grammar
-# ---------------------------------------------------------------------------
-
-@test "manage-story: SKILL.md argument grammar does NOT include a WSJF flag/token (I11 invariant)" {
-  # The argument grammar block — extract the fenced block following
-  # "## Argument grammar" and verify no WSJF token in it.
-  run grep -A 20 '^## Argument grammar' "$SKILL_FILE"
-  [[ "$output" != *"WSJF"* ]] && [[ "$output" != *"wsjf"* ]]
-}

package/skills/manage-story-map/test/manage-story-map-contract.bats

@@ -1,63 +0,0 @@
-#!/usr/bin/env bats
-# Behavioural contract fixtures for /wr-itil:manage-story-map (P170 Phase 2 Slice 4).
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../../.." && pwd)"
-  SKILL_FILE="${REPO_ROOT}/packages/itil/skills/manage-story-map/SKILL.md"
-}
-
-@test "manage-story-map: SKILL.md exists" {
-  [ -f "$SKILL_FILE" ]
-}
-
-@test "manage-story-map: SKILL.md frontmatter declares wr-itil:manage-story-map name" {
-  run grep -E '^name: wr-itil:manage-story-map$' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-story-map: SKILL.md declares I3 trace-to-problem invariant" {
-  run grep -E 'I3.*trace-to-problem|trace-to-problem.*I3' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-story-map: SKILL.md declares I4 trace-to-JTBD invariant" {
-  run grep -E 'I4.*trace-to-JTBD|trace-to-JTBD.*I4' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-story-map: SKILL.md declares I5 no-WSJF-leak invariant" {
-  run grep -E 'I5.*WSJF|WSJF.*I5|no-WSJF-leak' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-story-map: SKILL.md names 5 lifecycle states (draft / accepted / in-progress / completed / archived)" {
-  for state in draft accepted in-progress completed archived; do
-    run grep -E "${state}" "$SKILL_FILE"
-    [ "$status" -eq 0 ]
-  done
-}
-
-@test "manage-story-map: SKILL.md names bootstrap-exemption marker per ADR-060 line 339" {
-  run grep -iE 'bootstrap-exempt|ADR-053.*Bootstrapping' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-story-map: SKILL.md names reverse-trace refresh on problem parents via Story Maps section" {
-  run grep -E 'wr-itil-update-problem-references-section.*Story Maps' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-story-map: SKILL.md names reverse-trace refresh on JTBD parents via Story Maps section" {
-  run grep -E 'wr-itil-update-jtbd-references-section.*Story Maps' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-story-map: SKILL.md states map HTML files do NOT carry markdown reverse-trace section themselves (per Slice 7 architect amend finding 2)" {
-  run grep -iE 'do NOT carry.*markdown reverse-trace|authored manually|manual.*data-attribute' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "manage-story-map: argument grammar contains no WSJF token (I5 invariant)" {
-  run grep -A 10 '^## Argument grammar' "$SKILL_FILE"
-  [[ "$output" != *"WSJF"* ]] && [[ "$output" != *"wsjf"* ]]
-}