@windyroad/itil 0.61.1 → 0.61.2

package/hooks/test/itil-deferral-cadence-gate.bats

@@ -1,187 +0,0 @@
-#!/usr/bin/env bats
-
-# P375 — itil-deferral-cadence-gate.sh PostToolUse:Write|Edit|MultiEdit
-# advisory hook. The ratified Option-C "authoring-time enforcement gate"
-# (core slice, advisory rollout per ADR-057 staged-rollout + the P375
-# architect review 2026-06-28).
-#
-# The gate fires at AUTHORING time on the NEWLY-authored text (Edit
-# new_string / Write content / MultiEdit concatenated new_strings —
-# diff-aware so descriptive prose already on disk never re-triggers) of
-# a SHIPPED authoring surface (SKILL.md, docs/decisions/*.md ADRs,
-# docs/rfcs/*.md RFCs, hook *.sh). When the new text introduces an
-# uncadenced-deferral phrasing (`deferred to <on-demand re-entry>`,
-# `pending review`, `re-rate at next`, `(deferred …`, `next review`,
-# `when ready`, `lands in Slice N`) WITHOUT a cadence annotation naming
-# a SELF-FIRING trigger within the +/-5 line window, it emits a stderr
-# advisory citing P375 + the remedy.
-#
-# THE LOAD-BEARING P375 REFINEMENT vs the P234 sibling
-# (itil-fictional-defer-detect.sh): a bare named on-demand skill
-# (/wr-foo:bar) or a bare ticket ID (Pnnn / RFC-nnn / ADR-nnn) does NOT
-# satisfy the cadence requirement — naming an on-demand re-entry point
-# and treating it as a cadence IS the exact conflation P375 captures
-# ("BUT NOTHING TRIGGERS THAT WORK!!!"). Only a self-firing-CLASS
-# citation satisfies: a hook *.sh, SessionStart, PreToolUse/PostToolUse,
-# .github/workflows/ (CI), cron, or a work-problems Step-0x / pre-flight
-# reference. The existing P234 hook WRONGLY accepts skills + ticket IDs;
-# this hook does not.
-#
-# Core-slice boundary (architect refinement 1, named loudly): the gate
-# checks the cadence annotation names a self-firing-CLASS trigger — it
-# does NOT validate that the named trigger actually exists / fires (the
-# transitive-reachability graph is a deferred later slice). docs/problems/
-# tickets are EXCLUDED (they descriptively narrate deferrals — highest
-# false-positive surface, already covered by the ADR-084 census).
-#
-# Advisory only — NEVER blocks (exit 0 always), per ADR-040/045
-# declarative-first + ADR-013 Rule 6 fail-open. Behavioural per ADR-052
-# (asserts emitted stderr, never source-greps the hook). Hermetic per
-# P391 (unsets inherited suppress vars in setup).
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../.." && pwd)"
-  HOOK="$REPO_ROOT/packages/itil/hooks/itil-deferral-cadence-gate.sh"
-  # P391 hermeticity: AFK iters export suppress vars that would poison
-  # the advisory-fires assertions. Strip them for the test process.
-  unset WR_SUPPRESS_DEFERRAL_CADENCE_GATE
-  unset WR_SUPPRESS_OVERSIGHT_NUDGE
-}
-
-# Emit a PostToolUse stdin payload for an Edit tool call.
-# $1 = file_path, $2 = new_string (the newly-authored text).
-emit_edit() {
-  jq -n --arg p "$1" --arg n "$2" '{
-    session_id: "deferral-cadence-test",
-    tool_name: "Edit",
-    tool_input: { file_path: $p, old_string: "X", new_string: $n },
-    tool_response: { success: true }
-  }'
-}
-
-# Emit a PostToolUse stdin payload for a Write tool call.
-emit_write() {
-  jq -n --arg p "$1" --arg c "$2" '{
-    session_id: "deferral-cadence-test",
-    tool_name: "Write",
-    tool_input: { file_path: $p, content: $c },
-    tool_response: { success: true }
-  }'
-}
-
-fires() { [[ "$output" == *"P375"* ]] || [[ "$stderr" == *"P375"* ]]; }
-silent() { [[ "$output" != *"P375"* ]] && [[ "$stderr" != *"P375"* ]]; }
-
-# --- Positive: uncadenced deferral authored → advisory fires ---
-
-@test "fire: SKILL.md deferral citing a bare on-demand skill (the P375 conflation) fires" {
-  # This is the case the P234 sibling WRONGLY allows — a named on-demand
-  # skill is NOT a self-firing cadence.
-  run bash "$HOOK" <<<"$(emit_edit packages/foo/skills/bar/SKILL.md \
-    'Full scope deferred to /wr-itil:manage-rfc accepted transition.')"
-  [ "$status" -eq 0 ]
-  fires
-}
-
-@test "fire: ADR new_string deferral with no citation at all fires" {
-  run bash "$HOOK" <<<"$(emit_edit docs/decisions/099-thing.proposed.md \
-    'Considered Options and Consequences deferred — flesh out later.')"
-  [ "$status" -eq 0 ]
-  fires
-}
-
-@test "fire: RFC deferral citing only a bare ticket ID (not a cadence) fires" {
-  # A ticket ID is an audit annotation, not a self-firing trigger.
-  run bash "$HOOK" <<<"$(emit_edit docs/rfcs/RFC-099-thing.proposed.md \
-    'Scope deferred to next review per P375.')"
-  [ "$status" -eq 0 ]
-  fires
-}
-
-@test "fire: Write of a hook .sh whose content carries an uncadenced deferral fires" {
-  run bash "$HOOK" <<<"$(emit_write packages/foo/hooks/foo.sh \
-    '# TODO pending review when ready; not wired yet.')"
-  [ "$status" -eq 0 ]
-  fires
-}
-
-@test "fire: deferral with a cadence comment that names an ON-DEMAND skill still fires" {
-  # The annotation carrier is fine but its value must name a self-firing
-  # CLASS — naming an on-demand skill inside the comment does not help.
-  run bash "$HOOK" <<<"$(emit_edit packages/foo/skills/bar/SKILL.md \
-    'Tasks deferred. <!-- cadence: /wr-itil:manage-rfc accepted -->')"
-  [ "$status" -eq 0 ]
-  fires
-}
-
-# --- Negative: cadenced deferral or out-of-scope → silent ---
-
-@test "allow: deferral citing a self-firing hook .sh in window exits silent" {
-  run bash "$HOOK" <<<"$(emit_edit packages/foo/skills/bar/SKILL.md \
-    'Re-rate deferred; surfaced every session by retrospective-deferral-census.sh.')"
-  [ "$status" -eq 0 ]
-  silent
-}
-
-@test "allow: deferral with an explicit cadence annotation naming PostToolUse exits silent" {
-  run bash "$HOOK" <<<"$(emit_edit docs/decisions/099-thing.proposed.md \
-    'Section deferred. <!-- cadence: PostToolUse itil-foo-gate.sh fires on author -->')"
-  [ "$status" -eq 0 ]
-  silent
-}
-
-@test "allow: deferral citing SessionStart self-firing surface exits silent" {
-  run bash "$HOOK" <<<"$(emit_edit docs/rfcs/RFC-099-thing.proposed.md \
-    'Drain deferred; the SessionStart nudge re-surfaces it every session.')"
-  [ "$status" -eq 0 ]
-  silent
-}
-
-@test "allow: deferral citing a CI workflow exits silent" {
-  run bash "$HOOK" <<<"$(emit_edit packages/foo/skills/bar/SKILL.md \
-    'Validation deferred to .github/workflows/check.yml on every push.')"
-  [ "$status" -eq 0 ]
-  silent
-}
-
-@test "allow: deferral in a docs/problems/ ticket (excluded surface) exits silent" {
-  run bash "$HOOK" <<<"$(emit_edit docs/problems/open/099-thing.open.md \
-    'Severity deferred to investigation; expand at next review.')"
-  [ "$status" -eq 0 ]
-  silent
-}
-
-@test "allow: deferral phrasing in a non-authoring file (.ts) exits silent" {
-  run bash "$HOOK" <<<"$(emit_edit packages/foo/src/index.ts \
-    '// deferred to next review — TODO')"
-  [ "$status" -eq 0 ]
-  silent
-}
-
-@test "allow: authoring-surface edit with NO deferral phrasing exits silent" {
-  run bash "$HOOK" <<<"$(emit_edit packages/foo/skills/bar/SKILL.md \
-    'This step validates the marker and proceeds to Step 2.')"
-  [ "$status" -eq 0 ]
-  silent
-}
-
-# --- Robustness ---
-
-@test "fail-open: empty stdin exits 0 silent" {
-  run bash "$HOOK" <<<""
-  [ "$status" -eq 0 ]
-  silent
-}
-
-@test "fail-open: malformed JSON stdin exits 0 silent" {
-  run bash "$HOOK" <<<"{not json"
-  [ "$status" -eq 0 ]
-  silent
-}
-
-@test "suppress: WR_SUPPRESS_DEFERRAL_CADENCE_GATE=1 silences the advisory" {
-  run env WR_SUPPRESS_DEFERRAL_CADENCE_GATE=1 bash "$HOOK" <<<"$(emit_edit \
-    packages/foo/skills/bar/SKILL.md 'deferred to /wr-itil:manage-rfc accepted')"
-  [ "$status" -eq 0 ]
-  silent
-}

package/hooks/test/itil-fictional-defer-detect.bats

@@ -1,292 +0,0 @@
-#!/usr/bin/env bats
-
-# P234 Phase 1: itil-fictional-defer-detect.sh PostToolUse:Write|Edit
-# hook detects "fictional defer" rationales in `docs/retros/*.md` writes
-# — defer-rationale phrases (`next retro`, `next session`, `defer
-# pending`, `defer with cause:`, `deferred per`) that lack a
-# SCHEDULED-FUTURE-SURFACE citation in surrounding context.
-#
-# Detection signal (per ticket Investigation Task 2 two-axis test):
-#   1. tool_name is Write OR Edit AND file_path matches docs/retros/*.md
-#   2. Written file contains a defer-rationale phrase (case-insensitive)
-#   3. Within +/-5 lines of the match there is NO citation of a
-#      SCHEDULED-FUTURE-SURFACE (ticket ID P\d{3} / STORY-\d{3} / R\d{3},
-#      skill invocation /wr-[a-z-]+:[a-z-]+, hook script path .sh, CI
-#      workflow path .github/workflows/, dated ADR ADR-\d{3} ... YYYY-MM-DD)
-#   4. Match is NOT on the exception allowlist (e.g. `deferred per Branch B`).
-#
-# When all four hold, the hook emits a stderr advisory citing P234 + the
-# SCHEDULED-FUTURE-SURFACE definition. Advisory only — never blocks
-# (exit 0 always). Mirrors the itil-rfc-trailer-advisory.sh PostToolUse
-# precedent (stderr + exit 0) and the itil-mid-loop-ask-detect.sh
-# detection-pattern precedent (per-surface configuration at top).
-#
-# Per ADR-005 / ADR-052 — bats live under packages/<plugin>/hooks/test/
-# and assert on emitted stderr text, not source-content. Per P081 — no
-# source-grep on hook text.
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../.." && pwd)"
-  HOOK="$REPO_ROOT/packages/itil/hooks/itil-fictional-defer-detect.sh"
-  TMPDIR_="$(mktemp -d)"
-  RETRO_DIR="$TMPDIR_/docs/retros"
-  mkdir -p "$RETRO_DIR"
-  RETRO_FILE="$RETRO_DIR/2026-05-17-session-3.md"
-}
-
-teardown() {
-  rm -rf "$TMPDIR_"
-}
-
-# Helper: emit PostToolUse stdin payload for a Write tool call.
-emit_write_payload() {
-  local file_path="$1"
-  jq -n --arg p "$file_path" '{
-    session_id: "fictional-defer-test",
-    tool_name: "Write",
-    tool_input: { file_path: $p, content: "(content already on disk)" },
-    tool_response: { success: true }
-  }'
-}
-
-# Helper: emit PostToolUse stdin payload for an Edit tool call.
-emit_edit_payload() {
-  local file_path="$1"
-  jq -n --arg p "$file_path" '{
-    session_id: "fictional-defer-test",
-    tool_name: "Edit",
-    tool_input: { file_path: $p, old_string: "x", new_string: "y" },
-    tool_response: { success: true }
-  }'
-}
-
-run_hook_with_write() {
-  emit_write_payload "$RETRO_FILE" | bash "$HOOK"
-}
-
-run_hook_with_edit() {
-  emit_edit_payload "$RETRO_FILE" | bash "$HOOK"
-}
-
-# --- Positive detection: fictional defer ---
-
-@test "detect: defer-to-next-retro with no scheduled-future-surface emits advisory" {
-  # Faithful reproduction of the P234 worked-example fictional-defer
-  # class — the defer-rationale prose carries no SCHEDULED-FUTURE-
-  # SURFACE citation; no ticket ID, no skill invocation, no dated ADR
-  # appears in the +/-5 line window around the defer phrase.
-  cat > "$RETRO_FILE" <<'EOF'
-# Session 3 Retro
-
-## Signal-vs-Noise Pass
-
-Deferred this retro per session-length constraint (16+ briefing
-entries across 13 topic files would require ~30 min of per-entry
-scoring). Next retro should run a full pass.
-EOF
-  run run_hook_with_write
-  [ "$status" -eq 0 ]
-  [[ "$stderr" == *"P234"* ]] || [[ "$output" == *"P234"* ]]
-}
-
-@test "detect: deferred-pending-design-judgement with no scheduled-future-surface emits advisory" {
-  cat > "$RETRO_FILE" <<'EOF'
-# Session 4 Retro
-
-## Topic File Rotation Candidates
-
-| File | Action |
-|------|--------|
-| governance-workflow.md | deferred pending design judgement (cascade case) |
-| hooks-and-gates.md | deferred pending complexity review |
-EOF
-  run run_hook_with_write
-  [ "$status" -eq 0 ]
-  [[ "$stderr" == *"P234"* ]] || [[ "$output" == *"P234"* ]]
-}
-
-@test "detect: defer-with-cause-context-budget with no surface emits advisory" {
-  cat > "$RETRO_FILE" <<'EOF'
-# Session 5 Retro
-
-## Codification Candidates
-
-Deferred with cause: context budget pressure. Next session should
-revisit when fresh context is available.
-EOF
-  run run_hook_with_edit
-  [ "$status" -eq 0 ]
-  [[ "$stderr" == *"P234"* ]] || [[ "$output" == *"P234"* ]]
-}
-
-# --- Negative paths: legitimate citations (silent exit) ---
-
-@test "allow: defer citing P-ticket within +/-5 lines exits silent" {
-  cat > "$RETRO_FILE" <<'EOF'
-# Session 3 Retro
-
-## Signal-vs-Noise Pass
-
-Deferred per [[P235]] (briefing SVN backlog: 146 entries across 17
-topic files). Next retro will surface P235 if it has been promoted
-to actionable.
-EOF
-  run run_hook_with_write
-  [ "$status" -eq 0 ]
-  [[ "$stderr" != *"P234"* ]]
-  [[ "$output" != *"P234"* ]]
-}
-
-@test "allow: defer citing skill invocation within +/-5 lines exits silent" {
-  cat > "$RETRO_FILE" <<'EOF'
-# Session 4 Retro
-
-## Tickets Deferred
-
-Deferred pending /wr-itil:work-problems Step 6.5 above-appetite
-release-loop check.
-EOF
-  run run_hook_with_edit
-  [ "$status" -eq 0 ]
-  [[ "$stderr" != *"P234"* ]]
-  [[ "$output" != *"P234"* ]]
-}
-
-@test "allow: defer citing hook script path within +/-5 lines exits silent" {
-  cat > "$RETRO_FILE" <<'EOF'
-# Session 5 Retro
-
-## Codification Candidates
-
-Defer pending packages/itil/hooks/itil-fictional-defer-detect.sh
-extension to also cover the assistant-output review channel.
-EOF
-  run run_hook_with_write
-  [ "$status" -eq 0 ]
-  [[ "$stderr" != *"P234"* ]]
-  [[ "$output" != *"P234"* ]]
-}
-
-@test "allow: defer citing dated ADR within +/-5 lines exits silent" {
-  cat > "$RETRO_FILE" <<'EOF'
-# Session 6 Retro
-
-## Codification Candidates
-
-Deferred pending ADR-044 confirmation criterion 3 graduation
-(2026-05-25). Reassess after the criterion lands.
-EOF
-  run run_hook_with_write
-  [ "$status" -eq 0 ]
-  [[ "$stderr" != *"P234"* ]]
-  [[ "$output" != *"P234"* ]]
-}
-
-# --- Exception allowlist ---
-
-@test "allow: deferred-per-Branch-B allowlist phrase exits silent" {
-  cat > "$RETRO_FILE" <<'EOF'
-# Session 3 Retro
-
-## Topic File Rotation Candidates
-
-| File | Action |
-|------|--------|
-| governance-workflow.md (ratio 1.5x) | leave-as-is — deferred per Branch B |
-| hooks-and-gates.md (ratio 1.3x) | leave-as-is — deferred per Branch B |
-EOF
-  run run_hook_with_write
-  [ "$status" -eq 0 ]
-  [[ "$stderr" != *"P234"* ]]
-  [[ "$output" != *"P234"* ]]
-}
-
-# --- Path / tool short-circuits ---
-
-@test "allow: tool_name != Write/Edit exits silent" {
-  cat > "$RETRO_FILE" <<'EOF'
-Deferred this retro per session-length constraint. Next retro should run.
-EOF
-  payload=$(jq -n --arg p "$RETRO_FILE" '{
-    session_id: "x",
-    tool_name: "Bash",
-    tool_input: { command: "ls" },
-    tool_response: { stdout: "" }
-  }')
-  run bash -c "echo '$payload' | bash '$HOOK'"
-  [ "$status" -eq 0 ]
-  [[ "$stderr" != *"P234"* ]]
-  [[ "$output" != *"P234"* ]]
-}
-
-@test "allow: file_path outside docs/retros/ exits silent" {
-  OTHER="$TMPDIR_/docs/problems/foo.md"
-  mkdir -p "$(dirname "$OTHER")"
-  cat > "$OTHER" <<'EOF'
-Deferred this retro per session-length constraint. Next retro should run a full pass.
-EOF
-  payload=$(jq -n --arg p "$OTHER" '{
-    session_id: "x",
-    tool_name: "Write",
-    tool_input: { file_path: $p, content: "" },
-    tool_response: { success: true }
-  }')
-  run bash -c "echo '$payload' | bash '$HOOK'"
-  [ "$status" -eq 0 ]
-  [[ "$stderr" != *"P234"* ]]
-  [[ "$output" != *"P234"* ]]
-}
-
-@test "allow: missing file_path exits silent" {
-  payload=$(jq -n '{
-    session_id: "x",
-    tool_name: "Write",
-    tool_input: {},
-    tool_response: { success: true }
-  }')
-  run bash -c "echo '$payload' | bash '$HOOK'"
-  [ "$status" -eq 0 ]
-  [[ "$stderr" != *"P234"* ]]
-  [[ "$output" != *"P234"* ]]
-}
-
-# --- Crash safety ---
-
-@test "allow: malformed JSON input does not crash the hook" {
-  run bash -c "echo 'not-json' | bash '$HOOK'"
-  [ "$status" -eq 0 ]
-  # Either silent OR a single advisory — but never a non-zero exit.
-}
-
-@test "allow: non-existent retro file exits silent" {
-  GHOST="$TMPDIR_/docs/retros/does-not-exist.md"
-  payload=$(jq -n --arg p "$GHOST" '{
-    session_id: "x",
-    tool_name: "Write",
-    tool_input: { file_path: $p, content: "" },
-    tool_response: { success: true }
-  }')
-  run bash -c "echo '$payload' | bash '$HOOK'"
-  [ "$status" -eq 0 ]
-  [[ "$stderr" != *"P234"* ]]
-  [[ "$output" != *"P234"* ]]
-}
-
-# --- Advisory budget per ADR-045 ---
-
-@test "advisory output stays under ADR-045 1000-byte honour-system ceiling" {
-  cat > "$RETRO_FILE" <<'EOF'
-# Session 3 Retro
-
-## Signal-vs-Noise Pass
-
-Deferred this retro per session-length constraint. Next retro
-should run a full pass.
-EOF
-  emit_write_payload "$RETRO_FILE" > "$TMPDIR_/payload.json"
-  # Capture combined stdout+stderr; advisory channel is stderr per
-  # PostToolUse precedent (itil-rfc-trailer-advisory.sh).
-  combined=$(bash "$HOOK" < "$TMPDIR_/payload.json" 2>&1)
-  [ -n "$combined" ]
-  [ "${#combined}" -lt 1000 ]
-}

package/hooks/test/itil-fix-title-lifecycle-advisory.bats

@@ -1,129 +0,0 @@
-#!/usr/bin/env bats
-
-# P345/RFC-044/STORY-038: itil-fix-title-lifecycle-advisory.sh — post-commit
-# ADVISORY for fix-titled commits whose named P<NNN> ticket is still Open on
-# disk (no paired lifecycle transition in the commit). Advisory-only per
-# ADR-092: never blocks, exit 0 on every path. Behavioural (ADR-052).
-
-setup() {
-  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../.." && pwd)"
-  HOOK="$REPO_ROOT/packages/itil/hooks/itil-fix-title-lifecycle-advisory.sh"
-  DIR="$(mktemp -d)"; cd "$DIR"
-  git init -q; git config user.email t@e.x; git config user.name t
-  mkdir -p docs/problems/open docs/problems/known-error
-  echo x > seed; git add -A; git commit -qm "chore: seed"
-}
-teardown() { cd /; rm -rf "$DIR"; }
-
-# Feed the hook a PostToolUse Bash payload for a `git commit` command.
-run_hook() {
-  printf '{"tool_name":"Bash","tool_input":{"command":"git commit -m x"}}' | bash "$HOOK"
-}
-
-commit() { git commit -q --allow-empty -m "$1"; }
-
-@test "advises when a fix-titled commit names a still-Open ticket" {
-  echo t > docs/problems/open/501-widget-breaks.md
-  commit "fix(itil): P501 stop the widget breaking"
-  run run_hook
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"P501"* ]]
-  [[ "$output" == *"Open"* ]]
-  [[ "$output" == *"transition"* ]]
-}
-
-@test "silent when the named ticket is known-error (transition already happened)" {
-  echo t > docs/problems/known-error/501-widget-breaks.md
-  commit "fix(itil): P501 stop the widget breaking"
-  run run_hook
-  [ "$status" -eq 0 ]
-  [ -z "$output" ]
-}
-
-@test "silent when the named ticket does not exist on disk" {
-  commit "fix(itil): P999 phantom ticket"
-  run run_hook
-  [ "$status" -eq 0 ]
-  [ -z "$output" ]
-}
-
-@test "silent on a non-fix-typed subject naming a P token" {
-  echo t > docs/problems/open/501-widget-breaks.md
-  commit "docs(problems): P501 add investigation notes"
-  run run_hook
-  [ "$status" -eq 0 ]
-  [ -z "$output" ]
-}
-
-@test "silent on a fix-typed subject with no P token" {
-  echo t > docs/problems/open/501-widget-breaks.md
-  commit "fix(itil): tidy the widget"
-  run run_hook
-  [ "$status" -eq 0 ]
-  [ -z "$output" ]
-}
-
-@test "advises on the bare fix: form too" {
-  echo t > docs/problems/open/502-gadget-stuck.md
-  commit "fix: P502 unstick the gadget"
-  run run_hook
-  [ "$status" -eq 0 ]
-  [[ "$output" == *"P502"* ]]
-}
-
-@test "total emission stays at or under 300 bytes with many still-Open tickets" {
-  for n in 511 512 513 514 515 516; do
-    echo t > "docs/problems/open/${n}-thing-${n}.md"
-  done
-  commit "fix(itil): P511 P512 P513 P514 P515 P516 mega sweep"
-  run run_hook
-  [ "$status" -eq 0 ]
-  [ -n "$output" ]
-  [ "${#output}" -le 300 ]
-}
-
-@test "bypass env var suppresses the advisory" {
-  echo t > docs/problems/open/501-widget-breaks.md
-  commit "fix(itil): P501 stop the widget breaking"
-  run env BYPASS_FIX_TITLE_LIFECYCLE_ADVISORY=1 bash -c 'printf "{\"tool_name\":\"Bash\",\"tool_input\":{\"command\":\"git commit -m x\"}}" | bash "$0"' "$HOOK"
-  [ "$status" -eq 0 ]
-  [ -z "$output" ]
-}
-
-@test "silent on non-Bash tool payloads" {
-  echo t > docs/problems/open/501-widget-breaks.md
-  commit "fix(itil): P501 stop the widget breaking"
-  run bash -c 'printf "{\"tool_name\":\"Write\",\"tool_input\":{}}" | bash "$0"' "$HOOK"
-  [ "$status" -eq 0 ]
-  [ -z "$output" ]
-}
-
-@test "silent on non-commit Bash commands" {
-  echo t > docs/problems/open/501-widget-breaks.md
-  commit "fix(itil): P501 stop the widget breaking"
-  run bash -c 'printf "{\"tool_name\":\"Bash\",\"tool_input\":{\"command\":\"git log --oneline\"}}" | bash "$0"' "$HOOK"
-  [ "$status" -eq 0 ]
-  [ -z "$output" ]
-}
-
-@test "fail-open silent when docs/problems is absent" {
-  rm -rf docs/problems
-  commit "fix(itil): P501 stop the widget breaking"
-  run run_hook
-  [ "$status" -eq 0 ]
-  [ -z "$output" ]
-}
-
-@test "fail-open silent outside a git work tree" {
-  NOGIT="$(mktemp -d)"; cd "$NOGIT"
-  run run_hook
-  [ "$status" -eq 0 ]
-  [ -z "$output" ]
-  cd /; rm -rf "$NOGIT"
-}
-
-@test "fail-open silent on malformed JSON input" {
-  run bash -c 'printf "not json at all" | bash "$0"' "$HOOK"
-  [ "$status" -eq 0 ]
-  [ -z "$output" ]
-}