create-merlin-brain 3.15.1 → 3.15.2

package/files/hooks/lib/analytics.sh

@@ -28,11 +28,16 @@ ensure_session_file() {
 
 # Core event logger — all specialized loggers call this
 # Usage: log_event "type" '{"key":"value"}'
+# CRITICAL: Must NEVER fail (return non-zero) because callers run under
+# set -e and a failure here kills the entire hook before it can output
+# its additionalContext JSON. This was the root cause of Merlin going
+# silent — the session-start hook died inside log_event before reaching
+# the printf that outputs the boot sequence enforcement.
 log_event() {
   local etype="${1:-unknown}" edata="${2:-{}}"
-  ensure_session_file || return 1
+  ensure_session_file || return 0  # was return 1 — killed callers under set -e
   local evt
-  evt=$(printf '{"type":"%s","ts":"%s","data":%s}' "$etype" "$(date -u +"%Y-%m-%dT%H:%M:%SZ")" "$edata")
+  evt=$(printf '{"type":"%s","ts":"%s","data":%s}' "$etype" "$(date -u +"%Y-%m-%dT%H:%M:%SZ")" "$edata") || return 0
   if command -v jq >/dev/null 2>&1; then
     local tmp; tmp=$(mktemp 2>/dev/null || echo "/tmp/merlin-evt-$$")
     jq --argjson e "$evt" '.events += [$e]' "${MERLIN_SESSION_FILE}" > "$tmp" 2>/dev/null \
@@ -40,6 +45,7 @@ log_event() {
   else
     echo "$evt" >> "${MERLIN_SESSION_FILE}.log" 2>/dev/null
   fi
+  return 0  # never fail
 }
 
 # Specialized loggers — thin wrappers for structured events

package/files/hooks/session-start.sh

@@ -80,14 +80,16 @@ _merlin_cleanup_stale &
 AGENT_ID="${CLAUDE_AGENT_ID:-unknown}"
 AGENT_TYPE="${CLAUDE_AGENT_TYPE:-main}"
 
-ensure_session_file
+# Analytics calls MUST NOT kill the script — the additionalContext output
+# at the end is critical for Merlin enforcement. Use || true on every call.
+ensure_session_file || true
 cwd="${PWD:-$(pwd)}"
-log_event "session_start" "$(printf '{"cwd":"%s","agent_id":"%s","agent_type":"%s"}' "$cwd" "$AGENT_ID" "$AGENT_TYPE")"
+log_event "session_start" "$(printf '{"cwd":"%s","agent_id":"%s","agent_type":"%s"}' "$cwd" "$AGENT_ID" "$AGENT_TYPE")" || true
 
 # Warm up Sights context in background
 if command -v merlin >/dev/null 2>&1; then
   merlin context "session start" >/dev/null 2>&1 &
-  record_sights_call
+  record_sights_call || true
 fi
 
 # ── 2. Agent sync (background, max once/hour) ──────────────────
@@ -184,7 +186,9 @@ _merlin_check_voice_mode() {
   command -v jq >/dev/null 2>&1 || return 0
   local voice
   voice=$(jq -r '.voice_mode_concise // false' "${settings_file}" 2>/dev/null) || return 0
-  [ "${voice}" = "true" ] && export MERLIN_VOICE_MODE=1
+  # CRITICAL: [ x = y ] && cmd returns 1 when false, which under set -e kills
+  # the calling script. Use if/then instead.
+  if [ "${voice}" = "true" ]; then export MERLIN_VOICE_MODE=1; fi
 }
 _merlin_check_voice_mode
 

package/files/hooks/subagent-context.sh

@@ -25,8 +25,8 @@ if [ -n "$input" ] && command -v jq >/dev/null 2>&1; then
   task_desc=$(echo "$input" | jq -r '.task // .description // empty' 2>/dev/null || true)
 fi
 
-# Log subagent start event
-log_event "subagent_start" "$(printf '{"task":"%s"}' "${task_desc:-unknown}")"
+# Log subagent start event (|| true: never kill script before additionalContext output)
+log_event "subagent_start" "$(printf '{"task":"%s"}' "${task_desc:-unknown}")" || true
 
 # If Merlin CLI available, fetch context for the task and output as JSON
 if [ -n "$task_desc" ] && command -v merlin >/dev/null 2>&1; then

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "create-merlin-brain",
-  "version": "3.15.1",
+  "version": "3.15.2",
   "description": "Merlin - The Ultimate AI Brain for Claude Code. One install: workflows, agents, loop, and Sights MCP server.",
   "type": "module",
   "main": "./dist/server/index.js",