@windyroad/architect 0.20.3 → 0.20.4

package/hooks/test/architect-marker-only-exempt.bats

@@ -1,183 +0,0 @@
-#!/usr/bin/env bats
-
-# P301: architect-enforce-edit.sh exempts marker-only frontmatter diffs to
-# docs/decisions/*.md ADRs from the full architect review gate. The
-# architect-oversight-marker-discipline.sh hook remains the safety net for
-# `human-oversight: confirmed` introductions; this exemption only short-
-# circuits the enforce-edit drift/TTL gate when the diff adds/modifies
-# nothing but the narrow oversight-marker grammar.
-#
-# Behavioural — exercises the full hook with constructed PreToolUse stdin
-# JSON payloads under a sandbox project dir; asserts on stdout+exit.
-
-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 architect gate — only runs when docs/decisions/ exists.
-  mkdir -p docs/decisions
-  echo "# stub" > docs/decisions/001-stub.proposed.md
-  SID="marker-only-exempt-$$-$BATS_TEST_NUMBER"
-}
-
-teardown() {
-  cd "$ORIG_DIR"
-  rm -rf "$TEST_DIR"
-  rm -f "/tmp/architect-reviewed-${SID}" "/tmp/architect-reviewed-${SID}.hash"
-}
-
-# Helper: run the hook with constructed JSON. Echo to a file then pipe to
-# avoid heredoc / quote-escaping headaches with multi-line content.
-run_hook_json() {
-  local json_file="$1"
-  bash "$HOOK" < "$json_file"
-}
-
-# ── Marker-only ADD: should exempt ───────────────────────────────────────
-
-@test "P301: Edit adding only human-oversight + oversight-date is exempt (no architect marker required)" {
-  adr="$PWD/docs/decisions/100-some-adr.proposed.md"
-  cat > "$adr" <<'EOF'
----
-status: "proposed"
-date: 2026-06-08
----
-
-# 100 some adr
-
-Body content unchanged.
-EOF
-  old=$'---\nstatus: "proposed"\ndate: 2026-06-08\n---'
-  new=$'---\nstatus: "proposed"\ndate: 2026-06-08\nhuman-oversight: unconfirmed\noversight-date: 2026-06-08\n---'
-  json_file=$(mktemp)
-  jq -nc --arg p "$adr" --arg s "$SID" --arg o "$old" --arg n "$new" \
-    '{tool_name:"Edit",session_id:$s,tool_input:{file_path:$p,old_string:$o,new_string:$n}}' > "$json_file"
-  run run_hook_json "$json_file"
-  rm -f "$json_file"
-  [ "$status" -eq 0 ]
-  [[ "$output" != *"BLOCKED"* ]]
-}
-
-@test "P301: Edit updating human-oversight value (unconfirmed → confirmed) is exempt" {
-  adr="$PWD/docs/decisions/101-promote.proposed.md"
-  cat > "$adr" <<'EOF'
----
-status: "proposed"
-human-oversight: unconfirmed
----
-
-# 101 promote
-EOF
-  old=$'human-oversight: unconfirmed'
-  new=$'human-oversight: confirmed\noversight-date: 2026-06-08'
-  json_file=$(mktemp)
-  jq -nc --arg p "$adr" --arg s "$SID" --arg o "$old" --arg n "$new" \
-    '{tool_name:"Edit",session_id:$s,tool_input:{file_path:$p,old_string:$o,new_string:$n}}' > "$json_file"
-  run run_hook_json "$json_file"
-  rm -f "$json_file"
-  [ "$status" -eq 0 ]
-  [[ "$output" != *"BLOCKED"* ]]
-}
-
-@test "P301: Edit adding rejected-pending-supersede + supersede-ticket is exempt" {
-  adr="$PWD/docs/decisions/102-rejected.proposed.md"
-  cat > "$adr" <<'EOF'
----
-status: "proposed"
----
-
-# 102 rejected
-EOF
-  old=$'status: "proposed"'
-  new=$'status: "proposed"\nhuman-oversight: rejected-pending-supersede\nsupersede-ticket: P999'
-  json_file=$(mktemp)
-  jq -nc --arg p "$adr" --arg s "$SID" --arg o "$old" --arg n "$new" \
-    '{tool_name:"Edit",session_id:$s,tool_input:{file_path:$p,old_string:$o,new_string:$n}}' > "$json_file"
-  run run_hook_json "$json_file"
-  rm -f "$json_file"
-  [ "$status" -eq 0 ]
-  [[ "$output" != *"BLOCKED"* ]]
-}
-
-# ── Mixed marker + body: must NOT exempt ─────────────────────────────────
-
-@test "P301: Edit changing body content along with markers still gates" {
-  adr="$PWD/docs/decisions/110-mixed.proposed.md"
-  cat > "$adr" <<'EOF'
----
-status: "proposed"
----
-
-# 110 mixed
-
-Original body.
-EOF
-  old=$'---\nstatus: "proposed"\n---\n\n# 110 mixed\n\nOriginal body.'
-  new=$'---\nstatus: "proposed"\nhuman-oversight: confirmed\noversight-date: 2026-06-08\n---\n\n# 110 mixed\n\nRewritten body with new policy claim.'
-  json_file=$(mktemp)
-  jq -nc --arg p "$adr" --arg s "$SID" --arg o "$old" --arg n "$new" \
-    '{tool_name:"Edit",session_id:$s,tool_input:{file_path:$p,old_string:$o,new_string:$n}}' > "$json_file"
-  run run_hook_json "$json_file"
-  rm -f "$json_file"
-  [[ "$output" == *"BLOCKED"* ]]
-}
-
-@test "P301: Edit changing status field (not a marker line) still gates" {
-  adr="$PWD/docs/decisions/111-status-change.proposed.md"
-  cat > "$adr" <<'EOF'
----
-status: "proposed"
----
-
-# 111
-EOF
-  old=$'status: "proposed"'
-  new=$'status: "accepted"\nhuman-oversight: confirmed\noversight-date: 2026-06-08'
-  json_file=$(mktemp)
-  jq -nc --arg p "$adr" --arg s "$SID" --arg o "$old" --arg n "$new" \
-    '{tool_name:"Edit",session_id:$s,tool_input:{file_path:$p,old_string:$o,new_string:$n}}' > "$json_file"
-  run run_hook_json "$json_file"
-  rm -f "$json_file"
-  [[ "$output" == *"BLOCKED"* ]]
-}
-
-# ── Pure body change: must still gate (no marker involvement) ────────────
-
-@test "P301: Edit changing only body content with no marker lines still gates" {
-  adr="$PWD/docs/decisions/120-body.proposed.md"
-  cat > "$adr" <<'EOF'
----
-status: "proposed"
----
-
-# 120 body
-
-Some text.
-EOF
-  old='Some text.'
-  new='Some text. And new text.'
-  json_file=$(mktemp)
-  jq -nc --arg p "$adr" --arg s "$SID" --arg o "$old" --arg n "$new" \
-    '{tool_name:"Edit",session_id:$s,tool_input:{file_path:$p,old_string:$o,new_string:$n}}' > "$json_file"
-  run run_hook_json "$json_file"
-  rm -f "$json_file"
-  [[ "$output" == *"BLOCKED"* ]]
-}
-
-# ── Scope: exemption is narrow to docs/decisions/*.md ────────────────────
-
-@test "P301: marker-only diff to a NON docs/decisions/ path still gates (no path exemption)" {
-  mkdir -p "$PWD/src"
-  echo "// stub" > "$PWD/src/x.ts"
-  src="$PWD/src/x.ts"
-  old='// stub'
-  new='// stub'$'\n''human-oversight: confirmed'
-  json_file=$(mktemp)
-  jq -nc --arg p "$src" --arg s "$SID" --arg o "$old" --arg n "$new" \
-    '{tool_name:"Edit",session_id:$s,tool_input:{file_path:$p,old_string:$o,new_string:$n}}' > "$json_file"
-  run run_hook_json "$json_file"
-  rm -f "$json_file"
-  [[ "$output" == *"BLOCKED"* ]]
-}

package/hooks/test/architect-no-stop-hook.bats

@@ -1,17 +0,0 @@
-#!/usr/bin/env bats
-
-# P001 / ADR-009: Stop-hook marker reset removed. Marker lifecycle is
-# governed by TTL + drift detection. This test asserts the Stop hook
-# entry is absent from the plugin's hooks.json.
-
-setup() {
-  PLUGIN_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/../.." && pwd)"
-}
-
-@test "architect: hooks.json has no Stop hook entry (ADR-009)" {
-  ! grep -q '"Stop"' "$PLUGIN_DIR/hooks/hooks.json"
-}
-
-@test "architect: architect-reset-marker.sh has been removed" {
-  [ ! -f "$PLUGIN_DIR/hooks/architect-reset-marker.sh" ]
-}

package/hooks/test/architect-oversight-marker-discipline.bats

@@ -1,245 +0,0 @@
-#!/usr/bin/env bats
-
-# P348 / ADR-066 amendment 2026-06-02: architect-oversight-marker-discipline.sh
-# is a PreToolUse:Edit|Write hook that denies any Edit/Write that introduces
-# `human-oversight: confirmed` into a docs/decisions/ ADR's frontmatter unless
-# a session-scoped evidence marker `/tmp/oversight-confirmed-<sha>-<sid>`
-# exists for THAT specific ADR under THIS session.
-#
-# Behavioural — exercises the hook with constructed PreToolUse stdin JSON
-# payloads under SESSION_MARKER_DIR sandboxing, asserts on stdout+exit.
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../.." && pwd)"
-  HOOK="$REPO_ROOT/packages/architect/hooks/architect-oversight-marker-discipline.sh"
-  MARK_SCRIPT="$REPO_ROOT/packages/architect/scripts/mark-oversight-confirmed.sh"
-
-  DIR="$(mktemp -d)"
-  mkdir -p "$DIR/docs/decisions"
-  MARK_DIR="$(mktemp -d)"
-  export SESSION_MARKER_DIR="$MARK_DIR"
-
-  ORIG_DIR="$PWD"
-  cd "$DIR"
-
-  SID="discipline-test-$$"
-}
-
-teardown() {
-  cd "$ORIG_DIR"
-  rm -rf "$DIR" "$MARK_DIR"
-  unset SESSION_MARKER_DIR
-}
-
-# Helper: compute the marker path the hook expects for an ADR file.
-expected_marker() {
-  local f="$1"
-  local abs_dir abs path_hash
-  abs_dir="$(cd "$(dirname "$f")" && pwd)"
-  abs="$abs_dir/$(basename "$f")"
-  if command -v sha256sum >/dev/null 2>&1; then
-    path_hash=$(printf '%s' "$abs" | sha256sum | cut -d' ' -f1 | cut -c1-16)
-  else
-    path_hash=$(printf '%s' "$abs" | shasum -a 256 | cut -d' ' -f1 | cut -c1-16)
-  fi
-  printf '%s/oversight-confirmed-%s-%s\n' "$MARK_DIR" "$path_hash" "$SID"
-}
-
-mk_existing_adr() {
-  local name="$1"; shift
-  {
-    echo "---"
-    echo "status: \"proposed\""
-    echo "date: 2026-06-02"
-    for line in "$@"; do echo "$line"; done
-    echo "---"
-    echo "# $name"
-  } > "$DIR/docs/decisions/$name"
-}
-
-# ── Positive paths (marker present → allow) ──────────────────────────────
-
-@test "Write introducing 'human-oversight: confirmed' with marker present is allowed" {
-  mk_existing_adr "200-unconfirmed.proposed.md"
-  adr="$DIR/docs/decisions/200-unconfirmed.proposed.md"
-  : > "$(expected_marker "$adr")"
-  new_content=$'---\nstatus: "proposed"\ndate: 2026-06-02\nhuman-oversight: confirmed\noversight-date: 2026-06-02\n---\n\n# 200-unconfirmed\n'
-  json=$(jq -nc --arg p "$adr" --arg s "$SID" --arg c "$new_content" \
-    '{tool_name:"Write",session_id:$s,tool_input:{file_path:$p,content:$c}}')
-  run bash -c "echo '$(echo "$json" | sed "s/'/'\\\\''/g")' | bash '$HOOK'"
-  [ "$status" -eq 0 ]
-  [[ "$output" != *"BLOCKED"* ]]
-}
-
-@test "Edit introducing 'human-oversight: confirmed' with marker present is allowed" {
-  mk_existing_adr "201-pending.proposed.md"
-  adr="$DIR/docs/decisions/201-pending.proposed.md"
-  : > "$(expected_marker "$adr")"
-  old='date: 2026-06-02'
-  new=$'date: 2026-06-02\nhuman-oversight: confirmed\noversight-date: 2026-06-02'
-  json=$(jq -nc --arg p "$adr" --arg s "$SID" --arg o "$old" --arg n "$new" \
-    '{tool_name:"Edit",session_id:$s,tool_input:{file_path:$p,old_string:$o,new_string:$n}}')
-  run bash -c "echo '$(echo "$json" | sed "s/'/'\\\\''/g")' | bash '$HOOK'"
-  [ "$status" -eq 0 ]
-  [[ "$output" != *"BLOCKED"* ]]
-}
-
-# ── Negative paths (no marker → deny) ───────────────────────────────────
-
-@test "Write introducing 'human-oversight: confirmed' WITHOUT marker is denied" {
-  mk_existing_adr "210-orphan.proposed.md"
-  adr="$DIR/docs/decisions/210-orphan.proposed.md"
-  new_content=$'---\nstatus: "proposed"\ndate: 2026-06-02\nhuman-oversight: confirmed\noversight-date: 2026-06-02\n---\n\n# 210-orphan\n'
-  json=$(jq -nc --arg p "$adr" --arg s "$SID" --arg c "$new_content" \
-    '{tool_name:"Write",session_id:$s,tool_input:{file_path:$p,content:$c}}')
-  run bash -c "echo '$(echo "$json" | sed "s/'/'\\\\''/g")' | bash '$HOOK'"
-  [ "$status" -eq 0 ]
-  [[ "$output" == *'"permissionDecision": "deny"'* ]]
-  [[ "$output" == *"BLOCKED"* ]]
-  [[ "$output" == *"oversight-confirmed-"* ]]
-  [[ "$output" == *"wr-architect-mark-oversight-confirmed"* ]]
-}
-
-@test "Edit introducing 'human-oversight: confirmed' WITHOUT marker is denied" {
-  mk_existing_adr "211-no-evidence.proposed.md"
-  adr="$DIR/docs/decisions/211-no-evidence.proposed.md"
-  old='date: 2026-06-02'
-  new=$'date: 2026-06-02\nhuman-oversight: confirmed\noversight-date: 2026-06-02'
-  json=$(jq -nc --arg p "$adr" --arg s "$SID" --arg o "$old" --arg n "$new" \
-    '{tool_name:"Edit",session_id:$s,tool_input:{file_path:$p,old_string:$o,new_string:$n}}')
-  run bash -c "echo '$(echo "$json" | sed "s/'/'\\\\''/g")' | bash '$HOOK'"
-  [[ "$output" == *'"permissionDecision": "deny"'* ]]
-}
-
-# ── AFK-unconfirmed-write path (no marker required) ──────────────────────
-
-@test "Write introducing 'human-oversight: unconfirmed' is allowed without marker (AFK path)" {
-  mk_existing_adr "220-afk.proposed.md"
-  adr="$DIR/docs/decisions/220-afk.proposed.md"
-  new_content=$'---\nstatus: "proposed"\ndate: 2026-06-02\nhuman-oversight: unconfirmed\n---\n\n# 220-afk\n'
-  json=$(jq -nc --arg p "$adr" --arg s "$SID" --arg c "$new_content" \
-    '{tool_name:"Write",session_id:$s,tool_input:{file_path:$p,content:$c}}')
-  run bash -c "echo '$(echo "$json" | sed "s/'/'\\\\''/g")' | bash '$HOOK'"
-  [ "$status" -eq 0 ]
-  [[ "$output" != *"BLOCKED"* ]]
-}
-
-@test "Write introducing rejected-pending-supersede is allowed without marker" {
-  mk_existing_adr "221-rejected.proposed.md"
-  adr="$DIR/docs/decisions/221-rejected.proposed.md"
-  new_content=$'---\nstatus: "proposed"\ndate: 2026-06-02\nhuman-oversight: rejected-pending-supersede\nsupersede-ticket: P999\n---\n\n# 221-rejected\n'
-  json=$(jq -nc --arg p "$adr" --arg s "$SID" --arg c "$new_content" \
-    '{tool_name:"Write",session_id:$s,tool_input:{file_path:$p,content:$c}}')
-  run bash -c "echo '$(echo "$json" | sed "s/'/'\\\\''/g")' | bash '$HOOK'"
-  [ "$status" -eq 0 ]
-  [[ "$output" != *"BLOCKED"* ]]
-}
-
-# ── Scope / non-fire paths ────────────────────────────────────────────────
-
-@test "Edit to a non-ADR path (src/index.ts) exits 0 silently" {
-  mkdir -p "$DIR/src"
-  echo "// stub" > "$DIR/src/index.ts"
-  json=$(jq -nc --arg p "$DIR/src/index.ts" --arg s "$SID" \
-    '{tool_name:"Edit",session_id:$s,tool_input:{file_path:$p,old_string:"// stub",new_string:"// human-oversight: confirmed"}}')
-  run bash -c "echo '$(echo "$json" | sed "s/'/'\\\\''/g")' | bash '$HOOK'"
-  [ "$status" -eq 0 ]
-  [ -z "$output" ]
-}
-
-@test "Edit to docs/decisions/README.md exits 0 silently" {
-  echo "# index" > "$DIR/docs/decisions/README.md"
-  adr="$DIR/docs/decisions/README.md"
-  new=$'# index\nhuman-oversight: confirmed'
-  json=$(jq -nc --arg p "$adr" --arg s "$SID" --arg n "$new" \
-    '{tool_name:"Edit",session_id:$s,tool_input:{file_path:$p,old_string:"# index",new_string:$n}}')
-  run bash -c "echo '$(echo "$json" | sed "s/'/'\\\\''/g")' | bash '$HOOK'"
-  [ "$status" -eq 0 ]
-  [ -z "$output" ]
-}
-
-@test "Edit whose new content lacks the marker exits 0 silently" {
-  mk_existing_adr "230-unrelated.proposed.md"
-  adr="$DIR/docs/decisions/230-unrelated.proposed.md"
-  json=$(jq -nc --arg p "$adr" --arg s "$SID" \
-    '{tool_name:"Edit",session_id:$s,tool_input:{file_path:$p,old_string:"# 230-unrelated",new_string:"# 230 renamed"}}')
-  run bash -c "echo '$(echo "$json" | sed "s/'/'\\\\''/g")' | bash '$HOOK'"
-  [ "$status" -eq 0 ]
-  [ -z "$output" ]
-}
-
-@test "Write whose OLD content already had the marker (re-stamp / no-op) is allowed without re-evidencing" {
-  # Pre-existing ADR already carries the marker; new content keeps it.
-  mk_existing_adr "240-already.proposed.md" "human-oversight: confirmed" "oversight-date: 2026-05-30"
-  adr="$DIR/docs/decisions/240-already.proposed.md"
-  new_content=$'---\nstatus: "accepted"\ndate: 2026-06-02\nhuman-oversight: confirmed\noversight-date: 2026-05-30\n---\n\n# 240-already\n'
-  json=$(jq -nc --arg p "$adr" --arg s "$SID" --arg c "$new_content" \
-    '{tool_name:"Write",session_id:$s,tool_input:{file_path:$p,content:$c}}')
-  run bash -c "echo '$(echo "$json" | sed "s/'/'\\\\''/g")' | bash '$HOOK'"
-  [ "$status" -eq 0 ]
-  [[ "$output" != *"BLOCKED"* ]]
-}
-
-# ── End-to-end: mark-oversight-confirmed.sh + the hook ──────────────────
-
-@test "mark-oversight-confirmed.sh writes a marker that satisfies the hook" {
-  mk_existing_adr "250-e2e.proposed.md"
-  adr="$DIR/docs/decisions/250-e2e.proposed.md"
-  # Seed an announce marker so candidate enumeration finds the SID.
-  : > "$MARK_DIR/architect-announced-$SID"
-  bash "$MARK_SCRIPT" "$adr"
-  # The mark script writes under every candidate; the hook's expected marker
-  # filename for our SID must now exist.
-  [ -f "$(expected_marker "$adr")" ]
-  new_content=$'---\nstatus: "proposed"\ndate: 2026-06-02\nhuman-oversight: confirmed\noversight-date: 2026-06-02\n---\n\n# 250-e2e\n'
-  json=$(jq -nc --arg p "$adr" --arg s "$SID" --arg c "$new_content" \
-    '{tool_name:"Write",session_id:$s,tool_input:{file_path:$p,content:$c}}')
-  run bash -c "echo '$(echo "$json" | sed "s/'/'\\\\''/g")' | bash '$HOOK'"
-  [ "$status" -eq 0 ]
-  [[ "$output" != *"BLOCKED"* ]]
-}
-
-# P380: on macOS SESSION_MARKER_DIR defaults to /tmp, a symlink to /private/tmp.
-# `find <symlink> -maxdepth 1` in default (-P) mode refuses to descend the
-# start-point symlink, so candidate enumeration returns empty and the script
-# writes ZERO markers (silent cold-path exit 0). The `-L` flag follows it. This
-# test points SESSION_MARKER_DIR at a SYMLINK to the marker dir (reproducing the
-# macOS /tmp shape on any platform); RED without `-L`, GREEN with it.
-@test "mark-oversight-confirmed.sh enumerates candidates when SESSION_MARKER_DIR is a symlink (P380)" {
-  mk_existing_adr "251-symlink.proposed.md"
-  adr="$DIR/docs/decisions/251-symlink.proposed.md"
-  : > "$MARK_DIR/architect-announced-$SID"
-  link_dir="${MARK_DIR}.link"
-  ln -s "$MARK_DIR" "$link_dir"
-  SESSION_MARKER_DIR="$link_dir" bash "$MARK_SCRIPT" "$adr"
-  rm -f "$link_dir"
-  # Marker written under the discovered SID despite the symlinked marker dir.
-  [ -f "$(expected_marker "$adr")" ]
-}
-
-# P368: genuine cold path — no CLAUDE_SESSION_ID and no announce markers at all.
-# The shim correctly writes zero markers (nothing to key on) but must NOT do so
-# SILENTLY: a loud stderr diagnostic explains why the subsequent discipline-hook
-# deny happens, breaking the confusing "you didn't run the shim" loop the shim's
-# prior silent exit 0 produced. Exit stays 0 (documented contract: do not crash
-# SKILL flows before any hook has fired this session).
-@test "mark-oversight-confirmed.sh emits a loud stderr diagnostic on the no-candidate cold path (P368)" {
-  mk_existing_adr "252-coldpath.proposed.md"
-  adr="$DIR/docs/decisions/252-coldpath.proposed.md"
-  # MARK_DIR is empty (no *-announced-* markers); unset the env SID fast-path too.
-  run env -u CLAUDE_SESSION_ID bash "$MARK_SCRIPT" "$adr"
-  [ "$status" -eq 0 ]                              # contract: cold path still exits 0
-  [ ! -f "$(expected_marker "$adr")" ]             # no marker written
-  [[ "$output" == *"no candidate session id"* ]]   # loud, not silent
-  [[ "$output" == *"P368"* ]]
-}
-
-# ── Non-Edit/Write tool calls always exit 0 silently ────────────────────
-
-@test "tool_name=Bash exits 0 silently regardless of file path" {
-  json=$(jq -nc --arg s "$SID" \
-    '{tool_name:"Bash",session_id:$s,tool_input:{command:"echo human-oversight: confirmed"}}')
-  run bash -c "echo '$json' | bash '$HOOK'"
-  [ "$status" -eq 0 ]
-  [ -z "$output" ]
-}

package/hooks/test/architect-oversight-nudge.bats

@@ -1,75 +0,0 @@
-#!/usr/bin/env bats
-
-# ADR-066: architect-oversight-nudge.sh (SessionStart) emits a one-line nudge
-# when ADRs lack the human-oversight marker, is silent when none do, and
-# self-suppresses under the AFK guard (WR_SUPPRESS_OVERSIGHT_NUDGE=1) so the
-# interactive batch-confirm never fires into an absent-user iteration (JTBD-006).
-# Behavioural — exercises the hook against fixture trees and asserts on stdout.
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../.." && pwd)"
-  HOOK="$REPO_ROOT/packages/architect/hooks/architect-oversight-nudge.sh"
-  PLUGIN_ROOT="$REPO_ROOT/packages/architect"
-  DIR="$(mktemp -d)"
-  mkdir -p "$DIR/docs/decisions"
-  # Hermeticity (P391): the AFK work-problems orchestrator exports
-  # WR_SUPPRESS_OVERSIGHT_NUDGE=1, which the hook self-suppresses on. Unset it so
-  # count-emitting tests assert real behaviour; guard-specific tests set it
-  # per-invocation via `run env WR_SUPPRESS_OVERSIGHT_NUDGE=...`.
-  unset WR_SUPPRESS_OVERSIGHT_NUDGE
-}
-
-teardown() {
-  rm -rf "$DIR"
-}
-
-mk_unmarked() {
-  { echo "---"; echo "status: \"proposed\""; echo "date: 2026-05-25"; echo "---"; echo "# $1"; } \
-    > "$DIR/docs/decisions/$1"
-}
-mk_marked() {
-  { echo "---"; echo "status: \"proposed\""; echo "date: 2026-05-25"; echo "human-oversight: confirmed"; echo "---"; echo "# $1"; } \
-    > "$DIR/docs/decisions/$1"
-}
-
-@test "emits a count line when there are unoversighted ADRs" {
-  mk_unmarked "010-a.proposed.md"
-  mk_unmarked "011-b.proposed.md"
-  run env CLAUDE_PROJECT_DIR="$DIR" CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" bash "$HOOK"
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"2 recorded decisions lack human oversight"* ]]
-  [[ "$output" == *"/wr-architect:review-decisions"* ]]
-}
-
-@test "uses singular wording for exactly one unoversighted ADR" {
-  mk_unmarked "010-a.proposed.md"
-  run env CLAUDE_PROJECT_DIR="$DIR" CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" bash "$HOOK"
-  [[ "$output" == *"1 recorded decision lacks human oversight"* ]]
-}
-
-@test "silent when every ADR is confirmed" {
-  mk_marked "010-a.proposed.md"
-  run env CLAUDE_PROJECT_DIR="$DIR" CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" bash "$HOOK"
-  [ "$status" -eq 0 ]
-  [ -z "$output" ]
-}
-
-@test "AFK guard suppresses the nudge entirely" {
-  mk_unmarked "010-a.proposed.md"
-  run env WR_SUPPRESS_OVERSIGHT_NUDGE=1 CLAUDE_PROJECT_DIR="$DIR" CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" bash "$HOOK"
-  [ "$status" -eq 0 ]
-  [ -z "$output" ]
-}
-
-@test "guard value other than 1 does not suppress" {
-  mk_unmarked "010-a.proposed.md"
-  mk_unmarked "011-b.proposed.md"
-  run env WR_SUPPRESS_OVERSIGHT_NUDGE=0 CLAUDE_PROJECT_DIR="$DIR" CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" bash "$HOOK"
-  [[ "$output" == *"lack human oversight"* ]]
-}
-
-@test "silent when project has no docs/decisions dir" {
-  run env CLAUDE_PROJECT_DIR="$DIR/empty" CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" bash "$HOOK"
-  [ "$status" -eq 0 ]
-  [ -z "$output" ]
-}

package/hooks/test/architect-project-root.bats

@@ -1,81 +0,0 @@
-#!/usr/bin/env bats
-
-# Tests for architect-enforce-edit.sh project-root check (P004).
-# Verifies that absolute paths outside $PWD are exempted.
-
-setup() {
-  SCRIPT_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
-  HOOK="$SCRIPT_DIR/architect-enforce-edit.sh"
-  ORIG_DIR="$PWD"
-}
-
-teardown() {
-  cd "$ORIG_DIR"
-}
-
-run_hook_with_file() {
-  local file_path="$1"
-  local json="{\"tool_input\":{\"file_path\":\"${file_path}\"},\"session_id\":\"test-session-$$\"}"
-  echo "$json" | bash "$HOOK"
-}
-
-@test "project-root: absolute path outside project exits 0" {
-  run run_hook_with_file "/Users/other/somewhere/file.json"
-  [ "$status" -eq 0 ]
-  [[ "$output" != *"BLOCKED"* ]]
-}
-
-@test "project-root: home-dir config path exits 0" {
-  run run_hook_with_file "/Users/somebody/.claude/channels/discord/access.json"
-  [ "$status" -eq 0 ]
-  [[ "$output" != *"BLOCKED"* ]]
-}
-
-@test "project-root: absolute path inside \$PWD is still gated" {
-  # Use a temp dir as PWD with docs/decisions to trigger the gate
-  TEST_DIR=$(mktemp -d)
-  mkdir -p "$TEST_DIR/docs/decisions"
-  echo "# ADR" > "$TEST_DIR/docs/decisions/001-test.proposed.md"
-  cd "$TEST_DIR"
-  run run_hook_with_file "$TEST_DIR/src/index.ts"
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"BLOCKED"* ]]
-  rm -rf "$TEST_DIR"
-}
-
-# P191 Phase 2: the architect gate must resolve docs/decisions from the
-# project root (CLAUDE_PROJECT_DIR), NOT the hook's actual runtime CWD. Claude
-# Code can launch the hook with a working directory that differs from the
-# session/project dir; a relative `[ ! -d "docs/decisions" ]` then
-# false-negatives and the gate FAILS OPEN (exit 0) — silently DEACTIVATING the
-# architect gate so edits bypass review. This is a governance hole, strictly
-# more severe than the JTBD gate's fail-closed nuisance (P191 Phase 1).
-@test "project-root: gate stays ACTIVE via CLAUDE_PROJECT_DIR when hook CWD differs (P191 Phase 2)" {
-  local proj other json
-  proj="$(mktemp -d)"
-  other="$(mktemp -d)"        # a CWD that does NOT contain docs/decisions
-  mkdir -p "$proj/docs/decisions"
-  echo "# ADR" > "$proj/docs/decisions/001-test.proposed.md"
-  json="{\"tool_input\":{\"file_path\":\"${proj}/src/index.ts\"},\"session_id\":\"test-session-$$\"}"
-  # Fire from `other` (wrong CWD) with CLAUDE_PROJECT_DIR set to the real
-  # project. Pre-fix this silently exited 0 (gate inactive, edit allowed);
-  # post-fix the gate is ACTIVE and denies for the missing review marker.
-  run env CLAUDE_PROJECT_DIR="$proj" bash -c "cd '$other' && printf '%s' '$json' | bash '$HOOK'"
-  rm -rf "$proj" "$other"
-  [[ "$output" == *"BLOCKED"* ]]
-  [[ "$output" == *"without architecture review"* ]]
-}
-
-# P191 Phase 2 regression guard: when docs/decisions genuinely does not exist
-# under the project root, the gate correctly stays INACTIVE (fail-open exit 0).
-# The fix narrows the false-negative; it must not start gating projects that
-# have no architecture decisions.
-@test "project-root: genuinely-absent docs/decisions stays inactive (fail-open preserved, P191 Phase 2)" {
-  local proj json
-  proj="$(mktemp -d)"         # no docs/decisions created
-  json="{\"tool_input\":{\"file_path\":\"${proj}/src/index.ts\"},\"session_id\":\"test-session-$$\"}"
-  run env CLAUDE_PROJECT_DIR="$proj" bash -c "printf '%s' '$json' | bash '$HOOK'"
-  rm -rf "$proj"
-  [ "$status" -eq 0 ]
-  [[ "$output" != *"BLOCKED"* ]]
-}