@windyroad/retrospective 0.27.1 → 0.27.2

package/skills/migrate-briefing/test/migrate-briefing-fixture.bats

@@ -1,242 +0,0 @@
-#!/usr/bin/env bats
-
-# Behavioural fixture for /wr-retrospective:migrate-briefing per ADR-052.
-# Exercises the implementation script against synthetic legacy
-# docs/BRIEFING.md fixtures in a temp dir. Asserts observable
-# file-system outcomes — per feedback_behavioural_tests.md (P081),
-# not source content.
-#
-# Closes P204 verification surface.
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../../.." && pwd)"
-  SCRIPT="$REPO_ROOT/packages/retrospective/scripts/migrate-briefing.sh"
-  ORIG_DIR="$PWD"
-  TEST_DIR=$(mktemp -d -t migrate-briefing-bats.XXXXXX)
-  cd "$TEST_DIR"
-  mkdir -p docs
-}
-
-teardown() {
-  cd "$ORIG_DIR"
-  rm -rf "$TEST_DIR"
-}
-
-# ---------------------------------------------------------------------------
-# Idempotency direction 1: no legacy file → no-op exit 0
-# ---------------------------------------------------------------------------
-
-@test "fixture: no legacy file → no-op exit 0, tree not created" {
-  run bash "$SCRIPT"
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"nothing to migrate"* ]] || [[ "$output" == *"no action"* ]]
-  [ ! -d docs/briefing ]
-}
-
-@test "fixture: empty legacy file (zero-byte) → no-op exit 0" {
-  : > docs/BRIEFING.md
-  run bash "$SCRIPT"
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"no action"* ]]
-  [ ! -d docs/briefing ]
-}
-
-# ---------------------------------------------------------------------------
-# Idempotency direction 2: tree already present → no-op exit 0
-# ---------------------------------------------------------------------------
-
-@test "fixture: tree already present → no-op exit 0, legacy untouched" {
-  mkdir -p docs/briefing
-  cat > docs/briefing/README.md <<'EOF'
-# Project Briefing
-existing content
-EOF
-  cat > docs/BRIEFING.md <<'EOF'
-# Legacy
-## Hooks
-content
-EOF
-  run bash "$SCRIPT"
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"already migrated"* ]]
-  [ -f docs/BRIEFING.md ]
-  run grep -F 'existing content' docs/briefing/README.md
-  [ "$status" -eq 0 ]
-}
-
-# ---------------------------------------------------------------------------
-# Core migration: synthetic legacy file with three H2 sections
-# ---------------------------------------------------------------------------
-
-@test "fixture: three-section legacy → three topic files + README index" {
-  cat > docs/BRIEFING.md <<'EOF'
-# Project Briefing
-
-Preamble content here.
-
-## Hooks
-
-Hook entry one.
-
-## Releases
-
-Release entry one.
-
-## Plugin Distribution
-
-Plugin distribution entry one.
-EOF
-  run bash "$SCRIPT"
-  [ "$status" -eq 0 ]
-  [ -f docs/briefing/README.md ]
-  [ -f docs/briefing/hooks.md ]
-  [ -f docs/briefing/releases.md ]
-  [ -f docs/briefing/plugin-distribution.md ]
-  run grep -F 'Hook entry one.' docs/briefing/hooks.md
-  [ "$status" -eq 0 ]
-  run grep -F 'Release entry one.' docs/briefing/releases.md
-  [ "$status" -eq 0 ]
-  run grep -F 'Plugin distribution entry one.' docs/briefing/plugin-distribution.md
-  [ "$status" -eq 0 ]
-  run grep -F '[hooks.md](./hooks.md)' docs/briefing/README.md
-  [ "$status" -eq 0 ]
-  run grep -F 'Preamble content here.' docs/briefing/README.md
-  [ "$status" -eq 0 ]
-}
-
-@test "fixture: migration retires legacy under date-stamped suffix" {
-  cat > docs/BRIEFING.md <<'EOF'
-# Project Briefing
-
-## Hooks
-
-Hook content.
-EOF
-  run bash "$SCRIPT"
-  [ "$status" -eq 0 ]
-  [ ! -f docs/BRIEFING.md ]
-  run bash -c 'ls docs/BRIEFING.md.migrated-* 2>/dev/null'
-  [ "$status" -eq 0 ]
-  [ -n "$output" ]
-}
-
-# ---------------------------------------------------------------------------
-# Idempotent re-run after successful migration: tree present → no-op
-# ---------------------------------------------------------------------------
-
-@test "fixture: re-run after successful migration → no-op exit 0" {
-  cat > docs/BRIEFING.md <<'EOF'
-# Project Briefing
-
-## Hooks
-
-Hook content.
-EOF
-  bash "$SCRIPT" >/dev/null
-  # Second run sees the tree, no-ops, and does NOT re-process the
-  # already-retired legacy file.
-  run bash "$SCRIPT"
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"already migrated"* ]]
-}
-
-# ---------------------------------------------------------------------------
-# Slug collision: two H2 sections with the same heading text
-# ---------------------------------------------------------------------------
-
-@test "fixture: duplicate H2 headings → second slug gets -2 suffix" {
-  cat > docs/BRIEFING.md <<'EOF'
-# Project Briefing
-
-## Hooks
-
-First hooks section.
-
-## Hooks
-
-Second hooks section.
-EOF
-  run bash "$SCRIPT"
-  [ "$status" -eq 0 ]
-  [ -f docs/briefing/hooks.md ]
-  [ -f docs/briefing/hooks-2.md ]
-  run grep -F 'First hooks section.' docs/briefing/hooks.md
-  [ "$status" -eq 0 ]
-  run grep -F 'Second hooks section.' docs/briefing/hooks-2.md
-  [ "$status" -eq 0 ]
-}
-
-# ---------------------------------------------------------------------------
-# Code-fence awareness: H2 inside a fenced block must NOT be promoted
-# ---------------------------------------------------------------------------
-
-@test "fixture: H2 inside fenced code block → not promoted to topic marker" {
-  cat > docs/BRIEFING.md <<'EOF'
-# Project Briefing
-
-## Real Topic
-
-Real content.
-
-```bash
-## This looks like H2 but is inside a fence
-echo "do not promote"
-```
-
-More real content after the fence.
-
-## Second Real Topic
-
-Second real content.
-EOF
-  run bash "$SCRIPT"
-  [ "$status" -eq 0 ]
-  # Only two topic files — the fenced ## did NOT create a third.
-  [ -f docs/briefing/real-topic.md ]
-  [ -f docs/briefing/second-real-topic.md ]
-  [ ! -f "docs/briefing/this-looks-like-h2-but-is-inside-a-fence.md" ]
-  # The fenced ## line is preserved inside real-topic.md.
-  run grep -F '## This looks like H2 but is inside a fence' docs/briefing/real-topic.md
-  [ "$status" -eq 0 ]
-}
-
-# ---------------------------------------------------------------------------
-# Dry-run: prints plan, does NOT write
-# ---------------------------------------------------------------------------
-
-@test "fixture: --dry-run prints plan and does NOT write tree" {
-  cat > docs/BRIEFING.md <<'EOF'
-# Project Briefing
-
-## Hooks
-
-Hook content.
-EOF
-  run bash "$SCRIPT" --dry-run
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"dry-run"* ]]
-  [[ "$output" == *"hooks"* ]]
-  [ ! -d docs/briefing ]
-  [ -f docs/BRIEFING.md ]
-}
-
-# ---------------------------------------------------------------------------
-# --force: re-runs even when tree already exists
-# ---------------------------------------------------------------------------
-
-@test "fixture: --force overrides tree-present idempotency guard" {
-  mkdir -p docs/briefing
-  echo "# stale" > docs/briefing/README.md
-  cat > docs/BRIEFING.md <<'EOF'
-# Project Briefing
-
-## Hooks
-
-Hook content.
-EOF
-  run bash "$SCRIPT" --force
-  [ "$status" -eq 0 ]
-  [ -f docs/briefing/hooks.md ]
-  run grep -F 'Migrated from legacy' docs/briefing/README.md
-  [ "$status" -eq 0 ]
-}

package/skills/run-retro/test/run-retro-anti-pattern-clause.bats

@@ -1,87 +0,0 @@
-#!/usr/bin/env bats
-
-# P088: run-retro SKILL.md MUST carry a "Never invoke as a background
-# agent" anti-pattern clause that warns the agent off the
-# Agent(run_in_background: true) surface before it commits to that
-# invocation shape. The clause is the user-direction-settled outcome
-# of P088 ((b)): foreground /wr-retrospective:run-retro is the only
-# supported invocation; `claude -p` subprocess invocation (per P086)
-# remains supported because the subprocess has the iteration's context
-# naturally; background-agent invocation is deferred pending the
-# context-marshalling problem (ADR-032 capture-retro sibling, also
-# deferred).
-#
-# # Test shape: structural contract-assertion (ADR-037 fallback path)
-#
-# The architect-review verdict on P088 (2026-04-26 iter) was:
-# **structural-with-fallback-note, ship this iter**. P081 (architect-
-# design / open) flags structural grep tests on SKILL.md prose as
-# wasteful; the behavioural alternative would programmatically simulate
-# the subagent surface, invoke run-retro, and assert the skill detects
-# the surface and emits an anti-pattern denial. That requires
-# infrastructure (mock subagent stub, mock Agent-tool harness) which
-# does not exist in this repo today.
-#
-# Per ADR-037's "permitted exception" affordance for prose-only
-# contracts, this fixture takes the structural path. P081 follow-up
-# tracks the behavioural-test infrastructure build; once P081 lands a
-# subagent-surface mock, this file's structural assertions become
-# replaceable by behavioural assertions exercising the actual surface
-# detection. Until then, structural is the contract.
-#
-# # @adr ADR-037 fallback — P081 behavioural follow-up tracked.
-# # @ticket P088 — run-retro context-visibility settlement.
-
-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 carries 'Never invoke as a background agent' anti-pattern clause (P088)" {
-  run grep -F 'Never invoke as a background agent' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: anti-pattern clause cites P088 as driver" {
-  run grep -F 'P088' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: anti-pattern clause names the supported invocation surfaces" {
-  # Foreground /wr-retrospective:run-retro — the canonical invocation.
-  run grep -F 'Foreground' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  # claude -p subprocess invocation — supported per P086 (subprocess has
-  # iteration context naturally).
-  run grep -F 'claude -p' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'P086' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: anti-pattern clause names the deferred background-agent surface explicitly" {
-  # The clause MUST mention Agent(run_in_background: true) or the
-  # capture-retro sibling so a future contributor can pattern-match
-  # the surface to the warning.
-  run grep -E 'run_in_background|capture-retro' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: anti-pattern clause appears in the preamble (before Step 1)" {
-  # The anti-pattern note belongs near the top of the SKILL so the
-  # agent encounters it before committing to an invocation surface.
-  # Placement requirement: clause appears before the '### 1. Read the
-  # current briefing' section header.
-  pos_clause=$(grep -n 'Never invoke as a background agent' "$SKILL_MD" | head -1 | cut -d: -f1)
-  pos_step1=$(grep -n '^### 1\. Read the current briefing' "$SKILL_MD" | head -1 | cut -d: -f1)
-  [ -n "$pos_clause" ]
-  [ -n "$pos_step1" ]
-  [ "$pos_clause" -lt "$pos_step1" ]
-}
-
-@test "run-retro: anti-pattern clause cross-references ADR-032 capture-retro deferral" {
-  # The clause should pin the ADR amendment so a contributor reading
-  # the SKILL can trace the deferral decision back to the ADR.
-  run grep -F 'ADR-032' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}

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

@@ -1,168 +0,0 @@
-#!/usr/bin/env bats
-# Doc-lint guard: run-retro SKILL.md must include a generalised codification
-# branch that recommends agents, hooks, and other codifiable outputs — not
-# only skills. This is the P050 generalisation of P044's single-output-type
-# recommendation surface, extended by P051 with an improvement axis for
-# existing codifiables.
-#
-# Structural assertion — Permitted Exception to the source-grep ban (ADR-005 / P011).
-# These tests assert that the skill specification document includes the
-# multi-shape codification branch introduced by P050 and the improvement-axis
-# extension introduced by P051.
-#
-# Cross-reference:
-#   P051: docs/problems/051-run-retro-does-not-recommend-improvements-to-existing-codifiables.open.md
-#   P050: docs/problems/050-run-retro-does-not-recommend-other-codifiable-outputs.known-error.md
-#   P044: docs/problems/044-run-retro-does-not-recommend-new-skills.known-error.md (predecessor)
-#   ADR-013 Rule 1 / Rule 6 (docs/decisions/013-structured-user-interaction-for-governance-decisions.proposed.md)
-#   @jtbd JTBD-101 (extend the suite with clear patterns)
-#   @jtbd JTBD-006 (progress the backlog while I'm away — AFK-safe Rule 6 fallback)
-#   @jtbd JTBD-001 (enforce governance without slowing down)
-
-setup() {
-  SKILL_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  SKILL_FILE="${SKILL_DIR}/SKILL.md"
-}
-
-@test "SKILL.md Step 2 includes a generalised codification reflection category (P050)" {
-  # P050 fix: Step 2 must prompt for recurring patterns that would be better
-  # codified — not only as skills. The generalised wording names "codif"
-  # (codification / codifiable / codified) so reviewers and agents can tell
-  # P050 shipped.
-  run grep -in "codification candidate\|codifiable\|codify\|codified" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 2 names at least three codification shapes beyond skills (P050)" {
-  # The shape question is the point of P050. Names at minimum: agent, hook,
-  # and one of: settings, script, CI step, ADR, JTBD, guide, test. "skill"
-  # stays in the list as a worked example so P044 muscle memory survives.
-  run grep -ic "agent" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  [ "$output" -ge 1 ]
-  run grep -ic "hook" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  [ "$output" -ge 1 ]
-  run grep -icE "settings|script|ci step|ADR|JTBD|guide|test fixture" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  [ "$output" -ge 1 ]
-}
-
-@test "SKILL.md Step 4b recommendation branch covers multiple shapes (P050)" {
-  # Step 4b must route per-shape. The recommendation branch names more than
-  # just "skill" as an output type.
-  run grep -inE "(agent|hook).*stub|stub.*(agent|hook)|create.*(agent|hook)|(agent|hook).*candidate" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 4b uses a single AskUserQuestion with shape-prefixed options (ADR-013 Rule 1)" {
-  # Architect decision: flat AskUserQuestion with type-prefixed labels, not a
-  # two-step chained flow. At least three shape-prefixed option lines must
-  # appear in the Step 4b block. Match both plain and backticked forms:
-  #   "Skill — ..." / "`Skill — ...`" / "**Skill** — ..."
-  run grep -cE "(\`|\*\*)?(Skill|Agent|Hook|Settings|Script|CI|ADR|JTBD|Guide|Problem|Test fixture|Memory)(\`|\*\*)? +(—|-) " "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  [ "$output" -ge 3 ]
-}
-
-@test "SKILL.md Step 4b routes ADR candidates to wr-architect:create-adr (P050)" {
-  # Dedicated codification skills already exist — Step 4b must route to them,
-  # not duplicate intake.
-  run grep -in "wr-architect:create-adr\|/wr-architect:create-adr" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 4b routes JTBD candidates to wr-jtbd:update-guide (P050)" {
-  run grep -in "wr-jtbd:update-guide\|/wr-jtbd:update-guide" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 4b preserves non-interactive fallback for generalised shapes (ADR-013 Rule 6)" {
-  # The Rule 6 fallback that P044 introduced must cover the generalised surface.
-  # When AskUserQuestion is unavailable, all shape candidates are flagged rather
-  # than silently chosen.
-  run grep -in "non-interactive\|Rule 6" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 5 summary has a Codification Candidates section with Shape column (P050)" {
-  # P050 candidate fix (3): unified table. Either:
-  #   - a "### Codification Candidates" heading AND a "Shape" column, OR
-  #   - a "### Codification Candidates" heading with per-shape rows.
-  # Accept either shape but require the heading.
-  run grep -n "### Codification Candidates\|## Codification Candidates" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  run grep -in "shape" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md retains 'skill' as a worked example within the generalised category (backward compat with P044)" {
-  # Architect advisory: keep 'skill' in the shape list as one worked example so
-  # the existing P044 muscle memory and the run-retro-skill-candidates.bats
-  # line-30 grep still pass. This is the compatibility test.
-  run grep -in "would be better as a skill\|better as a skill\|as a skill\|skill candidate" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-# ---------------------------------------------------------------------------
-# P051 improvement-axis assertions
-#
-# P051 extends the P050 codification surface with an improvement axis so
-# existing skills, agents, hooks, ADRs, and guides can be recommended for
-# targeted edits (not only new creation). The architect decision requires:
-#   - flat shape-prefixed option list (no two-step create-vs-improve question)
-#   - parallel naming (e.g. `Skill — improvement stub` mirrors
-#     `Skill — create stub`)
-#   - Kind column in the Step 5 summary table (create / improve)
-#   - non-interactive fallback records the improvement Kind alongside Shape
-# ---------------------------------------------------------------------------
-
-@test "SKILL.md Step 2 includes an improvement reflection category for existing codifiables (P051)" {
-  # P051 fix: Step 2 must prompt for flaws observed in existing skills /
-  # agents / hooks / ADRs / guides — the improvement axis. The generalised
-  # phrasing names "improvement" or "improve" alongside "codification" so
-  # reviewers and agents can tell P051 shipped.
-  run grep -inE "existing (skill|agent|hook|codifiable).*(flaw|friction|gap|improve|improvement)|improvement(-| )shaped|improvement reflection|improvement candidate|improve an existing" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 4b names improvement-shaped options for multiple shapes (P051)" {
-  # P051 fix: the flat option list must carry `Skill — improvement ...`,
-  # `Agent — improvement ...`, and `Hook — improvement ...` rows in addition
-  # to the create-stub rows introduced by P050. Match at least three
-  # shape-prefixed improvement options.
-  run grep -cE "(\`|\*\*)?(Skill|Agent|Hook|ADR|Guide|Problem)(\`|\*\*)? +(—|-) +(improvement|supersede|amend|edit)" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-  [ "$output" -ge 3 ]
-}
-
-@test "SKILL.md Step 4b routes improvement-axis ADR candidates to create-adr with supersede hint (P051)" {
-  # ADR improvement shape is "supersede or amend an existing ADR". The
-  # routing target stays `/wr-architect:create-adr` (it writes the new ADR
-  # with a supersedes reference) — P051 adds the supersede/amend wording as
-  # a shape-prefixed option so the improvement axis is recognisable.
-  # Require both: an `ADR — supersede` (or equivalent) shape-prefixed option
-  # AND a route through `wr-architect:create-adr` near that option.
-  run grep -inE "(\`|\*\*)?ADR(\`|\*\*)? +(—|-) +(supersede|amend)" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 5 summary distinguishes create from improve via a Kind column (P051)" {
-  # P051 candidate fix (3): the Codification Candidates table gains a `Kind`
-  # column carrying `create` / `improve`. Accept either a literal "Kind"
-  # column header or explicit `create / improve` values cited in the summary
-  # template. Both forms signal that the summary separates the two axes.
-  run grep -inE "\| *Kind *\||Kind column|Kind.*create.*improve|create / improve|create/improve" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}
-
-@test "SKILL.md Step 4b non-interactive fallback covers improvement candidates (P051 + ADR-013 Rule 6)" {
-  # Rule 6 fallback must mention the improvement axis explicitly so an AFK
-  # loop that flags an improvement candidate records Kind=improve alongside
-  # Shape (per architect advisory — the audit trail needs both axes for
-  # improvements). Accept either explicit Kind=improve language in the
-  # fallback block, OR a statement that the fallback records the Kind
-  # alongside Shape.
-  run grep -inE "flagged.*improve|improvement.*flagged|Kind.*(Shape|alongside)|(Shape|alongside).*Kind" "$SKILL_FILE"
-  [ "$status" -eq 0 ]
-}

package/skills/run-retro/test/run-retro-context-usage-step-2c.bats

@@ -1,156 +0,0 @@
-#!/usr/bin/env bats
-
-# P101 / ADR-043: run-retro SKILL.md gains a Step 2c (Context-usage measurement
-# — cheap layer) between Step 2b (Pipeline-instability scan) and Step 3
-# (Update the briefing tree). The cheap layer invokes
-# packages/retrospective/scripts/measure-context-budget.sh and renders a
-# per-source-bucket table in the retro summary at < 5% of the session budget.
-#
-# Doc-lint structural test (Permitted Exception per ADR-005). Asserts SKILL.md
-# wording for: the step header, citation of ADR-043, citation of ADR-026
-# grounding rule, citation of the diagnostic script path, the AFK fallback
-# (ADR-013 Rule 6) prose, the defensive-trip fail-open contract, the
-# composition-with-P099 / P105 paragraphs, and the user-direction-only
-# discipline on the deep layer.
-#
-# Behavioural assertions on the script itself live in
-# packages/retrospective/scripts/test/measure-context-budget.bats.
-
-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 2c Context-usage measurement (P101)" {
-  run grep -F '### 2c. Context-usage measurement (cheap layer, P101)' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2c cites ADR-043 as the source decision" {
-  run grep -F 'ADR-043' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2c invokes measure-context-budget.sh as the primitive" {
-  run grep -F 'packages/retrospective/scripts/measure-context-budget.sh' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2c cites ADR-026 grounding rule" {
-  run grep -F 'ADR-026' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2c bans qualitative-only phrases per ADR-026" {
-  # Forbidden phrases listed verbatim in Step 2c step 4
-  run grep -F 'load is negligible' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'microseconds only' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2c documents the defensive-trip fail-open contract" {
-  run grep -F 'cheap layer disabled' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2c AFK Rule 6 fallback prose present" {
-  run grep -F 'ADR-013 Rule 6' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'AFK behaviour' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2c documents first-retro / no-prior-snapshot path" {
-  run grep -F 'no prior snapshot' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2c cites P099 + P105 composition (no double-counting)" {
-  run grep -F 'P099' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'P105' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2c references HTML-comment trailer for snapshot persistence" {
-  run grep -F 'context-snapshot' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2c routes deep analysis to /wr-retrospective:analyze-context only on user direction" {
-  run grep -F '/wr-retrospective:analyze-context' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2c is placed between Step 2b and Step 3" {
-  # Capture line numbers for ordering check
-  step_2b_line=$(grep -n '^### 2b\.' "$SKILL_MD" | head -1 | cut -d: -f1)
-  step_2c_line=$(grep -n '^### 2c\.' "$SKILL_MD" | head -1 | cut -d: -f1)
-  step_3_line=$(grep -n '^### 3\. Update the briefing tree' "$SKILL_MD" | head -1 | cut -d: -f1)
-
-  [ -n "$step_2b_line" ]
-  [ -n "$step_2c_line" ]
-  [ -n "$step_3_line" ]
-
-  [ "$step_2b_line" -lt "$step_2c_line" ]
-  [ "$step_2c_line" -lt "$step_3_line" ]
-}
-
-# --- P295 (ADR-043 Amendment 2026-06-08) — auto-invoke wiring ---
-
-@test "run-retro: Step 2c cites ADR-043 Amendment 2026-06-08 and P295 settlement" {
-  run grep -F 'Amendment 2026-06-08' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'P295' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2c declares the combined whichever-comes-first trigger (calendar + delta)" {
-  run grep -F 'combined whichever-comes-first trigger' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'Calendar-elapse' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'Delta-breach' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2c declares the 14-day calendar threshold and 20% delta threshold" {
-  run grep -F 'older than 14 days' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'more than 20%' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2c declares the once-per-day guard via TODAY snapshot file presence" {
-  run grep -F 'Once-per-day guard' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F '<TODAY>-context-analysis.md' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2c auto-invokes /wr-retrospective:analyze-context via the Skill tool" {
-  run grep -F 'invoke `/wr-retrospective:analyze-context` via the Skill tool' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2c declares identical interactive + AFK auto-fire behaviour (silent deep layer)" {
-  run grep -F 'Identical behaviour in interactive and AFK modes' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'never invokes `AskUserQuestion`' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "run-retro: Step 2c no longer carries the superseded 'never auto-routes' / 'Never auto-fires' prose (P295 supersession)" {
-  # The 'never auto-routes' / 'Never auto-fires from this step' clauses were
-  # the on-demand-only design now reversed by ADR-043 Amendment 2026-06-08.
-  # The Step 2c block MUST NOT carry both the new auto-fire wiring AND the
-  # contradicting "never" prose — or future agents read it as still-canonical
-  # and revert the wiring. Bound the regex to the Step 2c block by checking
-  # the original on-demand-only sentinel phrases are absent from the whole
-  # SKILL.md (they appeared nowhere else).
-  run grep -F 'The cheap layer never auto-routes.' "$SKILL_MD"
-  [ "$status" -ne 0 ]
-  run grep -F 'Never auto-fires from this step.' "$SKILL_MD"
-  [ "$status" -ne 0 ]
-}