@windyroad/retrospective 0.27.1 → 0.27.2

package/scripts/test/measure-context-budget.bats

@@ -1,298 +0,0 @@
-#!/usr/bin/env bats
-
-# @problem P101 — wr-retrospective has no context-usage analysis. ADR-043
-# (Progressive context-usage measurement and reporting for retrospective
-# sessions) introduces a read-only diagnostic script
-# `packages/retrospective/scripts/measure-context-budget.sh` as the cheap-
-# layer measurement primitive. This fixture pins the script's behavioural
-# contract.
-#
-# Contract: `measure-context-budget.sh [<project-root>]` walks the
-# session's on-disk context contributors and reports per-source bucket
-# byte totals. Default project-root is $CLAUDE_PROJECT_DIR if set, else
-# the current working directory.
-#
-# Threshold default 10240 (the 5% / 200K cheap-layer envelope per ADR-043),
-# overridable via CONTEXT_BUDGET_MAX_BYTES.
-#
-# Exit codes:
-#   0 = always (advisory only — overflow is signal, not failure)
-#   2 = parse error (project root missing or unreadable)
-#
-# Output format (one line per bucket, terse machine-readable per ADR-038
-# progressive-disclosure budget — ≤150 bytes per row):
-#   BUCKET <name> bytes=<N>
-#   BUCKET <name> not-measured reason=<reason>
-#
-# Plus one trailing diagnostic row carrying the threshold:
-#   THRESHOLD bytes=<N>
-#
-# @adr ADR-043 (Progressive context-usage measurement; this script is the
-#   measurement primitive)
-# @adr ADR-038 (Progressive disclosure — per-row byte budget)
-# @adr ADR-026 (Agent output grounding — explicit not-measured sentinels)
-# @adr ADR-013 Rule 1 / Rule 6 — interactive vs AFK
-# @adr ADR-005 (Plugin testing strategy — behavioural fixture)
-# @adr ADR-037 (Skill testing strategy — bats-contract precedent)
-# @jtbd JTBD-001 / JTBD-005 / JTBD-006
-
-setup() {
-  SCRIPTS_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  SCRIPT="$SCRIPTS_DIR/measure-context-budget.sh"
-  FIXTURE_DIR="$(mktemp -d)"
-}
-
-teardown() {
-  rm -rf "$FIXTURE_DIR"
-}
-
-# ── Existence + executable ──────────────────────────────────────────────────
-
-@test "measure-context-budget: script file exists at expected path" {
-  [ -f "$SCRIPT" ]
-}
-
-@test "measure-context-budget: script is executable" {
-  [ -x "$SCRIPT" ]
-}
-
-# ── Exit codes ──────────────────────────────────────────────────────────────
-
-@test "measure-context-budget: empty project root exits 0 (advisory)" {
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  [ "$status" -eq 0 ]
-}
-
-@test "measure-context-budget: missing project root exits 2 (parse error)" {
-  run bash "$SCRIPT" "/path/that/does/not/exist/zz_$$"
-  [ "$status" -eq 2 ]
-}
-
-# ── Output shape — every run emits all buckets ──────────────────────────────
-
-@test "measure-context-budget: output contains hooks bucket row" {
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  [ "$status" -eq 0 ]
-  echo "$output" | grep -q '^BUCKET hooks '
-}
-
-@test "measure-context-budget: output contains skills bucket row" {
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  echo "$output" | grep -q '^BUCKET skills '
-}
-
-@test "measure-context-budget: output contains briefing bucket row" {
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  echo "$output" | grep -q '^BUCKET briefing '
-}
-
-@test "measure-context-budget: output contains decisions bucket row" {
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  echo "$output" | grep -q '^BUCKET decisions '
-}
-
-@test "measure-context-budget: output contains problems bucket row" {
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  echo "$output" | grep -q '^BUCKET problems '
-}
-
-@test "measure-context-budget: output contains jtbd bucket row" {
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  echo "$output" | grep -q '^BUCKET jtbd '
-}
-
-@test "measure-context-budget: output contains project-claude-md bucket row" {
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  echo "$output" | grep -q '^BUCKET project-claude-md '
-}
-
-@test "measure-context-budget: output contains memory bucket row" {
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  echo "$output" | grep -q '^BUCKET memory '
-}
-
-# ── Framework-injected sentinel (ADR-026 ungrounded-field rule) ─────────────
-
-@test "measure-context-budget: framework-injected bucket emits not-measured sentinel" {
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  [ "$status" -eq 0 ]
-  echo "$output" | grep -q '^BUCKET framework-injected not-measured '
-  echo "$output" | grep -q 'reason=framework-injected-no-on-disk-source'
-}
-
-# ── Surface-absent sentinels (ADR-026 ungrounded-field rule) ────────────────
-# Empty fixture has no docs/decisions/, docs/problems/, docs/jtbd/, etc.
-
-@test "measure-context-budget: empty fixture marks decisions not-measured" {
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  echo "$output" | grep -q '^BUCKET decisions not-measured '
-}
-
-@test "measure-context-budget: empty fixture marks problems not-measured" {
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  echo "$output" | grep -q '^BUCKET problems not-measured '
-}
-
-@test "measure-context-budget: empty fixture marks jtbd not-measured" {
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  echo "$output" | grep -q '^BUCKET jtbd not-measured '
-}
-
-@test "measure-context-budget: empty fixture marks briefing not-measured" {
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  echo "$output" | grep -q '^BUCKET briefing not-measured '
-}
-
-@test "measure-context-budget: empty fixture marks project-claude-md not-measured" {
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  echo "$output" | grep -q '^BUCKET project-claude-md not-measured '
-}
-
-# ── Surface-present byte counts ─────────────────────────────────────────────
-
-@test "measure-context-budget: populated decisions bucket reports byte count" {
-  mkdir -p "$FIXTURE_DIR/docs/decisions"
-  printf '# ADR-001\nbody body body\n' > "$FIXTURE_DIR/docs/decisions/001-foo.proposed.md"
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  [ "$status" -eq 0 ]
-  echo "$output" | grep -qE '^BUCKET decisions bytes=[0-9]+$'
-  # Sanity: byte count is non-zero
-  decisions_line=$(echo "$output" | grep '^BUCKET decisions ')
-  bytes_value="${decisions_line##*bytes=}"
-  [ "$bytes_value" -gt 0 ]
-}
-
-@test "measure-context-budget: populated problems bucket reports byte count" {
-  mkdir -p "$FIXTURE_DIR/docs/problems"
-  printf '# Problem 001\nbody\n' > "$FIXTURE_DIR/docs/problems/001-foo.open.md"
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  echo "$output" | grep -qE '^BUCKET problems bytes=[0-9]+$'
-}
-
-# ── Dual-tolerant per-state subdir enumeration (P182 / RFC-002 T4 / ADR-031) ─
-# RFC-002 T5 migrated problem tickets from the flat layout
-# (docs/problems/<NNN>-*.<state>.md) to per-state subdirs
-# (docs/problems/<state>/<NNN>-*.md). The flat-only glob misses the subdir
-# tickets and under-counts the bucket ~99% post-migration (P182). The fix
-# walks BOTH layouts and dedups on ticket ID — the per-state subdir copy
-# wins on collision per ADR-031 §"Authoritative state signal" — mirroring
-# the proven reconcile-readme.sh pattern.
-
-@test "measure-context-budget: per-state subdir problem tickets are counted" {
-  mkdir -p "$FIXTURE_DIR/docs/problems/open" "$FIXTURE_DIR/docs/problems/known-error"
-  printf '# Problem 100\nbody body body\n' > "$FIXTURE_DIR/docs/problems/open/100-foo.md"
-  printf '# Problem 101\nbody body\n' > "$FIXTURE_DIR/docs/problems/known-error/101-bar.md"
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  [ "$status" -eq 0 ]
-  echo "$output" | grep -qE '^BUCKET problems bytes=[0-9]+$'
-  problems_line=$(echo "$output" | grep '^BUCKET problems ')
-  bytes_value="${problems_line##*bytes=}"
-  [ "$bytes_value" -gt 0 ]
-}
-
-@test "measure-context-budget: problems bucket sums flat + per-state subdir tickets" {
-  mkdir -p "$FIXTURE_DIR/docs/problems/open" "$FIXTURE_DIR/docs/problems/verifying"
-  # Flat top-level (README + a pre-migration flat ticket) and per-state
-  # subdir tickets must all contribute to the bucket total.
-  printf '# Problems index\nrow\n' > "$FIXTURE_DIR/docs/problems/README.md"
-  printf '# Problem 050 legacy flat\nbody\n' > "$FIXTURE_DIR/docs/problems/050-legacy.open.md"
-  printf '# Problem 100\nbody body\n' > "$FIXTURE_DIR/docs/problems/open/100-foo.md"
-  printf '# Problem 101\nbody body body body\n' > "$FIXTURE_DIR/docs/problems/verifying/101-bar.md"
-  expected=0
-  for f in "$FIXTURE_DIR/docs/problems/README.md" \
-           "$FIXTURE_DIR/docs/problems/050-legacy.open.md" \
-           "$FIXTURE_DIR/docs/problems/open/100-foo.md" \
-           "$FIXTURE_DIR/docs/problems/verifying/101-bar.md"; do
-    b=$(wc -c < "$f" | tr -d ' ')
-    expected=$(( expected + b ))
-  done
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  [ "$status" -eq 0 ]
-  problems_line=$(echo "$output" | grep '^BUCKET problems ')
-  bytes_value="${problems_line##*bytes=}"
-  [ "$bytes_value" -eq "$expected" ]
-}
-
-@test "measure-context-budget: same ticket in flat + per-state subdir counted once (per-state wins)" {
-  # Mid-migration race: the same ticket ID exists as a flat file
-  # (200-dup.open.md) AND a per-state subdir file (open/200-dup.md) with a
-  # DIFFERENT byte size. ID-keyed dedup must count it once; the per-state
-  # subdir copy wins (ADR-031). Asserting the total equals the subdir file's
-  # size alone proves both: counted-once (not flat+subdir) and per-state-wins
-  # (subdir size, not flat size).
-  mkdir -p "$FIXTURE_DIR/docs/problems/open"
-  printf 'flat\n' > "$FIXTURE_DIR/docs/problems/200-dup.open.md"
-  printf 'per-state subdir copy is longer\n' > "$FIXTURE_DIR/docs/problems/open/200-dup.md"
-  subdir_bytes=$(wc -c < "$FIXTURE_DIR/docs/problems/open/200-dup.md" | tr -d ' ')
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  [ "$status" -eq 0 ]
-  problems_line=$(echo "$output" | grep '^BUCKET problems ')
-  bytes_value="${problems_line##*bytes=}"
-  [ "$bytes_value" -eq "$subdir_bytes" ]
-}
-
-@test "measure-context-budget: populated briefing bucket reports byte count" {
-  mkdir -p "$FIXTURE_DIR/docs/briefing"
-  printf '# Topic\nentry\n' > "$FIXTURE_DIR/docs/briefing/foo.md"
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  echo "$output" | grep -qE '^BUCKET briefing bytes=[0-9]+$'
-}
-
-@test "measure-context-budget: project CLAUDE.md reports byte count" {
-  printf '# Project\ninstructions\n' > "$FIXTURE_DIR/CLAUDE.md"
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  echo "$output" | grep -qE '^BUCKET project-claude-md bytes=[0-9]+$'
-  claude_md_line=$(echo "$output" | grep '^BUCKET project-claude-md ')
-  bytes_value="${claude_md_line##*bytes=}"
-  [ "$bytes_value" -gt 0 ]
-}
-
-# ── Threshold ────────────────────────────────────────────────────────────────
-
-@test "measure-context-budget: trailing THRESHOLD row emitted" {
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  [ "$status" -eq 0 ]
-  echo "$output" | grep -qE '^THRESHOLD bytes=[0-9]+$'
-}
-
-@test "measure-context-budget: default threshold is 10240" {
-  unset CONTEXT_BUDGET_MAX_BYTES
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  echo "$output" | grep -q '^THRESHOLD bytes=10240$'
-}
-
-@test "measure-context-budget: CONTEXT_BUDGET_MAX_BYTES override respected" {
-  CONTEXT_BUDGET_MAX_BYTES=42 run bash "$SCRIPT" "$FIXTURE_DIR"
-  [ "$status" -eq 0 ]
-  echo "$output" | grep -q '^THRESHOLD bytes=42$'
-}
-
-# ── Per-row byte budget (ADR-038 progressive-disclosure ≤150 bytes/row) ─────
-
-@test "measure-context-budget: every row is at most 150 bytes" {
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  [ "$status" -eq 0 ]
-  while IFS= read -r line; do
-    [ "${#line}" -le 150 ]
-  done <<< "$output"
-}
-
-# ── Read-only contract — script does not mutate the project tree ────────────
-
-@test "measure-context-budget: read-only — fixture tree unchanged after run" {
-  mkdir -p "$FIXTURE_DIR/docs/decisions"
-  printf '# ADR-001\nbody\n' > "$FIXTURE_DIR/docs/decisions/001-foo.proposed.md"
-  pre_hash=$(find "$FIXTURE_DIR" -type f -exec cksum {} \; 2>/dev/null | sort | cksum | awk '{print $1}')
-  run bash "$SCRIPT" "$FIXTURE_DIR"
-  [ "$status" -eq 0 ]
-  post_hash=$(find "$FIXTURE_DIR" -type f -exec cksum {} \; 2>/dev/null | sort | cksum | awk '{print $1}')
-  [ "$pre_hash" = "$post_hash" ]
-}
-
-# ── CLAUDE_PROJECT_DIR fallback when no arg ─────────────────────────────────
-
-@test "measure-context-budget: CLAUDE_PROJECT_DIR env var respected when no arg" {
-  CLAUDE_PROJECT_DIR="$FIXTURE_DIR" run bash "$SCRIPT"
-  [ "$status" -eq 0 ]
-  echo "$output" | grep -q '^BUCKET hooks '
-}

package/skills/analyze-context/test/analyze-context-skill-contract.bats

@@ -1,142 +0,0 @@
-#!/usr/bin/env bats
-
-# P101 / ADR-043: new skill /wr-retrospective:analyze-context (deep layer)
-# at packages/retrospective/skills/analyze-context/SKILL.md. Doc-lint
-# structural test (Permitted Exception per ADR-005). Asserts the SKILL.md
-# carries: the canonical name, citations of ADR-043 / ADR-026 / ADR-014 /
-# ADR-013, references to the cheap-layer measurement primitive, the report
-# path convention, the HTML-comment-trailer snapshot schema, the AFK
-# never-auto-fires discipline, the ADR-014 commit-message convention, and
-# the ADR-026 forbidden-qualitative-phrase ban.
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../../.." && pwd)"
-  SKILL_MD="$REPO_ROOT/packages/retrospective/skills/analyze-context/SKILL.md"
-}
-
-@test "analyze-context: SKILL.md exists at expected path" {
-  [ -f "$SKILL_MD" ]
-}
-
-@test "analyze-context: frontmatter declares the wr-retrospective:analyze-context name" {
-  run grep -F 'name: wr-retrospective:analyze-context' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "analyze-context: cites ADR-043 as the source decision" {
-  run grep -F 'ADR-043' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "analyze-context: cites ADR-026 (Agent output grounding)" {
-  run grep -F 'ADR-026' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "analyze-context: cites ADR-014 (Governance skills commit own work)" {
-  run grep -F 'ADR-014' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "analyze-context: cites ADR-013 Rule 6 (AFK fallback)" {
-  run grep -F 'ADR-013 Rule 6' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "analyze-context: references the cheap-layer measurement primitive" {
-  run grep -F 'packages/retrospective/scripts/measure-context-budget.sh' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "analyze-context: declares the report path convention docs/retros/<date>-context-analysis.md" {
-  run grep -F 'docs/retros/' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  # `--` separates flags from pattern; `-context-analysis.md` would otherwise
-  # parse as an option flag.
-  run grep -F -- '-context-analysis.md' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "analyze-context: HTML-comment trailer schema present (context-snapshot)" {
-  run grep -F 'context-snapshot:' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "analyze-context: trailer schema cites measurement-method and measured-at fields" {
-  run grep -F 'measurement-method' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'measured-at' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "analyze-context: auto-fires from run-retro Step 2c per ADR-043 Amendment 2026-06-08 (P295)" {
-  # P295 settlement: deep layer auto-fires from cheap-layer Step 2c when the
-  # combined trigger holds. Replaces the prior 'never auto-invoked' clause.
-  run grep -iF 'auto-fires from run-retro Step 2c' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'Amendment 2026-06-08' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "analyze-context: SKILL.md no longer asserts 'never auto-invoked' / 'never auto-fires' (P295 supersedes)" {
-  # The supersession discipline: the post-P295 SKILL.md MUST NOT carry the
-  # contradicting 'never auto-fires' / 'never auto-invoked' prose, or future
-  # agents will read it as still-authoritative and revert the wiring.
-  run grep -F 'never auto-invoked' "$SKILL_MD"
-  [ "$status" -ne 0 ]
-  run grep -F 'Never auto-fires' "$SKILL_MD"
-  [ "$status" -ne 0 ]
-}
-
-@test "analyze-context: cites the combined-trigger threshold values (14 days + 20%)" {
-  # ADR-043 Amendment 2026-06-08 chosen initial thresholds per ADR-026
-  # grounding rule (concrete numerics + no-prior-data sentinel).
-  run grep -F '14 days' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F '20%' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "analyze-context: cites 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 "analyze-context: ADR-014 commit-message convention declared (docs(retros): context analysis)" {
-  run grep -F 'docs(retros): context analysis' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "analyze-context: bans qualitative-only phrases per ADR-026" {
-  run grep -F 'load is negligible' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -F 'microseconds only' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "analyze-context: requires comparable-prior citation in suggestions" {
-  run grep -F 'comparable prior' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "analyze-context: routes to /wr-risk-scorer:assess-release fallback when subagent unavailable" {
-  run grep -F '/wr-risk-scorer:assess-release' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "analyze-context: documents per-plugin decomposition step" {
-  run grep -F 'Per-Plugin Decomposition' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "analyze-context: documents policy-breach detection step" {
-  run grep -F 'Policy Breaches' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "analyze-context: cites P101 (driver ticket) and P091 (parent meta)" {
-  run grep -F 'P101' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}

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

@@ -1,105 +0,0 @@
-#!/usr/bin/env bats
-
-# Contract-level structural tests for /wr-retrospective:migrate-briefing.
-# Permitted Exception per ADR-005 — structural SKILL.md content checks,
-# mirrored on scaffold-intake-contract.bats. Behavioural coverage lives
-# in migrate-briefing-fixture.bats per ADR-052.
-#
-# Closes P204 verification surface.
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../../.." && pwd)"
-  SKILL_DIR="$REPO_ROOT/packages/retrospective/skills/migrate-briefing"
-  SKILL_MD="$SKILL_DIR/SKILL.md"
-  REFERENCE_MD="$SKILL_DIR/REFERENCE.md"
-  SCRIPT="$REPO_ROOT/packages/retrospective/scripts/migrate-briefing.sh"
-  SHIM="$REPO_ROOT/packages/retrospective/bin/wr-retrospective-migrate-briefing"
-}
-
-@test "migrate-briefing: SKILL.md exists" {
-  [ -f "$SKILL_MD" ]
-}
-
-@test "migrate-briefing: REFERENCE.md exists (ADR-038 split)" {
-  [ -f "$REFERENCE_MD" ]
-}
-
-@test "migrate-briefing: implementation script exists and is executable" {
-  [ -x "$SCRIPT" ]
-}
-
-@test "migrate-briefing: bin shim exists and is executable" {
-  [ -x "$SHIM" ]
-}
-
-@test "migrate-briefing: SKILL.md frontmatter declares the skill name" {
-  run grep -F 'name: wr-retrospective:migrate-briefing' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "migrate-briefing: SKILL.md cites ADR-040 (target tree shape — load-bearing per architect)" {
-  run grep -F 'ADR-040' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "migrate-briefing: SKILL.md cites ADR-032 foreground-synchronous pattern" {
-  run grep -F 'ADR-032' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -iE 'foreground.synchronous' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "migrate-briefing: SKILL.md cites ADR-014 self-commit pattern" {
-  run grep -F 'ADR-014' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "migrate-briefing: SKILL.md cites ADR-038 progressive-disclosure pattern" {
-  run grep -F 'ADR-038' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "migrate-briefing: SKILL.md cites ADR-052 behavioural-tests-default" {
-  run grep -F 'ADR-052' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "migrate-briefing: SKILL.md cites ADR-049 (no repo-relative paths — recurring class)" {
-  run grep -F 'ADR-049' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "migrate-briefing: SKILL.md declares two-direction idempotency clause" {
-  # Both no-op directions named: tree already present + no legacy file.
-  run grep -iE 'tree.already.present|already.migrated' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-  run grep -iE 'no.legacy.file|legacy.*does.not.exist|missing or empty' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "migrate-briefing: SKILL.md carries Rule 6 audit section (ADR-013)" {
-  run grep -F 'Rule 6' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "migrate-briefing: SKILL.md cross-references P204 (the closing ticket)" {
-  run grep -F 'P204' "$SKILL_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "migrate-briefing: SKILL.md does NOT carry repo-relative packages/ paths (P151/P153/P219/P317 class)" {
-  # No bash invocations that hardcode a packages/.../scripts/ path —
-  # the script ships via ADR-049 PATH shim.
-  run grep -E 'packages/[a-z]+/scripts/migrate-briefing\.sh' "$SKILL_MD"
-  [ "$status" -ne 0 ]
-}
-
-@test "migrate-briefing: REFERENCE.md documents the heading-extraction algorithm" {
-  run grep -iE 'heading.extraction|slug.derivation|collision' "$REFERENCE_MD"
-  [ "$status" -eq 0 ]
-}
-
-@test "migrate-briefing: REFERENCE.md documents code-fence-aware parsing" {
-  run grep -iE 'fence|code.fence|in_fence' "$REFERENCE_MD"
-  [ "$status" -eq 0 ]
-}