@windyroad/itil 0.61.1 → 0.61.2

package/skills/work-problems/test/work-problems-p342-retro-auto-ticket-carveout.bats

@@ -1,120 +0,0 @@
-#!/usr/bin/env bats
-
-# P342: Iter retros queue their own observations as outstanding-questions
-# for user-direction triage instead of auto-ticketing — same trust-
-# boundary as /wr-retrospective:run-retro Step 4a (verification close-on-
-# evidence).
-#
-# The fix shape amends the work-problems Step 5 iter-prompt body:
-#   - Relax the "no capture-* siblings mid-loop" rule for RETRO-surfaced
-#     observations specifically.
-#   - Direct retro to auto-ticket recurring class-of-behaviour
-#     observations via /wr-itil:capture-problem (mechanical-stage carve-
-#     out per run-retro Step 4a precedent).
-#   - Route ONLY direction-setting observations (genuine user-judgment-
-#     bound questions) to outstanding_questions.
-#   - Document the classification:
-#       recurring class-of-behaviour / SKILL-contract drift / hook
-#       misbehaviour / framework-gap → auto-ticket.
-#       Direction-setting (design choice, deviation-approval, framework
-#       boundary) → outstanding_questions.
-#       Ambiguous → default to auto-ticket.
-#
-# Doc-lint contract assertions per ADR-037 Permitted Exception.
-#
-# @problem P342
-# @adr ADR-044 (Decision-Delegation Contract — mechanical-stage carve-out per Step 4a precedent)
-# @adr ADR-013 Rule 5 (policy-authorised silent proceed)
-# @adr ADR-032 (governance skill invocation patterns — foreground-spawns-background fanout pattern for capture-*)
-# @jtbd JTBD-006 (Progress the Backlog While I'm Away — durable WSJF-ranked backlog accumulation)
-# @jtbd JTBD-201 (audit-trail — auto-ticketed observations become durable artefacts)
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../../.." && pwd)"
-  SKILL_MD="$REPO_ROOT/packages/itil/skills/work-problems/SKILL.md"
-}
-
-@test "work-problems P342: SKILL.md exists" {
-  [ -f "$SKILL_MD" ]
-}
-
-# ── Step 5 iter-prompt: capture-* carve-out for retro observations ──────────
-
-@test "work-problems P342: iter-prompt body carves out capture-* for retro-surfaced observations" {
-  # The fix relaxes the "No capture-* siblings mid-loop" rule for RETRO
-  # observations specifically. Existing P078-class spam rule remains for
-  # non-retro mid-iter capture; the carve-out is bounded to retro.
-  run grep -nE 'retro.*capture-\*|capture-\*.*retro|retro-surfaced.*capture|capture-problem.*retro' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P342: iter-prompt body directs retro to auto-ticket via /wr-itil:capture-problem" {
-  # Recurring class-of-behaviour observations MUST route to
-  # /wr-itil:capture-problem (mechanical-stage carve-out per run-retro
-  # Step 4a precedent). The skill is named explicitly so adopters know
-  # which capture sibling to invoke.
-  run grep -nE '/wr-itil:capture-problem' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P342: iter-prompt cites Step 4a precedent for mechanical-stage carve-out" {
-  # The carve-out's authority is the run-retro Step 4a verification
-  # close-on-evidence precedent. Cite it so future authors don't unwind
-  # the carve-out by re-applying the broad "no capture-* mid-loop" rule
-  # uniformly.
-  run grep -nE 'Step 4a precedent|run-retro Step 4a|Step 4a.*mechanical' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-# ── Classification taxonomy ────────────────────────────────────────────────
-
-@test "work-problems P342: iter-prompt classifies recurring class-of-behaviour as auto-ticket" {
-  # Recurring class-of-behaviour / SKILL-contract drift / hook
-  # misbehaviour / framework-gap → auto-ticket. The taxonomy MUST be
-  # documented so future authors don't drift on classification.
-  run grep -nE 'recurring class-of-behaviour|class-of-behaviour observation' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P342: iter-prompt classifies direction-setting as outstanding_questions" {
-  # Direction-setting (design choice, deviation-approval, framework
-  # boundary) → outstanding_questions. The route must be named so the
-  # framework boundary is preserved.
-  run grep -nE 'Direction-setting observation|direction-setting.*outstanding_questions' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P342: iter-prompt classifies ambiguous observations as default-to-auto-ticket" {
-  # Ambiguous → default to auto-ticket. This is the trust-boundary
-  # asymmetry that prevents observations from silently piling in the
-  # queue file (per P342 Description).
-  run grep -nE 'Ambiguous.*auto-ticket|default to auto-ticket|ambiguous.*default.*ticket' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-# ── Trust-boundary mirror in run-retro Step 4b ─────────────────────────────
-
-@test "work-problems P342: iter-prompt cross-references run-retro Step 4b carve-out symmetry" {
-  # The run-retro Step 4b stage classification mirrors this carve-out so
-  # the same trust-boundary applies whether retro fires in iter context
-  # OR standalone in main turn. Cross-reference the sibling locus so the
-  # symmetry is discoverable.
-  run grep -nE 'run-retro.*Step 4b|Step 4b.*mirror|symmetry.*Step 4b|Step 4b.*carve-out' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-# ── Cross-reference to P342 ────────────────────────────────────────────────
-
-@test "work-problems P342: Related section cites P342 as the originating ticket" {
-  run grep -nE '\*\*P342\*\*|P342\b' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-# ── Anti-pattern preservation: P130 mid-loop ask discipline unaffected ─────
-
-@test "work-problems P342: iter-prompt preserves P130 NEVER call AskUserQuestion mid-loop discipline" {
-  # The P342 carve-out is for capture-* siblings on the retro path only;
-  # the iter-prompt's mid-loop AskUserQuestion ban is unchanged.
-  run grep -nE 'NEVER call .?AskUserQuestion.? mid-loop|MUST NOT call .?AskUserQuestion.? between iter' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}

package/skills/work-problems/test/work-problems-preflight-failure-handling.bats

@@ -1,248 +0,0 @@
-#!/usr/bin/env bats
-# tdd-review: structural-permitted (justification: the doc-lint slice below
-# asserts SKILL.md / ADR-032 prose contract — SKILL.md is the contract
-# document per ADR-037 Permitted Exception; these guards catch prose drift
-# away from the behavioural revert-and-proceed contract exercised above. The
-# load-bearing core of this fixture is behavioural per ADR-052. harness-gap P012)
-#
-# Behavioural test: work-problems Step 0 PRE-FLIGHT subprocess failure handling
-# (P358). Step 0b / 0c / 0d dispatch a /wr-itil:review-problems (or
-# check-upstream-responses) pre-flight subprocess "same shape as Step 5". But a
-# pre-flight is a NON-load-bearing cache-refresh dependency, NOT an iter (the
-# loop body). When a pre-flight subprocess exits non-zero OR returns
-# `is_error: true` (e.g. `API Error: The socket connection was closed
-# unexpectedly`), the orchestrator's contract is NON-BLOCKING:
-#   (1) revert any dirty (unstaged) partial cache/audit/README write the
-#       pre-flight left (`git checkout -- docs/problems/ docs/audits/`);
-#   (1b) if a dead pre-flight left STAGED residue, `git reset` then revert
-#        (ADR-009 no-trust-window-extension — a dead is_error:true subprocess
-#        must not seed the parent's commit);
-#   (2) log a one-line annotation;
-#   (3) proceed to Step 1 with the existing README.
-# This is ORTHOGONAL to the Step 5 iter SALVAGE-vs-HALT axis (P261 / P214):
-# that axis classifies an *iter's* is_error:true; this fixture classifies the
-# *pre-flight role* — which NEVER salvages and NEVER halts the loop.
-#
-# The fake `claude` shim below re-creates the production shape: it dirties an
-# unstaged cache file in the repo, then emits an is_error:true socket-closed
-# JSON envelope (no commit). The harness re-implements the orchestrator's
-# pre-flight failure contract (faithful to SKILL.md § "Step 0 pre-flight
-# subprocess failure handling (P358)") and asserts the revert-and-proceed
-# outcome across the input shapes. Adopters who copy the SKILL.md pre-flight
-# block into their orchestrator should observe the same outcomes.
-#
-# @problem P358
-# @rfc RFC-024
-# @jtbd JTBD-006
-#
-# Cross-reference:
-#   P358 (claude -p subprocess dispatch socket-closed; pre-flight is_error
-#     handling gap) — driver ticket
-#   RFC-024 (work-problems pre-flight subprocess failure handling) — fix vehicle
-#   ADR-032 (governance skill invocation patterns — § "Pre-flight subprocess
-#     failure handling — non-blocking revert-and-proceed (P358 amendment)") —
-#     the iter-vs-pre-flight failure-semantics distinction this fixture pins
-#   ADR-009 (gate-marker / no-trust-window-extension — staged residue git reset)
-#   ADR-019 (preflight clean-tree reconciliation surface)
-#   P261 / P214 (the iter is_error:true SALVAGE/HALT axis this is orthogonal to)
-#   ADR-037 / ADR-052 (skill testing strategy — behavioural default; doc-lint
-#     contract assertion is the Permitted Exception, marked above)
-
-setup() {
-  TEST_TMP="$(mktemp -d)"
-  FAKE_BIN="${TEST_TMP}/bin"
-  mkdir -p "$FAKE_BIN"
-
-  # Fake `claude` binary simulating a pre-flight subprocess of the socket-closed
-  # class: it leaves a dirty (UNSTAGED by default) partial cache write in the
-  # CWD git repo, then emits an is_error:true JSON envelope carrying the
-  # socket-closed error string in `.result`. No commit. This matches the
-  # 2026-06-10 P358 shape: a /wr-itil:review-problems pre-flight that partially
-  # refreshed docs/problems/.upstream-cache.json then died with
-  # `API Error: The socket connection was closed unexpectedly`.
-  cat > "$FAKE_BIN/claude" <<'FAKE_EOF'
-#!/usr/bin/env bash
-# Test fake for work-problems Step 0 pre-flight failure-handling fixture.
-# Dirties a partial cache write, then emits is_error:true socket-closed JSON.
-mkdir -p docs/problems
-printf 'partial-refresh-mid-die\n' >> docs/problems/.upstream-cache.json
-if [ "${FAKE_STAGE_RESIDUE:-0}" = "1" ]; then
-  git add docs/problems/.upstream-cache.json 2>/dev/null || true
-fi
-if [ "${FAKE_EXIT:-0}" != "0" ]; then
-  printf '%s\n' '{"is_error":true,"result":"subprocess crashed"}'
-  exit "${FAKE_EXIT}"
-fi
-printf '%s\n' '{"is_error":true,"result":"API Error: The socket connection was closed unexpectedly. For more information, pass `verbose: true`","total_cost_usd":0.0,"duration_ms":727000}'
-FAKE_EOF
-  chmod +x "$FAKE_BIN/claude"
-  export PATH="$FAKE_BIN:$PATH"
-
-  # A throwaway git repo so dirty-detection + the revert are real. Seed a
-  # committed clean cache so the partial write is observably a dirty delta.
-  REPO="${TEST_TMP}/repo"
-  mkdir -p "$REPO/docs/problems"
-  git -C "$REPO" init -q
-  git -C "$REPO" config user.email "test@example.com"
-  git -C "$REPO" config user.name "Test"
-  printf 'clean-cache\n' > "$REPO/docs/problems/.upstream-cache.json"
-  git -C "$REPO" add docs/problems/.upstream-cache.json
-  git -C "$REPO" commit -q -m "root: clean cache"
-
-  SKILL_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  SKILL_FILE="${SKILL_DIR}/SKILL.md"
-  ADR_FILE="$(cd "${SKILL_DIR}/../../../.." && pwd)/docs/decisions/032-governance-skill-invocation-patterns.proposed.md"
-}
-
-teardown() {
-  if [ -n "${TEST_TMP:-}" ] && [ -d "$TEST_TMP" ]; then
-    rm -rf "$TEST_TMP"
-  fi
-}
-
-# Faithful re-implementation of SKILL.md § "Step 0 pre-flight subprocess failure
-# handling (P358)". Consumes the pre-flight JSON envelope + the pre-flight exit
-# code + the repo working-tree state, applies the non-blocking revert-and-proceed
-# contract, and returns the orchestrator's decision. KEY PROPERTY: a pre-flight
-# NEVER halts the loop and NEVER salvages — it reverts and proceeds.
-preflight_failure_outcome() {
-  local json="$1"
-  local exit_code="$2"
-  local repo="$3"
-
-  local is_error
-  is_error=$(printf '%s' "$json" | python3 -c 'import json,sys; print(str(json.load(sys.stdin).get("is_error")).lower())' 2>/dev/null || echo unknown)
-
-  # Pre-flight succeeded (exit 0 AND is_error:false) → cache refreshed; proceed.
-  if [ "$exit_code" = "0" ] && [ "$is_error" = "false" ]; then
-    printf 'DECISION=PROCEED reason=preflight-ok\n'
-    return 0
-  fi
-
-  # Pre-flight FAILED (non-zero exit OR is_error:true) → NON-BLOCKING contract.
-  # Step 1b: if the dead pre-flight left STAGED residue, unstage it first
-  # (ADR-009 — a dead is_error:true subprocess must not seed the parent index).
-  if [ -n "$(git -C "$repo" diff --cached --name-only)" ]; then
-    git -C "$repo" reset -q
-  fi
-  # Step 1: revert the whole contractually-touchable path set (not just cache).
-  # Per-path tolerant — a COMBINED `git checkout -- A B` errors and reverts
-  # NOTHING when B is absent (e.g. docs/audits/ on a fresh adopter repo), so
-  # revert each path independently.
-  git -C "$repo" checkout -- docs/problems/ 2>/dev/null || true
-  git -C "$repo" checkout -- docs/audits/ 2>/dev/null || true
-  # Step 3: proceed to Step 1 (never halt).
-  printf 'DECISION=PROCEED reason=preflight-failed-reverted\n'
-  return 0
-}
-
-# ---------------------------------------------------------------------------
-# Behavioural cases (the load-bearing core per ADR-052).
-# ---------------------------------------------------------------------------
-
-@test "P358: pre-flight is_error:true (socket-closed) + unstaged partial write -> revert + PROCEED (never halt)" {
-  export FAKE_STAGE_RESIDUE=0 FAKE_EXIT=0
-  local json
-  json=$( cd "$REPO" && claude -p --output-format json "PREFLIGHT" < /dev/null )
-  # The fake left a dirty partial cache write.
-  run git -C "$REPO" status --porcelain docs/problems/.upstream-cache.json
-  [ -n "$output" ]
-  run preflight_failure_outcome "$json" 0 "$REPO"
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"DECISION=PROCEED"* ]]
-  [[ "$output" == *"preflight-failed-reverted"* ]]
-  # The dirty partial write was reverted — tree is clean again.
-  run git -C "$REPO" status --porcelain
-  [ -z "$output" ]
-  # The committed clean cache content survived (revert restored it).
-  run cat "$REPO/docs/problems/.upstream-cache.json"
-  [[ "$output" == "clean-cache" ]]
-}
-
-@test "P358: pre-flight non-zero exit -> revert + PROCEED (non-blocking; not a loop halt)" {
-  export FAKE_STAGE_RESIDUE=0 FAKE_EXIT=1
-  local json
-  json=$( cd "$REPO" && claude -p --output-format json "PREFLIGHT" < /dev/null || true )
-  run preflight_failure_outcome "$json" 1 "$REPO"
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"DECISION=PROCEED"* ]]
-  # Crucially NOT a HALT — a pre-flight failure does not stop the loop.
-  [[ "$output" != *"HALT"* ]]
-  run git -C "$REPO" status --porcelain
-  [ -z "$output" ]
-}
-
-@test "P358: pre-flight left STAGED residue -> git reset then revert (ADR-009 no-seed-parent-index)" {
-  export FAKE_STAGE_RESIDUE=1 FAKE_EXIT=0
-  local json
-  json=$( cd "$REPO" && claude -p --output-format json "PREFLIGHT" < /dev/null )
-  # The fake STAGED the partial write.
-  run git -C "$REPO" diff --cached --name-only
-  [[ "$output" == *".upstream-cache.json"* ]]
-  run preflight_failure_outcome "$json" 0 "$REPO"
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"DECISION=PROCEED"* ]]
-  # Staged residue was unstaged AND reverted — index + tree both clean.
-  run git -C "$REPO" diff --cached --name-only
-  [ -z "$output" ]
-  run git -C "$REPO" status --porcelain
-  [ -z "$output" ]
-}
-
-@test "P358: pre-flight success (exit 0 + is_error:false) -> PROCEED without revert" {
-  # Hand-build a clean success envelope (the fake always fails by design).
-  local json='{"is_error":false,"result":"refreshed"}'
-  run preflight_failure_outcome "$json" 0 "$REPO"
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"DECISION=PROCEED"* ]]
-  [[ "$output" == *"preflight-ok"* ]]
-}
-
-# ---------------------------------------------------------------------------
-# Doc-lint contract assertions (Permitted Exception per ADR-037; structural
-# slice marked at top of file per ADR-052 Surface 2). These guard the SKILL.md
-# / ADR-032 prose against drift away from the behavioural contract above.
-# ---------------------------------------------------------------------------
-
-@test "P358: SKILL.md documents the Step 0 pre-flight failure-handling subsection" {
-  run grep -niE "pre-flight subprocess failure handling.{0,40}P358|Step 0 pre-flight subprocess failure" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "P358: SKILL.md pre-flight contract names the non-blocking revert-and-proceed rule" {
-  run grep -niE "non-?blocking" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  run grep -niE "revert.{0,40}(proceed|partial)|proceed to Step 1 with the existing" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "P358: SKILL.md distinguishes pre-flight (cache-refresh dependency) from iter (loop body)" {
-  run grep -niE "non-load-bearing cache-refresh dependency" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  run grep -niE "iter IS the loop body|the iter is the loop body" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "P358: SKILL.md pre-flight forward-pointer fires in the 0b/0c/0d dispatch paragraphs" {
-  # At least three "do NOT halt the loop (a failed pre-flight ...)" pointers.
-  run bash -c "grep -ciE 'do NOT halt the loop .a failed pre-flight' '$SKILL_FILE'"
-  [ "$status" -eq 0 ]
-  [ "$output" -ge 3 ]
-}
-
-@test "P358: SKILL.md pre-flight contract cites P358" {
-  run grep -nE "P358" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "P358: ADR-032 carries the pre-flight failure-handling P358 amendment" {
-  run grep -niE "Pre-flight subprocess failure handling.{0,60}P358 amendment" "$ADR_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "P358: ADR-032 amendment names the iter-vs-pre-flight failure-semantics axis as orthogonal to SALVAGE/HALT" {
-  run grep -niE "orthogonal" "$ADR_FILE"
-  [ "$status" -eq 0 ]
-  run grep -niE "No SALVAGE for a pre-flight|SALVAGE branch does NOT apply to a pre-flight" "$ADR_FILE"
-  [ "$status" -eq 0 ]
-}

package/skills/work-problems/test/work-problems-preflight-session-continuity.bats

@@ -1,139 +0,0 @@
-#!/usr/bin/env bats
-# Contract-assertion bats for work-problems Step 0 session-continuity
-# detection (the extension per P109).
-#
-# Per ADR-037 SKILL.md is a contract document; these assertions check the
-# contract strings the skill prose authoritatively pins for the Step 0
-# session-continuity detection pass. Follows the split pattern established
-# by work-problems-preflight.bats (fetch/divergence assertions) — this file
-# covers the second invariant family: prior-session partial-work signals +
-# interactive/AFK routing.
-#
-# Cross-reference:
-#   @problem P109 (work-problems preflight does not detect prior-session partial-work state)
-#   ADR-019 (AFK orchestrator preflight — extension scope)
-#   ADR-013 (structured user interaction — Rule 1 interactive, Rule 6 non-interactive fail-safe)
-#   ADR-032 (governance skill invocation patterns — .afk-run-state/iter-*.json contract)
-#   ADR-037 (skill testing strategy — contract-assertion framing)
-#   @jtbd JTBD-006 (Progress the Backlog While I'm Away — session-continuity detection belongs in Step 0)
-
-setup() {
-  SKILL_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  SKILL_FILE="${SKILL_DIR}/SKILL.md"
-}
-
-@test "SKILL.md exists" {
-  [ -f "$SKILL_FILE" ]
-}
-
-@test "SKILL.md Step 0 cites P109 (session-continuity driver)" {
-  # Contract criterion: the extension is traceable to its driver ticket.
-  run grep -n "P109" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 0 names the session-continuity detection pass" {
-  # Contract criterion: the new detection pass is named as a discrete
-  # concept in the Step 0 prose (not buried under the divergence check).
-  run grep -niE "session.continuity" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 0 enumerates untracked docs/decisions/*.proposed.md signal" {
-  # Contract criterion: drafted-but-unlanded ADRs are one of the signals
-  # the session-continuity detection pass MUST enumerate.
-  run grep -nE "docs/decisions/\*\.proposed\.md|docs/decisions/.*\.proposed\.md" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 0 enumerates untracked docs/problems/*.md signal" {
-  # Contract criterion: drafted-but-unlanded problem tickets are enumerated
-  # as one of the session-continuity signals. The preflight section already
-  # references docs/problems/ for the scan surface; the test checks that
-  # the Preflight section names untracked problem files as a detection
-  # signal (not merely the backlog-scan surface).
-  run grep -niE "untracked.*docs/problems|docs/problems/.*untracked" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 0 enumerates .afk-run-state/iter-*.json error signal" {
-  # Contract criterion: the .afk-run-state/iter-*.json subprocess artefacts
-  # (per ADR-032) with is_error: true or api_error_status >= 400 are named
-  # as a signal.
-  run grep -nE "\.afk-run-state/iter-\*\.json|\.afk-run-state/iter-.*\.json" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 0 names the is_error / api_error_status fields" {
-  # Contract criterion: the specific JSON fields the detection pass reads
-  # are named verbatim so the contract is unambiguous.
-  run grep -niE "is_error.*true|api_error_status" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 0 enumerates stale .claude/worktrees signal" {
-  # Contract criterion: stale subagent worktrees are a detection signal.
-  # Detection only (not cleanup/mutation — per P109 scope boundary).
-  run grep -nE "\.claude/worktrees" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 0 enumerates git worktree list signal for claude/* branches" {
-  # Contract criterion: git worktree list is the detection mechanism for
-  # claude/* branches adjacent to the .claude/worktrees/ dir check.
-  run grep -niE "git worktree list" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 0 enumerates uncommitted SKILL.md / source / ADR edits signal" {
-  # Contract criterion: mid-authoring source edits are a detection signal.
-  run grep -niE "uncommitted.*(SKILL\.md|source|ADR)|(SKILL\.md|source|ADR).*uncommitted" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 0 routes interactive via AskUserQuestion" {
-  # Contract criterion per ADR-013 Rule 1: the interactive branch uses
-  # AskUserQuestion.
-  run grep -nE "AskUserQuestion" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 0 interactive branch names the 4 option categories" {
-  # Contract criterion: the AskUserQuestion 4-option shape is pinned so
-  # adopters know the branch set. Resume / discard / leave-and-lower-priority / halt.
-  run grep -niE "resume" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  run grep -niE "discard" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  run grep -niE "leave.*lower.priority|leave.and.lower.priority" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 0 routes AFK via halt-with-report per ADR-013 Rule 6" {
-  # Contract criterion per ADR-013 Rule 6: the non-interactive / AFK branch
-  # halts with a report rather than silently choosing.
-  run grep -niE "halt.with.report|halt with report|Rule 6 fail.safe" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 0 cites ADR-013 Rule 6 for AFK fail-safe" {
-  # Contract criterion: ADR-013 Rule 6 is named as the authority for the
-  # non-interactive halt branch.
-  run grep -nE "ADR-013.*Rule 6|Rule 6.*ADR-013" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 0 emits a structured Prior-Session State report" {
-  # Contract criterion: the AFK halt branch surfaces a structured report,
-  # not a free-text prose blurb — so the user can act on it on return.
-  run grep -niE "Prior.Session State" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Non-Interactive Decision Making table covers session-continuity" {
-  # Contract criterion: the decision-matrix section surfaces the new branch
-  # so adopters of the skill can find the AFK default without reading
-  # Step 0 prose in full.
-  run grep -niE "Prior.session partial.work|session.continuity.*dirty|Prior-Session State.*AFK" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}

package/skills/work-problems/test/work-problems-preflight.bats

@@ -1,69 +0,0 @@
-#!/usr/bin/env bats
-# Doc-lint guard: work-problems SKILL.md must include the preflight
-# (fetch-origin + divergence handling) per ADR-019.
-#
-# Structural assertion — Permitted Exception to the source-grep ban (ADR-005 / P011).
-# These tests assert that the skill specification document contains the
-# preflight step so the AFK orchestrator does not iterate against a stale
-# local backlog when origin/<base> has advanced.
-#
-# Cross-reference:
-#   P040 (work-problems does not fetch origin before starting)
-#   ADR-019 (AFK orchestrator preflight)
-#   @jtbd JTBD-006 (Progress the Backlog While I'm Away)
-
-setup() {
-  SKILL_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  SKILL_FILE="${SKILL_DIR}/SKILL.md"
-}
-
-@test "SKILL.md cites ADR-019 (preflight)" {
-  # ADR-019 confirmation criterion: skill must reference the ADR.
-  run grep -n "ADR-019" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md preflight runs git fetch origin" {
-  # ADR-019 mechanism: must run `git fetch origin` before opening the
-  # work loop.
-  run grep -n "git fetch origin" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md preflight uses git pull --ff-only on trivial divergence" {
-  # ADR-019 mechanism: trivial fast-forward is policy-authorised; non-ff is
-  # not. The skill must cite --ff-only to prevent merge attempts.
-  run grep -n "ff-only" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md preflight stops the loop on non-fast-forward divergence" {
-  # Confirmation: divergence handling must include a stop-with-report path,
-  # not a silent retry or merge.
-  run grep -niE "non-fast-forward|non-ff" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md preflight has a discrete step before backlog scan" {
-  # The preflight should appear as a discrete step (e.g. Step 0 — Preflight)
-  # so it is not buried in prose.
-  run grep -niE "step 0|preflight" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Non-Interactive Decision Making table covers origin divergence" {
-  # ADR-019 confirmation: the non-interactive defaults table must include a
-  # row for the divergence decision.
-  run grep -niE "Origin diverged|origin.*divergence|fast-forward" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md explicitly forbids non-interactive merge/rebase remediation" {
-  # ADR-019 forbids non-interactive rebase/merge attempts. The skill must
-  # negate these (e.g. "Do NOT attempt to rebase or merge" or "are NOT
-  # policy-authorised") rather than silently allow them. We assert the
-  # negation appears, not that the words are absent — discussing forbidden
-  # operations in the negative is exactly what the ADR requires.
-  run grep -niE "Do NOT attempt to rebase or merge|NOT policy-authorised" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}

package/skills/work-problems/test/work-problems-release-cadence.bats

@@ -1,63 +0,0 @@
-#!/usr/bin/env bats
-# Doc-lint guard: work-problems SKILL.md must include the inter-iteration
-# release-cadence check per ADR-018.
-#
-# Structural assertion — Permitted Exception to the source-grep ban (ADR-005 / P011).
-# These tests assert that the skill specification document contains the
-# release-cadence step so the AFK orchestrator does not silently accumulate
-# unreleased changesets across iterations.
-#
-# Cross-reference:
-#   P041 (work-problems does not enforce release cadence)
-#   ADR-018 (inter-iteration release cadence for AFK loops)
-#   @jtbd JTBD-006 (Progress the Backlog While I'm Away)
-
-setup() {
-  SKILL_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  SKILL_FILE="${SKILL_DIR}/SKILL.md"
-}
-
-@test "SKILL.md exists" {
-  [ -f "$SKILL_FILE" ]
-}
-
-@test "SKILL.md cites ADR-018 (release cadence)" {
-  # ADR-018 confirmation criterion: skill must reference the ADR.
-  run grep -n "ADR-018" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md delegates to wr-risk-scorer:assess-release (preserves pure-scorer contract from ADR-015)" {
-  # ADR-018 mechanism: must delegate to the assess-release skill rather than
-  # re-implementing risk scoring inline.
-  run grep -n "assess-release" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md references release:watch as the drain mechanism" {
-  # ADR-018 mechanism: drain action runs npm run release:watch (after
-  # push:watch) when the queue hits appetite.
-  run grep -n "release:watch" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md references push:watch as part of the drain mechanism" {
-  # ADR-018 mechanism: drain action runs push:watch before release:watch.
-  run grep -n "push:watch" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md has a release-cadence step between iterations" {
-  # The inter-iteration check should appear as a discrete step or subsection
-  # so it is not buried in prose.
-  run grep -niE "release.cadence|cadence check|release queue" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Non-Interactive Decision Making table covers pipeline risk at appetite" {
-  # ADR-018 confirmation criterion: the non-interactive defaults table must
-  # include a row for the release-drain decision, otherwise an AFK reader
-  # cannot find the rule.
-  run grep -niE "Pipeline risk at appetite|release queue.*appetite|drain.*release" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}