@thebassclef/lite 1.0.2 → 1.0.3

package/dist/lite/.claude/hooks/session-reflection.d/tests/45-auto-sync-status.test.sh

@@ -0,0 +1,263 @@
+#!/usr/bin/env bash
+# tier: standard
+# Tests for .claude/hooks/session-reflection.d/45-auto-sync-status.sh
+#
+# V3 of bassclef bet 2026-06-15b — refactored from earned-banner Layer 2
+# to always-on version-status line per operator decision (always-on subsumes
+# earned-banner).
+#
+# New behavior:
+# - Silent when no .bassclef-source.json (bassclef's own repo)
+# - Always emits version-status line when .bassclef-source.json + V1 fields exist
+# - Critical-fix entries from adopter-notable-releases.json surface as
+#   additional warning lines BYPASSING age threshold
+#
+# Run: bash .claude/hooks/session-reflection.d/tests/45-auto-sync-status.test.sh
+
+set -u
+
+REPO_ROOT="$(cd "$(dirname "$0")/../../../.." && pwd)"
+HOOK="${REPO_ROOT}/.claude/hooks/session-reflection.d/45-auto-sync-status.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/state" "$TMPDIR_TEST/.claude/hooks"
+}
+
+teardown_tmpdir() {
+  rm -rf "$TMPDIR_TEST"
+}
+
+# Run hook with CWD=$TMPDIR_TEST + CLAUDE_PROJECT_DIR=$TMPDIR_TEST
+run_hook() {
+  HOOK_STDOUT=$(cd "$TMPDIR_TEST" && CLAUDE_PROJECT_DIR="$TMPDIR_TEST" bash "$HOOK" 2>&1)
+  HOOK_EXIT=$?
+}
+
+if [ ! -x "$HOOK" ]; then
+  chmod +x "$HOOK"
+fi
+
+echo "Testing $HOOK"
+echo "---"
+
+# === Silent paths ===
+
+setup_tmpdir
+# No .bassclef-source.json
+run_hook
+[ "$HOOK_EXIT" = "0" ] && [ -z "$HOOK_STDOUT" ] && \
+  assert "silent when .bassclef-source.json missing" "ok" || \
+  assert "silent when .bassclef-source.json missing" "exit=$HOOK_EXIT output=$HOOK_STDOUT"
+teardown_tmpdir
+
+# === Always-on version-status line ===
+
+setup_tmpdir
+cat > "$TMPDIR_TEST/.bassclef-source.json" <<EOF
+{
+  "source_url": "https://api.github.com/repos/sunj-labs/bassclef/tarball/main",
+  "auto_sync": false,
+  "installed_sha": "abc1234",
+  "installed_at": "2026-06-10T12:00:00Z"
+}
+EOF
+run_hook
+[ "$HOOK_EXIT" = "0" ] && echo "$HOOK_STDOUT" | grep -q "version status" && \
+  assert "always-on line fires with installed_sha + installed_at" "ok" || \
+  assert "always-on line fires with installed_sha + installed_at" "exit=$HOOK_EXIT output=$HOOK_STDOUT"
+teardown_tmpdir
+
+setup_tmpdir
+cat > "$TMPDIR_TEST/.bassclef-source.json" <<EOF
+{
+  "auto_sync": true,
+  "installed_sha": "abc1234",
+  "installed_at": "2026-06-15T12:00:00Z"
+}
+EOF
+run_hook
+[ "$HOOK_EXIT" = "0" ] && echo "$HOOK_STDOUT" | grep -q "version status" && \
+  assert "always-on line fires even with auto_sync ON" "ok" || \
+  assert "always-on line fires even with auto_sync ON" "exit=$HOOK_EXIT output=$HOOK_STDOUT"
+teardown_tmpdir
+
+# === Status line content ===
+
+setup_tmpdir
+cat > "$TMPDIR_TEST/.bassclef-source.json" <<EOF
+{
+  "auto_sync": false,
+  "installed_sha": "abc1234",
+  "installed_at": "2026-06-10T12:00:00Z"
+}
+EOF
+run_hook
+[ "$HOOK_EXIT" = "0" ] && echo "$HOOK_STDOUT" | grep -q "abc1234" && \
+  assert "status line includes installed_sha" "ok" || \
+  assert "status line includes installed_sha" "exit=$HOOK_EXIT output=$HOOK_STDOUT"
+teardown_tmpdir
+
+setup_tmpdir
+cat > "$TMPDIR_TEST/.bassclef-source.json" <<EOF
+{
+  "auto_sync": false,
+  "installed_sha": "abc1234",
+  "installed_at": "2026-06-10T12:00:00Z"
+}
+EOF
+run_hook
+[ "$HOOK_EXIT" = "0" ] && echo "$HOOK_STDOUT" | grep -qE "[0-9]+ day" && \
+  assert "status line includes days-behind" "ok" || \
+  assert "status line includes days-behind" "exit=$HOOK_EXIT output=$HOOK_STDOUT"
+teardown_tmpdir
+
+# === Auto-sync state surfaced ===
+
+setup_tmpdir
+cat > "$TMPDIR_TEST/.bassclef-source.json" <<EOF
+{
+  "auto_sync": false,
+  "installed_sha": "abc1234",
+  "installed_at": "2026-06-10T12:00:00Z"
+}
+EOF
+run_hook
+[ "$HOOK_EXIT" = "0" ] && echo "$HOOK_STDOUT" | grep -qiE "auto.update.*off|auto.sync.*off" && \
+  assert "auto_sync OFF surfaced in status" "ok" || \
+  assert "auto_sync OFF surfaced in status" "exit=$HOOK_EXIT output=$HOOK_STDOUT"
+teardown_tmpdir
+
+setup_tmpdir
+cat > "$TMPDIR_TEST/.bassclef-source.json" <<EOF
+{
+  "auto_sync": true,
+  "installed_sha": "abc1234",
+  "installed_at": "2026-06-15T12:00:00Z"
+}
+EOF
+run_hook
+[ "$HOOK_EXIT" = "0" ] && echo "$HOOK_STDOUT" | grep -qiE "auto.update.*on|auto.sync.*on" && \
+  assert "auto_sync ON surfaced in status" "ok" || \
+  assert "auto_sync ON surfaced in status" "exit=$HOOK_EXIT output=$HOOK_STDOUT"
+teardown_tmpdir
+
+# === Critical-fix bypass ===
+# Critical-fix entries shipped after install date should fire the warning,
+# regardless of age threshold (which previously gated the earned-banner).
+
+setup_tmpdir
+cat > "$TMPDIR_TEST/.bassclef-source.json" <<EOF
+{
+  "auto_sync": false,
+  "installed_sha": "abc1234",
+  "installed_at": "2026-06-10T12:00:00Z"
+}
+EOF
+cat > "$TMPDIR_TEST/state/adopter-notable-releases.json" <<EOF
+{
+  "releases": [
+    {
+      "published_at": "2026-06-15T12:00:00Z",
+      "kind": "critical_fix",
+      "summary": "fix: serious bug in auto-save"
+    }
+  ]
+}
+EOF
+run_hook
+[ "$HOOK_EXIT" = "0" ] && echo "$HOOK_STDOUT" | grep -qiE "critical.fix|critical fix" && \
+  assert "critical-fix entry surfaces warning" "ok" || \
+  assert "critical-fix entry surfaces warning" "exit=$HOOK_EXIT output=$HOOK_STDOUT"
+teardown_tmpdir
+
+# Critical-fix BEFORE install date should NOT fire.
+setup_tmpdir
+cat > "$TMPDIR_TEST/.bassclef-source.json" <<EOF
+{
+  "auto_sync": false,
+  "installed_sha": "abc1234",
+  "installed_at": "2026-06-15T12:00:00Z"
+}
+EOF
+cat > "$TMPDIR_TEST/state/adopter-notable-releases.json" <<EOF
+{
+  "releases": [
+    {
+      "published_at": "2026-06-10T12:00:00Z",
+      "kind": "critical_fix",
+      "summary": "fix: older bug"
+    }
+  ]
+}
+EOF
+run_hook
+[ "$HOOK_EXIT" = "0" ] && ! echo "$HOOK_STDOUT" | grep -qiE "critical.fix|critical fix" && \
+  assert "critical-fix before install does NOT fire" "ok" || \
+  assert "critical-fix before install does NOT fire" "exit=$HOOK_EXIT output=$HOOK_STDOUT"
+teardown_tmpdir
+
+# === Override path ===
+
+setup_tmpdir
+cat > "$TMPDIR_TEST/.bassclef-source.json" <<EOF
+{
+  "auto_sync": false,
+  "installed_sha": "abc1234",
+  "installed_at": "2026-06-10T12:00:00Z"
+}
+EOF
+HOOK_STDOUT=$(cd "$TMPDIR_TEST" && CLAUDE_PROJECT_DIR="$TMPDIR_TEST" SKIP_AUTO_SYNC_STATUS=1 bash "$HOOK" 2>&1)
+HOOK_EXIT=$?
+[ "$HOOK_EXIT" = "0" ] && [ -z "$HOOK_STDOUT" ] && \
+  assert "SKIP_AUTO_SYNC_STATUS=1 silences" "ok" || \
+  assert "SKIP_AUTO_SYNC_STATUS=1 silences" "exit=$HOOK_EXIT output=$HOOK_STDOUT"
+teardown_tmpdir
+
+# === Backward-compat: no V1 fields ===
+# Adopter that hasn't yet adopted V1 (no installed_sha/installed_at) —
+# hook should still be silent (not crash, not emit garbage).
+
+setup_tmpdir
+cat > "$TMPDIR_TEST/.bassclef-source.json" <<EOF
+{
+  "auto_sync": false
+}
+EOF
+run_hook
+[ "$HOOK_EXIT" = "0" ] && \
+  assert "hook does not crash on missing V1 fields" "ok" || \
+  assert "hook does not crash on missing V1 fields" "exit=$HOOK_EXIT output=$HOOK_STDOUT"
+teardown_tmpdir
+
+# === Summary ===
+
+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