@rulemetric/hooks 0.5.2 → 0.5.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rulemetric/hooks",
-  "version": "0.5.2",
+  "version": "0.5.4",
   "publishConfig": {
     "access": "public"
   },

package/scripts/_config.sh

@@ -75,7 +75,7 @@ _rulemetric_load_config() {
     if [ -n "$token" ]; then
       export RULEMETRIC_ACCESS_TOKEN="$token"
       # Default API URL if not set
-      export RULEMETRIC_API_URL="${RULEMETRIC_API_URL:-https://api.rulemetric.com}"
+      export RULEMETRIC_API_URL="${RULEMETRIC_API_URL:-https://rulemetric.com}"
       _rulemetric_load_active_org
       return 0
     fi

package/scripts/_post.sh

@@ -39,10 +39,13 @@ _rulemetric_post_event() {
     http_code=$(printf '%s' "$body" | tail -1)
     body=$(printf '%s' "$body" | sed '$d')
 
-    if [ "$http_code" = "200" ] || [ "$http_code" = "201" ]; then
+    # Any 2xx is success: the batched events route (Phase 2 admission control)
+    # returns 202 "buffered", the legacy path 200, session create 201.
+    case "$http_code" in 2??)
       [ -n "$hdr_file" ] && rm -f "$hdr_file"
       return 0
-    fi
+      ;;
+    esac
 
     # Transient (service busy / rate-limited / network) → retry unless this was
     # the last allowed attempt. Everything else (4xx) is permanent.
@@ -103,10 +106,13 @@ _rulemetric_send() {
     http_code=$(printf '%s' "$body" | tail -1)
     body=$(printf '%s' "$body" | sed '$d')
 
-    if [ "$http_code" = "200" ] || [ "$http_code" = "201" ]; then
+    # Any 2xx is success: the batched events route (Phase 2 admission control)
+    # returns 202 "buffered", the legacy path 200, session create 201.
+    case "$http_code" in 2??)
       [ -n "$hdr_file" ] && rm -f "$hdr_file"
       return 0
-    fi
+      ;;
+    esac
 
     if [ "$http_code" = "503" ] || [ "$http_code" = "429" ] || [ "$http_code" = "000" ]; then
       if [ "$attempt" -lt "$max_attempts" ]; then

package/scripts/post-retry.test.sh

@@ -32,7 +32,7 @@ trap cleanup EXIT
 # --- Stub server: counts requests, programmed by $MODE in a control file --------
 cat > "$WORK_ROOT/stub.py" <<'PY'
 import http.server, json, os, sys, threading
-mode = sys.argv[1]            # fail-once | always-503 | always-400
+mode = sys.argv[1]            # fail-once | always-503 | always-400 | always-202
 count_file = sys.argv[2]
 n = 0
 lock = threading.Lock()
@@ -50,6 +50,9 @@ class H(http.server.BaseHTTPRequestHandler):
         if mode == "always-503":
             self.send_response(503); self.send_header("Retry-After", "1"); self.end_headers()
             self.wfile.write(b'{"error":"busy"}'); return
+        if mode == "always-202":
+            self.send_response(202); self.end_headers()
+            self.wfile.write(b'{"buffered":true,"count":1,"durable":false}'); return
         # fail-once: 503 on first request, 200 thereafter
         if cur == 1:
             self.send_response(503); self.send_header("Retry-After", "1"); self.end_headers()
@@ -112,6 +115,18 @@ assert_eq "always-503: server saw exactly MAX_ATTEMPTS (3) requests" "3" "$(req_
 grep -q "http_503" "$LOG_FILE" && ok "always-503: logged an http_503 error" || fail "always-503: expected http_503 in log"
 stop_server
 
+# --- Test 4: batched-ingest 202 is SUCCESS (no retry, no error log) -----------
+# The Phase 2 batched events route returns 202 { buffered: true } instead of
+# 200 { recorded: true }; the helper must treat ANY 2xx as delivered.
+start_server always-202
+: > "$LOG_FILE"
+_rulemetric_post_event "http://127.0.0.1:$SERVER_PORT/events" "$AUTH" "$PAYLOAD" "test" 4
+rc=$?
+assert_eq "always-202: returns success (0) — buffered counts as delivered" "0" "$rc"
+assert_eq "always-202: server saw exactly 1 request (no retry)" "1" "$(req_count)"
+assert_eq "always-202: no error logged" "0" "$(wc -l < "$LOG_FILE" | tr -d ' ')"
+stop_server
+
 # --- Summary -----------------------------------------------------------------
 echo
 echo "Passed: $PASSES   Failed: $FAILURES"

package/scripts/session-end.sh

@@ -51,6 +51,12 @@ if [ -n "$HOOK_TRANSCRIPT_PATH" ] && [ -f "$HOOK_TRANSCRIPT_PATH" ]; then
   # blocks the hook, so cap at 2 attempts (worst-case tail ~18s on a hard API hang,
   # vs ~32s at 3) — the common fast-503 wedge still recovers in <1s. Fall back to
   # close if it still fails after retries — a dropped reimport leaves the session EMPTY.
+  # NOTE: under headless `claude -p`, Claude Code's print-mode teardown cancels this
+  # hook ("SessionEnd hook … failed: Hook cancelled") regardless of whether this runs
+  # in the foreground or is backgrounded — verified 2026-07-05. It is a Claude Code
+  # print-mode behavior, not a capture failure (per-turn hooks still land events); do
+  # NOT try to "fix" it by detaching this call (adds no benefit, and headless is a
+  # deliberately-excluded capture vehicle).
   _rulemetric_send POST "${RULEMETRIC_API_URL}/api/sessions/${RW_SESSION_ID}/reimport" \
     "$AUTH_HEADER" "$REIMPORT_PAYLOAD" "session-end-reimport" 8 2 \
     > /dev/null 2>&1 || {