Security hotfix — v0.15.1 Wave 1 fixes from 4-parallel audit. RED-1 (CVE): KEI_DISABLED_HOOKS tokenized match — was `*all*` substring-glob (trivially bypassable via "install", "wall-clock", etc.), now exact-token split on comma/space. Patched in all 9 hooks: no-hand-edit-agents, assemble-agents, assemble-validate, tomd-preread, agent-fork-logger, site-wysiwyd-check, error-spike-detector, milestone-commit-hook, session-end-dump. RED-2 (observability): minimal profile whitelist now includes agent-fork-logger and session-end-dump (ledger + trace paths) so observability is not silently lost on minimal installs. HIGH: review.json schema minItems:1 on findings — rejects empty reviews; new Rust test review_schema_rejects_empty_findings. HIGH: typed-handoff wire-up — produces_artifact declared at top level on 5 manifests (kei-security-auditor, kei-validator, kei-architect, kei-code-implementer, kei-critic); duplicate per-handoff declarations removed. MED: kei-artifact validate.rs gains warn_unsupported_keywords — non-fatal stderr warning when schema uses keywords outside the hand-rolled 2020-12 subset. LOW: CI Node matrix dropped 18, now ['20','22']. Doc drift: skills/hooks-control/SKILL.md reflects tokenized-match semantics and updated minimal-profile hook list. Tests: 191 Rust workspace + 30 assembler (both pass). RED-1 reproducer 10/10 (4 former-CVE vectors blocked, 5 legit vectors accepted, empty passes). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
113 lines
3.6 KiB
Bash
Executable file
113 lines
3.6 KiB
Bash
Executable file
#!/bin/sh
|
|
# error-spike-detector.sh — PostToolUse:* hook (RULE 0.14).
|
|
#
|
|
# Maintains a rolling 20-tool-call window of (timestamp, error_flag) pairs.
|
|
# When ≥3 errors land in the window, append to audit-backlog.md. NEVER
|
|
# blocks: every exit path is `exit 0`. Silent-first: stderr advisory only
|
|
# fires after 10 sessions are in the memory store.
|
|
|
|
command -v jq >/dev/null 2>&1 || exit 0
|
|
|
|
# --- RUNTIME CONTROLS (v0.15.1) ---
|
|
# KEI_DISABLED_HOOKS: tokenized exact-match list (comma- or space-separated).
|
|
# Repro of pre-v0.15.1 substring bypass (CVE-class): KEI_DISABLED_HOOKS="foo-all-bar"
|
|
# previously disabled every hook via `*all*`. v0.15.1 requires token equality.
|
|
_hook_name="$(basename "$0" .sh)"
|
|
_disabled=" $(printf '%s' "${KEI_DISABLED_HOOKS:-}" | tr ',' ' ') "
|
|
case "$_disabled" in
|
|
*" $_hook_name "*|*" all "*) unset _disabled; exit 0 ;;
|
|
esac
|
|
unset _disabled
|
|
case "${KEI_HOOK_PROFILE:-full}" in
|
|
off) exit 0 ;;
|
|
minimal)
|
|
case "$_hook_name" in
|
|
no-hand-edit-agents|assemble-validate|agent-fork-logger|session-end-dump) ;;
|
|
*) exit 0 ;;
|
|
esac
|
|
;;
|
|
advisory-off)
|
|
case "$_hook_name" in
|
|
recurrence-suggest|citation-verify|error-spike-detector|milestone-commit-hook) exit 0 ;;
|
|
esac
|
|
;;
|
|
full|*) ;;
|
|
esac
|
|
# --- end runtime controls ---
|
|
|
|
set -eu
|
|
|
|
input="$(cat)"
|
|
|
|
is_err=$(printf '%s' "$input" | jq -r '.tool_response.is_error // empty' 2>/dev/null || true)
|
|
msg=$(printf '%s' "$input" | jq -r '.tool_response.content // .tool_response // empty' 2>/dev/null || true)
|
|
|
|
# Classify current call as errored if explicit is_error=true OR message
|
|
# matches a common failure signature. Empty tool_response → treat as OK.
|
|
flag=0
|
|
case "$is_err" in
|
|
true|True|TRUE|1) flag=1 ;;
|
|
esac
|
|
if [ "$flag" -eq 0 ] && [ -n "$msg" ]; then
|
|
case "$(printf '%s' "$msg" | tr 'A-Z' 'a-z')" in
|
|
*error:*|*failed*|*panic*|*denied*) flag=1 ;;
|
|
esac
|
|
fi
|
|
|
|
window="${HOME}/.claude/memory/error-window.txt"
|
|
backlog="${HOME}/.claude/memory/audit-backlog.md"
|
|
mkdir -p "$(dirname "$window")" 2>/dev/null || exit 0
|
|
|
|
# Append this tool call to the rolling window.
|
|
ts=$(date +%s)
|
|
printf '%s %s\n' "$ts" "$flag" >> "$window"
|
|
|
|
# Trim to last 20 lines.
|
|
tmp="${window}.tmp.$$"
|
|
tail -n 20 "$window" > "$tmp" 2>/dev/null || true
|
|
mv -f "$tmp" "$window" 2>/dev/null || true
|
|
|
|
# Count errors in the window.
|
|
err_ct=$(awk '$2==1' "$window" 2>/dev/null | wc -l | tr -d ' ')
|
|
err_ct=${err_ct:-0}
|
|
|
|
if [ "$err_ct" -lt 3 ]; then
|
|
exit 0
|
|
fi
|
|
|
|
# Classify the spike by greping the recent error messages. Best-effort.
|
|
pattern="unclassified"
|
|
if [ -n "$msg" ]; then
|
|
case "$(printf '%s' "$msg" | tr 'A-Z' 'a-z')" in
|
|
*permission*denied*) pattern="permission_denied" ;;
|
|
*worktree*) pattern="worktree_error" ;;
|
|
*cargo*workspace*) pattern="cargo_workspace" ;;
|
|
*panic*) pattern="panic" ;;
|
|
*timeout*) pattern="timeout" ;;
|
|
esac
|
|
fi
|
|
|
|
# Ensure the backlog file exists.
|
|
if [ ! -f "$backlog" ]; then
|
|
{
|
|
printf '# Audit Backlog\n\n'
|
|
printf '<!-- session_count: 0 -->\n\n'
|
|
} > "$backlog"
|
|
fi
|
|
|
|
iso=$(date -u +%Y-%m-%dT%H:%M)
|
|
printf -- '- [ERROR-SPIKE %s] %s errors in last 20 tool calls. Pattern: %s\n' \
|
|
"$iso" "$err_ct" "$pattern" >> "$backlog"
|
|
|
|
# Reset window so we do not re-fire on every subsequent call.
|
|
: > "$window"
|
|
|
|
# Silent-first: advisory stderr only after 10 sessions.
|
|
session_count=$(grep -Eo 'session_count: [0-9]+' "$backlog" 2>/dev/null | head -1 | awk '{print $2}')
|
|
session_count=${session_count:-0}
|
|
if [ "$session_count" -ge 10 ]; then
|
|
printf 'kei-memory: error spike detected (%s in 20) — see %s\n' \
|
|
"$err_ct" "$backlog" >&2
|
|
fi
|
|
|
|
exit 0
|