@onlooker-community/ecosystem 0.9.0 → 0.14.0

package/test/bats/archivist-config-files.bats

@@ -0,0 +1,60 @@
+#!/usr/bin/env bats
+# Verifies the on-disk plugin manifests for archivist are wired up correctly.
+
+setup_file() {
+  source "${BATS_TEST_DIRNAME}/../helpers/setup.bash"
+}
+
+@test "archivist plugin.json is valid JSON with name and version" {
+  run jq -e '.name == "archivist" and (.version | length > 0)' \
+    "${REPO_ROOT}/plugins/archivist/.claude-plugin/plugin.json"
+  [ "$status" -eq 0 ]
+}
+
+@test "archivist config.json has plugin_name and archivist.enabled defaulting to false" {
+  run jq -e '.plugin_name == "archivist" and .archivist.enabled == false' \
+    "${REPO_ROOT}/plugins/archivist/config.json"
+  [ "$status" -eq 0 ]
+}
+
+@test "marketplace.json lists ecosystem first and archivist second" {
+  run jq -e '.plugins[0].name == "ecosystem" and .plugins[1].name == "archivist"' \
+    "${REPO_ROOT}/.claude-plugin/marketplace.json"
+  [ "$status" -eq 0 ]
+}
+
+@test "marketplace.json plugin entries omit version (claude reads version from plugin.json)" {
+  # See https://code.claude.com/docs/en/plugins-reference.md#version-management:
+  # plugin.json's version is the cache key. Setting it in both locations is a
+  # documented drift hazard.
+  run jq -e 'all(.plugins[]; has("version") | not)' "${REPO_ROOT}/.claude-plugin/marketplace.json"
+  [ "$status" -eq 0 ]
+}
+
+@test "release-please-manifest.json tracks plugins/archivist" {
+  run jq -e '.["plugins/archivist"]' "${REPO_ROOT}/.release-please-manifest.json"
+  [ "$status" -eq 0 ]
+}
+
+@test "release-please-config.json declares plugins/archivist as a package" {
+  run jq -e '.packages["plugins/archivist"] | type == "object"' \
+    "${REPO_ROOT}/release-please-config.json"
+  [ "$status" -eq 0 ]
+}
+
+@test "archivist hooks.json wires PreCompact manual+auto and SessionStart" {
+  local f="${REPO_ROOT}/plugins/archivist/hooks/hooks.json"
+  run jq -e '
+    (.hooks.PreCompact | length) == 2 and
+    .hooks.PreCompact[0].matcher == "manual" and
+    .hooks.PreCompact[1].matcher == "auto" and
+    .hooks.SessionStart[0].matcher == "*"
+  ' "$f"
+  [ "$status" -eq 0 ]
+}
+
+@test "archivist hook scripts are executable" {
+  for script in archivist-extract.sh archivist-inject.sh; do
+    test -x "${REPO_ROOT}/plugins/archivist/scripts/hooks/$script"
+  done
+}

package/test/bats/archivist-config.bats

@@ -0,0 +1,54 @@
+#!/usr/bin/env bats
+
+setup() {
+  source "${BATS_TEST_DIRNAME}/../helpers/setup.bash"
+  setup_test_env
+
+  PLUGIN_ROOT="${REPO_ROOT}/plugins/archivist"
+  export CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT"
+  # shellcheck disable=SC1091
+  source "${PLUGIN_ROOT}/scripts/lib/archivist-config.sh"
+}
+
+@test "config defaults from plugin config.json: disabled" {
+  archivist_config_load ""
+  run archivist_config_enabled
+  [ "$status" -ne 0 ]
+}
+
+@test "user-level settings.json overlay can enable archivist" {
+  mkdir -p "${HOME}/.claude"
+  printf '%s\n' '{"archivist":{"enabled":true}}' > "${HOME}/.claude/settings.json"
+  archivist_config_load ""
+  run archivist_config_enabled
+  [ "$status" -eq 0 ]
+}
+
+@test "repo-level settings.json overrides user-level" {
+  mkdir -p "${HOME}/.claude"
+  printf '%s\n' '{"archivist":{"enabled":true}}' > "${HOME}/.claude/settings.json"
+  local repo="${BATS_TEST_TMPDIR}/repo"
+  mkdir -p "${repo}/.claude"
+  printf '%s\n' '{"archivist":{"enabled":false}}' > "${repo}/.claude/settings.json"
+  archivist_config_load "$repo"
+  run archivist_config_enabled
+  [ "$status" -ne 0 ]
+}
+
+@test "injection.max_items defaults from config.json" {
+  archivist_config_load ""
+  local v
+  v=$(archivist_config_get '.archivist.injection.max_items')
+  [ "$v" = "8" ]
+}
+
+@test "settings overlay merges deeply (preserves unset defaults)" {
+  mkdir -p "${HOME}/.claude"
+  printf '%s\n' '{"archivist":{"injection":{"max_items":3}}}' > "${HOME}/.claude/settings.json"
+  archivist_config_load ""
+  local overridden default_model
+  overridden=$(archivist_config_get '.archivist.injection.max_items')
+  default_model=$(archivist_config_get '.archivist.extraction.model')
+  [ "$overridden" = "3" ]
+  [ -n "$default_model" ]
+}

package/test/bats/archivist-inject.bats

@@ -0,0 +1,73 @@
+#!/usr/bin/env bats
+
+setup() {
+  source "${BATS_TEST_DIRNAME}/../helpers/setup.bash"
+  setup_test_env
+
+  PLUGIN_ROOT="${REPO_ROOT}/plugins/archivist"
+  export CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT"
+  export ONLOOKER_ECOSYSTEM_ROOT="$REPO_ROOT"
+
+  # Stand up a fake project repo so project-key resolution succeeds.
+  PROJECT_REPO="${BATS_TEST_TMPDIR}/repo"
+  mkdir -p "$PROJECT_REPO"
+  git -C "$PROJECT_REPO" init -q
+  git -C "$PROJECT_REPO" config user.email t@example.com
+  git -C "$PROJECT_REPO" config user.name "Test"
+  git -C "$PROJECT_REPO" remote add origin git@github.com:org/archivist-inject-test.git
+
+  # Compute the project key the hook will use.
+  # shellcheck disable=SC1091
+  source "${PLUGIN_ROOT}/scripts/lib/archivist-project-key.sh"
+  PROJECT_KEY=$(archivist_project_key "$PROJECT_REPO")
+  [ -n "$PROJECT_KEY" ]
+
+  # Seed an artifact on disk for this project.
+  local kind_dir="${ONLOOKER_DIR}/archivist/${PROJECT_KEY}/decisions"
+  mkdir -p "$kind_dir"
+  printf '%s\n' '{
+    "id": "01TESTTESTTESTTESTTESTTEST",
+    "kind": "decision",
+    "summary": "use git remote SHA256 as project key",
+    "detail": "remote URL is stable across machines; falls back to repo path",
+    "files": [],
+    "created_at": "2026-05-22T10:00:00Z",
+    "updated_at": "2026-05-22T10:00:00Z"
+  }' > "${kind_dir}/01TESTTESTTESTTESTTESTTEST.json"
+
+  # Project-scoped settings.json that enables archivist.
+  mkdir -p "${PROJECT_REPO}/.claude"
+  printf '%s\n' '{"archivist":{"enabled":true}}' > "${PROJECT_REPO}/.claude/settings.json"
+}
+
+@test "inject hook is a no-op when archivist is disabled" {
+  rm -f "${PROJECT_REPO}/.claude/settings.json"
+  local input
+  input=$(jq -n --arg cwd "$PROJECT_REPO" '{cwd: $cwd, source: "startup", session_id: "s"}')
+  run bash -c "printf '%s' '$input' | '${PLUGIN_ROOT}/scripts/hooks/archivist-inject.sh'"
+  [ "$status" -eq 0 ]
+  echo "$output" | jq -e '.hookSpecificOutput.additionalContext == ""' >/dev/null
+}
+
+@test "inject hook emits seeded artifact when enabled" {
+  local input
+  input=$(jq -n --arg cwd "$PROJECT_REPO" '{cwd: $cwd, source: "startup", session_id: "s"}')
+  run bash -c "printf '%s' '$input' | '${PLUGIN_ROOT}/scripts/hooks/archivist-inject.sh'"
+  [ "$status" -eq 0 ]
+
+  echo "$output" | jq -e '.hookSpecificOutput.hookEventName == "SessionStart"' >/dev/null
+  local ctx
+  ctx=$(echo "$output" | jq -r '.hookSpecificOutput.additionalContext')
+  [[ "$ctx" == *"use git remote SHA256 as project key"* ]]
+  [[ "$ctx" == *"Archivist injected 1"* ]]
+}
+
+@test "inject hook skips when there is no git context" {
+  local non_git="${BATS_TEST_TMPDIR}/no-git"
+  mkdir -p "$non_git"
+  local input
+  input=$(jq -n --arg cwd "$non_git" '{cwd: $cwd, source: "startup", session_id: "s"}')
+  run bash -c "printf '%s' '$input' | '${PLUGIN_ROOT}/scripts/hooks/archivist-inject.sh'"
+  [ "$status" -eq 0 ]
+  echo "$output" | jq -e '.hookSpecificOutput.additionalContext == ""' >/dev/null
+}

package/test/bats/archivist-project-key.bats

@@ -0,0 +1,75 @@
+#!/usr/bin/env bats
+
+setup() {
+  # shellcheck source=../helpers/setup.bash
+  source "${BATS_TEST_DIRNAME}/../helpers/setup.bash"
+  setup_test_env
+
+  PLUGIN_ROOT="${REPO_ROOT}/plugins/archivist"
+  # shellcheck disable=SC1091
+  source "${PLUGIN_ROOT}/scripts/lib/archivist-project-key.sh"
+}
+
+@test "non-git directory returns empty key" {
+  local d="${BATS_TEST_TMPDIR}/non-git"
+  mkdir -p "$d"
+  run archivist_project_key "$d"
+  [ "$status" -eq 0 ]
+  [ -z "$output" ]
+}
+
+@test "git repo without remote falls back to repo-root hash" {
+  local d="${BATS_TEST_TMPDIR}/local-only-repo"
+  mkdir -p "$d"
+  git -C "$d" init -q
+  git -C "$d" config user.email t@example.com
+  git -C "$d" config user.name "Test"
+
+  local k1
+  k1=$(archivist_project_key "$d")
+  [ -n "$k1" ]
+  [ "${#k1}" -eq 12 ]
+
+  # Stability: a second call returns the same key.
+  local k2
+  k2=$(archivist_project_key "$d")
+  [ "$k1" = "$k2" ]
+}
+
+@test "git repo with remote uses remote hash and ignores local path" {
+  local a="${BATS_TEST_TMPDIR}/clone-a"
+  local b="${BATS_TEST_TMPDIR}/clone-b"
+  mkdir -p "$a" "$b"
+  for d in "$a" "$b"; do
+    git -C "$d" init -q
+    git -C "$d" config user.email t@example.com
+    git -C "$d" config user.name "Test"
+    git -C "$d" remote add origin git@github.com:org/proj.git
+  done
+
+  local ka kb
+  ka=$(archivist_project_key "$a")
+  kb=$(archivist_project_key "$b")
+  [ -n "$ka" ]
+  [ "$ka" = "$kb" ]
+}
+
+@test "different remotes yield different keys" {
+  local a="${BATS_TEST_TMPDIR}/proj-a"
+  local b="${BATS_TEST_TMPDIR}/proj-b"
+  mkdir -p "$a" "$b"
+  for d in "$a" "$b"; do
+    git -C "$d" init -q
+    git -C "$d" config user.email t@example.com
+    git -C "$d" config user.name "Test"
+  done
+  git -C "$a" remote add origin git@github.com:org/proj-a.git
+  git -C "$b" remote add origin git@github.com:org/proj-b.git
+
+  local ka kb
+  ka=$(archivist_project_key "$a")
+  kb=$(archivist_project_key "$b")
+  [ -n "$ka" ]
+  [ -n "$kb" ]
+  [ "$ka" != "$kb" ]
+}

package/test/bats/archivist-storage.bats

@@ -0,0 +1,119 @@
+#!/usr/bin/env bats
+
+setup() {
+  # shellcheck source=../helpers/setup.bash
+  source "${BATS_TEST_DIRNAME}/../helpers/setup.bash"
+  setup_test_env
+
+  PLUGIN_ROOT="${REPO_ROOT}/plugins/archivist"
+  # shellcheck disable=SC1091
+  source "${PLUGIN_ROOT}/scripts/lib/archivist-storage.sh"
+  # shellcheck disable=SC1091
+  source "${PLUGIN_ROOT}/scripts/lib/archivist-ulid.sh"
+
+  REPO="${BATS_TEST_TMPDIR}/repo"
+  mkdir -p "$REPO/src"
+  : > "$REPO/src/known.ts"
+  : > "$REPO/README.md"
+}
+
+@test "validate accepts an existing repo-relative path" {
+  run archivist_validate_repo_path "$REPO" "src/known.ts"
+  [ "$status" -eq 0 ]
+  [ "$output" = "src/known.ts" ]
+}
+
+@test "validate accepts an absolute path inside the repo" {
+  run archivist_validate_repo_path "$REPO" "${REPO}/src/known.ts"
+  [ "$status" -eq 0 ]
+  [ "$output" = "src/known.ts" ]
+}
+
+@test "validate rejects a path outside the repo" {
+  local outside="${BATS_TEST_TMPDIR}/outside.ts"
+  : > "$outside"
+  run archivist_validate_repo_path "$REPO" "$outside"
+  [ "$status" -eq 0 ]
+  [ -z "$output" ]
+}
+
+@test "validate rejects a ../ escape" {
+  run archivist_validate_repo_path "$REPO" "../escaped.ts"
+  [ "$status" -eq 0 ]
+  [ -z "$output" ]
+}
+
+@test "validate rejects a path that does not exist" {
+  run archivist_validate_repo_path "$REPO" "src/missing.ts"
+  [ "$status" -eq 0 ]
+  [ -z "$output" ]
+}
+
+@test "validate_paths_array strips invalid entries" {
+  local input='["src/known.ts","../escape.ts","src/missing.ts","README.md"]'
+  local cleaned compact
+  cleaned=$(archivist_validate_paths_array "$REPO" "$input")
+  compact=$(printf '%s' "$cleaned" | jq -c .)
+  [ "$compact" = '["src/known.ts","README.md"]' ]
+}
+
+@test "storage_init creates kind directories" {
+  local key="abc123def456"
+  archivist_storage_init "$key"
+  [ -d "${ONLOOKER_DIR}/archivist/${key}/decisions" ]
+  [ -d "${ONLOOKER_DIR}/archivist/${key}/dead_ends" ]
+  [ -d "${ONLOOKER_DIR}/archivist/${key}/open_questions" ]
+}
+
+@test "write_artifact creates a ULID-keyed file" {
+  local key="abc123def456"
+  local id
+  id=$(archivist_ulid)
+  local json='{"id":"'"$id"'","kind":"decision","summary":"hello"}'
+  run archivist_storage_write_artifact "$key" "decisions" "$id" "$json"
+  [ "$status" -eq 0 ]
+  [ -f "${ONLOOKER_DIR}/archivist/${key}/decisions/${id}.json" ]
+}
+
+@test "write_artifact rejects unknown kind" {
+  local key="abc123def456"
+  run archivist_storage_write_artifact "$key" "bogus_kind" "01J" '{}'
+  [ "$status" -ne 0 ]
+}
+
+@test "load_ranked sorts pinned items first" {
+  local key="abc123def456"
+  archivist_storage_init "$key"
+
+  # Write two decisions; pin the older one.
+  local older_id="01AAAAAAAAAAAAAAAAAAAAAAAA"
+  local newer_id="01ZZZZZZZZZZZZZZZZZZZZZZZZ"
+  printf '{"id":"%s","summary":"older","created_at":"2026-05-01T00:00:00Z","updated_at":"2026-05-01T00:00:00Z"}\n' "$older_id" \
+    > "${ONLOOKER_DIR}/archivist/${key}/decisions/${older_id}.json"
+  printf '{"id":"%s","summary":"newer","created_at":"2026-05-22T00:00:00Z","updated_at":"2026-05-22T00:00:00Z"}\n' "$newer_id" \
+    > "${ONLOOKER_DIR}/archivist/${key}/decisions/${newer_id}.json"
+  printf '{"ids":["%s"]}\n' "$older_id" > "${ONLOOKER_DIR}/archivist/${key}/pinned.json"
+
+  local ranked
+  ranked=$(archivist_storage_load_ranked "$key")
+  local first_id
+  first_id=$(printf '%s' "$ranked" | jq -r '.[0].id')
+  [ "$first_id" = "$older_id" ]
+}
+
+@test "load_ranked sorts non-pinned by recency desc" {
+  local key="abc123def456"
+  archivist_storage_init "$key"
+  local older_id="01AAAAAAAAAAAAAAAAAAAAAAAA"
+  local newer_id="01ZZZZZZZZZZZZZZZZZZZZZZZZ"
+  printf '{"id":"%s","summary":"older","created_at":"2026-05-01T00:00:00Z","updated_at":"2026-05-01T00:00:00Z"}\n' "$older_id" \
+    > "${ONLOOKER_DIR}/archivist/${key}/decisions/${older_id}.json"
+  printf '{"id":"%s","summary":"newer","created_at":"2026-05-22T00:00:00Z","updated_at":"2026-05-22T00:00:00Z"}\n' "$newer_id" \
+    > "${ONLOOKER_DIR}/archivist/${key}/decisions/${newer_id}.json"
+
+  local ranked
+  ranked=$(archivist_storage_load_ranked "$key")
+  local first_id
+  first_id=$(printf '%s' "$ranked" | jq -r '.[0].id')
+  [ "$first_id" = "$newer_id" ]
+}

package/test/bats/archivist-ulid.bats

@@ -0,0 +1,36 @@
+#!/usr/bin/env bats
+
+setup() {
+  source "${BATS_TEST_DIRNAME}/../helpers/setup.bash"
+  setup_test_env
+  # shellcheck disable=SC1091
+  source "${REPO_ROOT}/plugins/archivist/scripts/lib/archivist-ulid.sh"
+}
+
+@test "archivist_ulid returns a 26-char crockford base32 string" {
+  local id
+  id=$(archivist_ulid)
+  [ "${#id}" -eq 26 ]
+  [[ "$id" =~ ^[0123456789ABCDEFGHJKMNPQRSTVWXYZ]{26}$ ]]
+}
+
+@test "two ULIDs minted apart are lexicographically ordered" {
+  local a b
+  a=$(archivist_ulid)
+  sleep 0.01
+  b=$(archivist_ulid)
+  [[ "$a" < "$b" ]] || [ "$a" = "$b" ]
+}
+
+@test "many ULIDs are unique" {
+  local seen="${BATS_TEST_TMPDIR}/ulids.txt"
+  : > "$seen"
+  local i
+  for ((i = 0; i < 50; i++)); do
+    printf '%s\n' "$(archivist_ulid)" >> "$seen"
+  done
+  local total unique
+  total=$(wc -l < "$seen" | tr -d ' ')
+  unique=$(sort -u "$seen" | wc -l | tr -d ' ')
+  [ "$total" = "$unique" ]
+}

package/test/bats/config.bats

@@ -10,17 +10,15 @@ setup_file() {
   [ "$status" -eq 0 ]
 }
 
-@test "claude plugin versions match package.json" {
+@test "ecosystem plugin.json version matches package.json" {
   local pkg_ver
   pkg_ver=$(jq -r '.version' "${REPO_ROOT}/package.json")
 
+  # Claude Code reads version from plugin.json. marketplace.json should NOT
+  # carry plugins[].version (see plugins-reference: setting both is a drift
+  # hazard since plugin.json silently wins).
   run jq -e --arg v "$pkg_ver" '.version == $v' "${REPO_ROOT}/.claude-plugin/plugin.json"
   [ "$status" -eq 0 ]
-
-  run jq -e --arg v "$pkg_ver" \
-    '[.plugins[].version] | unique | length == 1 and .[0] == $v' \
-    "${REPO_ROOT}/.claude-plugin/marketplace.json"
-  [ "$status" -eq 0 ]
 }
 
 @test "hooks.json wildcard matcher references tool-sequence-tracker" {
@@ -97,17 +95,19 @@ setup_file() {
   [[ "$hook_cmd" == *tool-history-tracker.sh ]]
 }
 
-@test "hooks.json UserPromptSubmit references turn and session-duration trackers" {
-  run jq -e '.hooks.UserPromptSubmit[0].hooks | length == 2' "${REPO_ROOT}/hooks/hooks.json"
+@test "hooks.json UserPromptSubmit references turn, session-duration, and prompt-rule trackers" {
+  run jq -e '.hooks.UserPromptSubmit[0].hooks | length == 3' "${REPO_ROOT}/hooks/hooks.json"
   [ "$status" -eq 0 ]
 
-  local turn_cmd duration_cmd
+  local turn_cmd duration_cmd rule_cmd
   turn_cmd=$(jq -r '.hooks.UserPromptSubmit[0].hooks[0].command' "${REPO_ROOT}/hooks/hooks.json")
   duration_cmd=$(jq -r '.hooks.UserPromptSubmit[0].hooks[1].command' "${REPO_ROOT}/hooks/hooks.json")
+  rule_cmd=$(jq -r '.hooks.UserPromptSubmit[0].hooks[2].command' "${REPO_ROOT}/hooks/hooks.json")
   [[ "$turn_cmd" == *turn-tracker.sh ]]
   [[ "$duration_cmd" == *session-duration-tracker.sh ]]
+  [[ "$rule_cmd" == *prompt-rule-injector.sh ]]
 
-  for hook_cmd in "$turn_cmd" "$duration_cmd"; do
+  for hook_cmd in "$turn_cmd" "$duration_cmd" "$rule_cmd"; do
     local script_path="${hook_cmd//\$CLAUDE_PLUGIN_ROOT/$REPO_ROOT}"
     script_path="${script_path//\"/}"
     run test -x "$script_path"

package/test/bats/echo-config.bats

@@ -0,0 +1,90 @@
+#!/usr/bin/env bats
+
+setup() {
+	source "${BATS_TEST_DIRNAME}/../helpers/setup.bash"
+	setup_test_env
+
+	PLUGIN_ROOT="${REPO_ROOT}/plugins/echo"
+	export CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT"
+	# shellcheck disable=SC1091
+	source "${PLUGIN_ROOT}/scripts/lib/echo-config.sh"
+}
+
+@test "echo is disabled by default" {
+	CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" echo_config_load ""
+	run echo_config_enabled
+	[ "$status" -ne 0 ]
+}
+
+@test "settings.json echo.enabled=true enables echo" {
+	local repo="${BATS_TEST_TMPDIR}/repo"
+	mkdir -p "${repo}/.claude"
+	printf '%s\n' '{"echo":{"enabled":true}}' > "${repo}/.claude/settings.json"
+	CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" echo_config_load "$repo"
+	run echo_config_enabled
+	[ "$status" -eq 0 ]
+}
+
+@test "settings.json echo.enabled=false overrides plugin default" {
+	local repo="${BATS_TEST_TMPDIR}/repo"
+	mkdir -p "${repo}/.claude"
+	printf '%s\n' '{"echo":{"enabled":false}}' > "${repo}/.claude/settings.json"
+	CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" echo_config_load "$repo"
+	run echo_config_enabled
+	[ "$status" -ne 0 ]
+}
+
+@test "default model is claude-haiku-4-5-20251001" {
+	CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" echo_config_load ""
+	local m
+	m=$(CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" echo_config_model)
+	[ "$m" = "claude-haiku-4-5-20251001" ]
+}
+
+@test "default timeout is 60" {
+	CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" echo_config_load ""
+	local t
+	t=$(CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" echo_config_timeout)
+	[ "$t" = "60" ]
+}
+
+@test "default drift_threshold is 0.05" {
+	CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" echo_config_load ""
+	local d
+	d=$(CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" echo_config_drift_threshold)
+	[ "$d" = "0.05" ]
+}
+
+@test "default watch_paths includes plugins/*/agents/*.md" {
+	CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" echo_config_load ""
+	local paths
+	paths=$(CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" echo_config_watch_paths)
+	printf '%s\n' "$paths" | grep -q 'plugins/\*/agents/\*.md'
+}
+
+@test "exclude_paths always includes plugins/echo/** regardless of config" {
+	CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" echo_config_load ""
+	local excl
+	excl=$(CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" echo_config_exclude_paths)
+	printf '%s\n' "$excl" | grep -q 'plugins/echo/\*\*'
+}
+
+@test "settings.json model override wins over plugin default" {
+	local repo="${BATS_TEST_TMPDIR}/repo"
+	mkdir -p "${repo}/.claude"
+	printf '%s\n' '{"echo":{"evaluation":{"model":"claude-opus-4-7"}}}' > "${repo}/.claude/settings.json"
+	CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" echo_config_load "$repo"
+	local m
+	m=$(CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" echo_config_model)
+	[ "$m" = "claude-opus-4-7" ]
+}
+
+@test "settings.json drift_threshold override wins" {
+	local repo="${BATS_TEST_TMPDIR}/repo"
+	mkdir -p "${repo}/.claude"
+	printf '%s\n' '{"echo":{"drift_threshold":0.1}}' > "${repo}/.claude/settings.json"
+	CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" echo_config_load "$repo"
+	local d
+	d=$(CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT" echo_config_drift_threshold)
+	[ "$d" = "0.1" ]
+}

package/test/bats/echo-events.bats

@@ -0,0 +1,121 @@
+#!/usr/bin/env bats
+
+# Validates every emitted echo.* event against @onlooker-community/schema.
+
+setup() {
+	source "${BATS_TEST_DIRNAME}/../helpers/setup.bash"
+	setup_test_env
+
+	PLUGIN_ROOT="${REPO_ROOT}/plugins/echo"
+	export CLAUDE_PLUGIN_ROOT="$PLUGIN_ROOT"
+	export ONLOOKER_EVENTS_LOG="${ONLOOKER_DIR}/logs/onlooker-events.jsonl"
+	mkdir -p "$(dirname "$ONLOOKER_EVENTS_LOG")"
+
+	export _ONLOOKER_EVENT_JS="${REPO_ROOT}/scripts/lib/onlooker-event.mjs"
+	export CLAUDE_SESSION_ID="bats-session-$$"
+
+	# shellcheck disable=SC1091
+	source "${PLUGIN_ROOT}/scripts/lib/echo-events.sh"
+}
+
+_validate_latest_event() {
+	local last
+	last=$(tail -n 1 "$ONLOOKER_EVENTS_LOG")
+	[ -n "$last" ] || return 1
+	printf '%s' "$last" | ONLOOKER_DIR="$ONLOOKER_DIR" \
+		node "${REPO_ROOT}/scripts/lib/onlooker-event.mjs" validate >/dev/null
+}
+
+SUITE_ID="01J000000000000000000000SS"
+TEST_ID="01J000000000000000000000TT"
+
+@test "echo.suite.started validates" {
+	local p
+	p=$(jq -n --arg s "$SUITE_ID" '{
+		suite_id: $s,
+		test_count: 2,
+		trigger: "file_change",
+		changed_file: "plugins/tribunal/agents/tribunal-judge-standard.md"
+	}')
+	echo_emit_event "echo.suite.started" "$p"
+	run _validate_latest_event
+	[ "$status" -eq 0 ]
+}
+
+@test "echo.suite.complete validates without drift fields" {
+	local p
+	p=$(jq -n --arg s "$SUITE_ID" '{
+		suite_id: $s,
+		test_count: 1,
+		improved: 0,
+		degraded: 0,
+		neutral: 1,
+		merge_recommended: true,
+		duration_ms: 3200
+	}')
+	echo_emit_event "echo.suite.complete" "$p"
+	run _validate_latest_event
+	[ "$status" -eq 0 ]
+}
+
+@test "echo.suite.complete validates with drift fields" {
+	local p
+	p=$(jq -n --arg s "$SUITE_ID" '{
+		suite_id: $s,
+		test_count: 1,
+		improved: 1,
+		degraded: 0,
+		neutral: 0,
+		merge_recommended: true,
+		duration_ms: 3200,
+		baseline_score: 0.72,
+		score_after: 0.85,
+		drift: 0.13,
+		drift_threshold: 0.05
+	}')
+	echo_emit_event "echo.suite.complete" "$p"
+	run _validate_latest_event
+	[ "$status" -eq 0 ]
+}
+
+@test "echo.improvement.detected validates" {
+	local p
+	p=$(jq -n --arg s "$SUITE_ID" --arg t "$TEST_ID" '{
+		suite_id: $s,
+		test_id: $t,
+		test_name: "tribunal-judge-standard.md",
+		score_before: 0.70,
+		score_after: 0.85,
+		delta: 0.15,
+		confidence: 0.9
+	}')
+	echo_emit_event "echo.improvement.detected" "$p"
+	run _validate_latest_event
+	[ "$status" -eq 0 ]
+}
+
+@test "echo.regression.detected validates" {
+	local p
+	p=$(jq -n --arg s "$SUITE_ID" --arg t "$TEST_ID" '{
+		suite_id: $s,
+		test_id: $t,
+		test_name: "tribunal-judge-standard.md",
+		score_before: 0.85,
+		score_after: 0.62,
+		delta: -0.23,
+		confidence: 0.88
+	}')
+	echo_emit_event "echo.regression.detected" "$p"
+	run _validate_latest_event
+	[ "$status" -eq 0 ]
+}
+
+@test "emission fails on unknown event type" {
+	run echo_emit_event "echo.no.such.event" '{"suite_id":"x"}'
+	[ "$status" -ne 0 ]
+}
+
+@test "echo_emit_event returns 1 when payload is empty" {
+	run echo_emit_event "echo.suite.started" ""
+	[ "$status" -ne 0 ]
+}