@windyroad/itil 0.61.1 → 0.61.2

package/skills/work-problems/test/work-problems-inter-iteration-verify.bats

@@ -1,59 +0,0 @@
-#!/usr/bin/env bats
-# Doc-lint guard: work-problems SKILL.md must include an inter-iteration
-# verification check (P036). After each subagent returns, the orchestrator
-# must verify via `git status --porcelain` that the tree is clean (or
-# dirty for a known deliberate reason) before spawning the next iteration.
-#
-# Structural assertion — Permitted Exception to the source-grep ban
-# (ADR-005 / P011). Asserts the skill specification document contains the
-# inter-iteration check so a silent subagent commit failure cannot
-# compound across iterations.
-#
-# Cross-reference:
-#   P036 (work-problems orchestrator does not verify commit-landing between iterations)
-#   P035 (commit-gate fallback, the primary mitigation this backstops)
-#   @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 P036 (inter-iteration verification)" {
-  run grep -n "P036" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md has an inter-iteration verification step (after commit, before next iteration)" {
-  # The step should appear between the commit/release-cadence step and
-  # the "Step 7: Loop" section, with a recognisable heading.
-  run grep -niE "inter-iteration verification|verif.*iteration|post-iteration verif" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md inter-iteration check uses git status --porcelain" {
-  run grep -n "git status --porcelain" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md inter-iteration check halts the loop on unexpected dirty state" {
-  # Must describe halting / stopping when the tree is dirty without a
-  # known-deliberate reason.
-  run grep -niE "halt|stop the loop|dirty.*(halt|stop|block)|uncommitted.*(halt|stop|block|detected)" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md inter-iteration check distinguishes clean tree from deliberate dirty state" {
-  # Not all dirty states are errors — e.g. a governance doc transition
-  # the subagent deliberately left for the next iteration to pick up.
-  # The check should permit a documented dirty state and halt on an
-  # undocumented one.
-  run grep -niE "deliberate|known reason|expected|documented.*dirty|known.*dirty" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Non-Interactive Decision Making table covers unexpected dirty state" {
-  # AFK mode: inter-iteration dirty state should have a default action.
-  run grep -niE "inter[- ]iteration|unexpected dirty" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}

package/skills/work-problems/test/work-problems-mid-loop-userpromptsubmit-handler.bats

@@ -1,69 +0,0 @@
-#!/usr/bin/env bats
-#
-# packages/itil/skills/work-problems/test/work-problems-mid-loop-userpromptsubmit-handler.bats
-#
-# Behavioural tests for the mid-loop UserPromptSubmit handler contract
-# (P135 Phase 3 / R4 / ADR-044). The explicit corrective for the
-# 2026-04-27 iter-9-killed overcorrection.
-#
-# When the orchestrator receives a user message DURING an iter, the
-# in-flight iter MUST complete naturally to its ITERATION_SUMMARY
-# emission BEFORE the orchestrator surfaces the queue + new direction.
-# Killing the iter mid-flight (SIGTERM the iter PID) is forbidden —
-# it wastes in-flight work and breaks the iter subprocess contract.
-#
-# tdd-review: structural-permitted (justification: skill behavioural
-# harness pending P012 + P081 Phase 2; SKILL.md contract assertions
-# bridge until then; expected to migrate to behavioural form once the
-# harness exists)
-#
-# @problem P135 Phase 3 R4
-# @adr ADR-044 (Decision-Delegation Contract — mid-loop interrupt handling)
-# @adr ADR-032 (subprocess-boundary contract; iter completes naturally)
-# @adr ADR-005 / ADR-037 (testing strategy — bridge during harness build)
-# @jtbd JTBD-006 (AFK orchestrator must respect in-flight iter)
-
-SKILL_FILE="${BATS_TEST_DIRNAME}/../SKILL.md"
-
-setup() {
-  [ -f "$SKILL_FILE" ]
-}
-
-# ── Mid-loop UserPromptSubmit handler contract ──────────────────────────────
-
-@test "SKILL.md documents mid-loop UserPromptSubmit handler (P135 Phase 3 R4)" {
-  run grep -F "Mid-loop UserPromptSubmit handling" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md mid-loop handler clause MUST let the in-flight iter complete naturally to ITERATION_SUMMARY emission" {
-  run grep -F "complete naturally to its" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"ITERATION_SUMMARY"* ]]
-}
-
-@test "SKILL.md mid-loop handler clause forbids SIGTERM to the iter PID" {
-  run grep -F "no SIGTERM to the iter PID" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md mid-loop handler clause forbids killing iter mid-flight" {
-  run grep -F "Do NOT abort the iter mid-flight" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md cites the 2026-04-27 iter-9-killed overcorrection as the corrective precedent" {
-  run grep -F "iter-9-killed overcorrection" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md mid-loop handler surfaces the queue + new direction together AFTER iter completes" {
-  run grep -F "surfaces the queue + the new direction together" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md cites the ~$5 + 25 min wasted-work cost as the loss measurement for the corrective" {
-  # Concrete cost citation per ADR-026 grounding
-  run grep -F "$5 + 25 min" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}

package/skills/work-problems/test/work-problems-no-mid-loop-asking.bats

@@ -1,192 +0,0 @@
-#!/usr/bin/env bats
-
-# P130: /wr-itil:work-problems orchestrator must NOT call AskUserQuestion
-# mid-loop. The loop's purpose is progress + accumulation; user-interaction
-# is reserved for framework-prescribed halt points (Step 0, Step 2.5/2.5b,
-# Step 6.5 Rule 5 + CI-failure, Step 6.75 dirty-unknown). The user is
-# transient — presence-detection is unreliable and is not the goal; the
-# orchestrator must accumulate questions and surface them at halt points.
-#
-# Per the user's reframe (ticket P130 lines 95-123), the original dual-mode
-# dispatch fix-shape was rejected. The fix is SKILL.md prose discipline:
-# enumerate the framework-prescribed halt points + assert no mid-iter asks
-# elsewhere + state the accumulated-question discipline (direction-setting
-# only; no BUFD; no questions answerable by research/exploration/
-# experimentation).
-#
-# Doc-lint contract assertions per ADR-037 Permitted Exception (structural
-# checks on prose contract, sibling shape with P126 / P135 fixtures).
-#
-# @problem P130
-# @adr ADR-044 (Decision-Delegation Contract — framework-resolution boundary)
-# @adr ADR-013 Rule 1 (as amended by ADR-044) + Rule 6 (non-interactive fail-safe)
-# @adr ADR-032 (subprocess-boundary contract — unchanged)
-# @adr ADR-037 (skill-testing-strategy — Permitted Exception for prose contract)
-# @jtbd JTBD-006 (Progress the Backlog While I'm Away)
-# @jtbd JTBD-001 (Enforce Governance Without Slowing Down)
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../../.." && pwd)"
-  SKILL_MD="$REPO_ROOT/packages/itil/skills/work-problems/SKILL.md"
-}
-
-@test "work-problems P130: SKILL.md exists" {
-  [ -f "$SKILL_MD" ]
-}
-
-# ── Mid-loop ask discipline subsection presence ─────────────────────────────
-
-@test "work-problems P130: SKILL.md names a 'Mid-loop ask discipline' subsection (orchestrator main turn)" {
-  # The architect-approved placement is inside Non-Interactive Decision
-  # Making section as a subsection. The heading must exist as a navigable
-  # markdown anchor so cross-references resolve.
-  run grep -nE '^#{3,4} Mid-loop ask discipline' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P130: subsection scopes itself to the orchestrator main turn (not the subprocess)" {
-  # The subprocess-layer constraint (Step 5's iteration-prompt body)
-  # already exists; this subsection is the orchestrator-main-turn
-  # equivalent. The scope must be explicit so future readers do not
-  # confuse the layers.
-  run grep -nE 'orchestrator main turn|orchestrator.s main turn' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-# ── Core invariant: no mid-iter AskUserQuestion ─────────────────────────────
-
-@test "work-problems P130: subsection asserts no mid-loop AskUserQuestion between iters except at framework halts" {
-  # The load-bearing rule. The orchestrator must NOT call AskUserQuestion
-  # between iterations EXCEPT at the framework-prescribed halt points.
-  run grep -nE 'MUST NOT call .?AskUserQuestion.? between iter' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P130: subsection cites the loop's purpose as progress + accumulation" {
-  # The reframed direction (ticket lines 95-123): the loop's purpose is
-  # progress + accumulation, not interactive-vs-AFK routing. This phrasing
-  # must appear so future authors do not re-introduce mid-loop asks under
-  # 'the user might be present' rationalisations.
-  run grep -nE 'progress \+ accumulation|progress and accumulation' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P130: subsection states presence-detection is unreliable and not the goal" {
-  # The transient-user framing. Without this, future authors may try to
-  # add presence-detection (the originally-rejected fix-shape).
-  run grep -nE 'presence[- ]detection is unreliable|presence is unreliable|user as transient|treat the user as transient' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-# ── Halt-point enumeration ──────────────────────────────────────────────────
-
-@test "work-problems P130: subsection enumerates Step 0 session-continuity as a permitted halt point" {
-  run grep -nE 'Step 0.*session[- ]continuity' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P130: subsection enumerates Step 2.5 / 2.5b loop-end emit as a permitted halt point" {
-  run grep -nE 'Step 2\.5.*loop[- ]end|Step 2\.5b' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P130: subsection enumerates Step 6.5 above-appetite Rule 5 halt as a permitted halt point" {
-  run grep -nE 'Step 6\.5.*Rule 5|above[- ]appetite Rule 5' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P130: subsection enumerates Step 6.75 dirty-for-unknown-reason as a permitted halt point" {
-  run grep -nE 'Step 6\.75.*dirty[- ]for[- ]unknown|Step 6\.75.*unknown reason' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-# ── Accumulated-question discipline at surface time ─────────────────────────
-
-@test "work-problems P130: subsection states direction-setting only at surface time" {
-  # ADR-044's six-class taxonomy: only the user-answerable categories
-  # qualify. Direction-setting is the canonical example.
-  run grep -nE 'Direction[- ]setting only|direction[- ]setting only' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P130: subsection forbids BUFD-style questions" {
-  # No big-design-up-front; small actionable questions. Anti-BUFD is a
-  # project-wide value (cited in ADR-044's anti-BUFD-for-framework-
-  # evolution clause).
-  run grep -nE 'No BUFD|no BUFD' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P130: subsection forbids questions answerable by research / exploration / experimentation" {
-  # The agent should investigate itself rather than sub-contract routine
-  # work back to the user.
-  run grep -nE 'research.*exploration.*experimentation|prototype.*read code.*experiments' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-# ── ADR citations ──────────────────────────────────────────────────────────
-
-@test "work-problems P130: subsection cites ADR-044 (primary) as the framework-resolution authority" {
-  # ADR-044 is the parent decision narrowing ADR-013 Rule 1 to framework-
-  # unresolved decisions. The subsection must cite it so future readers
-  # follow the cross-reference for the full picture.
-  run grep -nE 'ADR-044' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P130: subsection cites ADR-013 Rule 1 as amended by ADR-044" {
-  # The narrowing happens at ADR-013 Rule 1's amendment. The subsection
-  # must name the rule so the chain is traceable.
-  run grep -nE 'ADR-013.*Rule 1|Rule 1.*amended by ADR-044' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P130: subsection cites ADR-013 Rule 6 as the non-interactive fail-safe underlying constraint" {
-  # When AFK, Rule 6 is the underlying constraint that forbids
-  # AskUserQuestion. The subsection cites it so the subprocess-boundary
-  # connection is explicit.
-  run grep -nE 'ADR-013.*Rule 6|Rule 6.*non[- ]interactive fail[- ]safe' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-# ── ADR-032 subprocess-boundary unchanged ──────────────────────────────────
-
-@test "work-problems P130: subsection cross-references ADR-032 subprocess-boundary contract as unchanged" {
-  # The architect's flag: this fix does NOT amend ADR-032. The subsection
-  # must say so explicitly to prevent future readers thinking the
-  # subprocess-boundary contract is in scope.
-  run grep -nE 'subprocess[- ]boundary contract.*unchanged|ADR-032.*unchanged' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-# ── Subprocess vs orchestrator layer cross-reference ───────────────────────
-
-@test "work-problems P130: subsection cross-references Step 5's per-subprocess constraint as the parallel layer" {
-  # The orchestrator-main-turn discipline parallels Step 5's iteration-
-  # prompt-body 'Do not call AskUserQuestion' constraint. Both layers
-  # together enforce the discipline end-to-end.
-  run grep -nE "Step 5.*iteration[- ]prompt.*AskUserQuestion|Step 5's iteration[- ]prompt" "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-# ── Step 5's iteration-prompt body augmented with transient-user framing ───
-
-@test "work-problems P130: Step 5 iteration-prompt body includes transient-user framing" {
-  # The reframed direction's core insight: the user is transient even
-  # when present. The iteration-prompt body (already-existing
-  # 'NEVER call AskUserQuestion mid-loop in AFK') gains the transient
-  # framing so future iter authors understand why the constraint is
-  # absolute.
-  run grep -nE 'transient|disappear for hours' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-# ── Decision Table row reflects the discipline ─────────────────────────────
-
-@test "work-problems P130: Non-Interactive Decision Making table carries a row for mid-loop ask discipline" {
-  # The decisions table (line ~487) summarises the orchestrator's
-  # non-interactive defaults. A row naming mid-loop ask discipline keeps
-  # the table consistent with the new subsection.
-  run grep -nE '\| Mid-loop ask|mid[- ]loop AskUserQuestion|mid-loop ask between iter' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}

package/skills/work-problems/test/work-problems-p341-pre-all-done-gate.bats

@@ -1,122 +0,0 @@
-#!/usr/bin/env bats
-
-# P341: work-problems SKILL must surface outstanding questions THEN run
-# a session-level retro BEFORE emitting ALL_DONE. The fix shape adds a
-# new step (Step 2.4 — Pre-ALL_DONE gate sequence) that fires
-# UNCONDITIONALLY before ALL_DONE emit, sequencing (a) outstanding-
-# questions surface + (b) session-level retro + (c) ALL_DONE.
-#
-# Hard-fail mode: if either gate cannot complete (user not present and
-# queue has user-input-required entries, retro fails), the SKILL.md MUST
-# direct the orchestrator to halt with a clear directive — NOT emit
-# ALL_DONE.
-#
-# Doc-lint contract assertions per ADR-037 Permitted Exception
-# (structural checks on prose contract; behavioural harness for SKILL.md
-# pending P081 Phase 2 / P012).
-#
-# @problem P341
-# @adr ADR-044 (Decision-Delegation Contract — direction-class observations are the protected surface this gate enforces)
-# @adr ADR-013 (structured user interaction — outstanding-questions surface is the load-bearing application)
-# @adr ADR-014 (governance skills commit own work — retro commits its own work)
-# @adr ADR-037 (skill-testing-strategy — Permitted Exception for prose contract)
-# @jtbd JTBD-006 (Progress the Backlog While I'm Away)
-# @jtbd JTBD-201 (audit-trail — ALL_DONE is honest sentinel post-amendment)
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../../.." && pwd)"
-  SKILL_MD="$REPO_ROOT/packages/itil/skills/work-problems/SKILL.md"
-}
-
-@test "work-problems P341: SKILL.md exists" {
-  [ -f "$SKILL_MD" ]
-}
-
-# ── Step 2.4 (or named equivalent) gate-sequence subsection presence ────────
-
-@test "work-problems P341: SKILL.md names a Pre-ALL_DONE gate sequence step" {
-  # The fix adds a new orchestrator-main-turn step that fires
-  # UNCONDITIONALLY before ALL_DONE emit. The step MUST be a markdown
-  # heading so cross-references resolve to a single source of truth.
-  run grep -nE '^#{3,4} Step 2\.4|Pre-`?ALL_DONE`? gate sequence' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P341: gate-sequence step is unconditional (fires before every ALL_DONE emit)" {
-  # The structural gap P341 closes is that Step 2.5 fires conditionally
-  # on stop-condition #2. The new step fires UNCONDITIONALLY before
-  # ALL_DONE emit regardless of stop-condition.
-  run grep -nE 'unconditionally|UNCONDITIONAL|every `?ALL_DONE`? emit|before every `?ALL_DONE`?' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P341: gate-sequence names outstanding-questions surface as gate (a)" {
-  # Sequence (a): Read .afk-run-state/outstanding-questions.jsonl; if
-  # non-empty, fire Step 2.5b's surfacing routine; truncate on completion.
-  run grep -nE 'outstanding-questions\.jsonl|outstanding-questions surface|Step 2\.5b.*surfacing' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P341: gate-sequence names session-level retro as gate (b)" {
-  # Sequence (b): Run session-level retro via /wr-retrospective:run-retro
-  # — covers cross-iter patterns, friction observations, framework-
-  # improvement candidates. Retro commits its own work per ADR-014.
-  run grep -nE 'session-level retro|/wr-retrospective:run-retro' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P341: gate-sequence names ALL_DONE emit as gate (c) ONLY after (a) and (b)" {
-  # Sequence (c): Emit ALL_DONE ONLY after both (a) and (b) complete.
-  # The ordering must be explicit so future authors don't re-permit a
-  # short-circuit.
-  run grep -nE 'ALL_DONE.*after.*both|ONLY after.*(a).*(b)|Emit `?ALL_DONE`? ONLY' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-# ── Hard-fail mode (halt instead of ALL_DONE) ──────────────────────────────
-
-@test "work-problems P341: gate-sequence directs halt-with-directive when either gate cannot complete" {
-  # Hard-fail mode: if outstanding-questions surface cannot complete OR
-  # retro fails, the SKILL.md MUST direct the orchestrator to halt with a
-  # clear directive — NOT emit ALL_DONE. Halt is recoverable; user
-  # returns, surfaces, completes the loop with ALL_DONE.
-  run grep -nE 'halt with.*directive|MUST.*halt.*NOT emit|halt instead of.*ALL_DONE|halt.*not.*ALL_DONE' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems P341: gate-sequence mentions ADR-014 commit-ownership for retro work" {
-  # Retro commits its own work per ADR-014 — the orchestrator MUST NOT
-  # re-commit retro's output, AND retro is not silently dropped because
-  # the orchestrator forgot to invoke it.
-  run grep -nE 'retro commits its own work|run-retro.*ADR-014|retro.*per ADR-014' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-# ── Output Format ALL_DONE positioning ──────────────────────────────────────
-
-@test "work-problems P341: Output Format section reflects new ALL_DONE sequence" {
-  # The Output Format section MUST reference the gate sequence so the
-  # rendered ALL_DONE position is documented to follow Step 2.4. This
-  # protects against future authors who add new sections between the
-  # gate sequence and the ALL_DONE marker.
-  run grep -nE 'Step 2\.4|gate sequence|Pre-`?ALL_DONE`? gate' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-# ── Decision Table row (cross-reference to gate sequence) ───────────────────
-
-@test "work-problems P341: Non-Interactive Decision Making table carries a pre-ALL_DONE gate row" {
-  # The decisions table at the bottom of SKILL.md must carry a row that
-  # names the pre-ALL_DONE gate sequence so the decision summary is
-  # consistent with the Step prose. This prevents future readers from
-  # missing the gate when scanning the decisions table only.
-  run grep -nE '\| Pre-`?ALL_DONE`? gate|\| ALL_DONE.*gate sequence|\| Loop-end.*gate sequence' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-# ── Cross-reference to P341 + sibling P342 ──────────────────────────────────
-
-@test "work-problems P341: Related section cites P341 as the originating ticket" {
-  run grep -nE '\*\*P341\*\*|P341\b' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}

package/skills/work-problems/test/work-problems-p342-r007-derive-then-pass-flags.bats

@@ -1,170 +0,0 @@
-#!/usr/bin/env bats
-
-# R007 paired-capability gap (post-commit 54ecf83 — ADR-060 Amendment
-# 2026-06-02 + I12 derive-then-ratify in capture-problem):
-#
-# The P342 mechanical-stage carve-out at Step 5 iter-prompt authorises
-# retro-surfaced recurring class-of-behaviour observations to auto-ticket
-# via `/wr-itil:capture-problem`. The new capture-problem contract
-# requires AFK callers (iter subprocesses) to pre-resolve persona + JTBD
-# via `--no-prompt --persona=<value> --jtbd=JTBD-NNN` flags, OR capture
-# halts-with-stderr-directive. The halt stderr is unobservable to the
-# AFK user — silent loop-stall.
-#
-# The fix shape amends the work-problems Step 5 iter-prompt body's P342
-# classification taxonomy bullet for "Recurring class-of-behaviour":
-#   - Derive persona from iter context; default `developer` on ambiguity.
-#   - Validate persona against the enum
-#     `{developer | tech-lead | plugin-developer | plugin-user}` before
-#     dispatch.
-#   - Derive JTBD from iter context. Cite JTBD-006 (AFK-loop-continuity),
-#     JTBD-001 (governance), JTBD-101 (plugin-developer).
-#   - Dispatch shape:
-#       /wr-itil:capture-problem --no-prompt --persona=<derived>
-#         --jtbd=<derived> "<description>"
-#   - On genuine derive-failure (cannot cleanly resolve, invalid enum):
-#     route to outstanding_questions, NOT capture-problem dispatch.
-#
-# Doc-lint contract assertions per ADR-037 Permitted Exception.
-#
-# @problem P342 (originating mechanical-stage carve-out)
-# @problem P078 (capture-on-correction; sibling caller for the same dispatch)
-# @adr ADR-060 Amendment 2026-06-02 (I12 derive-then-ratify)
-# @adr ADR-044 (Decision-Delegation Contract — silent-framework cat 4 on
-#               derive-success; direction cat 1 on outstanding_questions route)
-# @adr ADR-014 (single-commit grain — SKILL + bats + changeset)
-# @jtbd JTBD-006 (Progress the Backlog While I'm Away — audit-trail outcome:
-#                 halt-with-stderr in iter is invisible to AFK user; route to
-#                 outstanding_questions or pre-resolve flags to preserve trail)
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../../.." && pwd)"
-  SKILL_MD="$REPO_ROOT/packages/itil/skills/work-problems/SKILL.md"
-}
-
-@test "work-problems R007: SKILL.md exists" {
-  [ -f "$SKILL_MD" ]
-}
-
-# ── Dispatch shape: --no-prompt + --persona + --jtbd flags ─────────────────
-
-@test "work-problems R007: iter-prompt dispatch shape names --no-prompt flag" {
-  # AFK-mode marker. Without --no-prompt, capture-problem's I12 derive-
-  # then-ratify AskUserQuestion fallback could fire inside the iter
-  # subprocess where the user is absent.
-  run grep -nE '\-\-no-prompt' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems R007: iter-prompt dispatch shape names --persona flag" {
-  # Pre-resolves persona to silent-proceed path. Required by capture-
-  # problem SKILL.md Step 1.5b silent-framework branch.
-  run grep -nE '\-\-persona=' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems R007: iter-prompt dispatch shape names --jtbd flag" {
-  # Pre-resolves JTBD-trace to silent-proceed path.
-  run grep -nE '\-\-jtbd=' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems R007: iter-prompt cites I12 derive-then-ratify contract authority" {
-  # The dispatch contract's upstream authority is ADR-060 Amendment
-  # 2026-06-02 + I12 derive-then-ratify. Cite so future authors don't
-  # drift on the rationale.
-  run grep -nE 'I12 derive-then-ratify|derive-then-ratify.*ADR-060|ADR-060 Amendment 2026-06-02' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems R007: iter-prompt cites R007 paired-capability gap" {
-  # R007 names the gap this amendment closes. Cited so the iter-prompt
-  # ties back to the originating risk-register entry.
-  run grep -nE 'R007.*paired-capability|paired-capability.*R007|R007 paired-capability gap' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-# ── Persona derivation contract ────────────────────────────────────────────
-
-@test "work-problems R007: iter-prompt derives persona from iter context" {
-  # Derive-don't-ask: persona signals come from the ticket the iter
-  # is dispatched against (Origin + RFC + story trace).
-  run grep -nE 'derive persona|Persona derivation' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems R007: iter-prompt defaults persona to developer on ambiguity" {
-  # Default chosen per JTBD-006 + dominant-persona in this monorepo.
-  run grep -nE 'Default to .?developer.?|default to .?developer.? if|default to .?developer.? when' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems R007: iter-prompt names the persona enum for pre-dispatch validation" {
-  # capture-problem halts-with-directive on invalid --persona= value
-  # (SKILL.md Step 1.5b). Iter must validate against the enum BEFORE
-  # dispatch, or fall through to outstanding_questions.
-  run grep -nE 'developer.*tech-lead.*plugin-developer.*plugin-user|persona enum' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-# ── JTBD derivation contract ───────────────────────────────────────────────
-
-@test "work-problems R007: iter-prompt derives JTBD from iter context" {
-  # Derive-don't-ask: JTBD signals come from iter-prompt content.
-  run grep -nE 'derive.*JTBD|JTBD derivation' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems R007: iter-prompt cites JTBD-006 for AFK-loop-continuity contexts" {
-  run grep -nE 'JTBD-006.*AFK|AFK.*JTBD-006|JTBD-006.*loop-continuity|JTBD-006.*iter-dispatch' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems R007: iter-prompt cites JTBD-001 for governance contexts" {
-  run grep -nE 'JTBD-001.*governance|governance.*JTBD-001|JTBD-001.*ADR|JTBD-001.*decision' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems R007: iter-prompt cites JTBD-101 for plugin-developer contexts" {
-  run grep -nE 'JTBD-101.*plugin|plugin.*JTBD-101|JTBD-101.*discoverability|JTBD-101.*suite-extension' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-# ── Fall-through to outstanding_questions on derive-failure ────────────────
-
-@test "work-problems R007: iter-prompt routes genuinely-ambiguous derivation to outstanding_questions" {
-  # Architect AMEND closed: when derivation cannot cleanly resolve (or
-  # persona fails enum validation), do NOT invoke capture-problem with
-  # a bad value (would halt-with-stderr-directive into unobservable iter
-  # subprocess stderr); instead, queue for orchestrator main-turn Step
-  # 2.5 surfacing.
-  run grep -nE 'Genuinely-ambiguous|cannot pick persona/JTBD cleanly|fall through to .?outstanding_questions' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems R007: iter-prompt names the unobservable-stderr failure mode" {
-  # Documents WHY the dispatch contract matters: the halt stderr does
-  # not surface to the AFK user; the loop silently stalls. JTBD-006
-  # audit-trail violation framing.
-  run grep -nE 'stderr is unobservable|unobservable.*stderr|silent loop-stall|halt stderr' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-# ── Composition with the existing P342 carve-out ───────────────────────────
-
-@test "work-problems R007: amendment preserves the P342 mechanical-stage carve-out framing" {
-  # The R007 fix is ADDITIVE — it carries the new dispatch shape into
-  # the existing Recurring-class auto-ticket bullet. The P342 carve-out
-  # itself (mechanical-stage / Step 4a precedent / ADR-013 Rule 5)
-  # remains the authority for auto-ticketing at all.
-  run grep -nE 'mechanical-stage carve-out|policy-authorised silent proceed' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "work-problems R007: Ambiguous bullet reuses the persona+JTBD derivation contract" {
-  # The Ambiguous-classification default-to-auto-ticket path also goes
-  # through the new dispatch contract (same flag-resolution discipline);
-  # otherwise the Ambiguous branch becomes a halt vector.
-  run grep -nE 'same persona \+ JTBD derivation contract|using the same persona \+ JTBD derivation' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}