@windyroad/retrospective 0.27.1 → 0.27.2

package/skills/run-retro/test/run-retro-pipeline-instability-scan.bats

@@ -1,122 +0,0 @@
-#!/usr/bin/env bats
-
-# P074: run-retro SKILL.md documents a Pipeline-instability scan step
-# (Step 2b) that inspects session activity for tool-level friction
-# signals and funnels each detection into Step 4's problem-ticket
-# creation path. Shape mirrors P068's Step 4a (evidence-scan + ADR-026
-# grounding + interactive/AFK branches).
-#
-# Doc-lint structural test (Permitted Exception per ADR-005). Asserts
-# SKILL.md wording for: the step header, the placement between Step 2
-# and Step 4, the six signal categories enumerated in the RCA, the
-# ADR-026 grounding requirement, the interactive AskUserQuestion
-# contract (ADR-013 Rule 1), the AFK fallback (ADR-013 Rule 6), the
-# ownership-delegation boundary to /wr-itil:manage-problem, and the
-# Step 5 summary integration.
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../../.." && pwd)"
-  SKILL_MD="$REPO_ROOT/packages/retrospective/skills/run-retro/SKILL.md"
-}
-
-@test "run-retro: SKILL.md contains Step 2b Pipeline-instability scan (P074)" {
-  run grep -F '### 2b. Pipeline-instability scan (P074)' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2b enumerates all six signal categories" {
-  run grep -F 'Hook-protocol friction' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'Skill-contract violations' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'Release-path instability' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'Subagent-delegation friction' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'Repeat-work friction' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'Session-wrap silent drops' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2b requires specific-citation grounding (ADR-026)" {
-  run grep -F 'ADR-026' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'specific citations' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2b interactive AskUserQuestion contract per ADR-013 Rule 1" {
-  run grep -F 'ADR-013 Rule 1' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'Create new ticket' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'Append to P<NNN>' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'Skip — false positive' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2b AFK fallback defers ticket creation per ADR-013 Rule 6" {
-  run grep -F 'ADR-013 Rule 6' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'Pipeline Instability' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2b delegates ticket creation to /wr-itil:manage-problem" {
-  run grep -F '/wr-itil:manage-problem' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'run-retro surfaces the detection' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2b dedup checks existing tickets before creating" {
-  run grep -F 'Dedup against existing tickets' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'docs/problems/*.open.md' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'docs/problems/*.known-error.md' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2b ADR-032 supersession note documents post-supersession context handling" {
-  # ADR-027 was superseded by ADR-032 (2026-04-21). The former
-  # "ADR-027 compatibility note" was rewritten to an ADR-032 supersession
-  # note that records the obviation of any Step-0 subagent migration.
-  # Structural grep retained for now (P081 anti-pattern; convert to
-  # behavioural fixture in a follow-up — tracked separately).
-  run grep -F 'ADR-032 supersession note' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'No Step-0 subagent migration applies' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2b placement between Step 2 reflection and Step 4 ticket creation" {
-  # Section 2b must appear after 2 and before 4.
-  pos_2=$(grep -n '^### 2\. Reflect on this session' "$SKILL_MD" | head -1 | cut -d: -f1)
-  pos_2b=$(grep -n '^### 2b\. Pipeline-instability scan' "$SKILL_MD" | head -1 | cut -d: -f1)
-  pos_4=$(grep -n '^### 4\. Create or update problem tickets' "$SKILL_MD" | head -1 | cut -d: -f1)
-  [ -n "$pos_2" ]
-  [ -n "$pos_2b" ]
-  [ -n "$pos_4" ]
-  [ "$pos_2" -lt "$pos_2b" ]
-  [ "$pos_2b" -lt "$pos_4" ]
-}
-
-@test "run-retro: Step 5 summary adds a Pipeline Instability section" {
-  run grep -F '### Pipeline Instability' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Pipeline Instability summary table columns match Step 2b output" {
-  run grep -F '| Signal | Category | Citations | Decision |' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2b documents interaction with P068 Step 4a shape (shared evidence-scan pattern)" {
-  run grep -F 'P068' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'evidence-scan' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}

package/skills/run-retro/test/run-retro-signal-vs-noise.bats

@@ -1,137 +0,0 @@
-#!/usr/bin/env bats
-
-# P105: run-retro SKILL.md documents a signal-vs-noise pass (Step 1.5) that
-# scores every briefing entry per retro cycle, drives Critical Points curation,
-# and gates deletion behind a batched user-confirmation queue.
-#
-# Doc-lint structural test (Permitted Exception per ADR-005). Asserts
-# SKILL.md wording for: the step header, placement between Step 1 and Step 2,
-# the three scoring rules (signal +2, noise -1, decay -1), the HTML comment
-# persistence format, the ADR-026 grounding requirement, the threshold
-# actions (promote/keep/delete), the Tier 1 budget guard, the delete-queue
-# AskUserQuestion contract (ADR-013 Rule 1), the AFK fallback (ADR-013 Rule 6),
-# the Step 3 agent-driven promotion update, and the Step 5 summary integration.
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../../.." && pwd)"
-  SKILL_MD="$REPO_ROOT/packages/retrospective/skills/run-retro/SKILL.md"
-}
-
-@test "run-retro: SKILL.md contains Step 1.5 Briefing signal-vs-noise pass (P105)" {
-  run grep -F '### 1.5. Briefing signal-vs-noise pass (P105)' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 1.5 placement between Step 1 and Step 2" {
-  pos_1=$(grep -n '^### 1\. ' "$SKILL_MD" | head -1 | cut -d: -f1)
-  pos_1_5=$(grep -n '^### 1\.5\. ' "$SKILL_MD" | head -1 | cut -d: -f1)
-  pos_2=$(grep -n '^### 2\. ' "$SKILL_MD" | head -1 | cut -d: -f1)
-  [ -n "$pos_1" ]
-  [ -n "$pos_1_5" ]
-  [ -n "$pos_2" ]
-  [ "$pos_1" -lt "$pos_1_5" ]
-  [ "$pos_1_5" -lt "$pos_2" ]
-}
-
-@test "run-retro: Step 1.5 documents all three scoring events" {
-  run grep -F 'Signal | +2' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'Noise | -1' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'Decay | -1' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 1.5 documents the HTML comment persistence format" {
-  run grep -F 'signal-score:' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'last-classified:' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'first-written:' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 1.5 requires ADR-026 grounding for classifications" {
-  run grep -F 'ADR-026' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'specific citation' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'Bare classifications are forbidden' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 1.5 documents promote threshold (score >= +3)" {
-  run grep -F '>= +3' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'Promote to Critical Points' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 1.5 documents delete queue threshold (score <= -3)" {
-  run grep -F '<= -3' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'delete queue' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 1.5 delete queue is silent — no AskUserQuestion (ADR-044 / P132; reconciled by P393)" {
-  # Step 1.5 deletes are SILENT (framework-mediated per ADR-044 "Briefing add/remove/rotate"
-  # + the CLAUDE.md P132 worked-example "run-retro Step 1.5 silent classification, Step 3
-  # briefing removals"). The superseded "single batched AskUserQuestion to confirm deletes"
-  # prose was a self-contradiction against Step 3's "Removals are silent" clause; reconciled
-  # to silent-delete by P393. This asserts the reconciled contract.
-  run grep -F 'Silent removal' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'Delete handling — silent' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'do NOT reintroduce it' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  # negative: the superseded delete-confirmation prompt must NOT return
-  run grep -F 'Delete briefing entries?' "$SKILL_MD"
-  [ "$status" -ne 0 ]
-}
-
-@test "run-retro: Step 1.5 AFK fallback defers delete queue to retro summary (ADR-013 Rule 6)" {
-  run grep -F 'Non-interactive / AFK fallback (ADR-013 Rule 6)' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'Do NOT auto-delete' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'Signal-vs-Noise Pass' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 1.5 documents the Tier 1 budget guard" {
-  run grep -F 'Tier 1 budget guard' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F '2 KB' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 1.5 cites ADR-040 for the Critical Points budget" {
-  run grep -F 'ADR-040' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 1.5 classification is policy-authorised silent (ADR-013 Rule 5)" {
-  run grep -F 'policy-authorised per ADR-013 Rule 5' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'agent owns silent classification' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 3 acknowledges agent-driven promotion from Step 1.5 score" {
-  run grep -F 'agent-driven per Step 1.5' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'signal-score' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 5 summary adds a Signal-vs-Noise Pass section" {
-  run grep -F '### Signal-vs-Noise Pass (P105)' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Signal-vs-Noise Pass summary table columns match Step 1.5 output" {
-  run grep -F '| Entry | Topic file | Old score | New score | Classification | Citation |' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}

package/skills/run-retro/test/run-retro-skill-candidates.bats

@@ -1,116 +0,0 @@
-#!/usr/bin/env bats
-# Doc-lint guard: run-retro SKILL.md must include the skill-recommendation branch.
-#
-# Structural assertion — Permitted Exception to the source-grep ban (ADR-005 / P011).
-# These tests do not assert hook behaviour; they assert that the skill specification
-# document includes the skill-candidate branch added in P044.
-#
-# Cross-reference:
-#   P044 (docs/problems/044-run-retro-does-not-recommend-new-skills.known-error.md)
-#   ADR-013 Rule 1 / Rule 6 (docs/decisions/013-structured-user-interaction-for-governance-decisions.proposed.md)
-#   @jtbd JTBD-001 (enforce governance without slowing down)
-#   @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 and has frontmatter" {
-  [ -f "$SKILL_FILE" ]
-  run head -1 "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  [ "$output" = "---" ]
-}
-
-@test "SKILL.md Step 2 includes the skill-candidate reflection category (P044, updated by P050)" {
-  # P044 fix: Step 2 must prompt for recurring workflows that would be better
-  # as skills. P050 generalises this to a codification category, with "skill"
-  # retained as one worked example within the shape list. This test accepts
-  # either the original P044 phrasing OR the P050 generalised phrasing that
-  # still names "skill" as a shape.
-  run grep -in "recurring workflow.*better as a skill\|would be better as a skill\|recurring pattern.*better codified\|\*\*Skill\*\* — " "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md includes a Step 4b Recommend new skills branch (P044)" {
-  # P044 fix: a dedicated output branch for skill candidates, distinct from Step 4
-  # (problem tickets) and Step 5 (summary).
-  run grep -n "Recommend new skills\|Step 4b" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 4b uses AskUserQuestion (ADR-013 Rule 1)" {
-  # ADR-013 Rule 1: the skill-candidate decision branch must use AskUserQuestion,
-  # not prose '(a)/(b)/(c)' enumeration. Architect review flagged this as the
-  # gotcha to avoid when implementing P044.
-  run grep -n "AskUserQuestion" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 4b header matches ADR-013 structured-interaction pattern (P044, updated by P050)" {
-  # P044 used "Skill candidate" as the AskUserQuestion header. P050 generalises
-  # to "Codification candidate" with "Skill" as one shape option. Accept either —
-  # both preserve the ADR-013 Rule 1 structured-interaction shape.
-  run grep -in "Skill candidate\|Codification candidate" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 4b names the structured options for skill-shaped candidates (P044, updated by P050, reframed by P075)" {
-  # P044 required three specific option labels (Create a new skill / Track as
-  # a problem / Skip — not skill-worthy). P050 generalised to "Skill — create
-  # stub". P075 reframes Step 4b as a two-stage flow: Stage 1 tickets
-  # mechanically (the former "Track as a problem" decision disappears
-  # because ticketing is no longer a user choice); Stage 2 asks the
-  # fix-strategy question with four options. P044's recommend-new-skills
-  # intent now rides in Stage 2 Option 1 (`Skill — create stub`). The
-  # former "Skip — not skill-worthy / not codify-worthy" path becomes
-  # Stage 2 Option 4 (`Self-contained work — no codification stub`) with a
-  # Rule 6 audit note preventing silent-skip. Accept all three forms across
-  # the P044 / P050 / P075 lineage.
-  run grep -in "Create a new skill\|Skill — create stub\|Skill - create stub" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  # Stage 1 ticketing delegation is mechanical post-P075; assert the
-  # delegation path is named in SKILL.md so the P044 "Track as a problem
-  # ticket" enforcement intent still has a visible home.
-  run grep -in "Track as a problem ticket\|Problem — invoke manage-problem\|/wr-itil:manage-problem\|/wr-itil:capture-problem" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  # The skip-equivalent path. P075 renamed this to "Self-contained work" so
-  # P044's escape-hatch pain pattern does not leak; keep the legacy strings
-  # in the regex for backward compatibility across the P044 / P050 / P075
-  # lineage.
-  run grep -in "Skip — not skill-worthy\|Skip - not skill-worthy\|Skip — not codify-worthy\|Skip - not codify-worthy\|Self-contained work — no codification stub\|Self-contained work - no codification stub" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 4b has non-interactive fallback per ADR-013 Rule 6" {
-  # ADR-013 Rule 6: if AskUserQuestion is unavailable, flag candidates instead of
-  # silently choosing. P044 implementation uses "flagged — not actioned" wording.
-  run grep -n "non-interactive\|Rule 6" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 5 summary template has a Skill / Codification Candidates slot (P044, updated by P050)" {
-  # P044 fix: the summary template must include a Skill Candidates section so
-  # recommendations are visible in the session audit alongside BRIEFING changes
-  # and problem tickets. P050 generalises this to a unified "Codification
-  # Candidates" table with a Shape column; skill-shaped candidates still appear
-  # (as Shape: skill rows). Accept either heading.
-  run grep -n "### Skill Candidates\|### Codification Candidates" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md does not contain 'Options: (a)' prose option list (ADR-013)" {
-  # ADR-013 Rule 1: user-facing decisions must use AskUserQuestion, not prose
-  # "Options: (a)/(b)/(c)". This matches the narrow pattern used by
-  # manage-problem-no-prose-options.bats so criteria lists using (a)/(b)/(c)
-  # as internal enumeration (which are fine) don't trip the test.
-  run grep -n "Options: (a)" "$SKILL_FILE"
-  [ "$status" -ne 0 ]
-}
-
-@test "SKILL.md does not contain 'Your call:' prose option prompt (ADR-013)" {
-  # ADR-013 Rule 1: mirrors the manage-problem regression guard.
-  run grep -n "Your call:" "$SKILL_FILE"
-  [ "$status" -ne 0 ]
-}

package/skills/run-retro/test/run-retro-stage-1-fallback-gating.bats

@@ -1,65 +0,0 @@
-#!/usr/bin/env bats
-#
-# P148: run-retro SKILL.md Step 4b Stage 1 MUST carry a tightened
-# AFK-branch fallback-gating clause that (a) names `cause: skill_unavailable`
-# as the only valid fallback gate, (b) enumerates the rationalisation
-# anti-patterns the agent is forbidden to use, and (c) cites the
-# advisory script that mechanically enforces the cause allowlist.
-#
-# # Test shape: minimal structural backstop (ADR-037 permitted exception)
-#
-# Per P081 (structural-content tests are wasteful — behavioural preferred),
-# the LOAD-BEARING test for this fix lives at
-# `packages/retrospective/scripts/test/check-tickets-deferred-cause.bats`
-# (behavioural — exercises the script against fixture retro summary
-# directories with good / bad / legacy / mixed cause column shapes).
-#
-# This file is a TINY structural backstop that links the SKILL.md prose
-# (canonical human-readable source) to the script (mechanical enforcement).
-# Without this link, the prose contract and the script contract could
-# drift independently — agents reading the prose would see one set of
-# valid causes, the script would enforce a different set. The structural
-# assertion confirms the prose names the same allowlist token (`skill_unavailable`)
-# and the same script path the prose tells the reader to consult.
-#
-# Architect verdict (2026-04-29): keep the prose as canonical, the
-# script as mechanical enforcement, and ONE structural assertion linking
-# them. P081 narrows ADR-037's permitted exception; this file's three
-# assertions are the architect-approved minimum.
-#
-# # @adr ADR-037 permitted exception — narrowest justifiable scope.
-# # @adr ADR-044 framework-mediated surface boundary cited in SKILL.md.
-# # @ticket P148 — Stage 1 fallback-gating tightening (driver).
-# # @ticket P145 — sibling defer-pattern at Tier 3 rotation (composing surface).
-# # @ticket P081 — behavioural-tests-preferred direction (justifies tiny scope).
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../../.." && pwd)"
-  SKILL_MD="$REPO_ROOT/packages/retrospective/skills/run-retro/SKILL.md"
-}
-
-@test "Stage 1 AFK branch names 'cause: skill_unavailable' as the only valid fallback gate (P148)" {
-  # Links SKILL.md prose to the script's allowlist. If the prose adds
-  # a second valid cause without an matching script update, this
-  # assertion still passes — the goal is "the prose names the canonical
-  # token", not "the prose names the entire allowlist exhaustively".
-  run grep -F 'cause: skill_unavailable' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "Stage 1 AFK branch references the advisory script by path (P148)" {
-  # Without this reference, a reader of the SKILL.md prose has no path
-  # to the mechanical enforcement layer. The script reference IS the
-  # affordance to the tooling.
-  run grep -F 'check-tickets-deferred-cause.sh' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "Stage 1 AFK branch cites P148 as the anti-pattern driver (P148)" {
-  # Audit-trail link: the anti-pattern enumeration's authority is the
-  # 2026-04-29 user correction recorded on P148. This assertion confirms
-  # the prose names the driver ticket so future readers can locate the
-  # evidence trail.
-  run grep -F 'P148' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}

package/skills/run-retro/test/run-retro-step-2d-r6-auto-flag.bats

@@ -1,82 +0,0 @@
-#!/usr/bin/env bats
-#
-# packages/retrospective/skills/run-retro/test/run-retro-step-2d-r6-auto-flag.bats
-#
-# Behavioural tests for Step 2d's R6-numeric-gate auto-flag contract
-# (P135 follow-up / ADR-044 Reassessment Trigger). When the R6 gate
-# fires (lazy count ≥2 across 3 consecutive retros), Step 2d MUST
-# auto-queue a deviation-candidate so the framework reminds itself
-# that Phase 4 enforcement-hook work is warranted.
-#
-# 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 (ADR-044 Reassessment / Phase 4 gating)
-# @adr ADR-044 (Decision-Delegation Contract — R6 numeric gate)
-# @adr ADR-005 / ADR-037 (testing strategy)
-# @jtbd JTBD-001 / JTBD-006
-
-SKILL_FILE="${BATS_TEST_DIRNAME}/../SKILL.md"
-
-setup() {
-  [ -f "$SKILL_FILE" ]
-}
-
-@test "Step 2d documents R6 numeric gate auto-flag mechanism (P135 Reassessment Trigger)" {
-  run grep -F "R6 numeric gate auto-flag" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "Step 2d invokes check-ask-hygiene.sh to detect R6 condition" {
-  run grep -F "check-ask-hygiene.sh" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "Step 2d R6 condition explicit: lazy count >=2 across 3 consecutive retros" {
-  run grep -F "≥2 across 3 consecutive retros" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "Step 2d auto-queues a deviation-candidate when R6 fires" {
-  run grep -F "auto-queue a deviation-candidate" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "Step 2d auto-flag deviation-candidate cites P135 + ADR-044" {
-  # Bound to Step 2d section
-  run awk '/^### 2d\./,/^### 3\./ {print}' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"P135"* ]]
-  [[ "$output" == *"ADR-044"* ]]
-}
-
-@test "Step 2d auto-flag specifies the proposed_shape: amend (Phase 4 enforcement hook)" {
-  run awk '/^### 2d\./,/^### 3\./ {print}' "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"proposed_shape:"* ]]
-  [[ "$output" == *"amend"* ]]
-  [[ "$output" == *"Phase 4 enforcement hook"* ]]
-}
-
-@test "ADR-044 Reassessment cites R6 numeric gate as the explicit trigger" {
-  ADR_FILE="${BATS_TEST_DIRNAME}/../../../../../docs/decisions/044-decision-delegation-contract.proposed.md"
-  [ -f "$ADR_FILE" ]
-  run grep -F "R6 numeric gate fires" "$ADR_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "ADR-044 Reassessment explicitly cross-references Step 2d's auto-queue mechanism" {
-  ADR_FILE="${BATS_TEST_DIRNAME}/../../../../../docs/decisions/044-decision-delegation-contract.proposed.md"
-  [ -f "$ADR_FILE" ]
-  run grep -F "auto-queue a deviation-candidate" "$ADR_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "ADR-044 Reassessment names 'framework reminds itself' as the no-manual-tracking property" {
-  ADR_FILE="${BATS_TEST_DIRNAME}/../../../../../docs/decisions/044-decision-delegation-contract.proposed.md"
-  [ -f "$ADR_FILE" ]
-  run grep -F "framework reminds itself" "$ADR_FILE"
-  [ "$status" -eq 0 ]
-}