@windyroad/architect 0.20.3 → 0.20.4

package/hooks/test/architect-detect-once-per-session.bats

@@ -1,106 +0,0 @@
-#!/usr/bin/env bats
-
-# P095 / ADR-038: architect-detect.sh UserPromptSubmit hook must emit
-# the full MANDATORY instruction block only on the first prompt of a
-# session (identified by session_id on stdin). Subsequent prompts in
-# the same session emit only a terse reminder. New sessions or empty
-# session_ids fall back to the full block.
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../.." && pwd)"
-  HOOK="$REPO_ROOT/packages/architect/hooks/architect-detect.sh"
-
-  # Stub project working dir so the hook's docs/decisions/ detection fires.
-  WORKDIR="$(mktemp -d)"
-  mkdir -p "$WORKDIR/docs/decisions"
-  : > "$WORKDIR/docs/decisions/.keep"
-
-  SID="arch-detect-test-$$-$RANDOM"
-}
-
-teardown() {
-  rm -f "/tmp/architect-announced-${SID}"
-  rm -f "/tmp/architect-announced-${SID}-alt"
-  rm -rf "$WORKDIR"
-}
-
-# Helper: invoke the hook with a given session_id from stdin, while cd'd
-# into WORKDIR so the `docs/decisions` detection picks up the stub.
-run_hook() {
-  local sid="$1"
-  (cd "$WORKDIR" && echo "{\"session_id\":\"$sid\"}" | bash "$HOOK")
-}
-
-@test "architect-detect: first invocation emits the full MANDATORY block" {
-  run run_hook "$SID"
-  [ "$status" -eq 0 ]
-  [ "${#output}" -gt 500 ]
-  [[ "$output" == *"MANDATORY ARCHITECTURE CHECK"* ]]
-  [[ "$output" == *"wr-architect:agent"* ]]
-  [[ "$output" == *"SCOPE:"* ]]
-}
-
-@test "architect-detect: first invocation writes the announcement marker" {
-  run run_hook "$SID"
-  [ "$status" -eq 0 ]
-  [ -f "/tmp/architect-announced-${SID}" ]
-}
-
-@test "architect-detect: second invocation emits only a terse reminder" {
-  run_hook "$SID" >/dev/null
-  run run_hook "$SID"
-  [ "$status" -eq 0 ]
-  # Terse reminder is < 250 bytes; full block is > 500 bytes.
-  [ "${#output}" -lt 250 ]
-  # Terse reminder names the gate AND the delegation affordance.
-  [[ "$output" == *"architecture"* ]] || [[ "$output" == *"ARCHITECTURE"* ]]
-  [[ "$output" == *"wr-architect:agent"* ]]
-  # Full SCOPE block is NOT re-emitted.
-  [[ "$output" != *"Does NOT apply to"* ]]
-}
-
-@test "architect-detect: terse reminder preserves the MANDATORY signal word" {
-  run_hook "$SID" >/dev/null
-  run run_hook "$SID"
-  [ "$status" -eq 0 ]
-  # Per JTBD-lead condition: terse reminder must keep an imperative
-  # marker (MANDATORY / REQUIRED / NON-OPTIONAL) so the enforcement
-  # signal does not soften after turn 1.
-  [[ "$output" == *"MANDATORY"* ]] || [[ "$output" == *"REQUIRED"* ]] || [[ "$output" == *"NON-OPTIONAL"* ]]
-}
-
-@test "architect-detect: terse reminder names the trigger artifact" {
-  run_hook "$SID" >/dev/null
-  run run_hook "$SID"
-  [ "$status" -eq 0 ]
-  # Per JTBD-lead condition: reason for the gate must stay visible
-  # on subsequent turns.
-  [[ "$output" == *"docs/decisions"* ]]
-}
-
-@test "architect-detect: different session_id re-emits the full block" {
-  run_hook "$SID" >/dev/null
-  local SID2="${SID}-alt"
-  run run_hook "$SID2"
-  [ "$status" -eq 0 ]
-  [ "${#output}" -gt 500 ]
-  [[ "$output" == *"MANDATORY ARCHITECTURE CHECK"* ]]
-  rm -f "/tmp/architect-announced-${SID2}"
-}
-
-@test "architect-detect: empty session_id emits the full block and writes no marker" {
-  run run_hook ""
-  [ "$status" -eq 0 ]
-  [ "${#output}" -gt 500 ]
-  [[ "$output" == *"MANDATORY ARCHITECTURE CHECK"* ]]
-  [ ! -f "/tmp/architect-announced-" ]
-}
-
-@test "architect-detect: absent docs/decisions/ emits the no-decisions NOTE (unchanged by this ADR)" {
-  local NO_DECISIONS="$(mktemp -d)"
-  run bash -c "cd '$NO_DECISIONS' && echo '{\"session_id\":\"$SID\"}' | bash '$HOOK'"
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"no docs/decisions/ directory"* ]]
-  [[ "$output" != *"MANDATORY ARCHITECTURE CHECK"* ]]
-  rm -rf "$NO_DECISIONS"
-}

package/hooks/test/architect-detect-scope.bats

@@ -1,56 +0,0 @@
-#!/usr/bin/env bats
-
-# Tests for architect-detect.sh (UserPromptSubmit) — verifies the injected
-# scope exclusion text lists governance docs that the PreToolUse gate already
-# exempts (P029). Without this, the LLM wastes time delegating to the
-# architect for files the edit gate would allow anyway.
-
-setup() {
-  SCRIPT_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  HOOK="$SCRIPT_DIR/architect-detect.sh"
-  ORIG_DIR="$PWD"
-  TEST_DIR=$(mktemp -d)
-  cd "$TEST_DIR"
-  mkdir -p docs/decisions
-}
-
-teardown() {
-  cd "$ORIG_DIR"
-  rm -rf "$TEST_DIR"
-}
-
-@test "detect: scope text mentions problem files exemption (P029)" {
-  run bash "$HOOK" </dev/null
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"docs/problems/"* ]] || [[ "$output" == *"problem tickets"* ]]
-}
-
-@test "detect: scope text mentions BRIEFING.md exemption (P029)" {
-  run bash "$HOOK" </dev/null
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"BRIEFING"* ]]
-}
-
-@test "detect: scope text mentions docs/briefing/ exemption (P100)" {
-  run bash "$HOOK" </dev/null
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"docs/briefing/"* ]]
-}
-
-@test "detect: scope text mentions RISK-POLICY exemption (P029)" {
-  run bash "$HOOK" </dev/null
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"RISK-POLICY"* ]]
-}
-
-@test "detect: scope text mentions changeset exemption (P029)" {
-  run bash "$HOOK" </dev/null
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"changeset"* ]]
-}
-
-@test "detect: scope text mentions memory files exemption (P029)" {
-  run bash "$HOOK" </dev/null
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"memory"* ]] || [[ "$output" == *"MEMORY"* ]]
-}

package/hooks/test/architect-enforce-scope.bats

@@ -1,124 +0,0 @@
-#!/usr/bin/env bats
-
-# Tests for architect-enforce-edit.sh — verifies peer-plugin policy files are
-# exempt from the architect gate (P009). Each plugin governs its own policy
-# docs via its own enforce hook; the architect should not re-gate them.
-#
-# All tests are functional (P011): they execute the hook with mock JSON
-# input and assert on exit status + BLOCKED output. Source-grep assertions
-# were removed because they over-specified the implementation and gave
-# false confidence (a literal string can appear in source without the
-# matching case branch actually short-circuiting).
-
-setup() {
-  SCRIPT_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  HOOK="$SCRIPT_DIR/architect-enforce-edit.sh"
-  ORIG_DIR="$PWD"
-  TEST_DIR=$(mktemp -d)
-  cd "$TEST_DIR"
-  # Engage the gate: architect-enforce only runs when docs/decisions/ exists.
-  mkdir -p docs/decisions
-}
-
-teardown() {
-  cd "$ORIG_DIR"
-  rm -rf "$TEST_DIR"
-}
-
-# Helper: run the hook with a mock JSON input for a given file path.
-# Claude Code passes absolute file_path values, so tests use $PWD-prefixed
-# paths to match real shape (after the P004 root check).
-run_hook_with_file() {
-  local file_path="$1"
-  local json="{\"tool_input\":{\"file_path\":\"${file_path}\"},\"session_id\":\"test-session-$$\"}"
-  echo "$json" | bash "$HOOK"
-}
-
-assert_path_allowed() {
-  local file_path="$1"
-  run run_hook_with_file "$file_path"
-  [ "$status" -eq 0 ]
-  [[ "$output" != *"BLOCKED"* ]]
-}
-
-@test "architect: does NOT exempt legacy docs/JOBS_TO_BE_DONE.md (ADR-008 Option 3, P019)" {
-  # Legacy single-file path is no longer a recognised peer-plugin policy
-  # artefact — ADR-008 Option 3 removes it from runtime consideration.
-  # The architect gate treats it as ordinary content and applies its
-  # normal review rules.
-  run run_hook_with_file "$PWD/docs/JOBS_TO_BE_DONE.md"
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"BLOCKED"* ]]
-}
-
-@test "architect: exempts VCS-internal .git/ plumbing files (P458)" {
-  # A commit-message scratch file under .git/ is not a project artefact. Without
-  # the exclusion a .txt under the repo root is gated (no extension exemption).
-  assert_path_allowed "$PWD/.git/cruise-brake-commit-msg.txt"
-  assert_path_allowed "$PWD/.git/COMMIT_EDITMSG"
-}
-
-@test "architect: exempts JTBD directory file (P009)" {
-  assert_path_allowed "$PWD/docs/jtbd/solo-developer/persona.md"
-}
-
-@test "architect: exempts PRODUCT_DISCOVERY.md (P009)" {
-  assert_path_allowed "$PWD/docs/PRODUCT_DISCOVERY.md"
-}
-
-@test "architect: exempts voice-tone policy file (P009)" {
-  assert_path_allowed "$PWD/docs/VOICE-AND-TONE.md"
-}
-
-@test "architect: exempts style-guide policy file (P009)" {
-  assert_path_allowed "$PWD/docs/STYLE-GUIDE.md"
-}
-
-# --- Story maps + stories exemptions (P170 Phase 2 Slice 2.5 / ADR-060 amendment 2026-05-12) ---
-# ADR-060 § Phase 2 encoding amendment lines 481-496 mandates exempting
-# `docs/story-maps/**/*.html` and `docs/stories/**/*.md` from the 5
-# enforce-edit gates. Mirrors the docs/problems and docs/jtbd exemptions —
-# these are governance-managed surfaces with their own capture/manage skills.
-
-@test "architect: exempts docs/story-maps/ HTML in per-state subdir (P170 / ADR-060)" {
-  assert_path_allowed "$PWD/docs/story-maps/draft/STORY-MAP-001-rfc-framework.html"
-}
-
-@test "architect: exempts docs/story-maps/ HTML in completed subdir (P170 / ADR-060)" {
-  assert_path_allowed "$PWD/docs/story-maps/completed/STORY-MAP-002-foo.html"
-}
-
-@test "architect: exempts docs/story-maps/README.md (P170 / ADR-060)" {
-  assert_path_allowed "$PWD/docs/story-maps/README.md"
-}
-
-@test "architect: exempts docs/stories/ markdown in per-state subdir (P170 / ADR-060)" {
-  assert_path_allowed "$PWD/docs/stories/draft/STORY-001-some-slice.md"
-}
-
-@test "architect: exempts docs/stories/ markdown in done subdir (P170 / ADR-060)" {
-  assert_path_allowed "$PWD/docs/stories/done/STORY-002-other.md"
-}
-
-@test "architect: exempts docs/stories/README.md (P170 / ADR-060)" {
-  assert_path_allowed "$PWD/docs/stories/README.md"
-}
-
-# --- Retros exemption (P203) ---
-# `docs/retros/*` is the ask-hygiene + run-retro narrative trail written
-# routinely by `/wr-retrospective:run-retro` Step 2d + Step 5. Routine
-# appends should not fire the architect gate — they are not load-bearing
-# architecture content. Mirrors the docs/problems and docs/jtbd peer-
-# plugin-policy exemptions.
-
-@test "architect: exempts docs/retros/ ask-hygiene file (P203)" {
-  assert_path_allowed "$PWD/docs/retros/2026-06-06-some-iter-ask-hygiene.md"
-}
-
-@test "architect: exempts docs/retros/ analyze-context file (P203)" {
-  assert_path_allowed "$PWD/docs/retros/2026-06-06-context-analysis.md"
-}
-
-@test "architect: exempts docs/retros/ retro narrative file (P203)" {
-  assert_path_allowed "$PWD/docs/retros/2026-06-06-session-wrap.md"
-}

package/hooks/test/architect-gate.bats

@@ -1,102 +0,0 @@
-#!/usr/bin/env bats
-
-# Tests for architect-gate.sh (TTL, drift, marker)
-
-setup() {
-  SCRIPT_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  source "$SCRIPT_DIR/lib/architect-gate.sh"
-  TEST_SESSION="test-$$-$BATS_TEST_NUMBER"
-}
-
-teardown() {
-  rm -f "/tmp/architect-reviewed-${TEST_SESSION}"
-  rm -f "/tmp/architect-reviewed-${TEST_SESSION}.hash"
-}
-
-@test "gate denies when no marker exists" {
-  run check_architect_gate "$TEST_SESSION"
-  [ "$status" -ne 0 ]
-}
-
-@test "gate allows when marker exists and is fresh" {
-  touch "/tmp/architect-reviewed-${TEST_SESSION}"
-  run check_architect_gate "$TEST_SESSION"
-  [ "$status" -eq 0 ]
-}
-
-@test "gate denies when marker is expired" {
-  touch "/tmp/architect-reviewed-${TEST_SESSION}"
-  # Set TTL to 0 to force expiry
-  ARCHITECT_TTL=0 run check_architect_gate "$TEST_SESSION"
-  [ "$status" -ne 0 ]
-}
-
-# P215 / RFC-021 — ARCHITECT_GATE_REASON behaviour. The gate must expose a
-# differentiated reason per failure mode (no marker / TTL expired / drift
-# detected) so the downstream deny message carries a clear recovery directive.
-# Mirrors sibling REVIEW_GATE_REASON pattern in jtbd/voice-tone/style-guide
-# review-gate.sh.
-
-@test "ARCHITECT_GATE_REASON names re-delegate directive when no marker" {
-  ARCHITECT_GATE_REASON=""
-  check_architect_gate "$TEST_SESSION" || true
-  [[ "$ARCHITECT_GATE_REASON" == *"wr-architect:agent"* ]]
-  [[ "$ARCHITECT_GATE_REASON" == *"Agent tool"* ]]
-}
-
-# P400 — the no-marker reason must also document the SendMessage-resume
-# recovery path: a verdict upgrade after ISSUES FOUND is a FRESH Agent spawn
-# (a SendMessage resume does NOT fire the marker hook), or the manual marker
-# assertion. Without this directive the deny message is a dead-end for the
-# most common recovery flow.
-@test "ARCHITECT_GATE_REASON documents SendMessage-resume recovery when no marker" {
-  ARCHITECT_GATE_REASON=""
-  check_architect_gate "$TEST_SESSION" || true
-  [[ "$ARCHITECT_GATE_REASON" == *"SendMessage"* ]]
-  [[ "$ARCHITECT_GATE_REASON" == *"FRESH Agent spawn"* ]]
-  [[ "$ARCHITECT_GATE_REASON" == *"touch /tmp/architect-reviewed-"* ]]
-  [[ "$ARCHITECT_GATE_REASON" == *"rm -f /tmp/architect-reviewed-"* ]]
-}
-
-@test "ARCHITECT_GATE_REASON names re-delegate directive when TTL expired" {
-  touch "/tmp/architect-reviewed-${TEST_SESSION}"
-  ARCHITECT_GATE_REASON=""
-  ARCHITECT_TTL=0 check_architect_gate "$TEST_SESSION" || true
-  [[ "$ARCHITECT_GATE_REASON" == *"expired"* ]]
-  [[ "$ARCHITECT_GATE_REASON" == *"wr-architect:agent"* ]]
-  [[ "$ARCHITECT_GATE_REASON" == *"refresh the marker"* ]]
-}
-
-@test "ARCHITECT_GATE_REASON names drift directive when stored hash differs" {
-  # Set up project root with a docs/decisions directory and a stored hash that
-  # differs from the current substance hash to force the drift branch.
-  TEST_PROJECT_DIR=$(mktemp -d)
-  mkdir -p "$TEST_PROJECT_DIR/docs/decisions"
-  echo "# ADR-001 current content" > "$TEST_PROJECT_DIR/docs/decisions/001-x.md"
-  touch "/tmp/architect-reviewed-${TEST_SESSION}"
-  echo "stale-hash-that-will-not-match" > "/tmp/architect-reviewed-${TEST_SESSION}.hash"
-  ARCHITECT_GATE_REASON=""
-  CLAUDE_PROJECT_DIR="$TEST_PROJECT_DIR" check_architect_gate "$TEST_SESSION" || true
-  rm -rf "$TEST_PROJECT_DIR"
-  [[ "$ARCHITECT_GATE_REASON" == *"drift"* ]] || [[ "$ARCHITECT_GATE_REASON" == *"changed"* ]]
-  [[ "$ARCHITECT_GATE_REASON" == *"wr-architect:agent"* ]]
-  [[ "$ARCHITECT_GATE_REASON" == *"refresh the marker"* ]]
-}
-
-@test "architect-enforce-edit deny output includes ARCHITECT_GATE_REASON directive" {
-  HOOK="$SCRIPT_DIR/architect-enforce-edit.sh"
-  ORIG_DIR="$PWD"
-  TEST_DIR=$(mktemp -d)
-  cd "$TEST_DIR"
-  mkdir -p docs/decisions
-  echo "# adr stub" > docs/decisions/001-x.md
-  json="{\"tool_input\":{\"file_path\":\"$PWD/src/x.ts\"},\"session_id\":\"deny-test-$$\"}"
-  run bash -c "echo '$json' | bash '$HOOK'"
-  cd "$ORIG_DIR"
-  rm -rf "$TEST_DIR"
-  [[ "$output" == *"BLOCKED"* ]]
-  [[ "$output" == *"wr-architect:agent"* ]]
-  # No marker exists for this fresh session — deny reason must explicitly
-  # name the re-delegate directive (not vague "review required").
-  [[ "$output" == *"No architect review marker"* ]] || [[ "$output" == *"marker"* ]]
-}

package/hooks/test/architect-mark-reviewed-verdict-grep.bats

@@ -1,113 +0,0 @@
-#!/usr/bin/env bats
-
-# Behavioural tests for architect-mark-reviewed.sh verdict parsing (P181).
-# Drives the hook with realistic agent-output payloads and asserts marker
-# creation matches the heading-shape contract in
-# packages/architect/agents/agent.md "How to Report".
-#
-# P181 root cause: literal-substring grep `grep -q "ISSUES FOUND"` matches
-# anywhere in the response — including prose narrative that mentions the
-# verdict string without it being the canonical heading. Anchored heading
-# match fixes the false-positive FAIL → silent marker-drop → edit block.
-
-setup() {
-  HOOKS_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  HOOK="$HOOKS_DIR/architect-mark-reviewed.sh"
-  TEST_SESSION="bats-arch-verdict-$$-${BATS_TEST_NUMBER}"
-  REVIEW_MARKER="/tmp/architect-reviewed-${TEST_SESSION}"
-  HASH_MARKER="/tmp/architect-reviewed-${TEST_SESSION}.hash"
-  PLAN_MARKER="/tmp/architect-plan-reviewed-${TEST_SESSION}"
-  rm -f "$REVIEW_MARKER" "$HASH_MARKER" "$PLAN_MARKER"
-}
-
-teardown() {
-  rm -f "$REVIEW_MARKER" "$HASH_MARKER" "$PLAN_MARKER"
-}
-
-# Build a PostToolUse:Agent input JSON with the given agent-output text.
-# Uses python3 (already a hard dep of gate-helpers.sh) for safe escaping.
-_make_input() {
-  local text="$1"
-  python3 -c "
-import json, sys
-text = sys.argv[1]
-print(json.dumps({
-    'session_id': '$TEST_SESSION',
-    'tool_name': 'Agent',
-    'tool_input': {'subagent_type': 'wr-architect:agent'},
-    'tool_response': {'content': [{'type': 'text', 'text': text}]}
-}))
-" "$text"
-}
-
-# ---------------------------------------------------------------------------
-# Sanity: canonical headings classify correctly
-# ---------------------------------------------------------------------------
-
-@test "verdict-grep: marker drops on canonical PASS heading" {
-  INPUT=$(_make_input "**Architecture Review: PASS**
-
-No conflicts with existing decisions.")
-  echo "$INPUT" | "$HOOK"
-  [ -f "$REVIEW_MARKER" ]
-}
-
-@test "verdict-grep: marker NOT created on canonical ISSUES FOUND heading" {
-  INPUT=$(_make_input "**Architecture Review: ISSUES FOUND**
-
-1. [Decision Conflict] — ADR-009 violation.")
-  echo "$INPUT" | "$HOOK"
-  [ ! -f "$REVIEW_MARKER" ]
-}
-
-@test "verdict-grep: marker drops on PASS heading with blockquote prefix" {
-  INPUT=$(_make_input "> **Architecture Review: PASS**
-> No conflicts.")
-  echo "$INPUT" | "$HOOK"
-  [ -f "$REVIEW_MARKER" ]
-}
-
-@test "verdict-grep: marker NOT created on ISSUES FOUND heading with blockquote prefix" {
-  INPUT=$(_make_input "> **Architecture Review: ISSUES FOUND**
-> 1. [Conflict] ...")
-  echo "$INPUT" | "$HOOK"
-  [ ! -f "$REVIEW_MARKER" ]
-}
-
-# ---------------------------------------------------------------------------
-# P181 bug-fix cases: substring-anywhere false-positive FAIL
-# ---------------------------------------------------------------------------
-
-@test "verdict-grep: no marker without a canonical heading" {
-  INPUT=$(_make_input "I reviewed the change. The previous review surfaced ISSUES FOUND that have since been addressed; the current proposed change is fine.")
-  echo "$INPUT" | "$HOOK"
-  [ ! -f "$REVIEW_MARKER" ]
-}
-
-@test "verdict-grep: NEEDS DIRECTION does not unlock edits" {
-  INPUT=$(_make_input "**Architecture Review: NEEDS DIRECTION**
-
-A decision must be recorded. This differs from an ISSUES FOUND verdict because the option is not pinned.
-
-- Option A — ...
-- Option B — ...")
-  echo "$INPUT" | "$HOOK"
-  [ ! -f "$REVIEW_MARKER" ]
-  [ ! -f "$PLAN_MARKER" ]
-}
-
-@test "verdict-grep: marker drops when PASS heading present and body also says 'ISSUES FOUND' inline" {
-  # PASS check runs first and must win even with substring noise downstream.
-  # This works in both old and new code; sanity-anchors the precedence rule.
-  INPUT=$(_make_input "**Architecture Review: PASS**
-
-No conflicts. Note: earlier sessions reported ISSUES FOUND on adjacent files but those are out of scope here.")
-  echo "$INPUT" | "$HOOK"
-  [ -f "$REVIEW_MARKER" ]
-}
-
-@test "verdict-grep: prose-only approval does not unlock edits" {
-  INPUT=$(_make_input "Review complete — no issues found in the diff.")
-  echo "$INPUT" | "$HOOK"
-  [ ! -f "$REVIEW_MARKER" ]
-}

package/hooks/test/architect-mark-reviewed.bats

@@ -1,26 +0,0 @@
-#!/usr/bin/env bats
-
-# Tests for architect verdict parsing from output text
-
-@test "grep matches 'Architecture Review: PASS'" {
-  output="Some preamble text\n\n**Architecture Review: PASS**\n\nNo conflicts."
-  echo -e "$output" | grep -q "Architecture Review: PASS"
-}
-
-@test "grep matches 'ISSUES FOUND'" {
-  output="**Architecture Review: ISSUES FOUND**\n\n1. [Decision Conflict]"
-  echo -e "$output" | grep -q "ISSUES FOUND"
-}
-
-@test "grep does NOT match unrelated text" {
-  output="The review is complete. Everything looks good."
-  ! echo -e "$output" | grep -q "Architecture Review: PASS"
-}
-
-@test "subagent pattern matches wr-architect:agent" {
-  SUBAGENT="wr-architect:agent"
-  case "$SUBAGENT" in
-    *architect*) true ;;
-    *) false ;;
-  esac
-}