@windyroad/itil 0.61.1 → 0.61.2

package/skills/work-problems/test/work-problems-step-2-5-routing.bats

@@ -1,80 +0,0 @@
-#!/usr/bin/env bats
-
-# P122: work-problems Step 2.5 stop-condition #2 routing must default to
-# AskUserQuestion (interactive) and use the table emit only as the
-# AskUserQuestion-unavailable fallback. The "non-interactive default for
-# this skill" prose was wrong-by-design — JTBD-006 (AFK persona) is served
-# by the iteration subprocess workers, not by suppressing AskUserQuestion
-# at the orchestrator's main turn (which is interactive by construction).
-#
-# Doc-lint contract assertions per ADR-037 Permitted Exception (structural
-# checks on prose contract, not behavioural coverage).
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../../.." && pwd)"
-  SKILL_MD="$REPO_ROOT/packages/itil/skills/work-problems/SKILL.md"
-}
-
-@test "work-problems P122: SKILL.md exists" {
-  [ -f "$SKILL_MD" ]
-}
-
-@test "work-problems P122: Step 2.5 removes the legacy AFK-as-default prose" {
-  # The flipped default — the legacy "non-interactive path the default
-  # for this skill" prose at Step 2.5 must be gone. (Note: Step 0's
-  # session-continuity AFK branch intentionally retains its own AFK
-  # default for a different reason; this test is scoped to Step 2.5's
-  # specific phrasing only.)
-  run grep -F 'non-interactive path the default for this skill' "$SKILL_MD"
-  [ "$status" -ne 0 ]
-}
-
-@test "work-problems P122: Step 2.5 explicitly cites AskUserQuestion as the default branch" {
-  # The new default-selection prose names a "Default branch" that calls
-  # AskUserQuestion. Match the structural phrase rather than the
-  # backtick-wrapped tool name to keep the grep portable.
-  run grep -F 'Default branch' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P122: Step 2.5 cites the orchestrator-vs-subprocess principle" {
-  # The flip's load-bearing reasoning: the orchestrator's main turn is
-  # interactive by construction; AFK persona is served by the
-  # subprocess-boundary contract under ADR-032, not by suppressing
-  # AskUserQuestion at the orchestrator layer. (Architect FLAG —
-  # cross-skill principle sentence.)
-  run grep -F 'subprocess-boundary contract' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'interactive by construction' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P122: Step 2.5 preserves the user-answerable skip-reason scoping" {
-  # P103 anti-pattern boundary: the AskUserQuestion default must NOT
-  # broaden to architect-design or upstream-blocked skip-reasons. The
-  # user-answerable scoping from Step 4's classifier remains in force.
-  run grep -F 'user-answerable' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P122: Step 2.5 preserves the 4-question-per-call cap" {
-  # AskUserQuestion's documented per-call limit; sequential calls when
-  # the question set exceeds the cap.
-  run grep -F '4 questions per' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P122: Step 2.5 preserves the table-fallback-when-unavailable branch" {
-  # Rule 6 fail-safe: when AskUserQuestion is unavailable (restricted
-  # permission mode, etc.), emit the Outstanding Design Questions table.
-  run grep -F 'Outstanding Design Questions' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P122: Decisions Table row for stop-condition #2 names AskUserQuestion as default" {
-  # The Step 6.5 Decisions Table row at the bottom of the SKILL.md
-  # must agree with the Step 2.5 prose — the prior "table is the
-  # default" wording must be flipped here too.
-  run grep -E '^\| Stop-condition #2 .*AskUserQuestion' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}

package/skills/work-problems/test/work-problems-step-2-5b-cross-halt-routing.bats

@@ -1,149 +0,0 @@
-#!/usr/bin/env bats
-
-# P126: work-problems halt paths must route accumulated user-answerable
-# skips through Step 2.5's surfacing routine before emitting the AFK
-# summary. P122 fixed the routing at Step 2.5 stop-condition #2; P126
-# extends the same contract to the remaining halt paths (Step 0
-# session-continuity, Step 0 fetch failure, Step 6.5 CI-failure, Step
-# 6.5 ADR-042 Rule 5, Step 6.75 dirty-for-unknown-reason).
-#
-# The fix shape: extract Step 2.5's surfacing routine into a reusable
-# named sub-step (Step 2.5b) that every halt path cross-references
-# before emitting its summary. Empty-skip halts skip the round-trip
-# (gating clause: ≥1 user-answerable skip accumulated).
-#
-# Doc-lint contract assertions per ADR-037 Permitted Exception
-# (structural checks on prose contract, not behavioural coverage).
-#
-# @problem P126
-# @jtbd JTBD-001 (Enforce Governance Without Slowing Down)
-# @jtbd JTBD-006 (Progress the Backlog While I'm Away)
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../../.." && pwd)"
-  SKILL_MD="$REPO_ROOT/packages/itil/skills/work-problems/SKILL.md"
-  BRIEFING_MD="$REPO_ROOT/docs/briefing/afk-subprocess.md"
-}
-
-@test "work-problems P126: SKILL.md exists" {
-  [ -f "$SKILL_MD" ]
-}
-
-@test "work-problems P126: SKILL.md names Step 2.5b as the reusable surfacing routine" {
-  # The fix extracts Step 2.5's AskUserQuestion-when-available-else-table
-  # logic into a named reusable sub-step that halt paths can cross-reference
-  # uniformly. The name MUST appear as a heading or anchor so cross-references
-  # resolve to a single source of truth.
-  run grep -F 'Step 2.5b' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P126: Step 2.5b heading exists in SKILL.md" {
-  # Stronger structural check: Step 2.5b must be a markdown heading (### or
-  # ####) so it is a navigable anchor, not just an inline mention.
-  run grep -E '^#{3,4} Step 2\.5b' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P126: Step 2.5b preserves the AskUserQuestion default branch" {
-  # Step 2.5b must inherit P122's interactive-default routing — calling
-  # AskUserQuestion when available is the load-bearing fix.
-  run grep -F 'AskUserQuestion' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P126: Step 2.5b preserves the table fallback for ADR-013 Rule 6" {
-  # Rule 6 fail-safe: when AskUserQuestion is unavailable, emit the
-  # Outstanding Design Questions table.
-  run grep -F 'Outstanding Design Questions' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P126: Step 2.5b is gated on at least one accumulated user-answerable skip" {
-  # The gating clause prevents empty-skip halts from triggering an
-  # unnecessary round-trip. Architect-flagged refinement: the gate clause
-  # must be named in Step 2.5b so future authors don't copy-paste cross-
-  # references and forget the gate.
-  run grep -nE 'at least one accumulated user-answerable skip|≥ ?1 (accumulated )?user-answerable skip|>= ?1 user-answerable skip|one or more user-answerable skip' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P126: Step 0 session-continuity halt cross-references Step 2.5b" {
-  # P109's Step 0 AFK fallback halts with a Prior-Session State report.
-  # When iters have accumulated user-answerable skips before the halt fires
-  # (rare at Step 0 since iters haven't run yet, but the contract must be
-  # uniform), the halt must run Step 2.5b first.
-  run grep -nE 'Step 0.*Step 2\.5b|session-continuity.*Step 2\.5b' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P126: Step 0 fetch failure halt cross-references Step 2.5b" {
-  # Step 0's git fetch network failure halt also emits a final summary
-  # without iter context — the cross-reference is for contract uniformity
-  # even when no skips can accumulate at Step 0.
-  run grep -nE 'fetch.*Step 2\.5b|Network failure.*Step 2\.5b|fetch failure.*Step 2\.5b' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P126: Step 6.5 CI-failure halt cross-references Step 6.5b" {
-  # The Step 6.5 "Failure handling" clause halts on push:watch / release:watch
-  # failure. After N iters this halt path frequently has accumulated user-
-  # answerable skips (the empirically observed P126 surface).
-  run grep -nE 'Failure handling.*Step 2\.5b|CI failure.*Step 2\.5b|release:watch.*Step 2\.5b' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P126: Step 6.5 ADR-042 Rule 5 halt cross-references Step 2.5b with halt-vs-prior-skip guard" {
-  # Architect-flagged refinement: the cross-reference under Rule 5 halt
-  # must explicitly distinguish "Step 2.5b surfaces prior-iter accumulated
-  # skips" from "ADR-042 Rule 5 halt remains the bug signal — the user is
-  # NOT asked how to remediate the above-appetite state". Two grep checks:
-  # cross-reference present AND the guard prose present nearby.
-  run grep -nE 'Rule 5.*Step 2\.5b|halted-above-appetite.*Step 2\.5b' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  # Guard: prior-iter accumulated skips are surfaced; the halt-causing
-  # scorer-gap remains a halt with bug-signal.
-  run grep -nE 'prior[- ]iter accumulated|surfaces? prior[- ]iter|NOT ask the user how to remediate|halt-causing scorer[- ]gap' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P126: Step 6.75 dirty-for-unknown-reason halt cross-references Step 2.5b" {
-  # P036's Step 6.75 inter-iter verification halt emits a final summary
-  # when git status is dirty for unknown reason between iters. After N
-  # iters this halt path is the second most common P126 surface
-  # empirically.
-  run grep -nE 'Dirty for an unknown reason.*Step 2\.5b|dirty[- ]for[- ]unknown[- ]reason.*Step 2\.5b|Step 6\.75.*Step 2\.5b' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P126: Step 2.5 still cross-references Step 2.5b (single source of truth)" {
-  # The original Step 2.5 stop-condition #2 branch must also call into
-  # Step 2.5b — keeping the surfacing logic in one place rather than
-  # duplicated between Step 2.5 and Step 2.5b.
-  run grep -nE 'Step 2\.5 .*Step 2\.5b|Step 2\.5b.*surfacing|Step 2\.5\b.*calls? Step 2\.5b' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P126: Decisions Table row exists for halt-paths-cross-route via Step 2.5b" {
-  # The "Non-Interactive Decision Making" Decisions Table at the bottom
-  # of SKILL.md must carry a row that names the cross-halt routing so the
-  # decision summary is consistent with the Step prose.
-  run grep -nE '\| Halt[- ]path .*Step 2\.5b|halt path.*accumulated user-answerable' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P126: Briefing entry documents the halt-paths-must-route principle" {
-  # The cross-session briefing entry must record the principle alongside
-  # the existing P122 entry so future sessions inherit the reasoning.
-  [ -f "$BRIEFING_MD" ]
-  run grep -nE 'P126|halt[- ]paths[- ]must[- ]route|halt-paths-must-route-design-questions-through-Step-2\.5' "$BRIEFING_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P126: Briefing entry cross-references P122" {
-  # Architect-flagged refinement: the briefing entry must cite both
-  # P122 (parent) and P126 (extension) so the principle's evolution is
-  # traceable.
-  run grep -nE 'P122' "$BRIEFING_MD"
-  [ "$status" -eq 0 ]
-}

package/skills/work-problems/test/work-problems-step-3-5-jtbd-ratification-predicate.bats

@@ -1,191 +0,0 @@
-#!/usr/bin/env bats
-
-# Step 3.5 behavioural fixture per RFC-016 + P344:
-# work-problems orchestrator predicate-checks the cited JTBDs of the
-# selected ticket BEFORE dispatching the iter-worker. The predicate
-# decision lives in `packages/itil/scripts/check-ticket-jtbd-ratification.sh`
-# so the SKILL.md Step 3.5 prose is a thin source-and-call wrapper around
-# a behaviorally-testable shell script (P081 / user feedback: prefer
-# behavioural over structural-grep tests).
-#
-# Polarity (RFC-016 § Scope):
-#   exit 0 = all cited JTBDs ratified (or none cited) → orchestrator proceeds
-#            with the dispatch
-#   exit 1 = ≥1 cited JTBD unratified; one ID per stdout line, `(unresolved)`
-#            tag for the per-JTBD predicate's exit-2 cases → orchestrator
-#            routes to user-answerable skip + queues outstanding_question
-#   exit 2 = ticket file missing / unreadable → halt
-#
-# Cases covered (per RFC-016 § Verification + § Tasks bats coverage):
-#   1. Helper exists at the contracted PATH-shim location.
-#   2. Ticket cites no JTBDs at all → exit 0 (vacuous-pass).
-#   3. Ticket cites a ratified JTBD (frontmatter `human-oversight: confirmed`)
-#      → exit 0.
-#   4. Ticket cites an unratified JTBD → exit 1 + ID on stdout.
-#   5. Ticket cites an unresolved JTBD ID (no matching JTBD file) → exit 1
-#      + `(unresolved)` tag.
-#   6. Ticket file missing → exit 2.
-#   7. Missing per-JTBD predicate shim → silent-pass exit 0 (ADR-031
-#      degenerate adopter case).
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../../.." && pwd)"
-  SHIM="$REPO_ROOT/packages/itil/bin/wr-itil-check-ticket-jtbd-ratification"
-
-  FIXTURE="$(mktemp -d)"
-  mkdir -p "$FIXTURE/docs/jtbd/developer"
-  mkdir -p "$FIXTURE/docs/problems/open"
-
-  # Shim search PATH: put the real per-JTBD predicate shim on PATH so the
-  # helper can dispatch it. The helper resolves shims through PATH per
-  # ADR-049 — no source-relative paths.
-  export PATH="$REPO_ROOT/packages/jtbd/bin:$PATH"
-}
-
-teardown() {
-  rm -rf "$FIXTURE"
-}
-
-@test "shim exists at the contracted PATH location" {
-  [ -f "$SHIM" ]
-  [ -x "$SHIM" ]
-}
-
-@test "case 1: ticket cites no JTBDs → exit 0 (vacuous-pass)" {
-  cat > "$FIXTURE/docs/problems/open/999-no-jtbds.md" <<'EOF'
-# Problem 999: ticket with no JTBD citations
-
-**Status**: Open
-
-## Description
-
-This ticket has no Decision Drivers section and no JTBD citations.
-EOF
-  cd "$FIXTURE"
-  run "$SHIM" "docs/problems/open/999-no-jtbds.md"
-  [ "$status" -eq 0 ]
-  [ -z "$output" ]
-}
-
-@test "case 2: ticket cites a ratified JTBD → exit 0" {
-  cat > "$FIXTURE/docs/jtbd/developer/JTBD-501-ratified.md" <<'EOF'
----
-human-oversight: confirmed
----
-
-# JTBD-501: A ratified job
-EOF
-  cat > "$FIXTURE/docs/problems/open/998-cites-ratified.md" <<'EOF'
-# Problem 998: cites a ratified JTBD
-
-## Decision Drivers
-
-- JTBD-501 — A ratified job
-EOF
-  cd "$FIXTURE"
-  run "$SHIM" "docs/problems/open/998-cites-ratified.md"
-  [ "$status" -eq 0 ]
-  [ -z "$output" ]
-}
-
-@test "case 3: ticket cites an unratified JTBD → exit 1 + ID stdout" {
-  # Unratified = no `human-oversight: confirmed` frontmatter.
-  cat > "$FIXTURE/docs/jtbd/developer/JTBD-502-unratified.md" <<'EOF'
----
-status: proposed
----
-
-# JTBD-502: An unratified job (no human-oversight: confirmed)
-EOF
-  cat > "$FIXTURE/docs/problems/open/997-cites-unratified.md" <<'EOF'
-# Problem 997: cites an unratified JTBD
-
-## Decision Drivers
-
-- JTBD-502 — An unratified job
-EOF
-  cd "$FIXTURE"
-  run "$SHIM" "docs/problems/open/997-cites-unratified.md"
-  [ "$status" -eq 1 ]
-  [[ "$output" == *"JTBD-502"* ]]
-}
-
-@test "case 4: ticket cites an unresolved JTBD ID → exit 1 + (unresolved) tag" {
-  cat > "$FIXTURE/docs/problems/open/996-cites-unresolved.md" <<'EOF'
-# Problem 996: cites an unresolved JTBD ID
-
-## Decision Drivers
-
-- JTBD-999 — Does not exist
-EOF
-  cd "$FIXTURE"
-  run "$SHIM" "docs/problems/open/996-cites-unresolved.md"
-  [ "$status" -eq 1 ]
-  [[ "$output" == *"JTBD-999"* ]]
-  [[ "$output" == *"(unresolved)"* ]]
-}
-
-@test "case 5: ticket file missing → exit 2" {
-  cd "$FIXTURE"
-  run "$SHIM" "docs/problems/open/nonexistent.md"
-  [ "$status" -eq 2 ]
-}
-
-@test "case 6: missing per-JTBD predicate shim → silent-pass exit 0" {
-  # Strip ALL per-JTBD predicate shim locations from PATH (source-repo
-  # packages/jtbd/bin AND the global plugin install cache); degenerate
-  # adopter case (ADR-031). The helper should silent-pass (exit 0) rather
-  # than fail.
-  cat > "$FIXTURE/docs/problems/open/995-cites-unratified.md" <<'EOF'
-# Problem 995: cites an unratified JTBD but shim is missing
-
-## Decision Drivers
-
-- JTBD-502 — An unratified job
-EOF
-  cd "$FIXTURE"
-  # Strip every PATH entry that exposes wr-jtbd-is-job-or-persona-unconfirmed.
-  # Both packages/jtbd/bin (source-repo) and ~/.claude/plugins/cache/.../bin
-  # (plugin install) must go.
-  STRIPPED_PATH=""
-  while IFS= read -r entry; do
-    [ -z "$entry" ] && continue
-    if [ -x "$entry/wr-jtbd-is-job-or-persona-unconfirmed" ]; then
-      continue
-    fi
-    STRIPPED_PATH="${STRIPPED_PATH}${entry}:"
-  done < <(printf '%s' "$PATH" | tr ':' '\n')
-  PATH="${STRIPPED_PATH%:}" run "$SHIM" "docs/problems/open/995-cites-unratified.md"
-  [ "$status" -eq 0 ]
-}
-
-@test "case 7: ticket cites multiple JTBDs, one unratified → exit 1 + unratified ID only" {
-  cat > "$FIXTURE/docs/jtbd/developer/JTBD-503-ratified.md" <<'EOF'
----
-human-oversight: confirmed
----
-
-# JTBD-503: A ratified job
-EOF
-  cat > "$FIXTURE/docs/jtbd/developer/JTBD-504-unratified.md" <<'EOF'
----
-status: proposed
----
-
-# JTBD-504: An unratified job
-EOF
-  cat > "$FIXTURE/docs/problems/open/994-cites-mixed.md" <<'EOF'
-# Problem 994: cites both ratified and unratified JTBDs
-
-## Decision Drivers
-
-- JTBD-503 — A ratified job
-- JTBD-504 — An unratified job
-EOF
-  cd "$FIXTURE"
-  run "$SHIM" "docs/problems/open/994-cites-mixed.md"
-  [ "$status" -eq 1 ]
-  [[ "$output" == *"JTBD-504"* ]]
-  # Ratified IDs are NOT echoed — only the unratified set.
-  [[ "$output" != *"JTBD-503"* ]]
-}

package/skills/work-problems/test/work-problems-step-5-bats-polling-discipline.bats

@@ -1,91 +0,0 @@
-#!/usr/bin/env bats
-# Doc-lint guard: work-problems SKILL.md Step 5 iteration prompt body must
-# carry a prompt-discipline rule forbidding the bash-until-loop bats-output-
-# polling antipattern that caused P146.
-#
-# Antipattern (observed 2026-04-29, iter 1, PID 23580 child PID 16408 —
-# 68m34s wall-clock burn + lost JSON metadata):
-#   until [ -f $TASK_OUTPUT ] && grep -qE '^[0-9]+ tests?,' $TASK_OUTPUT \
-#     2>/dev/null; do sleep 5; done; tail -25 $TASK_OUTPUT
-# The regex `^[0-9]+ tests?,` matches bats's *default* (non-TAP) console-
-# summary line. `bats --tap` never emits that line, so the until-loop spins
-# forever after bats completes. Silent deadlock; no error; no exit.
-#
-# The polling idiom is NOT taught by any SKILL.md (grep across the repo
-# returns only this ticket itself). It is agent-learned from training data —
-# a generic bash + bats Bourne idiom. The fix per the ticket strategy is a
-# prompt-discipline rule + behavioural assertion: the iteration prompt body
-# must explicitly forbid the antipattern AND name a safe substitute, so the
-# iteration agent does not fall back on the learned idiom.
-#
-# Structural assertion — Permitted Exception under ADR-037 (SKILL.md is
-# explicitly a contract document; doc-lint contract assertion against the
-# contract document itself is the named permitted pattern; same rationale
-# as work-problems-step-5-delegation.bats's @problem P083 / P089 fixtures).
-# Behavioural alternative would require spawning a real `claude -p`
-# subprocess and observing its tool-call traces; that harness sits outside
-# the skill layer.
-#
-# @problem P146
-# @jtbd JTBD-006
-# @jtbd JTBD-001
-# @jtbd JTBD-101
-#
-# Cross-reference:
-#   P146 (bash until-loop polls bats output with bats-console regex against
-#     TAP-format) — driver for this clause
-#   P147 (sibling — SIGTERM-flush metadata-loss caveat fires when an iter
-#     deadlocks before ITERATION_SUMMARY emission; P146's regex-poll is the
-#     concrete mechanism behind today's stuck-before-emit incident)
-#   ADR-037 (skill testing strategy — contract-assertion Permitted Exception)
-#   ADR-014 (single-commit grain — fix + bats + ticket transition land
-#     together)
-
-setup() {
-  SKILL_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  SKILL_FILE="${SKILL_DIR}/SKILL.md"
-}
-
-@test "SKILL.md Step 5 iteration prompt forbids bats-console-summary regex polling (P146)" {
-  # The iteration prompt body MUST explicitly name the antipattern so the
-  # iteration agent does not reach for the learned bash idiom. The
-  # prohibition phrase must reference the regex-poll-on-bats-output shape
-  # OR the bats-console-summary line shape, since both are the failure
-  # surface today.
-  run grep -niE "do not poll.{0,40}bats|bats.{0,40}console.summary.{0,40}regex|never poll.{0,40}bats|grep.{0,20}tests\\?,.{0,80}(forbidden|antipattern|do not|never)" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 5 iteration prompt names a safe substitute for backgrounded bats waits (P146)" {
-  # Forbidding the antipattern without naming a substitute leaves the agent
-  # without a path forward, and the agent will silently fall back on the
-  # antipattern. The prompt must name `wait \$bg_pid` (Unix idiom) OR the
-  # Bash tool's `run_in_background=true` + BashOutput check (Claude Code
-  # idiom) as the recommended replacement.
-  run grep -niE "wait[[:space:]]+[\"']?\\\$[A-Z_]*pid|wait[[:space:]]+[\"']?\\\$[A-Z_]*PID|run_in_background.{0,80}BashOutput|BashOutput.{0,80}run_in_background|background.{0,40}wait.{0,40}exit" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 5 bats-polling clause cites P146 (self-documenting contract)" {
-  # The clause must cite P146 inline so a future contributor reading the
-  # rule understands why it exists before deleting it (mirrors the P083
-  # ScheduleWakeup-clause discipline already in place at line ~301).
-  run grep -nE "P146" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 5 bats-polling clause names the TAP/console-summary divergence (P146)" {
-  # The clause must explain WHY the regex `^[0-9]+ tests?,` fails — the
-  # divergence between bats's default (console-summary) and `--tap` output
-  # formats. Without this, a future contributor who switches from `--tap`
-  # to default formatting may "fix" the rule incorrectly.
-  run grep -niE "TAP.{0,80}console.summary|console.summary.{0,80}TAP|--tap.{0,80}(does not|never|no console summary)|tap.format.{0,80}(does not|never).{0,40}emit" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Related section cites P146" {
-  # Self-documenting contract: ticket cited in Related so a future reader
-  # of the file finds the driving incident without scrolling to find it.
-  run grep -nE "P146" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}