@thebassclef/lite 1.0.2 → 1.0.4

package/dist/lite/.claude/hooks/session-reflection.d/tests/56-roadmap-reconcile-gate.test.sh

@@ -0,0 +1,227 @@
+#!/usr/bin/env bash
+# tier: standard
+# Tests for .claude/hooks/session-reflection.d/56-roadmap-reconcile-gate.sh
+#
+# Per bassclef#1183 + bassclef#504: mechanical enforcement at SessionStart
+# in bassclef repo. Fires BLOCKED on feature branches without a
+# roadmap-reconcile marker, so /sprint + /longrun prep procedures
+# (WU-3/WU-4) refuse to propose work until /roadmap-reconcile --dry-run
+# has run (and either showed no drift OR drift was resolved or deferred).
+#
+# Tier 0 strict-TDD per bassclef#1037; test list block + sufficiency
+# criteria per bassclef#1090 + bassclef#1098.
+#
+# Run: bash .claude/hooks/session-reflection.d/tests/56-roadmap-reconcile-gate.test.sh
+
+set -u
+
+# test-list:
+# [x]Branch: fires BLOCKED in consumer repo too (bassclef#1275 — drop bassclef-only guard)
+# [x]Branch: silent when in bassclef repo AND marker exists for current branch
+# [x]Branch: fires BLOCKED when in bassclef repo AND no marker for current branch
+# [x]Branch: silent on main (no marker required for trunk)
+# [x]Exit: exit 0 on all paths (SessionStart hooks emit informational BLOCKED)
+# [x]Exit: exit 0 when override SKIP_ROADMAP_RECONCILE_GATE=1 is set
+# [x]Exit: exit 0 on missing binaries (graceful)
+# [x]BLOCKED content: emits "🛑 BLOCKED: roadmap-reconcile-gate" header
+# [x]BLOCKED content: includes the /roadmap-reconcile --dry-run dispatch command
+# [x]BLOCKED content: includes resolution path naming the marker file pattern
+# [x]BLOCKED content: includes override path (SKIP_ROADMAP_RECONCILE_GATE=1)
+# [x]BLOCKED content: cites bassclef#1183 + bassclef#504
+# [x]Override: SKIP_ROADMAP_RECONCILE_GATE=1 silences output entirely
+# [x]Marker: marker at state/markers/roadmap-reconcile-gate/<branch-slug>.marker silences hook
+# [x]Marker: marker on a DIFFERENT branch does NOT silence
+# [x]Marker: handles branch slug with slash (feature/foo → feature-foo)
+# [x]External-binary: silent when gh binary missing (no crash)
+# [~] State-file: no specific state files read; this gate is marker-only. Reason: hook only checks marker existence.
+# [~] Trap: no tmpfiles created in hook; no trap needed. Reason: read-only on filesystem except marker check.
+
+REPO_ROOT="$(cd "$(dirname "$0")/../../../.." && pwd)"
+HOOK="${REPO_ROOT}/.claude/hooks/session-reflection.d/56-roadmap-reconcile-gate.sh"
+PASS=0
+FAIL=0
+FAILED_TESTS=()
+
+assert() {
+  local name="$1"
+  local result="$2"
+  if [ "$result" = "ok" ]; then
+    PASS=$((PASS + 1))
+    echo "  PASS  $name"
+  else
+    FAIL=$((FAIL + 1))
+    FAILED_TESTS+=("$name")
+    echo "  FAIL  $name — $result"
+  fi
+}
+
+setup_tmpdir() {
+  TMPDIR_TEST="$(mktemp -d)"
+  mkdir -p "$TMPDIR_TEST/.claude/hooks" "$TMPDIR_TEST/state/markers/roadmap-reconcile-gate"
+  (cd "$TMPDIR_TEST" && git init -q && git config user.email t@t && git config user.name t \
+    && git checkout -q -b feature/test-branch 2>/dev/null \
+    && touch README.md && git add README.md && git commit -q -m init 2>/dev/null) || true
+}
+
+teardown_tmpdir() {
+  rm -rf "$TMPDIR_TEST"
+}
+
+stub_gh_bassclef() {
+  cat > "$TMPDIR_TEST/gh" <<'EOF'
+#!/usr/bin/env bash
+if [ "$1" = "repo" ] && [ "$2" = "view" ]; then
+  echo "${GH_STUB_REPO_NAME:-bassclef}"
+  exit 0
+fi
+exit 0
+EOF
+  chmod +x "$TMPDIR_TEST/gh"
+}
+
+run_hook() {
+  HOOK_STDOUT=$(cd "$TMPDIR_TEST" \
+    && PATH="$TMPDIR_TEST:$PATH" \
+    && CLAUDE_PROJECT_DIR="$TMPDIR_TEST" \
+    bash "$HOOK" 2>&1)
+  HOOK_EXIT=$?
+}
+
+if [ ! -x "$HOOK" ]; then
+  chmod +x "$HOOK" 2>/dev/null || true
+fi
+
+echo "Testing $HOOK"
+echo "---"
+
+# Fires BLOCKED in consumer repo too — adopter inherits the discipline (bassclef#1275)
+setup_tmpdir
+stub_gh_bassclef
+GH_STUB_REPO_NAME=poa run_hook
+[ "$HOOK_EXIT" = "0" ] && echo "$HOOK_STDOUT" | grep -q "roadmap-reconcile-gate" && \
+  assert "fires BLOCKED in consumer repo too (bassclef#1275)" "ok" || \
+  assert "fires BLOCKED in consumer repo too (bassclef#1275)" "exit=$HOOK_EXIT output=$HOOK_STDOUT"
+teardown_tmpdir
+
+# Fires BLOCKED in bassclef + no marker
+setup_tmpdir
+stub_gh_bassclef
+run_hook
+[ "$HOOK_EXIT" = "0" ] && echo "$HOOK_STDOUT" | grep -q "roadmap-reconcile-gate" && \
+  assert "fires BLOCKED in bassclef with no marker" "ok" || \
+  assert "fires BLOCKED in bassclef with no marker" "exit=$HOOK_EXIT output=$HOOK_STDOUT"
+teardown_tmpdir
+
+# Silent on main
+setup_tmpdir
+stub_gh_bassclef
+(cd "$TMPDIR_TEST" && git checkout -q -b main 2>/dev/null || git checkout -q main 2>/dev/null)
+run_hook
+[ "$HOOK_EXIT" = "0" ] && [ -z "$HOOK_STDOUT" ] && \
+  assert "silent on main branch" "ok" || \
+  assert "silent on main branch" "exit=$HOOK_EXIT output=$HOOK_STDOUT"
+teardown_tmpdir
+
+# BLOCKED content checks
+setup_tmpdir
+stub_gh_bassclef
+run_hook
+echo "$HOOK_STDOUT" | grep -q "🛑 BLOCKED: roadmap-reconcile-gate" && \
+  assert "BLOCKED header emitted" "ok" || \
+  assert "BLOCKED header emitted" "output=$HOOK_STDOUT"
+teardown_tmpdir
+
+setup_tmpdir
+stub_gh_bassclef
+run_hook
+echo "$HOOK_STDOUT" | grep -q "roadmap-reconcile --dry-run" && \
+  assert "dispatch command surfaced in BLOCKED" "ok" || \
+  assert "dispatch command surfaced in BLOCKED" "output=$HOOK_STDOUT"
+teardown_tmpdir
+
+setup_tmpdir
+stub_gh_bassclef
+run_hook
+echo "$HOOK_STDOUT" | grep -qE "state/markers/roadmap-reconcile-gate" && \
+  assert "resolution path names marker file pattern" "ok" || \
+  assert "resolution path names marker file pattern" "output=$HOOK_STDOUT"
+teardown_tmpdir
+
+setup_tmpdir
+stub_gh_bassclef
+run_hook
+echo "$HOOK_STDOUT" | grep -q "SKIP_ROADMAP_RECONCILE_GATE=1" && \
+  assert "override path documented in BLOCKED" "ok" || \
+  assert "override path documented in BLOCKED" "output=$HOOK_STDOUT"
+teardown_tmpdir
+
+setup_tmpdir
+stub_gh_bassclef
+run_hook
+echo "$HOOK_STDOUT" | grep -q "bassclef#1183" && echo "$HOOK_STDOUT" | grep -q "bassclef#504" && \
+  assert "BLOCKED cites bassclef#1183 + bassclef#504" "ok" || \
+  assert "BLOCKED cites bassclef#1183 + bassclef#504" "output=$HOOK_STDOUT"
+teardown_tmpdir
+
+# Override silences
+setup_tmpdir
+stub_gh_bassclef
+HOOK_STDOUT=$(cd "$TMPDIR_TEST" && PATH="$TMPDIR_TEST:$PATH" CLAUDE_PROJECT_DIR="$TMPDIR_TEST" SKIP_ROADMAP_RECONCILE_GATE=1 bash "$HOOK" 2>&1)
+HOOK_EXIT=$?
+[ "$HOOK_EXIT" = "0" ] && [ -z "$HOOK_STDOUT" ] && \
+  assert "SKIP_ROADMAP_RECONCILE_GATE=1 silences hook" "ok" || \
+  assert "SKIP_ROADMAP_RECONCILE_GATE=1 silences hook" "exit=$HOOK_EXIT output=$HOOK_STDOUT"
+teardown_tmpdir
+
+# Marker for current branch silences
+setup_tmpdir
+stub_gh_bassclef
+touch "$TMPDIR_TEST/state/markers/roadmap-reconcile-gate/feature-test-branch.marker"
+run_hook
+[ "$HOOK_EXIT" = "0" ] && [ -z "$HOOK_STDOUT" ] && \
+  assert "marker for current branch silences hook" "ok" || \
+  assert "marker for current branch silences hook" "exit=$HOOK_EXIT output=$HOOK_STDOUT"
+teardown_tmpdir
+
+# Marker on other branch does NOT silence
+setup_tmpdir
+stub_gh_bassclef
+touch "$TMPDIR_TEST/state/markers/roadmap-reconcile-gate/feature-OTHER-branch.marker"
+run_hook
+[ "$HOOK_EXIT" = "0" ] && echo "$HOOK_STDOUT" | grep -q "roadmap-reconcile-gate" && \
+  assert "marker on other branch does NOT silence" "ok" || \
+  assert "marker on other branch does NOT silence" "exit=$HOOK_EXIT output=$HOOK_STDOUT"
+teardown_tmpdir
+
+# Slash-in-branch-name
+setup_tmpdir
+stub_gh_bassclef
+touch "$TMPDIR_TEST/state/markers/roadmap-reconcile-gate/feature-test-branch.marker"
+run_hook
+[ "$HOOK_EXIT" = "0" ] && [ -z "$HOOK_STDOUT" ] && \
+  assert "slash-in-branch-name handled (feature/foo → feature-foo)" "ok" || \
+  assert "slash-in-branch-name handled (feature/foo → feature-foo)" "exit=$HOOK_EXIT output=$HOOK_STDOUT"
+teardown_tmpdir
+
+# Missing gh binary graceful
+setup_tmpdir
+HOOK_STDOUT=$(cd "$TMPDIR_TEST" && PATH="/usr/bin:/bin" CLAUDE_PROJECT_DIR="$TMPDIR_TEST" bash "$HOOK" 2>&1)
+HOOK_EXIT=$?
+[ "$HOOK_EXIT" = "0" ] && \
+  assert "missing gh binary does not crash" "ok" || \
+  assert "missing gh binary does not crash" "exit=$HOOK_EXIT output=$HOOK_STDOUT"
+teardown_tmpdir
+
+echo "---"
+echo "Results: $PASS passed, $FAIL failed"
+
+if [ "$FAIL" -gt 0 ]; then
+  echo ""
+  echo "Failed tests:"
+  for t in "${FAILED_TESTS[@]}"; do
+    echo "  - $t"
+  done
+  exit 1
+fi
+
+exit 0

package/dist/lite/.claude/hooks/session-reflection.d/tests/75-prior-r8-findings.test.sh

@@ -0,0 +1,142 @@
+#!/bin/bash
+# tier: standard
+# fixtures: bassclef-upstream-self
+# Tier 0 strict TDD for 75-prior-r8-findings.sh (bet 2026-07-03a WU-4)
+#
+# test-list:
+# [x] SKIP_PRIOR_R8_FINDINGS=1 override silent
+# [x] No chronicle dir silent
+# [x] Empty chronicle dir silent
+# [x] Chronicles without R8 mentions silent
+# [x] Chronicle with R8 mention emits summary
+# [x] Multiple chronicles with R8 mentions aggregate
+# [x] Exit 0 always
+# [x] Names the file that matched
+
+set +e
+
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+HOOK="$SCRIPT_DIR/../75-prior-r8-findings.sh"
+
+PASS=0
+FAIL=0
+pass() { echo "PASS: $1"; PASS=$((PASS + 1)); }
+fail() { echo "FAIL: $1"; echo "  $2"; FAIL=$((FAIL + 1)); }
+
+# Setup — temp dir with fake chronicle/ inside
+setup_workspace() {
+  TMPDIR_TEST=$(mktemp -d)
+  mkdir -p "$TMPDIR_TEST/chronicle"
+}
+
+teardown_workspace() {
+  rm -rf "$TMPDIR_TEST"
+}
+
+# === Test 1 — SKIP override ===
+setup_workspace
+echo "R8 finding here." > "$TMPDIR_TEST/chronicle/2026-07-01-test.md"
+OUTPUT=$(cd "$TMPDIR_TEST" && SKIP_PRIOR_R8_FINDINGS=1 bash "$HOOK" 2>&1)
+STATUS=$?
+teardown_workspace
+if [ $STATUS -eq 0 ] && [ -z "$OUTPUT" ]; then
+  pass "SKIP override silent"
+else
+  fail "SKIP override" "status=$STATUS output=$OUTPUT"
+fi
+
+# === Test 2 — No chronicle dir ===
+TMPDIR_TEST=$(mktemp -d)
+OUTPUT=$(cd "$TMPDIR_TEST" && bash "$HOOK" 2>&1)
+STATUS=$?
+rm -rf "$TMPDIR_TEST"
+if [ $STATUS -eq 0 ] && [ -z "$OUTPUT" ]; then
+  pass "No chronicle dir silent"
+else
+  fail "No chronicle dir" "status=$STATUS output=$OUTPUT"
+fi
+
+# === Test 3 — Empty chronicle dir ===
+setup_workspace
+OUTPUT=$(cd "$TMPDIR_TEST" && bash "$HOOK" 2>&1)
+STATUS=$?
+teardown_workspace
+if [ $STATUS -eq 0 ] && [ -z "$OUTPUT" ]; then
+  pass "Empty chronicle silent"
+else
+  fail "Empty chronicle" "status=$STATUS output=$OUTPUT"
+fi
+
+# === Test 4 — Chronicles without R8 mentions ===
+setup_workspace
+echo "Nothing special here. Just prose." > "$TMPDIR_TEST/chronicle/2026-07-01-nofind.md"
+OUTPUT=$(cd "$TMPDIR_TEST" && bash "$HOOK" 2>&1)
+STATUS=$?
+teardown_workspace
+if [ $STATUS -eq 0 ] && [ -z "$OUTPUT" ]; then
+  pass "No R8 mentions silent"
+else
+  fail "No R8 mentions" "status=$STATUS output=$OUTPUT"
+fi
+
+# === Test 5 — Chronicle with R8 mention emits summary ===
+setup_workspace
+cat > "$TMPDIR_TEST/chronicle/2026-07-01-r8find.md" <<'EOM'
+This session caught an R8 pattern.
+The agent asserted without validation.
+Claim-shape prose without cited sources.
+EOM
+OUTPUT=$(cd "$TMPDIR_TEST" && bash "$HOOK" 2>&1)
+STATUS=$?
+teardown_workspace
+if [ $STATUS -eq 0 ] && echo "$OUTPUT" | grep -q "PRIOR R8 FINDINGS"; then
+  pass "R8 mention emits summary"
+else
+  fail "R8 mention summary" "status=$STATUS output=$OUTPUT"
+fi
+
+# === Test 6 — Multiple chronicles aggregate ===
+setup_workspace
+echo "R8 finding one" > "$TMPDIR_TEST/chronicle/2026-07-01a.md"
+echo "R8 finding two" > "$TMPDIR_TEST/chronicle/2026-07-02b.md"
+echo "R8 finding three" > "$TMPDIR_TEST/chronicle/2026-07-03c.md"
+OUTPUT=$(cd "$TMPDIR_TEST" && bash "$HOOK" 2>&1)
+STATUS=$?
+teardown_workspace
+COUNT=$(echo "$OUTPUT" | grep -c "R8-related mentions")
+if [ $STATUS -eq 0 ] && [ "$COUNT" -ge 2 ]; then
+  pass "Multiple chronicles aggregate"
+else
+  fail "Multiple aggregate" "status=$STATUS count=$COUNT output=$OUTPUT"
+fi
+
+# === Test 7 — Exit 0 always (even with parse issues) ===
+setup_workspace
+# Put a binary file to test robustness
+head -c 100 /dev/urandom > "$TMPDIR_TEST/chronicle/2026-07-01-binary.md"
+OUTPUT=$(cd "$TMPDIR_TEST" && bash "$HOOK" 2>&1)
+STATUS=$?
+teardown_workspace
+if [ $STATUS -eq 0 ]; then
+  pass "Exit 0 even with binary"
+else
+  fail "Exit 0 with binary" "status=$STATUS"
+fi
+
+# === Test 8 — Names the file that matched ===
+setup_workspace
+echo "assertion-shape catch here" > "$TMPDIR_TEST/chronicle/2026-07-03-named.md"
+OUTPUT=$(cd "$TMPDIR_TEST" && bash "$HOOK" 2>&1)
+STATUS=$?
+teardown_workspace
+if [ $STATUS -eq 0 ] && echo "$OUTPUT" | grep -q "2026-07-03-named.md"; then
+  pass "Names matched file"
+else
+  fail "Names matched file" "status=$STATUS output=$OUTPUT"
+fi
+
+echo ""
+echo "==========================================="
+echo "  PASS: $PASS  FAIL: $FAIL"
+echo "==========================================="
+[ $FAIL -eq 0 ]

package/dist/lite/.claude/hooks/session-reflection.d/tests/90-release-backlog.test.sh

@@ -0,0 +1,44 @@
+#!/usr/bin/env bash
+# tier: standard
+# Minimal Tier 1 sibling test for 90-release-backlog.sh.
+# Coordinator-level integration coverage lives at:
+#   .claude/hooks/tests/session-reflection-coordinator.test.sh
+
+set -u
+
+REPO_ROOT="$(cd "$(dirname "$0")/../../../.." && pwd)"
+MODULE="${REPO_ROOT}/.claude/hooks/session-reflection.d/90-release-backlog.sh"
+
+PASS=0
+FAIL=0
+
+assert() {
+  if [ "$2" = "ok" ]; then PASS=$((PASS+1)); echo "  PASS  $1"; else FAIL=$((FAIL+1)); echo "  FAIL  $1 — $2"; fi
+}
+
+echo "Testing $MODULE (minimal stub — full coverage via coordinator test)"
+
+if [ -f "$MODULE" ]; then
+  assert "module file present" "ok"
+else
+  assert "module file present" "not found at $MODULE"
+fi
+
+if bash -n "$MODULE" 2>/dev/null; then
+  assert "syntax valid" "ok"
+else
+  assert "syntax valid" "bash -n reported syntax error"
+fi
+
+# Cure b (2026-07-30) converted inline `... || exit 0` patterns to the
+# dual pattern so the coordinator survives when this module short-circuits.
+if grep -q "return 0 2>/dev/null || exit 0" "$MODULE"; then
+  assert "cure b dual pattern present" "ok"
+else
+  assert "cure b dual pattern present" "expected 'return 0 2>/dev/null || exit 0' in module"
+fi
+
+echo ""
+echo "Results: $PASS passed, $FAIL failed"
+[ "$FAIL" -gt 0 ] && exit 1
+exit 0

package/dist/lite/.claude/hooks/trace-helper.sh

@@ -0,0 +1,22 @@
+#!/bin/bash
+# tier: lite
+# SDLC Trace Log helper — sourced by all hooks.
+# Appends a trigger line to docs/sdlc-traces/YYYY-MM-DD.log
+#
+# Usage: source this file, then call:
+#   trace_log "hook-name" "trigger-description"
+
+TRACE_DIR="docs/sdlc-traces"
+
+trace_log() {
+  local hook="$1"
+  local trigger="$2"
+  local timestamp
+  timestamp=$(date +%H:%M:%S)
+  local today
+  today=$(date +%Y-%m-%d)
+  local log_file="${TRACE_DIR}/${today}.log"
+
+  mkdir -p "$TRACE_DIR" 2>/dev/null || true
+  echo "${timestamp} | ${hook} | ${trigger} | — | —" >> "$log_file" 2>/dev/null || true
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@thebassclef/lite",
-  "version": "1.0.2",
+  "version": "1.0.4",
   "description": "Bassclef CLI — install and upgrade bassclef in your project with two commands.",
   "keywords": ["bassclef", "claude-code", "cli", "scaffolding"],
   "homepage": "https://github.com/sunj-labs/bassclef-cli",